opencode-swarm 7.121.0 → 7.121.2

package/dist/cli/index.js

@@ -7,28 +7,28 @@ import {
   getPluginLockFilePaths,
   package_default,
   resolveCommand
-} from "./index-gmc34yzt.js";
+} from "./index-y0qa3xf0.js";
 import"./index-kyvg2cmp.js";
 import"./index-vqcgmy4y.js";
-import"./index-j5xv8zbp.js";
+import"./index-emkb0bbe.js";
 import"./index-4905hd2m.js";
 import"./index-nprawcc0.js";
-import"./index-78b192v7.js";
-import"./index-0k4r8111.js";
-import"./index-0e1v02hq.js";
+import"./index-x4fwtd59.js";
+import"./index-ft6jw816.js";
+import"./index-j8349ac0.js";
 import"./index-vxv732ex.js";
 import"./index-c8s9a3zh.js";
 import"./index-816rkp4y.js";
 import"./index-22e127jw.js";
-import"./index-3366gs26.js";
-import"./index-69ms9ny1.js";
+import"./index-db82jxt7.js";
+import"./index-5a3xsnzg.js";
 import"./index-rtry5xyf.js";
-import"./index-rywkdacj.js";
-import"./index-nhgw7de8.js";
+import"./index-0f800qg7.js";
+import"./index-35xyj7gn.js";
 import"./index-ae75rja9.js";
 import"./index-3jvqw5hy.js";
 import"./index-9xpgytj9.js";
-import"./index-0mbmfmt0.js";
+import"./index-qt2yaghg.js";
 import"./index-gnp9x6se.js";
 import {
   DEFAULT_AGENT_CONFIGS

package/dist/cli/{knowledge-escalator-0ejqs8c9.js → knowledge-escalator-8ajq1fhb.js}

@@ -8,9 +8,9 @@ import {
   maybeEscalateOnViolation,
   maybeQuarantineOnContradiction,
   readRecentEscalations
-} from "./index-0k4r8111.js";
-import"./index-rywkdacj.js";
-import"./index-nhgw7de8.js";
+} from "./index-ft6jw816.js";
+import"./index-0f800qg7.js";
+import"./index-35xyj7gn.js";
 import"./index-ae75rja9.js";
 import"./index-3jvqw5hy.js";
 import"./index-4rhcth7a.js";

package/dist/cli/{knowledge-events-wx9e37ba.js → knowledge-events-2sjy3aa7.js}

@@ -25,7 +25,7 @@ import {
   resolveKnowledgeCounterBaselinePath,
   resolveKnowledgeEventsPath,
   resolveLegacyApplicationLogPath
-} from "./index-nhgw7de8.js";
+} from "./index-35xyj7gn.js";
 import"./index-ae75rja9.js";
 import"./index-3jvqw5hy.js";
 import"./index-zgwm4ryv.js";

package/dist/cli/{knowledge-store-rnajx33a.js → knowledge-store-27cesthc.js}

@@ -40,7 +40,7 @@ import {
   transactKnowledge,
   transactKnowledgeWithCas,
   wordBigrams
-} from "./index-rywkdacj.js";
+} from "./index-0f800qg7.js";
 import {
   resolveHiveDataDir
 } from "./index-ae75rja9.js";

package/dist/cli/{knowledge-validator-1f4mkzmc.js → knowledge-validator-9keh3p8j.js}

@@ -23,11 +23,11 @@ import {
   validateLesson,
   validateSkillCandidatePath,
   validateSkillPath
-} from "./index-3366gs26.js";
-import"./index-69ms9ny1.js";
+} from "./index-db82jxt7.js";
+import"./index-5a3xsnzg.js";
 import"./index-rtry5xyf.js";
-import"./index-rywkdacj.js";
-import"./index-nhgw7de8.js";
+import"./index-0f800qg7.js";
+import"./index-35xyj7gn.js";
 import"./index-ae75rja9.js";
 import"./index-3jvqw5hy.js";
 import"./index-4rhcth7a.js";

package/dist/cli/{scan-cursor-qapm0sxj.js → scan-cursor-qmsmmg1b.js}

@@ -4,8 +4,8 @@ import {
   alreadyCuratedThisGeneration,
   claimNextScanBatch,
   getScanStatus
-} from "./index-0e1v02hq.js";
-import"./index-rywkdacj.js";
+} from "./index-j8349ac0.js";
+import"./index-0f800qg7.js";
 import"./index-ae75rja9.js";
 import"./index-3jvqw5hy.js";
 import"./index-4rhcth7a.js";

package/dist/cli/{skill-generator-tkv7s12g.js → skill-generator-hkyxdc3j.js}

@@ -33,12 +33,12 @@ import {
   sanitizeSlug,
   selectCandidateEntries,
   writeEvalStub
-} from "./index-78b192v7.js";
-import"./index-3366gs26.js";
-import"./index-69ms9ny1.js";
+} from "./index-x4fwtd59.js";
+import"./index-db82jxt7.js";
+import"./index-5a3xsnzg.js";
 import"./index-rtry5xyf.js";
-import"./index-rywkdacj.js";
-import"./index-nhgw7de8.js";
+import"./index-0f800qg7.js";
+import"./index-35xyj7gn.js";
 import"./index-ae75rja9.js";
 import"./index-3jvqw5hy.js";
 import"./index-4rhcth7a.js";

package/dist/commands/issue.d.ts

@@ -10,4 +10,16 @@
  *   --no-repro    → appends noRepro=true to emitted signal
  *   no args       → returns usage string (no throw)
  */
+import * as fs from 'node:fs';
+/**
+ * DI seam for filesystem operations — allows tests to override synchronous fs calls.
+ */
+export declare const _internals: {
+    writeFileSync: typeof fs.writeFileSync;
+    mkdirSync: typeof fs.mkdirSync;
+    renameSync: typeof fs.renameSync;
+    unlinkSync: typeof fs.unlinkSync;
+    readFileSync: typeof fs.readFileSync;
+    existsSync: typeof fs.existsSync;
+};
 export declare function handleIssueCommand(directory: string, args: string[]): string;

package/dist/config/plan-schema.d.ts

@@ -69,6 +69,7 @@ export declare const TaskSchema: z.ZodObject<{
     files_touched: z.ZodDefault<z.ZodArray<z.ZodString>>;
     evidence_path: z.ZodOptional<z.ZodString>;
     blocked_reason: z.ZodOptional<z.ZodString>;
+    fr_refs: z.ZodOptional<z.ZodArray<z.ZodString>>;
 }, z.core.$strip>;
 export type Task = z.infer<typeof TaskSchema>;
 export declare const PhaseSchema: z.ZodObject<{
@@ -103,6 +104,7 @@ export declare const PhaseSchema: z.ZodObject<{
         files_touched: z.ZodDefault<z.ZodArray<z.ZodString>>;
         evidence_path: z.ZodOptional<z.ZodString>;
         blocked_reason: z.ZodOptional<z.ZodString>;
+        fr_refs: z.ZodOptional<z.ZodArray<z.ZodString>>;
     }, z.core.$strip>>>;
     type: z.ZodOptional<z.ZodEnum<{
         code: "code";
@@ -148,6 +150,7 @@ export declare const PlanSchema: z.ZodObject<{
             files_touched: z.ZodDefault<z.ZodArray<z.ZodString>>;
             evidence_path: z.ZodOptional<z.ZodString>;
             blocked_reason: z.ZodOptional<z.ZodString>;
+            fr_refs: z.ZodOptional<z.ZodArray<z.ZodString>>;
         }, z.core.$strip>>>;
         type: z.ZodOptional<z.ZodEnum<{
             code: "code";

package/dist/hooks/delegation-gate.d.ts

@@ -66,6 +66,20 @@ interface ValidationContext {
 /**
  * Validates a DelegationEnvelope against the current plan and agent list.
  * Returns { valid: true } on success, or { valid: false; reason: string } on failure.
+ *
+ * NOTE (#1687, kept SEPARATE from {@link validateCoderReviewerAcceptanceField}):
+ * this validator operates on a *structured* `DelegationEnvelope` object recovered
+ * from the OLD `KEY:VALUE` envelope format (`taskId:`/`targetAgent:`/`action:`/
+ * `commandType:`/`files:`/`acceptanceCriteria:`) and is consumed ADVISORY-ONLY,
+ * POST-execution via {@link appendDelegationEnvelopeAdvisory} in `toolAfter` (it
+ * never throws). The free-text coder/reviewer format
+ * (`TASK:`/`FILE:`/`ACCEPTANCE:`) does NOT parse into this shape
+ * (`parseDelegationEnvelope` returns `null`), so this path never fires on a real
+ * coder/reviewer dispatch. The pre-dispatch, BLOCKING acceptance-field gate for
+ * the free-text format lives in {@link validateCoderReviewerAcceptanceField};
+ * the two are intentionally not merged (advisory-vs-blocking, structured-vs-
+ * free-text are different contracts). Do not delete this function — it retains a
+ * real purpose for the old envelope format and other delegation types.
  */
 export declare function validateDelegationEnvelope(envelope: unknown, context: ValidationContext): EnvelopeValidationResult;
 /**
@@ -87,6 +101,107 @@ export declare function validateDelegationEnvelope(envelope: unknown, context: V
  *   the side effect (advisory push) is the production purpose.
  */
 export declare function appendDelegationEnvelopeAdvisory(session: AgentSessionState, promptText: string, context: ValidationContext): EnvelopeValidationResult | null;
+/**
+ * FR-003 / SC-003 / SC-004 (issue #1687): pre-dispatch acceptance-criteria
+ * enforcement for **coder and reviewer** delegations, operating on the REAL
+ * free-text delegation prompt (`TASK:`/`FILE:`/`INPUT:`/`OUTPUT:`/`CONSTRAINT:`/
+ * `ACCEPTANCE:`/`SKILLS:`) the architect actually sends via the native Task tool.
+ *
+ * Deliberately kept SEPARATE from {@link validateDelegationEnvelope} (see the
+ * sibling note there), NOT a rename/reuse of it, because the semantics differ on
+ * two axes: this function parses the FREE-TEXT `ACCEPTANCE:` line (not a
+ * structured envelope object) and is consumed as a BLOCKING, PRE-execution
+ * (`toolBefore`) gate (not the advisory, post-execution `toolAfter` path that
+ * `validateDelegationEnvelope` feeds via {@link appendDelegationEnvelopeAdvisory}).
+ * The two formats never overlap — `parseDelegationEnvelope` returns `null` for
+ * the free-text coder/reviewer format — so merging them would conflate
+ * advisory-vs-blocking and structured-vs-free-text contracts. M15's advisory
+ * mechanism is left fully intact for the old envelope format and other
+ * delegation types.
+ *
+ * A dispatch passes when an `ACCEPTANCE:` line exists AND the field carries
+ * non-empty, non-whitespace content — either INLINE on the header line (the
+ * architect-instructed same-line format) OR on the lines that FOLLOW a bare
+ * `ACCEPTANCE:` header, up to (but excluding) the next INPUT-FORMAT field header
+ * (`^[A-Z][A-Z0-9_]*:`, e.g. `SKILLS:`) or end of input. The multi-line form is
+ * a plausible way to paste a verbatim FR/SC body that spans lines (PR #1864
+ * review feedback), so treating a bare header + following content as EMPTY would
+ * false-block a good-faith copy. The next-field-header terminator ensures
+ * content belonging to the NEXT field (e.g. `SKILLS: none`) is never miscounted
+ * as ACCEPTANCE content — an empty `ACCEPTANCE:` immediately followed by another
+ * field is still correctly rejected as empty. Prompts are split on `\r?\n` so a
+ * trailing `\r` (CRLF-authored prompts) never leaks into the captured content.
+ *
+ * Only NON-EMPTINESS is enforced here; verbatim-coverage of the mapped FR/SC
+ * bodies is a SEPARATE, later check ({@link checkAcceptanceCoversFrRefs}) that
+ * already newline-normalizes the whole prompt, so multi-line ACCEPTANCE content
+ * is visible to it regardless.
+ *
+ * @param promptText the assembled delegation prompt (all text-bearing Task args
+ *   joined).
+ * @returns `{ valid: true }` when the ACCEPTANCE field is present and non-empty;
+ *   otherwise `{ valid: false, reason }` with a diagnosable reason the caller
+ *   turns into a blocking, pre-dispatch error.
+ */
+export declare function validateCoderReviewerAcceptanceField(promptText: string): {
+    valid: boolean;
+    reason?: 'acceptance_field_missing' | 'acceptance_field_empty';
+};
+/**
+ * FR-001/FR-002/FR-005/SC-001/SC-002/SC-006 (issue #1687): pull the verbatim
+ * requirement BODY for a single spec id (`FR-###` / `SC-###`) out of `spec.md`.
+ *
+ * spec.md requirements are bold-prefixed bullets, one per (possibly wrapped)
+ * bullet, e.g.
+ *   `- **FR-000 — Structured task-to-FR mapping.** The plan task model SHALL ...`
+ *   `- **SC-000 (FR-000).** Given a plan ..., then ...`
+ * The requirement BODY is the text AFTER the FIRST closing `**` of the leading
+ * bold span — i.e. everything past the `**...**` id/title prefix. Handles both
+ * FR-### and SC-### uniformly (no special-casing) and stitches on any
+ * immediately-following continuation lines of a wrapped bullet until the next
+ * bullet / next FR-|SC- bullet / a blank line / a markdown heading.
+ *
+ * @returns the raw (untrimmed) body string, or `null` when the id is not present
+ *   in the spec (unknown/typo/renamed) — the caller treats null as fail-open skip.
+ */
+export declare function extractSpecRequirementBodyById(specText: string, id: string): string | null;
+/**
+ * FR-001/FR-002 (issue #1687): symmetric normalization applied identically to
+ * BOTH the extracted spec body AND the ACCEPTANCE prompt text before the
+ * substring-coverage compare. Asymmetric normalization would false-block, so this
+ * is deliberately the single shared normalizer. Strips markdown emphasis/code
+ * markers and leading list markers, folds punctuation the architect's LLM copy
+ * routinely substitutes (dash-width, curly-vs-straight quotes) so a good-faith
+ * verbatim copy is not false-blocked on a `—`→`--` or `'`→`'` swap, collapses
+ * ALL whitespace (incl. newlines) to single spaces, trims, and lowercases. Every
+ * fold is symmetric (both sides go through this one function), so it can only
+ * REDUCE false-blocks — it never opens a bypass, since covered/uncovered still
+ * differ by whole words, not punctuation.
+ */
+export declare function normalizeAcceptanceText(s: string): string;
+/**
+ * FR-001/FR-002/FR-005/SC-001/SC-002/SC-006 (issue #1687): mechanical coverage
+ * check that the ACCEPTANCE text CONTAINS the verbatim requirement body for each
+ * mapped spec id. Fail-open by construction:
+ *  - an id not present in spec.md (unknown/typo/renamed) is SKIPPED, never blocked;
+ *  - an id whose body normalizes to empty is SKIPPED (neither trivially covered
+ *    nor a miss);
+ *  - each id is checked INDEPENDENTLY (per-id substring), because the architect
+ *    concatenates multiple requirement bodies with an UNSPECIFIED separator — a
+ *    whole-string equality compare would false-block multi-FR tasks.
+ *
+ * @returns `{ covered: true }` when every id is present-and-covered or skipped;
+ *   `{ covered: false, missingId }` naming the FIRST id whose body is not a
+ *   substring of the ACCEPTANCE text.
+ */
+export declare function checkAcceptanceCoversFrRefs(params: {
+    acceptanceText: string;
+    frRefs: string[];
+    specText: string;
+}): {
+    covered: boolean;
+    missingId?: string;
+};
 interface MessageInfo {
     role: string;
     agent?: string;
@@ -101,6 +216,11 @@ interface MessageWithParts {
     info: MessageInfo;
     parts: MessagePart[];
 }
+/**
+ * Returns whether the plan in the given directory has a valid plan-critic
+ * approval. Does not throw — returns `false` for any failure (fail-closed).
+ */
+export declare function isPlanCriticApproved(directory: string): Promise<boolean>;
 declare function resolveDelegatedPlanTaskId(args: Record<string, unknown>, knownPlanTaskIds?: ReadonlySet<string>): string | null;
 /**
  * Parses structured per-task verdict lines from agent dispatch output.

package/dist/hooks/issue-trace-reducer.d.ts

@@ -0,0 +1,66 @@
+/**
+ * Issue trace reducer — a pure transition function with zero I/O.
+ *
+ * Determines the next mode transition for an issue-trace workflow
+ * based on current trace state and workflow artifacts. Evaluated
+ * top-to-bottom (first-match-wins) against an 8-row decision table.
+ */
+export interface IssueReference {
+    url: string;
+    owner: string;
+    repo: string;
+    number: number;
+    timestamp: string;
+    flags: {
+        plan?: boolean;
+        trace?: boolean;
+        noRepro?: boolean;
+    };
+    noReproWaiver?: {
+        waived: boolean;
+        reason: string;
+        timestamp: string;
+    };
+}
+export interface TraceState {
+    issueNumber: number;
+    lastTransition: string | null;
+    completed: boolean;
+}
+export interface WorkflowArtifacts {
+    specExists: boolean;
+    specIssueNumber: number | null;
+    planExists: boolean;
+    criticApproved: boolean;
+    allPhasesComplete: boolean;
+}
+export interface TransitionResult {
+    nextMode: string | null;
+    directive: string | null;
+    nextLastTransition: string | null;
+    nextCompleted: boolean;
+}
+export interface ComputeNextModeParams {
+    issueReference: IssueReference | null;
+    traceState: TraceState;
+    workflowArtifacts: WorkflowArtifacts;
+}
+/**
+ * Pure reducer: given trace state + workflow artifacts, return the
+ * next mode transition (or a no-op).
+ *
+ * Decision table (top-to-bottom, first match wins):
+ *   (a) No issue reference or trace not requested        → no-op
+ *   (b) Trace already completed                           → no-op
+ *   (c) Cross-issue guard (spec issue ≠ current issue)   → no-op
+ *   (d) Spec does not exist                              → no-op
+ *   (e) Spec exists, no plan, never transitioned          → PLAN
+ *   (f) Plan exists but critic not approved               → no-op
+ *   (g) Critic approved, phases incomplete, not yet PLAN_TO_EXECUTE → EXECUTE
+ *   (h) All phases complete, not yet EXECUTE_TO_COMMIT    → COMMIT directive
+ *
+ * Idempotency: rows (e), (g), (h) return no-op when
+ * `traceState.lastTransition` already equals the target
+ * transition value.
+ */
+export declare function computeNextMode(params: ComputeNextModeParams): TransitionResult;

package/dist/hooks/issue-trace-state.d.ts

@@ -0,0 +1,64 @@
+/**
+ * Issue trace state adapter — reads and writes all artifacts for the
+ * issue-trace workflow engine.
+ *
+ * Uses the `_internals` DI seam pattern (AGENTS.md invariant 7) so
+ * tests can override filesystem calls without `mock.module` leakage.
+ */
+import { existsSync, mkdirSync, readFileSync, renameSync, unlinkSync, writeFileSync } from 'node:fs';
+import type { Plan } from '../config/plan-schema';
+import type { IssueReference, TraceState } from './issue-trace-reducer';
+declare function _defaultLoadPlanFromLedger(directory: string): Promise<Plan | null>;
+export declare const _internals: {
+    readFileSync: typeof readFileSync;
+    writeFileSync: typeof writeFileSync;
+    existsSync: typeof existsSync;
+    renameSync: typeof renameSync;
+    mkdirSync: typeof mkdirSync;
+    unlinkSync: typeof unlinkSync;
+    loadPlanFromLedger: typeof _defaultLoadPlanFromLedger;
+};
+/**
+ * Reads `.swarm/issue-reference.json` and returns the parsed object,
+ * or null if the file is absent or malformed.
+ */
+export declare function readIssueReference(directory: string): IssueReference | null;
+/**
+ * Reads `.swarm/issue-trace-state.json` and returns the parsed state,
+ * or a default `{issueNumber:0, lastTransition:null, completed:false}`
+ * if the file is absent or malformed.
+ */
+export declare function readTraceState(directory: string): TraceState;
+/**
+ * Writes `.swarm/issue-trace-state.json` atomically: creates a temp file
+ * inside `.swarm/`, writes JSON, then renames. Cleans up the temp on failure.
+ */
+export declare function writeTraceState(directory: string, state: TraceState): void;
+/**
+ * Reads `.swarm/spec.md` and extracts the issue number from the
+ * `## Source Issue` section. Looks for `- Number: N` or `- URL: ...issues/N`
+ * under the heading. Returns null if the section is absent or the number
+ * is unparseable.
+ */
+export declare function readSpecIssueNumber(directory: string): number | null;
+/**
+ * Loads the plan from the ledger-aware plan loader.
+ * Async because the underlying ledger loader is async.
+ */
+export declare function loadPlanFromLedger(directory: string): Promise<Plan | null>;
+/**
+ * Checks whether all phases in the plan have a completed status.
+ * Returns `{allComplete: false}` if the plan is null.
+ */
+export declare function readPlanPhaseStatus(directory: string): Promise<{
+    allComplete: boolean;
+}>;
+/**
+ * Returns whether `.swarm/spec.md` exists.
+ */
+export declare function specExists(directory: string): boolean;
+/**
+ * Returns whether `.swarm/plan.json` exists.
+ */
+export declare function planExists(directory: string): boolean;
+export {};

package/dist/hooks/issue-trace.d.ts

@@ -0,0 +1,31 @@
+/**
+ * Issue trace hook — the deterministic trace transition engine.
+ *
+ * Ties together the reducer (issue-trace-reducer), the state adapter
+ * (issue-trace-state), and the approval helper (delegation-gate) into
+ * a single `messagesTransform` hook that the composeHandlers chain
+ * calls on every architect message cycle.
+ *
+ * Uses the `_internals` DI seam pattern (AGENTS.md invariant 7) so
+ * tests can override adapter functions without `mock.module` leakage.
+ */
+import { isPlanCriticApproved } from './delegation-gate';
+import { planExists, readIssueReference, readPlanPhaseStatus, readSpecIssueNumber, readTraceState, specExists, writeTraceState } from './issue-trace-state';
+export declare const _internals: {
+    readIssueReference: typeof readIssueReference;
+    readTraceState: typeof readTraceState;
+    writeTraceState: typeof writeTraceState;
+    readSpecIssueNumber: typeof readSpecIssueNumber;
+    readPlanPhaseStatus: typeof readPlanPhaseStatus;
+    specExists: typeof specExists;
+    planExists: typeof planExists;
+    isPlanCriticApproved: typeof isPlanCriticApproved;
+};
+export declare function createIssueTraceHook(_config: unknown, directory: string, approvalTimeoutMs?: number): {
+    messagesTransform: (input: unknown, output: unknown) => Promise<void>;
+};
+/**
+ * Reset the approval cache. Called by tests in afterEach to avoid
+ * cross-test pollution from the module-level cache.
+ */
+export declare function resetApprovalCache(): void;