From 010cee2a27b16891630d51cc00c0183cb97a19cd Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Tue, 20 Oct 2026 05:03:07 +0000
Subject: [PATCH 1/3] fix(progress-report): show the build plan once, in the
 phase cards; open the now phase (#474)

Phase cards now carry the full row detail (Produces/Proves, Depends on, Scripts,
Claims) from build-plan-status.sh via --phase-detail; the duplicate 'Full plan
tables' block is removed and views A and B stay as their own section. The now
phase renders open + selected; phase cards behave as an accordion; #phase-N
links and saved reload state still work.

Field proof: no real project available in this session; ran against a fixture
architecture/build-plan.md (2 phases, 4 rows, claims). grep -c '<td>12</td>' = 1,
'<td>3</td>' = 1. Chromium at 375px: now phase open on load, click opens/closes
previous, #phase-2 hash opens it, no horizontal scroll. Fixtures run:
test-progress-report{,-polish,-pages,-field-fixes,-gitignore}, test-build-plan-json,
test-build-plan-status-args, test-windows-msys-paths (all pass).

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01XruEA24qrAkK7Zqxc5Z59E
---
 CHANGELOG.md                               |  2 ++
 project-bin/build-plan-status.sh           | 42 ++++++++++++----------
 project-bin/progress-report.py             | 35 +++++++++---------
 tests/wave2/test-progress-report-polish.sh | 19 ++++++----
 4 files changed, 56 insertions(+), 42 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index edafc87e..64add8d5 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -19,6 +19,8 @@ Sections dated before 2026-09-19 predate the cycle and stay as they are. Named m
 
 ## Unreleased
 
+- fix(progress-report): **the build plan is shown once: the phase cards are the plan.** Expanding a card now shows its full rows (#, Kind, Step, Produces/Proves, Depends on, State, Scripts) and Claims, which only the duplicate "Full plan tables" block carried; that block is gone and views A (script folders) and B (module proof) are their own section. The `now` phase renders open with a selected style, opening another phase closes the previous one (accordion; `#phase-N` links still open their card), and a viewer's saved open/closed state survives the 30 s reload and wins over the default (#474) — field feedback from a requirements-driven project at Stage 5
+
 ## v2026.10.19
 
 - new(progress-report): **the progress report gains a Project pages section: a compact folder tree of the HTML pages found in the project's output folders (gate dashboard, analysis reports, blueprint, design system, process map, wireframes, UI reviews, test reports), each a relative link from `architecture/` with its modified date.** Pages are found by scanning the project's files on every render, never from a stored list; only the top-level folders in `PAGE_DIRS` are scanned (each named with its producer; a symlinked one is skipped), `node_modules`, `.mpr-snapshots`, `deployment`, `mprcontents`, `theme`, `.git`, vendor and dot folders are never entered, a folder with more than 8 pages folds behind a count, every folder keeps its row and lists at most 100 links (`+K more`), and the report never lists itself. Dates are shown as days, so the section is unchanged on the same day unless a listed page changed and a `--refresh` leaves the page alone. Test: `tests/wave2/test-progress-report-pages.sh` (single and two-tree layouts, Windows hrefs) — MendixMau (request 2026-10-19)
diff --git a/project-bin/build-plan-status.sh b/project-bin/build-plan-status.sh
index b9530361..fd115d4f 100755
--- a/project-bin/build-plan-status.sh
+++ b/project-bin/build-plan-status.sh
@@ -532,10 +532,10 @@ render_plan_section() {  # stdout: the plan section's HTML
     echo "<h2>The plan</h2><p class=empty>Nothing to show yet: ${PLAN_MSG:-build-plan.md was not read}.</p>"
     return 0
   fi
-  "$PY" - "$PLAN_JSON" "$MDLSOURCE" <<'PYEOF' || echo "<h2>The plan</h2><p class=empty>Could not render the plan section from build-plan.md.</p>"
+  "$PY" - "$PLAN_JSON" "$MDLSOURCE" "${PHASE_DETAIL:-/dev/null}" <<'PYEOF' || echo "<h2>The plan</h2><p class=empty>Could not render the plan section from build-plan.md.</p>"
 import fnmatch, html, json, os, re, sys
 
-src, mdl = sys.argv[1], sys.argv[2]
+src, mdl, detail_out = sys.argv[1], sys.argv[2], sys.argv[3]
 d = json.load(open(src, encoding="utf-8"))
 e = lambda s: html.escape(s or "", quote=True)
 # `code` spans in plan cells render as <code>, not literal backticks
@@ -631,29 +631,32 @@ if named_total:
     print("<p class=note>Scripts named in the plan: %d, of which %d are renamed <code>done-</code> under <code>mdlsource/</code>.%s</p>" % (
         named_total, named_done,
         (" <span class=drift>&#9888; %s where the plan and the disk disagree.</span>" % plural(drift_rows, "row")) if drift_rows else ""))
+# One view of the plan: each phase's rows and claims go to the detail file, which progress-report.py
+# splices into that phase's card. They are not printed here, or the rows would appear twice.
+det = []
 for p, out in zip(phases, rows_html):
-    print('<section class=phase><h3>Phase %s &mdash; %s <span class="badge %s">%s</span></h3>' % (
-        e(p["id"]), e(p["name"]), CLS.get(p["state"], "st-unknown"), e(p["state"])))
+    h = ['<!--phase:%s-->' % e(p["id"]),
+         '<p class=note>Phase state: <span class="badge %s">%s</span></p>' % (CLS.get(p["state"], "st-unknown"), e(p["state"]))]
     if p.get("note"):
-        print("<p class=note>%s</p>" % e(p["note"]))
+        h.append("<p class=note>%s</p>" % e(p["note"]))
     if p["steps"]:
-        b = sum(1 for s in p["steps"] if classify(s["state"]) == "built")
-        print("<p class=note>%d of %s built</p>" % (b, plural(len(p["steps"]), "row")))
-        print("<div class=scroll><table><tr><th>#</th><th>Kind</th><th>Step</th><th>Produces / Proves</th>"
-              "<th>Depends on</th><th>State</th><th>Scripts</th></tr>")
-        print("\n".join(out))
-        print("</table></div>")
+        h.append("<div class=tw><table class=steps><tr><th>#</th><th>Kind</th><th>Step</th><th>Produces / Proves</th>"
+                 "<th>Depends on</th><th>State</th><th>Scripts</th></tr>")
+        h.append("\n".join(out))
+        h.append("</table></div>")
     else:
-        print("<p class=empty>No row table under this phase.</p>")
+        h.append("<p class=empty>No row table under this phase.</p>")
     if p.get("claims"):
-        print("<details><summary>Claims (%d)</summary><ul class=claims>" % len(p["claims"]))
+        h.append("<details><summary>Claims (%d)</summary><ul class=claims>" % len(p["claims"]))
         for c in p["claims"]:
             extra = (" (%d)" % c["count"] if c.get("count") is not None else "") + (" [%s]" % c["brd"] if c.get("brd") else "")
-            print("<li><code>%s</code>%s</li>" % (e(c["pointer"]), e(extra)))
-        print("</ul></details>")
+            h.append("<li><code>%s</code>%s</li>" % (e(c["pointer"]), e(extra)))
+        h.append("</ul></details>")
     elif p.get("claimsNote"):
-        print("<p class=note>Claims: %s</p>" % e(p["claimsNote"]))
-    print("</section>")
+        h.append("<p class=note>Claims: %s</p>" % e(p["claimsNote"]))
+    h.append('<!--/phase-->')
+    det.append("\n".join(h))
+open(detail_out, "w", encoding="utf-8").write("\n".join(det))
 w = d.get("warnings") or []
 if w:
     print("<details class=warn><summary>Parser notes (%d): lines in build-plan.md this page could not read as plan data</summary><ul>" % len(w))
@@ -694,6 +697,7 @@ if [ "$WRITE_HTML" -eq 1 ]; then
     mkdir -p "$ROOT/architecture"
     OUT_TMP="$OUT.tmp.$$"
     FRAG_TMP="$OUT.frag.$$"
+    PHASE_DETAIL="$OUT.phases.$$"
     {
       render_plan_section
 
@@ -725,7 +729,7 @@ if [ "$WRITE_HTML" -eq 1 ]; then
       fi
       echo "<p class=note>The plan: what build-plan.md says. A: built, from mdlsource/ done- prefixes. B: proven, from verify-module.sh + docs/improvement-register.md. A phase at 100% with no reviewed row in B is built, not proven.</p>"
     } > "$FRAG_TMP"
-    GEN_ARGS=(--out "$OUT_TMP" --fragment "$FRAG_TMP" --plan-msg "${PLAN_MSG:-}")
+    GEN_ARGS=(--out "$OUT_TMP" --fragment "$FRAG_TMP" --phase-detail "$PHASE_DETAIL" --plan-msg "${PLAN_MSG:-}")
     [ -n "$PLAN_JSON" ] && GEN_ARGS+=(--plan-json "$PLAN_JSON")
     for _sl in ${SESSION_LOGS[@]+"${SESSION_LOGS[@]}"}; do GEN_ARGS+=(--session-log "$_sl"); done
     if "$PY" "$GEN" "$ROOT" "${GEN_ARGS[@]}" >/dev/null 2>"$OUT.err.$$"; then
@@ -744,7 +748,7 @@ if [ "$WRITE_HTML" -eq 1 ]; then
       if [ "$QUIET" -eq 0 ]; then echo "" && echo "  $REPORT_FILE not written: the generator failed: $(head -c 300 "$OUT.err.$$" 2>/dev/null)"
       else echo "WARN: $REPORT_FILE not updated, progress-report.py failed: ${_why:-no message}" >&2; fi   # unattended callers stay quiet on success, never on a crash
     fi
-    rm -f "${FRAG_TMP:?}" "${OUT}.err.$$"
+    rm -f "${FRAG_TMP:?}" "${PHASE_DETAIL:?}" "${OUT}.err.$$"
   fi
 fi
 
diff --git a/project-bin/progress-report.py b/project-bin/progress-report.py
index 88b51bda..9e424353 100755
--- a/project-bin/progress-report.py
+++ b/project-bin/progress-report.py
@@ -1392,7 +1392,7 @@ def rule(ratio, kind, title, seen, matters, fix, why, lost_s=0.0, lost_tok=0.0,
     health = run_health(cs)
     return fired[:ADV["max_items"]], quiet + fired[ADV["max_items"]:], health, cs, hidden[0]
 
-JS = "<script>\n" + 'function openTarget(){var h=location.hash.slice(1);if(!h)return;var e=document.getElementById(h);if(!e)return;\n if(e.tagName===\'DETAILS\')e.open=true;var d=e.closest&&e.closest(\'details\');if(d)d.open=true;}\nwindow.addEventListener(\'hashchange\',openTarget);openTarget();\n(function(){var links=[].slice.call(document.querySelectorAll(\'.snav a[href^="#stage-"]\'));\n if(!(\'IntersectionObserver\' in window))return;var io=new IntersectionObserver(function(es){es.forEach(function(en){if(en.isIntersecting){\n  links.forEach(function(l){l.style.outline=l.getAttribute(\'href\')===\'#\'+en.target.id?\'2px solid var(--accent)\':\'none\'});}})},{rootMargin:\'-60px 0px -70% 0px\'});\n links.forEach(function(l){var t=document.querySelector(l.getAttribute(\'href\'));if(t)io.observe(t)});})();\n' + "</script>"
+JS = "<script>\n" + 'function openTarget(){var h=location.hash.slice(1);if(!h)return;var e=document.getElementById(h);if(!e)return;\n if(e.tagName===\'DETAILS\')e.open=true;var d=e.closest&&e.closest(\'details\');if(d)d.open=true;}\nwindow.addEventListener(\'hashchange\',openTarget);openTarget();\ndocument.addEventListener(\'toggle\',function(ev){var t=ev.target;if(!t.open||!t.classList||!t.classList.contains(\'ph\'))return;[].forEach.call(document.querySelectorAll(\'details.ph[open]\'),function(o){if(o!==t)o.open=false})},true);\ndocument.addEventListener(\'toggle\',function(ev){var t=ev.target;if(!t.open||!t.classList||!t.classList.contains(\'ph\'))return;[].forEach.call(document.querySelectorAll(\'details.ph[open]\'),function(o){if(o!==t)o.open=false})},true);\n(function(){var links=[].slice.call(document.querySelectorAll(\'.snav a[href^="#stage-"]\'));\n if(!(\'IntersectionObserver\' in window))return;var io=new IntersectionObserver(function(es){es.forEach(function(en){if(en.isIntersecting){\n  links.forEach(function(l){l.style.outline=l.getAttribute(\'href\')===\'#\'+en.target.id?\'2px solid var(--accent)\':\'none\'});}})},{rootMargin:\'-60px 0px -70% 0px\'});\n links.forEach(function(l){var t=document.querySelector(l.getAttribute(\'href\'));if(t)io.observe(t)});})();\n' + "</script>"
 # ----------------------------------------------------------------- render
 CSS = """
 :root{--bg:#f6f7f9;--card:#fff;--ink:#1c2430;--mut:#667085;--line:#e3e7ee;--accent:#2f6fed;--ok:#1f9d63;--warn:#d98a0b;--bad:#c93c3c;--idle:#d9dde5;--main:#2f6fed;--sub:#8b5cf6;--soft:#eef2fb;--shadow:0 1px 2px rgba(20,30,50,.06)}
@@ -1441,14 +1441,14 @@ def rule(ratio, kind, title, seen, matters, fix, why, lost_s=0.0, lost_tok=0.0,
 .layout{display:grid;grid-template-columns:minmax(0,1fr) 340px;gap:16px;align-items:start}.layout.solo{display:block}.content{min-width:0}.side{position:sticky;top:52px;max-height:calc(100vh - 64px);overflow:auto;padding-right:2px}.side h2{margin-top:0}.health{margin-top:8px}.health .hi{font-size:13px;margin-top:3px}.grp{margin-top:8px;border:1px solid var(--line);border-radius:10px;background:var(--card)}.grp>summary{cursor:pointer;padding:9px 12px;font-size:14px}.grp .speed,.grp>.split,.grp>div,.grp .tip{padding:0 12px 10px}.tip{border-top:1px solid var(--line);padding-top:10px}.tip h3{font-size:13px;margin:0 0 4px}.tip p{margin:0 0 4px;font-size:12.5px}.p-info{background:#8793a6;color:#fff}@media(max-width:1000px){.layout{grid-template-columns:minmax(0,1fr)}.side{position:static;max-height:none}}.dot{display:inline-block;width:9px;height:9px;border-radius:50%;margin-right:4px}.p-green{background:#2e9e5b;color:#fff}.p-amber{background:#e0a100;color:#fff}.p-red{background:#d1383d;color:#fff}.pill{padding:1px 9px;margin-left:6px;border-radius:99px;font-size:12px}.speed{margin-top:8px;font-size:13px}.split{display:flex;gap:2px;margin:6px 0}.seg{background:var(--accent);color:#fff;font-size:11px;padding:3px 6px;border-radius:4px;white-space:nowrap;overflow:hidden}.seg:nth-child(2){opacity:.75}.seg:nth-child(3){background:var(--idle);color:var(--ink)}.seg:nth-child(4){opacity:.55}.muted{color:var(--mut);font-size:12px}\n.advl{display:grid;grid-template-columns:repeat(auto-fit,minmax(340px,1fr));gap:10px;margin-top:8px}
 .advi{border-left:4px solid var(--warn)}.advi.hi{border-left-color:var(--bad)}.advi .rk{font-size:12px;color:var(--mut)}.advi h3{margin:2px 0 6px}.advi p{margin:4px 0;font-size:13px}.advi .why{color:var(--mut);font-size:12px}
 details.ph>summary{list-style:none;cursor:pointer;color:var(--ink)}details.ph>summary::-webkit-details-marker{display:none}
-details.ph[open]{grid-column:1/-1}.steps td{font-size:12px;padding:3px 6px}
+details.ph[open]{grid-column:1/-1;border-color:var(--accent);box-shadow:0 0 0 1px var(--accent);background:var(--soft)}.steps td{font-size:12px;padding:3px 6px}
 .nolog{margin:10px 0;font-size:13px}.note-card{margin-top:14px;border-left:4px solid var(--warn)}.live{font-size:12px;color:var(--muted)}.live a{color:inherit}details.how{margin:10px 0;font-size:13px;color:var(--muted)}details.how summary{cursor:pointer}
 .frag section.phase{margin-top:14px}.frag h2{font-size:16px;margin:24px 0 4px}.frag h3{margin:14px 0 2px}.frag .scroll{overflow-x:auto}
 .frag .lede{font-size:14px}.frag .empty{color:var(--mut)}.frag code{font-size:12px}
-.frag .st-built,.frag .done,.frag .CLEAN{color:var(--ok)}.frag .st-progress,.frag .in-progress,.frag .INCOMPLETE{color:var(--warn)}
-.frag .st-todo,.frag .pending{color:var(--mut)}.frag .st-person{color:#8b5cf6}.frag .st-unknown{color:var(--mut);font-style:italic}
-.frag .FINDINGS,.frag .drift{color:var(--bad)}.frag .drift{font-size:12px}.frag .claims{font-size:12px}.frag .warn summary{color:var(--warn)}
-.frag .badge{margin-left:6px;border:1px solid currentColor;background:transparent}
+.frag .st-built,.ph .st-built,.frag .done,.frag .CLEAN{color:var(--ok)}.frag .st-progress,.ph .st-progress,.frag .in-progress,.frag .INCOMPLETE{color:var(--warn)}
+.frag .st-todo,.ph .st-todo,.frag .pending{color:var(--mut)}.frag .st-person,.ph .st-person{color:#8b5cf6}.frag .st-unknown,.ph .st-unknown{color:var(--mut);font-style:italic}
+.frag .FINDINGS,.frag .drift,.ph .drift{color:var(--bad)}.frag .drift,.ph .drift{font-size:12px}.ph .scroll{overflow-x:auto}.ph .claims{font-size:12px}.frag .claims{font-size:12px}.frag .warn summary{color:var(--warn)}
+.frag .badge,.ph .badge{margin-left:6px;border:1px solid currentColor;background:transparent}
 .pages .tree{margin-top:6px;font-size:13px}.pg{display:grid;grid-template-columns:minmax(120px,220px) minmax(0,1fr);gap:2px 12px;padding:5px 0;border-bottom:1px solid var(--line)}.pg:last-child{border-bottom:0}
 .pg .dir{font-weight:600;overflow-wrap:anywhere}.pg ul{list-style:none;margin:0;padding:0;display:flex;flex-wrap:wrap;gap:3px 14px}.pg li{overflow-wrap:anywhere}.pg details{margin:0}.pg details ul{margin-top:4px}
 footer{margin-top:36px;font-size:13px;color:var(--mut)}footer li{margin:3px 0}
@@ -1739,8 +1739,8 @@ def chip(f, cls=""):
     if bp:
         kinds = bp["kinds"]; tr = ""
         runs_by = {str(k_[0]): d["runs"] for k_, d in (facts["phases"] if facts else []) if k_[0] != ""}
-        cur = now_phase(bp, proj)
-        w("<div class=card><div class=hd><b>%d of %d rows built%s</b><a href='#plan-tables'>Jump to the plan tables</a></div><div class='stack3' style='margin:8px 0' title='built, partial, not built'><i class=b style='width:%.1f%%'></i><i class=p style='width:%.1f%%'></i></div>" %
+        cur = now_phase(bp, proj); phase_detail = args.phase_detail
+        w("<div class=card><div class=hd><b>%d of %d rows built%s</b><a href='#plan-tables'>Jump to script folders and module proof</a></div><div class='stack3' style='margin:8px 0' title='built, partial, not built'><i class=b style='width:%.1f%%'></i><i class=p style='width:%.1f%%'></i></div>" %
           (bp["built"], bp["total"], (", %d partial" % bp["partial"]) if bp["partial"] else "", pct(bp["built"], bp["total"]), pct(bp["partial"], bp["total"])))
         w("<div class=legend><span><b style='background:var(--ok)'></b>built %d</span><span><b style='background:var(--warn)'></b>partial %d</span><span><b style='background:var(--idle)'></b>not built %d</span>%s</div>" % (bp["built"], bp["partial"], bp["todo"], ("<span>not needed %d (not counted)</span>" % bp["skip"]) if bp["skip"] else ""))
         w("<div class=stack>" + "".join("<i title='%s %d' style='width:%.1f%%;background:%s'></i>" % (k, v, pct(v, bp["total"]), KC.get(k, "#888")) for k, v in kinds.items()) + "</div>")
@@ -1753,16 +1753,18 @@ def chip(f, cls=""):
         w("<div class=phases style='margin-top:10px'>")
         for p in bp["phases"]:
             now = p is cur
-            w("<details class='card ph%s' id='phase-%s'><summary><b>Phase %s: %s</b>%s<div class='stack3' style='margin:6px 0'><i class=b style='width:%.0f%%'></i><i class=p style='width:%.0f%%'></i></div><div class=sub>%d of %d rows built%s%s</div><div class=k>%s</div><div class=k>module files: %s</div><div class=k>Click to show the rows</div></summary>" %
-              (" now" if now else "", esc(p["id"]), esc(p["id"]), esc(p["name"]), " <span class=nowtag>building now</span>" if now else "", pct(p["built"], p["n"]), pct(p["partial"], p["n"]), p["built"], p["n"], (", %d partial" % p["partial"]) if p["partial"] else "", (" &middot; %d runs" % runs_by.get(str(p["id"]), 0)) if runs_by.get(str(p["id"])) else "", esc(" ".join("%s %d" % kv for kv in p["kinds"].items())),
+            w("<details class='card ph%s' id='phase-%s'%s><summary><b>Phase %s: %s</b>%s<div class='stack3' style='margin:6px 0'><i class=b style='width:%.0f%%'></i><i class=p style='width:%.0f%%'></i></div><div class=sub>%d of %d rows built%s%s</div><div class=k>%s</div><div class=k>module files: %s</div><div class=k>Click to show the rows</div></summary>" %
+              (" now" if now else "", esc(p["id"]), " open" if now else "", esc(p["id"]), esc(p["name"]), " <span class=nowtag>building now</span>" if now else "", pct(p["built"], p["n"]), pct(p["partial"], p["n"]), p["built"], p["n"], (", %d partial" % p["partial"]) if p["partial"] else "", (" &middot; %d runs" % runs_by.get(str(p["id"]), 0)) if runs_by.get(str(p["id"])) else "", esc(" ".join("%s %d" % kv for kv in p["kinds"].items())),
                esc(" ".join(("%s %s" % (n, "yes" if v else "no")) for n, v in (("definition", p["def"]), ("brief", p["brief"]), ("ledger", p["ledger"]))))))
-            w("<div class=tw><table class=steps><tr><th>#</th><th>Kind</th><th>Row</th><th>State</th></tr>" + "".join("<tr class='r-%s'><td>%s</td><td>%s</td><td>%s</td><td>%s</td></tr>" % (x["cls"], esc(x["n"]), esc(x["kind"]), esc(x["step"]), esc(x["state"][:90])) for x in p["steps"]) + "</table></div></details>")
+            if str(p["id"]) in phase_detail: w(phase_detail[str(p["id"])])
+            else: w("<div class=tw><table class=steps><tr><th>#</th><th>Kind</th><th>Row</th><th>State</th></tr>" + "".join("<tr class='r-%s'><td>%s</td><td>%s</td><td>%s</td><td>%s</td></tr>" % (x["cls"], esc(x["n"]), esc(x["kind"]), esc(x["step"]), esc(x["state"][:90])) for x in p["steps"]) + "</table></div>")
+            w("</details>")
         w("</div>")
     else: w("<div class=card>No plan to summarise yet: %s</div>" % esc(args.plan_msg or "architecture/build-plan.md was not read."))
-    if args.fragment:   # the full plan tables repeat the phase cards above row by row, so they start folded
-        w("<details id=plan-tables class='card frag'><summary><b>Full plan tables and views A and B</b><div class=sub>Every row of build-plan.md with its claims and drift, the script folders (A) and module proof (B). The phase cards above already show each row.</div></summary>")
+    if args.fragment:   # plan tally, views A (script folders) and B (module proof); the per-phase rows live in the cards above
+        w("<section id=plan-tables class='card frag'>")
         w(args.fragment)
-        w("</details>")
+        w("</section>")
     w(render_pages(proj, args.outdir, [SELF_OUT]))
     # footer
     w("<footer><h2>How these numbers are derived</h2><ul>")
@@ -1812,6 +1814,7 @@ def main_():
     ap.add_argument("--out"); ap.add_argument("--toolkit")
     ap.add_argument("--rates", help="rates file (glob input cache-write cache-read output per line); default <project>/.claude/progress-report-rates.txt if present, else placeholder rates")
     ap.add_argument("--fragment", help="HTML file with the plan tables and views A and B, embedded as the build detail")
+    ap.add_argument("--phase-detail", help="HTML file of <!--phase:ID-->...<!--/phase--> blocks (rows with Produces, Depends on, Scripts, and Claims), spliced into each phase card")
     ap.add_argument("--plan-json", help="parsed build plan (build-plan-status.sh --json shape)")
     ap.add_argument("--plan-msg", default="")
     ap.add_argument("--logs-dir", help="folder holding the <slug>/ session log folders (default: <CLAUDE_CONFIG_DIR or ~/.claude>/projects)")
@@ -1819,7 +1822,7 @@ def main_():
     ap.add_argument("--session-cache", help="discovery cache file (default <project>/.claude/progress-report-sessions.txt)")
     ap.add_argument("--no-log", action="store_true", help="skip session log discovery")
     a = ap.parse_args()
-    for _k in ("out", "fragment", "plan_json", "logs_dir", "session_cache", "toolkit"):
+    for _k in ("out", "fragment", "phase_detail", "plan_json", "logs_dir", "session_cache", "toolkit"):
         if getattr(a, _k, None): setattr(a, _k, fix_msys(getattr(a, _k)))
     a.session_log = [fix_msys(x) for x in a.session_log]
     proj = os.path.abspath(os.path.expanduser(fix_msys(a.project)))
@@ -1842,7 +1845,7 @@ def main_():
     rates = load_rates(rf if os.path.isfile(rf) else None)
     a.rates_used = rf if os.path.isfile(rf) else ""; a.rates_default = rf
     bp = load_build_plan(a.plan_json or os.path.join(proj, "architecture", "build-plan.json"), proj)
-    a.fragment = read(a.fragment) if a.fragment else ""; a.plan_msg = a.plan_msg
+    a.fragment = read(a.fragment) if a.fragment else ""; a.phase_detail = {m.group(1): m.group(2) for m in re.finditer(r"<!--phase:(.*?)-->(.*?)<!--/phase-->", read(a.phase_detail) if a.phase_detail else "", re.S)}; a.plan_msg = a.plan_msg
     if a.session_log and not a.no_log:
         sessions, base = explicit_sessions(a.session_log), ""
     else: sessions, base = ([], "") if a.no_log else find_sessions(proj, a.logs_dir, a.session_cache)
diff --git a/tests/wave2/test-progress-report-polish.sh b/tests/wave2/test-progress-report-polish.sh
index 80ab7548..ad47c422 100644
--- a/tests/wave2/test-progress-report-polish.sh
+++ b/tests/wave2/test-progress-report-polish.sh
@@ -11,8 +11,8 @@
 #   P5  every State the build-plan skill lists (built / partial / not built / blocked) and the spellings the card
 #       counts (complete, verified, in progress, ...) are read the same way by the card, the plan header and the
 #       phase badge; a genuinely unreadable value is still reported as unreadable.
-#   P6  the full plan tables (the fragment) start folded under the phase cards, which already list every row; the
-#       "Jump to the plan tables" link still lands on them (openTarget opens a targeted <details>).
+#   P6  one plan view (#474): every row appears once, inside its phase card with Produces, Depends on, Scripts and
+#       Claims; the now phase renders open; views A and B stay as their own section.
 #   P3  no session log: one calm line (not the orange box), the regenerate hint inside "How this was measured";
 #       with --no-log the page never claims it looked in folders or logs.
 #
@@ -65,11 +65,16 @@ hasnt "$R" "cannot read" "no State cell is called unreadable"
 has "$R" "1 of 2 rows built" "Phase 2 shows 1 of 2 built"
 hasnt "$R" 'st-unknown">unknown' "no phase is labelled unknown"
 
-echo "P6 full plan tables start folded"
-has "$R" "<details id=plan-tables class='card frag'><summary>" "plan tables sit in a <details> with a summary"
-hasnt "$R" "<details id=plan-tables[^>]* open" "plan tables are closed by default"
-has "$R" "href='#plan-tables'" "the jump link still points at them"
-hasnt "$R" "<div id=plan-tables" "the old open <div> wrapper is gone"
+echo "P6 one plan view: rows once, in the phase cards; the now phase starts open; views A and B stay"
+for n in 3; do [ "$(grep -c "<td>$n</td>" "$R")" = 1 ] && ok "row $n appears once" || bad "row $n appears once"; done
+hasnt "$R" "Full plan tables" "the duplicate plan-tables block is gone"
+has "$R" "<details class='card ph now' id='phase-2' open>" "the now phase renders open"
+hasnt "$R" "<details class='card ph' id='phase-1'[^>]* open" "other phases start closed"
+has "$R" "<th>Produces / Proves</th><th>Depends on</th><th>State</th><th>Scripts</th>" "cards carry the full row columns"
+has "$R" "<h2>A\\. Script folders" "view A stays"
+has "$R" "<h2>B\\. Modules" "view B stays"
+has "$R" "href='#plan-tables'" "the jump link still points at views A and B"
+has "$R" "details.ph\\[open\\]" "accordion script present"
 
 echo "P2 placeholder failure on a later stage is pending"
 P2="$WORK/p2"; mkproj "$P2" "**Stage P - Kickoff**, in progress - gates passed: none"

From 08921f7e00017d67bd430f5437cacf87b37ee6fa Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Tue, 20 Oct 2026 05:06:56 +0000
Subject: [PATCH 2/3] fix(progress-report): review fixes for #474: one
 accordion handler, hash link wins on load, real JS assertions

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01XruEA24qrAkK7Zqxc5Z59E
---
 project-bin/progress-report.py             | 4 ++--
 tests/wave2/test-progress-report-polish.sh | 6 ++++--
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/project-bin/progress-report.py b/project-bin/progress-report.py
index 9e424353..9ca25da7 100755
--- a/project-bin/progress-report.py
+++ b/project-bin/progress-report.py
@@ -1392,7 +1392,7 @@ def rule(ratio, kind, title, seen, matters, fix, why, lost_s=0.0, lost_tok=0.0,
     health = run_health(cs)
     return fired[:ADV["max_items"]], quiet + fired[ADV["max_items"]:], health, cs, hidden[0]
 
-JS = "<script>\n" + 'function openTarget(){var h=location.hash.slice(1);if(!h)return;var e=document.getElementById(h);if(!e)return;\n if(e.tagName===\'DETAILS\')e.open=true;var d=e.closest&&e.closest(\'details\');if(d)d.open=true;}\nwindow.addEventListener(\'hashchange\',openTarget);openTarget();\ndocument.addEventListener(\'toggle\',function(ev){var t=ev.target;if(!t.open||!t.classList||!t.classList.contains(\'ph\'))return;[].forEach.call(document.querySelectorAll(\'details.ph[open]\'),function(o){if(o!==t)o.open=false})},true);\ndocument.addEventListener(\'toggle\',function(ev){var t=ev.target;if(!t.open||!t.classList||!t.classList.contains(\'ph\'))return;[].forEach.call(document.querySelectorAll(\'details.ph[open]\'),function(o){if(o!==t)o.open=false})},true);\n(function(){var links=[].slice.call(document.querySelectorAll(\'.snav a[href^="#stage-"]\'));\n if(!(\'IntersectionObserver\' in window))return;var io=new IntersectionObserver(function(es){es.forEach(function(en){if(en.isIntersecting){\n  links.forEach(function(l){l.style.outline=l.getAttribute(\'href\')===\'#\'+en.target.id?\'2px solid var(--accent)\':\'none\'});}})},{rootMargin:\'-60px 0px -70% 0px\'});\n links.forEach(function(l){var t=document.querySelector(l.getAttribute(\'href\'));if(t)io.observe(t)});})();\n' + "</script>"
+JS = "<script>\n" + 'function openTarget(){var h=location.hash.slice(1);if(!h)return;var e=document.getElementById(h);if(!e)return;\n if(e.tagName===\'DETAILS\')e.open=true;var d=e.closest&&e.closest(\'details\');if(d)d.open=true;var ph=d||e;if(ph.classList&&ph.classList.contains(\'ph\'))[].forEach.call(document.querySelectorAll(\'details.ph[open]\'),function(o){if(o!==ph)o.open=false});}\nwindow.addEventListener(\'hashchange\',openTarget);openTarget();\ndocument.addEventListener(\'toggle\',function(ev){var t=ev.target;if(!t.open||!t.classList||!t.classList.contains(\'ph\'))return;[].forEach.call(document.querySelectorAll(\'details.ph[open]\'),function(o){if(o!==t)o.open=false})},true);\n(function(){var links=[].slice.call(document.querySelectorAll(\'.snav a[href^="#stage-"]\'));\n if(!(\'IntersectionObserver\' in window))return;var io=new IntersectionObserver(function(es){es.forEach(function(en){if(en.isIntersecting){\n  links.forEach(function(l){l.style.outline=l.getAttribute(\'href\')===\'#\'+en.target.id?\'2px solid var(--accent)\':\'none\'});}})},{rootMargin:\'-60px 0px -70% 0px\'});\n links.forEach(function(l){var t=document.querySelector(l.getAttribute(\'href\'));if(t)io.observe(t)});})();\n' + "</script>"
 # ----------------------------------------------------------------- render
 CSS = """
 :root{--bg:#f6f7f9;--card:#fff;--ink:#1c2430;--mut:#667085;--line:#e3e7ee;--accent:#2f6fed;--ok:#1f9d63;--warn:#d98a0b;--bad:#c93c3c;--idle:#d9dde5;--main:#2f6fed;--sub:#8b5cf6;--soft:#eef2fb;--shadow:0 1px 2px rgba(20,30,50,.06)}
@@ -1753,7 +1753,7 @@ def chip(f, cls=""):
         w("<div class=phases style='margin-top:10px'>")
         for p in bp["phases"]:
             now = p is cur
-            w("<details class='card ph%s' id='phase-%s'%s><summary><b>Phase %s: %s</b>%s<div class='stack3' style='margin:6px 0'><i class=b style='width:%.0f%%'></i><i class=p style='width:%.0f%%'></i></div><div class=sub>%d of %d rows built%s%s</div><div class=k>%s</div><div class=k>module files: %s</div><div class=k>Click to show the rows</div></summary>" %
+            w("<details class='card ph%s' id='phase-%s'%s><summary><b>Phase %s: %s</b>%s<div class='stack3' style='margin:6px 0'><i class=b style='width:%.0f%%'></i><i class=p style='width:%.0f%%'></i></div><div class=sub>%d of %d rows built%s%s</div><div class=k>%s</div><div class=k>module files: %s</div><div class=k>Click to show or hide the rows</div></summary>" %
               (" now" if now else "", esc(p["id"]), " open" if now else "", esc(p["id"]), esc(p["name"]), " <span class=nowtag>building now</span>" if now else "", pct(p["built"], p["n"]), pct(p["partial"], p["n"]), p["built"], p["n"], (", %d partial" % p["partial"]) if p["partial"] else "", (" &middot; %d runs" % runs_by.get(str(p["id"]), 0)) if runs_by.get(str(p["id"])) else "", esc(" ".join("%s %d" % kv for kv in p["kinds"].items())),
                esc(" ".join(("%s %s" % (n, "yes" if v else "no")) for n, v in (("definition", p["def"]), ("brief", p["brief"]), ("ledger", p["ledger"]))))))
             if str(p["id"]) in phase_detail: w(phase_detail[str(p["id"])])
diff --git a/tests/wave2/test-progress-report-polish.sh b/tests/wave2/test-progress-report-polish.sh
index ad47c422..c8b96b41 100644
--- a/tests/wave2/test-progress-report-polish.sh
+++ b/tests/wave2/test-progress-report-polish.sh
@@ -66,7 +66,8 @@ has "$R" "1 of 2 rows built" "Phase 2 shows 1 of 2 built"
 hasnt "$R" 'st-unknown">unknown' "no phase is labelled unknown"
 
 echo "P6 one plan view: rows once, in the phase cards; the now phase starts open; views A and B stay"
-for n in 3; do [ "$(grep -c "<td>$n</td>" "$R")" = 1 ] && ok "row $n appears once" || bad "row $n appears once"; done
+[ "$(grep -c "<td>3</td>" "$R")" = 1 ] && ok "row 3 appears once" || bad "row 3 appears once"
+[ "$(grep -c ">done 2026-10-12</td>" "$R")" = 1 ] && ok "row 2 state appears once" || bad "row 2 state appears once"
 hasnt "$R" "Full plan tables" "the duplicate plan-tables block is gone"
 has "$R" "<details class='card ph now' id='phase-2' open>" "the now phase renders open"
 hasnt "$R" "<details class='card ph' id='phase-1'[^>]* open" "other phases start closed"
@@ -74,7 +75,8 @@ has "$R" "<th>Produces / Proves</th><th>Depends on</th><th>State</th><th>Scripts
 has "$R" "<h2>A\\. Script folders" "view A stays"
 has "$R" "<h2>B\\. Modules" "view B stays"
 has "$R" "href='#plan-tables'" "the jump link still points at views A and B"
-has "$R" "details.ph\\[open\\]" "accordion script present"
+has "$R" "addEventListener(.\\?'toggle" "accordion toggle handler present in the script"
+[ "$(grep -o "addEventListener(.\\?'toggle" "$R" | wc -l)" = 1 ] && ok "accordion handler is not duplicated" || bad "accordion handler is not duplicated"
 
 echo "P2 placeholder failure on a later stage is pending"
 P2="$WORK/p2"; mkproj "$P2" "**Stage P - Kickoff**, in progress - gates passed: none"

From 784d743f31f6a91dc6e2b97eed7034558e3d75a3 Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Tue, 20 Oct 2026 11:07:35 +0000
Subject: [PATCH 3/3] fix(progress-report): guard --phase-detail for older
 generators; real accordion tests (#474)

- build-plan-status.sh passes --phase-detail only when progress-report.py
  knows it, so a generator left behind by a partial sync still writes the
  page (cards fall back to the short row tables) instead of exiting on
  "unrecognized arguments".
- polish P6b: an old-generator copy without the flag still renders; the
  copy is checked to reject the flag directly.
- polish P6c: the page's own two scripts run in node against a DOM shim
  with coalesced toggle events: now phase open on load, opening phase 1
  closes phase 2, #phase-3 on load wins over the default, restored state
  wins over the default. Both new cases fail against a mutant.
- Remove dead CSS: .ph .scroll and .frag section.phase (.frag .scroll is
  still used by views A and B).
- CHANGELOG: credit MendixMau on the #474 line.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01VJgWP5vEoAsNsJYqCDMGNw
---
 CHANGELOG.md                               |  2 +-
 project-bin/build-plan-status.sh           |  6 +-
 project-bin/progress-report.py             |  4 +-
 tests/wave2/test-progress-report-polish.sh | 69 ++++++++++++++++++++++
 4 files changed, 77 insertions(+), 4 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 64add8d5..b5a253d7 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -19,7 +19,7 @@ Sections dated before 2026-09-19 predate the cycle and stay as they are. Named m
 
 ## Unreleased
 
-- fix(progress-report): **the build plan is shown once: the phase cards are the plan.** Expanding a card now shows its full rows (#, Kind, Step, Produces/Proves, Depends on, State, Scripts) and Claims, which only the duplicate "Full plan tables" block carried; that block is gone and views A (script folders) and B (module proof) are their own section. The `now` phase renders open with a selected style, opening another phase closes the previous one (accordion; `#phase-N` links still open their card), and a viewer's saved open/closed state survives the 30 s reload and wins over the default (#474) — field feedback from a requirements-driven project at Stage 5
+- fix(progress-report): **the build plan is shown once: the phase cards are the plan.** Expanding a card now shows its full rows (#, Kind, Step, Produces/Proves, Depends on, State, Scripts) and Claims, which only the duplicate "Full plan tables" block carried; that block is gone and views A (script folders) and B (module proof) are their own section. The `now` phase renders open with a selected style, opening another phase closes the previous one (accordion; `#phase-N` links still open their card), and a viewer's saved open/closed state survives the 30 s reload and wins over the default (#474) — field feedback from a requirements-driven project at Stage 5 — MendixMau (field feedback 2026-10-20)
 
 ## v2026.10.19
 
diff --git a/project-bin/build-plan-status.sh b/project-bin/build-plan-status.sh
index fd115d4f..c7f1ac32 100755
--- a/project-bin/build-plan-status.sh
+++ b/project-bin/build-plan-status.sh
@@ -729,7 +729,11 @@ if [ "$WRITE_HTML" -eq 1 ]; then
       fi
       echo "<p class=note>The plan: what build-plan.md says. A: built, from mdlsource/ done- prefixes. B: proven, from verify-module.sh + docs/improvement-register.md. A phase at 100% with no reviewed row in B is built, not proven.</p>"
     } > "$FRAG_TMP"
-    GEN_ARGS=(--out "$OUT_TMP" --fragment "$FRAG_TMP" --phase-detail "$PHASE_DETAIL" --plan-msg "${PLAN_MSG:-}")
+    GEN_ARGS=(--out "$OUT_TMP" --fragment "$FRAG_TMP" --plan-msg "${PLAN_MSG:-}")
+    # An older progress-report.py (a `sync-project.sh --only-bin build-plan-status.sh` run, or a hand-edited
+    # copy) has no --phase-detail and would exit "unrecognized arguments", writing no page at all. Pass the
+    # flag only when the generator knows it; without it the phase cards keep their short row tables.
+    grep -q -- '--phase-detail' "$GEN" 2>/dev/null && GEN_ARGS+=(--phase-detail "$PHASE_DETAIL")
     [ -n "$PLAN_JSON" ] && GEN_ARGS+=(--plan-json "$PLAN_JSON")
     for _sl in ${SESSION_LOGS[@]+"${SESSION_LOGS[@]}"}; do GEN_ARGS+=(--session-log "$_sl"); done
     if "$PY" "$GEN" "$ROOT" "${GEN_ARGS[@]}" >/dev/null 2>"$OUT.err.$$"; then
diff --git a/project-bin/progress-report.py b/project-bin/progress-report.py
index 9ca25da7..951675d7 100755
--- a/project-bin/progress-report.py
+++ b/project-bin/progress-report.py
@@ -1443,11 +1443,11 @@ def rule(ratio, kind, title, seen, matters, fix, why, lost_s=0.0, lost_tok=0.0,
 details.ph>summary{list-style:none;cursor:pointer;color:var(--ink)}details.ph>summary::-webkit-details-marker{display:none}
 details.ph[open]{grid-column:1/-1;border-color:var(--accent);box-shadow:0 0 0 1px var(--accent);background:var(--soft)}.steps td{font-size:12px;padding:3px 6px}
 .nolog{margin:10px 0;font-size:13px}.note-card{margin-top:14px;border-left:4px solid var(--warn)}.live{font-size:12px;color:var(--muted)}.live a{color:inherit}details.how{margin:10px 0;font-size:13px;color:var(--muted)}details.how summary{cursor:pointer}
-.frag section.phase{margin-top:14px}.frag h2{font-size:16px;margin:24px 0 4px}.frag h3{margin:14px 0 2px}.frag .scroll{overflow-x:auto}
+.frag h2{font-size:16px;margin:24px 0 4px}.frag h3{margin:14px 0 2px}.frag .scroll{overflow-x:auto}
 .frag .lede{font-size:14px}.frag .empty{color:var(--mut)}.frag code{font-size:12px}
 .frag .st-built,.ph .st-built,.frag .done,.frag .CLEAN{color:var(--ok)}.frag .st-progress,.ph .st-progress,.frag .in-progress,.frag .INCOMPLETE{color:var(--warn)}
 .frag .st-todo,.ph .st-todo,.frag .pending{color:var(--mut)}.frag .st-person,.ph .st-person{color:#8b5cf6}.frag .st-unknown,.ph .st-unknown{color:var(--mut);font-style:italic}
-.frag .FINDINGS,.frag .drift,.ph .drift{color:var(--bad)}.frag .drift,.ph .drift{font-size:12px}.ph .scroll{overflow-x:auto}.ph .claims{font-size:12px}.frag .claims{font-size:12px}.frag .warn summary{color:var(--warn)}
+.frag .FINDINGS,.frag .drift,.ph .drift{color:var(--bad)}.frag .drift,.ph .drift{font-size:12px}.ph .claims{font-size:12px}.frag .claims{font-size:12px}.frag .warn summary{color:var(--warn)}
 .frag .badge,.ph .badge{margin-left:6px;border:1px solid currentColor;background:transparent}
 .pages .tree{margin-top:6px;font-size:13px}.pg{display:grid;grid-template-columns:minmax(120px,220px) minmax(0,1fr);gap:2px 12px;padding:5px 0;border-bottom:1px solid var(--line)}.pg:last-child{border-bottom:0}
 .pg .dir{font-weight:600;overflow-wrap:anywhere}.pg ul{list-style:none;margin:0;padding:0;display:flex;flex-wrap:wrap;gap:3px 14px}.pg li{overflow-wrap:anywhere}.pg details{margin:0}.pg details ul{margin-top:4px}
diff --git a/tests/wave2/test-progress-report-polish.sh b/tests/wave2/test-progress-report-polish.sh
index c8b96b41..59fe2091 100644
--- a/tests/wave2/test-progress-report-polish.sh
+++ b/tests/wave2/test-progress-report-polish.sh
@@ -13,6 +13,9 @@
 #       phase badge; a genuinely unreadable value is still reported as unreadable.
 #   P6  one plan view (#474): every row appears once, inside its phase card with Produces, Depends on, Scripts and
 #       Claims; the now phase renders open; views A and B stay as their own section.
+#   P6b an older progress-report.py without --phase-detail (a partial sync) still renders, with the short row tables.
+#   P6c the page's own scripts, run in node against a DOM shim: the now phase is open on load, opening phase 1
+#       closes phase 2, a #phase-3 link on load wins over the default, and restored state wins over the default.
 #   P3  no session log: one calm line (not the orange box), the regenerate hint inside "How this was measured";
 #       with --no-log the page never claims it looked in folders or logs.
 #
@@ -78,6 +81,72 @@ has "$R" "href='#plan-tables'" "the jump link still points at views A and B"
 has "$R" "addEventListener(.\\?'toggle" "accordion toggle handler present in the script"
 [ "$(grep -o "addEventListener(.\\?'toggle" "$R" | wc -l)" = 1 ] && ok "accordion handler is not duplicated" || bad "accordion handler is not duplicated"
 
+echo "P6b an older progress-report.py without --phase-detail still renders the page, with the short row tables"
+OLD="$WORK/oldbin"; mkdir -p "$OLD"
+cp "$SCRIPT" "$ROOT/project-bin/_common.sh" "$OLD/"
+# The generator as it was before #474: no --phase-detail option (the attribute defaults to empty).
+sed 's/ap\.add_argument("--phase-detail", help="[^"]*")/ap.set_defaults(phase_detail=None)/' "$ROOT/project-bin/progress-report.py" > "$OLD/progress-report.py"
+if grep -q -- '--phase-detail' "$OLD/progress-report.py"; then bad "old-generator copy still names --phase-detail"
+else
+  "$PY" "$OLD/progress-report.py" "$P1" --no-log --out "$WORK/old-direct.html" --phase-detail /dev/null >/dev/null 2>&1 \
+    && bad "control: the old-generator copy should reject --phase-detail" || ok "control: the old-generator copy rejects --phase-detail"
+  P6B="$WORK/p6b"; mkproj "$P6B" "**Stage 5 - Build**, in progress - gates passed: P, 0, 1, 2, 3, 4"
+  cp "$P1/architecture/build-plan.md" "$P6B/architecture/"
+  CLAUDE_CONFIG_DIR="$EMPTY" bash "$OLD/build-plan-status.sh" "$P6B" --html >"$WORK/p6b.out" 2>&1
+  R6B="$P6B/architecture/progress-report.html"
+  [ -f "$R6B" ] && ok "page written by the older generator" || bad "page written by the older generator ($(tail -n 1 "$WORK/p6b.out"))"
+  has "$R6B" "<th>#</th><th>Kind</th><th>Row</th><th>State</th>" "phase cards fall back to the short row table"
+  hasnt "$R6B" "unrecognized arguments" "no argparse error on the page"
+fi
+
+echo "P6c accordion and load order, run in node against a DOM shim"
+if command -v node >/dev/null 2>&1; then
+P6="$WORK/p6"; mkproj "$P6" "**Stage 5 - Build**, in progress - gates passed: P, 0, 1, 2, 3, 4"
+cp "$P1/architecture/build-plan.md" "$P6/architecture/"
+printf '\n### Phase 3 - Reports\n\n| # | Kind | Step | Produces | Depends on | State |\n|---|---|---|---|---|---|\n| 4 | BUILD | report page | 04-report.mdl | 3 | not built |\n' >> "$P6/architecture/build-plan.md"
+mkdir -p "$P6/.claude/loop"   # recent activity, so the page carries its autoreload (state save/restore) script
+printf 'ts\tscript\tgate\tresult\telapsed_s\tmxbuild_errors\tlint_rc\tlint_total\trestored\tscore\n%s\t02-billing.mdl\tpass\tapplied\t40\t0\t0\t\t0\t90.0\n' "$(date -u +%Y-%m-%dT%H:%M:%SZ)" > "$P6/.claude/loop/metrics-ledger.tsv"
+CLAUDE_CONFIG_DIR="$EMPTY" bash "$SCRIPT" "$P6" --html >/dev/null 2>&1
+node - "$P6/architecture/progress-report.html" <<'JSEOF' && ok "node: now phase open on load; opening phase 1 closes phase 2; #phase-3 beats the default; saved state beats the default" || bad "accordion and load order (node)"
+const fs = require("fs"), assert = require("assert");
+const h = fs.readFileSync(process.argv[2], "utf8");
+const main = h.match(/<script>([\s\S]*?)<\/script>/)[1];
+const reload = h.match(/<script id=autoreload>([\s\S]*?)<\/script>/)[1];
+const cards = [...h.matchAll(/<details class='card ph[^']*' id='(phase-[^']+)'( open)?>/g)].map(m => [m[1], !!m[2]]);
+assert.deepStrictEqual(cards.map(c => c[0]), ["phase-1", "phase-2", "phase-3"]);
+// One page load: details elements whose open setter queues a coalesced 'toggle' (as browsers do, also for the
+// open attribute at parse time), the page's two scripts run in order, then the queued toggles are dispatched.
+function load(hash, store) {
+  const pending = [], caps = [], els = {};
+  const queue = e => { if (!pending.includes(e)) pending.push(e); };
+  for (const [id, open] of cards) {
+    const e = {id, tagName: "DETAILS", _o: open, classList: {contains: c => c === "ph"}, closest: s => s === "details" ? e : null};
+    Object.defineProperty(e, "open", {get() { return this._o; }, set(v) { v = !!v; if (v !== this._o) { this._o = v; queue(this); } }});
+    els[id] = e; if (open) queue(e);
+  }
+  const all = Object.values(els);
+  const doc = {documentElement: {style: {}, scrollTop: 0}, getElementById: id => els[id] || null,
+    querySelectorAll: s => s === "details.ph[open]" ? all.filter(e => e.open) : s === "details[id]" ? all : [],
+    addEventListener: (n, f, cap) => { if (n === "toggle") caps.push(f); }};
+  const win = {pageYOffset: 0, addEventListener: () => {}, scrollTo: () => {}};
+  const ss = {getItem: k => k in store ? store[k] : null, setItem: (k, v) => { store[k] = String(v); }, removeItem: k => { delete store[k]; }};
+  const run = src => new Function("window", "document", "history", "location", "sessionStorage", "setTimeout", src)(
+    win, doc, {}, {hash, pathname: "/r.html", reload() {}}, ss, () => {});
+  run(main); run(reload);
+  const flush = () => { while (pending.length) { const t = pending.shift(); caps.forEach(f => f({target: t})); } };
+  flush();
+  return {els, flush, open: () => all.filter(e => e.open).map(e => e.id)};
+}
+let p = load("", {});
+assert.deepStrictEqual(p.open(), ["phase-2"], "the now phase is open on load");
+p.els["phase-1"].open = true; p.flush();
+assert.deepStrictEqual(p.open(), ["phase-1"], "opening phase 1 closes phase 2");
+assert.deepStrictEqual(load("#phase-3", {}).open(), ["phase-3"], "#phase-3 on load wins over the default");
+const saved = {"pr-view:/r.html": JSON.stringify({y: 0, d: {"phase-1": true, "phase-2": false, "phase-3": false}})};
+assert.deepStrictEqual(load("", saved).open(), ["phase-1"], "restored state wins over the default");
+JSEOF
+else ok "SKIP: node not available for the accordion behaviour test"; fi
+
 echo "P2 placeholder failure on a later stage is pending"
 P2="$WORK/p2"; mkproj "$P2" "**Stage P - Kickoff**, in progress - gates passed: none"
 cat > "$P2/index.html" <<'HTML'