pi-goal-list-loop-audit 0.28.15 → 0.28.16

package/extensions/reviewer.ts

@@ -213,12 +213,22 @@ export interface ReviewerOutcome {
   reportPath?: string;
   enqueued: number;
   proposed: number;
-  /** v0.27.5: the cascade step that actually fired (notify-and-idle | convert-findings-to-list | queue-leftovers | fire-audit-on-clean | propose-goal | aggressive-relaunch | report-only) — surfaces in /goal status and tests. */
+  /** v0.27.5: the cascade step that actually fired (notify-and-idle | convert-findings-to-list | queue-leftovers | fire-audit-on-clean | propose-goal | aggressive-relaunch | duplicate-suppressed) — surfaces in /goal status and tests. v0.28.16: "duplicate-suppressed" = the clean completion WAS a regression scan, so the follow-up scan was deduped. */
   cascadeStep?: string;
 }
 
 export const REVIEWER_REFIRE_WINDOW_MS = 5 * 60_000;
 
+/** v0.28.16: normalize an objective for duplicate-compare — lowercase,
+ * goal-ids (yyyyMMddHHmmss-xxxxxx) become <id>, whitespace collapses. */
+export function normalizeObjective(s: string): string {
+  return s
+    .toLowerCase()
+    .replace(/\b\d{14}-[a-z0-9]{6}\b/g, "<id>")
+    .replace(/\s+/g, " ")
+    .trim();
+}
+
 /** The reviewer lifecycle (contract item 1). */
 export function runReviewer(
   config: ReviewerConfig,
@@ -310,7 +320,16 @@ export function runReviewer(
   // aggressive mode → relaunch the audit goal directly (no Confirm).
   if (findings.length === 0 && config.cascade.includes("fire-audit-on-clean")) {
     const auditObjective = `Post-completion regression scan after ${source.goalId} (${config.auditScope})`;
-    if (aggressive) {
+    // v0.28.16: duplicate-scan dedupe — the reviewer fires ON completion,
+    // so source.objective IS the most recent completion. If the goal that
+    // just completed was itself this same scan (normalized: goal-ids
+    // stripped), the proposal would be a scan-of-a-scan — the cascade loop
+    // that fired twice on 2026-07-28. Suppress the proposal/enqueue (the
+    // report is still written; the ledger records the reason).
+    if (normalizeObjective(source.objective) === normalizeObjective(auditObjective)) {
+      deps.ledger("reviewer_suppressed", { reason: "duplicate-scan", goalId: source.goalId, objective: auditObjective });
+      cascadeStep = "duplicate-suppressed";
+    } else if (aggressive) {
       if (deps.proposeGoal(auditObjective, "aggressive postaudit: clean completion — relaunching the regression scan as /goal")) {
         proposed++;
       }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-goal-list-loop-audit",
-  "version": "0.28.15",
+  "version": "0.28.16",
   "description": "Goal. Loop. Audit. Done. \u2014 a pi-coding-agent extension that supervises long-running work, with isolated auditor on each completion. Beat bamboozling by design: the auditor runs in a fresh session with no extensions, no skills, no editor \u2014 only the read tools needed to verify your goal.",
   "license": "MIT",
   "author": "dracon",