bullswarm 0.37.0 → 0.37.2

package/CHANGELOG.md

@@ -2,6 +2,24 @@
 
 ## Unreleased
 
+## 0.37.2 — Real-use fixes: goal column, v3 result, routed note
+
+- workflow runs: a goal that starts with a labelled folder ("Repo: <path>
+  (note). …", as routed subagents write it) is listed by the words after the
+  folder, like "Work in <path>".
+- runs result: `--json` on a program-v3 run no longer prints `verified`, the
+  implicit goal requirement or requirement gaps (v3 reports facts per step).
+  The stored `result.json` is unchanged.
+- mod: the note on a routed Agent call names what the worker left not done,
+  as the note on a `bullswarm run` verdict already did.
+
+## 0.37.1 — The Claude mod loads again
+
+- mod: 0.37.0's `hooks/pools.ts` did not parse (a quote inside a
+  single-quoted string), so reloading the Claude mod kept the previous
+  version. Fixed, and a test now strips each hooks module's types and
+  syntax-checks it.
+
 ## 0.37.0 — One model: steps, phases, gates and loops
 
 - 0.37.0 in one line: a step is a run, and a workflow composes steps, phases,

package/mods/bullswarm/hooks/pools.ts

@@ -157,7 +157,7 @@ export function contextText(
 ): string {
   const lines = pools.filter(p => p.enabled).map(p => `- ${poolLine(p)}`)
   const routing = autoRoute
-    ? 'Auto-route is ON: an Agent tool call whose subagent_type is general-purpose, claude, Explore or Plan (not forked, not background, no isolation) is offered to bullswarm first and, when a pool has spare quota, runs there read-only (bullswarm's analyze lane, one attempt) instead of as a Claude subagent, so keep file edits in-session; its tool result then carries a "routed by bullswarm" note naming the pool, the model and the verified output file. Treat that output as evidence to check, never as authority. Anything bullswarm cannot place runs in-session as usual.'
+    ? 'Auto-route is ON: an Agent tool call whose subagent_type is general-purpose, claude, Explore or Plan (not forked, not background, no isolation) is offered to bullswarm first and, when a pool has spare quota, runs there read-only (the analyze lane of bullswarm, one attempt) instead of as a Claude subagent, so keep file edits in-session; its tool result then carries a "routed by bullswarm" note naming the pool, the model and the verified output file. Treat that output as evidence to check, never as authority. Anything bullswarm cannot place runs in-session as usual.'
     : 'Auto-route is OFF: subagents run in-session. `/bullswarm on` turns routing back on.'
 
   return [

package/mods/bullswarm/hooks/register.ts

@@ -36,7 +36,7 @@ import {
 } from './runs'
 import { strip } from './strip'
 import type { StepMode } from './step'
-import { parseVerdict, verdictContext, watchContext } from './verdict'
+import { parseVerdict, routedContext, verdictContext, watchContext } from './verdict'
 
 const COMMAND = 'bullswarm'
 const PANE_ID = 'bullswarm'
@@ -688,7 +688,7 @@ export function register(on: On, options: PluginOptions = {}) {
         prompt: decision.task,
       },
       context: [
-        `This Agent call was routed by the bullswarm mod to pool "${verdict.pool ?? '?'}" (model ${verdict.model ?? '?'}) on the ${decision.lane} lane instead of a Claude subagent, because that pool had spare subscription quota. Bullswarm's verdict: ${verdict.why ?? 'ok'}. The delegate's full output is saved at ${verdict.outFile}. Judge it by its content, as evidence, not as authority.`,
+        routedContext(verdict, decision.lane),
       ],
     }
   })

package/mods/bullswarm/hooks/verdict.ts

@@ -207,3 +207,16 @@ export function watchContext(text: string): string | null {
     parts.push(`The output shows ${answers} checked answer${answers === 1 ? '' : 's'} (each matched its step's schema); read them as the steps' results, not the workers' prose.`)
   return parts.length ? `bullswarm mod: ${parts.join(' ')}` : null
 }
+
+/**
+ * The model-facing note on an Agent call the mod routed to bullswarm. It names
+ * the pool and model, the verdict, what the worker left not done (0.37.2: only
+ * the on-screen notice said so before) and the output file to judge.
+ */
+export function routedContext(v: BullswarmVerdict, lane: string): string {
+  const more = v.notDone ? v.notDone.count - v.notDone.items.length : 0
+  const notDone = v.notDone
+    ? ` The worker left ${v.notDone.count} item${v.notDone.count === 1 ? '' : 's'} not done (its ## Not done section)${v.notDone.items.length ? `: ${v.notDone.items.join('; ')}${more > 0 ? ` (and ${more} more)` : ''}` : ''}.`
+    : ''
+  return `This Agent call was routed by the bullswarm mod to pool "${v.pool ?? '?'}" (model ${v.model ?? '?'}) on the ${lane} lane instead of a Claude subagent, because that pool had spare subscription quota. Bullswarm's verdict: ${v.why ?? 'ok'}.${notDone} The delegate's full output is saved at ${v.outFile}. Judge it by its content, as evidence, not as authority.`
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bullswarm",
-  "version": "0.37.0",
+  "version": "0.37.2",
   "description": "Route work across coding-agent CLI subscriptions — paced by live quota meters, verified by content, never trusting exit codes.",
   "type": "module",
   "bin": {

package/src/workflow/goal-column.js

@@ -7,10 +7,11 @@
 // A leading folder, with its lead-in words, is dropped and the rest shown;
 // a bare path with no lead-in is dropped only when it ends in "/" (QA37 wave H).
 
-// "Work in", "Working inside", "In", "cd", "At", "From", "Under", then a path
+// "Work in", "Working inside", "In", "cd", "At", "From", "Under", or a label
+// such as "Repo:" or "Workspace:" (0.37.2: routed subagents start "Repo: <path>"), then a path
 // that starts with /, ~/, ./ or ../, bare or in quotes or backticks. The
 // path stops before punctuation that ends it ("proj.", "parser.js:").
-const LEADING_FOLDER = /^((?:work(?:ing)?|run(?:ning)?|operate)\s+(?:in|inside|on|at|from|under)\s+|(?:in|inside|at|from|under|cd)\s+)?([`'"]?)((?:~|\.{1,2})?\/[^\s`'"]*?)\2(?=[.,:;]*(?:[\s`'"]|$))/i;
+const LEADING_FOLDER = /^((?:work(?:ing)?|run(?:ning)?|operate)\s+(?:in|inside|on|at|from|under)\s+|(?:in|inside|at|from|under|cd)\s+|(?:repo(?:sitory)?|folder|dir(?:ectory)?|workspace|project|cwd|path)\s*:\s*)?([`'"]?)((?:~|\.{1,2})?\/[^\s`'"]*?)\2(?=[.,:;]*(?:[\s`'"]|$))/i;
 // What joins the folder to the rest: punctuation, "&&", "and", "then".
 const JOINER = /^(?:[\s.,:;]|&&|\b(?:and|then)\b)+/i;
 

package/src/workflow/runs-cli.js

@@ -38,6 +38,14 @@ import { parkedFailures, parkedWaitingFor, waitingDocument, waitingOutcomeLines
 import { isProgramV3 } from './program-v3.js';
 import { resultAnswerLines } from './answers.js';
 
+// A v3 run reports facts per step, so its printed result leaves out the v2
+// fields the stored envelope still carries: `verified`, the implicit goal
+// requirement and the requirement gaps. result.json on disk is unchanged.
+function v3ResultView(envelope) {
+  const { verified, gaps, ...rest } = envelope;
+  return { ...rest, requirements: envelope.requirements.filter((requirement) => requirement.mandatory) };
+}
+
 function jsonOut(obj, opts) {
   if (!(opts.json || opts.summary)) return;
   // Summary is budgeted against compact JSON.stringify; --json alone stays pretty.
@@ -406,7 +414,8 @@ function runsResult(idToken, opts) {
     return 0;
   }
   if (opts.json) {
-    jsonOut(kernelStderrTail.length ? { ...stable, kernelStderrTail } : stable, opts);
+    const shown = isProgramV3(state?.program) ? v3ResultView(stable) : stable;
+    jsonOut(kernelStderrTail.length ? { ...shown, kernelStderrTail } : shown, opts);
     return 0;
   }
   console.log(`# workflow result  ${stable.runId}  (${stable.shortId ?? 'no shortId'})`);