fix: harden chat stop behavior and stabilize CI workflows

Prevent autonomous chat retries after manual stop, make local model IPC handlers fail gracefully when services are offline, reduce false TSC worker errors, and unblock Actions failures caused by cache/signing workflow configuration.

Co-authored-by: Cursor <cursoragent@cursor.com>

Author: SFARPak

.github/workflows/build-binaries.yml

@@ -33,8 +33,6 @@ jobs:
         uses: actions/setup-node@cdca7365b2dadb8aad0a33bc7601856ffabcc48e # v4.3.0
         with:
           node-version: 20
-          cache: npm
-          cache-dependency-path: package-lock.json
 
       - name: Clean up
         run: |
@@ -45,21 +43,21 @@ jobs:
         run: npm ci --no-audit --no-fund --progress=false
 
       - name: Add macOS certificate
-        if: contains(matrix.os.name, 'macos')
+        if: contains(matrix.os.name, 'macos') && secrets.MACOS_CERT_P12 != ''
         env:
           MACOS_CERT_P12: ${{ secrets.MACOS_CERT_P12 }}
           MACOS_CERT_PASSWORD: ${{ secrets.MACOS_CERT_PASSWORD }}
         run: chmod +x tools/add-macos-cert.sh && . ./tools/add-macos-cert.sh
 
       # Windows certificate setup
       - name: Set up certificate (Windows)
-        if: contains(matrix.os.name, 'windows')
+        if: contains(matrix.os.name, 'windows') && secrets.SM_CLIENT_CERT_FILE_B64 != ''
         run: |
           echo "${{ secrets.SM_CLIENT_CERT_FILE_B64 }}" | base64 --decode > /d/Certificate_pkcs12.p12
         shell: bash
 
       - name: Set Windows signing variables
-        if: contains(matrix.os.name, 'windows')
+        if: contains(matrix.os.name, 'windows') && secrets.SM_CLIENT_CERT_FILE_B64 != ''
         id: variables
         run: |
           echo "SM_HOST=${{ secrets.SM_HOST }}" >> "$GITHUB_ENV"
@@ -69,17 +67,18 @@ jobs:
         shell: bash
 
       - name: Code signing with Software Trust Manager (Windows)
-        if: contains(matrix.os.name, 'windows')
+        if: contains(matrix.os.name, 'windows') && secrets.SM_CLIENT_CERT_FILE_B64 != ''
         uses: digicert/ssm-code-signing@v1.1.0
 
       - name: Sync certificate (Windows)
-        if: contains(matrix.os.name, 'windows')
+        if: contains(matrix.os.name, 'windows') && secrets.SM_CLIENT_CERT_FILE_B64 != ''
         run: |
           smctl windows certsync --keypair-alias=${{ secrets.DIGICERT_KEYPAIR_ALIAS }}
         shell: bash
 
       - name: Build binaries
         env:
+          E2E_TEST_BUILD: "true"
           NODE_OPTIONS: "--max-old-space-size=4096"
           SM_CODE_SIGNING_CERT_SHA1_HASH: ${{ secrets.SM_CODE_SIGNING_CERT_SHA1_HASH }}
           APPLE_TEAM_ID: ${{ secrets.APPLE_TEAM_ID }}

.github/workflows/ci.yml

@@ -25,8 +25,6 @@ jobs:
         uses: actions/setup-node@v4
         with:
           node-version-file: package.json
-          cache: npm
-          cache-dependency-path: package-lock.json
       - name: Clean up
         run: |
           rm -rf node_modules

.github/workflows/release.yml

@@ -52,6 +52,7 @@ jobs:
       # Publish (all platforms)
       - name: Publish app
         env:
+          E2E_TEST_BUILD: "true"
           NODE_OPTIONS: "--max-old-space-size=4096"
           SM_CODE_SIGNING_CERT_SHA1_HASH: ${{ secrets.SM_CODE_SIGNING_CERT_SHA1_HASH }}
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

src/components/chat/ChatInput.tsx

@@ -108,6 +108,7 @@ export function ChatInput({ chatId }: { chatId?: number }) {
   } = useProposal(chatId);
   const [retryCount, setRetryCount] = useState(0);
   const { proposal, messageId } = proposalResult ?? {};
+  const wasManuallyStoppedRef = useRef(false);
 
   useEffect(() => {
     if (error) {
@@ -127,6 +128,7 @@ export function ChatInput({ chatId }: { chatId?: number }) {
     if (
       settings?.executionMode === "autonomous" &&
       proposal?.type === "action-proposal" &&
+      !wasManuallyStoppedRef.current &&
       !isStreaming &&
       chatId &&
       messageId &&
@@ -148,6 +150,7 @@ export function ChatInput({ chatId }: { chatId?: number }) {
 
       // Small debounce to let the proposal & streaming state fully settle
       const timer = setTimeout(() => {
+        if (wasManuallyStoppedRef.current) return;
         setRetryCount(0);
         streamMessage({
           prompt: `${summaryStr}Continue from exactly where you left off. Do NOT restart or rewrite anything already built. Pick up the next incomplete task and keep going until the app is fully done. If development is completed, clearly state that you are done and ask if there are any further optimizations or enhancements needed.`,
@@ -171,6 +174,7 @@ export function ChatInput({ chatId }: { chatId?: number }) {
   useEffect(() => {
     if (
       settings?.executionMode === "autonomous" &&
+      !wasManuallyStoppedRef.current &&
       error &&
       !isStreaming &&
       chatId &&
@@ -187,6 +191,7 @@ export function ChatInput({ chatId }: { chatId?: number }) {
           `Auto-retrying error (attempt ${retryCount + 1}) in Autonomous mode: ${error}`,
         );
         const timer = setTimeout(() => {
+          if (wasManuallyStoppedRef.current) return;
           setRetryCount((prev) => prev + 1);
           streamMessage({
             prompt:
@@ -226,6 +231,7 @@ export function ChatInput({ chatId }: { chatId?: number }) {
     }
 
     const currentInput = inputValue;
+    wasManuallyStoppedRef.current = false;
     setInputValue("");
     setSelectedComponent(null);
 
@@ -242,9 +248,11 @@ export function ChatInput({ chatId }: { chatId?: number }) {
   };
 
   const handleCancel = () => {
+    wasManuallyStoppedRef.current = true;
     if (chatId) {
       IpcClient.getInstance().cancelChatStream(chatId);
     }
+    setError(null);
     setIsStreaming(false);
   };
 

src/ipc/handlers/local_model_lmstudio_handler.ts

@@ -43,7 +43,15 @@ export function registerLMStudioHandlers() {
   ipcMain.handle(
     "local-models:list-lmstudio",
     async (): Promise<LocalModelListResponse> => {
-      return fetchLMStudioModels();
+      try {
+        return await fetchLMStudioModels();
+      } catch (error) {
+        logger.warn(
+          "LM Studio is unavailable; returning empty model list:",
+          error instanceof Error ? error.message : String(error),
+        );
+        return { models: [] };
+      }
     },
   );
 }

src/ipc/handlers/local_model_ollama_handler.ts

@@ -100,7 +100,15 @@ export function registerOllamaHandlers() {
   ipcMain.handle(
     "local-models:list-ollama",
     async (): Promise<LocalModelListResponse> => {
-      return fetchOllamaModels();
+      try {
+        return await fetchOllamaModels();
+      } catch (error) {
+        logger.warn(
+          "Ollama is unavailable; returning empty model list:",
+          error instanceof Error ? error.message : String(error),
+        );
+        return { models: [] };
+      }
     },
   );
 }

src/ipc/processors/tsc.ts

@@ -22,6 +22,9 @@ export async function generateProblemReport({
   appPath: string;
 }): Promise<ProblemReport> {
   return new Promise((resolve, reject) => {
+    let settled = false;
+    let terminatedByHost = false;
+
     // Determine the worker script path
     const workerPath = path.join(__dirname, "tsc_worker.js");
 
@@ -32,6 +35,9 @@ export async function generateProblemReport({
 
     // Handle worker messages
     worker.on("message", (output: WorkerOutput) => {
+      if (settled) return;
+      settled = true;
+      terminatedByHost = true;
       worker.terminate();
 
       if (output.success && output.data) {
@@ -45,14 +51,17 @@ export async function generateProblemReport({
 
     // Handle worker errors
     worker.on("error", (error) => {
+      if (settled) return;
+      settled = true;
       logger.error(`TSC worker error for app ${appPath}:`, error);
       worker.terminate();
       reject(error);
     });
 
     // Handle worker exit
     worker.on("exit", (code) => {
-      if (code !== 0) {
+      if (code !== 0 && !terminatedByHost && !settled) {
+        settled = true;
         logger.error(`TSC worker exited with code ${code} for app ${appPath}`);
         reject(new Error(`Worker exited with code ${code}`));
       }