@open-agent-toolkit/cli 0.2.24 → 0.2.26

package/dist/review-remote/marker-parser.d.ts

@@ -13,7 +13,7 @@
  */
 /** Token that opens the marker comment block. */
 export declare const MARKER_BLOCK_OPEN = "oat-review-metadata";
-export type ReviewInvocation = 'manual' | 'auto';
+export type ReviewInvocation = 'manual' | 'auto' | 'gate';
 export interface MarkerBlock {
     /** Always `true` for an OAT provide-remote review; the discriminator. */
     oat_provide_remote: true;
@@ -26,8 +26,13 @@ export interface MarkerBlock {
      * `null` value) discriminates project rail from ad-hoc rail.
      */
     oat_project?: string;
-    /** How the review was invoked. Defaults to `manual` when omitted. */
-    oat_review_invocation: ReviewInvocation;
+    /**
+     * How the review was invoked. Missing and unknown values stay undefined so
+     * legacy markers cannot silently acquire lifecycle lineage.
+     */
+    oat_review_invocation?: ReviewInvocation;
+    /** Exact gate target. Required before a gate marker can establish lineage. */
+    oat_gate_target?: string;
     /** Forward-compat bag for unknown marker keys. Omitted when none seen. */
     extras?: Record<string, string>;
 }

package/dist/review-remote/marker-parser.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"marker-parser.d.ts","sourceRoot":"","sources":["../../src/review-remote/marker-parser.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;GAYG;AAEH,iDAAiD;AACjD,eAAO,MAAM,iBAAiB,wBAAwB,CAAC;AAavD,MAAM,MAAM,gBAAgB,GAAG,QAAQ,GAAG,MAAM,CAAC;AAEjD,MAAM,WAAW,WAAW;IAC1B,yEAAyE;IACzE,kBAAkB,EAAE,IAAI,CAAC;IACzB,oDAAoD;IACpD,mBAAmB,EAAE,MAAM,CAAC;IAC5B,gEAAgE;IAChE,gBAAgB,EAAE,MAAM,CAAC;IACzB;;;OAGG;IACH,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,qEAAqE;IACrE,qBAAqB,EAAE,gBAAgB,CAAC;IACxC,0EAA0E;IAC1E,MAAM,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;CACjC;AAUD;;;;;;;;GAQG;AACH,wBAAgB,gBAAgB,CAAC,IAAI,EAAE,MAAM,GAAG,WAAW,GAAG,IAAI,CAqEjE"}
+{"version":3,"file":"marker-parser.d.ts","sourceRoot":"","sources":["../../src/review-remote/marker-parser.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;GAYG;AAEH,iDAAiD;AACjD,eAAO,MAAM,iBAAiB,wBAAwB,CAAC;AAavD,MAAM,MAAM,gBAAgB,GAAG,QAAQ,GAAG,MAAM,GAAG,MAAM,CAAC;AAE1D,MAAM,WAAW,WAAW;IAC1B,yEAAyE;IACzE,kBAAkB,EAAE,IAAI,CAAC;IACzB,oDAAoD;IACpD,mBAAmB,EAAE,MAAM,CAAC;IAC5B,gEAAgE;IAChE,gBAAgB,EAAE,MAAM,CAAC;IACzB;;;OAGG;IACH,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB;;;OAGG;IACH,qBAAqB,CAAC,EAAE,gBAAgB,CAAC;IACzC,8EAA8E;IAC9E,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,0EAA0E;IAC1E,MAAM,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;CACjC;AAWD;;;;;;;;GAQG;AACH,wBAAgB,gBAAgB,CAAC,IAAI,EAAE,MAAM,GAAG,WAAW,GAAG,IAAI,CA+EjE"}

package/dist/review-remote/marker-parser.js

@@ -26,6 +26,7 @@ const KNOWN_KEYS = new Set([
     'oat_review_scope',
     'oat_project',
     'oat_review_invocation',
+    'oat_gate_target',
 ]);
 /**
  * Parse the first OAT marker block out of a posted-review body.
@@ -73,17 +74,27 @@ export function parseMarkerBlock(body) {
         return null;
     }
     const invocationRaw = raw['oat_review_invocation'];
-    const oat_review_invocation = invocationRaw === 'auto' ? 'auto' : 'manual';
+    const oat_review_invocation = invocationRaw === 'manual' ||
+        invocationRaw === 'auto' ||
+        invocationRaw === 'gate'
+        ? invocationRaw
+        : undefined;
     const block = {
         oat_provide_remote: true,
         oat_review_head_sha: headSha,
         oat_review_scope: scope,
-        oat_review_invocation,
     };
+    if (oat_review_invocation !== undefined) {
+        block.oat_review_invocation = oat_review_invocation;
+    }
     // Project key existence (not a null value) discriminates the rail.
     if ('oat_project' in raw && raw['oat_project'] !== '') {
         block.oat_project = raw['oat_project'];
     }
+    const gateTarget = raw['oat_gate_target']?.trim();
+    if (gateTarget) {
+        block.oat_gate_target = gateTarget;
+    }
     const extras = {};
     for (const [key, value] of Object.entries(raw)) {
         if (!KNOWN_KEYS.has(key)) {

package/dist/review-remote/narrowing.d.ts

@@ -9,8 +9,15 @@
  * range, so a rebased/force-pushed/shallow prior SHA never produces a
  * misleading partial range.
  */
-import type { ReviewInvocation } from './marker-parser.js';
+import type { MarkerBlock, ReviewInvocation } from './marker-parser.js';
 export type ReviewRail = 'ad-hoc' | 'project';
+export type ReviewLineage = {
+    kind: 'lifecycle';
+} | {
+    kind: 'gate';
+    target: string;
+};
+export type ReviewInvocationKind = ReviewInvocation;
 /** A prior provide-remote review, distilled from its parsed marker block. */
 export interface PriorReview {
     /** Full 40-char SHA recorded by the prior review. */
@@ -19,10 +26,34 @@ export interface PriorReview {
     scope: string;
     /** Project path — present only for project-rail reviews. */
     project?: string;
-    invocation: ReviewInvocation;
+    invocation?: ReviewInvocationKind;
+    /** Review lineage; absent on legacy records, which are not eligible. */
+    lineage?: ReviewLineage;
     /** ISO-8601 review submission timestamp, used for descending sort. */
     submittedAt: string;
 }
+/** Provenance persisted on one tracked Reviews ledger row. */
+export interface ReviewLedgerProvenance {
+    reviewedHead: string;
+    scope: string;
+    project?: string;
+    invocation?: string;
+    gateTarget?: string;
+    artifact: string;
+    submittedAt: string;
+}
+/**
+ * Normalize one parsed GitHub marker into a narrowing candidate. Unknown
+ * invocation and target-less gate markers preserve their metadata but carry no
+ * lineage, so tuple matching rejects them and fails open.
+ */
+export declare function priorReviewFromMarker(marker: MarkerBlock, submittedAt: string): PriorReview;
+/**
+ * Normalize durable ledger provenance into the same candidate shape used by
+ * artifact-sourced reviews. Both sources then pass through {@link matchesTuple}
+ * and cannot drift into separate lineage predicates.
+ */
+export declare function priorReviewFromLedger(row: ReviewLedgerProvenance): PriorReview;
 /**
  * Narrow git surface needed by the guard. Callers pass a worktree-bound
  * implementation (rich-context mode) or a fetch-capable one (diff-only mode);
@@ -35,6 +66,8 @@ export interface GitInvoker {
     isAncestor(sha: string, head: string): Promise<boolean>;
     /** `git fetch origin <sha>:<ref>` — fetch a single ref (diff-only mode). */
     fetchRef(sha: string): Promise<boolean>;
+    /** Files changed in `<priorSha>..<headSha>`. */
+    changedFiles(priorSha: string, headSha: string): Promise<string[]>;
 }
 export interface NarrowingInput {
     reviews: PriorReview[];
@@ -43,30 +76,33 @@ export interface NarrowingInput {
     project: string | null;
     /** Current scope token, or `ad-hoc`. */
     scope: string;
+    /** Current review lineage, including the target for gate invocations. */
+    lineage: ReviewLineage;
     /** Current PR HEAD SHA. */
     headSha: string;
     git: GitInvoker;
     /** `--narrow` was passed explicitly: guard failure becomes a hard error. */
     forceNarrow?: boolean;
-    /** `workflow.autoNarrowReReviewScope === true`: never prompt. */
-    autoNarrow?: boolean;
+    /** Resolved preference: unset and true narrow; false forces full scope. */
+    narrowingPreference?: boolean;
     /** Diff-only mode (no ephemeral worktree): fetch the single ref first. */
     diffOnly?: boolean;
 }
-export type FullScopeReason = 'no-prior-review' | 'stale-sha';
+export type FullScopeReason = 'narrowing-disabled' | 'no-prior-review' | 'stale-sha';
+export type RangeClassification = 'empty' | 'bookkeeping-only' | 'substantive';
 export interface NarrowRangeResult {
     kind: 'narrow-range';
     priorSha: string;
     headSha: string;
-    /** Whether the caller still owes the user a confirm prompt. */
-    prompted: boolean;
+    classification: RangeClassification;
+    /** Why classification conservatively fell back, when enumeration failed. */
+    classificationReason?: 'changed-files-unavailable';
 }
 export interface FullScopeFallbackResult {
     kind: 'full-scope-fallback';
     reason: FullScopeReason;
     /** The stale SHA that triggered the fallback, when applicable. */
     priorSha?: string;
-    prompted: boolean;
 }
 export interface HardErrorResult {
     kind: 'hard-error';

package/dist/review-remote/narrowing.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"narrowing.d.ts","sourceRoot":"","sources":["../../src/review-remote/narrowing.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AAEH,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,iBAAiB,CAAC;AAExD,MAAM,MAAM,UAAU,GAAG,QAAQ,GAAG,SAAS,CAAC;AAE9C,6EAA6E;AAC7E,MAAM,WAAW,WAAW;IAC1B,qDAAqD;IACrD,OAAO,EAAE,MAAM,CAAC;IAChB,4CAA4C;IAC5C,KAAK,EAAE,MAAM,CAAC;IACd,4DAA4D;IAC5D,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,UAAU,EAAE,gBAAgB,CAAC;IAC7B,sEAAsE;IACtE,WAAW,EAAE,MAAM,CAAC;CACrB;AAED;;;;GAIG;AACH,MAAM,WAAW,UAAU;IACzB,+DAA+D;IAC/D,YAAY,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,CAAC,CAAC;IAC5C,yEAAyE;IACzE,UAAU,CAAC,GAAG,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,CAAC,CAAC;IACxD,4EAA4E;IAC5E,QAAQ,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,CAAC,CAAC;CACzC;AAED,MAAM,WAAW,cAAc;IAC7B,OAAO,EAAE,WAAW,EAAE,CAAC;IACvB,IAAI,EAAE,UAAU,CAAC;IACjB,4EAA4E;IAC5E,OAAO,EAAE,MAAM,GAAG,IAAI,CAAC;IACvB,wCAAwC;IACxC,KAAK,EAAE,MAAM,CAAC;IACd,2BAA2B;IAC3B,OAAO,EAAE,MAAM,CAAC;IAChB,GAAG,EAAE,UAAU,CAAC;IAChB,4EAA4E;IAC5E,WAAW,CAAC,EAAE,OAAO,CAAC;IACtB,iEAAiE;IACjE,UAAU,CAAC,EAAE,OAAO,CAAC;IACrB,0EAA0E;IAC1E,QAAQ,CAAC,EAAE,OAAO,CAAC;CACpB;AAED,MAAM,MAAM,eAAe,GAAG,iBAAiB,GAAG,WAAW,CAAC;AAE9D,MAAM,WAAW,iBAAiB;IAChC,IAAI,EAAE,cAAc,CAAC;IACrB,QAAQ,EAAE,MAAM,CAAC;IACjB,OAAO,EAAE,MAAM,CAAC;IAChB,+DAA+D;IAC/D,QAAQ,EAAE,OAAO,CAAC;CACnB;AAED,MAAM,WAAW,uBAAuB;IACtC,IAAI,EAAE,qBAAqB,CAAC;IAC5B,MAAM,EAAE,eAAe,CAAC;IACxB,kEAAkE;IAClE,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,QAAQ,EAAE,OAAO,CAAC;CACnB;AAED,MAAM,WAAW,eAAe;IAC9B,IAAI,EAAE,YAAY,CAAC;IACnB,MAAM,EAAE,WAAW,CAAC;IACpB,QAAQ,EAAE,MAAM,CAAC;CAClB;AAED,MAAM,MAAM,eAAe,GACvB,iBAAiB,GACjB,uBAAuB,GACvB,eAAe,CAAC;AAiDpB;;GAEG;AACH,wBAAsB,mBAAmB,CACvC,KAAK,EAAE,cAAc,GACpB,OAAO,CAAC,eAAe,CAAC,CAuC1B"}
+{"version":3,"file":"narrowing.d.ts","sourceRoot":"","sources":["../../src/review-remote/narrowing.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AAEH,OAAO,KAAK,EAAE,WAAW,EAAE,gBAAgB,EAAE,MAAM,iBAAiB,CAAC;AAIrE,MAAM,MAAM,UAAU,GAAG,QAAQ,GAAG,SAAS,CAAC;AAE9C,MAAM,MAAM,aAAa,GACrB;IAAE,IAAI,EAAE,WAAW,CAAA;CAAE,GACrB;IAAE,IAAI,EAAE,MAAM,CAAC;IAAC,MAAM,EAAE,MAAM,CAAA;CAAE,CAAC;AAErC,MAAM,MAAM,oBAAoB,GAAG,gBAAgB,CAAC;AAEpD,6EAA6E;AAC7E,MAAM,WAAW,WAAW;IAC1B,qDAAqD;IACrD,OAAO,EAAE,MAAM,CAAC;IAChB,4CAA4C;IAC5C,KAAK,EAAE,MAAM,CAAC;IACd,4DAA4D;IAC5D,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,UAAU,CAAC,EAAE,oBAAoB,CAAC;IAClC,wEAAwE;IACxE,OAAO,CAAC,EAAE,aAAa,CAAC;IACxB,sEAAsE;IACtE,WAAW,EAAE,MAAM,CAAC;CACrB;AAED,8DAA8D;AAC9D,MAAM,WAAW,sBAAsB;IACrC,YAAY,EAAE,MAAM,CAAC;IACrB,KAAK,EAAE,MAAM,CAAC;IACd,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,QAAQ,EAAE,MAAM,CAAC;IACjB,WAAW,EAAE,MAAM,CAAC;CACrB;AAED;;;;GAIG;AACH,wBAAgB,qBAAqB,CACnC,MAAM,EAAE,WAAW,EACnB,WAAW,EAAE,MAAM,GAClB,WAAW,CAYb;AAgBD;;;;GAIG;AACH,wBAAgB,qBAAqB,CACnC,GAAG,EAAE,sBAAsB,GAC1B,WAAW,CAcb;AAqBD;;;;GAIG;AACH,MAAM,WAAW,UAAU;IACzB,+DAA+D;IAC/D,YAAY,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,CAAC,CAAC;IAC5C,yEAAyE;IACzE,UAAU,CAAC,GAAG,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,CAAC,CAAC;IACxD,4EAA4E;IAC5E,QAAQ,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,CAAC,CAAC;IACxC,gDAAgD;IAChD,YAAY,CAAC,QAAQ,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,EAAE,CAAC,CAAC;CACpE;AAED,MAAM,WAAW,cAAc;IAC7B,OAAO,EAAE,WAAW,EAAE,CAAC;IACvB,IAAI,EAAE,UAAU,CAAC;IACjB,4EAA4E;IAC5E,OAAO,EAAE,MAAM,GAAG,IAAI,CAAC;IACvB,wCAAwC;IACxC,KAAK,EAAE,MAAM,CAAC;IACd,yEAAyE;IACzE,OAAO,EAAE,aAAa,CAAC;IACvB,2BAA2B;IAC3B,OAAO,EAAE,MAAM,CAAC;IAChB,GAAG,EAAE,UAAU,CAAC;IAChB,4EAA4E;IAC5E,WAAW,CAAC,EAAE,OAAO,CAAC;IACtB,2EAA2E;IAC3E,mBAAmB,CAAC,EAAE,OAAO,CAAC;IAC9B,0EAA0E;IAC1E,QAAQ,CAAC,EAAE,OAAO,CAAC;CACpB;AAED,MAAM,MAAM,eAAe,GACvB,oBAAoB,GACpB,iBAAiB,GACjB,WAAW,CAAC;AAChB,MAAM,MAAM,mBAAmB,GAAG,OAAO,GAAG,kBAAkB,GAAG,aAAa,CAAC;AAE/E,MAAM,WAAW,iBAAiB;IAChC,IAAI,EAAE,cAAc,CAAC;IACrB,QAAQ,EAAE,MAAM,CAAC;IACjB,OAAO,EAAE,MAAM,CAAC;IAChB,cAAc,EAAE,mBAAmB,CAAC;IACpC,4EAA4E;IAC5E,oBAAoB,CAAC,EAAE,2BAA2B,CAAC;CACpD;AAED,MAAM,WAAW,uBAAuB;IACtC,IAAI,EAAE,qBAAqB,CAAC;IAC5B,MAAM,EAAE,eAAe,CAAC;IACxB,kEAAkE;IAClE,QAAQ,CAAC,EAAE,MAAM,CAAC;CACnB;AAED,MAAM,WAAW,eAAe;IAC9B,IAAI,EAAE,YAAY,CAAC;IACnB,MAAM,EAAE,WAAW,CAAC;IACpB,QAAQ,EAAE,MAAM,CAAC;CAClB;AAED,MAAM,MAAM,eAAe,GACvB,iBAAiB,GACjB,uBAAuB,GACvB,eAAe,CAAC;AAgGpB;;GAEG;AACH,wBAAsB,mBAAmB,CACvC,KAAK,EAAE,cAAc,GACpB,OAAO,CAAC,eAAe,CAAC,CAyC1B"}

package/dist/review-remote/narrowing.js

@@ -9,16 +9,85 @@
  * range, so a rebased/force-pushed/shallow prior SHA never produces a
  * misleading partial range.
  */
-/** Does a prior review match the current `(rail, project, scope)` tuple? */
+const FULL_COMMIT_SHA = /^[0-9a-f]{40}$/i;
+/**
+ * Normalize one parsed GitHub marker into a narrowing candidate. Unknown
+ * invocation and target-less gate markers preserve their metadata but carry no
+ * lineage, so tuple matching rejects them and fails open.
+ */
+export function priorReviewFromMarker(marker, submittedAt) {
+    const lineage = markerLineage(marker);
+    return {
+        headSha: marker.oat_review_head_sha,
+        scope: marker.oat_review_scope,
+        ...(marker.oat_project ? { project: marker.oat_project } : {}),
+        ...(marker.oat_review_invocation
+            ? { invocation: marker.oat_review_invocation }
+            : {}),
+        ...(lineage ? { lineage } : {}),
+        submittedAt,
+    };
+}
+function markerLineage(marker) {
+    if (marker.oat_review_invocation === 'manual' ||
+        marker.oat_review_invocation === 'auto') {
+        return { kind: 'lifecycle' };
+    }
+    const gateTarget = marker.oat_gate_target?.trim();
+    return marker.oat_review_invocation === 'gate' && gateTarget
+        ? { kind: 'gate', target: gateTarget }
+        : undefined;
+}
+/**
+ * Normalize durable ledger provenance into the same candidate shape used by
+ * artifact-sourced reviews. Both sources then pass through {@link matchesTuple}
+ * and cannot drift into separate lineage predicates.
+ */
+export function priorReviewFromLedger(row) {
+    const invocation = isReviewInvocationKind(row.invocation)
+        ? row.invocation
+        : undefined;
+    const lineage = ledgerLineage(row);
+    return {
+        headSha: row.reviewedHead,
+        scope: row.scope,
+        ...(row.project ? { project: row.project } : {}),
+        ...(invocation ? { invocation } : {}),
+        ...(lineage ? { lineage } : {}),
+        submittedAt: row.submittedAt,
+    };
+}
+function isReviewInvocationKind(invocation) {
+    return (invocation === 'manual' || invocation === 'auto' || invocation === 'gate');
+}
+function ledgerLineage(row) {
+    if (row.invocation === 'manual' || row.invocation === 'auto') {
+        return { kind: 'lifecycle' };
+    }
+    const gateTarget = row.gateTarget?.trim();
+    return row.invocation === 'gate' && gateTarget
+        ? { kind: 'gate', target: gateTarget }
+        : undefined;
+}
+/** Does a prior review match the current rail, project, scope, and lineage? */
 function matchesTuple(review, input) {
+    if (!FULL_COMMIT_SHA.test(review.headSha)) {
+        return false;
+    }
+    const sameLineage = review.lineage !== undefined &&
+        review.lineage.kind === input.lineage.kind &&
+        (input.lineage.kind === 'lifecycle' ||
+            (review.lineage.kind === 'gate' &&
+                review.lineage.target === input.lineage.target));
     if (input.rail === 'ad-hoc') {
         // Ad-hoc: scope sentinel must match AND there must be no project key.
-        return review.scope === 'ad-hoc' && review.project === undefined;
+        return (review.scope === 'ad-hoc' && review.project === undefined && sameLineage);
     }
     // Project rail: same project AND same scope.
     return (review.project !== undefined &&
         review.project === input.project &&
-        review.scope === input.scope);
+        review.scope === input.scope &&
+        sameLineage);
 }
 function mostRecentMatch(input) {
     const matches = input.reviews.filter((r) => matchesTuple(r, input));
@@ -48,16 +117,47 @@ async function guardPasses(priorSha, input) {
     }
     return input.git.isAncestor(priorSha, input.headSha);
 }
+async function classifyRange(priorSha, input) {
+    let files;
+    try {
+        files = await input.git.changedFiles(priorSha, input.headSha);
+    }
+    catch {
+        // Classification is reporting-only, so enumeration failure must not block
+        // dispatch. Treat unknown work as substantive and disclose the fallback.
+        return {
+            classification: 'substantive',
+            classificationReason: 'changed-files-unavailable',
+        };
+    }
+    if (files.length === 0) {
+        return { classification: 'empty' };
+    }
+    const projectPrefix = input.project === null ? null : `${input.project.replace(/\/+$/, '')}/`;
+    // Path-only classification is sufficient while it is reporting-only. If it
+    // ever authorizes skipping work, strengthen it with lifecycle corroboration.
+    return {
+        classification: projectPrefix !== null &&
+            files.every((file) => file.startsWith(projectPrefix))
+            ? 'bookkeeping-only'
+            : 'substantive',
+    };
+}
 /**
  * Pick the narrowing target for the current re-review pass.
  */
 export async function pickNarrowingTarget(input) {
+    if (input.narrowingPreference === false && !input.forceNarrow) {
+        return {
+            kind: 'full-scope-fallback',
+            reason: 'narrowing-disabled',
+        };
+    }
     const prior = mostRecentMatch(input);
     if (!prior) {
         return {
             kind: 'full-scope-fallback',
             reason: 'no-prior-review',
-            prompted: false,
         };
     }
     const passed = await guardPasses(prior.headSha, input);
@@ -66,8 +166,7 @@ export async function pickNarrowingTarget(input) {
             kind: 'narrow-range',
             priorSha: prior.headSha,
             headSha: input.headSha,
-            // Auto-narrow skips the prompt; otherwise the caller still confirms.
-            prompted: !input.autoNarrow,
+            ...(await classifyRange(prior.headSha, input)),
         };
     }
     // Guard failed. The user explicitly asked to narrow → hard error.
@@ -78,13 +177,10 @@ export async function pickNarrowingTarget(input) {
             priorSha: prior.headSha,
         };
     }
-    // Otherwise fall back to full scope. Auto-narrow surfaces this as the
-    // auto-fallback notice (no prompt); manual mode does not prompt here either —
-    // the fallback is informational.
+    // Otherwise fail open to full scope and preserve the guard failure reason.
     return {
         kind: 'full-scope-fallback',
         reason: 'stale-sha',
         priorSha: prior.headSha,
-        prompted: false,
     };
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@open-agent-toolkit/cli",
-  "version": "0.2.24",
+  "version": "0.2.26",
   "private": false,
   "description": "Open Agent Toolkit CLI",
   "homepage": "https://github.com/voxmedia/open-agent-toolkit/tree/main/packages/cli",
@@ -34,7 +34,7 @@
     "ora": "^9.0.0",
     "yaml": "2.8.2",
     "zod": "^3.25.76",
-    "@open-agent-toolkit/control-plane": "0.2.24"
+    "@open-agent-toolkit/control-plane": "0.2.26"
   },
   "devDependencies": {
     "@types/node": "^22.10.0",