specrails-hub 1.58.0 → 1.58.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "specrails-hub",
-  "version": "1.58.0",
+  "version": "1.58.2",
   "license": "MIT",
   "repository": {
     "type": "git",

package/server/dist/project-router.js

@@ -1831,7 +1831,11 @@ function createProjectRouter(registry) {
         // by the codex adapter (it doesn't read extraArgs that don't apply).
         const adapter = (0, providers_1.getAdapter)(provider);
         const toolFlags = provider === 'claude' ? (0, context_scope_1.toolFlagsForScope)(quickScope) : { args: [] };
-        const claudeMaxTurns = quickScope.full ? 6 : (hasAttachments ? 3 : 1);
+        // Full scope grants Read/Grep/Glob. The model spends turns exploring the
+        // repo before it writes the spec; 6 was too tight (a few tool calls on a
+        // sparse/empty repo hit error_max_turns → exit 1 → opaque failure). 15
+        // leaves comfortable headroom while --max-turns still bounds runaway loops.
+        const claudeMaxTurns = quickScope.full ? 15 : (hasAttachments ? 3 : 1);
         const args = adapter.buildArgs('spec-gen', {
             prompt: userPrompt,
             systemPrompt,
@@ -1964,7 +1968,18 @@ function createProjectRouter(registry) {
         child.on('close', async (code) => {
             clearSpecGenWatchdog();
             let createdTicketId = null;
-            if (code === 0 && buffer.trim()) {
+            // When claude burns its whole --max-turns budget it exits non-zero with
+            // a result event of subtype:error_max_turns — but it may already have
+            // emitted a complete spec. Salvage that usable output instead of failing
+            // the whole request on an exit code.
+            const resultSubtype = lastResultEvent?.subtype ?? null;
+            const hasUsableSpec = buffer.trim().length > 0 && /##\s*Spec Title/i.test(buffer);
+            const salvageMaxTurns = code !== 0 && resultSubtype === 'error_max_turns' && hasUsableSpec;
+            if ((code === 0 && buffer.trim()) || salvageMaxTurns) {
+                if (salvageMaxTurns) {
+                    console.warn(`[project-router] spec-gen salvaged partial output after error_max_turns (${requestId}); ` +
+                        `consider raising --max-turns if this recurs`);
+                }
                 // Extract title from generated spec
                 const titleMatch = buffer.match(/##\s*Spec Title\s*\n+(.+)/);
                 const specTitle = titleMatch ? titleMatch[1].trim() : idea.trim().slice(0, 80);
@@ -2088,7 +2103,11 @@ function createProjectRouter(registry) {
                 }
             }
             else {
-                const reason = code === 0 ? 'Empty response from AI' : `Process exited with code ${code}`;
+                const reason = code === 0
+                    ? 'Empty response from AI'
+                    : resultSubtype === 'error_max_turns'
+                        ? 'AI hit its turn limit before finishing the spec. Try again, or narrow the idea / turn off full-codebase context.'
+                        : `Process exited with code ${code}`;
                 console.error(`[project-router] spec-gen failed (${requestId}): ${reason}` +
                     (stderrBuf.trim() ? `\n  stderr: ${stderrBuf.trim()}` : '') +
                     (buffer.trim() ? `\n  stdout-buffer: ${buffer.trim().slice(0, 500)}` : ''));

package/server/dist/providers/claude-adapter.js

@@ -44,6 +44,14 @@ const COMMON_FLAGS = [
     '--tools', 'default',
     '--output-format', 'stream-json',
     '--verbose',
+    // Isolate hub-spawned claude from the *user's* global Claude config. Without
+    // this, the child loads ~/.claude (user CLAUDE.md memory, plugins like
+    // claude-mem, SessionStart hooks). That bled cross-project memory into
+    // Explore turns (e.g. an unrelated "fighting game" surfaced for a fresh
+    // project) and inflated spec-gen tool usage past --max-turns. `project,local`
+    // still loads the target repo's own .claude settings + CLAUDE.md, which is
+    // exactly the context a hub run should see.
+    '--setting-sources', 'project,local',
 ];
 function buildClaudeArgs(action, opts) {
     const args = [];