create-claude-workspace 1.1.78 → 1.1.80

package/dist/scripts/lib/claude-runner.mjs

@@ -321,6 +321,13 @@ export function runClaude(opts, log, runOpts = {}) {
                 const event = parseStreamEvent(line);
                 if (!event) {
                     log.debug(`Non-JSON: ${line.slice(0, 200)}`);
+                    // Check non-JSON stdout lines for error signals (e.g. "You've hit your limit" printed as plain text)
+                    const lower = line.toLowerCase();
+                    detectTextSignals(lower);
+                    // Append to stderr so parseUsageLimitResetMs can find reset time
+                    if (USAGE_LIMIT_RE.test(lower) || RATE_LIMIT_RE.test(lower) || AUTH_ERROR_RE.test(lower)) {
+                        stderr += line + '\n';
+                    }
                     continue;
                 }
                 // Extract session ID
@@ -400,7 +407,8 @@ export function runClaude(opts, log, runOpts = {}) {
             }
             // Flush remaining buffer
             if (buffer.trim()) {
-                const event = parseStreamEvent(buffer.replace(/\r/g, ''));
+                const cleaned = buffer.replace(/\r/g, '');
+                const event = parseStreamEvent(cleaned);
                 if (event) {
                     if (event.session_id && !sessionId)
                         sessionId = event.session_id;
@@ -409,6 +417,14 @@ export function runClaude(opts, log, runOpts = {}) {
                         resultReceived = true;
                     }
                 }
+                else {
+                    // Non-JSON buffer (e.g. "You've hit your limit" without trailing newline)
+                    const lower = cleaned.toLowerCase();
+                    detectTextSignals(lower);
+                    if (USAGE_LIMIT_RE.test(lower) || RATE_LIMIT_RE.test(lower) || AUTH_ERROR_RE.test(lower)) {
+                        stderr += cleaned + '\n';
+                    }
+                }
             }
             // Final stderr-based detection
             detectTextSignals(stderr.toLowerCase());
@@ -424,6 +440,7 @@ export function runClaude(opts, log, runOpts = {}) {
                 timedOut: killReason === 'process_timeout',
                 activityTimedOut: killReason === 'activity_timeout',
                 hasResult: resultReceived,
+                durationMs,
             });
             if (resolved)
                 return;

package/dist/scripts/lib/errors.mjs

@@ -49,7 +49,10 @@ export function classifyError(signals) {
             return category;
     }
     // Exit code 1 with no error indicators → likely max-turns
-    if (signals.code === 1 && stderr.trim().length < 50)
+    // But only if process ran long enough (>10s). Instant exit (e.g. usage limit printed
+    // as plain text that wasn't caught) should NOT be classified as max-turns to prevent
+    // infinite restart loops with no backoff.
+    if (signals.code === 1 && stderr.trim().length < 50 && (signals.durationMs ?? 60_000) > 10_000)
         return 'max_turns';
     // Non-zero exit with content
     if (signals.code !== null && signals.code !== 0)

package/dist/scripts/lib/errors.spec.js

@@ -56,8 +56,11 @@ describe('classifyError', () => {
     it('detects context exhaustion from stderr', () => {
         expect(classifyError({ ...base, code: 1, stderr: 'context_length_exceeded' })).toBe('context_exhausted');
     });
-    it('classifies exit code 1 with minimal stderr as max_turns', () => {
-        expect(classifyError({ ...base, code: 1, stderr: '' })).toBe('max_turns');
+    it('classifies exit code 1 with minimal stderr as max_turns when process ran >10s', () => {
+        expect(classifyError({ ...base, code: 1, stderr: '', durationMs: 60_000 })).toBe('max_turns');
+    });
+    it('classifies instant exit (code 1, <10s, no stderr) as cli_crash not max_turns', () => {
+        expect(classifyError({ ...base, code: 1, stderr: '', durationMs: 1_000 })).toBe('cli_crash');
     });
     it('classifies exit code 1 with content as cli_crash', () => {
         expect(classifyError({ ...base, code: 1, stderr: 'Some unexpected internal error happened during execution blah blah' })).toBe('cli_crash');

package/dist/scripts/lib/loop-integration.spec.js

@@ -674,9 +674,12 @@ describe('Loop integration: classifyError → getErrorAction → checkpoint', ()
             expect(classifyError({ ...base, code: 1, stderr: 'ENOTFOUND' })).toBe('network_error');
             expect(classifyError({ ...base, code: 1, stderr: 'context_length_exceeded' })).toBe('context_exhausted');
         });
-        it('exit 1 with minimal stderr → max_turns', () => {
-            expect(classifyError({ ...base, code: 1, stderr: '' })).toBe('max_turns');
-            expect(classifyError({ ...base, code: 1, stderr: 'short' })).toBe('max_turns');
+        it('exit 1 with minimal stderr and long duration → max_turns', () => {
+            expect(classifyError({ ...base, code: 1, stderr: '', durationMs: 60_000 })).toBe('max_turns');
+            expect(classifyError({ ...base, code: 1, stderr: 'short', durationMs: 60_000 })).toBe('max_turns');
+        });
+        it('exit 1 with minimal stderr and instant exit → cli_crash (not max_turns)', () => {
+            expect(classifyError({ ...base, code: 1, stderr: '', durationMs: 2_000 })).toBe('cli_crash');
         });
         it('exit 1 with long stderr → cli_crash', () => {
             expect(classifyError({ ...base, code: 1, stderr: 'a'.repeat(60) })).toBe('cli_crash');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "create-claude-workspace",
-  "version": "1.1.78",
+  "version": "1.1.80",
   "description": "Scaffold a project with Claude Code agents for autonomous AI-driven development",
   "type": "module",
   "bin": {