opencode-swarm 7.140.4 → 7.140.6

package/dist/plan/ledger.d.ts

@@ -78,6 +78,28 @@ declare function getLedgerPath(directory: string): string;
  * Get the path to plan.json
  */
 declare function getPlanJsonPath(directory: string): string;
+/**
+ * Durable atomic write: write `data` to `tempPath`, fsync the file descriptor so
+ * the bytes are flushed to stable storage, then atomically rename it over
+ * `targetPath`.
+ *
+ * WHY the fsync matters: a plain `writeFileSync` returns once the data is in the
+ * OS page cache, not on disk. A crash/power-loss between the write and the
+ * subsequent `rename` can publish a zero-length or partially-written temp file as
+ * the canonical ledger — silent truncation that later reads cannot distinguish
+ * from a legitimately short ledger. Forcing an fsync before the rename guarantees
+ * the temp file's bytes are durable before it becomes the canonical file.
+ *
+ * NOTE: the containing-directory fsync (which would also durably persist the
+ * rename's directory entry so the *rename itself* survives a crash) is
+ * intentionally OMITTED. The rename is atomic on POSIX, and the ledger's
+ * crash-consistency contract only requires that a published ledger never be
+ * partially-written; it tolerates losing the very last rename on power-loss (the
+ * prior canonical file is then still intact). Omitting the dir-fsync avoids the
+ * portability and cost concerns of opening the directory as a file descriptor.
+ */
+declare function writeFileFsyncedThenRename(tempPath: string, targetPath: string, data: string | Uint8Array): void;
+declare function fsyncRecoveryDirectory(directory: string): void;
 /**
  * Compute a SHA-256 hash of the plan state.
  * Uses deterministic JSON serialization for consistent hashing.
@@ -163,6 +185,13 @@ export declare function initLedger(directory: string, planId: string, initialPla
 export declare function appendLedgerEvent(directory: string, eventInput: LedgerEventInput, options?: {
     expectedSeq?: number;
     expectedHash?: string;
+    /**
+     * CAS against the durable ledger tail rather than plan.json. This is used
+     * only by stale-projection reconciliation, where plan.json intentionally
+     * differs from the ledger and a snapshot adopts that exact projection as
+     * the new authoritative state.
+     */
+    expectedLedgerHash?: string;
     planHashAfter?: string;
 }): Promise<LedgerEvent>;
 /**
@@ -244,7 +273,31 @@ export declare function takeSnapshotEvent(directory: string, plan: Plan, options
     source?: string;
     approvalMetadata?: Record<string, unknown>;
     payloadHashOverride?: string;
+    /** CAS binding for stale-projection recovery snapshots. */
+    expectedSeq?: number;
+    /** Previous durable hash used to preserve the ledger hash chain. */
+    expectedLedgerHash?: string;
 }): Promise<LedgerEvent>;
+export interface TruncatedLedgerRecoveryResult {
+    archivePath: string;
+    archiveSha256: string;
+    recoveryRoot: LedgerEvent;
+}
+/**
+ * Recover an explicitly verified stale projection from a malformed ledger.
+ *
+ * A snapshot appended after an unparsable line is not durable recovery because
+ * integrity replay stops at that line after restart. This transaction first
+ * writes and fsyncs a byte-for-byte archive of the complete canonical ledger,
+ * then atomically replaces the canonical file with a new `plan_created` root
+ * that embeds the verified projection and links to the preserved archive.
+ * A crash therefore leaves either the old canonical ledger or the new complete
+ * root; it never creates a missing-ledger window or silently discards history.
+ */
+export declare function replaceTruncatedLedgerWithRecoveryRoot(directory: string, plan: Plan, expected: {
+    seq: number;
+    ledgerHash: string;
+}): Promise<TruncatedLedgerRecoveryResult>;
 /**
  * Options for replayFromLedger
  */
@@ -461,5 +514,7 @@ export declare const _internals: {
     loadLastPlanCriticApprovedSnapshot: typeof loadLastPlanCriticApprovedSnapshot;
     getLedgerPath: typeof getLedgerPath;
     getPlanJsonPath: typeof getPlanJsonPath;
+    writeFileFsyncedThenRename: typeof writeFileFsyncedThenRename;
+    fsyncRecoveryDirectory: typeof fsyncRecoveryDirectory;
 };
 export {};

package/dist/plan/manager.d.ts

@@ -146,6 +146,15 @@ export declare function savePlan(directory: string, plan: Plan, options?: {
     acknowledged_removals?: AcknowledgedRemovals;
     /** True only when the caller already owns the canonical plan.json lock. */
     planLockAlreadyHeld?: boolean;
+    /**
+     * Narrow recovery mode for a semantically unreadable ledger tail. The
+     * caller must bind the exact tail observed while validating an unchanged
+     * `_ledgerReplayStale` projection.
+     */
+    staleProjectionReconcile?: {
+        expectedSeq: number;
+        expectedLedgerHash: string;
+    };
 }): Promise<void>;
 /**
  * Rebuild plan from ledger events.

package/dist/plan/planning-profile.d.ts

@@ -0,0 +1,27 @@
+import type { ExecutionProfile, Plan } from '../config/plan-schema';
+import type { PluginConfig } from '../config/schema';
+export type PlanningProfile = NonNullable<NonNullable<ExecutionProfile['planning_profile']>>;
+export interface ResolvedPlanningProfile {
+    effective: PlanningProfile;
+    persisted: PlanningProfile | undefined;
+    source: 'incoming' | 'persisted' | 'repository_default' | 'legacy_locked_default';
+}
+export type PlanningProfileDirectiveContext = 'repository_default' | 'runtime';
+/**
+ * Render the authoritative planning ceremony for the architect prompt.
+ *
+ * Keep this policy next to the resolver so static agent construction and
+ * per-turn persisted-plan injection cannot drift into different meanings for
+ * the same resolved profile.
+ */
+export declare function renderPlanningProfileDirective(resolution: ResolvedPlanningProfile, context?: PlanningProfileDirectiveContext): string;
+interface ResolvePlanningProfileOptions {
+    directory: string;
+    incomingExecutionProfile?: Partial<ExecutionProfile>;
+    existingExecutionProfile?: Plan['execution_profile'];
+    resetStatuses?: boolean;
+    config?: Pick<PluginConfig, 'execution_mode'>;
+}
+export declare function resolvePlanningProfile({ directory, incomingExecutionProfile, existingExecutionProfile, resetStatuses, config, }: ResolvePlanningProfileOptions): ResolvedPlanningProfile;
+export declare function normalizeExecutionProfileForHash(profile: Plan['execution_profile']): Record<string, unknown> | undefined;
+export {};

package/dist/services/spec-drift-recovery.d.ts

@@ -0,0 +1,84 @@
+import { validateSwarmPath } from '../hooks/utils.js';
+import { tryAcquireLock } from '../parallel/file-locks.js';
+import { takeSnapshotEvent } from '../plan/ledger.js';
+import { loadPlanJsonOnly, savePlan } from '../plan/manager.js';
+import { readEffectiveSpecSync } from '../sdd/effective-spec.js';
+import type { SpecDriftAcknowledgedEvent, SpecDriftRepairedEvent } from '../types/events.js';
+import { bunWrite } from '../utils/bun-compat.js';
+import { assertProjectRoot } from '../utils/project-boundary.js';
+import { invalidateCachedArtifact } from '../utils/swarm-artifact-cache.js';
+type RecoveryMode = 'acknowledge' | 'repair';
+type RecoveryActor = 'cli' | 'spec_write' | 'unknown' | 'user';
+type RecoveryState = 'COMMITTED' | 'PREPARED';
+interface SpecStalenessMarker {
+    planTitle: string;
+    phase: number;
+    specHash_plan: string | null;
+    specHash_current: string | null;
+    reason: string;
+    timestamp: string;
+}
+interface SpecDriftRecoveryWal {
+    version: 2;
+    mode: RecoveryMode;
+    state: RecoveryState;
+    transitionId: string;
+    markerHash: string;
+    planTitle: string;
+    phase: number;
+    previousHash: string | null;
+    newHash: string | null;
+    specContent: string | null;
+    actor: RecoveryActor;
+    recordedAt: string;
+}
+export interface SpecDriftRecoveryResult {
+    status: 'applied' | 'cleanup_pending' | 'corrupt_marker' | 'failed' | 'no_marker' | 'retry_later';
+    mode: RecoveryMode;
+    message: string;
+    phase?: number;
+    planTitle?: string;
+    previousHash?: string | null;
+    newHash?: string | null;
+    transitionId?: string;
+}
+export declare const _internals: {
+    appendEvent: typeof appendEvent;
+    assertProjectRoot: typeof assertProjectRoot;
+    bunWrite: typeof bunWrite;
+    invalidateCachedArtifact: typeof invalidateCachedArtifact;
+    loadPlanJsonOnly: typeof loadPlanJsonOnly;
+    now: typeof now;
+    parseMarker: typeof parseMarker;
+    parseWal: typeof parseWal;
+    readEffectiveSpecSync: typeof readEffectiveSpecSync;
+    readFileText: typeof readFileText;
+    savePlan: typeof savePlan;
+    tryAcquireLock: typeof tryAcquireLock;
+    takeSnapshotEvent: typeof takeSnapshotEvent;
+    unlinkIfExists: typeof unlinkIfExists;
+    validateSwarmPath: typeof validateSwarmPath;
+    verifyEventPresence: typeof verifyEventPresence;
+    ensureSnapshotEvent: typeof ensureSnapshotEvent;
+    verifySnapshot: typeof verifySnapshot;
+    writeWal: typeof writeWal;
+};
+declare function now(): string;
+declare function parseMarker(raw: string): SpecStalenessMarker | null;
+declare function parseWal(raw: string): SpecDriftRecoveryWal | null;
+declare function readFileText(filePath: string): Promise<string | null>;
+declare function unlinkIfExists(filePath: string): Promise<boolean>;
+declare function writeWal(walPath: string, wal: SpecDriftRecoveryWal): Promise<void>;
+declare function verifySnapshot(snapshotPath: string, currentSpec: {
+    content: string;
+} | null): Promise<void>;
+declare function ensureSnapshotEvent(directory: string, plan: NonNullable<Awaited<ReturnType<typeof loadPlanJsonOnly>>>, transitionId: string): Promise<void>;
+declare function appendEvent(eventsPath: string, event: SpecDriftAcknowledgedEvent | SpecDriftRepairedEvent): Promise<void>;
+declare function verifyEventPresence(eventsPath: string, eventType: 'spec_drift_acknowledged' | 'spec_drift_repaired', transitionId: string): Promise<boolean>;
+export declare function reconcileSpecDrift(directory: string, options: {
+    mode: RecoveryMode;
+    actor: RecoveryActor;
+    /** Caller already holds the spec.md lock across the canonical spec write. */
+    specLockAlreadyHeld?: boolean;
+}): Promise<SpecDriftRecoveryResult>;
+export {};

package/dist/session/snapshot-writer.d.ts

@@ -2,6 +2,7 @@
  * Session snapshot writer for OpenCode Swarm plugin.
  * Serializes swarmState to .swarm/session/state.json using atomic write (temp-file + rename).
  */
+import { TASK_WORKFLOW_SCHEMA_MARKER } from '../gate-evidence.js';
 import type { AgentSessionState, DelegationEntry, ToolAggregate } from '../state';
 import { swarmState } from '../state';
 /**
@@ -114,8 +115,9 @@ export interface SerializedInvocationWindow {
  * Snapshot data structure written to disk
  */
 export interface SnapshotData {
-    version: 1 | 2;
+    version: 1 | 2 | 3;
     writtenAt: number;
+    workflowSchema?: typeof TASK_WORKFLOW_SCHEMA_MARKER;
     toolAggregates: Record<string, ToolAggregate>;
     activeAgent: Record<string, string>;
     delegationChains: Record<string, DelegationEntry[]>;

package/dist/state.d.ts

@@ -10,6 +10,7 @@ import type { OpencodeClient } from '@opencode-ai/sdk';
 import type { CouncilAgent } from './council/types';
 import { type QaGates } from './db/qa-gate-profile.js';
 import { type EnvironmentProfile } from './environment/profile.js';
+import { type TaskWorkflowMetadata } from './gate-evidence';
 import type { EscalationTracker } from './prm/escalation.js';
 import type { PatternMatch } from './prm/types.js';
 import { AgentRunContext } from './state/agent-run-context.js';
@@ -63,7 +64,14 @@ export interface PrSubscriptionState {
  * Per-task workflow state for gate progression tracking.
  * Transitions must be forward-only: idle → coder_delegated → pre_check_passed → reviewer_run → tests_run → complete
  */
-export type TaskWorkflowState = 'idle' | 'coder_delegated' | 'pre_check_passed' | 'reviewer_run' | 'tests_run' | 'complete';
+export type TaskWorkflowState = 'idle' | 'coder_delegated' | 'pre_check_passed' | 'reviewer_run' | 'tests_run' | 'rework_required' | 'complete' | 'blocked';
+export interface TaskWorkflowCacheEntry {
+    generation: number;
+    retryCount: number;
+    lastOutcome: TaskWorkflowMetadata['lastOutcome'];
+    lastTransitionId: string | null;
+    updatedAt: string;
+}
 /**
  * Upper bound for per-session task file-attribution entries.
  *
@@ -187,6 +195,8 @@ export interface AgentSessionState {
     qaSkipTaskIds: string[];
     /** Per-task workflow state — taskId → current state */
     taskWorkflowStates: Map<string, TaskWorkflowState>;
+    /** Exact-task workflow cache keyed by taskId. Durable evidence remains authoritative. */
+    taskWorkflowCache?: Map<string, TaskWorkflowCacheEntry>;
     /**
      * PR 2 Stage B barrier: per-task set of completed Stage B agents.
      * Order-independent — either 'reviewer' or 'test_engineer' may complete first.
@@ -213,6 +223,8 @@ export interface AgentSessionState {
          */
         rewarded?: boolean;
     }>;
+    /** Generation captured when the first member of a task council is dispatched. */
+    taskCouncilWorkflowGeneration?: Map<string, number>;
     /**
      * Per-(task,round) required council members for the next submission attempt.
      * Key format: `${taskId}:${roundNumber}`.
@@ -1026,6 +1038,7 @@ export declare function resetModifiedFilesForTask(session: AgentSessionState, ta
  * Epic sessions retain it until `epic_record_divergence` consumes the entry.
  */
 export declare function completeModifiedFilesForTask(session: AgentSessionState, taskId: string): void;
+export declare function updateTaskWorkflowCache(session: AgentSessionState, taskId: string, overrides: Partial<TaskWorkflowCacheEntry>): void;
 /**
  * Advance a task's workflow state. Validates forward-only transitions.
  * Throws 'INVALID_TASK_STATE_TRANSITION: [taskId] [current] → [requested]' on illegal transition.
@@ -1061,23 +1074,12 @@ export declare function canAdvanceTaskState(session: AgentSessionState, taskId:
     requireAllMembers?: boolean;
 }): boolean;
 /**
- * Advance the per-task workflow state machine AND persist the corresponding
- * plan.json status at meaningful workflow boundaries.
- *
- * The two-layer model splits in-memory workflow state (Layer 1, fast, used by
- * gates) from the durable plan (Layer 2, projected to plan.md). Without this
- * bridge, council APPROVE → 'complete' updates Layer 1 only and plan.md goes
- * stale. This helper closes the gap by mapping:
- *   - 'coder_delegated' → plan.json status 'in_progress'
- *   - 'complete'        → plan.json status 'completed'
- * Other transitions are in-memory only (the task is already in_progress on disk
- * once coder_delegated has fired).
- *
- * Persistence errors are logged and swallowed so a transient disk failure does
- * not break the in-memory state machine — matches the existing defensive
- * pattern around advanceTaskState call sites.
+ * Legacy compatibility wrapper for diagnostic-only intermediate projections.
+ * Durable coder/terminal boundaries must use the exact-task evidence reducer
+ * and the plan→evidence transaction/WAL; this wrapper intentionally refuses
+ * those states so no caller can recreate the former split-brain writer.
  */
-export declare function advanceTaskStateAndPersist(session: AgentSessionState, taskId: string, newState: TaskWorkflowState, directory: string, options?: {
+export declare function advanceTaskStateAndPersist(session: AgentSessionState, taskId: string, newState: TaskWorkflowState, _directory: string, options?: {
     telemetrySessionId?: string;
     emitTelemetry?: boolean;
 }, councilConfig?: {

package/dist/summaries/schema.d.ts

@@ -22,8 +22,8 @@ export declare const MAX_LIST_ITEMS = 5;
 /** Verdict vocabulary — mirrors the phase-council gate (APPROVE | CONCERNS | REJECT). */
 export declare const SupervisorVerdictSchema: z.ZodEnum<{
     APPROVE: "APPROVE";
-    REJECT: "REJECT";
     CONCERNS: "CONCERNS";
+    REJECT: "REJECT";
 }>;
 export type SupervisorVerdict = z.infer<typeof SupervisorVerdictSchema>;
 /** Count whitespace-delimited words in a string. */
@@ -130,8 +130,8 @@ export declare const ArchitectureSupervisorReportSchema: z.ZodObject<{
     phase: z.ZodNumber;
     verdict: z.ZodEnum<{
         APPROVE: "APPROVE";
-        REJECT: "REJECT";
         CONCERNS: "CONCERNS";
+        REJECT: "REJECT";
     }>;
     findings: z.ZodDefault<z.ZodArray<z.ZodObject<{
         severity: z.ZodEnum<{

package/dist/tools/convene-council.d.ts

@@ -8,23 +8,23 @@ export declare const ArgsSchema: z.ZodObject<{
     verdicts: z.ZodArray<z.ZodObject<{
         agent: z.ZodEnum<{
             reviewer: "reviewer";
-            test_engineer: "test_engineer";
+            critic: "critic";
             explorer: "explorer";
+            test_engineer: "test_engineer";
             sme: "sme";
-            critic: "critic";
         }>;
         verdict: z.ZodEnum<{
             APPROVE: "APPROVE";
-            REJECT: "REJECT";
             CONCERNS: "CONCERNS";
+            REJECT: "REJECT";
         }>;
         confidence: z.ZodNumber;
         findings: z.ZodArray<z.ZodObject<{
             severity: z.ZodEnum<{
-                LOW: "LOW";
-                MEDIUM: "MEDIUM";
-                HIGH: "HIGH";
                 CRITICAL: "CRITICAL";
+                HIGH: "HIGH";
+                MEDIUM: "MEDIUM";
+                LOW: "LOW";
             }>;
             category: z.ZodString;
             location: z.ZodString;

package/dist/tools/save-plan.d.ts

@@ -82,6 +82,12 @@ export interface SavePlanArgs {
      * Omit to leave the current profile unchanged.
      */
     execution_profile?: Partial<ExecutionProfile>;
+    /**
+     * Narrow recovery mode for stale plan.json projections whose ledger replay
+     * previously failed. When true, save_plan will only accept an unchanged
+     * semantic re-save that reconverges plan.json with the authoritative ledger.
+     */
+    reconcile_ledger_projection?: boolean;
 }
 /**
  * Result from executing save_plan
@@ -115,6 +121,8 @@ interface RequirementCoverageResult {
     missing: RequirementCoverageEntry[];
     blocking_missing: RequirementCoverageEntry[];
 }
+export declare function isLedgerProjectionReconcileRequest(args: Pick<SavePlanArgs, 'reconcile_ledger_projection'>): boolean;
+export declare function isPureLedgerProjectionReconcileRequest(args: SavePlanArgs): boolean;
 /**
  * Detect template placeholder content (e.g., [task], [Project], [description], [N]).
  * These patterns indicate the LLM reproduced template examples literally rather than

package/dist/tools/submit-phase-council-verdicts.d.ts

@@ -8,23 +8,23 @@ export declare const ArgsSchema: z.ZodObject<{
     verdicts: z.ZodArray<z.ZodObject<{
         agent: z.ZodEnum<{
             reviewer: "reviewer";
-            test_engineer: "test_engineer";
+            critic: "critic";
             explorer: "explorer";
+            test_engineer: "test_engineer";
             sme: "sme";
-            critic: "critic";
         }>;
         verdict: z.ZodEnum<{
             APPROVE: "APPROVE";
-            REJECT: "REJECT";
             CONCERNS: "CONCERNS";
+            REJECT: "REJECT";
         }>;
         confidence: z.ZodNumber;
         findings: z.ZodArray<z.ZodObject<{
             severity: z.ZodEnum<{
-                LOW: "LOW";
-                MEDIUM: "MEDIUM";
-                HIGH: "HIGH";
                 CRITICAL: "CRITICAL";
+                HIGH: "HIGH";
+                MEDIUM: "MEDIUM";
+                LOW: "LOW";
             }>;
             category: z.ZodString;
             location: z.ZodString;

package/dist/tools/tool-metadata.d.ts

@@ -34,7 +34,7 @@ export interface ToolMeta {
 export declare const TOOL_METADATA: {
     diff: {
         description: string;
-        agents: ("reviewer" | "test_engineer" | "coder" | "critic_oversight" | "critic_finding_validator" | "architect")[];
+        agents: ("reviewer" | "critic_oversight" | "critic_finding_validator" | "coder" | "test_engineer" | "architect")[];
         prWorkflow: {
             modes: ("PR_REVIEW" | "PR_FEEDBACK")[];
             capability: "observe";
@@ -50,7 +50,7 @@ export declare const TOOL_METADATA: {
     };
     syntax_check: {
         description: string;
-        agents: ("test_engineer" | "coder" | "architect")[];
+        agents: ("coder" | "test_engineer" | "architect")[];
         prWorkflow: {
             modes: "PR_REVIEW"[];
             capability: "validate";
@@ -66,7 +66,7 @@ export declare const TOOL_METADATA: {
     };
     imports: {
         description: string;
-        agents: ("reviewer" | "test_engineer" | "coder" | "docs" | "sme" | "critic" | "critic_sounding_board" | "critic_drift_verifier" | "critic_hallucination_verifier" | "critic_finding_validator" | "architect" | "researcher" | "docs_design")[];
+        agents: ("reviewer" | "critic" | "critic_finding_validator" | "coder" | "test_engineer" | "architect" | "sme" | "researcher" | "docs" | "docs_design" | "critic_sounding_board" | "critic_drift_verifier" | "critic_hallucination_verifier")[];
         prWorkflow: {
             modes: ("PR_REVIEW" | "PR_FEEDBACK")[];
             capability: "observe";
@@ -74,7 +74,7 @@ export declare const TOOL_METADATA: {
     };
     lint: {
         description: string;
-        agents: ("reviewer" | "coder" | "critic_finding_validator" | "architect")[];
+        agents: ("reviewer" | "critic_finding_validator" | "coder" | "architect")[];
         prWorkflow: {
             modes: "PR_REVIEW"[];
             capability: "validate";
@@ -98,7 +98,7 @@ export declare const TOOL_METADATA: {
     };
     build_check: {
         description: string;
-        agents: ("test_engineer" | "coder" | "architect")[];
+        agents: ("coder" | "test_engineer" | "architect")[];
     };
     pre_check_batch: {
         description: string;
@@ -114,15 +114,15 @@ export declare const TOOL_METADATA: {
     };
     symbols: {
         description: string;
-        agents: ("reviewer" | "test_engineer" | "coder" | "docs" | "designer" | "sme" | "critic" | "critic_sounding_board" | "critic_drift_verifier" | "critic_hallucination_verifier" | "critic_finding_validator" | "architect" | "researcher" | "docs_design" | "spec_writer")[];
+        agents: ("reviewer" | "critic" | "critic_finding_validator" | "coder" | "test_engineer" | "architect" | "sme" | "researcher" | "docs" | "docs_design" | "designer" | "critic_sounding_board" | "critic_drift_verifier" | "critic_hallucination_verifier" | "spec_writer")[];
     };
     complexity_hotspots: {
         description: string;
-        agents: ("reviewer" | "test_engineer" | "explorer" | "sme" | "critic" | "critic_sounding_board" | "critic_drift_verifier" | "critic_hallucination_verifier" | "critic_oversight" | "critic_finding_validator" | "architect" | "researcher")[];
+        agents: ("reviewer" | "critic" | "critic_oversight" | "critic_finding_validator" | "explorer" | "test_engineer" | "architect" | "sme" | "researcher" | "critic_sounding_board" | "critic_drift_verifier" | "critic_hallucination_verifier")[];
     };
     schema_drift: {
         description: string;
-        agents: ("docs" | "explorer" | "sme" | "architect" | "researcher")[];
+        agents: ("explorer" | "architect" | "sme" | "researcher" | "docs")[];
         prWorkflow: {
             modes: ("PR_REVIEW" | "PR_FEEDBACK")[];
             capability: "observe";
@@ -130,7 +130,7 @@ export declare const TOOL_METADATA: {
     };
     todo_extract: {
         description: string;
-        agents: ("docs" | "explorer" | "architect" | "researcher")[];
+        agents: ("explorer" | "architect" | "researcher" | "docs")[];
         prWorkflow: {
             modes: ("PR_REVIEW" | "PR_FEEDBACK")[];
             capability: "observe";
@@ -210,7 +210,7 @@ export declare const TOOL_METADATA: {
     };
     pkg_audit: {
         description: string;
-        agents: ("reviewer" | "test_engineer" | "critic_hallucination_verifier" | "critic_oversight" | "critic_finding_validator" | "architect")[];
+        agents: ("reviewer" | "critic_oversight" | "critic_finding_validator" | "test_engineer" | "architect" | "critic_hallucination_verifier")[];
         prWorkflow: {
             modes: "PR_REVIEW"[];
             capability: "validate";
@@ -238,7 +238,7 @@ export declare const TOOL_METADATA: {
     };
     test_runner: {
         description: string;
-        agents: ("reviewer" | "test_engineer" | "critic_finding_validator" | "architect")[];
+        agents: ("reviewer" | "critic_finding_validator" | "test_engineer" | "architect")[];
         prWorkflow: {
             modes: "PR_REVIEW"[];
             capability: "validate";
@@ -246,7 +246,7 @@ export declare const TOOL_METADATA: {
     };
     test_impact: {
         description: string;
-        agents: ("reviewer" | "test_engineer" | "critic_oversight" | "critic_finding_validator" | "architect")[];
+        agents: ("reviewer" | "critic_oversight" | "critic_finding_validator" | "test_engineer" | "architect")[];
         prWorkflow: {
             modes: ("PR_REVIEW" | "PR_FEEDBACK")[];
             capability: "observe";
@@ -262,11 +262,11 @@ export declare const TOOL_METADATA: {
     };
     detect_domains: {
         description: string;
-        agents: ("docs" | "explorer" | "sme" | "critic" | "critic_sounding_board" | "critic_drift_verifier" | "critic_hallucination_verifier" | "critic_oversight" | "architect" | "docs_design")[];
+        agents: ("critic" | "critic_oversight" | "explorer" | "architect" | "sme" | "docs" | "docs_design" | "critic_sounding_board" | "critic_drift_verifier" | "critic_hallucination_verifier")[];
     };
     git_blame: {
         description: string;
-        agents: ("reviewer" | "explorer" | "critic_finding_validator" | "architect")[];
+        agents: ("reviewer" | "critic_finding_validator" | "explorer" | "architect")[];
         prWorkflow: {
             modes: ("PR_REVIEW" | "PR_FEEDBACK")[];
             capability: "observe";
@@ -274,7 +274,7 @@ export declare const TOOL_METADATA: {
     };
     gitingest: {
         description: string;
-        agents: ("docs" | "explorer" | "architect")[];
+        agents: ("explorer" | "architect" | "docs")[];
         prWorkflow: {
             modes: ("PR_REVIEW" | "PR_FEEDBACK")[];
             capability: "observe";
@@ -282,7 +282,7 @@ export declare const TOOL_METADATA: {
     };
     retrieve_summary: {
         description: string;
-        agents: ("reviewer" | "test_engineer" | "coder" | "docs" | "designer" | "sme" | "critic" | "critic_sounding_board" | "critic_drift_verifier" | "critic_hallucination_verifier" | "critic_architecture_supervisor" | "critic_finding_validator" | "architect" | "docs_design" | "spec_writer")[];
+        agents: ("reviewer" | "critic" | "critic_finding_validator" | "coder" | "test_engineer" | "architect" | "sme" | "docs" | "docs_design" | "designer" | "critic_sounding_board" | "critic_drift_verifier" | "critic_hallucination_verifier" | "critic_architecture_supervisor" | "spec_writer")[];
         prWorkflow: {
             modes: ("PR_REVIEW" | "PR_FEEDBACK")[];
             capability: "observe";
@@ -298,7 +298,7 @@ export declare const TOOL_METADATA: {
     };
     extract_code_blocks: {
         description: string;
-        agents: ("test_engineer" | "coder" | "docs" | "designer" | "docs_design" | "spec_writer")[];
+        agents: ("coder" | "test_engineer" | "docs" | "docs_design" | "designer" | "spec_writer")[];
     };
     phase_complete: {
         description: string;
@@ -362,7 +362,7 @@ export declare const TOOL_METADATA: {
     };
     knowledge_recall: {
         description: string;
-        agents: ("reviewer" | "test_engineer" | "coder" | "docs" | "designer" | "explorer" | "sme" | "critic" | "critic_sounding_board" | "critic_drift_verifier" | "critic_hallucination_verifier" | "critic_architecture_supervisor" | "critic_oversight" | "critic_finding_validator" | "architect" | "docs_design" | "curator_init" | "curator_phase" | "skill_improver" | "spec_writer")[];
+        agents: ("reviewer" | "critic" | "critic_oversight" | "critic_finding_validator" | "explorer" | "coder" | "test_engineer" | "architect" | "sme" | "docs" | "docs_design" | "designer" | "critic_sounding_board" | "critic_drift_verifier" | "critic_hallucination_verifier" | "critic_architecture_supervisor" | "curator_init" | "curator_phase" | "skill_improver" | "spec_writer")[];
     };
     knowledge_remove: {
         description: string;
@@ -378,11 +378,11 @@ export declare const TOOL_METADATA: {
     };
     search: {
         description: string;
-        agents: ("reviewer" | "test_engineer" | "coder" | "docs" | "designer" | "explorer" | "sme" | "critic_hallucination_verifier" | "critic_oversight" | "critic_finding_validator" | "architect" | "researcher" | "docs_design" | "skill_improver" | "spec_writer")[];
+        agents: ("reviewer" | "critic_oversight" | "critic_finding_validator" | "explorer" | "coder" | "test_engineer" | "architect" | "sme" | "researcher" | "docs" | "docs_design" | "designer" | "critic_hallucination_verifier" | "skill_improver" | "spec_writer")[];
     };
     ast_grep: {
         description: string;
-        agents: ("test_engineer" | "coder" | "docs" | "explorer" | "sme" | "critic_hallucination_verifier" | "architect" | "researcher" | "docs_design" | "spec_writer")[];
+        agents: ("explorer" | "coder" | "test_engineer" | "architect" | "sme" | "researcher" | "docs" | "docs_design" | "critic_hallucination_verifier" | "spec_writer")[];
     };
     actionlint_scan: {
         description: string;
@@ -418,7 +418,7 @@ export declare const TOOL_METADATA: {
     };
     batch_symbols: {
         description: string;
-        agents: ("reviewer" | "explorer" | "critic_hallucination_verifier" | "critic_oversight" | "critic_finding_validator" | "architect")[];
+        agents: ("reviewer" | "critic_oversight" | "critic_finding_validator" | "explorer" | "architect" | "critic_hallucination_verifier")[];
     };
     suggest_patch: {
         description: string;
@@ -426,15 +426,15 @@ export declare const TOOL_METADATA: {
     };
     req_coverage: {
         description: string;
-        agents: ("critic" | "critic_sounding_board" | "critic_drift_verifier" | "critic_hallucination_verifier" | "critic_oversight" | "spec_writer")[];
+        agents: ("critic" | "critic_oversight" | "critic_sounding_board" | "critic_drift_verifier" | "critic_hallucination_verifier" | "spec_writer")[];
     };
     get_approved_plan: {
         description: string;
-        agents: ("critic" | "critic_drift_verifier" | "critic_oversight")[];
+        agents: ("critic" | "critic_oversight" | "critic_drift_verifier")[];
     };
     repo_map: {
         description: string;
-        agents: ("reviewer" | "coder" | "explorer" | "critic" | "critic_sounding_board" | "critic_drift_verifier" | "critic_hallucination_verifier" | "critic_architecture_supervisor" | "critic_oversight" | "critic_finding_validator" | "architect")[];
+        agents: ("reviewer" | "critic" | "critic_oversight" | "critic_finding_validator" | "explorer" | "coder" | "architect" | "critic_sounding_board" | "critic_drift_verifier" | "critic_hallucination_verifier" | "critic_architecture_supervisor")[];
         prWorkflow: {
             modes: ("PR_REVIEW" | "PR_FEEDBACK")[];
             capability: "observe";
@@ -502,7 +502,7 @@ export declare const TOOL_METADATA: {
     };
     knowledge_receipt: {
         description: string;
-        agents: ("reviewer" | "test_engineer" | "coder" | "docs" | "designer" | "sme" | "critic" | "critic_sounding_board" | "critic_drift_verifier" | "critic_hallucination_verifier" | "critic_architecture_supervisor" | "critic_finding_validator" | "architect" | "docs_design" | "curator_init" | "curator_phase" | "skill_improver" | "spec_writer")[];
+        agents: ("reviewer" | "critic" | "critic_finding_validator" | "coder" | "test_engineer" | "architect" | "sme" | "docs" | "docs_design" | "designer" | "critic_sounding_board" | "critic_drift_verifier" | "critic_hallucination_verifier" | "critic_architecture_supervisor" | "curator_init" | "curator_phase" | "skill_improver" | "spec_writer")[];
     };
     knowledge_archive: {
         description: string;
@@ -518,7 +518,7 @@ export declare const TOOL_METADATA: {
     };
     swarm_command: {
         description: string;
-        agents: ("reviewer" | "test_engineer" | "coder" | "docs" | "designer" | "explorer" | "sme" | "critic" | "critic_finding_validator" | "architect" | "researcher" | "docs_design")[];
+        agents: ("reviewer" | "critic" | "critic_finding_validator" | "explorer" | "coder" | "test_engineer" | "architect" | "sme" | "researcher" | "docs" | "docs_design" | "designer")[];
     };
     dispatch_lanes: {
         description: string;
@@ -534,7 +534,7 @@ export declare const TOOL_METADATA: {
     };
     summarize_work: {
         description: string;
-        agents: ("test_engineer" | "coder" | "docs" | "designer" | "explorer" | "sme" | "architect" | "researcher" | "docs_design")[];
+        agents: ("explorer" | "coder" | "test_engineer" | "architect" | "sme" | "researcher" | "docs" | "docs_design" | "designer")[];
     };
     write_architecture_supervisor_evidence: {
         description: string;
@@ -566,7 +566,7 @@ export declare const TOOL_METADATA: {
     };
     swarm_apply_patch: {
         description: string;
-        agents: ("test_engineer" | "coder")[];
+        agents: ("coder" | "test_engineer")[];
     };
     external_skill_discover: {
         description: string;