Prettier run (#425)

* Add prettier gh action

* Add build check

* rm prettierignore

* Prettier run

* prettier

* use npm ci instead

* rename

* rm lockfiles

* npm run prettier-fix

Author: oso-jukumari

.github/workflows/lint.yml

@@ -1,27 +1,39 @@
-name: Lint and Build Check
+name: Prettier and Build Check
 
 on:
   pull_request:
-    branches: [ main ]
+    branches: [main]
   push:
-    branches: [ main ]
+    branches: [main]
 
 jobs:
-  lint-and-build:
+  prettier-and-build:
     runs-on: ubuntu-latest
-    
+
     steps:
       - name: Checkout code
         uses: actions/checkout@v4
 
       - name: Setup Node.js
         uses: actions/setup-node@v4
         with:
-          node-version: '18'
-          cache: 'npm'
+          node-version: "18"
+          cache: "npm"
 
       - name: Install dependencies
-        run: npm install
+        run: |
+          # Clean npm cache
+          npm cache clean --force
+
+          # Remove lock files and node_modules to fix rollup binary issues
+          rm -rf package-lock.json node_modules
+          rm -rf client/package-lock.json client/node_modules
+          rm -rf server/package-lock.json server/node_modules
+
+          # Install dependencies
+          npm install
+          cd client && npm install
+          cd ../server && npm install
 
       - name: Check Prettier formatting
         run: |
@@ -41,4 +53,4 @@ jobs:
         run: |
           echo "🏗️ Building project..."
           npm run build
-          echo "✅ Build completed successfully!"
+          echo "✅ Build completed successfully!"

client/src/components/connection/ServerConnectionCard.tsx

@@ -177,7 +177,9 @@ export function ServerConnectionCard({
 
             <div className="flex items-center gap-2">
               <div className="flex items-center gap-2 pr-2 text-xs text-muted-foreground leading-none">
-                <span className="leading-none">{server.enabled === false ? "Disabled" : "Enabled"}</span>
+                <span className="leading-none">
+                  {server.enabled === false ? "Disabled" : "Enabled"}
+                </span>
                 <Switch
                   checked={server.enabled !== false}
                   onCheckedChange={(checked) => {
@@ -227,7 +229,11 @@ export function ServerConnectionCard({
                   <Separator />
                   <DropdownMenuItem
                     className="text-destructive text-xs cursor-pointer"
-                    onClick={() => (onRemove ? onRemove(server.name) : onDisconnect(server.name))}
+                    onClick={() =>
+                      onRemove
+                        ? onRemove(server.name)
+                        : onDisconnect(server.name)
+                    }
                   >
                     <Link2Off className="h-3 w-3 mr-2" />
                     Remove server

client/src/hooks/use-chat.ts

@@ -72,7 +72,7 @@ export function useChat(options: UseChatOptions = {}) {
           id: modelName,
           name: modelName,
           provider: "ollama" as const,
-        })
+        }),
       );
 
       setOllamaModels(ollamaModelDefinitions);
@@ -93,15 +93,15 @@ export function useChat(options: UseChatOptions = {}) {
         setModel(ollamaModels[0]);
       } else if (hasToken("anthropic")) {
         const claudeModel = SUPPORTED_MODELS.find(
-          (m) => m.id === Model.CLAUDE_3_5_SONNET_LATEST
+          (m) => m.id === Model.CLAUDE_3_5_SONNET_LATEST,
         );
         if (claudeModel) setModel(claudeModel);
       } else if (hasToken("openai")) {
         const gptModel = SUPPORTED_MODELS.find((m) => m.id === Model.GPT_4O);
         if (gptModel) setModel(gptModel);
       } else if (hasToken("deepseek")) {
         const deepseekModel = SUPPORTED_MODELS.find(
-          (m) => m.id === Model.DEEPSEEK_CHAT
+          (m) => m.id === Model.DEEPSEEK_CHAT,
         );
         if (deepseekModel) setModel(deepseekModel);
       } else {
@@ -116,7 +116,7 @@ export function useChat(options: UseChatOptions = {}) {
         // For Ollama, return "local" if it's running and the model is available
         return isOllamaRunning &&
           ollamaModels.some(
-            (om) => om.id === model.id || om.id.startsWith(`${model.id}:`)
+            (om) => om.id === model.id || om.id.startsWith(`${model.id}:`),
           )
           ? "local"
           : "";
@@ -133,7 +133,7 @@ export function useChat(options: UseChatOptions = {}) {
         onModelChange(newModel);
       }
     },
-    [onModelChange]
+    [onModelChange],
   );
 
   // Available models with API keys or local Ollama models
@@ -165,7 +165,7 @@ export function useChat(options: UseChatOptions = {}) {
       assistantMessage: ChatMessage,
       assistantContent: { current: string },
       toolCalls: { current: any[] },
-      toolResults: { current: any[] }
+      toolResults: { current: any[] },
     ) => {
       // Handle text content
       if (
@@ -178,7 +178,7 @@ export function useChat(options: UseChatOptions = {}) {
           messages: prev.messages.map((msg) =>
             msg.id === assistantMessage.id
               ? { ...msg, content: assistantContent.current }
-              : msg
+              : msg,
           ),
         }));
         return;
@@ -196,7 +196,7 @@ export function useChat(options: UseChatOptions = {}) {
           messages: prev.messages.map((msg) =>
             msg.id === assistantMessage.id
               ? { ...msg, toolCalls: [...toolCalls.current] }
-              : msg
+              : msg,
           ),
         }));
         return;
@@ -218,7 +218,7 @@ export function useChat(options: UseChatOptions = {}) {
                 ...tc,
                 status: toolResult.error ? "error" : "completed",
               }
-            : tc
+            : tc,
         );
 
         setState((prev) => ({
@@ -230,7 +230,7 @@ export function useChat(options: UseChatOptions = {}) {
                   toolCalls: [...toolCalls.current],
                   toolResults: [...toolResults.current],
                 }
-              : msg
+              : msg,
           ),
         }));
         return;
@@ -261,14 +261,14 @@ export function useChat(options: UseChatOptions = {}) {
         throw new Error(parsed.error);
       }
     },
-    []
+    [],
   );
 
   const sendChatRequest = useCallback(
     async (userMessage: ChatMessage) => {
       if (!serverConfigs || !model || !currentApiKey) {
         throw new Error(
-          "Missing required configuration: serverConfig, model, and apiKey are required"
+          "Missing required configuration: serverConfig, model, and apiKey are required",
         );
       }
 
@@ -349,7 +349,7 @@ export function useChat(options: UseChatOptions = {}) {
                       assistantMessage,
                       assistantContent,
                       toolCalls,
-                      toolResults
+                      toolResults,
                     );
                   } catch (parseError) {
                     console.warn("Failed to parse SSE data:", data, parseError);
@@ -388,7 +388,7 @@ export function useChat(options: UseChatOptions = {}) {
       onMessageReceived,
       handleStreamingEvent,
       getOllamaBaseUrl,
-    ]
+    ],
   );
 
   const sendMessage = useCallback(
@@ -429,7 +429,7 @@ export function useChat(options: UseChatOptions = {}) {
         }
       }
     },
-    [state.isLoading, onMessageSent, sendChatRequest, onError]
+    [state.isLoading, onMessageSent, sendChatRequest, onError],
   );
 
   const stopGeneration = useCallback(() => {
@@ -480,7 +480,7 @@ export function useChat(options: UseChatOptions = {}) {
         }
       }
     },
-    [sendChatRequest, onError]
+    [sendChatRequest, onError],
   );
 
   const deleteMessage = useCallback((messageId: string) => {
@@ -502,7 +502,7 @@ export function useChat(options: UseChatOptions = {}) {
   const handleElicitationResponse = useCallback(
     async (
       action: "accept" | "decline" | "cancel",
-      parameters?: Record<string, any>
+      parameters?: Record<string, any>,
     ) => {
       if (!elicitationRequest) {
         console.warn("Cannot handle elicitation response: no active request");
@@ -554,7 +554,7 @@ export function useChat(options: UseChatOptions = {}) {
         setElicitationLoading(false);
       }
     },
-    [elicitationRequest, onError]
+    [elicitationRequest, onError],
   );
 
   // Cleanup on unmount

client/src/lib/chat-utils.ts

@@ -46,7 +46,7 @@ export function formatMessageDate(date: Date): string {
 export function createMessage(
   role: "user" | "assistant",
   content: string,
-  attachments?: any[]
+  attachments?: any[],
 ): ChatMessage {
   return {
     id: generateId(),
@@ -103,7 +103,7 @@ export function scrollToBottom(element?: Element | null) {
 
 export function debounce<T extends (...args: any[]) => any>(
   func: T,
-  wait: number
+  wait: number,
 ): (...args: Parameters<T>) => void {
   let timeout: NodeJS.Timeout;
   return (...args: Parameters<T>) => {