specrails-desktop 2.57.0 → 2.58.0

package/server/dist/desktop-db.js

@@ -647,6 +647,34 @@ function applyDesktopMigrations(db) {
             if (!cols.includes('intent'))
                 db.exec(`ALTER TABLE agent_messages ADD COLUMN intent TEXT;`);
         },
+        // 30: preserve the original legacy loop graph before its first Core conversion.
+        // Column guards also support the existing migration replay fixtures.
+        () => {
+            const columns = db.prepare('PRAGMA table_info(loops)').all().map(column => column.name);
+            if (!columns.includes('graph_legacy'))
+                db.exec('ALTER TABLE loops ADD COLUMN graph_legacy TEXT');
+            if (!columns.includes('graph_legacy_saved_at'))
+                db.exec('ALTER TABLE loops ADD COLUMN graph_legacy_saved_at TEXT');
+        },
+        // 31: editable built-in loops. `builtin_id` marks the row that IS a built-in
+        // (its canonical factory id), `builtin_default_hash` records the default
+        // content last seeded (an unedited row may be refreshed when Core changes)
+        // and `published_graph` snapshots the last published graph so rails keep
+        // launching it while the loop is edited. Existing published loops are
+        // backfilled with their current graph. Column-guarded for replay fixtures.
+        () => {
+            const columns = db.prepare('PRAGMA table_info(loops)').all().map(column => column.name);
+            if (!columns.includes('builtin_id'))
+                db.exec('ALTER TABLE loops ADD COLUMN builtin_id TEXT');
+            if (!columns.includes('builtin_default_hash'))
+                db.exec('ALTER TABLE loops ADD COLUMN builtin_default_hash TEXT');
+            if (!columns.includes('published_graph'))
+                db.exec('ALTER TABLE loops ADD COLUMN published_graph TEXT');
+            db.exec(`
+        CREATE UNIQUE INDEX IF NOT EXISTS idx_loops_builtin_id ON loops(builtin_id) WHERE builtin_id IS NOT NULL;
+        UPDATE loops SET published_graph = graph WHERE status = 'published' AND published_graph IS NULL;
+      `);
+        },
     ];
     (0, sqlite_migrations_1.applyNumberedMigrations)(db, migrations);
 }

package/server/dist/desktop-router.js

@@ -325,8 +325,10 @@ function createDesktopRouter(registry, broadcast) {
     // routes live on the global `/api` router (loops are NOT project-scoped). The
     // A loop is "running" iff any project's per-project DB has an in-flight
     // loop_run for it — so edit/unpublish/delete are blocked (409) while it runs.
+    // Built-ins are seeded as editable rows here (startup) and on `GET /loops`.
     (0, loops_router_1.registerLoopsRoutes)(router, {
         db: registry.desktopDb,
+        builtinLoops: {},
         isLoopRunning: (loopId) => registry.listContexts().some((c) => (0, loop_runs_store_1.countRunningForLoop)(c.db, loopId) > 0),
     });
     // Lazy per-provider workspace assembly (provider-auto-detection spec): when

package/server/dist/mcp/guide.js

@@ -74,6 +74,8 @@ Codex, Gemini, Kimi Code) to implement specs.
   Do not patch statuses to simulate acceptance or repair a failed delivery.
 - **Loop**: an APP-LEVEL saved workflow graph (not project-scoped). Author with
   \`specrails_loops\`; RUN it with \`specrails_rails(launch, mode:'loop', loopId)\`.
+  Built-ins (\`factory:*\` ids) are editable in place: update + publish changes the
+  built-in everywhere it is used; restore_builtin resets it.
 - **Profile**: per-project, provider-scoped agent configuration (agents, models,
   routing). Explicit named profiles are validated against the selected provider;
   do not silently discard a user's profile when choosing an engine.
@@ -189,9 +191,10 @@ heading inside the description); \`labels\`; \`priority\`. Spec content is Engli
 - Configure: \`set_tickets\` (replaces the assigned set), \`set_profile\` (null =
   legacy), \`set_engine\` (null = project primary), \`set_name\`.
 - Launch modes:
-  - \`implement\`: one pipeline job (Architect → Developer → Reviewer → Ship)
-    over the rail's tickets.
-  - \`batch-implement\`: dependency-aware waves across many tickets.
+  - \`implement\`: one aggregate pipeline job (Architect → Developer → Reviewer
+    → Ship) over all the rail's tickets. Put at most 3 specs on one rail; split
+    larger sets across several rails (they run in parallel) or sequential
+    launches. There is no separate batch mode.
   - Freestyle: sends a free-form autonomous prompt to a capable provider — one
     job per ticket; Claude and Kimi; \`model\` picker. Claude additionally
     supports optional \`interactive\` in-job chat (settle with

package/server/dist/mcp/tools/jobs.js

@@ -41,6 +41,7 @@ function jobsTools() {
                 'runtime_runs (read the programmatic runtime state of jobId — or every run when omitted — status, nextStep, canResume, recoverableSteps, pendingApproval, pendingQuestion; THIS is how you learn why a run failed and what recovery it offers), ' +
                 'runtime_diagnose (read-only recovery assessment for jobId: original scope, completed steps, repeated failures and verification invalidation reasons; use before recommending retry or relaunch), ' +
                 'runtime_evidence (read the durable runtime evidence of jobId; page with evidenceId/section/sourceId/cursor/limit), ' +
+                'runtime_steer (write — queue steeringText in an engine v2 run with a stable requestId; acceptance is not consumption; inspect runtime_runs steering receipts), ' +
                 'runtime_resume (ai-spawn — continue a resumable run: optional approve/recover/invalidate step-id lists + answer for a pending question; act ONLY after the user confirmed on the card), ' +
                 'runtime_recover (ai-spawn — shorthand for runtime_resume with recover:[stepIds] over the recoverable steps), ' +
                 'runtime_approve (write — shorthand for runtime_resume with approve:[stepId] for a pending approval), ' +
@@ -53,7 +54,7 @@ function jobsTools() {
                     return 'destructive';
                 if (['spawn', 'interactive_turn', 'runtime_resume', 'runtime_recover'].includes(action))
                     return 'ai-spawn';
-                if (['pause', 'resume', 'reorder', 'priority', 'finalize', 'runtime_approve', 'runtime_settle', 'runtime_dismiss'].includes(action))
+                if (['pause', 'resume', 'reorder', 'priority', 'finalize', 'runtime_approve', 'runtime_settle', 'runtime_dismiss', 'runtime_steer'].includes(action))
                     return 'write';
                 return 'read';
             },
@@ -89,6 +90,7 @@ function jobsTools() {
                     'runtime_runs',
                     'runtime_diagnose',
                     'runtime_evidence',
+                    'runtime_steer',
                     'runtime_resume',
                     'runtime_recover',
                     'runtime_approve',
@@ -148,6 +150,8 @@ function jobsTools() {
                 provider: zod_1.z.string().optional().describe('Provider to resolve the default spec model for (default_spec_model)'),
                 // ── runtime_* (mission-rail-cards: the agent's eyes + hands on a failed run) ──
                 railIndex: zod_1.z.number().int().nonnegative().optional().describe('runtime_runs: only the latest continuation of this rail (0-based)'),
+                steeringText: zod_1.z.string().min(1).max(20_000).optional().describe('runtime_steer: operator instruction consumed at the next eligible AI attempt'),
+                requestId: zod_1.z.string().regex(/^[A-Za-z0-9][A-Za-z0-9._:-]{0,127}$/).optional().describe('runtime_steer: stable idempotency key; reuse only with identical text'),
                 approve: zod_1.z.array(zod_1.z.string()).optional().describe('runtime_resume/runtime_approve: step ids to approve'),
                 recover: zod_1.z.array(zod_1.z.string()).optional().describe('runtime_resume/runtime_recover: step ids to recover (default for runtime_recover: every recoverableStep)'),
                 invalidate: zod_1.z.array(zod_1.z.string()).optional().describe('runtime_resume: step ids whose results must be discarded before continuing'),
@@ -374,6 +378,15 @@ function jobsTools() {
                                 query.set(key, String(args[key]));
                         return (0, types_1.apiCall)(ctx, 'GET', `${base}/agent-runtime/runs/${encodeURIComponent(id)}/evidence${query.size ? '?' + query.toString() : ''}`);
                     }
+                    case 'runtime_steer': {
+                        const id = args.jobId;
+                        if (!id)
+                            throw new Error('runtime_steer requires a "jobId".');
+                        if (typeof args.steeringText !== 'string' || !args.steeringText.trim() || args.steeringText.length > 20_000 || typeof args.requestId !== 'string' || !/^[A-Za-z0-9][A-Za-z0-9._:-]{0,127}$/.test(args.requestId))
+                            throw new Error('runtime_steer requires steeringText and a stable requestId.');
+                        const result = await (0, types_1.apiCall)(ctx, 'POST', `${base}/agent-runtime/runs/${encodeURIComponent(id)}/steer`, { text: args.steeringText, requestId: args.requestId });
+                        return { ...result, hint: 'Accepted into the durable inbox; consumption happens at the next eligible AI attempt. Inspect runtime_runs steering receipts.' };
+                    }
                     case 'runtime_resume':
                     case 'runtime_recover':
                     case 'runtime_approve': {

package/server/dist/mcp/tools/loops.js

@@ -15,7 +15,7 @@ const types_1 = require("./types");
  *   - read:        list, get, templates, factory, commands, constants_list,
  *                  preview, run_get
  *   - write:       constant_create, constant_update, import, create, from_template,
- *                  fork, update, publish, unpublish, duplicate
+ *                  fork, update, publish, unpublish, duplicate, restore_builtin
  *   - ai-spawn:    run (spawns an AI CLI, incurs cost, async 202)
  *   - destructive: constant_delete, delete
  */
@@ -32,7 +32,8 @@ function loopsTools() {
                 'list (all saved loops), ' +
                 'get (one loop by id), ' +
                 'templates (built-in template catalog with graphs), ' +
-                'factory (built-in locked factory loops), ' +
+                'factory (the built-in loops — factory:implement, factory:freestyle, factory:sdd-quick-openspec — with the graph a rail launch runs; ' +
+                'built-ins are ordinary editable loops whose loopId IS the factory id: change them with update + publish and the change applies to every rail, chat launch and companion that uses that built-in), ' +
                 'commands (magic-command catalog for the builder palette), ' +
                 'constants_list (global cross-loop constants), ' +
                 'constant_create (write), constant_update (write), constant_delete (destructive), ' +
@@ -40,11 +41,12 @@ function loopsTools() {
                 'import (import loops from an export envelope), ' +
                 'create (new Draft loop), ' +
                 'from_template (instantiate from a template), ' +
-                'fork (fork a factory loop into an editable Draft), ' +
-                'update (name/description/graph — reverts to Draft, 409 while running), ' +
-                'publish (graph-validated), unpublish (409 while running), ' +
+                'fork (same as duplicate for a built-in: copies it into a separate Draft; kept for back-compat), ' +
+                'update (name/description/graph — reverts to Draft; 409 while running except for built-ins, whose rails keep using the last published version until you publish), ' +
+                'publish (graph-validated), unpublish (409 while running; 409 builtin_loop for built-ins), ' +
                 'duplicate (copy a loop), ' +
-                'delete (destructive — 409 while running), ' +
+                'restore_builtin (reset a built-in to its original default, Published — 409 while running), ' +
+                'delete (destructive — 409 while running; built-ins cannot be deleted: 409 builtin_loop, use restore_builtin), ' +
                 'run (ai-spawn — standalone ticket-less run of a Published loop against a project: spawns an AI CLI, incurs token cost, returns 202 with loopRunId), ' +
                 'run_get (read one loop run\'s live/terminal state — the post-timeout recovery read).',
             hintTier: 'read',
@@ -65,6 +67,7 @@ function loopsTools() {
                     'publish',
                     'unpublish',
                     'duplicate',
+                    'restore_builtin',
                 ].includes(action)) {
                     return 'write';
                 }
@@ -91,6 +94,7 @@ function loopsTools() {
                     'publish',
                     'unpublish',
                     'duplicate',
+                    'restore_builtin',
                     'delete',
                     'run',
                     'run_get',
@@ -100,12 +104,12 @@ function loopsTools() {
                     .string()
                     .optional()
                     .describe('Project id (run / run_get only — defaults to the active project; ignored by every other action)'),
-                loopId: zod_1.z.string().optional().describe('Loop id (for get / update / publish / unpublish / duplicate / delete / run — must be Published for run)'),
+                loopId: zod_1.z.string().optional().describe('Loop id (for get / update / publish / unpublish / duplicate / restore_builtin / delete / run — must be Published for run). Built-ins use their factory id, e.g. factory:implement'),
                 repositoryIds: zod_1.z.array(zod_1.z.string().min(1)).min(1).max(50).optional().describe('run only: repository memberships to include in one coordinated standalone execution; omission retains primary-only behavior.'),
                 loopRunId: zod_1.z.string().optional().describe('Loop run id (for run_get — returned by run)'),
                 constantId: zod_1.z.string().optional().describe('Constant id (for constant_update / constant_delete)'),
                 templateId: zod_1.z.string().optional().describe('Template id (for from_template)'),
-                factoryId: zod_1.z.string().optional().describe('Factory loop id (for fork)'),
+                factoryId: zod_1.z.string().optional().describe('Built-in (factory) loop id (for fork, e.g. factory:implement)'),
                 name: zod_1.z
                     .string()
                     .optional()
@@ -246,6 +250,11 @@ function loopsTools() {
                             ...(args.name !== undefined ? { name: args.name } : {}),
                         });
                     }
+                    case 'restore_builtin': {
+                        if (!loopId)
+                            throw new Error('restore_builtin requires a "loopId" (a built-in id such as factory:implement).');
+                        return (0, types_1.apiCall)(ctx, 'POST', `/loops/${encodeURIComponent(loopId)}/restore-builtin`);
+                    }
                     // ── Standalone runs (project-scoped) ────────────────────────────────
                     case 'run': {
                         if (!loopId)

package/server/dist/mcp/tools/rails.js

@@ -47,7 +47,7 @@ function railsTools() {
                 'stop (destructive — kills all active jobs and loop runs for the rail). ' +
                 'For on_review tickets with an already-open GitHub PR (including a published pr_ready delivery), launch automatically tries to continue that PR head branch; Jira-linked in_progress tickets can do the same when the PR match is explicit; fresh tickets still start from the project integration branch. ' +
                 'When launched from the in-app agent chat without an explicit aiEngine, the engine defaults to your conversation\'s provider. On that same provider, an omitted model inherits the conversation model; an explicit model wins (pass aiEngine to override; launch_all always uses each rail\'s stored engine). ' +
-                'For Implement and Batch Implement, the selected launch provider applies to ALL runtime roles (architect, developer, reviewer); an explicit launch model/effort also applies to each role. Resuming an existing execution keeps its original frozen role configuration; start a new launch to change providers. ' +
+                'For Implement, the selected launch provider applies to ALL runtime roles (architect, developer, reviewer); an explicit launch model/effort also applies to each role. Resuming an existing execution keeps its original frozen role configuration; start a new launch to change providers. ' +
                 'User-facing naming: call the free-form autonomous mode "Freestyle"; use "freestyle" as the canonical API enum value for that same capability. ' +
                 'For small OpenSpec-governed work, recommend "SDD Quick (OpenSpec)" and launch with mode "loop" plus loopId "factory:sdd-quick-openspec"; keep Freestyle for ticket-local implementation-only work. ' +
                 'NAMING: railIndex is the 0-BASED internal identity; the dashboard shows rails 1-based ("Rail N" = railIndex N-1). When talking to the user, ALWAYS say "Rail <railIndex + 1>" (or the rail\'s custom name) — results include railLabel with the correct user-facing label.',
@@ -101,9 +101,9 @@ function railsTools() {
                     .describe('Rail display label (<=60 chars); null/empty clears to default (set_name). Also the optional initial name for create_rail.'),
                 // launch
                 mode: zod_1.z
-                    .enum(['implement', 'batch-implement', 'freestyle', 'loop'])
+                    .enum(['implement', 'freestyle', 'loop'])
                     .optional()
-                    .describe('Launch mode (launch; default "implement"). Use "freestyle" as the canonical API enum value for Freestyle. It invokes the selected provider\'s native free-form autonomous workflow (currently Claude and Kimi), one job per ticket. In prose, call it "Freestyle". Loop runs an app-driven loop per ticket.'),
+                    .describe('Launch mode (launch; default "implement"). Implement runs all of the rail\'s tickets in one aggregate run; propose at most 3 specs per rail and split larger sets across rails. Use "freestyle" as the canonical API enum value for Freestyle. It invokes the selected provider\'s native free-form autonomous workflow (currently Claude and Kimi), one job per ticket. In prose, call it "Freestyle". Loop runs an app-driven loop per ticket.'),
                 model: zod_1.z
                     .string()
                     .optional()