opencode-swarm 7.193.0 → 7.195.0

package/dist/pr-review/legacy-transcript-adapter.d.ts

@@ -52,6 +52,14 @@ export interface PrReviewItemClaim {
      * Reviewer claims only — this is what a critic batch binds to per item.
      */
     rowDigest?: string;
+    /**
+     * Issue #2906: reviewer claims only. Whether this item's verdict was stable
+     * across the counterbalanced presentation orders — 'agree' when both the A
+     * and B passes produced the same (classification, severity) pair, 'disagree'
+     * when they flipped, 'single' when fewer than two marked orders observed the
+     * item. The label describes cross-order evidence, not winner provenance.
+     */
+    orderAgreement?: 'agree' | 'disagree' | 'single';
 }
 export interface PrReviewPhaseComposition {
     /** Item id -> winning claim. Most recent successful lane per item wins. */
@@ -59,6 +67,19 @@ export interface PrReviewPhaseComposition {
     requiredInventory: string[];
     unclaimed: string[];
     contributingBatchIds: string[];
+    /**
+     * Issue #2906 review finding F-04: reviewer phase only (empty for critic).
+     * Batch ids that recorded at least one ORDER OBSERVATION the agreement
+     * labels read — a strictly different contribution than a claim. A batch
+     * whose newer counterbalanced pass claimed every item contributes no claim
+     * yet still supplies the second marked order every `agree`/`disagree`
+     * label is computed from, so a capacity prune that retains only claim
+     * contributors would silently degrade those labels to `single`. The prune
+     * retains these ids as well; an older batch whose observations were
+     * already recorded by a newer batch of the same order is absent (its
+     * removal cannot change any label).
+     */
+    orderObservingBatchIds: string[];
     diagnostics: string[];
 }
 /** Validation-batch shape, as the transcript composition reads it. */
@@ -87,6 +108,8 @@ export interface PrReviewTranscriptBatchCoherence {
     reviewerItemBindings?: Record<string, string>;
     /** Key encoding for reviewerItemBindings; absent means legacy raw item IDs. */
     reviewerItemBindingKeyEncoding?: 'prefixed-v1';
+    /** Issue #2906: reviewer batches only — the counterbalanced presentation order. */
+    presentationOrder?: 'A' | 'B';
 }
 /** Settled PR_FEEDBACK verification batch, as the adapter reads it. */
 export interface PrReviewTranscriptFeedbackVerification {
@@ -95,6 +118,20 @@ export interface PrReviewTranscriptFeedbackVerification {
         laneId: string;
     }>;
 }
+/**
+ * One journaled `write_pr_review_artifact` rejection (issue #3098 — the
+ * verdict-lane mirror of the #2859 submit-rejection journal). Structural twin
+ * of the gate's `PrReviewVerdictRejectionRecord`; declared locally because the
+ * adapter cannot import the gate.
+ */
+export interface PrReviewVerdictRejectionEntry {
+    sessionId: string;
+    laneId?: string;
+    workflowLane?: string;
+    boundary?: 'post_explorer' | 'post_reviewer' | 'post_critic';
+    at: number;
+    message: string;
+}
 /**
  * The PR-review slice of the owning gate's `PrWorkflowGateState`: exactly the
  * fields the moved conversion/composition machinery reads directly. The gate's
@@ -104,6 +141,7 @@ export interface PrReviewTranscriptState {
     sessionID: string;
     prReviewBatchCoherence?: Record<string, PrReviewTranscriptBatchCoherence>;
     prFeedbackVerifications?: PrReviewTranscriptFeedbackVerification[];
+    prReviewVerdictRejections?: PrReviewVerdictRejectionEntry[];
 }
 /** The gate-context surface the transcript composition consumes (full ctx is gate-owned). */
 export interface PrReviewTranscriptContext {
@@ -133,6 +171,16 @@ export interface PrReviewTranscriptGateHelpers {
     batchMayContributeClaims(directory: string, state: PrReviewTranscriptState, batch: PrReviewTranscriptBatch, phase: PrReviewComposablePhase, requiredInventory: readonly string[], forbiddenSubagentSessionIds: ReadonlySet<string>, reviewerClaims: ReadonlyMap<string, PrReviewItemClaim> | undefined, ctx: PrReviewTranscriptContext): boolean;
     recordsPassingBatchIntegrity(directory: string, state: PrReviewTranscriptState, batchId: string, expectedLanes: ReadonlyArray<PrReviewTranscriptLane>, expectedMode: string, validatedAt: string, checkWorkflowLane?: boolean, forbiddenSubagentSessionIds?: ReadonlySet<string>, diagnostics?: string[]): PrReviewTranscriptQualifiedBatchRecord[];
     loadArtifactPassingLaneIntegrity(directory: string, state: PrReviewTranscriptState, record: BackgroundDelegationRecord, expectedMode: string, expectedWorkflowLane: string, expectedRevisionDigest: string, diagnostics?: string[]): LaneOutputArtifact | null;
+    /**
+     * Issue #3098: newest journaled `write_pr_review_artifact` rejection
+     * attributable to a lane (workflowLane/laneId join keys). Bound by the
+     * gate, which owns the journal's reader.
+     */
+    latestPrReviewVerdictRejectionForLane(state: PrReviewTranscriptState, matchers: {
+        workflowLane?: string;
+        laneIds?: readonly string[];
+        boundary?: 'post_explorer' | 'post_reviewer' | 'post_critic';
+    }): string | undefined;
 }
 export declare function bindPrReviewTranscriptAdapterHelpers(helpers: PrReviewTranscriptGateHelpers): void;
 /**
@@ -161,6 +209,12 @@ export declare function reviewerItemBindingKey(itemId: string): string;
  * complete claim set is unbounded blocking I/O for no verdict change — first
  * write wins, and every required item has already been written.
  *
+ * Issue #2906 adds ONE bounded exception: when the claim set completes but
+ * marked presentation orders remain, the scan continues only far enough to
+ * collect the ORDER OBSERVATIONS of the remaining batches, which no later
+ * batch can supply. That remainder is bounded by the same
+ * `MAX_WORKFLOW_BATCHES` window and changes no claim.
+ *
  * `exhaustive` turns the exit off for the batch GC, which prunes a reviewer
  * batch on "it contributed no claim". Being precise about what that buys: with
  * *this* exit condition the two scans yield the same `contributingBatchIds`,

package/dist/pr-review/render-review-body.d.ts

@@ -17,6 +17,12 @@ export interface RendererFinding {
     evidence: string;
     next_action: string;
     severity?: string;
+    /**
+     * Issue #2906: order-stability of this finding's reviewer verdict across
+     * counterbalanced presentation orders. Only 'disagree' renders a marker
+     * (the token `order-unstable`); 'agree'/'single'/absent render nothing.
+     */
+    order_agreement?: string;
 }
 export interface RendererCoverage {
     kind: string;
@@ -29,6 +35,15 @@ export interface RenderReviewBodyInput {
     base_verification?: string;
     coverage: RendererCoverage;
     findings: RendererFinding[];
+    /**
+     * Issue #2906: order-unstable findings whose FINAL settled status keeps
+     * them out of `findings` (a DISPROVED row is not a finding and is never
+     * published as one). They are still order-unstable — the reviewer verdict
+     * flipped with presentation order — so they get their own section and are
+     * counted. Suppressing them entirely hid exactly the signal the
+     * counterbalancing control exists to produce.
+     */
+    order_unstable_suppressed?: RendererFinding[];
     /** Previously posted comment/review bodies; findings whose rendered marker already appears are skipped (AC4 dedupe). */
     existingComments?: string[];
 }
@@ -47,6 +62,11 @@ export interface RenderedReviewBody {
     truncatedInlineComments: number;
     /** findings omitted from the body as dismissed (severity NONE at settlement). */
     dismissedCount: number;
+    /**
+     * Issue #2906: findings rendered with the order-unstable marker — their
+     * reviewer verdict flipped across counterbalanced presentation orders.
+     */
+    orderUnstableCount: number;
     /** true when the body exceeded PR_REVIEW_BODY_CHAR_CAP and was truncated with a disclosure. */
     bodyTruncated: boolean;
 }

package/dist/tools/dispatch-lanes.d.ts

@@ -682,7 +682,7 @@ interface PrReviewReceiptFallbackState {
     }>;
 }
 declare function resolvePrReviewReceiptFallbacksFromState(records: readonly BackgroundDelegationRecord[], state: PrReviewReceiptFallbackState): ReadonlyMap<string, PrReviewVerdictCollectionReceipt>;
-declare function appendPrReviewCollectionReceipt(record: BackgroundDelegationRecord, result: BackgroundDelegationResult, receipt: PrReviewVerdictCollectionReceipt): BackgroundDelegationResult | null;
+declare function appendPrReviewCollectionReceipt(record: BackgroundDelegationRecord, result: BackgroundDelegationResult, receipt: PrReviewVerdictCollectionReceipt, presentationOrder?: 'A' | 'B'): BackgroundDelegationResult | null;
 declare function parsePrReviewCollectionReceipt(record: BackgroundDelegationRecord): PrReviewVerdictCollectionReceipt | null;
 declare function recordToLaneResult(record: BackgroundDelegationRecord, batchId: string, deliveryContext?: LaneResultDeliveryContext): DispatchLaneResult;
 declare function buildReadOnlyTools(mode?: string): ReadOnlyToolPermissions;
@@ -749,6 +749,21 @@ declare function applyPrWorkflowPromptContract(lanes: DispatchLaneSpec[], option
     revisionDigest?: string;
     scope?: string;
     callerFocus?: string;
+    /**
+     * Issue #2906: reviewer lanes only. 'A' renders assigned_item_ids in
+     * canonical sorted order, 'B' in the exact reverse, plus a
+     * presentation_order contract line. Undefined is exactly absent —
+     * prompts stay byte-identical to the pre-#2906 rendering.
+     */
+    presentationOrder?: 'A' | 'B';
+    /**
+     * Issue #2906: critic lanes only. Item ids whose verdicts flipped across
+     * counterbalanced reviewer passes; rendered as an
+     * order_unstable_item_ids line (intersected with the lane's assigned
+     * ids; '(none)' when the intersection is empty). Undefined is exactly
+     * absent — no line is rendered.
+     */
+    orderUnstableItemIds?: string[];
 }): ApplyCommonPromptResult;
 declare function formatError(error: unknown): string;
 declare function buildLaneSessionCreateArgs(directory: string, lane: DispatchLaneSpec, context: Pick<DispatchLanesExecutionContext, 'sessionID'>): {

package/dist/tools/pr-workflow-status.d.ts

@@ -1,4 +1,4 @@
-import { readDelegationsDetailed } from '../background/pending-delegations';
+import { findByBatchIdDetailed, readDelegationsDetailed } from '../background/pending-delegations';
 import { resolveCurrentGitHeadAsync, resolveIsWorkingTreeCleanAsync } from '../background/workspace-snapshot';
 import { classifyPrWorkflowGitState } from '../git/pr-workflow-state';
 import { isPrWorkflowAutoWakeSuppressed } from '../hooks/pr-workflow-auto-wake';
@@ -14,12 +14,30 @@ import { createSwarmTool } from './create-tool';
  * severed at the boundary instead of leaving a lone surrogate.
  */
 declare function truncateToByteBudget(value: string, maxBytes: number): string;
+/**
+ * Lightweight neutralizer for short, author-influenced Git strings (branch
+ * names, remote URLs, changed paths). A PR branch/remote is attacker-controlled
+ * text; this strips control characters and the markdown/HTML metacharacters that
+ * could break a downstream renderer, then bounds the length. It deliberately
+ * does NOT use the heavyweight `neutralizeUntrustedMarkdown` fenced wrapper,
+ * which is for large prose blobs, not single structured fields.
+ */
+declare function boundUntrusted(value: string, maxLen: number): string;
 /**
  * Bounded, killable, non-interactive Git read (AGENTS.md invariant 3). Array
  * form, explicit cwd, ignored stdin, capped stdout, timeout, best-effort kill in
  * finally. Never runs a PR-controlled script — only fixed read verbs are passed.
  */
 declare function runGitCapture(directory: string, args: string[]): Promise<string | null>;
+/**
+ * Bound a one-line finding summary for display: strip control characters and
+ * the markdown/HTML metacharacters (same character classes as
+ * {@link boundUntrusted} — receipt fields are lane-agent-authored text and a
+ * schema cap bounds length, not content), collapse whitespace, then cap to
+ * the char limit without splitting a surrogate pair (code-point-aware, unlike
+ * a bare `slice`).
+ */
+declare function boundFindingLine(line: string): string;
 declare function resolveWorkflowGateSession(directory: string, sessionID: string): Promise<string | null>;
 export declare const pr_workflow_status: ReturnType<typeof createSwarmTool>;
 /** Test seam mirroring gh-evidence's `_internals` (issue #507 DI convention). */
@@ -34,5 +52,10 @@ export declare const _internals: {
     classifyGitState: typeof classifyPrWorkflowGitState;
     truncateToByteBudget: typeof truncateToByteBudget;
     resolveWorkflowGateSession: typeof resolveWorkflowGateSession;
+    /** Batch receipt lookup for the lane-coverage projection (F-01 test seam). */
+    findByBatchIdDetailed: typeof findByBatchIdDetailed;
+    /** Pure field sanitizers (direct hostile-input unit suites). */
+    boundUntrusted: typeof boundUntrusted;
+    boundFindingLine: typeof boundFindingLine;
 };
 export {};

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "opencode-swarm",
-	"version": "7.193.0",
+	"version": "7.195.0",
 	"description": "Architect-centric agentic swarm plugin for OpenCode - hub-and-spoke orchestration with SME consultation, code generation, and QA review",
 	"main": "dist/index.js",
 	"types": "dist/index.d.ts",

package/dist/cli/curator-llm-factory-8gt3mvg9.js

@@ -1,82 +0,0 @@
-// @bun
-import {
-  createCuratorLLMDelegate
-} from "./index-jx80ed6g.js";
-import"./index-bkpyff61.js";
-import"./index-4mq8r4be.js";
-import"./index-ma71x1aa.js";
-import"./index-j2vv0h13.js";
-import"./index-668npagp.js";
-import"./index-7njm9jk1.js";
-import"./index-txdagbh7.js";
-import"./index-prvancd0.js";
-import"./index-rdec0y3z.js";
-import"./index-d2emv8ap.js";
-import"./index-ymnathfr.js";
-import"./index-r3mffz78.js";
-import"./index-5n760fzn.js";
-import"./index-fd6b9f38.js";
-import"./index-hfnyykvm.js";
-import"./index-cjapb4hy.js";
-import"./index-6s3ab22d.js";
-import"./index-pt7h91zc.js";
-import"./index-q7sjgr1y.js";
-import"./index-nktwf2xm.js";
-import"./index-he2nyr7x.js";
-import"./index-6b295cas.js";
-import"./index-9nyqt3kb.js";
-import"./index-bgtfvst1.js";
-import"./index-v3bmwmqw.js";
-import"./index-4gacy6y6.js";
-import"./index-nz6ehpjn.js";
-import"./index-x2phyvcv.js";
-import"./index-g4fs6z4e.js";
-import"./index-6dn16dre.js";
-import"./index-tr47q89d.js";
-import"./index-cs0p582p.js";
-import"./index-m9zzj4ee.js";
-import"./index-84k9zzv1.js";
-import"./index-2s7dvzff.js";
-import"./index-jxs9qnad.js";
-import"./index-w2vbaczx.js";
-import"./index-rtry5xyf.js";
-import"./index-vbp35frb.js";
-import"./index-g22dps7x.js";
-import"./index-0725jpp7.js";
-import"./index-1p24bfn2.js";
-import"./index-5ev6f1gj.js";
-import"./index-qh93qjc2.js";
-import"./index-st80sjg7.js";
-import"./index-vg8sxg12.js";
-import"./index-0e318vgt.js";
-import"./index-ry16d01s.js";
-import"./index-rdm3px0g.js";
-import"./index-7pmbnjvj.js";
-import"./index-ndec8akq.js";
-import"./index-bg19b84s.js";
-import"./index-vzm6zre7.js";
-import"./index-ae75rja9.js";
-import"./index-3atkcqpb.js";
-import"./index-5rkh69f3.js";
-import"./index-e8k0v19e.js";
-import"./index-t7hmqgjy.js";
-import"./index-mtqwyjrg.js";
-import"./index-fzpt2djt.js";
-import"./index-ejhxgyvm.js";
-import"./index-7g4c7s5r.js";
-import"./index-hfssjyxs.js";
-import"./index-gx2vp8gx.js";
-import"./index-ngk8hpc3.js";
-import"./index-zxdawwc7.js";
-import"./index-1dx5zyss.js";
-import"./index-py8wpe81.js";
-import"./index-bk5tah7q.js";
-import"./index-gknz3bv1.js";
-import"./index-zpsr8rpc.js";
-import"./index-bzfn85kq.js";
-import"./index-9rwhas09.js";
-import"./index-zgwm4ryv.js";
-import"./index-a76rekgs.js";
-export {
-  createCuratorLLMDelegate
-};

package/dist/cli/guardrail-explain-hpgecr7x.js

@@ -1,83 +0,0 @@
-// @bun
-import {
-  handleGuardrailExplain
-} from "./index-cpt38e0e.js";
-import"./index-jx80ed6g.js";
-import"./index-bkpyff61.js";
-import"./index-4mq8r4be.js";
-import"./index-ma71x1aa.js";
-import"./index-j2vv0h13.js";
-import"./index-668npagp.js";
-import"./index-7njm9jk1.js";
-import"./index-txdagbh7.js";
-import"./index-prvancd0.js";
-import"./index-rdec0y3z.js";
-import"./index-d2emv8ap.js";
-import"./index-ymnathfr.js";
-import"./index-r3mffz78.js";
-import"./index-5n760fzn.js";
-import"./index-fd6b9f38.js";
-import"./index-hfnyykvm.js";
-import"./index-cjapb4hy.js";
-import"./index-6s3ab22d.js";
-import"./index-pt7h91zc.js";
-import"./index-q7sjgr1y.js";
-import"./index-nktwf2xm.js";
-import"./index-he2nyr7x.js";
-import"./index-6b295cas.js";
-import"./index-9nyqt3kb.js";
-import"./index-bgtfvst1.js";
-import"./index-v3bmwmqw.js";
-import"./index-4gacy6y6.js";
-import"./index-nz6ehpjn.js";
-import"./index-x2phyvcv.js";
-import"./index-g4fs6z4e.js";
-import"./index-6dn16dre.js";
-import"./index-tr47q89d.js";
-import"./index-cs0p582p.js";
-import"./index-m9zzj4ee.js";
-import"./index-84k9zzv1.js";
-import"./index-2s7dvzff.js";
-import"./index-jxs9qnad.js";
-import"./index-w2vbaczx.js";
-import"./index-rtry5xyf.js";
-import"./index-vbp35frb.js";
-import"./index-g22dps7x.js";
-import"./index-0725jpp7.js";
-import"./index-1p24bfn2.js";
-import"./index-5ev6f1gj.js";
-import"./index-qh93qjc2.js";
-import"./index-st80sjg7.js";
-import"./index-vg8sxg12.js";
-import"./index-0e318vgt.js";
-import"./index-ry16d01s.js";
-import"./index-rdm3px0g.js";
-import"./index-7pmbnjvj.js";
-import"./index-ndec8akq.js";
-import"./index-bg19b84s.js";
-import"./index-vzm6zre7.js";
-import"./index-ae75rja9.js";
-import"./index-3atkcqpb.js";
-import"./index-5rkh69f3.js";
-import"./index-e8k0v19e.js";
-import"./index-t7hmqgjy.js";
-import"./index-mtqwyjrg.js";
-import"./index-fzpt2djt.js";
-import"./index-ejhxgyvm.js";
-import"./index-7g4c7s5r.js";
-import"./index-hfssjyxs.js";
-import"./index-gx2vp8gx.js";
-import"./index-ngk8hpc3.js";
-import"./index-zxdawwc7.js";
-import"./index-1dx5zyss.js";
-import"./index-py8wpe81.js";
-import"./index-bk5tah7q.js";
-import"./index-gknz3bv1.js";
-import"./index-zpsr8rpc.js";
-import"./index-bzfn85kq.js";
-import"./index-9rwhas09.js";
-import"./index-zgwm4ryv.js";
-import"./index-a76rekgs.js";
-export {
-  handleGuardrailExplain
-};