@deftai/directive-core 0.73.0 → 0.74.0

package/dist/pr-merge-readiness/mergeability.js

@@ -65,13 +65,15 @@ export function verdictShaIsStale(verdict, headSha) {
  * Classify whether the verdict-based merge block is ONLY "soft" (#2260).
  *
  * A soft block means the review verdict is absent or pinned to a prior head
- * SHA (rebased staleness) -- i.e. the review has not spoken about the CURRENT
- * head. It is safe to reconcile a soft block against GitHub mergeability.
+ * SHA without carrying real blocker-class findings (rebased staleness with no
+ * P0/P1/errored/low-confidence) -- i.e. the review has not spoken about the
+ * CURRENT head in a blocking way. It is safe to reconcile a soft block against
+ * GitHub mergeability.
  *
- * A HARD block -- a genuine P0/P1 finding, an ERRORED review, or a low
- * confidence score on the current head -- is NEVER soft; those must keep
- * blocking regardless of GitHub mergeability (guardrail: do not merge a PR
- * with a real P0/P1 review finding).
+ * A HARD block -- a genuine P0/P1 finding (even on a stale SHA), an ERRORED
+ * review, or a low confidence score -- is NEVER soft; those must keep blocking
+ * regardless of GitHub mergeability (guardrail: do not merge a PR with a real
+ * P0/P1 review finding).
  */
 export function verdictBlockIsSoftOnly(verdict, headSha) {
     // Absent: no Greptile rolling-summary comment at all.
@@ -83,12 +85,11 @@ export function verdictBlockIsSoftOnly(verdict, headSha) {
     if (verdict.informalClean) {
         return false;
     }
-    // Stale: verdict pinned to a prior head SHA -> its findings pertain to old code.
-    if (verdictShaIsStale(verdict, headSha)) {
+    // Excluded-author skip is an intentional N/A reviewer state (#2375).
+    if (verdict.excludedAuthor) {
         return true;
     }
-    // Verdict is current (or its SHA is unparseable). A genuine current-head
-    // finding is a hard block and must not be overridden.
+    // Blocker-class signals apply even when the verdict SHA is stale (#2382).
     if (verdict.errored) {
         return false;
     }
@@ -98,6 +99,10 @@ export function verdictBlockIsSoftOnly(verdict, headSha) {
     if (verdict.p0Count > 0 || verdict.p1Count > 0) {
         return false;
     }
+    // Stale without blocker findings: review has not spoken about the current head.
+    if (verdictShaIsStale(verdict, headSha)) {
+        return true;
+    }
     // No genuine finding: the block is a missing-canonical-field / not-yet-posted
     // wait, which GitHub mergeability may resolve.
     return true;

package/dist/pr-merge-readiness/output.js

@@ -10,6 +10,7 @@ function verdictToDict(verdict) {
         p1_count: verdict.p1Count,
         p2_count: verdict.p2Count,
         informal_clean: verdict.informalClean,
+        excluded_author: verdict.excludedAuthor,
         raw_body_excerpt: verdict.rawBodyExcerpt,
     };
 }
@@ -47,12 +48,17 @@ export function printHuman(result) {
     const lines = [];
     lines.push(`PR #${result.prNumber} merge-readiness check  (via=${result.via})`);
     lines.push(`  HEAD SHA:           ${result.headSha ?? "<unknown>"}`);
-    lines.push(`  Greptile reviewed:  ${result.verdict.lastReviewedSha ?? "<not parsed>"}`);
-    const confidenceStr = result.verdict.confidence !== null ? String(result.verdict.confidence) : "<not parsed>";
-    lines.push(`  Confidence:         ${confidenceStr}/5`);
-    lines.push(`  Findings:           P0=${result.verdict.p0Count}  ` +
-        `P1=${result.verdict.p1Count}  P2=${result.verdict.p2Count}`);
-    lines.push(`  Errored sentinel:   ${result.verdict.errored ? "True" : "False"}`);
+    if (result.verdict.excludedAuthor) {
+        lines.push("  Greptile review:    skipped (author excluded)");
+    }
+    else {
+        lines.push(`  Greptile reviewed:  ${result.verdict.lastReviewedSha ?? "<not parsed>"}`);
+        const confidenceStr = result.verdict.confidence !== null ? String(result.verdict.confidence) : "<not parsed>";
+        lines.push(`  Confidence:         ${confidenceStr}/5`);
+        lines.push(`  Findings:           P0=${result.verdict.p0Count}  ` +
+            `P1=${result.verdict.p1Count}  P2=${result.verdict.p2Count}`);
+        lines.push(`  Errored sentinel:   ${result.verdict.errored ? "True" : "False"}`);
+    }
     const ciBlock = result.partialData.ci;
     if (ciBlock !== null && typeof ciBlock === "object" && !Array.isArray(ciBlock)) {
         const ci = ciBlock;

package/dist/pr-merge-readiness/parse.d.ts

@@ -1,5 +1,6 @@
 import type { GreptileVerdict } from "./types.js";
 export declare function emptyVerdict(): GreptileVerdict;
+export declare function isGreptileExcludedAuthor(body: string): boolean;
 export declare function isInformalCleanMissingCanonicalFields(verdict: GreptileVerdict, body: string): boolean;
 /** Parse a Greptile rolling-summary comment body into a structured verdict. */
 export declare function parseGreptileBody(body: string): GreptileVerdict;

package/dist/pr-merge-readiness/parse.js

@@ -1,5 +1,5 @@
 import { findLastReviewedCommitSha } from "../text/redos-safe.js";
-import { CONFIDENCE_RE, GREPTILE_ERRORED_SENTINEL, INFORMAL_CLEAN_SIGNAL_RE, P0_BADGE, P1_BADGE, SECTION_RE, } from "./constants.js";
+import { CONFIDENCE_RE, GREPTILE_ERRORED_SENTINEL, GREPTILE_EXCLUDED_AUTHOR_PHRASE, GREPTILE_STATUS_MARKER, INFORMAL_CLEAN_SIGNAL_RE, P0_BADGE, P1_BADGE, SECTION_RE, } from "./constants.js";
 export function emptyVerdict() {
     return {
         found: false,
@@ -10,9 +10,15 @@ export function emptyVerdict() {
         p1Count: 0,
         p2Count: 0,
         informalClean: false,
+        excludedAuthor: false,
         rawBodyExcerpt: "",
     };
 }
+export function isGreptileExcludedAuthor(body) {
+    const lower = body.toLowerCase();
+    return (lower.includes(GREPTILE_EXCLUDED_AUTHOR_PHRASE) &&
+        (lower.includes("greptile-status") || body.includes(GREPTILE_STATUS_MARKER)));
+}
 export function isInformalCleanMissingCanonicalFields(verdict, body) {
     if (!verdict.found || verdict.errored) {
         return false;
@@ -42,6 +48,7 @@ export function parseGreptileBody(body) {
         return emptyVerdict();
     }
     const errored = body.trim().startsWith(GREPTILE_ERRORED_SENTINEL);
+    const excludedAuthor = isGreptileExcludedAuthor(body);
     const lastReviewedSha = findLastReviewedCommitSha(body);
     const confMatch = CONFIDENCE_RE.exec(body);
     const confidence = confMatch?.groups?.score !== undefined ? Number(confMatch.groups.score) : null;
@@ -73,6 +80,7 @@ export function parseGreptileBody(body) {
         p1Count,
         p2Count,
         informalClean: false,
+        excludedAuthor,
         rawBodyExcerpt: body.slice(0, 200),
     };
     if (isInformalCleanMissingCanonicalFields(verdict, body)) {

package/dist/pr-merge-readiness/types.d.ts

@@ -7,6 +7,8 @@ export interface GreptileVerdict {
     readonly p1Count: number;
     readonly p2Count: number;
     readonly informalClean: boolean;
+    /** Greptile deliberately skipped review for an excluded PR author (#2375). */
+    readonly excludedAuthor: boolean;
     readonly rawBodyExcerpt: string;
 }
 export interface GateResult {

package/dist/preflight-cache/evaluate.d.ts

@@ -17,6 +17,8 @@ export declare const CANDIDATES_RELPATH: string;
 export declare const DEFAULT_MAX_AGE_HOURS = 24;
 export declare const ENV_MAX_AGE_HOURS = "DEFT_CACHE_MAX_AGE_HOURS";
 export declare const ENV_TRIAGE_REPO = "DEFT_TRIAGE_REPO";
+/** Set by releaseSubprocessEnv() during task release Step 5 (#2386). */
+export declare const ENV_RELEASE_PREFLIGHT = "DEFT_RELEASE_PREFLIGHT";
 export declare const REQUIRED_DECISION = "accept";
 export interface GateResult {
     readonly code: number;

package/dist/preflight-cache/evaluate.js

@@ -27,7 +27,14 @@ export const CANDIDATES_RELPATH = join("xbrief", ".triage-cache", "candidates.js
 export const DEFAULT_MAX_AGE_HOURS = 24;
 export const ENV_MAX_AGE_HOURS = "DEFT_CACHE_MAX_AGE_HOURS";
 export const ENV_TRIAGE_REPO = "DEFT_TRIAGE_REPO";
+/** Set by releaseSubprocessEnv() during task release Step 5 (#2386). */
+export const ENV_RELEASE_PREFLIGHT = "DEFT_RELEASE_PREFLIGHT";
 export const REQUIRED_DECISION = "accept";
+const TRUTHY = new Set(["1", "true", "yes", "on"]);
+function releasePreflightBypassActive() {
+    const raw = process.env[ENV_RELEASE_PREFLIGHT] ?? "";
+    return TRUTHY.has(raw.trim().toLowerCase());
+}
 // ---------------------------------------------------------------------------
 // Repo discovery helpers
 // ---------------------------------------------------------------------------
@@ -327,6 +334,7 @@ const REMEDIATION_NO_CANDIDATES = [
 export function evaluate(projectRoot, options = {}) {
     const source = options.source ?? DEFAULT_SOURCE;
     const allowStale = options.allowStale ?? false;
+    const releasePreflightStaleBypass = releasePreflightBypassActive();
     const allowMissingBootstrap = options.allowMissingBootstrap ?? false;
     const nowFn = options.nowFn ?? (() => new Date());
     const envMaxAge = process.env[ENV_MAX_AGE_HOURS];
@@ -423,11 +431,14 @@ export function evaluate(projectRoot, options = {}) {
     // empty-cache case (no entries at all) preserves its prior staleness result.
     const allCandidatesClosed = minFetchedAt === null && rawCandidatePaths.length > 0;
     const stale = allCandidatesClosed ? false : ageH > maxAgeHours;
-    // Step 5b: --allow-stale bypass
-    if (stale && allowStale) {
+    // Step 5b: --allow-stale bypass (or release Step-5 age bypass only, #2386)
+    if (stale && (allowStale || releasePreflightStaleBypass)) {
+        const bypassReason = options.allowStale
+            ? "--allow-stale is set"
+            : `${ENV_RELEASE_PREFLIGHT} is set (release pre-flight)`;
         const warning = [
             `⚠ deft cache-fresh: cache is ${ageH.toFixed(1)}h old (max-age=${maxAgeHours}h) but`,
-            `  --allow-stale is set -- proceeding with audit-trail warning.`,
+            `  ${bypassReason} -- proceeding with audit-trail warning.`,
         ].join("\n");
         if (options.forIssue !== undefined && options.forIssue !== null) {
             const forResult = evaluateForIssue(resolvedRepo, options.forIssue, candidatesPath, cacheRoot, source, scopeRules, projectRoot);

package/dist/preflight-cache/index.d.ts

@@ -1,3 +1,3 @@
 export type { EvaluateOptions, GateResult } from "./evaluate.js";
-export { CACHE_DIR_NAME, CANDIDATES_RELPATH, DEFAULT_MAX_AGE_HOURS, DEFAULT_SOURCE, ENV_MAX_AGE_HOURS, ENV_TRIAGE_REPO, evaluate, REQUIRED_DECISION, } from "./evaluate.js";
+export { CACHE_DIR_NAME, CANDIDATES_RELPATH, DEFAULT_MAX_AGE_HOURS, DEFAULT_SOURCE, ENV_MAX_AGE_HOURS, ENV_RELEASE_PREFLIGHT, ENV_TRIAGE_REPO, evaluate, REQUIRED_DECISION, } from "./evaluate.js";
 //# sourceMappingURL=index.d.ts.map

package/dist/preflight-cache/index.js

@@ -1,2 +1,2 @@
-export { CACHE_DIR_NAME, CANDIDATES_RELPATH, DEFAULT_MAX_AGE_HOURS, DEFAULT_SOURCE, ENV_MAX_AGE_HOURS, ENV_TRIAGE_REPO, evaluate, REQUIRED_DECISION, } from "./evaluate.js";
+export { CACHE_DIR_NAME, CANDIDATES_RELPATH, DEFAULT_MAX_AGE_HOURS, DEFAULT_SOURCE, ENV_MAX_AGE_HOURS, ENV_RELEASE_PREFLIGHT, ENV_TRIAGE_REPO, evaluate, REQUIRED_DECISION, } from "./evaluate.js";
 //# sourceMappingURL=index.js.map

package/dist/release/constants.d.ts

@@ -16,6 +16,8 @@ export declare const VERIFY_DRAFT_INTERVAL_SECONDS = 1;
 export declare const RELEASE_ARTIFACTS: readonly ["CHANGELOG.md", "ROADMAP.md"];
 export declare const BRANCH_GATE_BYPASS_ENV = "DEFT_ALLOW_DEFAULT_BRANCH_COMMIT";
 export declare const DESTRUCTIVE_GH_GATE_BYPASS_ENV = "DEFT_ALLOW_DESTRUCTIVE_GH_VERBS";
+/** Set by releaseSubprocessEnv() so Step-5 pre-flight skips triage-cache staleness (#2386). */
+export declare const RELEASE_PREFLIGHT_ENV = "DEFT_RELEASE_PREFLIGHT";
 export declare const PYPROJECT_VERSION_LINE_RE: RegExp;
 /** Byte-identical argparse --help from scripts/release.py (Python 3.12). */
 export declare const RELEASE_HELP: string;

package/dist/release/constants.js

@@ -24,6 +24,8 @@ export const VERIFY_DRAFT_INTERVAL_SECONDS = 1.0;
 export const RELEASE_ARTIFACTS = ["CHANGELOG.md", "ROADMAP.md"];
 export const BRANCH_GATE_BYPASS_ENV = "DEFT_ALLOW_DEFAULT_BRANCH_COMMIT";
 export const DESTRUCTIVE_GH_GATE_BYPASS_ENV = "DEFT_ALLOW_DESTRUCTIVE_GH_VERBS";
+/** Set by releaseSubprocessEnv() so Step-5 pre-flight skips triage-cache staleness (#2386). */
+export const RELEASE_PREFLIGHT_ENV = "DEFT_RELEASE_PREFLIGHT";
 export const PYPROJECT_VERSION_LINE_RE = /version\s*=\s*"[^"]*"/;
 /** Byte-identical argparse --help from scripts/release.py (Python 3.12). */
 export const RELEASE_HELP = "usage: release [-h] [--dry-run] [--skip-tag] [--skip-release] [--allow-dirty]\n" +

package/dist/release/preflight.d.ts

@@ -8,6 +8,8 @@
  * dispatches `check:framework-source`.
  */
 import { type CheckOrchestratorSeams } from "../check/orchestrator.js";
+/** Step-5-only env: branch bypass plus release pre-flight cache staleness tolerance (#2386). */
+export declare function releaseCheckEnv(base?: NodeJS.ProcessEnv): NodeJS.ProcessEnv;
 /** Seams for test isolation of the native release pre-flight. */
 export interface ReleasePreflightSeams {
     /** Override the check dispatcher (default: dispatchTaskCheck from check/orchestrator). */

package/dist/release/preflight.js

@@ -8,6 +8,15 @@
  * dispatches `check:framework-source`.
  */
 import { dispatchTaskCheck } from "../check/orchestrator.js";
+import { RELEASE_PREFLIGHT_ENV } from "./constants.js";
+import { releaseSubprocessEnv } from "./git.js";
+/** Step-5-only env: branch bypass plus release pre-flight cache staleness tolerance (#2386). */
+export function releaseCheckEnv(base = process.env) {
+    return {
+        ...releaseSubprocessEnv(base),
+        [RELEASE_PREFLIGHT_ENV]: "1",
+    };
+}
 /**
  * Run the native TypeScript `task check` as the release pre-flight.
  *
@@ -18,7 +27,11 @@ import { dispatchTaskCheck } from "../check/orchestrator.js";
  */
 export function runReleaseCheck(projectRoot, seams = {}) {
     const dispatch = seams.dispatchCheck ?? dispatchTaskCheck;
-    const code = dispatch(projectRoot, projectRoot, seams.checkSeams);
+    const checkSeams = {
+        ...seams.checkSeams,
+        env: releaseCheckEnv(seams.checkSeams?.env ?? process.env),
+    };
+    const code = dispatch(projectRoot, projectRoot, checkSeams);
     if (code === 0) {
         return [true, "ran native TypeScript task check"];
     }

package/dist/scope/index.d.ts

@@ -3,6 +3,7 @@ export * from "./constants.js";
 export * from "./decomposed-refs.js";
 export * from "./demote.js";
 export * from "./main.js";
+export * from "./open-umbrella-warning.js";
 export * from "./project-context.js";
 export * from "./transition.js";
 export * from "./undo.js";

package/dist/scope/index.js

@@ -3,6 +3,7 @@ export * from "./constants.js";
 export * from "./decomposed-refs.js";
 export * from "./demote.js";
 export * from "./main.js";
+export * from "./open-umbrella-warning.js";
 export * from "./project-context.js";
 export * from "./transition.js";
 export * from "./undo.js";

package/dist/scope/main.js

@@ -1,8 +1,10 @@
 import { existsSync } from "node:fs";
-import { join } from "node:path";
+import { dirname, join } from "node:path";
+import { reconcileUmbrellas, renderUmbrellasReport } from "../vbrief-reconcile/umbrellas.js";
 import { canonicalLogPath, readAll } from "./audit-log.js";
 import { TRANSITIONS } from "./constants.js";
 import { batchDemote, DEFAULT_OLDER_THAN_DAYS, demoteOne, resolveDemoteFilePath, resolveFilePath, resolveProjectRootStrict, } from "./demote.js";
+import { completedPathForScopeMove, findOpenUmbrellaReferences, renderOpenUmbrellaWarning, } from "./open-umbrella-warning.js";
 import { resolveProjectRoot } from "./project-context.js";
 import { recordWipCapOverride, runTransition } from "./transition.js";
 import { findByDecisionId, isAlreadyUndone, REVERSIBLE_ACTIONS, undoBatch, undoOne, } from "./undo.js";
@@ -88,6 +90,27 @@ export function lifecycleMain(argv) {
                 "audit: vbrief/.eval/scope-lifecycle.jsonl entry tagged wip_cap_override (#1124).\n");
         }
         process.stdout.write(`${result.message}\n`);
+        if (action === "complete") {
+            try {
+                const completedPath = completedPathForScopeMove(filePath);
+                const rootForWarning = resolveProjectRoot(projectRoot) ?? dirname(dirname(dirname(completedPath)));
+                const refs = findOpenUmbrellaReferences(rootForWarning, completedPath);
+                const warning = renderOpenUmbrellaWarning(refs);
+                if (warning.length > 0) {
+                    process.stderr.write(`${warning}\n`);
+                    const [, outcome] = reconcileUmbrellas(rootForWarning);
+                    // Only surface a report when reconcile actually changed something or
+                    // recorded per-epic errors — skip vacuous setup failures (exit 2 with
+                    // empty buckets) so a missing lifecycle root does not spam stderr.
+                    if (outcome.changed.length > 0 || outcome.errors.length > 0) {
+                        process.stderr.write(`${renderUmbrellasReport(outcome)}\n`);
+                    }
+                }
+            }
+            catch {
+                /* best-effort drift warning + reconcile; lifecycle success remains authoritative */
+            }
+        }
         return 0;
     }
     process.stderr.write(`Error: ${result.message}\n`);

package/dist/scope/open-umbrella-warning.d.ts

@@ -0,0 +1,12 @@
+export interface OpenUmbrellaReference {
+    readonly repo: string | null;
+    readonly issueNumber: number | null;
+    readonly title: string;
+    readonly path: string | null;
+    readonly sources: readonly string[];
+}
+/** Find open umbrella/tracker surfaces that still mention a just-completed scope. */
+export declare function findOpenUmbrellaReferences(projectRoot: string, completedScopePath: string): OpenUmbrellaReference[];
+export declare function renderOpenUmbrellaWarning(refs: readonly OpenUmbrellaReference[]): string;
+export declare function completedPathForScopeMove(filePath: string): string;
+//# sourceMappingURL=open-umbrella-warning.d.ts.map