amsd-pipeline-workflow 0.1.24 → 0.1.26

package/generator/plan-no-change-structured.js

@@ -0,0 +1,25 @@
+#!/usr/bin/env node
+'use strict';
+// DOES THE DETECTIVE'S STRUCTURED ANALYSIS GIVE THIS CODELINE NO CHANGE? (run 11, 2026-10-21.) The pipeline's plan-gives-no-change.js reads prose (the consumer's `change` line must be exactly "none"); the same story carries the
+// detective's structured findings — every fix site has `changeRequired` — and a codeline whose sites ALL say false needs no edit: its files are "NO EDIT REQUIRED", verified, not changed. Exit 0 = no change (and the reason on
+// stdout as JSON); exit 1 = otherwise: no analysis for the lane, no sites, or ANY site that is true or does not say (no evidence is not "no change").
+//   plan-no-change-structured.js <prd> <story-id> <lane codeline>
+const fs = require('fs');
+
+try {
+  const [prdFile, storyId, lane] = process.argv.slice(2);
+  const prd = JSON.parse(fs.readFileSync(prdFile, 'utf8'));
+  const s = (prd.stories || []).find((x) => x && x.id === storyId);
+  if (!s) process.exit(1);
+  // ONLY A LANE THE PLAN SAYS MERELY RECEIVES THE CHANGE: a consumer in the story's overall plan and the provider of none of its packages. The provider is where the change is made — and run 9's detective marked the two
+  // sites cxshared edited changeRequired:false — so for it the field is not evidence of anything.
+  const pkgs = (s.overallPlan && Array.isArray(s.overallPlan.packages)) ? s.overallPlan.packages : [];
+  const isConsumer = pkgs.some((p) => p && (p.consumers || []).some((c) => c && c.codeline === lane));
+  const isProvider = pkgs.some((p) => p && p.providedBy === lane);
+  if (!isConsumer || isProvider) process.exit(1);
+  const per = s.fixSiteAnalysisPerCodeline && typeof s.fixSiteAnalysisPerCodeline === 'object' ? s.fixSiteAnalysisPerCodeline : {};
+  const sites = Array.isArray(per[lane]) ? per[lane] : (s.codeline === lane && Array.isArray(s.fixSiteAnalysis) ? s.fixSiteAnalysis : null);
+  if (!sites || !sites.length || !sites.every((x) => x && x.changeRequired === false)) process.exit(1);
+  process.stdout.write(`${JSON.stringify({ sites: sites.length, reason: `none of its ${sites.length} fix sites has changeRequired (the detective's structured analysis)` })}\n`);
+  process.exit(0);
+} catch { process.exit(1); }

package/generator/plan-no-change.sh

@@ -14,7 +14,13 @@ LANE_LOG="${LOG_DIR:-}"
 set +e
 for _np in "${PRD_FILE:-}" "${MAIN_PRD_FILE:-}"; do
     [ -n "$_np" ] && [ -f "$_np" ] || continue
-    "$NODE_BIN" "$SCRIPT_DIR/lib/handlers/plan-gives-no-change.js" "$_np" "$STORY" "$LANE" 2>/dev/null || continue
+    # THE PIPELINE'S ROUTE (the consumer's change line is exactly "none"), else THE STRUCTURED ONE (generator/plan-no-change-structured.js: every fix site of this lane's analysis says changeRequired false) — run 11:
+    # the plan's prose was "Verify … no manifest change." and a Writer was sent into 8 failing attempts over files the detective had already marked NO EDIT REQUIRED
+    _why=""
+    if ! "$NODE_BIN" "$SCRIPT_DIR/lib/handlers/plan-gives-no-change.js" "$_np" "$STORY" "$LANE" 2>/dev/null; then
+        _sj=$("$NODE_BIN" "$(dirname "${BASH_SOURCE[0]}")/plan-no-change-structured.js" "$_np" "$STORY" "$LANE" 2>/dev/null) || continue
+        _why=" ($(printf '%s' "$_sj" | jq -r '.reason // empty' 2>/dev/null))"
+    fi
     _ref=""
     if [ -e "$PROJECT_ROOT/.git" ]; then
         [ -n "$LANE_LOG" ] && [ -f "$LANE_LOG/phase-baseline-sha.txt" ] && _ref=$(tr -d '[:space:]' < "$LANE_LOG/phase-baseline-sha.txt" 2>/dev/null)
@@ -26,7 +32,7 @@ for _np in "${PRD_FILE:-}" "${MAIN_PRD_FILE:-}"; do
             '{noChange:false, message:$m}'
         exit 0
     fi
-    jq -cn --arg m "Story $STORY: the story's plan gives $LANE no change — no writer is run; the lane's gates prove it against the release it receives" \
+    jq -cn --arg m "Story $STORY: the story's plan gives $LANE no change${_why} — no writer is run; the lane's gates prove it against the release it receives" \
         '{noChange:true, message:$m}'
     exit 0
 done

package/hooks/attempt-verify.sh

@@ -43,12 +43,27 @@ ruled=false
 # the rung attribution.
 retry_count="${ATTEMPT_RETRY:-}"; STORY_MODEL="${ATTEMPT_MODEL:-}"
 _err="$TMPDIR/attempt-verify.log"; _out="$TMPDIR/attempt-output.log"; : > "$_out"
-stage=""
+stage=""; plan_note=""
+# THE STRUCTURED NO-CHANGE ROUTE (run 11, 2026-10-21): verify_story_deliverables excuses "all declared files unchanged" only through the
+# pipeline's prose handler (the consumer's change line is exactly "none"). A consumer whose fix sites ALL say changeRequired:false
+# (generator/plan-no-change-structured.js) has had its plan carried out by a Writer that edited nothing. Only that one failure class
+# (STORY_REJECTION_KEY unchanged-all:) is excused — a missing file or any other finding is not.
+planned_no_change() {
+    case "${STORY_REJECTION_KEY:-}" in unchanged-all:*) ;; *) return 1 ;; esac
+    local _p _sj
+    for _p in "${PRD_FILE:-}" "${MAIN_PRD_FILE:-}"; do
+        [ -n "$_p" ] && [ -f "$_p" ] || continue
+        _sj=$("$NODE_BIN" "$(dirname "${BASH_SOURCE[0]}")/../generator/plan-no-change-structured.js" "$_p" "$STORY" "$EPAM_CODELINE" 2>/dev/null) || continue
+        plan_note="the story's plan gives ${EPAM_CODELINE} no change ($(printf '%s' "$_sj" | jq -r '.reason // empty')) — its declared files are unchanged, as planned"
+        STORY_REJECTION_KEY=""; return 0
+    done
+    return 1
+}
 complete_codeline_manifests "$PROJECT_ROOT" >/dev/null 2>&1
 for s in deliverables typecheck lint tests; do
     : > "$_err"
     case "$s" in
-        deliverables) verify_story_deliverables "$STORY" 2>"$_err" ;;
+        deliverables) verify_story_deliverables "$STORY" 2>"$_err" || planned_no_change ;;
         typecheck)    run_tsc_verification "$STORY" "$_out" 2>"$_err" ;;
         lint)         run_repo_lint_verification "$STORY" "$_out" 2>"$_err" ;;
         tests)        run_external_verification "$STORY" "$_out" 2>"$_err" \
@@ -83,6 +98,6 @@ fi
 # a passing attempt ends this writer process; a failing one leaves its globals to the failure handling and the next attempt
 if [ "$passed" = true ]; then wp_clear "$STATE"; else wp_save "$STATE/writer-process.json"; fi
 jq -cn --argjson p "$passed" --argjson ruled "$ruled" --arg st "$stage" --argjson d "$det" --arg vf "$VERIFICATION_FAILURE" --rawfile m "$_err" \
-    --argjson s "$same" --arg k "$([ "$passed" = false ] && printf '%s' "${STORY_REJECTION_KEY:-}")" --arg t "${LAST_TEST_CMD:-}" \
+    --arg pn "$plan_note" --argjson s "$same" --arg k "$([ "$passed" = false ] && printf '%s' "${STORY_REJECTION_KEY:-}")" --arg t "${LAST_TEST_CMD:-}" \
     '{passed:$p, stage:(if $st == "" then null else $st end), deterministic:$d, verificationFailure:$vf, message:$m,
-      sameViolation:$s, rejectionKey:$k, testCommand:$t, ruledEnvironmental:$ruled}'
+      planNote:$pn, sameViolation:$s, rejectionKey:$k, testCommand:$t, ruledEnvironmental:$ruled}'

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "amsd-pipeline-workflow",
-  "version": "0.1.24",
+  "version": "0.1.26",
   "description": "The AMSD pipeline as Claude workflows. Installed by the amsd-pipeline installer: install.sh --workflows",
   "license": "UNLICENSED",
   "bin": {