Skip to content

new(progress-report): replace build-plan.html with a progress report - #434

Merged
MendixMau merged 13 commits into
release/v1.0from
feat/progress-report
Oct 16, 2026
Merged

MendixMau merged 13 commits into
release/v1.0from
feat/progress-report

Conversation

@MendixMau

@MendixMau MendixMau commented Oct 16, 2026 •

Copy link
Copy Markdown
Owner

What

Replaces architecture/build-plan.html with a generated progress report (architecture/progress-report.html, self-contained, stdlib Python behind build-plan-status.sh). It keeps everything the old page showed (phases, built and proven views) and adds stages P to 6 with artifact links, a progress bar, a sticky stage nav, and, when a Claude Code session log is available, time per stage, sub-agent use, context size, an estimated token and cost figure, a run health strip, a speed panel and a right-hand column of actionable tips.

  • Session logs are optional. Without one the project sections render and time and token data are marked unavailable. Only numbers, stage names, script names, artifact paths and short sub-agent descriptions are read; no prompt or message text is copied.
  • Tips are grouped by theme, collapsed by default, with calm severity. Costs inherent to the toolkit or mxcli (builds, required gates, waiting) are counted but never shown.
  • Context window is read per model from a small table and raised if the log shows more; never a percentage over 100.
  • Setting Progress report: off | ask | auto in PROJECT.md (default ask). Regenerated on every exec.sh slice; the 30 s self-reload applies only while a build is active.
  • An existing build-plan.html is still accepted by gate-check and status.sh and never deleted. --json and build-plan.json are unchanged.

Impact

Row Answer
Process Surface artifact for stage 4 is now progress-report.html; build-plan.html still accepted. New optional PROJECT.md setting.
Dependencies None new. python3 stdlib only; page skipped when python3 is absent.
Context cost render-routing.sh --check: base 75204 words, head 75204 words (budget 80000). Skill words total about 308277 vs 308384 on master.
Behaviour RUN on a real in-progress project (Stage 5, 153 plan rows) with its real session log: stage switch from the first metrics-ledger row, Stage 4 and Stage 5 windows match the chart band and the sub-agent tables, last build step matches the newest ledger row. Fixture tests cover the rest (F10 to F14).
Runtime Generation is a single Python pass over the session log; no network.

Tests

  • tests/wave2/test-progress-report.sh (new): 39 checks on a synthetic project and synthetic JSONL log.
  • tests/wave2/test-build-plan-json.sh: 66 pass. tests/wave2/test-surface-column.sh: 13 pass.
  • bin/check-portability.sh on changed files and render-routing.sh --check: clean.

Not verified

Visual layout (sticky side column, phone order, severity dots) was not viewed in a browser. Full test suite and CI not run locally.

Round 5 measured (head 0a908a3)

  • Generation time on the real project above with an explicit session log: 1 s. With discovery against a 300-file ~/.claude/projects: about 1 s cold, under 1 s warm (discovery cached in .claude/progress-report-sessions.txt; F13 asserts a cached run does not reopen session files).
  • Autoreload keeps the reader's place: scroll position and open sections are saved in sessionStorage and restored, history.scrollRestoration='manual', no reload within 20 s of interaction, and a Live / Pause control (F14).
  • Tests: test-progress-report-field-fixes 90, test-progress-report 101, test-build-plan-json 70, test-build-plan-status-args 6, all passing.

🤖 Generated with Claude Code

MendixMau and others added 3 commits October 16, 2026 19:19
Generated architecture/progress-report.html covers phases, built and proven views, stages P to 6, progress bar, and optional time, token and tip data from session logs (numbers only, no message text). Setting 'Progress report: off | ask | auto'. An existing build-plan.html is still accepted and never deleted.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
… builtin-free repeat tips, exact --open URL

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
MendixMau and others added 5 commits October 16, 2026 20:37
… window from log or inferred, inherent never hangs, red at 3+ runs

Context window is 1M only when the log records it, else 200k; a context above 200k with auto compactions caps at the compaction point and is labelled inferred. Inherent scripts never make hang tips or health. Red for a key script needs 3+ runs. Tips under a minute and non-script tokens dropped. CHANGELOG matches the code.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Wrap code and values, stack stage label/value rows, let the gates label
drop below the header, scroll the stage nav inside its own container.
Add a static narrow-width CSS test.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…wider log discovery, ledger fallback, Build progress)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@MendixMau
MendixMau changed the base branch from master to release/v1.0 October 16, 2026 16:21
MendixMau and others added 4 commits October 17, 2026 02:32
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…ecision and config-folder scoping

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
… cold or warm cache

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…e it to project path and rules, cap cold skips, ledger decides the switch

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
MendixMau added a commit that referenced this pull request Oct 16, 2026
Remove conflict markers in CHANGELOG, correct the exec timing source, the merged claim and the #434 claims, add the decompose, rehost and mxpatch bullets, and apply the cheap wording fixes.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@MendixMau
MendixMau merged commit 59cb777 into release/v1.0 Oct 16, 2026
2 checks passed
MendixMau added a commit that referenced this pull request Oct 16, 2026
Drop the stale #434 clause, align the paste-in test rule with the agreed
moments, add the interview-gate hook re-install step, and use the real
skill name learned-multilingual-app.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
MendixMau added a commit that referenced this pull request Oct 16, 2026
#454)

Opus review 454-r1: APPROVE

# PR #454 review, round 1

PR: fix(progress-report): polish counts, stage 0 pending, calm no-log line
Base: release/v1.0, head 98649f9 (branch fix/progress-report-polish). Up to date with base, merges clean.

Verdict: APPROVE

## Must-fix

None.

## What I checked

1. **"done" rows count as built (build-plan-status.sh).** Both `classify()` copies (the plan-table rollup at line 303 and the HTML tally at line 559) now use `startswith(("built", "done"))`, with "not built" still checked first. progress-report.py `classify_state` already treats `done` as built, so the two "done" readings now match. `**done** 2026-...` and `Done` both count in both parsers (probe run). `not done` does not match `done` in either.
2. **Future-stage placeholder FAIL shows PENDING (progress-report.py ~1475).** It only relabels when all of these hold: status is `FAIL`, the detail contains "placeholder", `info["current"]` is a known stage, and the stage index is strictly greater than the current one. The current stage and earlier stages keep FAIL. With an unknown current stage (no or unparseable PROJECT.md) nothing is relabelled, which is the safe default. Only two gate-check.sh FAIL messages contain "placeholder" (lines 827 and 835, both the `Confirmed by:` sign-off check), so a substantive failure on a later stage stays FAIL unless its detail happens to say "placeholder". The stage badge for a later stage is already "pending" (`stage_state` comes from PROJECT.md), so the sub line now agrees with the badge.
3. **Calm no-log line.** The orange `note-card` box is replaced by `<p class=nolog>`. The looked-in counts and the regenerate hint move into the `How this was measured` details. That block always exists when there is no log, so the line's pointer to it never dangles. `--no-log` drops the "Looked in" text and the regenerate hint. The ledger card still renders under the second `if not L:`.
4. **Test-moments wording.** Every change = "Gate check plus only the tests that touch the change" (impacted only). Build complete = "Everything, once". Before every later deploy = "Everything". Module done now names the per-flow unit tests chosen at CAC-5, listed in the module brief. That matches the rule (impacted-only per change, full run at build complete and before each deploy, unit-test choice at CAC-5) and the `Unit flows:` brief list from #417 and its follow-up.
5. **Prose and hygiene.** No em or en dashes in added prose. The only one is the CHANGELOG attribution separator ` — field review of the #434 report`, which follows the existing entry convention. No private or client names: the fixtures use Orders, Billing and Kickoff. No conflict markers. The CHANGELOG entry sits under Unreleased, is accurate and is credited, and check-pr-discipline is clean.

## Tests run (detached worktree at 98649f9, removed afterwards)

| Test | Result |
|---|---|
| tests/wave2/test-progress-report-polish.sh (new) | 20 / 0 |
| tests/wave2/test-progress-report.sh | 101 / 0 |
| tests/wave2/test-progress-report-field-fixes.sh | 101 / 0 |
| tests/wave2/test-progress-report-gitignore.sh | 6 / 0 |
| tests/wave2/test-build-plan-json.sh | 70 / 0 |
| tests/wave2/test-build-plan-status-args.sh | 6 / 0 |
| bin/check-portability.sh, bin/check-pr-discipline.sh | clean |

The new test against the base scripts (project-bin from release/v1.0): 12 of 20 fail. So the test pins the new behaviour. The 8 that pass on base are the pre-existing parts: report written, Stage 5 card count, Phase 2 count, and the `--no-log` "never claims it looked" checks. Discovery is already off under `--no-log`, which makes the new `and not args.no_log` guard defensive only.

## Nice-to-haves

1. **"One plan count" holds for `done`, not for the rest of the vocabulary.** progress-report.py also reads `complete`, `verified`, `proven` and `passed` as built, and `partial`, `in progress`, `started` and `wip` as partial. build-plan-status.sh reads all of these as unknown. That includes `partial` and `blocked`, which brd-to-build-plan.md line 333 lists as legal State values. Probe: one plan with rows `complete (...)`, `**done** ...` and `Done` gives "3 of 3 rows built" on the card. The plan header shows "2 built, 1 with a State cell this page cannot read" and the phase is labelled "unknown". This disagreement predates the PR, but the PR body promises the card, header and badge agree. Follow-up: share one vocabulary, or at least add `partial`/`blocked` to the sh classifier as non-built. Alternatively, reword the CHANGELOG to say "done rows" only.
2. **Narrow the PENDING relabel to `states[s] == "pending"`.** Today the relabel also applies to a later stage that PROJECT.md lists as passed, when the gate page is not stale. This happens when the current stage moves back during a change round. A real conflict, "passed but the sign-off is still a placeholder", would then read PENDING. Adding the state check costs one condition.
3. **Test gaps for item 2.** Add one test where a non-placeholder FAIL on a later stage stays FAIL, and one where an unknown current stage (no stage line) keeps FAIL.
4. **Module done wording is a little narrow.** Unit tests can also be switched on per module (`Unit modules:` in PROJECT.md, `Unit tests: yes` in the brief) or per project (`Unit tests default: yes`). "listed in the module brief" covers the per-flow default only. A shorter alternative: "plus the unit tests the CAC-5 test strategy selects for the module".
5. **Reading order.** The calm line renders after the collapsed details block, so the pointer "details in How this was measured" points upward. Consider writing the line first.
6. **Pre-existing, outside this PR's scope.** `build-plan-status.sh` called by a relative path (`bash project-bin/build-plan-status.sh <proj> --html`) says "progress-report.py is not next to this script". GEN is `$(dirname "${BASH_SOURCE[0]}")/...`, which probably breaks after the script changes directory (cause not confirmed). An absolute path works.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant