@codedrifters/configulator 0.0.235 → 0.0.236

package/lib/index.js

@@ -10588,7 +10588,7 @@ var requirementsWriterBundle = {
 // src/agent/bundles/requirements-reviewer.ts
 var requirementsReviewerSubAgent = {
   name: "requirements-reviewer",
-  description: "Audits and deprecates existing requirement documents (BR, FR, NFR, TR, ADR, SEC, DR, INT, OPS, UX, MT). In `req:review` mode, runs the 11-check audit (structural compliance, categorization, traceability, cross-reference integrity, sequence integrity, content quality, registry sync, decision-authority compliance, tier classification, cross-referencing conventions) and produces a report grouped by Critical / Warning / Info. In `req:deprecate` mode, transitions target documents to `Deprecated` or `Superseded`, updates the category index row, and files `req:write` follow-ups for every back-reference so the writer can rewrite them. One phase per session.",
+  description: "Audits and deprecates existing requirement documents (BR, FR, NFR, TR, ADR, SEC, DR, INT, OPS, UX, MT). In `req:review` mode, runs the 12-check audit (structural compliance, categorization, traceability, cross-reference integrity, sequence integrity, content quality, registry sync, decision-authority compliance, tier classification, cross-referencing conventions, stale `Proposed` ADR/TR decisions) and produces a report grouped by Critical / Warning / Info (plus a dedicated Stale decisions section). In `req:deprecate` mode, transitions target documents to `Deprecated` or `Superseded`, updates the category index row, and files `req:write` follow-ups for every back-reference so the writer can rewrite them. One phase per session.",
   model: AGENT_MODEL.POWERFUL,
   maxTurns: 80,
   platforms: { cursor: { exclude: true } },
@@ -10715,6 +10715,7 @@ var requirementsReviewerSubAgent = {
     "| `<REVIEW_REPORTS_ROOT>` | Where review reports are written | `docs/research/reviews/` |",
     `| \`<TEMPLATES_ROOT>\` | Where the writer bundle ships category templates (read-only for this agent) | \`${REQUIREMENTS_WRITER_PATHS.templatesRoot}\` |`,
     `| \`<STANDARDS_REF>\` | Standards & frameworks reference shipped by the writer bundle | \`${REQUIREMENTS_WRITER_PATHS.standardsRef}\` |`,
+    "| `<STALE_PROPOSED_THRESHOLD_DAYS>` | Age (in days) after which a `Proposed` ADR/TR is flagged as a stale decision by Check 12 | `14` |",
     "",
     "If `docs/project-context.md` specifies a different requirements",
     "tree, prefer that. Otherwise fall back to the defaults above.",
@@ -10764,7 +10765,7 @@ var requirementsReviewerSubAgent = {
     "",
     "## Review Checklist",
     "",
-    "Apply the following 11 checks. Each check produces zero or more",
+    "Apply the following 12 checks. Each check produces zero or more",
     "findings. Findings are graded Critical / Warning / Info using the",
     "severity ladder in the next section.",
     "",
@@ -11009,6 +11010,60 @@ var requirementsReviewerSubAgent = {
     "  reverse-link structure (e.g., a `referencedIn.meetings[]`",
     "  frontmatter block) and that structure is in active use.",
     "",
+    "### 12. Stale Proposed Decisions",
+    "",
+    "ADR and TR documents ship as `Status: Proposed` because they",
+    "encode architectural or technology decisions that require explicit",
+    "human acceptance (see Check 9 and the `requirements-writer`",
+    "decision-authority rules). Once `Proposed`, there is no automatic",
+    "signal that surfaces long-`Proposed` documents for decision \u2014 the",
+    "backlog silently accumulates. Check 12 surfaces them.",
+    "",
+    "For every document under `<REQUIREMENTS_ROOT>` whose category is",
+    "**ADR** or **TR** and whose `Status` is `Proposed`:",
+    "",
+    "- Determine the document's last-touched date. Prefer the YAML",
+    "  frontmatter `updated` field when present. If frontmatter does",
+    "  not carry an `updated` field, fall back to the most recent entry",
+    "  in the `## Revision History` section. If neither exists, fall",
+    "  back to `git log -1 --format=%cI -- <path>` for the file.",
+    "- Compare that date to today. If the document has carried",
+    "  `Status: Proposed` for longer than",
+    "  `<STALE_PROPOSED_THRESHOLD_DAYS>` days (default 14), record a",
+    "  **Warning** finding that names the document, the age in days,",
+    "  the threshold, and the source used for the last-touched date.",
+    "- Do **not** flag `Proposed` ADRs or TRs that are below the",
+    "  threshold \u2014 they are in active decision and not stale.",
+    "- Do **not** auto-transition any status. Deprecation and",
+    "  acceptance both remain human decisions: Check 12 only surfaces",
+    "  the backlog for the human running `/review-requirements` (or",
+    "  for the follow-up issue it files) to act on.",
+    "",
+    "The threshold placeholder `<STALE_PROPOSED_THRESHOLD_DAYS>` is",
+    "configurable \u2014 override it in the `req:review` issue body, in",
+    "`docs/project-context.md`, or via `agentConfig.rules`. The default",
+    "of 14 days sits above the orchestrator's existing 72h / 168h stale",
+    "thresholds for `status:in-progress` / `status:blocked` and fits",
+    "the pace at which architectural decisions are typically ratified.",
+    "",
+    "Findings from Check 12 describe a decision the human must make \u2014",
+    "not an edit to the requirement document itself. Follow-up routing",
+    "is therefore different from Checks 3 and 4:",
+    "",
+    "- Surface every stale `Proposed` ADR/TR in a distinct",
+    "  **Stale decisions** section in the review report (see Reporting",
+    "  Format below), separate from Critical / Warning / Info.",
+    "- File **one** follow-up issue covering the stale decisions found",
+    "  in this review. Title:",
+    "  `req(review): decide on stale Proposed ADR/TR documents`.",
+    "  Labels: `type:requirement`, `req:review`, `priority:medium`,",
+    "  `status:needs-attention`. Body must list each stale document",
+    "  path, its age in days, and a link back to the review report.",
+    "  Do **not** file one follow-up per document \u2014 humans triage the",
+    "  backlog in a single pass.",
+    "- When no stale documents are found, omit the Stale decisions",
+    "  section entirely and do not file the follow-up issue.",
+    "",
     "---",
     "",
     "## Severity Ladder",
@@ -11044,6 +11099,13 @@ var requirementsReviewerSubAgent = {
     "- Missing cross-tier traceability; missing Implementor / Customer",
     "  fields on Customer Workflow / Consumer Application requirements",
     "  when the project tracks those fields",
+    "- Stale `Proposed` ADR/TR documents that have carried the",
+    "  `Proposed` status longer than",
+    "  `<STALE_PROPOSED_THRESHOLD_DAYS>` (default 14) days. The",
+    "  underlying document is valid; only the absence of human",
+    "  decision is stale, so these are reported in the report's",
+    "  dedicated **Stale decisions** section rather than inline in",
+    "  the Warning list.",
     "",
     "### Info (Consider)",
     "",
@@ -11087,6 +11149,20 @@ var requirementsReviewerSubAgent = {
     "1. **[<file path>] <short title>** `Info`",
     "   ...",
     "",
+    "## Stale decisions",
+    "",
+    "*Omit this section entirely when Check 12 found no stale",
+    "`Proposed` ADR/TR documents.*",
+    "",
+    "| Document | Age (days) | Threshold | Last-touched source |",
+    "|----------|------------|-----------|---------------------|",
+    "| `<file path>` | N | <STALE_PROPOSED_THRESHOLD_DAYS> | frontmatter `updated` / revision history / git log |",
+    "",
+    "Human decision required for each row. A single follow-up",
+    "`req:review` issue (labels `type:requirement`, `req:review`,",
+    "`priority:medium`, `status:needs-attention`) is filed to",
+    "track triage; see Check 12 for the exact follow-up contract.",
+    "",
     "## Review Summary",
     "",
     "| Severity | Count |",
@@ -11094,6 +11170,7 @@ var requirementsReviewerSubAgent = {
     "| Critical | N |",
     "| Warning | N |",
     "| Info | N |",
+    "| Stale decisions | N |",
     "",
     "**Categories reviewed:** [list]",
     "**Documents reviewed:** N",
@@ -11155,7 +11232,7 @@ var requirementsReviewerSubAgent = {
     "1. Read scope from the `req:review` issue.",
     "2. Read the matching templates from `<TEMPLATES_ROOT>` for every",
     "   category in scope.",
-    "3. Run the 11 checks. For audits >10 documents, write the",
+    "3. Run the 12 checks. For audits >10 documents, write the",
     "   verification script to disk first and run it.",
     "4. Write the report to",
     "   `<REVIEW_REPORTS_ROOT>/review-<scope>-YYYY-MM-DD.md`.",
@@ -11208,6 +11285,28 @@ var requirementsReviewerSubAgent = {
     "- Adds `status:ready` (or `status:blocked` when the finding",
     "  declares a `Depends on: #N` on another open issue)",
     "",
+    "**Check 12 exception \u2014 stale `Proposed` ADR/TR triage.** Check 12",
+    "findings are the one case where follow-up flows through a new",
+    "`req:review` issue rather than `req:write` / `req:trace` /",
+    "`req:scan`. The finding is a decision a human must make, not an",
+    "edit to a requirement document, so:",
+    "",
+    "- File **one** aggregated follow-up (not one per document). The",
+    "  title is",
+    "  `req(review): decide on stale Proposed ADR/TR documents`.",
+    "- Labels: `type:requirement`, `req:review`, `priority:medium`,",
+    "  `status:needs-attention`.",
+    "- Body lists every stale document surfaced in this review with",
+    "  its path, age in days, and the threshold used, and links back",
+    "  to the Stale decisions section of the review report.",
+    "- This is the **only** case in which the reviewer opens a",
+    "  `req:review` issue \u2014 it is explicitly not self-referential",
+    "  because the new issue triages decisions rather than triggering",
+    "  another review.",
+    "- The `req:write` follow-up schema below does **not** apply to",
+    "  this Check 12 issue; the schema governs `req:write` payloads,",
+    "  and this follow-up is a `req:review` triage item.",
+    "",
     "### `req:write` follow-up schema",
     "",
     "Because this bundle is the second-largest producer of `req:write`",
@@ -11258,7 +11357,9 @@ var requirementsReviewerSubAgent = {
     "",
     "**Do NOT create:**",
     "",
-    "- `req:review` issues \u2014 that would be self-referential",
+    "- `req:review` issues for any reason other than the Check 12",
+    "  stale-decisions triage documented above \u2014 that would be",
+    "  self-referential",
     "- `bcm:*`, `people:*`, `company:*`, `software:*`, `research:*`,",
     "  or `industry:*` issues \u2014 those belong to their respective",
     "  bundles. If the review surfaces work for one of those bundles,",
@@ -11539,7 +11640,7 @@ var requirementsReviewerSubAgent = {
 };
 var reviewRequirementsSkill = {
   name: "review-requirements",
-  description: "Audit existing requirement documents (BR / FR / NFR / TR / ADR / SEC / DR / INT / OPS / UX / MT) for structural compliance, categorization, traceability, cross-reference integrity, sequence integrity, content quality, registry sync, decision-authority compliance, tier classification, and cross-referencing conventions. Supports four scopes (full audit, category, single document, targeted check) and dispatches the requirements-reviewer agent. Audits documents \u2014 never writes them. Soft dependency: expects requirement templates at `" + REQUIREMENTS_WRITER_PATHS.templatesRoot + "`, shipped by the requirements-writer bundle.",
+  description: "Audit existing requirement documents (BR / FR / NFR / TR / ADR / SEC / DR / INT / OPS / UX / MT) for structural compliance, categorization, traceability, cross-reference integrity, sequence integrity, content quality, registry sync, decision-authority compliance, tier classification, cross-referencing conventions, and stale `Proposed` ADR/TR decisions. Supports four scopes (full audit, category, single document, targeted check) and dispatches the requirements-reviewer agent. Audits documents \u2014 never writes them. Soft dependency: expects requirement templates at `" + REQUIREMENTS_WRITER_PATHS.templatesRoot + "`, shipped by the requirements-writer bundle.",
   disableModelInvocation: true,
   userInvocable: true,
   context: "fork",
@@ -11571,9 +11672,10 @@ var reviewRequirementsSkill = {
     "- `category:<slug>` \u2014 audit one category directory (e.g.,",
     "  `category:functional` audits every FR)",
     "- `doc:<path>` \u2014 audit one specific requirement file",
-    "- `check:<n>[,<n>...]` \u2014 run one or more checks (1\u201311) across the",
+    "- `check:<n>[,<n>...]` \u2014 run one or more checks (1\u201312) across the",
     "  documents in scope (e.g., `check:4` runs cross-reference",
-    "  integrity only)",
+    "  integrity only, `check:12` runs the stale-proposed-decision",
+    "  audit only)",
     "",
     "Optional extensions in the issue body:",
     "",
@@ -11581,6 +11683,9 @@ var reviewRequirementsSkill = {
     "- `requirements-root: <path>` \u2014 override `<REQUIREMENTS_ROOT>`",
     "- `targets: <glob>` \u2014 restrict the document set with a glob",
     "  pattern relative to `<REQUIREMENTS_ROOT>`",
+    "- `stale-proposed-threshold-days: <n>` \u2014 override the default",
+    "  14-day threshold used by Check 12 (stale `Proposed` ADR/TR",
+    "  decisions) for this review",
     "",
     "## Default Paths",
     "",
@@ -11599,10 +11704,12 @@ var reviewRequirementsSkill = {
     "   scope, the requirements root (or accept the default), and the",
     "   output path.",
     "2. Execute the review phase of the requirements-reviewer agent.",
-    "3. The agent runs the 11-check audit, writes a structured report",
-    "   grouped by Critical / Warning / Info, files follow-up issues",
-    "   for actionable findings, and comments on the `req:review`",
-    "   issue with the summary.",
+    "3. The agent runs the 12-check audit, writes a structured report",
+    "   grouped by Critical / Warning / Info (plus a separate",
+    "   **Stale decisions** section when Check 12 surfaces stale",
+    "   `Proposed` ADR/TR documents), files follow-up issues for",
+    "   actionable findings, and comments on the `req:review` issue",
+    "   with the summary.",
     "",
     "## Audits >10 Documents",
     "",
@@ -11622,6 +11729,11 @@ var reviewRequirementsSkill = {
     "- Follow-up GitHub issues for every Critical and Warning finding,",
     "  carrying the appropriate phase label (`req:write`, `req:trace`,",
     "  or `req:scan`) for the downstream agent that should act",
+    "- An aggregated `req:review` follow-up issue labeled",
+    "  `status:needs-attention` when Check 12 surfaces stale `Proposed`",
+    "  ADR/TR documents (one issue per review, not one per document);",
+    "  the review report also includes a dedicated **Stale decisions**",
+    "  section in that case",
     "- A summary comment on the originating `req:review` issue"
   ].join("\n")
 };
@@ -11729,7 +11841,7 @@ var deprecateRequirementSkill = {
 };
 var requirementsReviewerBundle = {
   name: "requirements-reviewer",
-  description: "Requirements reviewer agent bundle. Audits existing requirement documents (BR, FR, NFR, TR, ADR, SEC, DR, INT, OPS, UX, MT) via the 11-check review phase, and transitions requirements to `Deprecated` or `Superseded` via the deprecation phase (narrow writes to Status, Revision History, Superseded-by, and category index row, with `req:write` follow-ups for every back-reference). Reads templates from the requirements-writer bundle's reference directory; ships no templates of its own.",
+  description: "Requirements reviewer agent bundle. Audits existing requirement documents (BR, FR, NFR, TR, ADR, SEC, DR, INT, OPS, UX, MT) via the 12-check review phase (including Check 12: stale `Proposed` ADR/TR decisions), and transitions requirements to `Deprecated` or `Superseded` via the deprecation phase (narrow writes to Status, Revision History, Superseded-by, and category index row, with `req:write` follow-ups for every back-reference). Reads templates from the requirements-writer bundle's reference directory; ships no templates of its own.",
   appliesWhen: () => true,
   rules: [
     {
@@ -11754,14 +11866,26 @@ var requirementsReviewerBundle = {
         "Use `/review-requirements <scope>` to audit existing requirement",
         "documents. Four scopes are supported: `full` (every document),",
         "`category:<slug>` (one category directory), `doc:<path>` (one",
-        "specific document), and `check:<n>` (one or more of the 11",
+        "specific document), and `check:<n>` (one or more of the 12",
         "checks across the documents in scope).",
         "",
+        "Check 12 flags ADR and TR documents that have carried",
+        "`Status: Proposed` longer than",
+        "`<STALE_PROPOSED_THRESHOLD_DAYS>` days (default 14, configurable",
+        "via the `req:review` issue body, `docs/project-context.md`, or",
+        "`agentConfig.rules`). Stale decisions are reported in a separate",
+        "**Stale decisions** section of the review report and trigger one",
+        "aggregated `req:review` follow-up issue labeled",
+        "`status:needs-attention` so a human can triage the backlog. The",
+        "reviewer never auto-transitions status \u2014 acceptance and",
+        "deprecation both remain human decisions.",
+        "",
         "The review phase produces **review reports and follow-up issues",
         "only** \u2014 it never edits requirement documents, capability",
         "models, or research notes. It files follow-up issues with the",
-        "appropriate phase label (`req:write`, `req:trace`, or",
-        "`req:scan`) for the downstream agent to pick up.",
+        "appropriate phase label (`req:write`, `req:trace`, `req:scan`,",
+        "or \u2014 only for Check 12 stale-decision triage \u2014 `req:review`)",
+        "for the downstream agent or human to pick up.",
         "",
         "For audits covering more than 10 documents, the reviewer writes",
         "a read-only Python verification script alongside the report and",