@windyroad/retrospective 0.23.0-preview.521 → 0.23.1

package/.claude-plugin/plugin.json

@@ -66,5 +66,5 @@
     }
   },
   "name": "wr-retrospective",
-  "version": "0.23.0"
+  "version": "0.23.1"
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@windyroad/retrospective",
-  "version": "0.23.0-preview.521",
+  "version": "0.23.1",
   "description": "Session retrospectives that update briefings and create problem tickets",
   "bin": {
     "windyroad-retrospective": "./bin/install.mjs"

package/scripts/check-briefing-budgets.sh

@@ -38,13 +38,25 @@
 #   2. Then all MUST_SPLIT lines, sorted by basename.
 #
 # Output is empty (no lines) when no topic files exceed the threshold.
-# README.md is excluded from the scan — it is Tier 2, not Tier 3.
+#
+# Two scan exclusions:
+#   1. README.md — the Tier 2 index, not a Tier 3 topic file.
+#   2. *-archive*.md — rotation SINKS produced by split-by-date.
+#      Archives are loaded on-demand only (per docs/briefing/README.md
+#      "load alongside when full historical context needed"); they are
+#      NOT session-start-loaded and therefore sit outside the per-topic
+#      session-surface budget that ADR-040 Tier 3 governs. Flagging them
+#      forces a churn-vs-defer choice with no correct destination
+#      (a sink has no chronologically-correct sibling slot for further
+#      splitting). P322 closes this — archives are explicitly out of
+#      scope for the OVER / MUST_SPLIT pass.
 #
 # Read-only — does NOT mutate any briefing file. Rotation is surfaced
 # to the user via run-retro Step 3.
 #
 # @problem P099 (initial OVER advisory)
 # @problem P145 (MUST_SPLIT escalation — closes the defer-recurrence gap)
+# @problem P322 (archive-sink exclusion — closes the sink-rotation forced-choice)
 # @adr ADR-040 (Session-start briefing surface — Tier 3 budget; this
 #   script promotes Tier 3 from informational to advisory enforcement;
 #   MUST_SPLIT promotes the 2× reassessment trigger to per-cycle)
@@ -87,6 +99,12 @@ for path in "${files[@]}"; do
   if [ "$base" = "README.md" ]; then
     continue
   fi
+  # *-archive*.md files are rotation SINKS — loaded on-demand only,
+  # NOT session-start-loaded; out of scope for the session-surface
+  # budget. See P322 + header rationale.
+  case "$base" in
+    *-archive*.md) continue ;;
+  esac
   bytes=$(wc -c < "$path" | tr -d ' ')
   entries+=("$base $bytes")
 done

package/scripts/test/check-briefing-budgets.bats

@@ -146,6 +146,48 @@ write_briefing_entry() {
   [ -z "$output" ]
 }
 
+# ── Archive-sink exclusion (P322) ───────────────────────────────────────────
+#
+# Archive files match `*-archive*.md` and are rotation SINKS — destination
+# of split-by-date — loaded on-demand only, NOT session-start-loaded.
+# Holding them to the per-topic session-surface budget forces a churn-vs-
+# defer choice with no correct answer (re-rotating a sink has no
+# chronologically-correct destination among existing siblings). The
+# session-start surface budget (ADR-040 Tier 3) targets session-start-
+# loaded files only. P322 closes this by excluding `*-archive*.md` from
+# the OVER / MUST_SPLIT pass — same shape as the README.md exclusion.
+#
+# @problem P322
+
+@test "check-briefing-budgets: archive file at or above threshold is excluded (P322 rotation sink)" {
+  mkdir -p "$FIXTURE_DIR/briefing"
+  # Archive sink that would otherwise trip OVER + MUST_SPLIT
+  write_briefing_entry "$FIXTURE_DIR/briefing/governance-workflow-archive.md" 20000
+  run "$SCRIPT" "$FIXTURE_DIR/briefing"
+  [ "$status" -eq 0 ]
+  [ -z "$output" ]
+}
+
+@test "check-briefing-budgets: archive variant -archive-2026-05.md is also excluded" {
+  mkdir -p "$FIXTURE_DIR/briefing"
+  # Date-suffixed archive sibling created by split-by-date rotation
+  write_briefing_entry "$FIXTURE_DIR/briefing/hooks-and-gates-archive-2026-05.md" 12000
+  run "$SCRIPT" "$FIXTURE_DIR/briefing"
+  [ "$status" -eq 0 ]
+  [ -z "$output" ]
+}
+
+@test "check-briefing-budgets: non-archive over-threshold file is still flagged alongside archive exclusion" {
+  mkdir -p "$FIXTURE_DIR/briefing"
+  # Mixed case: session-start surface file IS flagged; archive sink IS NOT.
+  write_briefing_entry "$FIXTURE_DIR/briefing/active-topic.md" 8000
+  write_briefing_entry "$FIXTURE_DIR/briefing/active-topic-archive.md" 8000
+  run "$SCRIPT" "$FIXTURE_DIR/briefing"
+  [ "$status" -eq 0 ]
+  echo "$output" | grep -E "^OVER active-topic.md bytes=[0-9]+ threshold=5120"
+  ! echo "$output" | grep -q "active-topic-archive.md"
+}
+
 # ── Configurable threshold via env var ──────────────────────────────────────
 
 @test "check-briefing-budgets: BRIEFING_TIER3_MAX_BYTES env var overrides default" {