@context-forge/core 0.8.6 → 0.9.1

package/dist/config/ConfigKeys.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"ConfigKeys.d.ts","sourceRoot":"","sources":["../../src/config/ConfigKeys.ts"],"names":[],"mappings":"AAAA,MAAM,WAAW,mBAAmB;IAClC,IAAI,EAAE,QAAQ,GAAG,SAAS,GAAG,QAAQ,CAAC;IACtC,OAAO,EAAE,MAAM,GAAG,OAAO,GAAG,MAAM,CAAC;IACnC,WAAW,EAAE,MAAM,CAAC;IACpB,IAAI,CAAC,EAAE,MAAM,EAAE,CAAC;IAChB,QAAQ,CAAC,EAAE,CAAC,KAAK,EAAE,MAAM,GAAG,OAAO,GAAG,MAAM,KAAK,MAAM,GAAG,IAAI,CAAC;CAChE;AAED,eAAO,MAAM,WAAW,EAAE,MAAM,CAAC,MAAM,EAAE,mBAAmB,CA4C3D,CAAC"}
+{"version":3,"file":"ConfigKeys.d.ts","sourceRoot":"","sources":["../../src/config/ConfigKeys.ts"],"names":[],"mappings":"AAAA,MAAM,WAAW,mBAAmB;IAClC,IAAI,EAAE,QAAQ,GAAG,SAAS,GAAG,QAAQ,CAAC;IACtC,OAAO,EAAE,MAAM,GAAG,OAAO,GAAG,MAAM,CAAC;IACnC,WAAW,EAAE,MAAM,CAAC;IACpB,IAAI,CAAC,EAAE,MAAM,EAAE,CAAC;IAChB,QAAQ,CAAC,EAAE,CAAC,KAAK,EAAE,MAAM,GAAG,OAAO,GAAG,MAAM,KAAK,MAAM,GAAG,IAAI,CAAC;CAChE;AAED,eAAO,MAAM,WAAW,EAAE,MAAM,CAAC,MAAM,EAAE,mBAAmB,CAuG3D,CAAC"}

package/dist/config/ConfigKeys.js

@@ -25,10 +25,65 @@ export const CONFIG_KEYS = {
         default: false,
         description: 'Automatically apply non-destructive corrections when running consistency checks',
     },
-    'git.branch_root': {
+    'workflow.review_enabled': {
+        type: 'boolean',
+        default: false,
+        description: 'Enable review gating in the workflow navigator (off by default; no behavior change when false)',
+    },
+    'workflow.review_threshold': {
+        type: 'string',
+        default: 'concerns',
+        description: 'Verdict floor that clears a review gate: "pass" requires PASS; "concerns" clears on PASS or CONCERNS',
+        enum: ['pass', 'concerns'],
+    },
+    'workflow.review_unknown_as': {
+        type: 'string',
+        default: 'fail',
+        description: 'How to treat an UNKNOWN/absent/unparseable verdict: "fail" blocks, "concerns" treats as CONCERNS, "pass" clears',
+        enum: ['fail', 'concerns', 'pass'],
+    },
+    'workflow.review_gates.code.threshold': {
+        type: 'string',
+        default: '',
+        description: 'Per-gate override: verdict floor for the code (pre-advance) review gate (empty = use workflow.review_threshold)',
+        enum: ['', 'pass', 'concerns'],
+    },
+    'workflow.review_gates.arch.threshold': {
+        type: 'string',
+        default: '',
+        description: 'Per-gate override: verdict floor for the arch (pre-slice-plan) review gate (empty = use workflow.review_threshold)',
+        enum: ['', 'pass', 'concerns'],
+    },
+    'workflow.review_gates.slice.threshold': {
+        type: 'string',
+        default: '',
+        description: 'Per-gate override: verdict floor for the slice (pre-tasks) review gate (empty = use workflow.review_threshold)',
+        enum: ['', 'pass', 'concerns'],
+    },
+    'workflow.review_gates.tasks.threshold': {
+        type: 'string',
+        default: '',
+        description: 'Per-gate override: verdict floor for the tasks (pre-implementation) review gate (empty = use workflow.review_threshold)',
+        enum: ['', 'pass', 'concerns'],
+    },
+    'workflow.review_gate_effective_date': {
+        type: 'string',
+        default: '',
+        description: 'Grandfathers slices/architecture designed before this date (YYYYMMDD, compared against the artifact\'s own dateCreated) out of every review gate boundary. Empty (default) applies no cutoff — every slice is subject to gating. Lets a project turn on review_enabled without retroactively demanding reviews for work completed before the gate existed.',
+        validate: (value) => {
+            if (typeof value !== 'string')
+                return 'must be a string';
+            if (value === '')
+                return null; // empty = no cutoff (identity default)
+            if (!/^\d{8}$/.test(value))
+                return 'must be in YYYYMMDD format (e.g. "20260701")';
+            return null;
+        },
+    },
+    'git.integration_branch': {
         type: 'string',
         default: '',
-        description: 'Optional path prefix prepended to work branch names (e.g. "myroot" yields "myroot/910-slice.foo"). Empty means no prefix. Relative and contained: must not be absolute or escape via "..".',
+        description: 'Optional long-lived integration branch that work branches fork from and merge into instead of main (e.g. "dev/erik" yields "dev/erik/910-slice.foo", forked from and merged into "dev/erik"). Empty means no integration branch — work branches fork from and merge into main directly. Relative and contained: must not be absolute or escape via "..".',
         validate: (value) => {
             if (typeof value !== 'string')
                 return 'must be a string';

package/dist/config/ConfigKeys.js.map

@@ -1 +1 @@
-{"version":3,"file":"ConfigKeys.js","sourceRoot":"","sources":["../../src/config/ConfigKeys.ts"],"names":[],"mappings":"AAQA,MAAM,CAAC,MAAM,WAAW,GAAwC;IAC9D,mBAAmB,EAAE;QACnB,IAAI,EAAE,SAAS;QACf,OAAO,EAAE,KAAK;QACd,WAAW,EAAE,sDAAsD;KACpE;IACD,cAAc,EAAE;QACd,IAAI,EAAE,QAAQ;QACd,OAAO,EAAE,EAAE;QACX,WAAW,EAAE,4CAA4C;KAC1D;IACD,oBAAoB,EAAE;QACpB,IAAI,EAAE,QAAQ;QACd,OAAO,EAAE,WAAW;QACpB,WAAW,EAAE,oDAAoD;QACjE,IAAI,EAAE,CAAC,WAAW,EAAE,OAAO,EAAE,QAAQ,CAAC;KACvC;IACD,uBAAuB,EAAE;QACvB,IAAI,EAAE,SAAS;QACf,OAAO,EAAE,KAAK;QACd,WAAW,EAAE,qDAAqD;KACnE;IACD,mBAAmB,EAAE;QACnB,IAAI,EAAE,SAAS;QACf,OAAO,EAAE,KAAK;QACd,WAAW,EAAE,iFAAiF;KAC/F;IACD,iBAAiB,EAAE;QACjB,IAAI,EAAE,QAAQ;QACd,OAAO,EAAE,EAAE;QACX,WAAW,EACT,4LAA4L;QAC9L,QAAQ,EAAE,CAAC,KAAK,EAAE,EAAE;YAClB,IAAI,OAAO,KAAK,KAAK,QAAQ;gBAAE,OAAO,kBAAkB,CAAC;YACzD,IAAI,KAAK,KAAK,EAAE;gBAAE,OAAO,IAAI,CAAC,CAAC,uCAAuC;YACtE,IAAI,KAAK,CAAC,UAAU,CAAC,GAAG,CAAC;gBAAE,OAAO,gCAAgC,CAAC;YACnE,IAAI,KAAK,CAAC,UAAU,CAAC,IAAI,CAAC,IAAI,YAAY,CAAC,IAAI,CAAC,KAAK,CAAC,EAAE,CAAC;gBACvD,OAAO,gDAAgD,CAAC;YAC1D,CAAC;YACD,IAAI,KAAK,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,QAAQ,CAAC,IAAI,CAAC;gBAAE,OAAO,gCAAgC,CAAC;YACjF,IAAI,KAAK,CAAC,QAAQ,CAAC,GAAG,CAAC,IAAI,KAAK,CAAC,QAAQ,CAAC,IAAI,CAAC;gBAAE,OAAO,gCAAgC,CAAC;YACzF,OAAO,IAAI,CAAC;QACd,CAAC;KACF;CACF,CAAC"}
+{"version":3,"file":"ConfigKeys.js","sourceRoot":"","sources":["../../src/config/ConfigKeys.ts"],"names":[],"mappings":"AAQA,MAAM,CAAC,MAAM,WAAW,GAAwC;IAC9D,mBAAmB,EAAE;QACnB,IAAI,EAAE,SAAS;QACf,OAAO,EAAE,KAAK;QACd,WAAW,EAAE,sDAAsD;KACpE;IACD,cAAc,EAAE;QACd,IAAI,EAAE,QAAQ;QACd,OAAO,EAAE,EAAE;QACX,WAAW,EAAE,4CAA4C;KAC1D;IACD,oBAAoB,EAAE;QACpB,IAAI,EAAE,QAAQ;QACd,OAAO,EAAE,WAAW;QACpB,WAAW,EAAE,oDAAoD;QACjE,IAAI,EAAE,CAAC,WAAW,EAAE,OAAO,EAAE,QAAQ,CAAC;KACvC;IACD,uBAAuB,EAAE;QACvB,IAAI,EAAE,SAAS;QACf,OAAO,EAAE,KAAK;QACd,WAAW,EAAE,qDAAqD;KACnE;IACD,mBAAmB,EAAE;QACnB,IAAI,EAAE,SAAS;QACf,OAAO,EAAE,KAAK;QACd,WAAW,EAAE,iFAAiF;KAC/F;IACD,yBAAyB,EAAE;QACzB,IAAI,EAAE,SAAS;QACf,OAAO,EAAE,KAAK;QACd,WAAW,EAAE,gGAAgG;KAC9G;IACD,2BAA2B,EAAE;QAC3B,IAAI,EAAE,QAAQ;QACd,OAAO,EAAE,UAAU;QACnB,WAAW,EACT,sGAAsG;QACxG,IAAI,EAAE,CAAC,MAAM,EAAE,UAAU,CAAC;KAC3B;IACD,4BAA4B,EAAE;QAC5B,IAAI,EAAE,QAAQ;QACd,OAAO,EAAE,MAAM;QACf,WAAW,EACT,iHAAiH;QACnH,IAAI,EAAE,CAAC,MAAM,EAAE,UAAU,EAAE,MAAM,CAAC;KACnC;IACD,sCAAsC,EAAE;QACtC,IAAI,EAAE,QAAQ;QACd,OAAO,EAAE,EAAE;QACX,WAAW,EACT,iHAAiH;QACnH,IAAI,EAAE,CAAC,EAAE,EAAE,MAAM,EAAE,UAAU,CAAC;KAC/B;IACD,sCAAsC,EAAE;QACtC,IAAI,EAAE,QAAQ;QACd,OAAO,EAAE,EAAE;QACX,WAAW,EACT,oHAAoH;QACtH,IAAI,EAAE,CAAC,EAAE,EAAE,MAAM,EAAE,UAAU,CAAC;KAC/B;IACD,uCAAuC,EAAE;QACvC,IAAI,EAAE,QAAQ;QACd,OAAO,EAAE,EAAE;QACX,WAAW,EACT,gHAAgH;QAClH,IAAI,EAAE,CAAC,EAAE,EAAE,MAAM,EAAE,UAAU,CAAC;KAC/B;IACD,uCAAuC,EAAE;QACvC,IAAI,EAAE,QAAQ;QACd,OAAO,EAAE,EAAE;QACX,WAAW,EACT,yHAAyH;QAC3H,IAAI,EAAE,CAAC,EAAE,EAAE,MAAM,EAAE,UAAU,CAAC;KAC/B;IACD,qCAAqC,EAAE;QACrC,IAAI,EAAE,QAAQ;QACd,OAAO,EAAE,EAAE;QACX,WAAW,EACT,4VAA4V;QAC9V,QAAQ,EAAE,CAAC,KAAK,EAAE,EAAE;YAClB,IAAI,OAAO,KAAK,KAAK,QAAQ;gBAAE,OAAO,kBAAkB,CAAC;YACzD,IAAI,KAAK,KAAK,EAAE;gBAAE,OAAO,IAAI,CAAC,CAAC,uCAAuC;YACtE,IAAI,CAAC,SAAS,CAAC,IAAI,CAAC,KAAK,CAAC;gBAAE,OAAO,8CAA8C,CAAC;YAClF,OAAO,IAAI,CAAC;QACd,CAAC;KACF;IACD,wBAAwB,EAAE;QACxB,IAAI,EAAE,QAAQ;QACd,OAAO,EAAE,EAAE;QACX,WAAW,EACT,0VAA0V;QAC5V,QAAQ,EAAE,CAAC,KAAK,EAAE,EAAE;YAClB,IAAI,OAAO,KAAK,KAAK,QAAQ;gBAAE,OAAO,kBAAkB,CAAC;YACzD,IAAI,KAAK,KAAK,EAAE;gBAAE,OAAO,IAAI,CAAC,CAAC,uCAAuC;YACtE,IAAI,KAAK,CAAC,UAAU,CAAC,GAAG,CAAC;gBAAE,OAAO,gCAAgC,CAAC;YACnE,IAAI,KAAK,CAAC,UAAU,CAAC,IAAI,CAAC,IAAI,YAAY,CAAC,IAAI,CAAC,KAAK,CAAC,EAAE,CAAC;gBACvD,OAAO,gDAAgD,CAAC;YAC1D,CAAC;YACD,IAAI,KAAK,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,QAAQ,CAAC,IAAI,CAAC;gBAAE,OAAO,gCAAgC,CAAC;YACjF,IAAI,KAAK,CAAC,QAAQ,CAAC,GAAG,CAAC,IAAI,KAAK,CAAC,QAAQ,CAAC,IAAI,CAAC;gBAAE,OAAO,gCAAgC,CAAC;YACzF,OAAO,IAAI,CAAC;QACd,CAAC;KACF;CACF,CAAC"}

package/dist/introspection/ConsistencyChecker.d.ts

@@ -1,13 +1,15 @@
 import type { ProjectData } from '../types/project.js';
 import type { IArtifactIntrospector } from './interfaces.js';
 import type { ConsistencyCheckResult, ConsistencyFixResult } from './types.js';
+import type { ConfigManager } from '../config/ConfigManager.js';
 /**
  * Compares related artifact states within a project and flags mismatches.
  * Optionally applies non-destructive corrections to fixable findings.
  */
 export declare class ConsistencyChecker {
     private readonly introspector;
-    constructor(introspector: IArtifactIntrospector);
+    private readonly config?;
+    constructor(introspector: IArtifactIntrospector, config?: ConfigManager | undefined);
     /**
      * Run detection rules against the active slice and return structured findings.
      */
@@ -39,10 +41,31 @@ export declare class ConsistencyChecker {
     private rulePlanVsFrontmatter;
     /** Rule 5: Task file frontmatter status vs. computed task completion */
     private ruleTaskFileStatus;
+    /** Rule: slice marked complete in the plan but its code review is absent or failing. */
+    /**
+     * Wraps a single evaluateReviewGate() call so a throw (e.g. malformed review-gate
+     * frontmatter) degrades to one error-severity finding instead of aborting the whole
+     * check/checkAll run. Follows the safe*() convention used elsewhere in this class.
+     */
+    private safeEvaluateGate;
+    /**
+     * Builds the pending-review/review-failed finding for a resolved (non-null, non-error)
+     * gate evaluation. Shared by ruleReviewGate (slice boundaries) and ruleArchReviewGate
+     * (arch boundary) — they differ only in entity label and pending-review location.
+     */
+    private buildGateFinding;
+    private ruleReviewGate;
     /** Rule 6: Duplicate slice index detection */
     private ruleDuplicateIndex;
     /** Rule 7: Plan status vs. all-entries-complete */
     private rulePlanStatusVsEntries;
+    /**
+     * Rule 9: pre-slice-plan (arch) review gate — checkAll() only. The arch boundary
+     * is keyed by arch index, not slice index, so it doesn't belong in per-slice
+     * ruleReviewGate(); it audits every discovered architecture file the same way
+     * ruleArchStatusVsPlans does.
+     */
+    private ruleArchReviewGate;
     /** Rule 8: Architecture status vs. all-plans-complete */
     private ruleArchStatusVsPlans;
     /** Find the initiative plan file for a project. Delegates to shared utility. */

package/dist/introspection/ConsistencyChecker.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"ConsistencyChecker.d.ts","sourceRoot":"","sources":["../../src/introspection/ConsistencyChecker.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,qBAAqB,CAAC;AAGvD,OAAO,KAAK,EAAE,qBAAqB,EAAE,MAAM,iBAAiB,CAAC;AAC7D,OAAO,KAAK,EAEV,sBAAsB,EACtB,oBAAoB,EAKrB,MAAM,YAAY,CAAC;AAcpB;;;GAGG;AACH,qBAAa,kBAAkB;IAC7B,OAAO,CAAC,QAAQ,CAAC,YAAY,CAAwB;gBAEzC,YAAY,EAAE,qBAAqB;IAI/C;;OAEG;IACG,KAAK,CAAC,OAAO,EAAE,WAAW,GAAG,OAAO,CAAC,sBAAsB,CAAC;IAqBlE;;OAEG;IACG,QAAQ,CAAC,OAAO,EAAE,WAAW,GAAG,OAAO,CAAC,sBAAsB,CAAC;IA6FrE;;OAEG;IACG,GAAG,CAAC,OAAO,EAAE,WAAW,GAAG,OAAO,CAAC,oBAAoB,CAAC;IAK9D;;;OAGG;IACG,MAAM,CAAC,OAAO,EAAE,WAAW,GAAG,OAAO,CAAC,oBAAoB,CAAC;IAKjE,oEAAoE;IAC9D,UAAU,CAAC,WAAW,EAAE,sBAAsB,GAAG,OAAO,CAAC,oBAAoB,CAAC;IAoCpF,gFAAgF;YAClE,UAAU;IAsCxB,sDAAsD;IACtD,OAAO,CAAC,cAAc;IAiDtB,oDAAoD;IACpD,OAAO,CAAC,yBAAyB;IAoDjC,gDAAgD;IAChD,OAAO,CAAC,oBAAoB;IAuC5B,yDAAyD;IACzD,OAAO,CAAC,qBAAqB;IAoD7B,wEAAwE;IACxE,OAAO,CAAC,kBAAkB;IAmD1B,8CAA8C;IAC9C,OAAO,CAAC,kBAAkB;IA6B1B,mDAAmD;YACrC,uBAAuB;IAyDrC,yDAAyD;YAC3C,qBAAqB;IAsDnC,gFAAgF;YAClE,kBAAkB;IAIhC,kEAAkE;YACpD,yBAAyB;IA0FvC,0EAA0E;YAC5D,iCAAiC;IAuD/C,kFAAkF;YACpE,qBAAqB;IAqDnC,mFAAmF;IACnF,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,aAAa,CAOnC;IAEF,iEAAiE;YACnD,oBAAoB;IAqBlC,gEAAgE;YAClD,qBAAqB;IA8CnC,OAAO,CAAC,WAAW;IAYnB,OAAO,CAAC,WAAW;IAoBnB,mEAAmE;YACrD,qBAAqB;IAanC,qEAAqE;YACvD,oBAAoB;IAalC,iFAAiF;IACjF,OAAO,CAAC,MAAM,CAAC,gBAAgB;IAM/B;;;OAGG;YACW,qBAAqB;IA8CnC,OAAO,CAAC,oBAAoB;YAMd,mBAAmB;YAYnB,kBAAkB;YAalB,iBAAiB;YAajB,oBAAoB;YAYpB,4BAA4B;CAY3C"}
+{"version":3,"file":"ConsistencyChecker.d.ts","sourceRoot":"","sources":["../../src/introspection/ConsistencyChecker.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,qBAAqB,CAAC;AAGvD,OAAO,KAAK,EAAE,qBAAqB,EAAE,MAAM,iBAAiB,CAAC;AAE7D,OAAO,KAAK,EAEV,sBAAsB,EACtB,oBAAoB,EAKrB,MAAM,YAAY,CAAC;AAIpB,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,4BAA4B,CAAC;AAmBhE;;;GAGG;AACH,qBAAa,kBAAkB;IAE3B,OAAO,CAAC,QAAQ,CAAC,YAAY;IAC7B,OAAO,CAAC,QAAQ,CAAC,MAAM,CAAC;gBADP,YAAY,EAAE,qBAAqB,EACnC,MAAM,CAAC,EAAE,aAAa,YAAA;IAGzC;;OAEG;IACG,KAAK,CAAC,OAAO,EAAE,WAAW,GAAG,OAAO,CAAC,sBAAsB,CAAC;IAsBlE;;OAEG;IACG,QAAQ,CAAC,OAAO,EAAE,WAAW,GAAG,OAAO,CAAC,sBAAsB,CAAC;IAwGrE;;OAEG;IACG,GAAG,CAAC,OAAO,EAAE,WAAW,GAAG,OAAO,CAAC,oBAAoB,CAAC;IAK9D;;;OAGG;IACG,MAAM,CAAC,OAAO,EAAE,WAAW,GAAG,OAAO,CAAC,oBAAoB,CAAC;IAKjE,oEAAoE;IAC9D,UAAU,CAAC,WAAW,EAAE,sBAAsB,GAAG,OAAO,CAAC,oBAAoB,CAAC;IAoCpF,gFAAgF;YAClE,UAAU;IA2CxB,sDAAsD;IACtD,OAAO,CAAC,cAAc;IAgEtB,oDAAoD;IACpD,OAAO,CAAC,yBAAyB;IAuEjC,gDAAgD;IAChD,OAAO,CAAC,oBAAoB;IAuC5B,yDAAyD;IACzD,OAAO,CAAC,qBAAqB;IAoD7B,wEAAwE;IACxE,OAAO,CAAC,kBAAkB;IAiD1B,wFAAwF;IACxF;;;;OAIG;YACW,gBAAgB;IA6B9B;;;;OAIG;IACH,OAAO,CAAC,gBAAgB;YA6BV,cAAc;IA4C5B,8CAA8C;IAC9C,OAAO,CAAC,kBAAkB;IA6B1B,mDAAmD;YACrC,uBAAuB;IAyDrC;;;;;OAKG;YACW,kBAAkB;IAqChC,yDAAyD;YAC3C,qBAAqB;IAsDnC,gFAAgF;YAClE,kBAAkB;IAIhC,kEAAkE;YACpD,yBAAyB;IA0FvC,0EAA0E;YAC5D,iCAAiC;IAuD/C,kFAAkF;YACpE,qBAAqB;IAqDnC,mFAAmF;IACnF,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,aAAa,CAOnC;IAEF,iEAAiE;YACnD,oBAAoB;IAqBlC,gEAAgE;YAClD,qBAAqB;IA8CnC,OAAO,CAAC,WAAW;IAYnB,OAAO,CAAC,WAAW;IAoBnB,mEAAmE;YACrD,qBAAqB;IAanC,qEAAqE;YACvD,oBAAoB;IAalC,iFAAiF;IACjF,OAAO,CAAC,MAAM,CAAC,gBAAgB;IAM/B;;;OAGG;YACW,qBAAqB;IA8CnC,OAAO,CAAC,oBAAoB;YAMd,mBAAmB;YAYnB,kBAAkB;YAalB,iBAAiB;YAajB,oBAAoB;YAYpB,4BAA4B;CAY3C"}

package/dist/introspection/ConsistencyChecker.js

@@ -2,9 +2,11 @@ import { join, relative } from 'node:path';
 import { existsSync } from 'node:fs';
 import { readdir } from 'node:fs/promises';
 import { validateFrontmatter } from '../schema/frontmatterSchema.js';
+import { STATUS } from './types.js';
 import { resolveArtifactPath } from '../schema/resolveFileByIndex.js';
 import { updateCheckbox, updateFrontmatterField } from './writers/markdownWriter.js';
 import { resolveInitiativePlanPath } from './ArtifactIntrospector.js';
+import { resolveGateConfig, evaluateReviewGate, positionToReviewType, } from './reviewGate.js';
 /**
  * Extract numeric slice index from a fileSlice value like "165-slice.workflow-navigator".
  */
@@ -20,8 +22,10 @@ function extractSliceIndex(fileSlice) {
  */
 export class ConsistencyChecker {
     introspector;
-    constructor(introspector) {
+    config;
+    constructor(introspector, config) {
         this.introspector = introspector;
+        this.config = config;
     }
     /**
      * Run detection rules against the active slice and return structured findings.
@@ -37,7 +41,8 @@ export class ConsistencyChecker {
         }
         const slicePlanResult = await this.safeParseSlicePlan(project, projectPath);
         const slicePlanPath = this.resolveSlicePlanPath(project, projectPath);
-        const findings = await this.checkSlice(projectPath, sliceIndex, slicePlanResult, slicePlanPath);
+        const resolvedGate = this.config ? await resolveGateConfig(this.config) : null;
+        const findings = await this.checkSlice(projectPath, sliceIndex, slicePlanResult, slicePlanPath, resolvedGate);
         return this.buildResult(projectPath, findings);
     }
     /**
@@ -63,6 +68,12 @@ export class ConsistencyChecker {
                 const planResult = await this.introspector.parseSlicePlan(planPath);
                 parsedPlans.set(planPath, planResult);
                 for (const entry of planResult.entries) {
+                    // Fallback (unindexed-format) indices are per-file sequential counters,
+                    // not real project-wide slice indices — they must never enter the
+                    // cross-plan index space, or they collide with real indices from other
+                    // plans (see slice 913 TD-1).
+                    if (entry.indexSource === 'fallback')
+                        continue;
                     if (!uniqueEntries.has(entry.index)) {
                         uniqueEntries.set(entry.index, { entry, planPath, planResult });
                     }
@@ -73,9 +84,10 @@ export class ConsistencyChecker {
             }
         }
         const allFindings = [];
+        const resolvedGate = this.config ? await resolveGateConfig(this.config) : null;
         // Run rules 1-5 for each unique slice entry across all plans
         for (const [, { entry, planPath, planResult }] of uniqueEntries) {
-            const sliceFindings = await this.checkSlice(projectPath, entry.index, planResult, planPath);
+            const sliceFindings = await this.checkSlice(projectPath, entry.index, planResult, planPath, resolvedGate);
             // Prefix each finding's description with [sliceIndex] for attribution
             for (const finding of sliceFindings) {
                 finding.description = `[${entry.index}] ${finding.description}`;
@@ -92,6 +104,8 @@ export class ConsistencyChecker {
         for (const { archPath, planResult } of archPlanPairs) {
             allFindings.push(...await this.ruleArchStatusVsPlans(archPath, planResult));
         }
+        // Rule 9: pre-slice-plan (arch) review gate — every discovered arch file, not just paired ones
+        allFindings.push(...await this.ruleArchReviewGate(project, projectPath, resolvedGate));
         // Rule 10: stale worktree paths
         allFindings.push(...await this.ruleStaleWorktreePath(project, projectPath));
         // Rules 13 & 14: initiative plan checks
@@ -158,7 +172,7 @@ export class ConsistencyChecker {
     }
     // --- Per-slice check logic ---
     /** Run rules 1-5 against a single slice, returning raw findings (no prefix). */
-    async checkSlice(projectPath, sliceIndex, slicePlanResult, slicePlanPath) {
+    async checkSlice(projectPath, sliceIndex, slicePlanResult, slicePlanPath, resolvedGate) {
         const docs = await this.safeDetectDocuments(projectPath, sliceIndex);
         const taskResult = await this.safeParseTaskFile(docs?.taskFile, projectPath);
         const sliceFrontmatter = await this.safeParseFrontmatter(docs?.sliceDesign, projectPath);
@@ -171,6 +185,7 @@ export class ConsistencyChecker {
         findings.push(...this.rulePlanVsFrontmatter(planEntry, sliceFrontmatter, slicePlanPath, sliceDesignRel, projectPath));
         const taskFileFrontmatter = await this.safeParseTaskFileFrontmatter(docs?.taskFile, projectPath);
         findings.push(...this.ruleTaskFileStatus(taskResult, taskFileFrontmatter));
+        findings.push(...await this.ruleReviewGate(planEntry, sliceIndex, projectPath, slicePlanPath, resolvedGate, docs));
         return findings;
     }
     // --- Detection Rules ---
@@ -179,7 +194,7 @@ export class ConsistencyChecker {
         const findings = [];
         if (!taskResult || !planEntry || !slicePlanPath || !slicePlanResult)
             return findings;
-        const tasksComplete = taskResult.inferredStatus === 'complete';
+        const tasksComplete = taskResult.inferredStatus === STATUS.Complete;
         const sliceChecked = planEntry.isChecked;
         if (tasksComplete && !sliceChecked) {
             findings.push({
@@ -211,6 +226,20 @@ export class ConsistencyChecker {
                 },
             });
         }
+        // Not-started boundary (TD-4): tasks have started (any checked, not all) but
+        // the plan entry is still unchecked. Not fixable — there is no single correct
+        // auto-fix for "in progress" (don't check the box; there's no in-progress
+        // checkbox state to write).
+        if (taskResult.inferredStatus === STATUS.InProgress && !sliceChecked) {
+            findings.push({
+                rule: 'task-vs-plan',
+                severity: 'warning',
+                location: slicePlanPath,
+                description: `Tasks in progress (${taskResult.completedTasks}/${taskResult.totalTasks}) but slice ${sliceIndex} is unchecked in plan`,
+                suggestedFix: `No single auto-fix for in-progress — leave the slice plan entry for (${sliceIndex}) unchecked until tasks complete`,
+                fixable: false,
+            });
+        }
         return findings;
     }
     /** Rule 2: Frontmatter status vs. computed state */
@@ -221,11 +250,11 @@ export class ConsistencyChecker {
         if (!taskResult)
             return findings;
         const fmStatus = frontmatter.data.status.toLowerCase();
-        const tasksComplete = taskResult.inferredStatus === 'complete';
+        const tasksComplete = taskResult.inferredStatus === STATUS.Complete;
         const sliceDesignFullPath = sliceDesignRelPath
             ? join(projectPath, sliceDesignRelPath)
             : frontmatter.filePath;
-        if (fmStatus === 'complete' && !tasksComplete) {
+        if (fmStatus === STATUS.Complete && !tasksComplete) {
             findings.push({
                 rule: 'frontmatter-vs-computed',
                 severity: 'error',
@@ -236,11 +265,11 @@ export class ConsistencyChecker {
                 fixAction: {
                     type: 'update-frontmatter',
                     filePath: sliceDesignFullPath,
-                    detail: { key: 'status', value: 'in-progress' },
+                    detail: { key: 'status', value: STATUS.InProgress },
                 },
             });
         }
-        if ((fmStatus === 'in-progress' || fmStatus === 'not-started') && tasksComplete) {
+        if ((fmStatus === STATUS.InProgress || fmStatus === STATUS.NotStarted) && tasksComplete) {
             findings.push({
                 rule: 'frontmatter-vs-computed',
                 severity: 'warning',
@@ -251,7 +280,25 @@ export class ConsistencyChecker {
                 fixAction: {
                     type: 'update-frontmatter',
                     filePath: sliceDesignFullPath,
-                    detail: { key: 'status', value: 'complete' },
+                    detail: { key: 'status', value: STATUS.Complete },
+                },
+            });
+        }
+        // Not-started boundary (TD-4): tasks have started but design frontmatter
+        // still says not-started. Mirrors the branch above — same fix-action shape,
+        // different target value.
+        if (fmStatus === STATUS.NotStarted && taskResult.inferredStatus === STATUS.InProgress) {
+            findings.push({
+                rule: 'frontmatter-vs-computed',
+                severity: 'warning',
+                location: sliceDesignFullPath,
+                description: `Frontmatter status is "not-started" but tasks are in progress (${taskResult.completedTasks}/${taskResult.totalTasks})`,
+                suggestedFix: 'Update frontmatter status to "in-progress"',
+                fixable: true,
+                fixAction: {
+                    type: 'update-frontmatter',
+                    filePath: sliceDesignFullPath,
+                    detail: { key: 'status', value: STATUS.InProgress },
                 },
             });
         }
@@ -298,7 +345,7 @@ export class ConsistencyChecker {
         const sliceDesignFullPath = sliceDesignRelPath
             ? join(projectPath, sliceDesignRelPath)
             : frontmatter.filePath;
-        if (planChecked && fmStatus !== 'complete') {
+        if (planChecked && fmStatus !== STATUS.Complete) {
             findings.push({
                 rule: 'plan-vs-frontmatter',
                 severity: 'warning',
@@ -309,11 +356,11 @@ export class ConsistencyChecker {
                 fixAction: {
                     type: 'update-frontmatter',
                     filePath: sliceDesignFullPath,
-                    detail: { key: 'status', value: 'complete' },
+                    detail: { key: 'status', value: STATUS.Complete },
                 },
             });
         }
-        if (!planChecked && fmStatus === 'complete' && slicePlanPath) {
+        if (!planChecked && fmStatus === STATUS.Complete && slicePlanPath) {
             findings.push({
                 rule: 'plan-vs-frontmatter',
                 severity: 'warning',
@@ -336,11 +383,11 @@ export class ConsistencyChecker {
         if (!taskResult || !taskFrontmatter?.found || !taskFrontmatter.data.status)
             return findings;
         const fmStatus = taskFrontmatter.data.status.toLowerCase().replace(/_/g, '-');
-        const computed = taskResult.inferredStatus; // 'complete' | 'in-progress' | 'not-started'
+        const computed = taskResult.inferredStatus;
         const taskFilePath = taskFrontmatter.filePath;
         // Map task file status values to normalized form for comparison
         // Task files use not_started/in_progress/complete in frontmatter
-        if (fmStatus === 'complete' && computed !== 'complete') {
+        if (fmStatus === STATUS.Complete && computed !== STATUS.Complete) {
             findings.push({
                 rule: 'task-file-status',
                 severity: 'error',
@@ -355,7 +402,7 @@ export class ConsistencyChecker {
                 },
             });
         }
-        if (fmStatus !== 'complete' && computed === 'complete') {
+        if (fmStatus !== STATUS.Complete && computed === STATUS.Complete) {
             findings.push({
                 rule: 'task-file-status',
                 severity: 'warning',
@@ -366,12 +413,93 @@ export class ConsistencyChecker {
                 fixAction: {
                     type: 'update-frontmatter',
                     filePath: taskFilePath,
-                    detail: { key: 'status', value: 'complete' },
+                    detail: { key: 'status', value: STATUS.Complete },
                 },
             });
         }
         return findings;
     }
+    /** Rule: slice marked complete in the plan but its code review is absent or failing. */
+    /**
+     * Wraps a single evaluateReviewGate() call so a throw (e.g. malformed review-gate
+     * frontmatter) degrades to one error-severity finding instead of aborting the whole
+     * check/checkAll run. Follows the safe*() convention used elsewhere in this class.
+     */
+    async safeEvaluateGate(projectPath, index, boundary, resolvedGate, fallbackLocation) {
+        try {
+            const gate = await evaluateReviewGate(projectPath, index, boundary, this.config, resolvedGate);
+            return { gate, errorFinding: null };
+        }
+        catch (err) {
+            const reviewType = positionToReviewType(boundary);
+            const message = err instanceof Error ? err.message : String(err);
+            return {
+                gate: null,
+                errorFinding: {
+                    rule: 'review-gate',
+                    severity: 'error',
+                    location: fallbackLocation,
+                    description: `Failed to evaluate the ${reviewType} review gate for ${index}: ${message}`,
+                    suggestedFix: `Fix the frontmatter for ${index} and re-run the check`,
+                    fixable: false,
+                },
+            };
+        }
+    }
+    /**
+     * Builds the pending-review/review-failed finding for a resolved (non-null, non-error)
+     * gate evaluation. Shared by ruleReviewGate (slice boundaries) and ruleArchReviewGate
+     * (arch boundary) — they differ only in entity label and pending-review location.
+     */
+    buildGateFinding(result, boundary, entityLabel, index, projectPath, pendingLocation) {
+        const reviewType = positionToReviewType(boundary);
+        if (result.status === 'pending-review') {
+            return {
+                rule: 'review-gate',
+                severity: 'warning',
+                location: pendingLocation,
+                description: result.rationale,
+                suggestedFix: `Run the ${reviewType} review for ${entityLabel} ${index}`,
+                fixable: false,
+            };
+        }
+        return {
+            rule: 'review-gate',
+            severity: 'error',
+            location: join(projectPath, result.artifactPath),
+            description: result.rationale,
+            suggestedFix: `Resolve the review findings or rerun the ${reviewType} review for ${entityLabel} ${index}`,
+            fixable: false,
+        };
+    }
+    async ruleReviewGate(planEntry, sliceIndex, projectPath, slicePlanPath, resolvedGate, docs) {
+        if (!this.config || resolvedGate === null)
+            return [];
+        if (slicePlanPath === null)
+            return [];
+        const findings = [];
+        const boundaries = [
+            { boundary: 'preTasks', guard: docs?.sliceDesign !== null && docs?.sliceDesign !== undefined },
+            { boundary: 'preImplementation', guard: !!docs?.taskFile && docs.taskFile.length > 0 },
+            {
+                boundary: 'preAdvance',
+                guard: !!planEntry?.isChecked && docs?.sliceDesign !== null && docs?.sliceDesign !== undefined,
+            },
+        ];
+        for (const { boundary, guard } of boundaries) {
+            if (!guard)
+                continue;
+            const { gate: result, errorFinding } = await this.safeEvaluateGate(projectPath, sliceIndex, boundary, resolvedGate, slicePlanPath);
+            if (errorFinding) {
+                findings.push(errorFinding);
+                continue;
+            }
+            if (result === null)
+                continue;
+            findings.push(this.buildGateFinding(result, boundary, 'slice', sliceIndex, projectPath, slicePlanPath));
+        }
+        return findings;
+    }
     // --- Aggregate Rules (checkAll only) ---
     /** Rule 6: Duplicate slice index detection */
     ruleDuplicateIndex(entries, slicePlanPath) {
@@ -413,7 +541,7 @@ export class ConsistencyChecker {
         if (!planFrontmatter.data.status)
             return findings;
         const planStatus = planFrontmatter.data.status.toLowerCase();
-        if (planStatus === 'complete' && !allComplete) {
+        if (planStatus === STATUS.Complete && !allComplete) {
             findings.push({
                 rule: 'plan-status-vs-entries',
                 severity: 'warning',
@@ -424,11 +552,11 @@ export class ConsistencyChecker {
                 fixAction: {
                     type: 'update-frontmatter',
                     filePath: slicePlanPath,
-                    detail: { key: 'status', value: 'in-progress' },
+                    detail: { key: 'status', value: STATUS.InProgress },
                 },
             });
         }
-        if (planStatus !== 'complete' && allComplete && slicePlanResult.totalSlices > 0) {
+        if (planStatus !== STATUS.Complete && allComplete && slicePlanResult.totalSlices > 0) {
             findings.push({
                 rule: 'plan-status-vs-entries',
                 severity: 'warning',
@@ -439,12 +567,44 @@ export class ConsistencyChecker {
                 fixAction: {
                     type: 'update-frontmatter',
                     filePath: slicePlanPath,
-                    detail: { key: 'status', value: 'complete' },
+                    detail: { key: 'status', value: STATUS.Complete },
                 },
             });
         }
         return findings;
     }
+    /**
+     * Rule 9: pre-slice-plan (arch) review gate — checkAll() only. The arch boundary
+     * is keyed by arch index, not slice index, so it doesn't belong in per-slice
+     * ruleReviewGate(); it audits every discovered architecture file the same way
+     * ruleArchStatusVsPlans does.
+     */
+    async ruleArchReviewGate(project, projectPath, resolvedGate) {
+        if (!this.config || resolvedGate === null)
+            return [];
+        const archFiles = await this.discoverAllArchFiles(projectPath);
+        const allArchPaths = new Set(archFiles);
+        if (project.fileArch) {
+            const configuredArchRel = resolveArtifactPath('fileArch', project.fileArch);
+            if (configuredArchRel)
+                allArchPaths.add(join(projectPath, configuredArchRel));
+        }
+        const findings = [];
+        for (const archPath of allArchPaths) {
+            const archIndex = ConsistencyChecker.extractFileIndex(archPath);
+            if (archIndex === null)
+                continue;
+            const { gate: result, errorFinding } = await this.safeEvaluateGate(projectPath, archIndex, 'preSlicePlan', resolvedGate, archPath);
+            if (errorFinding) {
+                findings.push(errorFinding);
+                continue;
+            }
+            if (result === null)
+                continue;
+            findings.push(this.buildGateFinding(result, 'preSlicePlan', 'architecture', archIndex, projectPath, archPath));
+        }
+        return findings;
+    }
     /** Rule 8: Architecture status vs. all-plans-complete */
     async ruleArchStatusVsPlans(archPath, slicePlanResult) {
         const findings = [];
@@ -459,7 +619,7 @@ export class ConsistencyChecker {
             return findings;
         const archStatus = archFrontmatter.data.status.toLowerCase();
         const allComplete = slicePlanResult.completedSlices === slicePlanResult.totalSlices;
-        if (archStatus === 'complete' && !allComplete) {
+        if (archStatus === STATUS.Complete && !allComplete) {
             findings.push({
                 rule: 'arch-status-vs-plans',
                 severity: 'warning',
@@ -470,11 +630,11 @@ export class ConsistencyChecker {
                 fixAction: {
                     type: 'update-frontmatter',
                     filePath: archPath,
-                    detail: { key: 'status', value: 'in-progress' },
+                    detail: { key: 'status', value: STATUS.InProgress },
                 },
             });
         }
-        if (archStatus !== 'complete' && allComplete && slicePlanResult.totalSlices > 0) {
+        if (archStatus !== STATUS.Complete && allComplete && slicePlanResult.totalSlices > 0) {
             findings.push({
                 rule: 'arch-status-vs-plans',
                 severity: 'warning',
@@ -485,7 +645,7 @@ export class ConsistencyChecker {
                 fixAction: {
                     type: 'update-frontmatter',
                     filePath: archPath,
-                    detail: { key: 'status', value: 'complete' },
+                    detail: { key: 'status', value: STATUS.Complete },
                 },
             });
         }
@@ -520,7 +680,7 @@ export class ConsistencyChecker {
             if (!archFrontmatter.found || !archFrontmatter.data.status)
                 continue;
             const archStatus = archFrontmatter.data.status.toLowerCase();
-            const archComplete = archStatus === 'complete';
+            const archComplete = archStatus === STATUS.Complete;
             if (archComplete && !entry.isChecked) {
                 findings.push({
                     rule: 'initiative-entry-vs-arch',
@@ -547,7 +707,7 @@ export class ConsistencyChecker {
                     fixAction: {
                         type: 'update-frontmatter',
                         filePath: archPath,
-                        detail: { key: 'status', value: 'complete' },
+                        detail: { key: 'status', value: STATUS.Complete },
                     },
                 });
             }
@@ -591,7 +751,7 @@ export class ConsistencyChecker {
         const planStatus = planFrontmatter.data.status.toLowerCase();
         const allComplete = initiativePlanResult.totalSlices > 0 &&
             initiativePlanResult.completedSlices === initiativePlanResult.totalSlices;
-        if (planStatus === 'complete' && !allComplete) {
+        if (planStatus === STATUS.Complete && !allComplete) {
             findings.push({
                 rule: 'initiative-plan-status-vs-entries',
                 severity: 'warning',
@@ -602,11 +762,11 @@ export class ConsistencyChecker {
                 fixAction: {
                     type: 'update-frontmatter',
                     filePath: initiativePlanPath,
-                    detail: { key: 'status', value: 'in-progress' },
+                    detail: { key: 'status', value: STATUS.InProgress },
                 },
             });
         }
-        if (planStatus !== 'complete' && allComplete) {
+        if (planStatus !== STATUS.Complete && allComplete) {
             findings.push({
                 rule: 'initiative-plan-status-vs-entries',
                 severity: 'warning',
@@ -617,7 +777,7 @@ export class ConsistencyChecker {
                 fixAction: {
                     type: 'update-frontmatter',
                     filePath: initiativePlanPath,
-                    detail: { key: 'status', value: 'complete' },
+                    detail: { key: 'status', value: STATUS.Complete },
                 },
             });
         }