Apply suggestions from code review

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Author: yomybaby

pnpm-lock.yaml

@@ -111,7 +111,7 @@ const AppLauncherModal: React.FC<AppLauncherModalProps> = ({
 
     // Tensorboard - show path input modal BEFORE starting proxy
     // This matches legacy behavior where modal is shown first, then proxy starts after path submission
-    if (app?.name === 'tensorboard') {
+    if (app.name === 'tensorboard') {
       setOpenTensorboardModal(true);
       return;
     }

react/src/components/ComputeSessionNodeItems/AppLauncherModal.tsx

@@ -1,6 +1,4 @@
-'use memo';
-
-import SourceCodeViewer from '../SourceCodeViewer';
+import SourceCodeView from '../SourceCodeView';
 import { Descriptions, Skeleton, Typography } from 'antd';
 import {
   BAIButton,
@@ -22,6 +20,7 @@ interface VSCodeDesktopConnectionModalProps extends BAIModalProps {
 const VSCodeDesktopConnectionModal: React.FC<
   VSCodeDesktopConnectionModalProps
 > = ({ sessionId, host = '127.0.0.1', port, ...modalProps }) => {
+  'use memo';
   const { t } = useTranslation();
   const baiClient = useSuspendedBackendaiClient();
 
@@ -90,7 +89,7 @@ const VSCodeDesktopConnectionModal: React.FC<
         <Typography.Text>
           {t('session.VSCodeRemoteNoticeSSHConfig')}
         </Typography.Text>
-        <SourceCodeViewer language="shell">{sshConfig}</SourceCodeViewer>
+        <SourceCodeView language="shell">{sshConfig}</SourceCodeView>
 
         <BAIButton href={vscodeUri} target="_blank" type="primary" size="large">
           {t('session.appLauncher.OpenVSCodeRemote')}

react/src/components/ComputeSessionNodeItems/VSCodeDesktopConnectionModal.tsx

@@ -393,8 +393,7 @@ export const useBackendAIAppLauncher = (
     const { appConnectUrl, reused, redirectUrl } = await _connectToProxyWorker(
       response.url,
       '',
-      !TCP_APPS.includes(app) || globalThis.isElectron,
-      // ^^ false for TCP apps in browser, true otherwise
+      !TCP_APPS.includes(app) || globalThis.isElectron, // Enable direct TCP for non-TCP apps, or when running in Electron
     );
 
     // Initialize TCP connection info variables
@@ -746,7 +745,6 @@ export const useBackendAIAppLauncher = (
     };
   };
 
-  // @ts-ignore
   return {
     /**
      * Convenience method to launch terminal (ttyd) app.
@@ -831,18 +829,6 @@ export const useBackendAIAppLauncher = (
      * Used when re-launching apps like tensorboard that need to be restarted with different args.
      */
     closeWsproxy: _close_wsproxy,
-    // TODO: implement below function
-    // showLauncher: (params: {
-    //   'session-uuid'?: string;
-    //   'access-key'?: string;
-    //   'app-services'?: string;
-    //   mode?: string;
-    //   'app-services-option'?: string;
-    //   'service-ports'?: string;
-    //   runtime?: string;
-    //   filename?: string;
-    //   arguments?: string;
-    // }) => {},
   };
 };
 
@@ -874,7 +860,11 @@ class AppLaunchError extends Error {
  * @param request - Request configuration object
  * @returns Parsed response body or error object with status
  */
-async function sendRequest(request: any) {
+interface SendRequestConfig extends RequestInit {
+  uri: string;
+  method?: string;
+}
+async function sendRequest(request: SendRequestConfig) {
   try {
     if (request.method === 'GET') {
       request.body = undefined;