mandrel 1.67.0 → 1.68.0

package/.agents/scripts/lib/config-settings-schema-delivery.js

@@ -300,6 +300,22 @@ const ACCEPTANCE_EVAL_SCHEMA = {
   additionalProperties: false,
 };
 
+/**
+ * `delivery.feedbackLoop` — opt-out toggles consumed by the Epic finalize
+ * listener's auto-file graduators (`lib/feedback-loop/*-graduator.js`, read
+ * via `graduator-core.js#makeIsAutoFileEnabled`). Both default to `true`
+ * (auto-file on); set either to `false` to suppress auto-filing the
+ * corresponding non-blocking findings as follow-up issues.
+ */
+const FEEDBACK_LOOP_SCHEMA = {
+  type: 'object',
+  properties: {
+    codeReviewAutoFile: { type: 'boolean' },
+    auditResultsAutoFile: { type: 'boolean' },
+  },
+  additionalProperties: false,
+};
+
 export const DELIVERY_SCHEMA = {
   type: 'object',
   properties: {
@@ -318,6 +334,7 @@ export const DELIVERY_SCHEMA = {
     retro: RETRO_SCHEMA,
     refactorStage: REFACTOR_STAGE_SCHEMA,
     acceptanceEval: ACCEPTANCE_EVAL_SCHEMA,
+    feedbackLoop: FEEDBACK_LOOP_SCHEMA,
     ci: CI_DELIVERY_SCHEMA,
     preflight: PREFLIGHT_SCHEMA,
     // Cross-Story concurrency-hazard gate (Story #2297). When true,

package/.agents/scripts/lib/config-settings-schema.js

@@ -282,6 +282,17 @@ const PLANNING_SCHEMA = {
     // decompose loop's re-prompt gate.
     failOnSharedEditors: { type: 'boolean' },
     requireExplicitCrossStoryDeps: { type: 'boolean' },
+    // Cross-cutting registry conflict knobs consumed by
+    // `ticket-validator-conflicts.js` (wired through
+    // `epic-plan-decompose/phases/planning-artifacts.js`).
+    // `crossCuttingRegistries` names the registry paths whose concurrent
+    // edits are flagged; `failOnRegistryConflicts` upgrades that finding to
+    // `'hard'`. `failOnLargeFanOut` / `largeFanOutThreshold` gate the
+    // single-Story fan-out finding.
+    crossCuttingRegistries: LIST_OR_EXTENDER_OF_STRINGS,
+    failOnRegistryConflicts: { type: 'boolean' },
+    failOnLargeFanOut: { type: 'boolean' },
+    largeFanOutThreshold: { type: 'integer', minimum: 0 },
     // Navigability-reachability config consumed by the epic-plan-healthcheck
     // --paranoid reachability check (Epic #4131, F7). Opt-in: absent or empty
     // routeGlobs degrades to a silent no-op.

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

@@ -7,6 +7,7 @@
  */
 
 import path from 'node:path';
+import { verifyBddRunnerPendingTag } from '../../../bdd-runner-detect.js';
 import { Logger } from '../../../Logger.js';
 import { AGENT_LABELS, TYPE_LABELS } from '../../../label-constants.js';
 import { cleanupPhaseTempFiles } from '../../../plan-phase-cleanup.js';
@@ -45,12 +46,20 @@ function buildRiskVerdictCommentBody({ epicId, riskVerdict, planningRisk }) {
     verdict: riskVerdict,
     planningRisk,
   };
+  // Story #4145 — when the disposition was forced to not-applicable because
+  // no BDD runner exists, make the waiver operator-visible in the rendered
+  // comment (not just the fenced JSON record) so a reviewer sees why an
+  // otherwise-required AC table was waived.
+  const waiverNote = planningRisk.acceptanceWaivedReason
+    ? ['', `> ⚠️ **Acceptance waived** — ${planningRisk.acceptanceWaivedReason}`]
+    : [];
   return [
     `### 🧭 Planning Risk Verdict — ${planningRisk.overallLevel} · ${planningRisk.gateDecision}`,
     '',
     riskVerdict.summary,
     '',
     ...axisTable,
+    ...waiverNote,
     '',
     '```json',
     JSON.stringify(record, null, 2),
@@ -99,7 +108,29 @@ export async function runSpecPhase(
       '[epic-plan-spec] risk verdict is required — author risk-verdict.json via the epic-plan-spec-author Skill and pass it with --risk-verdict.',
     );
   }
-  const planningRisk = deriveRiskEnvelope(riskVerdict);
+
+  // Story #4145 — probe the project's BDD runner. When none is detected
+  // (`fallback === true`, e.g. a node:test repo with no tests/features/**),
+  // the acceptance disposition is forced to not-applicable inside
+  // deriveRiskEnvelope: an authored AC table could never be reconciled by
+  // `@epic-<id>-ac-*` feature tags, so /deliver finalize would otherwise
+  // abort and require a manual `acceptance::n-a`. The probe is static and
+  // best-effort — a detection failure degrades to "runner present" (no
+  // forced waiver), preserving the BDD-repo path, and never blocks Phase 7.
+  let bddRunner = null;
+  try {
+    bddRunner = await verifyBddRunnerPendingTag({ cwd: PROJECT_ROOT });
+  } catch (err) {
+    Logger.warn(
+      `[epic-plan-spec] BDD runner probe skipped (${err.message}); acceptance disposition derived from risk axes only.`,
+    );
+  }
+  const planningRisk = deriveRiskEnvelope(riskVerdict, { bddRunner });
+  if (planningRisk.acceptanceWaivedReason) {
+    Logger.info(
+      `[epic-plan-spec] Acceptance disposition forced to not-applicable for Epic #${epicId}: ${planningRisk.acceptanceWaivedReason}`,
+    );
+  }
 
   const epic = await provider.getEpic(epicId);
   if (!epic) {

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

@@ -358,10 +358,15 @@ export async function pollUntilTerminal({
  * @param {number} opts.maxPolls           Hard cap on total poll iterations.
  * @param {number} opts.maxUpdates         Cap on `gh pr update-branch` recovery calls.
  * @param {number} opts.pollIntervalMs     Delay between poll ticks.
- * @param {Function} opts.ghPrChecksFn
- * @param {Function} opts.ghPrViewFn
- * @param {Function} opts.ghPrUpdateBranchFn
- * @param {Function} opts.sleepFn
+ * @param {Function} [opts.ghPrChecksFn]   `gh pr checks` invoker. Defaults
+ *   to the real `gh pr checks` spawn so the CLI path (which injects no
+ *   port) works; tests override it with a stub. Story #4144.
+ * @param {Function} [opts.ghPrViewFn]     `gh pr view` invoker. Defaults
+ *   to the real spawn; tests override.
+ * @param {Function} [opts.ghPrUpdateBranchFn] `gh pr update-branch`
+ *   invoker. Defaults to the real spawn; tests override.
+ * @param {Function} [opts.sleepFn]        Poll-tick delay. Defaults to a
+ *   real `setTimeout`-backed sleep; tests override with a no-op.
  * @param {{ info?: Function, warn?: Function, debug?: Function }} opts.logger
  * @param {{status:number,stdout:string,stderr:string}} [opts.firstProbe]
  *   Optional already-issued `gh pr checks` result. When the caller (the
@@ -388,10 +393,10 @@ export async function watchPrToTerminal({
   maxPolls,
   maxUpdates,
   pollIntervalMs,
-  ghPrChecksFn,
-  ghPrViewFn,
-  ghPrUpdateBranchFn,
-  sleepFn,
+  ghPrChecksFn = ghPrChecks,
+  ghPrViewFn = ghPrView,
+  ghPrUpdateBranchFn = ghPrUpdateBranch,
+  sleepFn = defaultSleep,
   logger,
   firstProbe,
 }) {

package/.agents/scripts/lib/orchestration/planning-risk.js

@@ -37,6 +37,18 @@
  * @property {boolean} requiresReview
  * @property {AcceptanceDisposition} acceptanceDisposition
  * @property {GateDecision} gateDecision
+ * @property {string} [acceptanceWaivedReason] Present only when the
+ *   acceptance disposition was forced to `not-applicable` by a non-axis
+ *   signal (currently: no BDD runner detected). An operator-visible
+ *   rationale so the override is never silent (Story #4145).
+ */
+
+/**
+ * @typedef {Object} BddRunnerProbe
+ * @property {string|null} runner
+ * @property {boolean} fallback `true` when no supported BDD runner was
+ *   detected in the project (`verifyBddRunnerPendingTag`).
+ * @property {string} [reason]
  */
 
 const LEVEL_RANK = Object.freeze({ low: 0, medium: 1, high: 2 });
@@ -129,27 +141,54 @@ function resolveRequiresReview(overallLevel, axes) {
  * (`epic-plan-spec.js`), never here, so a malformed verdict fails closed
  * before this function runs.
  *
+ * **No-BDD-runner waiver (Story #4145).** The acceptance disposition the risk
+ * axes derive presumes a BDD runner exists to satisfy an authored AC table.
+ * When `opts.bddRunner.fallback === true` (no supported runner detected — e.g.
+ * a `node:test` repo with no `tests/features/**`), an authored AC table can
+ * never be reconciled by `@epic-<id>-ac-*` feature tags, so `/deliver`
+ * finalize would abort. In that case the disposition is **forced** to
+ * `not-applicable` regardless of the risk axes, and `acceptanceWaivedReason`
+ * records the override so it is operator-visible, not silent. The
+ * `requiresReview` / `gateDecision` outputs are unaffected — a high-risk
+ * Epic still routes to review; only the acceptance-spec requirement is
+ * waived. Repos that ship a BDD runner (`fallback !== true`) are unaffected.
+ *
  * @param {RiskVerdict} [verdict]
+ * @param {{ bddRunner?: BddRunnerProbe|null }} [opts]
  * @returns {PlanningRiskEnvelope}
  */
-export function deriveRiskEnvelope(verdict = {}) {
+export function deriveRiskEnvelope(verdict = {}, { bddRunner = null } = {}) {
   const axes = (Array.isArray(verdict.axes) ? verdict.axes : []).map(
     ({ axis, level, rationale }) => ({ axis, level, rationale }),
   );
 
   const overallLevel = resolveOverallLevel(axes);
-  const acceptanceDisposition = resolveAcceptanceDisposition(
-    axes,
-    overallLevel,
-  );
+  const axisDisposition = resolveAcceptanceDisposition(axes, overallLevel);
   const requiresReview = resolveRequiresReview(overallLevel, axes);
   const gateDecision = requiresReview ? 'review-required' : 'auto-proceed';
 
-  return {
+  const noBddRunner = bddRunner?.fallback === true;
+  // Force the waiver only when the axes would otherwise have required (or
+  // recommended) an AC table; if the disposition is already not-applicable
+  // there is nothing to override and no waiver rationale to surface.
+  const forceWaiver = noBddRunner && axisDisposition !== 'not-applicable';
+  const acceptanceDisposition = forceWaiver
+    ? 'not-applicable'
+    : axisDisposition;
+
+  /** @type {PlanningRiskEnvelope} */
+  const envelope = {
     axes,
     overallLevel,
     requiresReview,
     acceptanceDisposition,
     gateDecision,
   };
+  if (forceWaiver) {
+    envelope.acceptanceWaivedReason =
+      `no BDD runner detected (${bddRunner?.reason ?? 'no-bdd-runner-detected'}) — ` +
+      `an authored acceptance-spec AC table cannot be reconciled by feature tags, ` +
+      `so the acceptance disposition is waived to not-applicable (was ${axisDisposition}).`;
+  }
+  return envelope;
 }

package/docs/CHANGELOG.md

@@ -2,6 +2,18 @@
 
 All notable changes to this project will be documented in this file.
 
+## [1.68.0](https://github.com/dsj1984/mandrel/compare/mandrel-v1.67.0...mandrel-v1.68.0) (2026-06-15)
+
+
+### Added
+
+* **plan:** waive acceptance disposition when no BDD runner detected (refs [#4145](https://github.com/dsj1984/mandrel/issues/4145)) ([#4147](https://github.com/dsj1984/mandrel/issues/4147)) ([f119919](https://github.com/dsj1984/mandrel/commit/f119919c4e69b4a583eeec7c8fe860116dcdf87c))
+
+
+### Fixed
+
+* **watcher:** default gh ports in watchPrToTerminal so the CLI path runs (refs [#4144](https://github.com/dsj1984/mandrel/issues/4144)) ([#4148](https://github.com/dsj1984/mandrel/issues/4148)) ([764cc05](https://github.com/dsj1984/mandrel/commit/764cc05b4414f2dff0fb1247b647460df32d037d))
+
 ## [1.67.0](https://github.com/dsj1984/mandrel/compare/mandrel-v1.66.0...mandrel-v1.67.0) (2026-06-15)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mandrel",
-  "version": "1.67.0",
+  "version": "1.68.0",
   "description": "Claude Code-first opinionated workflow framework: instructions, personas, skills, and SDLC workflows that govern AI coding assistants.",
   "files": [
     ".agents/",