mandrel 1.81.0 → 1.83.0

package/.agents/scripts/lib/orchestration/epic-plan-spec/phases/run-spec-phase.js

@@ -82,15 +82,15 @@ async function setEpicLabel(provider, epicId, targetLabel) {
  *
  * @param {number} epicId
  * @param {import('../../../ITicketingProvider.js').ITicketingProvider} provider
- * @param {{ prdContent: string, techSpecContent: string, acceptanceSpecContent?: string|null }} artifacts
+ * @param {{ techSpecContent: string, acceptanceSpecContent?: string|null }} artifacts
  * @param {object} settings
  * @param {{ force?: boolean, forceReview?: boolean, steal?: boolean, config?: object, riskVerdict?: import('../../planning-risk.js').RiskVerdict }} [opts]
- * @returns {Promise<{ epicId: number, prdId: number|null, techSpecId: number|null, acceptanceSpecId: number|null, checkpoint: object, planningRisk: import('../../planning-risk.js').PlanningRiskEnvelope, reviewRouting: import('../../plan-review-routing.js').ReviewRoutingEnvelope }>}
+ * @returns {Promise<{ epicId: number, techSpecPersisted: boolean, acceptanceTable: 'persisted'|'waived'|'none', checkpoint: object, planningRisk: import('../../planning-risk.js').PlanningRiskEnvelope, reviewRouting: import('../../plan-review-routing.js').ReviewRoutingEnvelope }>}
  */
 export async function runSpecPhase(
   epicId,
   provider,
-  { prdContent, techSpecContent, acceptanceSpecContent = null },
+  { techSpecContent, acceptanceSpecContent = null },
   settings = {},
   {
     force = false,
@@ -153,7 +153,7 @@ export async function runSpecPhase(
   const planResult = await planEpic(
     epicId,
     provider,
-    { prdContent, techSpecContent, acceptanceSpecContent },
+    { techSpecContent, acceptanceSpecContent },
     settings,
     {
       force,
@@ -163,15 +163,13 @@ export async function runSpecPhase(
   const specChanged = planResult?.persisted !== false;
 
   const afterPlan = await provider.getEpic(epicId);
-  const prdId = afterPlan.linkedIssues?.prd ?? null;
-  const techSpecId = afterPlan.linkedIssues?.techSpec ?? null;
-  const acceptanceSpecId = afterPlan.linkedIssues?.acceptanceSpec ?? null;
 
   // Story #2635 — cross-validate the authored Tech Spec body against the
-  // base branch and surface any stale path-shaped references on the Tech
-  // Spec issue. Non-blocking: a missing base ref, an unreadable temp
-  // directory, or a provider failure downgrades to a warning so Phase 7
-  // never fails on the advisory check.
+  // base branch and surface any stale path-shaped references. Story #4324:
+  // with the Tech Spec folded into the Epic body, the advisory comment
+  // lands on the Epic itself. Non-blocking: a missing base ref, an
+  // unreadable temp directory, or a provider failure downgrades to a
+  // warning so Phase 7 never fails on the advisory check.
   const baseBranchRef = settings?.baseBranch ?? 'main';
   const tempRoot = path.resolve(
     PROJECT_ROOT,
@@ -179,7 +177,6 @@ export async function runSpecPhase(
   );
   const freshness = await runSpecFreshnessCheck({
     epicId,
-    techSpecId,
     techSpecContent,
     baseBranchRef,
     tempRoot,
@@ -220,9 +217,8 @@ export async function runSpecPhase(
       },
       spec: {
         ...currentState.spec,
-        prdId,
-        techSpecId,
-        acceptanceSpecId,
+        techSpecPersisted: planResult?.techSpecPersisted === true,
+        acceptanceTable: planResult?.acceptanceTable ?? 'none',
         completedAt: new Date().toISOString(),
       },
     },
@@ -256,13 +252,17 @@ export async function runSpecPhase(
   const cleanup = await cleanupPhaseTempFiles({ phase: 'spec', epicId });
 
   const acceptanceSummary =
-    acceptanceSpecId !== null ? `, Acceptance Spec #${acceptanceSpecId}` : '';
+    planResult?.acceptanceTable === 'persisted'
+      ? ' + ## Acceptance Table section'
+      : planResult?.acceptanceTable === 'waived'
+        ? ' (acceptance waived)'
+        : '';
   const freshnessSummary =
     freshness.stale > 0 || freshness.ambiguous > 0
       ? ` ⚠️ Spec freshness: ${freshness.stale} stale / ${freshness.ambiguous} ambiguous reference(s) — see ${freshness.reportPath ?? 'report'}.`
       : '';
   Logger.info(
-    `[epic-plan-spec] ✅ Spec phase complete for Epic #${epicId}. PRD #${prdId}, Tech Spec #${techSpecId}${acceptanceSummary}.${freshnessSummary}`,
+    `[epic-plan-spec] ✅ Spec phase complete for Epic #${epicId}. Tech Spec sections folded into the Epic body${acceptanceSummary}.${freshnessSummary}`,
   );
   if (cleanup.deleted.length > 0) {
     Logger.info(
@@ -272,9 +272,8 @@ export async function runSpecPhase(
 
   return {
     epicId,
-    prdId,
-    techSpecId,
-    acceptanceSpecId,
+    techSpecPersisted: planResult?.techSpecPersisted === true,
+    acceptanceTable: planResult?.acceptanceTable ?? 'none',
     checkpoint,
     cleanup,
     freshness,

package/.agents/scripts/lib/orchestration/epic-plan-spec/phases/spec-authoring-grounding.js

@@ -1,7 +1,7 @@
 /**
  * phases/spec-authoring-grounding.js — F10 spec-authoring code-grounding.
  *
- * Story #4139 (Epic #4131). `/plan` Phase 7 authors the PRD + Tech Spec
+ * Story #4139 (Epic #4131). `/plan` Phase 7 authors the Tech Spec
  * from the Epic body, the scraped project docs, and the
  * `codebaseSnapshot` structural view of the consumer repo. Two failure
  * modes made that grounding silently partial:

package/.agents/scripts/lib/orchestration/epic-plan-spec/phases/spec-freshness.js

@@ -2,15 +2,16 @@
  * phases/spec-freshness.js — Tech Spec freshness check phase.
  *
  * Cross-validates the authored Tech Spec body against the base branch and
- * surfaces any stale path-shaped references on the Tech Spec issue. Two
- * side effects, both best-effort and non-blocking:
+ * surfaces any stale path-shaped references on the Epic (Story #4324: the
+ * Tech Spec is folded into the Epic body, so the Epic is the comment
+ * target). Two side effects, both best-effort and non-blocking:
  *
  *   1. Write the full `{ stale, fresh, ambiguous }` report to
  *      `<tempRoot>/epic-<id>-spec-freshness.json` so downstream tooling
  *      (the `--code-freshness` health check on the roadmap, or an
  *      operator inspecting the run) can read it without re-probing git.
  *   2. When `stale.length > 0`, upsert a `spec-freshness` structured
- *      comment on the Tech Spec issue listing each suspect citation.
+ *      comment on the Epic listing each suspect citation.
  *      The comment is upserted, so re-running spec re-renders the same
  *      comment in place rather than spamming the issue.
  *
@@ -32,7 +33,6 @@ import { upsertStructuredComment } from '../../ticketing.js';
 /**
  * @param {object} opts
  * @param {number} opts.epicId
- * @param {number|null} opts.techSpecId
  * @param {string} opts.techSpecContent
  * @param {string} opts.baseBranchRef
  * @param {string} opts.tempRoot
@@ -44,7 +44,6 @@ import { upsertStructuredComment } from '../../ticketing.js';
  */
 export async function runSpecFreshnessCheck({
   epicId,
-  techSpecId,
   techSpecContent,
   baseBranchRef,
   tempRoot,
@@ -64,7 +63,6 @@ export async function runSpecFreshnessCheck({
     );
     const payload = {
       epicId,
-      techSpecId,
       baseBranchRef,
       generatedAt: new Date().toISOString(),
       summary: {
@@ -77,13 +75,12 @@ export async function runSpecFreshnessCheck({
     await fileWriter(reportPath, `${JSON.stringify(payload, null, 2)}\n`);
 
     let commentPosted = false;
-    if (report.stale.length > 0 && Number.isFinite(techSpecId)) {
+    if (report.stale.length > 0 && Number.isFinite(epicId)) {
       const body = renderSpecFreshnessComment(report, {
         baseBranchRef,
-        techSpecId,
         epicId,
       });
-      await commentUpserter(provider, techSpecId, 'spec-freshness', body);
+      await commentUpserter(provider, epicId, 'spec-freshness', body);
       commentPosted = true;
     }
 

package/.agents/scripts/lib/orchestration/epic-plan-state-store.js

@@ -18,7 +18,7 @@
  *   "epicId": 349,
  *   "startedAt": "...",
  *   "lastUpdatedAt": "...",
- *   "spec": { "prdId": null, "techSpecId": null, "acceptanceSpecId": null, "completedAt": null },
+ *   "spec": { "techSpecPersisted": false, "acceptanceTable": "none", "completedAt": null },
  *   "decompose": { "ticketCount": null, "completedAt": null },
  *   "planningRisk": { "overallLevel": "...", "requiresReview": true, "gateDecision": "..." },
  *   "reviewRouting": { "decision": "review-required|auto-proceed|operator-override-review", "requiresStop": true, "forceReviewApplied": false },
@@ -106,9 +106,8 @@ export async function initialize({ provider, epicId, seed = {} } = {}) {
     epicId,
     startedAt: now,
     spec: {
-      prdId: null,
-      techSpecId: null,
-      acceptanceSpecId: null,
+      techSpecPersisted: false,
+      acceptanceTable: 'none',
       completedAt: null,
     },
     decompose: { ticketCount: null, completedAt: null },

package/.agents/scripts/lib/orchestration/epic-runner/phases/build-wave-dag.js

@@ -35,7 +35,7 @@ export async function discoverOpenStories({ epicId, provider }) {
   for (const t of descendants) {
     const labels = t.labels ?? [];
     if (!labels.includes(TYPE_LABELS.STORY)) continue;
-    // Defense-in-depth: never enumerate a context spec ticket (PRD / Tech
+    // Defense-in-depth: never enumerate a context spec ticket (Tech
     // Spec / Acceptance Spec) as a deliverable Story even if it carries
     // `type::story`. The `createTicket` factory no longer stamps context
     // tickets that way, but a context ticket mislabelled out-of-band must

package/.agents/scripts/lib/orchestration/epic-runner/phases/snapshot.js

@@ -5,20 +5,19 @@
  * Auto-close is now the default for `/deliver` (the human PR-merge is
  * the gate); no per-Epic label snapshot is required.
  *
- * Acceptance-spec start gate (relaxed): an Epic may be delivered when
- * *either* the operator has explicitly waived the acceptance-spec
- * requirement via the `acceptance::n-a` label, *or* a
- * `context::acceptance-spec` ticket is linked to the Epic. The ticket's
- * GitHub state (open / closed) is **not** checked — presence is
- * sufficient, matching the PRD and Tech Spec contract. The reviewer's
- * OK during /plan Phase 7 is the approval signal, not a manual
- * ticket-close action. This still refuses to launch Epics that skipped
- * the /plan Phase 7 acceptance-spec authoring step (or didn't
- * waive), surfacing the gap at delivery time rather than letting Story
- * dispatch race ahead without a spec at all.
+ * Acceptance-spec start gate (Story #4324): an Epic may be delivered when
+ * *either* the operator has explicitly waived the acceptance requirement
+ * via the `acceptance::n-a` label, *or* the Epic body carries the
+ * `## Acceptance Table` managed section (the folded home of the retired
+ * `context::acceptance-spec` ticket's AC-ID table). Presence is
+ * sufficient — the reviewer's OK during /plan Phase 7 is the approval
+ * signal. This still refuses to launch Epics that skipped the /plan
+ * Phase 7 acceptance authoring step (or didn't waive), surfacing the gap
+ * at delivery time rather than letting Story dispatch race ahead without
+ * a spec at all.
  */
 
-import { parseLinkedIssues } from '../../../issue-link-parser.js';
+import { hasEpicSection } from '../../../epic-body-sections.js';
 import { ACCEPTANCE_NA } from '../../../label-constants.js';
 import { discoverOpenStories } from './build-wave-dag.js';
 
@@ -82,29 +81,23 @@ async function discoverStoryIds({ epicId, provider }) {
 }
 
 /**
- * Refuse to launch /deliver when the acceptance-spec precondition has
- * not been satisfied. Throws a clear `Error` (per
- * orchestration-error-handling rule) so the `runAsCli` boundary maps it to
- * `process.exit(1)` with the operator-visible message intact.
+ * Refuse to launch /deliver when the acceptance precondition has not been
+ * satisfied. Throws a clear `Error` (per orchestration-error-handling
+ * rule) so the `runAsCli` boundary maps it to `process.exit(1)` with the
+ * operator-visible message intact.
  *
- * The gate now only checks presence (or waiver). Ticket state is
- * deliberately ignored — closure is no longer required as the approval
- * signal.
+ * The gate checks section presence (or the waiver label) only.
  *
- * @param {{ epic: { labels?: string[], linkedIssues?: { acceptanceSpec?: number|null }|null, body?: string }, epicId: number }} args
+ * @param {{ epic: { labels?: string[], body?: string }, epicId: number }} args
  */
 function assertAcceptanceSpecGate({ epic, epicId }) {
   const labels = epic?.labels ?? [];
   if (labels.includes(ACCEPTANCE_NA)) return;
 
-  const linkedIssues =
-    epic?.linkedIssues ?? parseLinkedIssues(epic?.body ?? '');
-  const acceptanceSpecId = linkedIssues?.acceptanceSpec ?? null;
-
-  if (!acceptanceSpecId) {
+  if (!hasEpicSection(epic?.body ?? '', 'acceptanceTable')) {
     throw new Error(
-      `[epic-deliver] Epic #${epicId} cannot launch: no context::acceptance-spec is linked and the acceptance::n-a waiver label is absent. ` +
-        'Run /plan Phase 7 to author an acceptance-spec, or apply the acceptance::n-a label to the Epic to opt out.',
+      `[epic-deliver] Epic #${epicId} cannot launch: the Epic body has no ## Acceptance Table section and the acceptance::n-a waiver label is absent. ` +
+        'Run /plan Phase 7 to author the acceptance table, or apply the acceptance::n-a label to the Epic to opt out.',
     );
   }
 }

package/.agents/scripts/lib/orchestration/lifecycle/listeners/acceptance-reconciler.js

@@ -23,8 +23,10 @@
  *   - Per-instance `Set<string>` of `${event}:${seqId}` keys. A repeat
  *     `(event, seqId)` short-circuits without re-running reconciliation
  *     and emits no events.
- *   - The reconciler helper itself is read-only with respect to runtime
- *     state, so the seqId guard is the only defence the listener needs.
+ *   - The reconciler helper's only write is the section-scoped
+ *     disposition record inside the Epic body's ## Acceptance Table
+ *     region (Story #4324) — itself idempotent — so the seqId guard is
+ *     the only defence the listener needs.
  *
  * Side-effect firewall: the listener owns the reconciliation call and
  * the four outcome emits. It does NOT mutate labels, post comments, or
@@ -233,10 +235,14 @@ export class AcceptanceReconciler {
         loggerImpl: this.logger,
         // Honour `acceptance::n-a` — the helper already returns
         // `status: 'waived'` for that label, but `skipWhenWaived: true`
-        // additionally tolerates a missing spec link under the waiver
-        // (we want the same forgiving stance the inline finalize call
-        // had).
+        // additionally tolerates a missing acceptance-table section under
+        // the waiver (we want the same forgiving stance the inline
+        // finalize call had).
         skipWhenWaived: true,
+        // Close time is the one place the verification outcome is
+        // recorded into the Epic body's ## Acceptance Table section
+        // (section-scoped write, Story #4324).
+        writeDispositions: true,
       });
     } catch (err) {
       thrown = err;

package/.agents/scripts/lib/orchestration/lifecycle/listeners/finalizer.js

@@ -24,9 +24,10 @@
  *   4. Otherwise, invoke `runFinalizeFn`. The production default
  *      (`composeBusOwnedFinalize`) chains
  *        a. `openOrLocatePr({ epicId, headBranch, baseBranch })`
- *        b. `closePlanningTickets({ epicId, provider })`
- *        c. `postHandoffComment({ epicId, prNumber, prUrl, provider })`
- *      and returns `{ prNumber, prUrl, planningClose, handoff }`.
+ *        b. `postHandoffComment({ epicId, prNumber, prUrl, provider })`
+ *      and returns `{ prNumber, prUrl, handoff }`. (Story #4324 retired
+ *      the `closePlanningTickets` sweep with the context-ticket classes —
+ *      there are no planning tickets to close.)
  *   5. Emit `pr.created` then `epic.finalize.end`.
  *
  * Why the Finalizer does NOT emit `epic.merge.ready` (Story #3367):
@@ -65,31 +66,27 @@ import {
 } from '../../../feedback-loop/audit-results-graduator.js';
 import { graduateFindings as defaultGraduateFindings } from '../../../feedback-loop/code-review-graduator.js';
 import { parsePrNumberFromUrl } from '../../../github-url.js';
-import { closePlanningTickets as defaultClosePlanningTickets } from '../../finalize/close-planning-tickets.js';
 import { openOrLocatePr as defaultOpenOrLocatePr } from '../../finalize/open-or-locate-pr.js';
 import { postHandoffComment as defaultPostHandoffComment } from '../../finalize/post-handoff-comment.js';
 
 /**
- * Build the production default `runFinalizeFn` that composes the three
+ * Build the production default `runFinalizeFn` that composes the
  * bus-owned finalize helpers. Exported as a factory so the listener can
  * inject the provider (which the default needs but tests inject by
  * supplying their own `runFinalizeFn` outright).
  *
- * Returns `{ prNumber, prUrl, planningClose, handoff }` on success, or
+ * Returns `{ prNumber, prUrl, handoff }` on success, or
  * `{ blocker: { reason, detail } }` when a step fails with an
  * unrecoverable error that should keep the Epic at `agent::blocked`.
  *
  * @param {{
  *   provider?: object|null,
  *   openOrLocatePrFn?: typeof defaultOpenOrLocatePr,
- *   closePlanningTicketsFn?: typeof defaultClosePlanningTickets,
  *   postHandoffCommentFn?: typeof defaultPostHandoffComment,
  * }} deps
  */
 export function composeBusOwnedFinalize(deps = {}) {
   const openOrLocatePrFn = deps.openOrLocatePrFn ?? defaultOpenOrLocatePr;
-  const closePlanningTicketsFn =
-    deps.closePlanningTicketsFn ?? defaultClosePlanningTickets;
   const postHandoffCommentFn =
     deps.postHandoffCommentFn ?? defaultPostHandoffComment;
   const provider = deps.provider ?? null;
@@ -132,28 +129,10 @@ export function composeBusOwnedFinalize(deps = {}) {
       };
     }
 
-    // Planning-ticket close + handoff comment require a provider.
-    // The lifecycle-emit CLI may construct the Finalizer without one;
-    // in that case both steps short-circuit with a 'skipped' marker
-    // (the run itself still succeeds — the PR is open and the bus can
-    // arm auto-merge).
-    let planningClose = null;
-    if (provider) {
-      try {
-        planningClose = await closePlanningTicketsFn({
-          epicId,
-          provider,
-        });
-      } catch (err) {
-        return {
-          blocker: {
-            reason: 'close-planning-tickets-failed',
-            detail: err?.message ?? String(err),
-          },
-        };
-      }
-    }
-
+    // The handoff comment requires a provider. The lifecycle-emit CLI
+    // may construct the Finalizer without one; in that case the step
+    // short-circuits (the run itself still succeeds — the PR is open and
+    // the bus can arm auto-merge).
     let handoff = null;
     if (provider) {
       try {
@@ -180,7 +159,6 @@ export function composeBusOwnedFinalize(deps = {}) {
       prNumber: openResult.prNumber,
       prUrl: openResult.url,
       created: openResult.created,
-      planningClose,
       handoff,
     };
   };
@@ -252,8 +230,8 @@ export class Finalizer {
    * @param {Function} [opts.ghPrListHeadFn] override of the
    *   idempotency probe.
    * @param {object} [opts.provider] Ticketing provider forwarded to
-   *   the default `runFinalizeFn` (for `closePlanningTickets` /
-   *   `postHandoffComment`) and to the graduators.
+   *   the default `runFinalizeFn` (for `postHandoffComment`) and to the
+   *   graduators.
    * @param {object} [opts.config] Resolved agent config; forwarded to
    *   the graduators.
    * @param {{owner:string,repo:string}} [opts.currentRepo]
@@ -283,11 +261,9 @@ export class Finalizer {
       composeBusOwnedFinalize({ provider: this.provider });
     this.ghPrListHeadFn = opts.ghPrListHeadFn ?? ghPrListHead;
     // ultrareview bug_007: the existing-PR short-circuit must run the
-    // planning-ticket close and the handoff-comment upsert (both
-    // idempotent) so crash-recovery replays don't skip them. Stored as
-    // instance overrides so tests can swap them with no-op stubs.
-    this.closePlanningTicketsFn =
-      opts.closePlanningTicketsFn ?? defaultClosePlanningTickets;
+    // handoff-comment upsert (idempotent) so crash-recovery replays
+    // don't skip it. Stored as an instance override so tests can swap
+    // it with a no-op stub.
     this.postHandoffCommentFn =
       opts.postHandoffCommentFn ?? defaultPostHandoffComment;
     this.config = opts.config ?? null;
@@ -375,24 +351,11 @@ export class Finalizer {
         );
         const prNumber = parsePrNumberFromUrl(existingUrl);
         // ultrareview bug_007: even on the short-circuit path we MUST
-        // run the planning-ticket close and the handoff-comment upsert.
-        // Both helpers are idempotent — closePlanningTickets counts
-        // already-closed tickets under `alreadyClosed`, postHandoffComment
-        // edits an existing marker comment in place. Without these calls,
-        // a crash-recovery replay against an already-open PR leaves the
-        // three planning context tickets open and never posts the
-        // handoff comment, even though epic.merge.ready is still emitted.
-        try {
-          await this.closePlanningTicketsFn({
-            epicId: this.epicId,
-            provider: this.provider,
-            cwd: this.cwd,
-          });
-        } catch (err) {
-          this.logger.warn?.(
-            `[Finalizer] closePlanningTickets on short-circuit failed (swallowed; replay will retry): ${err?.message ?? err}`,
-          );
-        }
+        // run the handoff-comment upsert. The helper is idempotent —
+        // postHandoffComment edits an existing marker comment in place.
+        // Without this call, a crash-recovery replay against an
+        // already-open PR never posts the handoff comment, even though
+        // epic.merge.ready is still emitted.
         try {
           await this.postHandoffCommentFn({
             epicId: this.epicId,
@@ -425,8 +388,8 @@ export class Finalizer {
 
     // 3. Run the composed bus-owned finalize (or the test-injected
     //    `runFinalizeFn`). The default chains openOrLocatePr →
-    //    closePlanningTickets → postHandoffComment in order and
-    //    returns `{ prNumber, prUrl, planningClose, handoff }`.
+    //    postHandoffComment in order and returns
+    //    `{ prNumber, prUrl, handoff }`.
     let finalize;
     try {
       finalize = await this.runFinalizeFn({

package/.agents/scripts/lib/orchestration/lifecycle/listeners/index.js

@@ -183,7 +183,7 @@ export async function buildDefaultListenerChain(opts = {}) {
   // 3. Finalizer — opens the PR on acceptance.reconcile.ok or .waived
   //    (Story #2893 split waiver out of .skipped) via the bus-owned
   //    default (`composeBusOwnedFinalize`, Story #2894), which chains
-  //    openOrLocatePr → closePlanningTickets → postHandoffComment and
+  //    openOrLocatePr → postHandoffComment and
   //    emits `epic.merge.ready` on success. The legacy
   //    `d1-default-no-op` blocker is gone; the listener constructed
   //    with no `runFinalizeFn` override always runs the real flow.

package/.agents/scripts/lib/orchestration/planning-context-budget.js

@@ -3,7 +3,7 @@
  *
  * Bounded planning-context budget for the `--emit-context` planning scripts
  * (Epic #817 Story 9). Provides a summary representation of large markdown
- * payloads (project docs, PRD body, Tech Spec body) so the host LLM is not
+ * payloads (project docs, Epic body, Tech Spec body) so the host LLM is not
  * handed multi-100KB envelopes when the underlying material exceeds the
  * configured `maxBytes` budget.
  *

package/.agents/scripts/lib/orchestration/preflight-cache.js

@@ -22,7 +22,7 @@
  *     call (`getTicket(epicId)`) used to walk the hierarchy. The hash
  *     covers the Epic's id, body, labels (sorted), and (when present)
  *     `updatedAt` — the same fields that drive Story discovery and the
- *     acceptance-spec gate.
+ *     acceptance start gate (the Epic body's ## Acceptance Table).
  *   - Cache miss is signalled by `readCache` returning `null`. Callers
  *     MUST fall back to a fresh snapshot/DAG pass.
  *   - `writeCache` creates the parent directory recursively and writes

package/.agents/scripts/lib/orchestration/spec-freshness.js

@@ -2,7 +2,7 @@
  * spec-freshness.js — Tech Spec post-author cross-validation against the
  * current codebase.
  *
- * `/plan` Phase 7 authors PRD + Tech Spec from documentation alone.
+ * `/plan` Phase 7 authors the Tech Spec from documentation alone.
  * When `project.docsContextFiles` (architecture.md, etc.) drift from
  * the real source tree, the Architect persona happily cites modules and
  * paths that no longer exist. The mismatch only surfaces at delivery time,
@@ -12,7 +12,7 @@
  * probes each against `baseBranchRef` (via `git cat-file -e`), and returns a
  * `{ stale, fresh, ambiguous }` envelope. The caller (epic-plan-spec.js)
  * uses the result to write a JSON report and post an advisory structured
- * comment on the Tech Spec issue. The check is intentionally non-blocking —
+ * comment on the Epic. The check is intentionally non-blocking —
  * planning continues even when stale references are present, because the
  * operator may legitimately be planning to create the path the Tech Spec
  * cites.
@@ -274,7 +274,7 @@ export function validateSpecFreshness(specBody, opts = {}) {
  * exact rendering contract.
  *
  * @param {{ stale: Array, ambiguous: Array, fresh: Array }} report
- * @param {{ baseBranchRef: string, techSpecId?: number, epicId?: number }} ctx
+ * @param {{ baseBranchRef: string, epicId?: number }} ctx
  * @returns {string}
  */
 export function renderSpecFreshnessComment(report, ctx) {

package/.agents/scripts/lib/orchestration/spec-section-validator.js

@@ -1,7 +1,7 @@
 /**
  * spec-section-validator.js — Phase 7.5 Tech Spec post-authoring section gate.
  *
- * `/plan` Phase 7 authors a Tech Spec from documentation and the PRD.
+ * `/plan` Phase 7 authors a Tech Spec from documentation and the Epic body.
  * Phase 8.3 (Holistic Consolidation) then reconciles the draft ticket array
  * against the Tech Spec's `## Delivery Slicing` section, which the
  * decompose-author skill uses as the capability-boundary anchor. When the

package/.agents/scripts/lib/orchestration/story-close/baseline-attribution/phases/refresh-commit.js

@@ -32,6 +32,7 @@ import { Logger as DefaultLogger } from '../../../../Logger.js';
 import {
   calculateAll as defaultCalculateAll,
   scanDirectory as defaultScanDirectory,
+  isIgnoredByGlobs as isIgnoredByGlobsMi,
 } from '../../../../maintainability-utils.js';
 
 /**
@@ -101,7 +102,20 @@ export function buildKindScorer({
           const underTarget = targetAbsDirs.some(
             (root) => abs === root || abs.startsWith(`${root}${path.sep}`),
           );
-          if (underTarget) sourceList.push(abs);
+          // Apply `ignoreGlobs` here too — the full-scope walk drops
+          // ignore-matched files via `scanDirectory`, so the diff-scope path
+          // must do the same or an ignored-but-changed file (e.g. one matched
+          // by `config-settings-schema*.js`) enters `rows` and drags the
+          // `rollup["*"].min` below the maintainability floor. This mirrors the
+          // canonical `buildDefaultMaintainabilityScorer` (refresh-service.js,
+          // Story #4293); the story-close auto-refresh routes through this
+          // scorer, not that one, so the fix has to live here too.
+          if (
+            underTarget &&
+            !isIgnoredByGlobsMi(abs, miIgnoreGlobs, effectiveCwd)
+          ) {
+            sourceList.push(abs);
+          }
         }
       }
       const scores = await calculateAll(sourceList);