specrails-desktop 2.30.2 → 2.31.0

package/server/dist/plugins/claude-md-mutation.js

@@ -10,9 +10,10 @@ const fs_1 = __importDefault(require("fs"));
 const path_1 = __importDefault(require("path"));
 const json_mutation_1 = require("./json-mutation");
 /**
- * Plugins contribute named sections to the project's top-level instructions
- * file (`CLAUDE.md` for claude projects, `AGENTS.md` for codex projects, and
- * the adapter-declared filename for future providers) so the agent's global
+ * Plugins contribute named sections to the project's provider-native
+ * instructions path (`CLAUDE.md` for Claude, `AGENTS.md` for Codex,
+ * `.kimi-code/AGENTS.md` for Kimi, and the adapter-declared path for future
+ * providers) so the agent's global
  * context includes per-plugin usage hints (e.g. "prefer Serena tools over
  * raw Read/Grep when locating symbols"). Each plugin owns one named block
  * delimited by HTML-comment markers; multiple plugins coexist without
@@ -88,7 +89,7 @@ function getBlockContent(projectPath, pluginName, filename = 'CLAUDE.md') {
  * file does not exist, it is created with just the block. If it exists
  * without our block, the block is appended at the end (separated by a blank
  * line). The filename defaults to `CLAUDE.md` for backwards compatibility;
- * codex projects pass `AGENTS.md`.
+ * other providers pass their adapter-declared relative path.
  */
 async function upsertBlock(projectPath, pluginName, content, filename = 'CLAUDE.md') {
     const file = instructionsMdPath(projectPath, filename);

package/server/dist/plugins/drift.js

@@ -5,8 +5,8 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.detectMcpDrift = detectMcpDrift;
 const fs_1 = __importDefault(require("fs"));
-const paths_1 = require("./paths");
 const claude_md_mutation_1 = require("./claude-md-mutation");
+const providers_1 = require("../providers");
 /**
  * Returns true when the project's `.mcp.json` entry for any of the plugin's
  * owned mcpServers no longer matches the bundled manifest. Used to surface
@@ -15,42 +15,45 @@ const claude_md_mutation_1 = require("./claude-md-mutation");
  *
  * Only inspects entries the plugin owns; user-authored entries are ignored.
  */
-function detectMcpDrift(projectPath, plugin) {
+function detectMcpDrift(projectPath, plugin, providerId = 'claude') {
     const ownedKeys = plugin.manifest.owns.mcpServers ?? [];
-    if (ownedKeys.length === 0)
-        return false;
+    const adapter = (0, providers_1.hasAdapter)(providerId) ? (0, providers_1.getAdapter)(providerId) : (0, providers_1.getAdapter)('claude');
     // The plugin tells us what its current canonical entry should look like
     // via `previewInstall` is too heavy; we instead introspect via a marker:
     // plugins that ship their canonical entry expose it via `expectedMcpEntry`.
     // We accept Plugins without it (drift detection silently disabled).
-    const expected = plugin
-        .expectedMcpEntry?.();
-    if (!expected)
-        return false;
-    const file = (0, paths_1.mcpJsonPath)(projectPath);
-    if (!fs_1.default.existsSync(file))
-        return false;
-    let parsed;
-    try {
-        parsed = JSON.parse(fs_1.default.readFileSync(file, 'utf8'));
+    const expected = plugin.expectedMcpEntry?.();
+    // CLI-managed registries (Codex) have no project JSON to compare. Their
+    // registration is checked by the plugin's verify hook instead.
+    if (expected &&
+        ownedKeys.length > 0 &&
+        adapter.mcpRegistration === 'project-json' &&
+        adapter.projectMcpPath) {
+        const file = adapter.projectMcpPath(projectPath);
+        if (fs_1.default.existsSync(file)) {
+            let parsed;
+            try {
+                parsed = JSON.parse(fs_1.default.readFileSync(file, 'utf8'));
+            }
+            catch {
+                return false;
+            }
+            const servers = parsed.mcpServers ?? {};
+            for (const key of ownedKeys) {
+                const current = servers[key];
+                if (current === undefined)
+                    continue;
+                if (JSON.stringify(current) !== JSON.stringify(expected))
+                    return true;
+            }
+        }
     }
-    catch {
-        return false;
-    }
-    const servers = parsed.mcpServers ?? {};
-    for (const key of ownedKeys) {
-        const current = servers[key];
-        if (current === undefined)
-            continue;
-        if (JSON.stringify(current) !== JSON.stringify(expected))
-            return true;
-    }
-    // Shared-file contributor drift. Today only CLAUDE.md is contributed to;
-    // we hardcode the comparison here. If/when more shared-file contributors
-    // land we move this to a `compareDrift` hook on the contributor type.
+    // Provider-instructions contributor drift. The adapter path may be nested
+    // (for example Kimi's `.kimi-code/AGENTS.md`). If/when more shared-file
+    // contributors land we move this to a `compareDrift` hook.
     if (plugin.manifest.claudeMdInstructions) {
         const expectedMd = plugin.manifest.claudeMdInstructions.trim();
-        const actualMd = ((0, claude_md_mutation_1.getBlockContent)(projectPath, plugin.manifest.name) ?? '').trim();
+        const actualMd = ((0, claude_md_mutation_1.getBlockContent)(projectPath, plugin.manifest.name, adapter.instructionsFilename) ?? '').trim();
         if (actualMd !== expectedMd)
             return true;
     }

package/server/dist/plugins/rail-integration.js

@@ -20,7 +20,7 @@ const json_mutation_1 = require("./json-mutation");
  * actually changed since the last check (no churn on the steady-state path).
  * Otherwise this performs no project-file mutation.
  */
-async function resolvePluginsForSpawn(projectPath, projectId, _jobId) {
+async function resolvePluginsForSpawn(projectPath, projectId, _jobId, providerId = 'claude', legacyProviderId = providerId, slug) {
     const mgr = (0, manager_1.getPluginManager)();
     const state = mgr.getProjectState(projectPath);
     const active = [];
@@ -28,13 +28,24 @@ async function resolvePluginsForSpawn(projectPath, projectId, _jobId) {
     // Run verify in parallel for all installed plugins. Each verify is
     // already timeout-wrapped inside PluginManager.
     const checks = await Promise.all(Object.entries(state.plugins).map(async ([name, entry]) => {
+        const scopedProviderState = entry.providers?.[providerId];
+        const providerInstalled = entry.providers
+            ? scopedProviderState !== undefined
+            : providerId === legacyProviderId;
+        // A plugin installed only for another provider must not appear in this
+        // rail's snapshot. Explicitly deactivated installs are intentionally
+        // absent rather than misreported as degraded.
+        if (!providerInstalled || scopedProviderState?.active === false)
+            return null;
         // Skip orphans — they are not in the registry, so verify cannot run.
         if (!mgr.registry.byName.has(name))
             return { name, version: entry.version, result: { ok: false, reason: 'orphan', checkedAt: new Date().toISOString() } };
-        const result = await mgr.verify(projectPath, projectId, name);
+        const result = await mgr.verify(projectPath, projectId, name, undefined, providerId, legacyProviderId, slug);
         return { name, version: entry.version, result };
     }));
     for (const c of checks) {
+        if (!c)
+            continue;
         if (c.result.ok)
             active.push({ name: c.name, version: c.version });
         else

package/server/dist/plugins/serena/install.js

@@ -50,15 +50,20 @@ async function installSerena(ctx) {
         }
         return;
     }
-    // Claude path: surgical merge of project's `.mcp.json` + custom agent
-    // fragment. Unchanged from the pre-§14 behaviour.
-    ctx.log('Adding mcpServers.serena to .mcp.json');
-    await plugin_manager_1.PluginManager.mergeMcpServers(ctx.projectPath, { serena: manifest_1.SERENA_MCP_ENTRY });
-    ctx.log(`Writing ${CLAUDE_FRAGMENT_REL}`);
-    const fragmentDest = path_1.default.join(ctx.projectPath, CLAUDE_FRAGMENT_REL);
-    fs_1.default.mkdirSync(path_1.default.dirname(fragmentDest), { recursive: true });
-    fs_1.default.writeFileSync(fragmentDest, instructions_content_1.SERENA_INSTRUCTIONS_MD, 'utf8');
-    ctx.recordInstalledFile(CLAUDE_FRAGMENT_REL);
+    const adapter = (0, providers_1.getAdapter)(ctx.providerId ?? 'claude');
+    const mcpFile = adapter.projectMcpPath?.(ctx.projectPath);
+    ctx.log(`Adding mcpServers.serena to ${mcpFile ?? '.mcp.json'}`);
+    await plugin_manager_1.PluginManager.mergeMcpServers(ctx.projectPath, { serena: manifest_1.SERENA_MCP_ENTRY }, adapter.id);
+    // Claude additionally receives its historical custom-agent fragment. Kimi
+    // consumes the provider-aware AGENTS.md contributor and native MCP config;
+    // its built-in subagents are not modelled as SpecRails roles.
+    if (adapter.id === 'claude') {
+        ctx.log(`Writing ${CLAUDE_FRAGMENT_REL}`);
+        const fragmentDest = path_1.default.join(ctx.projectPath, CLAUDE_FRAGMENT_REL);
+        fs_1.default.mkdirSync(path_1.default.dirname(fragmentDest), { recursive: true });
+        fs_1.default.writeFileSync(fragmentDest, instructions_content_1.SERENA_INSTRUCTIONS_MD, 'utf8');
+        ctx.recordInstalledFile(CLAUDE_FRAGMENT_REL);
+    }
 }
 async function uninstallSerena(ctx) {
     if (isCodex(ctx)) {
@@ -79,14 +84,17 @@ async function uninstallSerena(ctx) {
         }
         return;
     }
-    ctx.log('Removing mcpServers.serena from .mcp.json');
-    await plugin_manager_1.PluginManager.removeMcpServers(ctx.projectPath, ['serena']);
-    const fragmentDest = path_1.default.join(ctx.projectPath, CLAUDE_FRAGMENT_REL);
-    if (fs_1.default.existsSync(fragmentDest)) {
-        ctx.log(`Removing ${CLAUDE_FRAGMENT_REL}`);
-        try {
-            fs_1.default.unlinkSync(fragmentDest);
+    const adapter = (0, providers_1.getAdapter)(ctx.providerId ?? 'claude');
+    ctx.log(`Removing mcpServers.serena from ${adapter.projectMcpPath?.(ctx.projectPath) ?? '.mcp.json'}`);
+    await plugin_manager_1.PluginManager.removeMcpServers(ctx.projectPath, ['serena'], adapter.id);
+    if (adapter.id === 'claude') {
+        const fragmentDest = path_1.default.join(ctx.projectPath, CLAUDE_FRAGMENT_REL);
+        if (fs_1.default.existsSync(fragmentDest)) {
+            ctx.log(`Removing ${CLAUDE_FRAGMENT_REL}`);
+            try {
+                fs_1.default.unlinkSync(fragmentDest);
+            }
+            catch { /* best-effort */ }
         }
-        catch { /* best-effort */ }
     }
 }

package/server/dist/plugins/serena/manifest.js

@@ -60,13 +60,30 @@ exports.serenaManifest = {
                 ],
             },
         },
+        kimi: {
+            mcpEntry: {
+                command: 'uvx',
+                args: [
+                    '--from',
+                    'git+https://github.com/oraios/serena',
+                    'serena',
+                    'start-mcp-server',
+                    '--context',
+                    'ide-assistant',
+                    '--project',
+                    '.',
+                    '--enable-web-dashboard',
+                    'false',
+                ],
+            },
+        },
     },
     // Format: structured "when / tools / why / fallback" block. Lets generic
     // agent prompts (specrails-core sr-* "Tool Selection — Honor Project-
     // Documented MCP Tools" section) match the right entry to the task.
     //
-    // Tool names below are verified against Serena's actual exposed tools
-    // (visible in Claude session init's tools list). Keep this list aligned
+    // Tool names below are verified against Serena's actual exposed MCP tools.
+    // Keep this list aligned
     // with upstream — wrong names cause agents to mistrust the whole block.
     claudeMdInstructions: `## Plugin: serena (semantic code navigation)
 
@@ -85,11 +102,11 @@ exports.serenaManifest = {
 - \`replace_content\` — surgical text replace inside a single file when symbol-aware tools don't fit.
 - \`get_diagnostics_for_file\` — language-server diagnostics for a file.
 
-**Why prefer over built-ins**: Serena returns only the relevant symbol body, not whole files. Empirically cuts input tokens 40–60% on real workloads compared to \`Read\` / \`Grep\`.
+**Why prefer over built-ins**: Serena returns only the relevant symbol body, not whole files. Empirically cuts input tokens 40–60% on real workloads compared with whole-file reads or broad text search.
 
-**Fallback to \`Read\` / \`Grep\` for**: binary files, free-form prose, structured data without symbols (logs, JSON without schema, plain Markdown).
+**Fallback to the provider's built-in file/search tools for**: binary files, free-form prose, structured data without symbols (logs, JSON without schema, plain Markdown).
 
-**Subagents**: MCP access is inherited from the parent Claude session; pass these tool names through when delegating via the \`Task\` tool.`,
+**Delegated agents**: when the active provider supports delegation, tell the delegated role to use these Serena tools. MCP inheritance is provider-owned, so verify availability in the delegated context and fall back to built-in file/search tools when needed.`,
 };
 exports.SERENA_MCP_ENTRY = {
     command: 'uvx',

package/server/dist/plugins/serena/verify.js

@@ -1,24 +1,51 @@
 "use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.verifySerena = verifySerena;
 const child_process_1 = require("child_process");
+const fs_1 = __importDefault(require("fs"));
+const path_1 = __importDefault(require("path"));
 const win_spawn_1 = require("../../util/win-spawn");
+const providers_1 = require("../../providers");
+const codex_mcp_1 = require("../codex-mcp");
 const TIMEOUT_MS = 1800;
-/**
- * Verify Serena availability. We only probe `uv --version` here — proxy for
- * "uvx will be able to launch serena-mcp-server when claude actually calls it".
- *
- * We intentionally do NOT shell out to `uvx --from git+... serena-mcp-server`:
- * that would download the git repo + dependencies on every verify, which is
- * slow (multi-second) and would fail offline. The Claude CLI itself triggers
- * the lazy install through uvx's own cache, so as long as `uv` is on PATH and
- * executable, install + spawn-time verify can both pass quickly.
- *
- * Cross-platform spawn: on Windows we set `shell: true` so PATH resolution
- * picks up `uv.exe` (or, if astral installed via winget, the .cmd shim) the
- * same way it does in `setup-prerequisites.ts`.
- */
-async function verifySerena() {
+function registrationProblem(ctx) {
+    // Direct callers from the pre-provider API used verifySerena() as a cheap uv
+    // probe. Keep that contract; PluginManager always supplies provider context
+    // and therefore receives the stronger registration check below.
+    if (!ctx?.providerId)
+        return null;
+    if (!(0, providers_1.hasAdapter)(ctx.providerId))
+        return `unknown-provider:${ctx.providerId}`;
+    const adapter = (0, providers_1.getAdapter)(ctx.providerId);
+    if (adapter.mcpRegistration === 'cli-add') {
+        const slug = ctx.slug?.trim() || path_1.default.basename(ctx.projectPath);
+        const listing = (0, codex_mcp_1.codexMcpList)(slug);
+        if (!listing.ok)
+            return 'mcp-registry-unavailable';
+        return listing.servers.includes('serena') ? null : 'mcp-registration-missing';
+    }
+    const file = adapter.projectMcpPath?.(ctx.projectPath);
+    if (!file || !fs_1.default.existsSync(file))
+        return 'mcp-registration-missing';
+    try {
+        const parsed = JSON.parse(fs_1.default.readFileSync(file, 'utf8'));
+        if (!parsed.mcpServers ||
+            typeof parsed.mcpServers !== 'object' ||
+            Array.isArray(parsed.mcpServers)) {
+            return 'mcp-registration-invalid';
+        }
+        return Object.prototype.hasOwnProperty.call(parsed.mcpServers, 'serena')
+            ? null
+            : 'mcp-registration-missing';
+    }
+    catch {
+        return 'mcp-registration-invalid';
+    }
+}
+async function verifySerena(ctx) {
     const checkedAt = new Date().toISOString();
     const isWin = process.platform === 'win32';
     return new Promise((resolve) => {
@@ -68,7 +95,10 @@ async function verifySerena() {
             settled = true;
             clearTimeout(timer);
             if (code === 0) {
-                resolve({ ok: true, reason: undefined, checkedAt });
+                const reason = registrationProblem(ctx);
+                resolve(reason
+                    ? { ok: false, reason, checkedAt }
+                    : { ok: true, reason: undefined, checkedAt });
             }
             else {
                 resolve({

package/server/dist/plugins-router.js

@@ -8,6 +8,7 @@ const prereq_installer_1 = require("./plugins/prereq-installer");
 const path_resolver_1 = require("./path-resolver");
 const claude_approval_1 = require("./plugins/claude-approval");
 const workspace_resolution_1 = require("./workspace-resolution");
+const providers_1 = require("./providers");
 /**
  * Relocate-artifacts gate: the dir the PluginManager mutates (`.mcp.json` +
  * `.specrails/plugins/` state + `.claude/agents/custom-*.md`). Relocated ⇒ the
@@ -41,6 +42,32 @@ function handleError(res, err) {
     const message = err instanceof Error ? err.message : 'unknown error';
     res.status(500).json({ error: message });
 }
+/** Resolve the explicit provider projection for one plugin operation. Legacy
+ * clients that omit it keep targeting the project's primary provider. */
+function requestedProvider(req, res, project) {
+    const bodyProvider = req.body?.provider;
+    const queryProvider = req.query.provider;
+    const raw = bodyProvider ?? queryProvider;
+    if (raw !== undefined &&
+        (typeof raw !== 'string' || raw.trim().length === 0)) {
+        res.status(400).json({ error: 'provider must be a non-empty string' });
+        return null;
+    }
+    const provider = typeof raw === 'string' ? raw.trim() : project.provider;
+    const configured = project.providers?.length > 0 ? project.providers : [project.provider];
+    if (!configured.includes(provider)) {
+        res.status(400).json({
+            error: `provider '${provider}' is not configured for this project`,
+            providers: configured,
+        });
+        return null;
+    }
+    if (!(0, providers_1.hasAdapter)(provider)) {
+        res.status(400).json({ error: `unknown provider '${provider}'` });
+        return null;
+    }
+    return provider;
+}
 function createPluginsRouter() {
     const router = (0, express_1.Router)({ mergeParams: true });
     function ctx(req) {
@@ -57,7 +84,10 @@ function createPluginsRouter() {
     router.get('/', async (req, res) => {
         try {
             const { project } = ctx(req);
-            const list = await (0, manager_1.getPluginManager)().listAvailable(pluginRoot(project), project.provider);
+            const provider = requestedProvider(req, res, project);
+            if (!provider)
+                return;
+            const list = await (0, manager_1.getPluginManager)().listAvailable(pluginRoot(project), provider, project.provider);
             res.json({ plugins: list });
         }
         catch (err) {
@@ -68,7 +98,10 @@ function createPluginsRouter() {
     router.get('/:name/preview-install', async (req, res) => {
         try {
             const { project } = ctx(req);
-            const result = await (0, manager_1.getPluginManager)().previewInstall(pluginRoot(project), project.id, req.params.name);
+            const provider = requestedProvider(req, res, project);
+            if (!provider)
+                return;
+            const result = await (0, manager_1.getPluginManager)().previewInstall(pluginRoot(project), project.id, req.params.name, provider, project.slug);
             res.json(result);
         }
         catch (err) {
@@ -79,7 +112,10 @@ function createPluginsRouter() {
     router.post('/:name/install', async (req, res) => {
         try {
             const { project, broadcast } = ctx(req);
-            await (0, manager_1.getPluginManager)().install(pluginRoot(project), project.id, req.params.name, broadcast, project.provider, project.slug);
+            const provider = requestedProvider(req, res, project);
+            if (!provider)
+                return;
+            await (0, manager_1.getPluginManager)().install(pluginRoot(project), project.id, req.params.name, broadcast, provider, project.slug, project.provider);
             res.status(200).json({ ok: true });
         }
         catch (err) {
@@ -90,7 +126,10 @@ function createPluginsRouter() {
     router.delete('/:name', async (req, res) => {
         try {
             const { project, broadcast } = ctx(req);
-            await (0, manager_1.getPluginManager)().uninstall(pluginRoot(project), project.id, req.params.name, broadcast, project.provider, project.slug);
+            const provider = requestedProvider(req, res, project);
+            if (!provider)
+                return;
+            await (0, manager_1.getPluginManager)().uninstall(pluginRoot(project), project.id, req.params.name, broadcast, provider, project.slug, project.provider);
             res.status(200).json({ ok: true });
         }
         catch (err) {
@@ -182,7 +221,10 @@ function createPluginsRouter() {
     router.post('/:name/activate', async (req, res) => {
         try {
             const { project, broadcast } = ctx(req);
-            await (0, manager_1.getPluginManager)().setActive(pluginRoot(project), project.id, req.params.name, true, broadcast, project.provider);
+            const provider = requestedProvider(req, res, project);
+            if (!provider)
+                return;
+            await (0, manager_1.getPluginManager)().setActive(pluginRoot(project), project.id, req.params.name, true, broadcast, provider, project.slug, project.provider);
             res.json({ ok: true });
         }
         catch (err) {
@@ -193,7 +235,10 @@ function createPluginsRouter() {
     router.post('/:name/deactivate', async (req, res) => {
         try {
             const { project, broadcast } = ctx(req);
-            await (0, manager_1.getPluginManager)().setActive(pluginRoot(project), project.id, req.params.name, false, broadcast, project.provider);
+            const provider = requestedProvider(req, res, project);
+            if (!provider)
+                return;
+            await (0, manager_1.getPluginManager)().setActive(pluginRoot(project), project.id, req.params.name, false, broadcast, provider, project.slug, project.provider);
             res.json({ ok: true });
         }
         catch (err) {
@@ -204,7 +249,10 @@ function createPluginsRouter() {
     router.post('/:name/update', async (req, res) => {
         try {
             const { project, broadcast } = ctx(req);
-            await (0, manager_1.getPluginManager)().updateMcpEntry(pluginRoot(project), project.id, req.params.name, broadcast, project.provider);
+            const provider = requestedProvider(req, res, project);
+            if (!provider)
+                return;
+            await (0, manager_1.getPluginManager)().updateMcpEntry(pluginRoot(project), project.id, req.params.name, broadcast, provider, project.slug, project.provider);
             res.json({ ok: true });
         }
         catch (err) {
@@ -215,7 +263,10 @@ function createPluginsRouter() {
     router.get('/:name/health', async (req, res) => {
         try {
             const { project, broadcast } = ctx(req);
-            const result = await (0, manager_1.getPluginManager)().verify(pluginRoot(project), project.id, req.params.name, broadcast);
+            const provider = requestedProvider(req, res, project);
+            if (!provider)
+                return;
+            const result = await (0, manager_1.getPluginManager)().verify(pluginRoot(project), project.id, req.params.name, broadcast, provider, project.provider, project.slug);
             res.json(result);
         }
         catch (err) {