opencode-swarm 7.134.0 → 7.134.1

package/dist/prm/escalation.d.ts

@@ -50,19 +50,9 @@ export declare class EscalationTracker {
     getState(): EscalationState;
     /**
      * Resets all escalation counts and flags to their default values.
-     * Clears pattern counts, corrections pending, and all flags.
+     * Clears pattern counts and all flags.
      */
     reset(): void;
-    /**
-     * Returns defensive copies of all pending course corrections.
-     *
-     * @returns Array of pending CourseCorrection objects
-     */
-    getPendingCorrections(): CourseCorrection[];
-    /**
-     * Clears all pending course corrections.
-     */
-    clearPendingCorrections(): void;
     /**
      * Returns whether a hard stop is pending.
      * This flag is read by messagesTransform to halt agent execution.

package/dist/prm/index.d.ts

@@ -75,6 +75,7 @@ interface ResettablePrmSessionState {
     prmLastPatternDetected?: unknown;
     prmHardStopPending?: boolean;
     prmTrajectoryStep?: number;
+    prmInjectedAdvisoryKeys?: Set<string>;
     replayArtifactPath?: string | null;
 }
 export declare function resetPrmSessionState(session: ResettablePrmSessionState, sessionId?: string): void;

package/dist/prm/types.d.ts

@@ -115,6 +115,4 @@ export interface EscalationState {
     lastPatternDetected: PatternMatch | null;
     /** Whether a hard stop has been triggered */
     hardStopPending: boolean;
-    /** Queue of correction messages to inject */
-    correctionsPending: CourseCorrection[];
 }

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

@@ -32,6 +32,7 @@ export interface SerializedAgentSession {
         timestamp: number;
     } | null;
     partialGateWarningsIssuedForTask: string[];
+    completionGateWarnedForTask: string[];
     selfFixAttempted: boolean;
     selfCodingWarnedAtCount: number;
     catastrophicPhaseWarnings: number[];

package/dist/state.d.ts

@@ -169,6 +169,10 @@ export interface AgentSessionState {
     } | null;
     /** Task IDs for which partial gate warning has already been issued (prevents per-task spam) */
     partialGateWarningsIssuedForTask: Set<string>;
+    /** Task IDs for which the completion-gate violation advisory has already been issued
+     *  (issue #1976 B3: prevents re-injecting the identical, unactionable directive on
+     *  every subsequent Task tool call while the same task stays stuck in tests_run). */
+    completionGateWarnedForTask: Set<string>;
     /** Whether architect attempted self-fix write after gate failure */
     selfFixAttempted: boolean;
     /** Value of architectWriteCount at the time the self-coding warning was last injected.
@@ -329,6 +333,13 @@ export interface AgentSessionState {
     prmHardStopPending: boolean;
     /** Per-session escalation tracker instance (set lazily by PRM hook) */
     prmEscalationTracker?: EscalationTracker;
+    /** Cross-turn set of already-injected PRM advisory dedupe keys
+     *  (issue #1976 B1). The pushAdvisory helper only dedupes WITHIN a turn
+     *  (the drain clears pendingAdvisoryMessages each turn); this set suppresses
+     *  re-injecting the same pattern@level on subsequent tool calls until the
+     *  pattern's count advances escalation. Bounded by distinct (pattern, level)
+     *  pairs — at most (numPatterns × 3 levels). */
+    prmInjectedAdvisoryKeys: Set<string>;
     /** Active PR subscriptions for the background poller, keyed by `${repoFullName}::${prNumber}` */
     prSubscriptions: Map<string, PrSubscriptionState>;
     /**

package/dist/utils/advisory-queue.d.ts

@@ -0,0 +1,60 @@
+import type { AgentSessionState } from '../state';
+/**
+ * Shared advisory-queue push helper (issue #1976).
+ *
+ * Problem it fixes: `session.pendingAdvisoryMessages` is a bare `string[]`
+ * that ~70 producer sites pushed onto directly via `.push()`, with no shared
+ * gate. Re-firing triggers (per-tool-call, per-Task-call, per-poll, on retry)
+ * stacked near-identical or byte-identical advisories that the drain rendered
+ * verbatim with no dedupe or size bound — which produced the PR_REVIEW banner
+ * flood (55.3% of non-blank lines in a real transcript).
+ *
+ * This helper gives every producer a gate by construction:
+ *   1. empty/whitespace messages are skipped (drain hygiene),
+ *   2. near-identical messages already queued are suppressed (dedupe),
+ *   3. the queue is length-bounded (keep-latest) so volume is capped.
+ *
+ * IMPORTANT — dedupe scope. The advisory drain (`messagesTransform`) clears
+ * `pendingAdvisoryMessages = []` on every turn, so this helper's dedupe only
+ * suppresses duplicates that accumulate WITHIN a single turn (between pushes
+ * and the next drain). CROSS-turn re-injection (the dominant failure mode for
+ * per-tool/per-Task repeaters) is NOT handled here — those producers carry
+ * their own session-scoped "already said" state keyed on a stable identity.
+ * The drain-level byte budget (`messagesTransform`) bounds rendered output
+ * across all producers, including any not yet migrated to this helper.
+ */
+/**
+ * Maximum number of pending advisories retained in the queue at once
+ * (defense-in-depth, in addition to the drain byte budget). Keep-latest: when
+ * the cap is exceeded the OLDEST entry is dropped. The issue explicitly flags
+ * "keep earliest" as a priority inversion because high-value advisories tend
+ * to arrive LATE in a turn while low-value ones arrive early.
+ */
+export declare const MAX_PENDING_ADVISORIES = 25;
+/**
+ * Options for {@link pushAdvisory}.
+ */
+export interface PushAdvisoryOptions {
+    /**
+     * Stable semantic identity for dedupe. Use this when the rendered message
+     * embeds volatile fields (counts, step ranges, ids) that vary across
+     * logically-equivalent re-fires. When omitted, the full normalized message
+     * text is used as the fingerprint.
+     */
+    dedupeKey?: string;
+    /**
+     * Override the default queue length cap ({@link MAX_PENDING_ADVISORIES}).
+     */
+    maxPending?: number;
+}
+/**
+ * Push an advisory onto the session's pending queue with dedupe + length cap.
+ *
+ * @returns `true` if the message was enqueued, `false` if it was suppressed
+ *   (empty, or a near-identical/duplicate message was already queued). Callers
+ *   that maintain cross-turn "already said" state should record that state ONLY
+ *   when this returns `true` — and must NEVER early-return out of other
+ *   mandatory work (escalation counting, telemetry, replay) based on this
+ *   result. Suppression applies to the ADVISORY INJECTION only.
+ */
+export declare function pushAdvisory(session: Pick<AgentSessionState, 'pendingAdvisoryMessages'>, message: string, opts?: PushAdvisoryOptions): boolean;

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "opencode-swarm",
-	"version": "7.134.0",
+	"version": "7.134.1",
 	"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",