@codedrifters/configulator 0.0.232 → 0.0.233

package/lib/index.mjs

@@ -10590,6 +10590,16 @@ var requirementsReviewerSubAgent = {
     "- Bidirectional links exist where expected (if A references B in",
     "  Traceability, B should reference A)",
     "",
+    "Findings from this check describe edits inside an existing",
+    "requirement document (the broken link lives in the body of the",
+    "doc and must be rewritten or removed). Route follow-up issues to",
+    "`req:write` so the `requirements-writer` \u2014 the only agent scoped",
+    "to edit under `<REQUIREMENTS_ROOT>` \u2014 picks them up. Do **not**",
+    "route Check 4 findings to `req:trace`; the analyst bundle does",
+    "not edit requirement documents and the issue would sit",
+    "unresolved. See the Follow-Up Issues section below for the full",
+    "Check 3 vs. Check 4 routing split.",
+    "",
     "### 5. Sequence Number Integrity",
     "",
     "Within each category directory under `<REQUIREMENTS_ROOT>`:",
@@ -10915,12 +10925,23 @@ var requirementsReviewerSubAgent = {
     "- Carries `type:requirement` (the type owned by the upstream",
     "  `requirements-analyst` bundle)",
     "- Carries the appropriate phase label for the work needed:",
-    "  - Structural / content fixes that an existing document needs \u2192",
-    "    `req:write` (the `requirements-writer` agent picks it up to",
-    "    revise the document)",
-    "  - Missing or broken traceability links \u2192 `req:trace` (the",
-    "    `requirements-analyst` agent picks it up to backfill",
-    "    traceability)",
+    "  - Structural / content fixes that an existing requirement",
+    "    document needs (including broken internal cross-references",
+    "    from Check 4 \u2014 e.g., a `See: [FR-012](...)` link pointing at",
+    "    a renamed or deleted requirement) \u2192 `req:write` (the",
+    "    `requirements-writer` agent picks it up to revise the",
+    "    document). Only the writer is scoped to edit requirement",
+    "    documents, so anything that requires an in-place edit under",
+    "    `<REQUIREMENTS_ROOT>` routes here.",
+    "  - Missing traceability in **source documents** \u2014 BCM model",
+    "    docs, competitive-analysis notes, product-roadmap entries,",
+    "    meeting extracts \u2014 that should link forward to an existing",
+    "    requirement (Check 3: orphaned requirements whose fix lives",
+    "    in the source tree, not the requirement doc) \u2192 `req:trace`",
+    "    (the `requirements-analyst` agent picks it up to backfill",
+    "    source-doc traceability). **Do not** route Check 4 broken",
+    "    cross-references here \u2014 the analyst bundle never writes to",
+    "    `<REQUIREMENTS_ROOT>` and the issue would sit unresolved.",
     "  - A new requirement is needed (e.g., a `Proposed` ADR is missing",
     "    for a deferred technology choice) \u2192 `req:scan` (the analyst",
     "    runs a scoped scan to confirm the gap, then drafts the new",