@open-agent-toolkit/cli 0.2.79 → 0.3.0

package/assets/skills/oat-review-receive-remote/SKILL.md

@@ -5,7 +5,7 @@ disable-model-invocation: true
 user-invocable: true
 allowed-tools: Read, Write, Bash, Glob, Grep, AskUserQuestion
 metadata:
-  version: 1.3.1
+  version: 1.3.2
 ---
 
 # Remote Review Receive (Ad-hoc GitHub PR)
@@ -71,8 +71,8 @@ Normalize every finding to this shape:
 
 ```yaml
 finding:
-  id: "C1" | "I1" | "M1" | "m1"
-  severity: critical | important | medium | minor
+  id: "C1" | "H1" | "M1" | "L1"
+  severity: critical | high | medium | low
   title: string
   file: string | null
   line: number | null
@@ -86,9 +86,9 @@ finding:
 Severity conventions:
 
 - `critical`: Broken behavior, security risk, or missing P0 requirement.
-- `important`: Missing P1 requirement, major robustness issue.
+- `high`: Missing P1 requirement, major robustness issue.
 - `medium`: Meaningful but non-blocking quality/maintainability issue.
-- `minor`: Cosmetic/style/documentation issue.
+- `low`: Cosmetic/style/documentation issue.
 
 ## Process
 
@@ -126,16 +126,16 @@ For each item in JSON:
 
 - Capture `type` (`review_comment`, `issue_comment`, `review`).
 - Map location fields (`path`, `line`) when present.
-- Use comment body + context to assign severity (`critical`, `important`, `medium`, `minor`).
-- Treat `CHANGES_REQUESTED` review state as a strong hint toward `important+`, not an automatic override.
+- Use comment body + context to assign severity (`critical`, `high`, `medium`, `low`).
+- Treat `CHANGES_REQUESTED` review state as a strong hint toward `high+`, not an automatic override.
 - Normalize into the shared findings model.
 
 ID assignment per severity bucket:
 
 - Critical: `C1`, `C2`, ...
-- Important: `I1`, `I2`, ...
+- High: `H1`, `H2`, ...
 - Medium: `M1`, `M2`, ...
-- Minor: `m1`, `m2`, ...
+- Low: `L1`, `L2`, ...
 
 ### Step 4: Present Findings Overview and Triage
 
@@ -146,21 +146,21 @@ Before triage prompts, output:
 
 Disposition options per finding:
 
-- `convert` (default for critical/important/medium/minor)
+- `convert` (default for critical/high/medium/low)
 - `defer`
 - `dismiss`
 
 Rules:
 
-- Require concrete rationale for `defer`/`dismiss` at any severity, including minor. Small findings are usually cheaper to fix inline than to track as backlog items, so a minor `defer` must be justified just like any other deferral.
-- For any deferral (minor included), require a concrete reason (duplicate, dependency, explicit out-of-scope follow-up, risky churn).
+- Require concrete rationale for `defer`/`dismiss` at any severity, including low. Small findings are usually cheaper to fix inline than to track as backlog items, so a low `defer` must be justified just like any other deferral.
+- For any deferral (low included), require a concrete reason (duplicate, dependency, explicit out-of-scope follow-up, risky churn).
 
 ### Step 5: Generate Standalone Task List
 
 Task entry format:
 
 ```markdown
-- [ ] [important] Add null-guard in OAuth callback parser (`packages/auth/src/callback.ts:142`) - Validate provider payload before dereference.
+- [ ] [high] Add null-guard in OAuth callback parser (`packages/auth/src/callback.ts:142`) - Validate provider payload before dereference.
 ```
 
 Output modes:

package/assets/skills/oat-wave-execute/SKILL.md

@@ -6,7 +6,7 @@ disable-model-invocation: false
 user-invocable: true
 allowed-tools: Read, Write, Edit, Bash, Grep, Glob, Task
 metadata:
-  version: 1.9.4
+  version: 1.9.5
 ---
 
 # Execute a Wave of External Plans
@@ -181,7 +181,7 @@ excluded from groups) — never mid-run. **Reconciliation contract:** when recon
 reveals a stale plan premise, the reconciliation must be NON-NARROWING (WHERE
 the work happens may change; WHAT must be true may not), recorded exactly once
 in the Drift Refresh Record with pointer-only references elsewhere — a
-reconciliation that waives a source-plan requirement is a plan-gate Important
+reconciliation that waives a source-plan requirement is a plan-gate High
 (wave-4 evidence).
 
 Recon is delegated; the Drift Refresh Record entries and any reconciliation

package/assets/templates/plan.md

@@ -189,7 +189,7 @@ cell; never truncate a widened row back to five columns.
 - `received`: review artifact exists (not yet converted into fix tasks)
 - `fixes_added`: fix tasks were added to the plan (work queued)
 - `fixes_completed`: fix tasks implemented, awaiting re-review
-- `passed`: re-review run and recorded as passing (no Critical/Important)
+- `passed`: re-review run and recorded as passing (no Critical/High)
 
 ---
 

package/dist/commands/gate/__fixtures__/fake-runtime.mjs

@@ -55,9 +55,9 @@ async function writeArtifact() {
         `oat_invocation_reasoning_effort: ${promptValue('oat_invocation_reasoning_effort')}`,
         `oat_invocation_source: ${promptValue('oat_invocation_source')}`,
         'oat_review_critical_count: 0',
-        'oat_review_important_count: 0',
+        'oat_review_high_count: 0',
         'oat_review_medium_count: 0',
-        'oat_review_minor_count: 0',
+        'oat_review_low_count: 0',
         '---',
         '',
         '# Fake runtime review',
@@ -68,7 +68,7 @@ async function writeArtifact() {
         '',
         'None',
         '',
-        '### Important',
+        '### High',
         '',
         'None',
         '',
@@ -76,7 +76,7 @@ async function writeArtifact() {
         '',
         'None',
         '',
-        '### Minor',
+        '### Low',
         '',
         'None',
         '',

package/dist/commands/gate/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/commands/gate/index.ts"],"names":[],"mappings":"AAaA,OAAO,EAEL,KAAK,cAAc,EACnB,KAAK,aAAa,EACnB,MAAM,sBAAsB,CAAC;AAC9B,OAAO,EACL,gBAAgB,EAKhB,KAAK,qBAAqB,EAE3B,MAAM,8BAA8B,CAAC;AAYtC,OAAO,EAWL,KAAK,UAAU,EAIf,KAAK,SAAS,EACd,KAAK,cAAc,EACnB,KAAK,UAAU,EAEhB,MAAM,oBAAoB,CAAC;AAE5B,OAAO,EAOL,KAAK,cAAc,EACpB,MAAM,iBAAiB,CAAC;AAWzB,OAAO,EAEL,KAAK,WAAW,EACjB,MAAM,4BAA4B,CAAC;AACpC,OAAO,EAEL,KAAK,kBAAkB,EACvB,KAAK,kBAAkB,EAExB,MAAM,gCAAgC,CAAC;AAExC,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAGpC,OAAO,EACL,uBAAuB,EAExB,MAAM,mBAAmB,CAAC;AAC3B,OAAO,EACL,wBAAwB,EACxB,oBAAoB,EACpB,oBAAoB,EACpB,wBAAwB,EAEzB,MAAM,oBAAoB,CAAC;AAC5B,OAAO,EAEL,KAAK,iBAAiB,EACtB,KAAK,gBAAgB,EACtB,MAAM,iBAAiB,CAAC;AAEzB,OAAO,EACL,sBAAsB,EAIvB,MAAM,kBAAkB,CAAC;AAG1B,UAAU,uBAAuB;IAC/B,mBAAmB,EAAE,CAAC,OAAO,EAAE,aAAa,KAAK,cAAc,CAAC;IAChE,kBAAkB,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,OAAO,CAAC,MAAM,CAAC,CAAC;IACrD,aAAa,EAAE,CAAC,QAAQ,EAAE,MAAM,KAAK,OAAO,CAAC,SAAS,CAAC,CAAC;IACxD,cAAc,EAAE,CAAC,QAAQ,EAAE,MAAM,EAAE,MAAM,EAAE,SAAS,KAAK,OAAO,CAAC,IAAI,CAAC,CAAC;IACvE,kBAAkB,EAAE,CAAC,QAAQ,EAAE,MAAM,KAAK,OAAO,CAAC,cAAc,CAAC,CAAC;IAClE,mBAAmB,EAAE,CACnB,QAAQ,EAAE,MAAM,EAChB,MAAM,EAAE,cAAc,KACnB,OAAO,CAAC,IAAI,CAAC,CAAC;IACnB,cAAc,EAAE,CAAC,aAAa,EAAE,MAAM,KAAK,OAAO,CAAC,UAAU,CAAC,CAAC;IAC/D,eAAe,EAAE,CAAC,aAAa,EAAE,MAAM,EAAE,MAAM,EAAE,UAAU,KAAK,OAAO,CAAC,IAAI,CAAC,CAAC;IAC9E,sBAAsB,EAAE,CACtB,QAAQ,EAAE,MAAM,EAChB,aAAa,EAAE,MAAM,EACrB,GAAG,EAAE,MAAM,CAAC,UAAU,KACnB,OAAO,CAAC,cAAc,CAAC,CAAC;IAC7B,uBAAuB,EAAE,OAAO,uBAAuB,CAAC;IACxD,wBAAwB,EAAE,OAAO,wBAAwB,CAAC;IAC1D,oBAAoB,EAAE,OAAO,oBAAoB,CAAC;IAClD,wBAAwB,EAAE,OAAO,wBAAwB,CAAC;IAC1D,oBAAoB,EAAE,OAAO,oBAAoB,CAAC;IAClD,UAAU,EAAE,CACV,OAAO,EAAE,MAAM,EACf,IAAI,EAAE,MAAM,EAAE,EACd,OAAO,EAAE,iBAAiB,KACvB,OAAO,CAAC,gBAAgB,CAAC,CAAC;IAC/B,sBAAsB,EAAE,OAAO,sBAAsB,CAAC;IACtD,gBAAgB,EAAE,OAAO,gBAAgB,CAAC;IAC1C,UAAU,EAAE,MAAM,CAAC,UAAU,CAAC;IAC9B,kBAAkB,EAAE,CAClB,IAAI,EAAE,MAAM,EACZ,MAAM,EAAE,aAAa,EACrB,IAAI,EAAE,CAAC,OAAO,EAAE,MAAM,KAAK,IAAI,KAC5B,OAAO,CAAC,OAAO,CAAC,CAAC;IACtB,mBAAmB,EAAE,CACnB,IAAI,EAAE,MAAM,EACZ,IAAI,EAAE,CAAC,OAAO,EAAE,MAAM,KAAK,IAAI,KAC5B,OAAO,CAAC,IAAI,CAAC,CAAC;IACnB,0BAA0B,EAAE,CAC1B,IAAI,EAAE,MAAM,EACZ,OAAO,EAAE,qBAAqB,EAC9B,IAAI,EAAE,CAAC,OAAO,EAAE,MAAM,KAAK,IAAI,KAC5B,OAAO,CAAC,OAAO,CAAC,CAAC;IACtB,KAAK,EAAE,CAAC,EAAE,EAAE,MAAM,KAAK,OAAO,CAAC,IAAI,CAAC,CAAC;IACrC,eAAe,EAAE,CAAC,OAAO,EAAE,MAAM,KAAK,IAAI,CAAC;CAC5C;AAuDD,KAAK,iBAAiB,GAClB,KAAK,GACL,QAAQ,GACR,QAAQ,GACR,KAAK,GACL,eAAe,GACf,SAAS,CAAC;AAOd,UAAU,aAAa;IACrB,KAAK,EAAE,MAAM,CAAC;IACd,QAAQ,EAAE,MAAM,CAAC;IACjB,OAAO,EAAE,MAAM,CAAC;IAChB,UAAU,EAAE,MAAM,GAAG,IAAI,CAAC;IAC1B,WAAW,EAAE,MAAM,GAAG,IAAI,CAAC;IAC3B,OAAO,EAAE,MAAM,CAAC;IAChB,SAAS,EAAE,MAAM,CAAC;IAClB,QAAQ,EAAE,MAAM,CAAC;IACjB,YAAY,EAAE,iBAAiB,CAAC;CACjC;AAcD,KAAK,kBAAkB,GAAG,aAAa,GAAG,cAAc,GAAG,MAAM,CAAC;AAClE,KAAK,qBAAqB,GACtB,kBAAkB,GAClB,4BAA4B,GAC5B,2CAA2C,GAC3C,kBAAkB,CAAC;AA4CvB,MAAM,WAAW,kBAAkB;IACjC,EAAE,EAAE,MAAM,CAAC;IACX,MAAM,EAAE,UAAU,CAAC;IACnB,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,WAAW,CAAC;IACpB,SAAS,CAAC,EAAE,qBAAqB,CAAC;IAClC,uBAAuB,CAAC,EAAE,OAAO,CAAC;CACnC;AAED,UAAU,oBAAoB;IAC5B,KAAK,EAAE,MAAM,CAAC;IACd,UAAU,EAAE,kBAAkB,CAAC;IAC/B,UAAU,EAAE,kBAAkB,CAAC;IAC/B,MAAM,EAAE,WAAW,CAAC;IACpB,aAAa,EAAE,WAAW,EAAE,CAAC;IAC7B,kBAAkB,CAAC,EAAE,MAAM,EAAE,CAAC;IAC9B,sBAAsB,CAAC,EAAE,MAAM,CAAC;IAChC,kBAAkB,EAAE,OAAO,CAAC;IAC5B,MAAM,EAAE,MAAM,GAAG,OAAO,GAAG,mBAAmB,GAAG,aAAa,GAAG,SAAS,CAAC;CAC5E;AAED,UAAU,qBAAqB;IAC7B,KAAK,EAAE,kBAAkB,CAAC;IAC1B,QAAQ,EAAE,qBAAqB,CAAC;IAChC,QAAQ,EAAE;QACR,KAAK,EAAE,MAAM,CAAC;QACd,UAAU,EAAE,kBAAkB,CAAC;QAC/B,UAAU,EAAE,kBAAkB,CAAC;QAC/B,MAAM,EAAE,WAAW,CAAC;QACpB,MAAM,EAAE,oBAAoB,CAAC,QAAQ,CAAC,CAAC;QACvC,aAAa,EAAE,WAAW,EAAE,CAAC;QAC7B,kBAAkB,CAAC,EAAE,MAAM,EAAE,CAAC;QAC9B,sBAAsB,CAAC,EAAE,MAAM,CAAC;KACjC,CAAC;IACF,QAAQ,EAAE;QACR,MAAM,EAAE,MAAM,CAAC;QACf,OAAO,EAAE,MAAM,CAAC;QAChB,MAAM,EAAE,WAAW,CAAC;QACpB,KAAK,CAAC,EAAE,MAAM,CAAC;KAChB,CAAC;IACF,OAAO,CAAC,EAAE,MAAM,CAAC;CAClB;AAs2CD,wBAAgB,gBAAgB,CAC9B,QAAQ,EAAE,QAAQ,CAAC,MAAM,CAAC,MAAM,EAAE,UAAU,CAAC,CAAC,EAC9C,cAAc,EAAE,MAAM,EACtB,KAAK,EAAE,kBAAkB,EACzB,gBAAgB,CAAC,EAAE,oBAAoB,GACtC,kBAAkB,GAAG,IAAI,CAS3B;AA+xFD,wBAAgB,iBAAiB,CAC/B,SAAS,GAAE,OAAO,CAAC,uBAAuB,CAAM,GAC/C,OAAO,CAsOT"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/commands/gate/index.ts"],"names":[],"mappings":"AAaA,OAAO,EAEL,KAAK,cAAc,EACnB,KAAK,aAAa,EACnB,MAAM,sBAAsB,CAAC;AAC9B,OAAO,EACL,gBAAgB,EAKhB,KAAK,qBAAqB,EAE3B,MAAM,8BAA8B,CAAC;AAYtC,OAAO,EAWL,KAAK,UAAU,EAIf,KAAK,SAAS,EACd,KAAK,cAAc,EACnB,KAAK,UAAU,EAEhB,MAAM,oBAAoB,CAAC;AAE5B,OAAO,EAOL,KAAK,cAAc,EACpB,MAAM,iBAAiB,CAAC;AAWzB,OAAO,EAEL,KAAK,WAAW,EACjB,MAAM,4BAA4B,CAAC;AACpC,OAAO,EAEL,KAAK,kBAAkB,EACvB,KAAK,kBAAkB,EAExB,MAAM,gCAAgC,CAAC;AAExC,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAGpC,OAAO,EACL,uBAAuB,EAExB,MAAM,mBAAmB,CAAC;AAC3B,OAAO,EACL,wBAAwB,EACxB,oBAAoB,EACpB,oBAAoB,EACpB,wBAAwB,EAEzB,MAAM,oBAAoB,CAAC;AAC5B,OAAO,EAEL,KAAK,iBAAiB,EACtB,KAAK,gBAAgB,EACtB,MAAM,iBAAiB,CAAC;AAEzB,OAAO,EACL,sBAAsB,EAIvB,MAAM,kBAAkB,CAAC;AAG1B,UAAU,uBAAuB;IAC/B,mBAAmB,EAAE,CAAC,OAAO,EAAE,aAAa,KAAK,cAAc,CAAC;IAChE,kBAAkB,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,OAAO,CAAC,MAAM,CAAC,CAAC;IACrD,aAAa,EAAE,CAAC,QAAQ,EAAE,MAAM,KAAK,OAAO,CAAC,SAAS,CAAC,CAAC;IACxD,cAAc,EAAE,CAAC,QAAQ,EAAE,MAAM,EAAE,MAAM,EAAE,SAAS,KAAK,OAAO,CAAC,IAAI,CAAC,CAAC;IACvE,kBAAkB,EAAE,CAAC,QAAQ,EAAE,MAAM,KAAK,OAAO,CAAC,cAAc,CAAC,CAAC;IAClE,mBAAmB,EAAE,CACnB,QAAQ,EAAE,MAAM,EAChB,MAAM,EAAE,cAAc,KACnB,OAAO,CAAC,IAAI,CAAC,CAAC;IACnB,cAAc,EAAE,CAAC,aAAa,EAAE,MAAM,KAAK,OAAO,CAAC,UAAU,CAAC,CAAC;IAC/D,eAAe,EAAE,CAAC,aAAa,EAAE,MAAM,EAAE,MAAM,EAAE,UAAU,KAAK,OAAO,CAAC,IAAI,CAAC,CAAC;IAC9E,sBAAsB,EAAE,CACtB,QAAQ,EAAE,MAAM,EAChB,aAAa,EAAE,MAAM,EACrB,GAAG,EAAE,MAAM,CAAC,UAAU,KACnB,OAAO,CAAC,cAAc,CAAC,CAAC;IAC7B,uBAAuB,EAAE,OAAO,uBAAuB,CAAC;IACxD,wBAAwB,EAAE,OAAO,wBAAwB,CAAC;IAC1D,oBAAoB,EAAE,OAAO,oBAAoB,CAAC;IAClD,wBAAwB,EAAE,OAAO,wBAAwB,CAAC;IAC1D,oBAAoB,EAAE,OAAO,oBAAoB,CAAC;IAClD,UAAU,EAAE,CACV,OAAO,EAAE,MAAM,EACf,IAAI,EAAE,MAAM,EAAE,EACd,OAAO,EAAE,iBAAiB,KACvB,OAAO,CAAC,gBAAgB,CAAC,CAAC;IAC/B,sBAAsB,EAAE,OAAO,sBAAsB,CAAC;IACtD,gBAAgB,EAAE,OAAO,gBAAgB,CAAC;IAC1C,UAAU,EAAE,MAAM,CAAC,UAAU,CAAC;IAC9B,kBAAkB,EAAE,CAClB,IAAI,EAAE,MAAM,EACZ,MAAM,EAAE,aAAa,EACrB,IAAI,EAAE,CAAC,OAAO,EAAE,MAAM,KAAK,IAAI,KAC5B,OAAO,CAAC,OAAO,CAAC,CAAC;IACtB,mBAAmB,EAAE,CACnB,IAAI,EAAE,MAAM,EACZ,IAAI,EAAE,CAAC,OAAO,EAAE,MAAM,KAAK,IAAI,KAC5B,OAAO,CAAC,IAAI,CAAC,CAAC;IACnB,0BAA0B,EAAE,CAC1B,IAAI,EAAE,MAAM,EACZ,OAAO,EAAE,qBAAqB,EAC9B,IAAI,EAAE,CAAC,OAAO,EAAE,MAAM,KAAK,IAAI,KAC5B,OAAO,CAAC,OAAO,CAAC,CAAC;IACtB,KAAK,EAAE,CAAC,EAAE,EAAE,MAAM,KAAK,OAAO,CAAC,IAAI,CAAC,CAAC;IACrC,eAAe,EAAE,CAAC,OAAO,EAAE,MAAM,KAAK,IAAI,CAAC;CAC5C;AAuDD,KAAK,iBAAiB,GAClB,KAAK,GACL,QAAQ,GACR,QAAQ,GACR,KAAK,GACL,eAAe,GACf,SAAS,CAAC;AAOd,UAAU,aAAa;IACrB,KAAK,EAAE,MAAM,CAAC;IACd,QAAQ,EAAE,MAAM,CAAC;IACjB,OAAO,EAAE,MAAM,CAAC;IAChB,UAAU,EAAE,MAAM,GAAG,IAAI,CAAC;IAC1B,WAAW,EAAE,MAAM,GAAG,IAAI,CAAC;IAC3B,OAAO,EAAE,MAAM,CAAC;IAChB,SAAS,EAAE,MAAM,CAAC;IAClB,QAAQ,EAAE,MAAM,CAAC;IACjB,YAAY,EAAE,iBAAiB,CAAC;CACjC;AAcD,KAAK,kBAAkB,GAAG,aAAa,GAAG,cAAc,GAAG,MAAM,CAAC;AAClE,KAAK,qBAAqB,GACtB,kBAAkB,GAClB,4BAA4B,GAC5B,2CAA2C,GAC3C,kBAAkB,CAAC;AA4CvB,MAAM,WAAW,kBAAkB;IACjC,EAAE,EAAE,MAAM,CAAC;IACX,MAAM,EAAE,UAAU,CAAC;IACnB,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,WAAW,CAAC;IACpB,SAAS,CAAC,EAAE,qBAAqB,CAAC;IAClC,uBAAuB,CAAC,EAAE,OAAO,CAAC;CACnC;AAED,UAAU,oBAAoB;IAC5B,KAAK,EAAE,MAAM,CAAC;IACd,UAAU,EAAE,kBAAkB,CAAC;IAC/B,UAAU,EAAE,kBAAkB,CAAC;IAC/B,MAAM,EAAE,WAAW,CAAC;IACpB,aAAa,EAAE,WAAW,EAAE,CAAC;IAC7B,kBAAkB,CAAC,EAAE,MAAM,EAAE,CAAC;IAC9B,sBAAsB,CAAC,EAAE,MAAM,CAAC;IAChC,kBAAkB,EAAE,OAAO,CAAC;IAC5B,MAAM,EAAE,MAAM,GAAG,OAAO,GAAG,mBAAmB,GAAG,aAAa,GAAG,SAAS,CAAC;CAC5E;AAED,UAAU,qBAAqB;IAC7B,KAAK,EAAE,kBAAkB,CAAC;IAC1B,QAAQ,EAAE,qBAAqB,CAAC;IAChC,QAAQ,EAAE;QACR,KAAK,EAAE,MAAM,CAAC;QACd,UAAU,EAAE,kBAAkB,CAAC;QAC/B,UAAU,EAAE,kBAAkB,CAAC;QAC/B,MAAM,EAAE,WAAW,CAAC;QACpB,MAAM,EAAE,oBAAoB,CAAC,QAAQ,CAAC,CAAC;QACvC,aAAa,EAAE,WAAW,EAAE,CAAC;QAC7B,kBAAkB,CAAC,EAAE,MAAM,EAAE,CAAC;QAC9B,sBAAsB,CAAC,EAAE,MAAM,CAAC;KACjC,CAAC;IACF,QAAQ,EAAE;QACR,MAAM,EAAE,MAAM,CAAC;QACf,OAAO,EAAE,MAAM,CAAC;QAChB,MAAM,EAAE,WAAW,CAAC;QACpB,KAAK,CAAC,EAAE,MAAM,CAAC;KAChB,CAAC;IACF,OAAO,CAAC,EAAE,MAAM,CAAC;CAClB;AAi3CD,wBAAgB,gBAAgB,CAC9B,QAAQ,EAAE,QAAQ,CAAC,MAAM,CAAC,MAAM,EAAE,UAAU,CAAC,CAAC,EAC9C,cAAc,EAAE,MAAM,EACtB,KAAK,EAAE,kBAAkB,EACzB,gBAAgB,CAAC,EAAE,oBAAoB,GACtC,kBAAkB,GAAG,IAAI,CAS3B;AA+xFD,wBAAgB,iBAAiB,CAC/B,SAAS,GAAE,OAAO,CAAC,uBAAuB,CAAM,GAC/C,OAAO,CAsOT"}

package/dist/commands/gate/index.js

@@ -65,10 +65,18 @@ const VALID_CROSS_PROVIDER_AVOIDS = [
 ];
 const VALID_REVIEW_GATE_THRESHOLDS = [
     'critical',
-    'important',
+    'high',
     'medium',
-    'minor',
+    'low',
 ];
+/**
+ * Threshold aliases accepted from configuration written before the severity
+ * rename, so an existing `exit_nonzero_on: important` keeps its meaning.
+ */
+const LEGACY_REVIEW_GATE_THRESHOLDS = new Map([
+    ['important', 'high'],
+    ['minor', 'low'],
+]);
 const VALID_IDENTITY_PROVENANCES = [
     'declared',
     'observed',
@@ -76,7 +84,7 @@ const VALID_IDENTITY_PROVENANCES = [
     'unknown',
 ];
 const REVIEW_GATE_CONTEXT_NOTE = [
-    'This review is gate-originated. If you run `oat-project-review-provide`, set `oat_review_invocation: gate` in the review artifact. Write a canonical review artifact with `### Critical`, `### Important`, `### Medium`, and `### Minor` headings in that order, using `None` for empty sections.',
+    'This review is gate-originated. If you run `oat-project-review-provide`, set `oat_review_invocation: gate` in the review artifact. Write a canonical review artifact with `### Critical`, `### High`, `### Medium`, and `### Low` headings in that order, using `None` for empty sections.',
     'Complete the review, artifact write, and required bookkeeping inline or through a synchronously awaited child before this headless process exits. Do not start background tasks, monitors, or waiters that outlive this turn.',
     "Artifact hygiene contract: Before finishing or committing, format every file you created or edited. Use the concrete write/fix formatting command supplied by the governing plan, task, or brief. If none is usable, discover the repository's documented write/fix command from applicable `AGENTS.md`/`CLAUDE.md` instructions and relevant package manifests; do not infer or hardcode a formatter. Prefer a file-scoped invocation when supported, and avoid rewriting unrelated files. If no command is discoverable, warn once with `no format command discovered in repo instructions; skipping`, then continue.",
 ].join('\n\n');
@@ -349,9 +357,10 @@ function parseCrossProviderAvoid(value) {
     return avoid;
 }
 function parseReviewGateThreshold(value) {
-    const threshold = value?.trim() || 'important';
+    const raw = value?.trim() || 'high';
+    const threshold = LEGACY_REVIEW_GATE_THRESHOLDS.get(raw) ?? raw;
     if (!VALID_REVIEW_GATE_THRESHOLDS.includes(threshold)) {
-        throw new Error('--exit-nonzero-on must be one of critical | important | medium | minor.');
+        throw new Error('--exit-nonzero-on must be one of critical | high | medium | low.');
     }
     return threshold;
 }
@@ -1421,10 +1430,10 @@ function reviewBlocksAtThreshold(verdict, threshold) {
     if (threshold === 'critical') {
         return false;
     }
-    if (verdict.counts.important > 0) {
+    if (verdict.counts.high > 0) {
         return true;
     }
-    if (threshold === 'important') {
+    if (threshold === 'high') {
         return false;
     }
     if (verdict.counts.medium > 0) {
@@ -1433,7 +1442,7 @@ function reviewBlocksAtThreshold(verdict, threshold) {
     if (threshold === 'medium') {
         return false;
     }
-    return verdict.counts.minor > 0;
+    return verdict.counts.low > 0;
 }
 function buildReviewGateHandoff(options) {
     const receiveInstruction = `Run oat-project-review-receive for ${options.artifactPath} before treating this gate review as consumed.`;
@@ -1481,7 +1490,7 @@ function writeReviewGateResult(context, payload) {
     if (payload.normalization) {
         context.logger.info(`${payload.normalization.persisted ? 'Artifact normalized' : 'Artifact snapshot normalized in memory; source file unchanged'}: inserted ${payload.normalization.insertedSeverities.map((severity) => severityDisplayName(severity)).join(', ')} empty Findings section(s).`);
     }
-    context.logger.info(`Verdict: ${payload.status} (critical=${payload.counts.critical}, important=${payload.counts.important}, medium=${payload.counts.medium}, minor=${payload.counts.minor})`);
+    context.logger.info(`Verdict: ${payload.status} (critical=${payload.counts.critical}, high=${payload.counts.high}, medium=${payload.counts.medium}, low=${payload.counts.low})`);
     if (payload.diversity) {
         if (payload.diversity.warning) {
             context.logger.warn(payload.diversity.warning);
@@ -1893,7 +1902,7 @@ async function emitGateProjectLogPartialFinalization(options) {
 }
 async function finalizeReviewGateProjectLog(context, dependencies, finalization) {
     const findings = finalization.counts
-        ? ` findings=critical:${finalization.counts.critical},important:${finalization.counts.important},medium:${finalization.counts.medium},minor:${finalization.counts.minor}`
+        ? ` findings=critical:${finalization.counts.critical},high:${finalization.counts.high},medium:${finalization.counts.medium},low:${finalization.counts.low}`
         : '';
     const artifact = finalization.artifactPath
         ? ` artifact=${finalization.artifactPath}`
@@ -3012,7 +3021,7 @@ export function createGateCommand(overrides = {}) {
         .option('--project <path-or-name>', 'Project path or name to review; defaults to the active project')
         .option('--review-scope <scope>', 'Review scope hint for the provider')
         .option('--review-type <type>', 'Review type hint for the provider')
-        .option('--exit-nonzero-on <severity>', 'Lowest severity that exits nonzero: critical, important, medium, or minor')
+        .option('--exit-nonzero-on <severity>', 'Lowest severity that exits nonzero: critical, high, medium, or low')
         .argument('<prompt...>', 'Review prompt arguments appended to the target command')
         .action(async (prompt, options, command) => {
         const context = dependencies.buildCommandContext(readGlobalOptions(command));

package/dist/commands/gate/review-verdict.d.ts

@@ -7,9 +7,9 @@ export interface ReviewGateVerdict {
     gateInvocation?: ReviewArtifactGateInvocation;
     counts: {
         critical: number;
-        important: number;
+        high: number;
         medium: number;
-        minor: number;
+        low: number;
     };
     blocking: boolean;
     normalization?: {

package/dist/commands/gate/review-verdict.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"review-verdict.d.ts","sourceRoot":"","sources":["../../../src/commands/gate/review-verdict.ts"],"names":[],"mappings":"AAMA,MAAM,WAAW,iBAAiB;IAChC,YAAY,EAAE,MAAM,CAAC;IACrB,UAAU,EAAE,MAAM,GAAG,UAAU,GAAG,SAAS,CAAC;IAC5C,KAAK,EAAE,MAAM,GAAG,IAAI,CAAC;IACrB,UAAU,EAAE,MAAM,GAAG,IAAI,CAAC;IAC1B,OAAO,EAAE,MAAM,GAAG,IAAI,CAAC;IACvB,cAAc,CAAC,EAAE,4BAA4B,CAAC;IAC9C,MAAM,EAAE;QACN,QAAQ,EAAE,MAAM,CAAC;QACjB,SAAS,EAAE,MAAM,CAAC;QAClB,MAAM,EAAE,MAAM,CAAC;QACf,KAAK,EAAE,MAAM,CAAC;KACf,CAAC;IACF,QAAQ,EAAE,OAAO,CAAC;IAClB,aAAa,CAAC,EAAE;QACd,kBAAkB,EAAE,QAAQ,EAAE,CAAC;QAC/B,SAAS,EAAE,OAAO,CAAC;KACpB,CAAC;CACH;AAED,MAAM,WAAW,4BAA4B;IAC3C,KAAK,EAAE,MAAM,GAAG,IAAI,CAAC;IACrB,QAAQ,EAAE,MAAM,GAAG,IAAI,CAAC;IACxB,OAAO,EAAE,MAAM,GAAG,IAAI,CAAC;IACvB,KAAK,EAAE,MAAM,GAAG,IAAI,CAAC;IACrB,eAAe,EAAE,MAAM,GAAG,IAAI,CAAC;IAC/B,MAAM,EAAE,MAAM,GAAG,IAAI,CAAC;CACvB;AAED,MAAM,MAAM,QAAQ,GAAG,MAAM,iBAAiB,CAAC,QAAQ,CAAC,CAAC;AAEzD,MAAM,WAAW,0BAA0B;IACzC,QAAQ,CAAC,OAAO,EAAE,MAAM,CAAC;IACzB,QAAQ,CAAC,SAAS,EAAE,MAAM,CAAC;CAC5B;AAED,MAAM,WAAW,6BAA6B;IAC5C,qCAAqC,CAAC,EAAE,OAAO,CAAC;IAChD,gBAAgB,CAAC,EAAE,0BAA0B,CAAC;CAC/C;AA6LD,wBAAgB,mBAAmB,CAAC,QAAQ,EAAE,QAAQ,GAAG,MAAM,CAE9D;AAgSD,wBAAsB,sBAAsB,CAC1C,YAAY,EAAE,MAAM,EACpB,OAAO,GAAE,6BAAkC,GAC1C,OAAO,CAAC,iBAAiB,CAAC,CAiF5B"}
+{"version":3,"file":"review-verdict.d.ts","sourceRoot":"","sources":["../../../src/commands/gate/review-verdict.ts"],"names":[],"mappings":"AAMA,MAAM,WAAW,iBAAiB;IAChC,YAAY,EAAE,MAAM,CAAC;IACrB,UAAU,EAAE,MAAM,GAAG,UAAU,GAAG,SAAS,CAAC;IAC5C,KAAK,EAAE,MAAM,GAAG,IAAI,CAAC;IACrB,UAAU,EAAE,MAAM,GAAG,IAAI,CAAC;IAC1B,OAAO,EAAE,MAAM,GAAG,IAAI,CAAC;IACvB,cAAc,CAAC,EAAE,4BAA4B,CAAC;IAC9C,MAAM,EAAE;QACN,QAAQ,EAAE,MAAM,CAAC;QACjB,IAAI,EAAE,MAAM,CAAC;QACb,MAAM,EAAE,MAAM,CAAC;QACf,GAAG,EAAE,MAAM,CAAC;KACb,CAAC;IACF,QAAQ,EAAE,OAAO,CAAC;IAClB,aAAa,CAAC,EAAE;QACd,kBAAkB,EAAE,QAAQ,EAAE,CAAC;QAC/B,SAAS,EAAE,OAAO,CAAC;KACpB,CAAC;CACH;AAED,MAAM,WAAW,4BAA4B;IAC3C,KAAK,EAAE,MAAM,GAAG,IAAI,CAAC;IACrB,QAAQ,EAAE,MAAM,GAAG,IAAI,CAAC;IACxB,OAAO,EAAE,MAAM,GAAG,IAAI,CAAC;IACvB,KAAK,EAAE,MAAM,GAAG,IAAI,CAAC;IACrB,eAAe,EAAE,MAAM,GAAG,IAAI,CAAC;IAC/B,MAAM,EAAE,MAAM,GAAG,IAAI,CAAC;CACvB;AAED,MAAM,MAAM,QAAQ,GAAG,MAAM,iBAAiB,CAAC,QAAQ,CAAC,CAAC;AAEzD,MAAM,WAAW,0BAA0B;IACzC,QAAQ,CAAC,OAAO,EAAE,MAAM,CAAC;IACzB,QAAQ,CAAC,SAAS,EAAE,MAAM,CAAC;CAC5B;AAED,MAAM,WAAW,6BAA6B;IAC5C,qCAAqC,CAAC,EAAE,OAAO,CAAC;IAChD,gBAAgB,CAAC,EAAE,0BAA0B,CAAC;CAC/C;AAmOD,wBAAgB,mBAAmB,CAAC,QAAQ,EAAE,QAAQ,GAAG,MAAM,CAE9D;AA2eD,wBAAsB,sBAAsB,CAC1C,YAAY,EAAE,MAAM,EACpB,OAAO,GAAE,6BAAkC,GAC1C,OAAO,CAAC,iBAAiB,CAAC,CA8F5B"}

package/dist/commands/gate/review-verdict.js

@@ -2,17 +2,12 @@ import { createHash } from 'node:crypto';
 import { readFile, writeFile } from 'node:fs/promises';
 import { getFrontmatterBlock } from '../shared/frontmatter.js';
 import YAML from 'yaml';
-const SEVERITIES = [
-    'critical',
-    'important',
-    'medium',
-    'minor',
-];
+const SEVERITIES = ['critical', 'high', 'medium', 'low'];
 const FRONTMATTER_COUNT_KEYS = {
     critical: ['oat_review_critical_count', 'critical'],
-    important: ['oat_review_important_count', 'important'],
+    high: ['oat_review_high_count', 'high'],
     medium: ['oat_review_medium_count', 'medium'],
-    minor: ['oat_review_minor_count', 'minor'],
+    low: ['oat_review_low_count', 'low'],
 };
 function artifactContentSignature(content) {
     return createHash('sha256').update(content).digest('hex');
@@ -59,6 +54,9 @@ function readGateInvocation(frontmatter) {
     };
 }
 function parseCountValue(value) {
+    if (typeof value === 'string' && value.trim().length === 0) {
+        return null;
+    }
     const numberValue = typeof value === 'number'
         ? value
         : typeof value === 'string'
@@ -69,40 +67,58 @@ function parseCountValue(value) {
     }
     return numberValue;
 }
-function readFrontmatterCounts(frontmatter) {
+function readFrontmatterCounts(frontmatter, artifactPath) {
+    const hasNestedCounts = Object.hasOwn(frontmatter, 'oat_review_counts');
+    if (hasNestedCounts &&
+        (typeof frontmatter['oat_review_counts'] !== 'object' ||
+            frontmatter['oat_review_counts'] === null ||
+            Array.isArray(frontmatter['oat_review_counts']))) {
+        throw new Error(`Review artifact at ${artifactPath} declares an invalid oat_review_counts block; it must be an object of non-negative integer counts.`);
+    }
     const nestedCounts = typeof frontmatter['oat_review_counts'] === 'object' &&
         frontmatter['oat_review_counts'] !== null &&
         !Array.isArray(frontmatter['oat_review_counts'])
         ? frontmatter['oat_review_counts']
         : null;
-    const counts = {
-        critical: 0,
-        important: 0,
-        medium: 0,
-        minor: 0,
-    };
+    const parsed = new Map();
+    // Validate every supplied alias before deciding whether the block is
+    // complete. A partial block must not be able to hide a conflicting pair and
+    // fall through to a summary line that reports fewer findings.
     for (const severity of SEVERITIES) {
         const candidateValues = [
-            ...FRONTMATTER_COUNT_KEYS[severity].map((key) => frontmatter[key]),
+            ...FRONTMATTER_COUNT_KEYS[severity].flatMap((key) => Object.hasOwn(frontmatter, key) ? [frontmatter[key]] : []),
             ...(nestedCounts
-                ? FRONTMATTER_COUNT_KEYS[severity].map((key) => nestedCounts[key])
+                ? FRONTMATTER_COUNT_KEYS[severity].flatMap((key) => Object.hasOwn(nestedCounts, key) ? [nestedCounts[key]] : [])
                 : []),
-        ].filter((value) => value !== undefined && value !== null);
-        if (candidateValues.length === 0) {
-            return null;
+        ];
+        const hasInvalidValue = candidateValues.some((value) => parseCountValue(value) === null);
+        if (hasInvalidValue) {
+            throw new Error(`Review artifact at ${artifactPath} declares an invalid ${severity} count; counts must be non-negative integers.`);
         }
-        const parsedCount = candidateValues.reduce((parsed, value) => {
-            if (parsed !== null) {
-                return parsed;
-            }
-            return parseCountValue(value);
-        }, null);
-        if (parsedCount === null) {
-            return null;
+        const distinct = [
+            ...new Set(candidateValues
+                .map((value) => parseCountValue(value))
+                .filter((value) => value !== null)),
+        ];
+        if (distinct.length > 1) {
+            throw new Error(`Review artifact at ${artifactPath} declares conflicting ${severity} counts (${distinct.join(', ')}); resolve the frontmatter before the gate can evaluate the review.`);
+        }
+        if (distinct.length === 1) {
+            parsed.set(severity, distinct[0]);
         }
-        counts[severity] = parsedCount;
     }
-    return counts;
+    if (parsed.size === 0) {
+        return null;
+    }
+    return {
+        counts: {
+            critical: parsed.get('critical') ?? 0,
+            high: parsed.get('high') ?? 0,
+            medium: parsed.get('medium') ?? 0,
+            low: parsed.get('low') ?? 0,
+        },
+        tiers: [...parsed.keys()],
+    };
 }
 function sectionContentIsEmpty(content) {
     const cleaned = content
@@ -146,15 +162,25 @@ function markdownLines(content) {
     return lines;
 }
 function fenceMarker(line) {
-    const match = line.match(/^\s*(`{3,}|~{3,})/);
+    const match = line.match(/^\s*(`{3,}|~{3,})(.*)$/);
     if (!match?.[1]) {
         return null;
     }
     return {
         marker: match[1].startsWith('`') ? '`' : '~',
         length: match[1].length,
+        trailing: match[2] ?? '',
     };
 }
+/**
+ * Lines outside fenced code blocks.
+ *
+ * An opening fence may carry an info string (```ts), but a closing fence may
+ * not: a marker run followed by other text is fence content, not a delimiter.
+ * Treating such a line as a closer ends the fence early and exposes quoted
+ * examples to the count scans, which is the failure the fence-awareness is
+ * there to prevent.
+ */
 function linesOutsideFences(content) {
     const outsideFenceLines = [];
     let activeFence = null;
@@ -165,7 +191,8 @@ function linesOutsideFences(content) {
                 activeFence = marker;
             }
             else if (marker.marker === activeFence.marker &&
-                marker.length >= activeFence.length) {
+                marker.length >= activeFence.length &&
+                marker.trailing.trim() === '') {
                 activeFence = null;
             }
             continue;
@@ -211,39 +238,114 @@ function missingSeverityHeadings(severityHeadings) {
     const seenSeverities = new Set(severityHeadings.map((heading) => heading.severity));
     return SEVERITIES.filter((severity) => !seenSeverities.has(severity));
 }
-function parseFindingsSummaryCounts(content) {
-    const match = content.match(/^Findings:\s*(\d+)\s+critical,\s*(\d+)\s+important,\s*(\d+)\s+medium,\s*(\d+)\s+minor\s*$/im);
-    if (!match) {
+const FINDINGS_SUMMARY_PATTERN = /^Findings by severity:\s*(\d+)\s+critical,\s*(\d+)\s+high,\s*(\d+)\s+medium,\s*(\d+)\s+low\s*$/i;
+/**
+ * Read the canonical count line. Only lines outside fenced code blocks count,
+ * so a quoted example cannot be mistaken for the artifact's own counts, and two
+ * disagreeing count lines are rejected rather than silently resolved in favor
+ * of whichever appears first.
+ */
+function parseFindingsSummaryCounts(content, artifactPath) {
+    const found = [];
+    for (const line of linesOutsideFences(content)) {
+        const match = FINDINGS_SUMMARY_PATTERN.exec(line.text.trim());
+        if (!match) {
+            continue;
+        }
+        found.push({
+            critical: Number.parseInt(match[1] ?? '0', 10),
+            high: Number.parseInt(match[2] ?? '0', 10),
+            medium: Number.parseInt(match[3] ?? '0', 10),
+            low: Number.parseInt(match[4] ?? '0', 10),
+        });
+    }
+    if (found.length === 0) {
         return null;
     }
-    return {
-        critical: Number.parseInt(match[1] ?? '0', 10),
-        important: Number.parseInt(match[2] ?? '0', 10),
-        medium: Number.parseInt(match[3] ?? '0', 10),
-        minor: Number.parseInt(match[4] ?? '0', 10),
-    };
+    const distinct = new Map(found.map((counts) => [JSON.stringify(counts), counts]));
+    if (distinct.size > 1) {
+        throw new Error(`Review artifact at ${artifactPath} contains ${found.length} conflicting "Findings by severity" count lines; keep exactly one so the gate can evaluate the review.`);
+    }
+    return found[0];
 }
-function parseFindingsSectionCounts(content, artifactPath) {
+/**
+ * Tier spellings retired by the severity rename. They are no longer read, and
+ * detecting them fails the artifact with a migration error rather than letting
+ * it parse. A retired heading that parsed alongside canonical ones would have
+ * its findings attributed to whichever canonical section precedes it, which can
+ * move a finding below the blocking threshold.
+ */
+function describeLegacySeverityUsage(content, frontmatter) {
+    const found = new Set();
+    const findingsSection = findFindingsSection(content);
+    for (const line of linesOutsideFences(findingsSection?.content ?? '')) {
+        if (/^#{1,6}\s+Important\s*#*\s*$/i.test(line.text)) {
+            found.add('a `### Important` heading');
+        }
+        if (/^#{1,6}\s+Minor\s*#*\s*$/i.test(line.text)) {
+            found.add('a `### Minor` heading');
+        }
+    }
+    for (const line of linesOutsideFences(content)) {
+        if (/^Findings:\s*\d+\s+critical,/i.test(line.text.trim())) {
+            found.add('a legacy `Findings:` count line');
+        }
+    }
+    const nested = typeof frontmatter['oat_review_counts'] === 'object' &&
+        frontmatter['oat_review_counts'] !== null &&
+        !Array.isArray(frontmatter['oat_review_counts'])
+        ? frontmatter['oat_review_counts']
+        : {};
+    for (const key of [...Object.keys(frontmatter), ...Object.keys(nested)]) {
+        if (/^(oat_review_)?(important|minor)(_count)?$/.test(key)) {
+            found.add(`the \`${key}\` count key`);
+        }
+    }
+    return [...found];
+}
+function legacySeverityError(artifactPath, legacyUsage) {
+    if (legacyUsage.length === 0) {
+        return null;
+    }
+    return new Error(`Review artifact at ${artifactPath} uses retired severity tiers (${legacyUsage.join(', ')}). The tiers are now Critical, High, Medium, and Low: rename \`### Important\` to \`### High\` and \`### Minor\` to \`### Low\`, and \`oat_review_important_count\`/\`oat_review_minor_count\` to \`oat_review_high_count\`/\`oat_review_low_count\`, or re-run the review. If the reviewer that wrote this artifact still emits the retired tiers, its installed instructions are stale: run \`oat tools update\` to refresh the installed tools and their provider projections.`);
+}
+/**
+ * Per-tier counts for every severity heading the body actually has.
+ *
+ * Deliberately tolerant: it counts what is present without requiring the full
+ * canonical heading set, so it can cross-check explicit counts without changing
+ * how a body-only artifact is validated.
+ */
+function tallySeveritySections(content) {
     const findingsSection = findFindingsSection(content);
     const severityHeadings = findSeverityHeadings(content);
     if (severityHeadings.length === 0) {
         return null;
     }
-    const missingSeverities = missingSeverityHeadings(severityHeadings);
-    if (missingSeverities.length > 0) {
-        throw new Error(`Review artifact at ${artifactPath} has an incomplete Findings section; expected headings for Critical, Important, Medium, and Minor. Missing: ${missingSeverities.join(', ')}.`);
-    }
     const counts = {
         critical: 0,
-        important: 0,
+        high: 0,
         medium: 0,
-        minor: 0,
+        low: 0,
     };
     for (const [offset, heading] of severityHeadings.entries()) {
         const nextHeading = severityHeadings[offset + 1];
         const sectionStart = heading.index + heading.headingLength;
         const sectionEnd = nextHeading?.index ?? findingsSection?.end ?? content.length;
-        counts[heading.severity] = countFindingsInSection(content.slice(sectionStart, sectionEnd));
+        // Accumulate rather than assign: a repeated heading for one tier must not
+        // erase findings already counted under an earlier occurrence.
+        counts[heading.severity] += countFindingsInSection(content.slice(sectionStart, sectionEnd));
+    }
+    return counts;
+}
+function parseFindingsSectionCounts(content, artifactPath) {
+    const counts = tallySeveritySections(content);
+    if (counts === null) {
+        return null;
+    }
+    const missingSeverities = missingSeverityHeadings(findSeverityHeadings(content));
+    if (missingSeverities.length > 0) {
+        throw new Error(`Review artifact at ${artifactPath} has an incomplete Findings section; expected headings for ${SEVERITIES.map(severityDisplayName).join(', ')}. Missing: ${missingSeverities.join(', ')}.`);
     }
     return counts;
 }
@@ -263,7 +365,7 @@ function parseFrontmatterObject(frontmatter, artifactPath) {
     }
 }
 function hasBlockingFindings(counts) {
-    return counts.critical > 0 || counts.important > 0;
+    return counts.critical > 0 || counts.high > 0;
 }
 function insertionTextForSeverity(severity) {
     return `\n### ${severityDisplayName(severity)}\n\nNone\n`;
@@ -307,16 +409,72 @@ async function normalizeMissingEmptySeveritySections(artifactPath, content, coun
         persisted: insertedSeverities.length > 0 && persist,
     };
 }
-function resolveCounts(content, frontmatter) {
-    const frontmatterCounts = readFrontmatterCounts(frontmatter);
+function formatCounts(counts) {
+    return SEVERITIES.map((severity) => `${counts[severity]} ${severity}`).join(', ');
+}
+/**
+ * Fail closed when two count sources disagree.
+ *
+ * No single source may win by precedence: a stale all-zero frontmatter block or
+ * count line would otherwise mask real findings written in the body, and the
+ * gate would pass a review that clearly reports a blocking finding.
+ */
+function assertAgreeingCounts(artifactPath, left, right, tiers = SEVERITIES) {
+    const differing = tiers.filter((severity) => left.counts[severity] !== right.counts[severity]);
+    if (differing.length === 0) {
+        return;
+    }
+    throw new Error(`Review artifact at ${artifactPath} contradicts itself about finding counts (${differing.join(', ')}): declared counts are ${formatCounts(left.counts)} (${left.name}) but ${formatCounts(right.counts)} (${right.name}). Reconcile them before the gate can evaluate the review.`);
+}
+/**
+ * Resolve the artifact's counts, cross-checking every source that is present.
+ *
+ * Returns `null` when only the body carries counts, so the caller keeps the
+ * completeness-checked body path for body-only artifacts.
+ */
+function resolveCounts(content, frontmatter, artifactPath) {
+    const explicitSources = [];
+    const frontmatterCounts = readFrontmatterCounts(frontmatter, artifactPath);
     if (frontmatterCounts) {
-        return frontmatterCounts;
+        explicitSources.push({
+            name: 'the frontmatter count fields',
+            counts: frontmatterCounts.counts,
+            tiers: frontmatterCounts.tiers,
+        });
     }
-    const summaryCounts = parseFindingsSummaryCounts(content);
+    const summaryCounts = parseFindingsSummaryCounts(content, artifactPath);
     if (summaryCounts) {
-        return summaryCounts;
+        explicitSources.push({
+            name: 'the "Findings by severity" count line',
+            counts: summaryCounts,
+            tiers: SEVERITIES,
+        });
     }
-    return null;
+    const [primarySource, ...otherSources] = explicitSources;
+    for (const source of otherSources) {
+        const comparableTiers = primarySource.tiers.filter((severity) => source.tiers.includes(severity));
+        assertAgreeingCounts(artifactPath, primarySource, source, comparableTiers);
+    }
+    const bodyCounts = tallySeveritySections(content);
+    if (bodyCounts) {
+        // Only tiers the body actually has a heading for are comparable. A missing
+        // heading is not a claim of zero findings, it is a malformed body, and the
+        // completeness and normalization checks own that diagnosis.
+        const bodyTiers = [
+            ...new Set(findSeverityHeadings(content).map((heading) => heading.severity)),
+        ];
+        const bodySource = {
+            name: 'the Findings sections',
+            counts: bodyCounts,
+            tiers: bodyTiers,
+        };
+        for (const source of explicitSources) {
+            const comparableTiers = source.tiers.filter((severity) => bodyTiers.includes(severity));
+            assertAgreeingCounts(artifactPath, source, bodySource, comparableTiers);
+        }
+    }
+    return (explicitSources.find((source) => source.tiers.length === SEVERITIES.length)
+        ?.counts ?? null);
 }
 export async function parseReviewGateVerdict(artifactPath, options = {}) {
     let content;
@@ -340,7 +498,15 @@ export async function parseReviewGateVerdict(artifactPath, options = {}) {
     const frontmatter = frontmatterBlock
         ? parseFrontmatterObject(frontmatterBlock, artifactPath)
         : {};
-    let counts = resolveCounts(content, frontmatter);
+    // Retired tiers fail the artifact outright rather than being ignored, and do
+    // so before normalization can rewrite the file. Ignoring them is not safe: a
+    // retired heading that parses alongside canonical ones has its findings
+    // attributed to whichever canonical section precedes it.
+    const retiredTierError = legacySeverityError(artifactPath, describeLegacySeverityUsage(content, frontmatter));
+    if (retiredTierError) {
+        throw retiredTierError;
+    }
+    let counts = resolveCounts(content, frontmatter, artifactPath);
     let insertedSeverities = [];
     let normalizationPersisted = false;
     if (counts && options.normalizeMissingEmptySeveritySections) {