specrails-desktop 2.30.2 → 2.31.0

package/server/dist/result-event.js

@@ -37,6 +37,11 @@ function finaliseInvocationResult(adapter, events, opts = {}) {
     if (!cloned.model && opts.fallbackModel) {
         cloned.model = opts.fallbackModel;
     }
+    if (cloned.duration_ms == null &&
+        typeof opts.durationMs === 'number' &&
+        Number.isFinite(opts.durationMs)) {
+        cloned.duration_ms = Math.max(0, opts.durationMs);
+    }
     let estimated = false;
     // Guard: when the breakdown carries NO billable usage (every token field
     // undefined/0 — e.g. a codex/gemini turn aborted/crashed before its

package/server/dist/schemas/profile.v1.json

@@ -26,7 +26,7 @@
         "provider": {
             "type": "string",
             "pattern": "^[a-z0-9][a-z0-9-]*$",
-            "description": "Optional provider id (e.g. `claude`, `codex`). When set, the profile's model values are validated against that provider's adapter catalog. When omitted, validation falls back to the project's resolved provider."
+            "description": "Optional registered provider id (for example `claude`, `codex`, `gemini`, or `kimi`). When set, the profile's model values are validated according to that provider adapter: normally against its catalog, while adapters that support configured aliases (such as Kimi) accept any schema-valid non-empty alias. When omitted, validation falls back to the project's resolved provider."
         },
         "orchestrator": {
             "type": "object",
@@ -82,8 +82,7 @@
         "modelAlias": {
             "type": "string",
             "minLength": 1,
-            "maxLength": 64,
-            "description": "Accepted model alias. JSON schema accepts any non-empty string; provider-specific catalog enforcement runs at runtime (see profile-manager.validateStructural)."
+            "description": "Accepted model alias. JSON schema accepts any non-empty string without truncating provider-configured aliases; provider-specific catalog enforcement runs at runtime (see profile-manager.validateStructural)."
         },
         "agentEntry": {
             "type": "object",
@@ -93,7 +92,7 @@
                 "id": {
                     "type": "string",
                     "pattern": "^(sr|custom)-[a-z0-9][a-z0-9-]*$",
-                    "description": "Agent identifier. MUST correspond to a file at `.claude/agents/<id>.md`."
+                    "description": "Agent identifier. MUST correspond to the selected provider's provider-native role artifact. File-based providers use their own agents catalog (for example `.claude/agents/<id>.md`); Kimi uses the directly discoverable `.kimi-code/skills/<id>/SKILL.md` layout."
                 },
                 "model": {
                     "$ref": "#/$defs/modelAlias",

package/server/dist/setup-manager.js

@@ -20,6 +20,7 @@ const win_spawn_1 = require("./util/win-spawn");
 const setup_prerequisites_1 = require("./setup-prerequisites");
 const core_package_1 = require("./core-package");
 const providers_1 = require("./providers");
+const runtime_1 = require("./providers/runtime");
 const result_event_1 = require("./result-event");
 const ai_invocations_1 = require("./ai-invocations");
 const crypto_1 = require("crypto");
@@ -240,21 +241,29 @@ exports.QUICK_CHECKPOINTS = [
     { key: 'base_install', name: 'Base installation' },
     { key: 'quick_complete', name: 'Quick install complete' },
 ];
-function checkFilesystem(projectPath) {
-    const dir = SPECRAILS_DIR;
+function checkFilesystem(projectPath, provider = 'claude') {
+    const adapter = (0, providers_1.hasAdapter)(provider) ? (0, providers_1.getAdapter)(provider) : (0, providers_1.getAdapter)('claude');
+    const dir = adapter.projectDirName;
+    const artifactState = setupArtifactState(projectPath, 'full', provider);
     const hasBaseInstall = (0, fs_1.existsSync)((0, path_1.join)(projectPath, '.specrails', 'specrails-version')) ||
         (0, fs_1.existsSync)((0, path_1.join)(projectPath, '.specrails-version'));
     const hasSetupTemplates = (0, fs_1.existsSync)((0, path_1.join)(projectPath, '.specrails', 'setup-templates')) ||
         (0, fs_1.existsSync)((0, path_1.join)(projectPath, dir, 'setup-templates'));
     const hasRules = (0, fs_1.existsSync)((0, path_1.join)(projectPath, dir, 'rules')) &&
         hasFiles((0, path_1.join)(projectPath, dir, 'rules'), /\.md$/);
-    const hasPersonas = (0, fs_1.existsSync)((0, path_1.join)(projectPath, dir, 'agents', 'personas')) &&
-        hasFiles((0, path_1.join)(projectPath, dir, 'agents', 'personas'), /\.md$/);
-    const hasAgents = (0, fs_1.existsSync)((0, path_1.join)(projectPath, dir, 'agents')) &&
-        hasFiles((0, path_1.join)(projectPath, dir, 'agents'), /^sr-.*\.md$/);
-    const hasCommands = (((0, fs_1.existsSync)((0, path_1.join)(projectPath, dir, 'commands', 'sr')) && hasFiles((0, path_1.join)(projectPath, dir, 'commands', 'sr'), /\.md$/)) ||
-        ((0, fs_1.existsSync)((0, path_1.join)(projectPath, dir, 'commands', 'specrails')) && hasFiles((0, path_1.join)(projectPath, dir, 'commands', 'specrails'), /\.md$/)));
-    const hasCLAUDE = (0, fs_1.existsSync)((0, path_1.join)(projectPath, 'CLAUDE.md'));
+    const hasPersonas = artifactState.hasPersonas;
+    const hasAgents = artifactState.hasAgents;
+    const hasCommands = artifactState.hasCommands;
+    const instructionCandidates = [(0, path_1.join)(projectPath, adapter.instructionsFilename)];
+    // Older Core releases sometimes placed a top-level instructions filename
+    // inside the provider directory. Preserve that compatibility only for
+    // actual filenames: Kimi's adapter already declares the provider-relative
+    // `.kimi-code/AGENTS.md` path, so prefixing projectDirName again would probe
+    // the nonsensical `.kimi-code/.kimi-code/AGENTS.md`.
+    if (adapter.instructionsFilename === (0, path_1.basename)(adapter.instructionsFilename)) {
+        instructionCandidates.push((0, path_1.join)(projectPath, dir, adapter.instructionsFilename));
+    }
+    const hasInstructions = instructionCandidates.some((candidate) => (0, fs_1.existsSync)(candidate));
     const hasInstallConfig = (0, fs_1.existsSync)((0, path_1.join)(projectPath, '.specrails', 'install-config.yaml')) ||
         (0, fs_1.existsSync)((0, path_1.join)(projectPath, dir, 'install-config.yaml'));
     const hasAgentConfig = (0, fs_1.existsSync)((0, path_1.join)(projectPath, dir, 'agents.yaml'));
@@ -264,7 +273,7 @@ function checkFilesystem(projectPath) {
         // Back-compat filesystem signals from older cores are translated to the
         // current integration-contract checkpoint keys.
         agent_selection: hasInstallConfig || hasAgentConfig || hasBacklogConfig || hasAgents || hasCommands,
-        codebase_analysis: hasBaseInstall && (hasCLAUDE || hasSetupTemplates),
+        codebase_analysis: hasBaseInstall && (hasInstructions || hasSetupTemplates),
         vpc_discovery: hasPersonas,
         persona_synthesis: hasPersonas,
         agent_generation: hasAgents,
@@ -355,19 +364,19 @@ function detectCheckpointFromText(text) {
     if (/\/agents\/sr-[^/]+\.md/.test(text)) {
         hits.push({ key: 'agent_generation', detail: 'Writing agents...' });
     }
-    // Codex path: .codex/skills/sr-<name>/SKILL.md (rail skills ship in
-    // specrails-core 4.6.0+ — see openspec/.../specs/setup-wizard… for the
-    // checkpoint protocol shared across providers).
-    if (/\.codex\/skills\/sr-[^/]+\/SKILL\.md/.test(text)) {
+    // Provider-native rail skills. Kimi's upstream discovery contract scans only
+    // direct children of `.kimi-code/skills`, so its roles are top-level
+    // `.kimi-code/skills/sr-*/SKILL.md` entries. Codex may also use `rails/`.
+    if (/\.(?:codex|kimi-code)\/skills\/(?:rails\/)?sr-[^/]+\/SKILL\.md/.test(text)) {
         hits.push({ key: 'agent_generation', detail: 'Writing agent skills...' });
     }
     if ((text.includes('/commands/sr/') || text.includes('/commands/specrails/')) && text.includes('.md')) {
         hits.push({ key: 'command_generation', detail: 'Writing commands...' });
     }
-    // Codex enrich/doctor skills (the non-rail commands) also indicate
+    // Provider-native enrich/doctor skills (the non-rail commands) also indicate
     // command_generation progress.
-    if (/\.codex\/skills\/(enrich|doctor)\/SKILL\.md/.test(text)) {
-        hits.push({ key: 'command_generation', detail: 'Writing codex command skills...' });
+    if (/\.(?:codex|kimi-code)\/skills\/(?:specrails-)?(?:enrich|doctor)\/SKILL\.md/.test(text)) {
+        hits.push({ key: 'command_generation', detail: 'Writing provider command skills...' });
     }
     if (text.includes('/rules/') && text.includes('.md')) {
         hits.push({ key: 'command_generation', detail: 'Writing conventions...' });
@@ -456,24 +465,33 @@ function computeSummary(projectPath, tier, provider = 'claude') {
     let specrailsCommands = 0;
     let opsxCommands = 0;
     try {
-        if (provider === 'codex') {
-            // Codex layout: every artefact ships as a SKILL under `.codex/skills/`.
+        if (provider === 'codex' || provider === 'kimi') {
+            // Codex/Kimi layout: every artefact ships as a SKILL under the provider's
+            // `skills/` root.
             // - agents  = rail personas (`skills/rails/sr-*/SKILL.md`) + orchestrator
-            //             skills at the root with an `sr-` prefix (sr-implement,
-            //             sr-batch-implement, …)
+            //             skills at the root with an `sr-` prefix. Kimi roles MUST be
+            //             direct children (`skills/sr-*/SKILL.md`) because upstream
+            //             Kimi discovery is intentionally one level deep.
             // - opsxCommands     = `skills/openspec-*/SKILL.md`
             // - specrailsCommands = everything else under `skills/` (ported claude
             //   slash commands like propose-spec, explore-spec, retry, doctor,
             //   enrich, vpc-drift, …)
-            // - personas = 0 today; codex VPC pass not implemented yet.
-            const skillsDir = (0, path_1.join)(projectPath, '.codex', 'skills');
+            // - personas = Kimi's provider-local `.kimi-code/personas/*.md`; Codex
+            //              keeps its legacy nested skill representation.
+            const skillsDir = (0, path_1.join)(projectPath, provider === 'kimi' ? '.kimi-code' : '.codex', 'skills');
             if ((0, fs_1.existsSync)(skillsDir)) {
-                // Rails (always counted as agents).
+                // Codex retains its nested Rails catalog. Kimi cannot: nested skills
+                // are not discoverable by the Kimi CLI.
                 const railsDir = (0, path_1.join)(skillsDir, 'rails');
-                if ((0, fs_1.existsSync)(railsDir)) {
+                if (provider === 'codex' && (0, fs_1.existsSync)(railsDir)) {
                     for (const entry of (0, fs_1.readdirSync)(railsDir)) {
-                        if ((0, fs_1.existsSync)((0, path_1.join)(railsDir, entry, 'SKILL.md')))
+                        if (entry === 'personas') {
+                            const personasDir = (0, path_1.join)(railsDir, entry);
+                            personas = (0, fs_1.readdirSync)(personasDir).filter((persona) => (0, fs_1.existsSync)((0, path_1.join)(personasDir, persona, 'SKILL.md'))).length;
+                        }
+                        else if ((0, fs_1.existsSync)((0, path_1.join)(railsDir, entry, 'SKILL.md'))) {
                             agents++;
+                        }
                     }
                 }
                 // Top-level skill dirs.
@@ -490,6 +508,14 @@ function computeSummary(projectPath, tier, provider = 'claude') {
                         specrailsCommands++;
                 }
             }
+            if (provider === 'kimi') {
+                const personasDir = (0, path_1.join)(projectPath, '.kimi-code', 'personas');
+                if ((0, fs_1.existsSync)(personasDir)) {
+                    personas = (0, fs_1.readdirSync)(personasDir)
+                        .filter((entry) => entry.endsWith('.md'))
+                        .length;
+                }
+            }
         }
         else {
             // Claude / Gemini layout: agents at `<dir>/agents/sr-*.md`, slash commands
@@ -510,10 +536,18 @@ function computeSummary(projectPath, tier, provider = 'claude') {
                 }
             }
             const commandsDirSpecrails = (0, path_1.join)(projectPath, dir, 'commands', 'specrails');
+            const commandsDirLegacySr = (0, path_1.join)(projectPath, dir, 'commands', 'sr');
             const commandsDirOpsx = (0, path_1.join)(projectPath, dir, 'commands', 'opsx');
             if ((0, fs_1.existsSync)(commandsDirSpecrails)) {
                 specrailsCommands = (0, fs_1.readdirSync)(commandsDirSpecrails).filter((f) => f.endsWith(commandExt)).length;
             }
+            if (specrailsCommands === 0 && (0, fs_1.existsSync)(commandsDirLegacySr)) {
+                // Core <=4.1 used commands/sr. Count it only when the canonical
+                // specrails tree has no commands so a transition install is never
+                // doubled but an empty pre-created directory cannot hide real legacy
+                // artefacts.
+                specrailsCommands = (0, fs_1.readdirSync)(commandsDirLegacySr).filter((f) => f.endsWith(commandExt)).length;
+            }
             if ((0, fs_1.existsSync)(commandsDirOpsx)) {
                 opsxCommands = (0, fs_1.readdirSync)(commandsDirOpsx).filter((f) => f.endsWith(commandExt)).length;
             }
@@ -524,6 +558,35 @@ function computeSummary(projectPath, tier, provider = 'claude') {
     }
     return { agents, specrailsCommands, opsxCommands, personas, legacySrRemoved: 0, tier, provider };
 }
+/**
+ * Resolve completion against the selected provider's real filesystem layout.
+ * A legacy `.claude` fallback remains intentional only for Codex/Gemini:
+ * older Core releases scaffolded Claude artefacts for those selections.
+ * Kimi never had such a released legacy Core target; accepting a mixed
+ * project's `.claude` tree would falsely report a missing `.kimi-code` setup
+ * as complete.
+ */
+function setupArtifactState(projectPath, tier, provider) {
+    const nativeSummary = computeSummary(projectPath, tier, provider);
+    const supportsLegacyClaudeFallback = provider === 'codex' || provider === 'gemini';
+    const legacySummary = supportsLegacyClaudeFallback
+        ? computeSummary(projectPath, tier, 'claude')
+        : nativeSummary;
+    const nativeHasCommands = nativeSummary.specrailsCommands + nativeSummary.opsxCommands > 0;
+    const legacyHasCommands = legacySummary.specrailsCommands + legacySummary.opsxCommands > 0;
+    const nativeComplete = nativeSummary.agents > 0 && nativeHasCommands;
+    const legacyComplete = legacySummary.agents > 0 && legacyHasCommands;
+    const selected = nativeComplete || !legacyComplete ? nativeSummary : legacySummary;
+    return {
+        // Preserve the actual selected provider for UI labels even when the
+        // artefacts came from the legacy Claude compatibility tree.
+        summary: { ...selected, provider },
+        hasAgents: nativeSummary.agents > 0 || legacySummary.agents > 0,
+        hasCommands: nativeHasCommands || legacyHasCommands,
+        hasPersonas: nativeSummary.personas > 0 || legacySummary.personas > 0,
+        complete: nativeComplete || legacyComplete,
+    };
+}
 /**
  * Deletes the deprecated `.claude/commands/sr/` directory (if present) and returns
  * the number of `.md` files that were removed. Safe to call even if the directory
@@ -934,16 +997,22 @@ class SetupManager {
             this._projectNames.set(projectId, projectName);
         this._projectTiers.set(projectId, 'full');
         this._initCheckpoints(projectId);
+        // Resolve the relocated artifact workspace before touching any provider
+        // directories. The imported source repo must remain pristine.
+        const enrichSlug = (0, artifact_registry_1.resolveArtifacts)(projectPath).entry?.slug;
+        if (enrichSlug)
+            this._projectSlugs.set(projectId, enrichSlug);
+        const enrichRoot = this._artifactRoot(projectId, projectPath);
         // Pre-create the directory structure that /specrails:enrich will write to.
         // Claude Code's Write tool does not create parent directories automatically —
         // if a target directory doesn't exist the write fails and Claude reports a
         // misleading "write permissions aren't enabled" error.  Creating the dirs
         // here ensures enrich runs transparently without any user intervention.
         try {
-            (0, fs_1.mkdirSync)((0, path_1.join)(projectPath, SPECRAILS_DIR, 'agents', 'personas'), { recursive: true });
-            (0, fs_1.mkdirSync)((0, path_1.join)(projectPath, SPECRAILS_DIR, 'commands', 'sr'), { recursive: true });
-            (0, fs_1.mkdirSync)((0, path_1.join)(projectPath, SPECRAILS_DIR, 'commands', 'specrails'), { recursive: true });
-            (0, fs_1.mkdirSync)((0, path_1.join)(projectPath, SPECRAILS_DIR, 'rules'), { recursive: true });
+            (0, fs_1.mkdirSync)((0, path_1.join)(enrichRoot, SPECRAILS_DIR, 'agents', 'personas'), { recursive: true });
+            (0, fs_1.mkdirSync)((0, path_1.join)(enrichRoot, SPECRAILS_DIR, 'commands', 'sr'), { recursive: true });
+            (0, fs_1.mkdirSync)((0, path_1.join)(enrichRoot, SPECRAILS_DIR, 'commands', 'specrails'), { recursive: true });
+            (0, fs_1.mkdirSync)((0, path_1.join)(enrichRoot, SPECRAILS_DIR, 'rules'), { recursive: true });
         }
         catch (err) {
             console.warn(`[SetupManager] Failed to pre-create enrich directories: ${err}`);
@@ -952,7 +1021,6 @@ class SetupManager {
         // The slug was allocated at addProject and mirrored into the shared registry
         // before/at install time, so resolve it from there (legacy full-flow path —
         // not exercised by the app's quick flow).
-        const enrichSlug = (0, artifact_registry_1.resolveArtifacts)(projectPath).entry?.slug;
         const configPath = (0, install_config_path_1.installConfigPath)({ slug: enrichSlug, path: projectPath });
         const hasConfig = (0, fs_1.existsSync)(configPath);
         const enrichCmd = hasConfig ? '/specrails:enrich --from-config' : '/specrails:enrich';
@@ -1034,8 +1102,10 @@ class SetupManager {
         if (!db)
             return;
         try {
+            const finishedAt = new Date().toISOString();
             const { result, estimated } = (0, result_event_1.finaliseInvocationResult)(adapter, events, {
                 fallbackModel: adapter.defaultModel(),
+                durationMs: Math.max(0, Date.parse(finishedAt) - Date.parse(startedAtIso)),
             });
             (0, ai_invocations_1.recordInvocation)(db, {
                 id: (0, crypto_1.randomUUID)(),
@@ -1045,7 +1115,7 @@ class SetupManager {
                 surface_ref_id: projectId,
                 status,
                 started_at: startedAtIso,
-                finished_at: new Date().toISOString(),
+                finished_at: finishedAt,
                 total_cost_usd_estimated: estimated,
                 ...result,
             });
@@ -1068,37 +1138,64 @@ class SetupManager {
             console.warn('[SetupManager] No AI CLI detected. Falling back to claude.');
         }
         const adapter = (0, providers_1.getAdapter)(resolvedProvider ?? 'claude');
+        const execution = (0, workspace_resolution_1.resolveProjectExecution)({
+            slug: this._projectSlugs.get(projectId),
+            path: projectPath,
+        });
         // Provider-aware prompt resolution:
         //   - claude: pass the slash command unresolved so the CLI looks up
         //     `.claude/commands/specrails/enrich.md` natively. Honours the
         //     skills-resolution priority over CLAUDE.md.
         //   - codex: no slash-command support; fold the enrich.md content into
         //     the prompt with the PROJECT context header so codex knows the cwd.
-        let effectivePrompt = opts.prompt;
-        if (!adapter.capabilities.systemPromptArg &&
+        let effectivePrompt;
+        try {
+            effectivePrompt = adapter.formatCoreCommand?.(opts.prompt, execution.cwd) ?? opts.prompt;
+        }
+        catch (error) {
+            const message = error instanceof Error ? error.message : String(error);
+            console.error(`[SetupManager] provider command resolution failed for ${projectId}: ${message}`);
+            this._stopFilesystemPoll(projectId);
+            this._broadcast({
+                type: 'setup_error',
+                projectId,
+                error: message,
+            });
+            return;
+        }
+        if (adapter.id === 'codex' &&
+            !adapter.capabilities.systemPromptArg &&
             (opts.prompt === '/specrails:enrich' || opts.prompt === '/specrails:enrich --from-config')) {
-            const enrichContent = readEnrichMdContent(projectPath);
+            const enrichContent = readEnrichMdContent(execution.cwd);
             if (enrichContent) {
                 const projectName = this._projectNames.get(projectId);
+                const cwdContext = execution.relocated
+                    ? `CWD: ${execution.cwd}\nREPOSITORY: ${execution.repoDir}`
+                    : `CWD: ${projectPath}`;
                 effectivePrompt = projectName
-                    ? `PROJECT: ${projectName}\nCWD: ${projectPath}\n\n---\n\n${enrichContent}`
+                    ? `PROJECT: ${projectName}\n${cwdContext}\n\n---\n\n${enrichContent}`
                     : enrichContent;
             }
             else {
                 console.warn(`[SetupManager] Could not read enrich.md or setup.md — falling back to literal prompt`);
             }
         }
-        const args = adapter.buildArgs(opts.action, {
+        const repoAccessArgs = execution.relocated
+            ? (0, runtime_1.buildProviderRepoAccessArgs)(adapter, [execution.repoDir])
+            : [];
+        const spawnOpts = {
             prompt: effectivePrompt,
             model: adapter.defaultModel(),
             sessionId: opts.sessionId,
-        });
+            extraArgs: repoAccessArgs,
+        };
+        const args = adapter.buildArgs(opts.action, spawnOpts);
         // No OTEL env injection here — SetupManager spawns drive the initial project
         // setup wizard, not repeatable pipeline jobs. Telemetry is scoped to
         // QueueManager pipeline runs only.
         const child = (0, cli_prompt_1.spawnAiCli)(adapter.binary, args, {
-            cwd: projectPath,
-            env: process.env,
+            cwd: execution.cwd,
+            env: (0, runtime_1.buildProviderEnv)(adapter, spawnOpts, { ...process.env, ...execution.env }),
             stdio: ['ignore', 'pipe', 'pipe'],
         });
         this._setupProcesses.set(projectId, child);
@@ -1122,65 +1219,71 @@ class SetupManager {
         // Start periodic filesystem polling for checkpoint detection
         this._startFilesystemPoll(projectId, projectPath);
         let capturedSessionId = null;
+        let providerError = null;
         const stdoutReader = (0, readline_1.createInterface)({ input: child.stdout, crlfDelay: Infinity });
         const stderrReader = (0, readline_1.createInterface)({ input: child.stderr, crlfDelay: Infinity });
         stdoutReader.on('line', (line) => {
-            const ev = adapter.parseStreamLine(line);
-            if (ev)
-                adapterEvents.push(ev);
-            if (!ev) {
+            const parsedEvents = (0, runtime_1.parseStreamEvents)(adapter, line);
+            if (parsedEvents.length === 0) {
                 // Non-parseable line — emit as raw log.
                 if (line)
                     this._broadcast({ type: 'setup_log', projectId, line, stream: 'stdout' });
                 return;
             }
-            switch (ev.kind) {
-                case 'session-started': {
-                    if (!capturedSessionId) {
-                        capturedSessionId = ev.sessionId;
-                        this._onSessionCaptured?.(projectId, ev.sessionId);
+            for (const ev of parsedEvents) {
+                adapterEvents.push(ev);
+                switch (ev.kind) {
+                    case 'session-started': {
+                        if (!capturedSessionId) {
+                            capturedSessionId = ev.sessionId;
+                            this._onSessionCaptured?.(projectId, ev.sessionId);
+                        }
+                        break;
                     }
-                    break;
-                }
-                case 'text-delta': {
-                    // Run checkpoint detection over the assistant text and surface it
-                    // both to the collapsible log viewer and the wizard chat panel.
-                    this._broadcast({ type: 'setup_log', projectId, line: ev.text, stream: 'stdout' });
-                    this._broadcast({ type: 'setup_chat', projectId, text: ev.text, role: 'assistant' });
-                    const hits = detectCheckpointFromText(ev.text);
-                    for (const hit of hits) {
-                        this._advanceCheckpoint(projectId, hit.key, hit.detail);
+                    case 'text-delta': {
+                        // Run checkpoint detection over the assistant text and surface it
+                        // both to the collapsible log viewer and the wizard chat panel.
+                        this._broadcast({ type: 'setup_log', projectId, line: ev.text, stream: 'stdout' });
+                        this._broadcast({ type: 'setup_chat', projectId, text: ev.text, role: 'assistant' });
+                        const hits = detectCheckpointFromText(ev.text);
+                        for (const hit of hits) {
+                            this._advanceCheckpoint(projectId, hit.key, hit.detail);
+                        }
+                        // Also sync filesystem (cheap mtime/exists checks, see helper).
+                        this._syncFilesystemCheckpoints(projectId, projectPath);
+                        break;
                     }
-                    // Also sync filesystem (cheap mtime/exists checks, see helper).
-                    this._syncFilesystemCheckpoints(projectId, projectPath);
-                    break;
-                }
-                case 'tool-use': {
-                    this._broadcast({ type: 'setup_log', projectId, line: `[tool] ${ev.name}`, stream: 'stdout' });
-                    // Tool inputs commonly mention the paths being written — feed the
-                    // input preview into the checkpoint detector so writes to
-                    // .claude/agents/sr-*.md or .codex/skills/sr-*/SKILL.md advance
-                    // the checkpoint state immediately.
-                    const hits = detectCheckpointFromText(ev.inputPreview);
-                    for (const hit of hits) {
-                        this._advanceCheckpoint(projectId, hit.key, hit.detail);
+                    case 'tool-use': {
+                        this._broadcast({ type: 'setup_log', projectId, line: `[tool] ${ev.name}`, stream: 'stdout' });
+                        // Tool inputs commonly mention the paths being written — feed the
+                        // input preview into the checkpoint detector so writes to
+                        // .claude/agents/sr-*.md or .codex/skills/sr-*/SKILL.md advance
+                        // the checkpoint state immediately.
+                        const hits = detectCheckpointFromText(ev.inputPreview);
+                        for (const hit of hits) {
+                            this._advanceCheckpoint(projectId, hit.key, hit.detail);
+                        }
+                        break;
                     }
-                    break;
-                }
-                case 'result': {
-                    // Claude's `result` event also carries session_id; use it as a
-                    // backstop if `session-started` wasn't observed earlier.
-                    const sid = ev.payload.session_id;
-                    if (sid && !capturedSessionId) {
-                        capturedSessionId = sid;
-                        this._onSessionCaptured?.(projectId, sid);
+                    case 'result': {
+                        // Claude's `result` event also carries session_id; use it as a
+                        // backstop if `session-started` wasn't observed earlier.
+                        const sid = ev.payload.session_id;
+                        if (sid && !capturedSessionId) {
+                            capturedSessionId = sid;
+                            this._onSessionCaptured?.(projectId, sid);
+                        }
+                        break;
                     }
-                    break;
+                    case 'error':
+                        providerError = ev.message;
+                        this._broadcast({ type: 'setup_log', projectId, line: ev.message, stream: 'stderr' });
+                        break;
+                    case 'other':
+                        // Other event types (system progress markers) — already broadcast
+                        // implicitly through the line itself if useful. Skip.
+                        break;
                 }
-                case 'other':
-                    // Other event types (system progress markers) — already broadcast
-                    // implicitly through the line itself if useful. Skip.
-                    break;
             }
         });
         stderrReader.on('line', (line) => {
@@ -1198,25 +1301,29 @@ class SetupManager {
             }
             // Record the completed setup turn's spend (success on clean exit, failed
             // otherwise). LOW-2.
-            this._recordSetupInvocation(projectId, adapter, adapterEvents, code === 0 ? 'success' : 'failed', turnStartedAt);
+            const providerReportedFailure = providerError !== null;
+            this._recordSetupInvocation(projectId, adapter, adapterEvents, code === 0 && !providerReportedFailure ? 'success' : 'failed', turnStartedAt);
             // Final filesystem sync
             this._syncFilesystemCheckpoints(projectId, projectPath);
-            if (code === 0) {
+            if (code === 0 && !providerReportedFailure) {
                 // Sync filesystem checkpoints
                 this._syncFilesystemCheckpoints(projectId, projectPath);
                 // Relocated installs write artifacts to the workspace, not the repo —
                 // resolve the has-agents/has-commands gate + summary against it.
                 const artifactRoot = this._artifactRoot(projectId, projectPath);
-                // Check if setup is truly complete — real artifacts must exist
-                const hasAgents = (0, fs_1.existsSync)((0, path_1.join)(artifactRoot, SPECRAILS_DIR, 'agents')) &&
-                    hasFiles((0, path_1.join)(artifactRoot, SPECRAILS_DIR, 'agents'), /^sr-.*\.md$/);
-                const hasCommands = (((0, fs_1.existsSync)((0, path_1.join)(artifactRoot, SPECRAILS_DIR, 'commands', 'sr')) && hasFiles((0, path_1.join)(artifactRoot, SPECRAILS_DIR, 'commands', 'sr'), /\.md$/)) ||
-                    ((0, fs_1.existsSync)((0, path_1.join)(artifactRoot, SPECRAILS_DIR, 'commands', 'specrails')) && hasFiles((0, path_1.join)(artifactRoot, SPECRAILS_DIR, 'commands', 'specrails'), /\.md$/)));
-                const isComplete = hasAgents && hasCommands;
+                // Check completion against the selected provider's native artefacts.
+                // In particular, Kimi rail roles are direct-child SKILL.md files under
+                // `.kimi-code/skills/`, not Markdown agents/commands in `.claude`.
+                const tier = this._projectTiers.get(projectId) ?? 'full';
+                const provider = this._projectProviders.get(projectId) ?? 'claude';
+                const artifactState = setupArtifactState(artifactRoot, tier, provider);
+                const isComplete = artifactState.complete;
                 if (isComplete) {
                     const legacySrRemoved = sweepLegacySrCommands(artifactRoot);
-                    const tier = this._projectTiers.get(projectId) ?? 'full';
-                    const summary = { ...computeSummary(artifactRoot, tier, this._projectProviders.get(projectId) ?? 'claude'), legacySrRemoved };
+                    const summary = {
+                        ...artifactState.summary,
+                        legacySrRemoved,
+                    };
                     this._onSetupDone?.(projectId);
                     this._broadcast({
                         type: 'setup_complete',
@@ -1240,7 +1347,7 @@ class SetupManager {
                 this._broadcast({
                     type: 'setup_error',
                     projectId,
-                    error: `${adapter.binary} enrich exited with code ${code ?? 'unknown'}`,
+                    error: providerError ?? `${adapter.binary} enrich exited with code ${code ?? 'unknown'}`,
                 });
             }
         });
@@ -1306,8 +1413,6 @@ class SetupManager {
      *  known or the project isn't relocated. */
     _artifactRoot(projectId, projectPath) {
         const slug = this._projectSlugs.get(projectId);
-        if (!slug)
-            return projectPath;
         try {
             return (0, workspace_resolution_1.resolveProjectExecution)({ slug, path: projectPath }).cwd;
         }
@@ -1319,7 +1424,7 @@ class SetupManager {
         const statuses = this._checkpoints.get(projectId);
         if (!statuses)
             return;
-        const fsChecks = checkFilesystem(this._artifactRoot(projectId, projectPath));
+        const fsChecks = checkFilesystem(this._artifactRoot(projectId, projectPath), this._projectProviders.get(projectId) ?? 'claude');
         for (const [key, exists] of Object.entries(fsChecks)) {
             if (!exists)
                 continue;
@@ -1426,7 +1531,23 @@ class SetupManager {
         catch {
             // Missing install-config — stay on claude default.
         }
-        return computeSummary(project.path, tier, provider);
+        // Relocated installs intentionally keep the imported repository pristine:
+        // provider artefacts live in the populated workspace. The immediate install
+        // completion path already uses `_artifactRoot`; a later GET after reopening
+        // must resolve the same root or Kimi (and every relocated provider) is
+        // incorrectly summarised as 0 agents / 0 commands.
+        let artifactRoot = project.path;
+        try {
+            artifactRoot = (0, workspace_resolution_1.resolveProjectExecution)({
+                slug: project.slug,
+                path: project.path,
+            }).cwd;
+        }
+        catch {
+            // Registry/workspace resolution is fail-open by contract. Legacy projects
+            // continue to read their in-repo artefacts.
+        }
+        return computeSummary(artifactRoot, tier, provider);
     }
 }
 exports.SetupManager = SetupManager;

package/server/dist/setup-prerequisites.js

@@ -42,7 +42,7 @@ function runVersionSpawn(cmd, args, timeoutMs = 5_000) {
 // budget; the `where`/`which` LOCATE probe stays fast (5s) since it's a builtin.
 const VERSION_PROBE_TIMEOUT_MS = process.platform === 'win32' ? 20_000 : 8_000;
 exports.MIN_VERSIONS = {
-    node: '18.0.0',
+    node: '20.19.0',
     npm: '9.0.0',
     git: '2.20.0',
     uv: '0.1.0',
@@ -518,6 +518,8 @@ function providerInstallUrl(id) {
             return 'https://developers.openai.com/codex';
         case 'gemini':
             return 'https://github.com/google-gemini/gemini-cli';
+        case 'kimi':
+            return 'https://www.kimi.com/code/docs/en/';
         default:
             return 'https://github.com';
     }
@@ -534,6 +536,10 @@ function providerInstallHint(id, platform) {
                     : 'Install Codex CLI from https://developers.openai.com/codex (or `pipx install codex-cli`), authenticate with `codex login`, then restart Specrails.';
         case 'gemini':
             return 'Install Gemini CLI via `npm i -g @google/gemini-cli` (or see https://github.com/google-gemini/gemini-cli), set GEMINI_API_KEY, then restart Specrails.';
+        case 'kimi':
+            return platform === 'win32'
+                ? 'Install Kimi Code from https://www.kimi.com/code/docs/en/, run `kimi login`, then restart Specrails so Windows refreshes PATH.'
+                : 'Install Kimi Code from https://www.kimi.com/code/docs/en/, run `kimi login`, then restart Specrails.';
         default:
             return `Install the ${id} CLI and restart Specrails.`;
     }