Skip to content

new(brd-report): process map at the top, open-questions folded in - #438

Merged
MendixMau merged 16 commits into
release/v1.0from
feat/brd-report-process-map
Oct 16, 2026
Merged

MendixMau merged 16 commits into
release/v1.0from
feat/brd-report-process-map

Conversation

@MendixMau

Copy link
Copy Markdown
Owner

Stacks on #429, which is now merged into master; this branch carries only the fuse and fix work.

What changes

  • Stage 2: analysis/brd-report.html opens with the pass-1 process map (drawn by process-map.js build --embed, same renderer). Requirements and open questions are listed under it, each tied to its step. A click on a step filters the list; items on no step stay under "whole app" (collapsed when there are many).
  • open-questions.html and bin/questions-report.sh are retired. The gate and interview hook write the BRD report. An old docs/open-questions.html is never read and never fails the gate.
  • Stage 3: the prototype links to the full map and the map links back.
  • Field fixes in process-map.js: migrate (old anchors path, keeps *.bak), anchor reports carried-over judged entries, long processes (above 12 steps) get compact cards and scale to fit with a validate warning, BRD discovery also in analysis/knowledge-base/brd with an error that names the folders searched and brdDirs, PAGE-MAP.tsv page resolution, one CAUSE line in validate when most errors share a cause, globals grouped and collapsed in the map.
  • Skills updated: process-map, runbook, interview-protocol, coverage-ledger, report-schema. Always-loaded word counts equal master.

Tests

test-brd-report-process-map.sh (58), test-process-map-field.sh (24), test-process-map.sh (74), gate, interview-mode and open-questions tests pass. render-routing --check, check-skill-routing and check-portability are clean. Screenshots at 1440 and 420 checked; no horizontal page scroll at 420.

🤖 Generated with Claude Code

MendixMau and others added 12 commits October 16, 2026 17:33
…ess step

One swimlane board per business process. Each step sits in its role lane,
shows its wireframe and what the system does, and carries a pin for every
open question and ASSUMED decision about that moment, so a reviewer can
answer a question where it happens instead of in a flat list.

- skills/process-board.md: when, model authoring, anchoring, build, verify
  by looking
- project-bin/process-board.js + template: validate, anchor (derived plus
  judged by fingerprint, global counted), build (self-contained, light and
  dark, deterministic)
- Stage 3 Agent produces and Surface rows, process-board manifest row,
  swimlanes opt-in discharged by the model, CAC-4 line, routing row,
  install pair, check_stage_3 FAIL on a missing or stale board
- tests/wave2/test-process-board.sh on a synthetic order approval fixture

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
… default, visible click cues

Pass 1 at Stage 2 close shows each consolidated question in its place in the work before it is raised, and feeds module boundaries, journeys and the screen list. Pass 2 only re-runs the script once wireframes exist.
Board opens light; a how-to bar, an Open footer per card and a first-step pulse make the click targets obvious.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…d theme

Cards render the wireframe at 640px wide (was 1280) so text is legible; the drawer preview is larger. Screens drop any hard-coded theme and follow the Theme button. iframe sandbox now allow-same-origin (still no scripts): an empty sandbox left srcdoc frames blank in Chrome.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Fallback path, anchor --json stdout, judged entry placement, id examples,
standard buckets, exit codes, screenshot flags, step count guidance,
ignorePages option, dashboard and export rule.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
… committed, pass 1 without screens, CAC-3 wiring, optin row, routing re-render)

Merges origin/master (CHANGELOG both sides kept).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…hitecture

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The Stage 2 report opens with the pass-1 process map (process-map.js build
--embed) and lists requirements and open questions under it; a click on a step
filters the list. open-questions.html and questions-report.sh are retired; an
old page is never a gate failure. Stage 3 links the prototype and the full map
both ways. Also: process-map.js migrate, carried-over report in anchor, long
processes scaled to fit, BRD discovery in analysis/knowledge-base/brd,
PAGE-MAP.tsv page resolution, one-line cause summary in validate.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…cess-map

# Conflicts:
#	CHANGELOG.md
#	project-bin/process-map.js
#	project-bin/process-map.template.html
#	skills/conversion-runbook.md
#	skills/process-map.md
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 00:21
Fit the embedded map for every overflowing board and fix the hint text; render a questions-only
report when no BRDs exist; migrate keeps judged placements, reports collisions and empty entries,
keeps an existing .bak and writes atomically; PAGE-MAP.tsv beats the filename stem and warns on a
missing file; fit re-runs on resize; tests for each.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@MendixMau
MendixMau merged commit d99c83d into release/v1.0 Oct 16, 2026
2 checks passed
MendixMau added a commit that referenced this pull request Oct 16, 2026
Follow-up to #438 review. Review: APPROVE.

# PR 453 (process-map migrate follow-ups to #438 r2) review r1 (delta self-review)
Verdict: APPROVE
- Nested-key: regex result is kept only if JSON.parse(patched).anchors === want; otherwise full re-write. Output claim now matches what happened. Test with a nested meta.anchors look-alike plus a plain-model control.
- String targets: one line in anchorEntries wraps a non-array in a list; test checks it becomes one step target and rc 0.
- Unfilled current entries: listed with ids in the output; test names F001:Q77.
- 6 of 7 new checks fail on the pre-fix code (worker report). Field 59, process-map 74, brd-report 73, all 0 FAIL; portability and discipline clean.
- No behaviour change outside migrate; no names in the diff.
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