specrails-desktop 2.7.0 → 2.9.0

package/server/dist/chat-manager.js

@@ -21,6 +21,7 @@ const context_scope_1 = require("./context-scope");
 const user_mcp_config_1 = require("./user-mcp-config");
 const binary_probe_1 = require("./binary-probe");
 const explore_stdin_session_1 = require("./explore-stdin-session");
+const workspace_resolution_1 = require("./workspace-resolution");
 const COMMAND_INSTRUCTION = 'When you want to suggest a SpecRails command for the user to execute, wrap it in a command block like this: ' +
     ':::command\n/specrails:implement #42\n::: ' +
     'The user will be prompted to confirm before the command runs.';
@@ -263,9 +264,33 @@ class ChatManager {
      *
      * See openspec/changes/accelerate-spec-chat-first-token/design.md D1+D4.
      */
+    /**
+     * Relocate-artifacts: resolve execution for a NON-explore (sidebar / rail-like)
+     * spawn. Explore keeps its own explore-cwd logic (untouched). Returns the
+     * relocated cwd + env when relocated, else legacy (cwd = project.path, empty
+     * env). Cached per call — cheap registry read.
+     */
+    _resolveNonExploreExecution() {
+        if (!this._projectSlug || !this._cwd)
+            return null;
+        return (0, workspace_resolution_1.resolveProjectExecution)({ slug: this._projectSlug, path: this._cwd });
+    }
+    /**
+     * Relocate-artifacts: the dir whose `.specrails/local-tickets.json` the scoped
+     * system-prompt prefix reads. Workspace when relocated, else project.path.
+     */
+    _specrailsRoot() {
+        if (!this._cwd)
+            return undefined;
+        const exec = (0, workspace_resolution_1.resolveProjectExecution)({ path: this._cwd });
+        return exec.relocated && exec.workspaceDir ? exec.workspaceDir : this._cwd;
+    }
     _resolveSpawnCwd(kind, scope, providerId) {
-        if (kind !== 'explore')
-            return this._cwd;
+        if (kind !== 'explore') {
+            // Non-explore sidebar: route through the relocate-artifacts gate.
+            const exec = this._resolveNonExploreExecution();
+            return exec ? exec.cwd : this._cwd;
+        }
         if (!this._projectSlug || !this._cwd || !this._projectName)
             return this._cwd;
         // Per-conversation scope.mcp is the only source of truth. Legacy null
@@ -412,7 +437,7 @@ class ChatManager {
             `For roadmap-style requests like "suggest the next best spec", ground the answer in that context, avoid duplicates, and propose one concrete next spec instead of generic directions.`;
         if (!scope || !this._cwd)
             return scopedBase;
-        const prefix = (0, context_scope_1.buildScopedSystemPromptPrefix)(scope, this._cwd);
+        const prefix = (0, context_scope_1.buildScopedSystemPromptPrefix)(scope, this._cwd, this._specrailsRoot());
         if (!prefix)
             return scopedBase;
         return `${scopedBase}\n\n${prefix}`;
@@ -528,7 +553,7 @@ class ChatManager {
             }
             let promptForAdapter = resolvedText;
             if (conversation.kind === 'explore' && adapter.id === 'codex' && conversationScope && this._cwd) {
-                const scopedContext = (0, context_scope_1.buildScopedSystemPromptPrefix)(conversationScope, this._cwd);
+                const scopedContext = (0, context_scope_1.buildScopedSystemPromptPrefix)(conversationScope, this._cwd, this._specrailsRoot());
                 if (scopedContext) {
                     promptForAdapter =
                         `Project context selected in Add Spec. Use it to avoid duplicate specs and to make project-specific recommendations.\n\n` +
@@ -568,6 +593,18 @@ class ChatManager {
             // not pipeline jobs. Telemetry is scoped to QueueManager pipeline runs only.
             // spawnAiCli reroutes multi-line argv values through stdin on Windows.
             const spawnCwd = this._resolveSpawnCwd(conversation.kind, conversationScope, adapter.id);
+            // Relocate-artifacts env for NON-explore (sidebar) spawns. Explore keeps its
+            // own explore-cwd path (no relocation env — it reaches the repo via the
+            // explore-cwd `./project` link). Legacy ⇒ process.env (byte-identical).
+            let spawnEnv = process.env;
+            if (conversation.kind !== 'explore') {
+                const exec = this._resolveNonExploreExecution();
+                if (exec?.relocated) {
+                    spawnEnv = { ...process.env, ...exec.env };
+                    if (adapter.id === 'gemini')
+                        spawnEnv = { ...spawnEnv, GEMINI_CLI_TRUST_WORKSPACE: 'true' };
+                }
+            }
             // Big bet #3 fast-path: persistent-stdin multi-turn for Explore (claude
             // only, flag-gated default OFF). Reuses a single long-lived child across
             // turns so turns 2+ skip spawn + session rehydration. Full fallback to the
@@ -582,7 +619,7 @@ class ChatManager {
                 });
             }
             const child = (0, cli_prompt_1.spawnAiCli)(binary, args, {
-                env: process.env,
+                env: spawnEnv,
                 stdio: ['ignore', 'pipe', 'pipe'],
                 cwd: spawnCwd,
             });

package/server/dist/code-explorer-router.js

@@ -335,6 +335,8 @@ function createCodeExplorerRouter(deps) {
     const router = (0, express_1.Router)({ mergeParams: true });
     const listByPath = deps.listProvenanceByPath ?? file_provenance_1.listProvenanceByPath;
     const listByTicket = deps.listProvenanceByTicket ?? file_provenance_1.listProvenanceByTicket;
+    // Summary OUTPUT root (workspace when relocated). Source reads use projectPath.
+    const summaryRoot = () => deps.resolveSummaryRoot?.() ?? deps.projectPath;
     // Short-TTL per-project cache so paginating a large `all` tree reuses ONE
     // synchronous filesystem walk instead of re-walking (and re-statting) on every
     // page — the cursor only slices an already-materialized array. Also caches the
@@ -354,7 +356,7 @@ function createCodeExplorerRouter(deps) {
     function getSummaryHashesCached() {
         if (summaryHashCache && nowMs() - summaryHashCache.at < WALK_CACHE_TTL_MS)
             return summaryHashCache.set;
-        const set = readSummaryHashSet(deps.projectPath);
+        const set = readSummaryHashSet(summaryRoot());
         summaryHashCache = { at: nowMs(), set };
         return set;
     }
@@ -369,7 +371,7 @@ function createCodeExplorerRouter(deps) {
         // that broke terminals); attachWatcher is idempotent, so this is cheap on
         // every subsequent request.
         try {
-            deps.fileSummaryManager.attachWatcher(deps.projectId, deps.projectPath);
+            deps.fileSummaryManager.attachWatcher(deps.projectId, deps.projectPath, summaryRoot());
         }
         catch { /* non-fatal */ }
         next();
@@ -463,7 +465,7 @@ function createCodeExplorerRouter(deps) {
         catch {
             // Honour the staleness scenario: even if content is unavailable, return
             // the existing summary so the client can render a "not found" banner.
-            const summary = (0, file_summary_manager_1.readSummary)(deps.projectPath, rel);
+            const summary = (0, file_summary_manager_1.readSummary)(summaryRoot(), rel);
             const provenance = listByPath(deps.db, deps.projectId, rel);
             if (summary || provenance.length > 0) {
                 res.json({
@@ -487,7 +489,7 @@ function createCodeExplorerRouter(deps) {
                 tooLarge: true,
                 sizeBytes: stat.size,
                 provenance: provenanceRowsToJson(listByPath(deps.db, deps.projectId, rel)),
-                summary: (0, file_summary_manager_1.readSummary)(deps.projectPath, rel),
+                summary: (0, file_summary_manager_1.readSummary)(summaryRoot(), rel),
                 absolutePath: abs,
             });
             return;
@@ -514,7 +516,7 @@ function createCodeExplorerRouter(deps) {
                 sizeBytes: stat.size,
                 mime: 'application/octet-stream',
                 provenance: provenanceRowsToJson(listByPath(deps.db, deps.projectId, rel)),
-                summary: (0, file_summary_manager_1.readSummary)(deps.projectPath, rel),
+                summary: (0, file_summary_manager_1.readSummary)(summaryRoot(), rel),
                 absolutePath: abs,
             });
             return;
@@ -527,7 +529,7 @@ function createCodeExplorerRouter(deps) {
             res.status(500).json({ error: 'failed to read file' });
             return;
         }
-        const summary = (0, file_summary_manager_1.readSummary)(deps.projectPath, rel);
+        const summary = (0, file_summary_manager_1.readSummary)(summaryRoot(), rel);
         const summaryStale = await computeStaleness(abs, summary);
         res.json({
             content,
@@ -637,7 +639,7 @@ function createCodeExplorerRouter(deps) {
             res.status(403).json({ error: 'path is gitignored' });
             return;
         }
-        const summary = (0, file_summary_manager_1.readSummary)(deps.projectPath, rel);
+        const summary = (0, file_summary_manager_1.readSummary)(summaryRoot(), rel);
         if (!summary) {
             res.json({ summary: null });
             return;
@@ -711,6 +713,7 @@ function createCodeExplorerRouter(deps) {
             // the content hash is unchanged (e.g. after an app language switch).
             const result = await deps.fileSummaryManager.enqueue({
                 projectPath: deps.projectPath,
+                summaryRoot: summaryRoot(),
                 projectId: deps.projectId,
                 projectSlug: deps.projectId,
                 relPath: rel,

package/server/dist/config.js

@@ -218,7 +218,12 @@ function loadPersistedConfig(db) {
         return { active: null, labelFilter: '' };
     }
 }
-function getConfig(cwd, db, projectName) {
+function getConfig(cwd, db, projectName, 
+/** Relocate-artifacts: the dir whose `.claude/commands/sr` holds the
+ *  materialized sr/specrails commands — the workspace when the project is
+ *  relocated, else === the project root. Git/repo detection ALWAYS uses the
+ *  repo root (`projectRoot`). Defaults to `projectRoot` (byte-identical). */
+commandsRoot) {
     // Resolve project root. Super mode passes project.path directly, which has a
     // `.claude` directory at its root; the walk-up fallback covers callers that
     // pass a manager-relative cwd.
@@ -229,7 +234,7 @@ function getConfig(cwd, db, projectName) {
     else {
         projectRoot = path_1.default.resolve(cwd, '../..');
     }
-    const commandsDir = path_1.default.join(projectRoot, '.claude', 'commands', 'sr');
+    const commandsDir = path_1.default.join(commandsRoot ?? projectRoot, '.claude', 'commands', 'sr');
     const commands = scanCommands(commandsDir);
     const { github, jira } = detectTrackers();
     const repo = getGitRepoName(projectRoot);

package/server/dist/context-budget.js

@@ -15,6 +15,7 @@ exports.getContextBudget = getContextBudget;
 const node_fs_1 = __importDefault(require("node:fs"));
 const node_path_1 = __importDefault(require("node:path"));
 const context_scope_1 = require("./context-scope");
+const workspace_resolution_1 = require("./workspace-resolution");
 const BYTES_PER_TOKEN = 4;
 const CACHE_TTL_MS = 60_000;
 const SKIP_DIRS = new Set([
@@ -72,8 +73,8 @@ function walkCodebase(root) {
     }
     return { fileCount, bytes };
 }
-function readMcpServers(projectPath) {
-    const file = node_path_1.default.join(projectPath, '.mcp.json');
+function readMcpServers(mcpRoot) {
+    const file = node_path_1.default.join(mcpRoot, '.mcp.json');
     if (!node_fs_1.default.existsSync(file))
         return [];
     try {
@@ -87,11 +88,17 @@ function readMcpServers(projectPath) {
 function bytesOfSection(section) {
     return section ? Buffer.byteLength(section, 'utf8') : 0;
 }
-function computeContextBudget(projectPath) {
+function computeContextBudget(projectPath, 
+/** Relocate-artifacts: the dir whose `.specrails/local-tickets.json` + `.mcp.json`
+ *  the relocated CLI actually loads — the workspace when relocated, else
+ *  === projectPath. OpenSpec specs + the codebase walk ALWAYS read the repo
+ *  (projectPath). Defaults to `projectPath` (byte-identical legacy). */
+specrailsRoot) {
+    const root = specrailsRoot ?? projectPath;
     // Use the same builders the spawn path uses — token estimates reflect what
     // will actually land in the system prompt (caps + formatting included), not
     // the raw on-disk byte count of source files.
-    const ticketsBytes = bytesOfSection((0, context_scope_1.buildSpecrailsTicketsSection)(projectPath));
+    const ticketsBytes = bytesOfSection((0, context_scope_1.buildSpecrailsTicketsSection)(root));
     const openspecBytes = bytesOfSection((0, context_scope_1.buildOpenSpecSpecsSection)(projectPath));
     const { fileCount, bytes: codebaseBytes } = walkCodebase(projectPath);
     return {
@@ -99,7 +106,7 @@ function computeContextBudget(projectPath) {
         openspecSpecsTokens: Math.round(openspecBytes / BYTES_PER_TOKEN),
         codebaseFileCount: fileCount,
         codebaseEstimatedTokens: Math.round(codebaseBytes / BYTES_PER_TOKEN),
-        mcpServers: readMcpServers(projectPath),
+        mcpServers: readMcpServers(root),
     };
 }
 function getContextBudget(projectId, projectPath) {
@@ -107,7 +114,11 @@ function getContextBudget(projectId, projectPath) {
     const hit = cache.get(projectId);
     if (hit && hit.expiresAt > now)
         return hit.value;
-    const value = computeContextBudget(projectPath);
+    // Relocate-artifacts: resolve the workspace root so the tickets + .mcp.json
+    // estimates reflect what the relocated CLI loads (legacy ⇒ === projectPath).
+    const exec = (0, workspace_resolution_1.resolveProjectExecution)({ path: projectPath });
+    const specrailsRoot = exec.relocated && exec.workspaceDir ? exec.workspaceDir : projectPath;
+    const value = computeContextBudget(projectPath, specrailsRoot);
     cache.set(projectId, { value, expiresAt: now + CACHE_TTL_MS });
     return value;
 }

package/server/dist/context-scope.js

@@ -247,10 +247,14 @@ function buildOpenSpecSpecsSection(projectPath) {
         return null;
     return `## OpenSpec Specs\n\n${text}${truncated ? '\n(truncated)\n' : ''}`;
 }
-function buildScopedSystemPromptPrefix(scope, projectPath) {
+function buildScopedSystemPromptPrefix(scope, projectPath, 
+/** Relocate-artifacts: where `.specrails/local-tickets.json` lives — the
+ *  workspace dir when relocated, else === projectPath. OpenSpec specs always
+ *  read from the repo (`projectPath`). Defaults to `projectPath` (legacy). */
+specrailsRoot) {
     const sections = [];
     if (scope.specrails) {
-        const s = buildSpecrailsTicketsSection(projectPath);
+        const s = buildSpecrailsTicketsSection(specrailsRoot ?? projectPath);
         if (s)
             sections.push(s);
     }

package/server/dist/contract-refine-runner.js

@@ -27,7 +27,18 @@ const explore_cwd_manager_1 = require("./explore-cwd-manager");
 const ai_invocations_1 = require("./ai-invocations");
 const result_event_1 = require("./result-event");
 const ticket_store_1 = require("./ticket-store");
+const workspace_resolution_1 = require("./workspace-resolution");
 const REFINE_TIMEOUT_MS = 60_000;
+/**
+ * Relocate-artifacts: resolve the ticket store path honouring the gate.
+ * Relocated ⇒ the registry entry's tickets path (workspace); legacy ⇒
+ * `resolveTicketStoragePath(projectPath)` (preserves integration-contract.json
+ * custom-storagePath behaviour for existing repos).
+ */
+function resolveContractTicketsPath(projectPath) {
+    const exec = (0, workspace_resolution_1.resolveProjectExecution)({ path: projectPath });
+    return exec.relocated ? exec.ticketsPath : (0, ticket_store_1.resolveTicketStoragePath)(projectPath);
+}
 function normalizeClaudeCodeModel(model) {
     if (!model || typeof model !== 'string')
         return 'sonnet';
@@ -61,7 +72,13 @@ function prepareContractRefineSpawn(deps, conversation) {
             /* default false */
         }
     }
-    let cwd = deps.projectPath;
+    // Relocate-artifacts: when mcp is on, the spawn cwd is the project's spec root,
+    // which becomes the workspace when relocated (else project.path). When mcp is
+    // off the explore-cwd is used as before. Env carries SPECRAILS_REPO_DIR for the
+    // relocated case so source/openspec I/O re-points into the repo.
+    const exec = (0, workspace_resolution_1.resolveProjectExecution)({ slug: deps.projectSlug, path: deps.projectPath });
+    let cwd = exec.cwd;
+    let env = exec.relocated ? { ...process.env, ...exec.env } : undefined;
     if (!mcpEnabled) {
         try {
             cwd = (0, explore_cwd_manager_1.ensureExploreCwd)({
@@ -69,13 +86,15 @@ function prepareContractRefineSpawn(deps, conversation) {
                 projectPath: deps.projectPath,
                 projectName: deps.projectName,
             });
+            // explore-cwd reaches the repo via its own `./project` link; keep the
+            // relocation env so SPECRAILS_REPO_DIR is still exported when relocated.
         }
         catch {
-            cwd = deps.projectPath;
+            cwd = exec.cwd;
         }
     }
     const args = buildRefineArgs(conversation.model ?? 'sonnet', systemPrompt, conversation.session_id ?? '');
-    return { args, cwd, systemPrompt };
+    return { args, cwd, systemPrompt, env };
 }
 /**
  * Test-friendly inner runner: takes a child-like object and returns the parsed
@@ -238,7 +257,7 @@ async function runContractRefine(deps, conversationId, ticketId) {
         ticketId,
         timestamp: now().toISOString(),
     });
-    const { args, cwd } = prepareContractRefineSpawn({
+    const { args, cwd, env: refineEnv } = prepareContractRefineSpawn({
         projectSlug: deps.projectSlug,
         projectPath: deps.projectPath,
         projectName: deps.projectName,
@@ -256,7 +275,7 @@ async function runContractRefine(deps, conversationId, ticketId) {
         binary: 'claude',
         argv: args,
         cwd,
-        env: process.env,
+        env: refineEnv ?? process.env,
         spawn,
         timeoutMs,
         onStdoutLine: (line) => {
@@ -344,7 +363,7 @@ async function runContractRefine(deps, conversationId, ticketId) {
     // Patch the ticket description.
     let updated = null;
     try {
-        const filePath = (0, ticket_store_1.resolveTicketStoragePath)(deps.projectPath);
+        const filePath = resolveContractTicketsPath(deps.projectPath);
         updated = applyContractLayerToTicket(filePath, ticketId, parse.value, finishedAt);
     }
     catch (err) {
@@ -416,12 +435,15 @@ async function runContractRefineForQuick(deps, ticketId, generatedTitle, generat
         ticketId,
         timestamp: startedAt,
     });
+    // Relocate-artifacts gate: spawn from the workspace + SPECRAILS_REPO_DIR when
+    // relocated, else cwd = project.path + process.env (byte-identical legacy).
+    const quickExec = (0, workspace_resolution_1.resolveProjectExecution)({ slug: deps.projectSlug, path: deps.projectPath });
     let child;
     try {
         child = spawn('claude', args, {
-            env: process.env,
+            env: quickExec.relocated ? { ...process.env, ...quickExec.env } : process.env,
             stdio: ['ignore', 'pipe', 'pipe'],
-            cwd: deps.projectPath,
+            cwd: quickExec.cwd,
         });
     }
     catch (err) {
@@ -480,7 +502,7 @@ async function runContractRefineForQuick(deps, ticketId, generatedTitle, generat
     }
     let updated = null;
     try {
-        const filePath = (0, ticket_store_1.resolveTicketStoragePath)(deps.projectPath);
+        const filePath = resolveContractTicketsPath(deps.projectPath);
         updated = applyContractLayerToTicket(filePath, ticketId, parse.value, finishedAt);
     }
     catch (err) {

package/server/dist/desktop-router.js

@@ -21,6 +21,25 @@ const terminal_settings_1 = require("./terminal-settings");
 function slugify(name) {
     return name.toLowerCase().replace(/[^a-z0-9]+/g, '-').replace(/^-|-$/g, '');
 }
+/**
+ * Deterministic slug allocation with a `-N` dedup suffix — byte-parity with
+ * specrails-core's `allocateSlug` (and artifact-registry). Without this, two
+ * repos that share a basename (e.g. `frontend` in two parent dirs) both slugify
+ * to `frontend`, and the second `addProject` dies on the `projects.slug UNIQUE`
+ * constraint with a misleading 409 ("already registered") even though the PATH
+ * is new. The `-2`, `-3`… suffix gives the second repo a distinct slug.
+ */
+function allocateSlug(name, existing) {
+    let base = slugify(name);
+    if (base === '')
+        base = 'project';
+    if (!existing.has(base))
+        return base;
+    let n = 2;
+    while (existing.has(`${base}-${n}`))
+        n += 1;
+    return `${base}-${n}`;
+}
 // Emergency rollback for the codex provider: SPECRAILS_CODEX_BETA=0 forces
 // codex back to "unavailable" without redeploying. The pre-rebrand
 // SPECRAILS_HUB_CODEX_BETA name is read as a legacy fallback when the new
@@ -29,13 +48,15 @@ function isCodexBetaDisabled() {
     const v = process.env.SPECRAILS_CODEX_BETA ?? process.env.SPECRAILS_HUB_CODEX_BETA;
     return v === '0';
 }
-// Gemini is opt-IN (default off): unlike codex, its stream-json schema has not
-// yet been validated against a live binary, so it stays hidden + unselectable
-// until SPECRAILS_GEMINI_BETA=1 (or 'true'). The adapter is always registered
-// (pricing/getAdapter work); only project selection is gated.
-function isGeminiBetaEnabled() {
-    const v = (process.env.SPECRAILS_GEMINI_BETA ?? '').toLowerCase();
-    return v === '1' || v === 'true';
+// Gemini is enabled by DEFAULT now that its stream-json schema and the full rails
+// pipeline (architect→developer→reviewer delegation, headless agent loading, the
+// MAX_TURNS resume + reviewer gate) are validated against the live binary
+// (0.46/0.47). Emergency rollback: SPECRAILS_GEMINI_BETA=0 forces it back to
+// "unavailable" without redeploying — parity with codex's SPECRAILS_CODEX_BETA.
+// The adapter is always registered (pricing/getAdapter work); only project
+// selection is gated.
+function isGeminiBetaDisabled() {
+    return process.env.SPECRAILS_GEMINI_BETA === '0';
 }
 // Theme allow-list. Mirror of THEME_IDS in `client/src/lib/themes.ts` —
 // kept duplicated to avoid pulling client code into the server bundle.
@@ -160,10 +181,10 @@ function createDesktopRouter(registry, broadcast) {
         const gated = { ...providers };
         if (isCodexBetaDisabled())
             gated.codex = false;
-        // Gemini is opt-in: omit it entirely (not just `false`) when the beta flag is
-        // off, so it stays fully invisible in the UI until SPECRAILS_GEMINI_BETA=1.
-        if (!isGeminiBetaEnabled())
-            delete gated.gemini;
+        // Gemini: enabled by default; forced unavailable only when
+        // SPECRAILS_GEMINI_BETA=0 (emergency rollback, parity with codex).
+        if (isGeminiBetaDisabled())
+            gated.gemini = false;
         res.json({ ...gated, tiers });
     });
     router.get('/setup-prerequisites', (req, res) => {
@@ -231,9 +252,9 @@ function createDesktopRouter(registry, broadcast) {
             });
             return;
         }
-        if (providers.includes('gemini') && !isGeminiBetaEnabled()) {
+        if (providers.includes('gemini') && isGeminiBetaDisabled()) {
             res.status(400).json({
-                error: 'Gemini provider is in beta and disabled by default. Set SPECRAILS_GEMINI_BETA=1 to enable.',
+                error: 'Gemini provider is currently disabled (SPECRAILS_GEMINI_BETA=0). Unset or set to 1 to enable.',
             });
             return;
         }
@@ -252,7 +273,11 @@ function createDesktopRouter(registry, broadcast) {
         const derivedName = (name && typeof name === 'string' && name.trim())
             ? name.trim()
             : deriveProjectName(canonicalPath);
-        const slug = slugify(derivedName);
+        // Dedup the slug against already-registered projects so two same-basename
+        // repos don't collide on the projects.slug UNIQUE constraint (which would
+        // surface as a misleading 409 for a brand-new PATH).
+        const existingSlugs = new Set((0, desktop_db_1.listProjects)(registry.desktopDb).map((p) => p.slug));
+        const slug = allocateSlug(derivedName, existingSlugs);
         const id = (0, crypto_1.randomUUID)();
         const specrailsInstalled = hasSpecrails(canonicalPath);
         try {