Merge branch 'main' into fix/remove-auto-append-functionality

Author: deep1401

.github/workflows/prettier.yml

@@ -0,0 +1,27 @@
+name: Prettier Check
+
+on:
+    push:
+        branches: [main]
+    pull_request:
+        branches: [main]
+
+jobs:
+    prettier-check:
+        runs-on: ubuntu-latest
+
+        steps:
+            - name: Checkout code
+              uses: actions/checkout@v4
+
+            - name: Setup Node.js
+              uses: actions/setup-node@v4
+              with:
+                  node-version: "22"
+                  cache: "npm"
+
+            - name: Install Prettier
+              run: npm install -g prettier
+
+            - name: Check Prettier formatting
+              run: npm run format:check

.github/workflows/pytest.yml

@@ -25,7 +25,7 @@ jobs:
           python-version: "3.10"
 
       - name: Create virtualenv
-        run: uv venv --python 3.10 --seed
+        run: uv venv --python 3.10 --seed --clear
 
       - name: Install dependencies
         run: uv pip install -e .[dev]

frontend/package-lock.json

@@ -20,6 +20,7 @@
     "@xterm/xterm": "^5.5.0",
     "axios": "^1.10.0",
     "lucide-react": "^0.525.0",
+    "prettier": "^3.6.2",
     "react": "^19.1.0",
     "react-dom": "^19.1.0",
     "react-router-dom": "^7.7.0",
@@ -33,7 +34,9 @@
     "start": "vite",
     "dev": "vite",
     "build": "vite build",
-    "preview": "vite preview"
+    "preview": "vite preview",
+    "format:check": "prettier --check \"src/**/*.{js,jsx,ts,tsx,json,css,scss,md}\"",
+    "format": "prettier --write \"src/**/*.{js,jsx,ts,tsx,json,css,scss,md}\""
   },
   "proxy": "http://localhost:8000",
   "eslintConfig": {

frontend/package.json

@@ -1,8 +1,8 @@
-import React from 'react';
-import { render, screen } from '@testing-library/react';
-import App from './App';
+import React from "react";
+import { render, screen } from "@testing-library/react";
+import App from "./App";
 
-test('renders learn react link', () => {
+test("renders learn react link", () => {
   render(<App />);
   const linkElement = screen.getByText(/learn react/i);
   expect(linkElement).toBeInTheDocument();

frontend/src/App.test.tsx

@@ -37,15 +37,15 @@ const CLIAuthorizePage: React.FC = () => {
           buildApiUrl(`auth/cli/session/${sessionId}`),
           {
             credentials: "include",
-          }
+          },
         );
         console.log(`Response status: ${response.status}`);
 
         if (!response.ok) {
           const errorText = await response.text();
           console.error(`Error response: ${errorText}`);
           throw new Error(
-            `Session not found or expired: ${response.status} - ${errorText}`
+            `Session not found or expired: ${response.status} - ${errorText}`,
           );
         }
 

frontend/src/components/CLIAuthorizePage.tsx

@@ -85,14 +85,14 @@ const ClusterCard: React.FC<ClusterCardProps> = ({
     clusterType === "cloud" ? buildApiUrl("instances/status") : null,
     (url: string) =>
       apiFetch(url, { credentials: "include" }).then((res) => res.json()),
-    { refreshInterval: 2000 }
+    { refreshInterval: 2000 },
   );
 
   // Process nodes based on cluster type
   let processedNodes = cluster.nodes;
   let activeCount = cluster.nodes.filter((n) => n.status === "active").length;
   let assignedToYouCount = cluster.nodes.filter(
-    (n) => n.user === currentUser
+    (n) => n.user === currentUser,
   ).length;
 
   if (clusterType === "cloud") {
@@ -104,7 +104,7 @@ const ClusterCard: React.FC<ClusterCardProps> = ({
             ac.status === "ClusterStatus.UP" ||
             ac.status === "ClusterStatus.INIT" ||
             ac.status === "UP" ||
-            ac.status === "INIT"
+            ac.status === "INIT",
         ) || false;
 
       // Update the node status based on active clusters
@@ -116,7 +116,7 @@ const ClusterCard: React.FC<ClusterCardProps> = ({
               ac.status === "ClusterStatus.UP" ||
               ac.status === "ClusterStatus.INIT" ||
               ac.status === "UP" ||
-              ac.status === "INIT"
+              ac.status === "INIT",
           );
 
           return {
@@ -144,7 +144,7 @@ const ClusterCard: React.FC<ClusterCardProps> = ({
       // For other cloud providers (RunPod, Azure), use the old SkyPilot status matching
       const skyPilotClusters = skyPilotStatus?.clusters || [];
       const skyPilotCluster = skyPilotClusters.find(
-        (c: any) => c.cluster_name === displayName
+        (c: any) => c.cluster_name === displayName,
       );
       const isActiveCluster =
         skyPilotCluster?.status === "ClusterStatus.UP" ||
@@ -168,7 +168,7 @@ const ClusterCard: React.FC<ClusterCardProps> = ({
       }));
 
       activeCount = processedNodes.filter(
-        (n: Node) => n.status === "active"
+        (n: Node) => n.status === "active",
       ).length;
       assignedToYouCount = 0; // All nodes are available, none assigned
     }
@@ -177,7 +177,7 @@ const ClusterCard: React.FC<ClusterCardProps> = ({
   const sortedNodes = [...processedNodes].sort(
     (a, b) =>
       getStatusOrder(a.status, a.type, a.user, currentUser) -
-      getStatusOrder(b.status, b.type, b.user, currentUser)
+      getStatusOrder(b.status, b.type, b.user, currentUser),
   );
 
   const handleReserveNode = () => {
@@ -284,7 +284,7 @@ const ClusterCard: React.FC<ClusterCardProps> = ({
 
                         // Helper to parse strings like "1 of 2 free"
                         const parseUtilization = (
-                          s?: string
+                          s?: string,
                         ): { free?: number; total?: number } => {
                           if (!s) return {};
                           const m = s.match(/(\d+)\s*of\s*(\d+)/i);
@@ -302,18 +302,18 @@ const ClusterCard: React.FC<ClusterCardProps> = ({
                             const util = parseUtilization(e.utilization);
                             const totalParsed = Number.parseInt(
                               e.total ?? "",
-                              10
+                              10,
                             );
                             const freeParsed = Number.parseInt(
                               e.free ?? "",
-                              10
+                              10,
                             );
                             const total = Number.isFinite(totalParsed)
                               ? totalParsed
-                              : util.total ?? 0;
+                              : (util.total ?? 0);
                             const free = Number.isFinite(freeParsed)
                               ? freeParsed
-                              : util.free ?? 0;
+                              : (util.free ?? 0);
 
                             totalGpus += total;
                             usedGpus += Math.max(0, total - free);
@@ -326,7 +326,7 @@ const ClusterCard: React.FC<ClusterCardProps> = ({
                       } else {
                         // Fall back to capacity calculation for non-SSH clusters
                         return Math.round(
-                          (activeCount / processedNodes.length) * 100
+                          (activeCount / processedNodes.length) * 100,
                         );
                       }
                     })()}
@@ -389,7 +389,7 @@ const ClusterCard: React.FC<ClusterCardProps> = ({
                 // For cloud and regular clusters, show in two columns
                 ["dedicated", "on-demand"].map((nodeType) => {
                   const nodesOfType = nodesToShow.filter(
-                    (node) => node.type === nodeType
+                    (node) => node.type === nodeType,
                   );
                   if (nodesOfType.length === 0) return null;
 

frontend/src/components/ClusterCard.tsx

@@ -48,7 +48,7 @@ const ClusterManagement: React.FC<ClusterManagementProps> = ({
         buildApiUrl("node-pools/ssh-node-pools"),
         {
           credentials: "include",
-        }
+        },
       );
       if (response.ok) {
         const data = await response.json();
@@ -70,7 +70,7 @@ const ClusterManagement: React.FC<ClusterManagementProps> = ({
         buildApiUrl(`node-pools/ssh-node-pools/${clusterName}`),
         {
           credentials: "include",
-        }
+        },
       );
       if (response.ok) {
         const data = await response.json();

frontend/src/components/ClusterManagement.tsx

@@ -45,7 +45,7 @@ export const NotificationProvider: React.FC<NotificationProviderProps> = ({
 
   const removeNotification = useCallback((id: string) => {
     setNotifications((prev) =>
-      prev.filter((notification) => notification.id !== id)
+      prev.filter((notification) => notification.id !== id),
     );
   }, []);
 
@@ -67,7 +67,7 @@ export const NotificationProvider: React.FC<NotificationProviderProps> = ({
         }, newNotification.duration);
       }
     },
-    [removeNotification]
+    [removeNotification],
   );
 
   const getIcon = (type: Notification["type"]) => {
@@ -85,7 +85,7 @@ export const NotificationProvider: React.FC<NotificationProviderProps> = ({
 
   const contextValue = useMemo(
     () => ({ addNotification, removeNotification }),
-    [addNotification, removeNotification]
+    [addNotification, removeNotification],
   );
 
   return (

frontend/src/components/NotificationSystem.tsx

@@ -85,7 +85,7 @@ const SkyPilotClusterStatus: React.FC = () => {
   const { data, isLoading, mutate } = useSWR(
     buildApiUrl("instances/status"),
     fetcher,
-    { refreshInterval: 2000 }
+    { refreshInterval: 2000 },
   );
 
   const clusters = data?.clusters || [];
@@ -94,7 +94,7 @@ const SkyPilotClusterStatus: React.FC = () => {
   const { data: jobsData, mutate: mutateJobs } = useSWR(
     expandedCluster ? buildApiUrl(`jobs/${expandedCluster}`) : null,
     fetcher,
-    { refreshInterval: 5000 }
+    { refreshInterval: 5000 },
   );
 
   // Fetch cluster type info for each cluster
@@ -109,7 +109,7 @@ const SkyPilotClusterStatus: React.FC = () => {
         try {
           const typeResponse = await apiFetch(
             buildApiUrl(`instances/cluster-type/${cluster.cluster_name}`),
-            { credentials: "include" }
+            { credentials: "include" },
           );
           if (typeResponse.ok) {
             const typeData: ClusterTypeInfo = await typeResponse.json();
@@ -118,7 +118,7 @@ const SkyPilotClusterStatus: React.FC = () => {
         } catch (err) {
           console.error(
             `Failed to fetch cluster type for ${cluster.cluster_name}:`,
-            err
+            err,
           );
         }
         return null;
@@ -169,7 +169,7 @@ const SkyPilotClusterStatus: React.FC = () => {
     try {
       const response = await apiFetch(
         buildApiUrl(`jobs/${clusterName}/${jobId}/logs?tail_lines=100`),
-        { credentials: "include" }
+        { credentials: "include" },
       );
       if (!response.ok) {
         throw new Error(`Failed to fetch logs: ${response.statusText}`);
@@ -178,7 +178,7 @@ const SkyPilotClusterStatus: React.FC = () => {
       setSelectedJobLogs(data.logs || "No logs available");
     } catch (err) {
       setSelectedJobLogs(
-        err instanceof Error ? err.message : "Failed to fetch logs"
+        err instanceof Error ? err.message : "Failed to fetch logs",
       );
     } finally {
       setLogsLoading(false);
@@ -195,7 +195,7 @@ const SkyPilotClusterStatus: React.FC = () => {
         {
           method: "POST",
           credentials: "include",
-        }
+        },
       );
       if (!response.ok) {
         throw new Error(`Failed to cancel job: ${response.statusText}`);
@@ -640,7 +640,7 @@ const SkyPilotClusterStatus: React.FC = () => {
                                             <Chip
                                               size="sm"
                                               color={getJobStatusColor(
-                                                job.status
+                                                job.status,
                                               )}
                                               variant="soft"
                                             >
@@ -713,7 +713,7 @@ const SkyPilotClusterStatus: React.FC = () => {
                                               onClick={() =>
                                                 fetchJobLogs(
                                                   cluster.cluster_name,
-                                                  job.job_id
+                                                  job.job_id,
                                                 )
                                               }
                                             >
@@ -734,7 +734,7 @@ const SkyPilotClusterStatus: React.FC = () => {
                                                 onClick={() =>
                                                   cancelJob(
                                                     cluster.cluster_name,
-                                                    job.job_id
+                                                    job.job_id,
                                                   )
                                                 }
                                                 loading={
@@ -786,7 +786,7 @@ const SkyPilotClusterStatus: React.FC = () => {
                                         </Box>
                                       </ListItem>
                                     </React.Fragment>
-                                  )
+                                  ),
                                 )
                               )}
                             </List>
@@ -818,7 +818,7 @@ const SkyPilotClusterStatus: React.FC = () => {
         isSshCluster={!!clusterTypes[jobModalCluster]?.is_ssh}
         availableResources={
           clusters.find(
-            (c: ClusterStatus) => c.cluster_name === jobModalCluster
+            (c: ClusterStatus) => c.cluster_name === jobModalCluster,
           )?.resources_str || ""
         }
       />