@volter/supercode-orchestrator 0.4.25 → 0.4.27

package/board/cli.mjs

@@ -20,10 +20,10 @@ Hermes Kanban's verbs, with their flags:
   init · boards list|create|rm|switch|show|rename|set-default-workdir|export|import
   create <title> [--body] [--assignee] [--parent]... [--workspace scratch|worktree[:<path>]|dir:<path>] [--branch] [--project]
          [--tenant] [--priority] [--triage] [--idempotency-key] [--max-runtime 30m] [--created-by] [--skill]... [--max-retries]
-         [--model] [--provider] [--goal] [--goal-max-turns] [--initial-status blocked|running]
+         [--model] [--provider] [--goal|--no-goal] [--goal-max-turns] [--initial-status blocked|running]
   swarm <goal> --worker PROFILE:TITLE[:SKILL,SKILL]... --verifier P --synthesizer P
   list [--mine] [--assignee] [--status] [--tenant] [--session] [--archived] [--sort]   show <id>   context <id>   runs <id>
-  assign <id> <profile|none>   reassign <id> <profile> [--reclaim] [--reason]   set-model <id> [model] [--provider]
+  assign <id> <profile|none>   reassign <id> <profile> [--reclaim] [--reason]   set-model <id> [model] [--provider]   set-goal <id>... [--goal-max-turns N] [--off]
   claim <id> [--ttl]   reclaim <id> [--reason]   heartbeat <id> [--note]   link|unlink <parent> <child>
   comment <id> <text> [--author] [--max-len]   attach <id> <path> [--name] [--content-type]   attachments <id>   attach-rm <n>
   complete <id>... [--result] [--summary] [--metadata JSON]   edit <id> --result [--summary] [--metadata]
@@ -51,7 +51,7 @@ supercode's own:
 
 const MULTI = new Set(['parent', 'skill', 'worker', 'child', 'ids', 'rm']);
 const FLAGS = new Set(['json', 'triage', 'no-notify', 'no-start', 'all', 'force', 'help', 'mine', 'archived', 'reclaim', 'dry-run',
-  'goal', 'switch', 'delete', 'no-attachments', 'include-logs', 'verbose']);
+  'goal', 'no-goal', 'off', 'switch', 'delete', 'no-attachments', 'include-logs', 'verbose']);
 // flags that take every following word up to the next flag (Hermes's nargs='+')
 const GREEDY = new Set(['ids', 'rm']);
 
@@ -235,13 +235,15 @@ export async function runWorkflow(argv) {
       const meta = boardMeta(root, slug);
       let workspace = args.workspace;
       if ((workspace === 'dir' || workspace === 'worktree') && meta.default_workdir) workspace = `${workspace}:${meta.default_workdir}`;
+      // a home whose `kanban.goal_mode_default` is true makes every new card a goal card (a subtask is never started alone)
+      const goalDefault = !args.goal && !args['no-goal'] && !args['subtask-of'] && (await laneState(root)).kanban.goal_mode_default === true;
       const created = withBoard((b) => engine.createCard(b, {
         title: rest.join(' '), body: args.body, assignee: args.assignee === 'none' ? null : args.assignee, machine: args.machine,
         workspace, branch: args.branch, parents: args.parent, priority: args.priority, tenant: args.tenant,
         triage: args.triage, idempotencyKey: args['idempotency-key'], maxRuntime: duration(args['max-runtime']),
         maxRetries, skills: args.skill, model: args.model, provider: args.provider, createdBy: args['created-by'] ?? callerName(),
         sessionId: process.env.CLAUDE_CODE_SESSION_ID ?? null, subtaskOf: args['subtask-of'] ?? null, initialStatus: args['initial-status'] ?? null,
-        project: args.project ?? meta.project_id ?? null, goal: args.goal ? { maxTurns: args['goal-max-turns'] ? Number(args['goal-max-turns']) : null } : null,
+        project: args.project ?? meta.project_id ?? null, goal: args.goal || goalDefault ? { maxTurns: args['goal-max-turns'] ? Number(args['goal-max-turns']) : null } : null,
       }), true);
       let answer = { id: created.id, existing: created.existing };
       if (!args['no-start'] && !created.existing && !args['subtask-of']) {
@@ -377,6 +379,14 @@ export async function runWorkflow(argv) {
       say(args, `Reassigned ${id} to ${rest[1]}`, { ok: true }); return 0;
     }
     case 'set-model': need(1, 'a card id'); withBoard((b) => engine.setModel(b, id, rest[1] ?? null, args.provider ?? null)); say(args, rest[1] ? `${id} runs on model ${rest[1]}${args.provider ? ` (${args.provider})` : ''} from its next start` : `${id} uses its profile's model`, { ok: true }); return 0;
+    case 'set-goal': {
+      // a card into goal mode (or out of it with --off) from its next idle turn: the heartbeat keeps its session going
+      need(1, 'a card id');
+      const maxTurns = args['goal-max-turns'] ? Number(args['goal-max-turns']) : null;
+      if (maxTurns !== null && !(Number.isInteger(maxTurns) && maxTurns >= 1)) throw new Error('--goal-max-turns is a whole number of turns, at least 1');
+      for (const t of ids()) withBoard((b) => engine.setGoal(b, t, { on: !args.off, maxTurns }));
+      say(args, ids().map((t) => args.off ? `${t} is no longer a goal card` : `${t} is a goal card${maxTurns ? ` (${maxTurns} turns)` : ''}`).join('\n'), { ok: true }); return 0;
+    }
     case 'move': need(2, 'a card id and a machine (or none)'); withBoard((b) => engine.setMachine(b, id, rest[1] === 'none' ? null : rest[1])); say(args, `${id} runs on ${rest[1]}`, { ok: true }); return 0;
 
     case 'claim': {
@@ -942,7 +952,7 @@ function authorize(db, verb, rest, args, own) {
   }
   if (verb === 'boards') { if (!['list', 'ls', 'show', 'current', undefined].includes(rest[0])) refuse('does not change the home\'s boards'); return; }
   // verbs that act on the board rather than on a card are the manager's and the board's, never a card's session's
-  if (['pause', 'resume', 'swarm', 'dispatch', 'daemon', 'serve', 'gc', 'repair', 'adopt', 'close', 'move', 'reassign', 'assign', 'set-model', 'claim', 'reclaim', 'archive', 'promote', 'schedule', 'link', 'unlink', 'edit', 'reopen-review', 'reopen', 'attach-rm'].includes(verb)) refuse(`does not ${verb}; the fleet's manager does`);
+  if (['pause', 'resume', 'swarm', 'dispatch', 'daemon', 'serve', 'gc', 'repair', 'adopt', 'close', 'move', 'reassign', 'assign', 'set-model', 'set-goal', 'claim', 'reclaim', 'archive', 'promote', 'schedule', 'link', 'unlink', 'edit', 'reopen-review', 'reopen', 'attach-rm'].includes(verb)) refuse(`does not ${verb}; the fleet's manager does`);
   if (args.all) refuse(`does not ${verb} every card; only its own`);
   const card = engine.getTask(db, rest[0]);
   if (!card) return; // the verb itself answers "no card"

package/board/dispatch.mjs

@@ -23,7 +23,7 @@ import { fileURLToPath } from 'node:url';
 import { boardPath, event, json, listBoards, now, openBoard, tx } from './store.mjs';
 import { holdBoardTickLock } from '../hermes-locks.mjs';
 import {
-  cardContext, claimIn, claimTtl, currentRun, DEFAULT_FAILURE_LIMIT, extendClaim, getTask, lastSession,
+  blockIn, cardContext, claimIn, claimTtl, currentRun, DEFAULT_FAILURE_LIMIT, extendClaim, getTask, lastSession,
   recomputeReady, recordFailure, release, setRunSession, subscribeIn, subscriptions,
 } from './engine.mjs';
 
@@ -474,8 +474,9 @@ function launchPane(board, task, runId, lane, fleet, { input = null } = {}) {
   let replaced = null;
   if (prev && prev.machine === machine) {
     if (sessionAlive(fleet, prev)) {
-      session = { ...prev, mode: 'adopt', launched_at: now(), seen_at: now() };
-      delete session.missing_since;
+      // the session is kept; the heartbeat's count belongs to the run that ended, so the new run starts it again
+      const { missing_since: _m, goal_turns: _t, goal_nudged_at: _n, idle_since: _i, worked_since_ask: _w, ...kept } = prev;
+      session = { ...kept, mode: 'adopt', launched_at: now(), seen_at: now() };
     } else if (prev.resumable !== false) {
       const r = sc([...base, '--key', `board-${task.id}-r${runId}-resume`, '--input',
         `The board resumed this session after it was lost; you are still on card ${task.id}: ${task.title}. Carry on.`, '--', '--resume', prev.session_id, ...tail], { timeout: 180_000 });
@@ -942,9 +943,7 @@ function pursueGoals(board, fleet, settings, report) {
       const turns = session.worked_since_ask && !task.goal_mode ? 0 : Number(session.goal_turns ?? 0);
       const budget = task.goal_max_turns ?? settings.goalMaxTurns;
       if (turns >= budget) {
-        db.prepare("UPDATE tasks SET status = 'blocked', claim_lock = NULL, claim_expires = NULL WHERE id = ?").run(id);
-        const ended = endRun(db, id, { outcome: 'blocked', summary: `idle ${budget} times without deciding done or blocked` });
-        event(db, id, 'blocked', { reason: `its session went idle ${budget} times without deciding done or blocked; review it`, kind: 'needs_input', source_status: 'running' }, ended);
+        blockIn(db, id, { reason: `its session went idle ${budget} times without deciding done or blocked; review it`, kind: 'needs_input' });
         return null;
       }
       setRunSession(db, runRow.id, { ...session, goal_turns: turns + 1, goal_nudged_at: now(), worked_since_ask: false });

package/board/engine.mjs

@@ -396,25 +396,28 @@ function completeIn(board, id, { result = null, summary = null, metadata = null
 }
 
 export function block(board, id, { reason = null, kind = null } = {}) {
+  return tx(board, (db) => blockIn(db, id, { reason, kind }));
+}
+
+/** `block` inside the caller's transaction (the dispatcher's own blocks keep the same recurrence accounting). */
+export function blockIn(db, id, { reason = null, kind = null } = {}) {
   if (kind && !BLOCK_KINDS.includes(kind)) throw new Error(`block kind is one of ${BLOCK_KINDS.join(', ')}`);
-  return tx(board, (db) => {
-    const task = must(db, id);
-    if (!['running', 'ready'].includes(task.status)) throw new Error(`${id} is ${task.status}; only a running or ready card blocks`);
-    const source = task.status === 'running' ? retryStatus(db, id) : 'ready';
-    if (kind === 'dependency') {
-      run(db, "UPDATE tasks SET status = 'todo', claim_lock = NULL, claim_expires = NULL, worker_pid = NULL, block_kind = ? WHERE id = ?", kind, id);
-      const runId = endRun(db, id, { outcome: 'blocked', summary: reason }) ?? (reason ? synthesizeRun(db, id, { outcome: 'blocked', summary: reason }) : null);
-      event(db, id, 'dependency_wait', { reason, kind, source_status: source }, runId);
-      return 'todo';
-    }
-    const recurrences = task.block_kind === kind ? Number(task.block_recurrences || 0) + 1 : 1;
-    const loop = recurrences >= BLOCK_RECURRENCE_LIMIT;
-    const status = loop ? 'triage' : 'blocked';
-    run(db, 'UPDATE tasks SET status = ?, claim_lock = NULL, claim_expires = NULL, worker_pid = NULL, block_kind = ?, block_recurrences = ? WHERE id = ?', status, kind, recurrences, id);
+  const task = must(db, id);
+  if (!['running', 'ready'].includes(task.status)) throw new Error(`${id} is ${task.status}; only a running or ready card blocks`);
+  const source = task.status === 'running' ? retryStatus(db, id) : 'ready';
+  if (kind === 'dependency') {
+    run(db, "UPDATE tasks SET status = 'todo', claim_lock = NULL, claim_expires = NULL, worker_pid = NULL, block_kind = ? WHERE id = ?", kind, id);
     const runId = endRun(db, id, { outcome: 'blocked', summary: reason }) ?? (reason ? synthesizeRun(db, id, { outcome: 'blocked', summary: reason }) : null);
-    event(db, id, loop ? 'block_loop_detected' : 'blocked', { reason, kind, recurrences, ...(loop ? { limit: BLOCK_RECURRENCE_LIMIT } : {}), source_status: source }, runId);
-    return status;
-  });
+    event(db, id, 'dependency_wait', { reason, kind, source_status: source }, runId);
+    return 'todo';
+  }
+  const recurrences = task.block_kind === kind ? Number(task.block_recurrences || 0) + 1 : 1;
+  const loop = recurrences >= BLOCK_RECURRENCE_LIMIT;
+  const status = loop ? 'triage' : 'blocked';
+  run(db, 'UPDATE tasks SET status = ?, claim_lock = NULL, claim_expires = NULL, worker_pid = NULL, block_kind = ?, block_recurrences = ? WHERE id = ?', status, kind, recurrences, id);
+  const runId = endRun(db, id, { outcome: 'blocked', summary: reason }) ?? (reason ? synthesizeRun(db, id, { outcome: 'blocked', summary: reason }) : null);
+  event(db, id, loop ? 'block_loop_detected' : 'blocked', { reason, kind, recurrences, ...(loop ? { limit: BLOCK_RECURRENCE_LIMIT } : {}), source_status: source }, runId);
+  return status;
 }
 
 export function unblock(board, id) {
@@ -768,6 +771,15 @@ export function setModel(board, id, model = null, provider = null) {
   });
 }
 
+/** Goal mode on a card (Hermes's goal loop, which Hermes sets only at creation): on or off, with its turn budget. */
+export function setGoal(board, id, { on = true, maxTurns = null } = {}) {
+  return tx(board, (db) => {
+    must(db, id);
+    run(db, 'UPDATE tasks SET goal_mode = ?, goal_max_turns = ? WHERE id = ?', on ? 1 : 0, on ? maxTurns : null, id);
+    event(db, id, 'goal_set', { goal_mode: Boolean(on), goal_max_turns: on ? maxTurns : null });
+  });
+}
+
 export function addAttachment(board, id, { file, name, contentType = null, author = 'user', dir }) {
   return tx(board, (db) => {
     must(db, id);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@volter/supercode-orchestrator",
-  "version": "0.4.25",
+  "version": "0.4.27",
   "type": "module",
   "description": "The orchestrator runtime over the Volter Harness ontology: one typed operational model whose folder is its serialization, read and written through the harness orchestration doors (docs/ORCHESTRATOR-IR.md)",
   "exports": {