Repository navigation
new(poc-build): POC build — design first, one approval stop, a 12-row must-have list (#729) - #743
Conversation
#729) Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013jQvaonym9V1wKKfAEPmRr
…s, strict order, three styles, skip list, 11-row checklist Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013jQvaonym9V1wKKfAEPmRr
…load nav, phone nav, unreachable rules, no always-true checks, clean demo data, list order, forbidden options, 403 limit Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013jQvaonym9V1wKKfAEPmRr
…ign paths, approval stop) Review: MERGE-AFTER-FIX applied Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013jQvaonym9V1wKKfAEPmRr
…hree restatements (#729) Second review: the status block said 'no failures seen' although two development runs failed before the last fix; it was also twice the size it needed to be. Same facts, about 125 words. Removed a duplicated never-waive instruction, a question-moment count that read as conflicting with the no-prototype path, and a repeated because-clause in Precedence. Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013jQvaonym9V1wKKfAEPmRr
|
Review: MERGE-AFTER-FIX: the skill asks "Normal build or POC build?" whenever Value: this takes #729's light mode as triage option 2: an on-demand skill, with no hook or gate changes. It also folds in #730's measurements:
Text only, about 1,575 words, and not overbuilt. "Generic first" holds: it lives in Regression risk: no script, gate, template or hook changes, so
Correctness:
The PR adds no fixtures, and none were needed. Context cost: 66,014 → 66,014 baseline words (budget 80,000), measured with Time cost: none per build row. The flow itself was reported at about 36 minutes for one build (not re-measured). Exact fixes required:
No merge conflict with origin/master. Generated by Claude Code |
…ontrol step, intake skipped, gate/review roles (#729) Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013jQvaonym9V1wKKfAEPmRr
…ntities; a missing Build type is full otherwise (#729) Review finding: every fresh scaffold already carries a Current stage line, so 'a stage line exists' could not tell a new project from one in flight, and 'model content beyond the blank scaffold' was not checkable. Birth is now the exact scaffold register line plus SHOW ENTITIES listing none; an explicit switch of a project past birth is confirmed once and recorded. Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013jQvaonym9V1wKKfAEPmRr
|
The fixes from the review above are in
Also: the test app's identifier used as an example is replaced by a neutral description. The skill does not cite #730. Re-review walked six cases (fresh scaffold with and without a POC ask; Stage 5 with "demo this week"; Stage 5 with an explicit switch; Generated by Claude Code |
…ake Q16, Build type: full | poc) (#729) Mirrors Pipeline mode (6f51620): question in bin/lib/intake-template.sh, scaffold line in init-project.sh's PROJECT.md, Stage P row names it. Existing projects: sync reports the new question only; no line = full. poc-build.md 'Who asks' simplified. Baseline 66014 -> 66025 words. Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013jQvaonym9V1wKKfAEPmRr
…k the scaffolded Build type as a default (#729) Review findings: Q16 is the last heading in intake.md and only its own text said 'ask first', so a lead reading top-down would walk Q1-Q15 before it; the scaffolded 'Build type: full' read as an answered question. The intake preamble now names Q16 as asked first at project birth, the PROJECT.md comment says the line is only the default until answered, and the skill's status block lists the kickoff question as not yet run by a fresh agent. Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013jQvaonym9V1wKKfAEPmRr
|
Added since the last note (head
Files beyond the skill: Review at Generated by Claude Code |
|
Duplicate of the note above: the first attempt to post it timed out and landed late, after the retry. Nothing new here. Generated by Claude Code |
…sign message, re-seed without a command, page pre-flight precedence, second proof build (#729) Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013jQvaonym9V1wKKfAEPmRr
… by explicit paths (#729) Review findings: the question-moments line said the three design questions are asked 'if it is answered Nothing' although they go in the same message as the prototype question; the no-prototype path repeated the ambiguity; the commit rule now names the working form. One restated clause removed. Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013jQvaonym9V1wKKfAEPmRr
…pproval is one more stop (#729) Review finding: 'exactly one stop, the approval' conflicted with step 1's 'ONE message, then stop'. Both lines now say the same thing: one message of questions, wait for the reply, then one more stop for approval. Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013jQvaonym9V1wKKfAEPmRr
CHANGELOG.md, ROUTING.md and bin/lib/skill-routing.tsv: both rows kept; routing re-rendered. Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013jQvaonym9V1wKKfAEPmRr
Direct lane (changes to
skills/,bug-logs/,bin/,project-bin/,project-tests/, pipelines)What changed and why (one paragraph): Refs #729. New on-demand, experimental skill
skills/poc-build.md. A POC build is a build type (Build type: full | pocinPROJECT.md), chosen by the kickoff interview's first question (intake Q16, "Which kind of build?": Full build (Recommended) or POC build, rendered perinterview-protocol.md§3) and independent of the entry mode and ofPipeline mode:. It mirrors how Pipeline mode (intake Q15, #700) was added: the question lives inbin/lib/intake-template.sh, aBuild type: fullline is in the PROJECT.md scaffold, the Stage P row names it. No line meansfull;Interview mode: autonever pickspoc; an existing mid-build project is not asked. Flow: after the build-type answer ONE message with the design question ("Do you have a prototype or a design?", plus three short questions on the Nothing path); design artifacts first; ONE human stop on the clickable prototype; then model work (security and theme first, wireframe-built pages one at a time, looked at on 1280 and 390); a journey that covers the happy path and every refusal; a markdown hand-over so the POC can continue throughexisting-app-change.md. It names what a POC build skips and says to state that in the hand-over. Size is warn-only. Second proof build fixes in this push: the skill now tells the lead to write a fixedUnverified — how to verify: not asked, POC build …under the intake questions a POC build skips (the escape form intake.md already offers), so Stage P stops reporting FAIL; the RESUME starter says to ask Q16 first; re-seed without an invented command; page pre-flight precedence (skip Step 0 and Step 3 ofui-preflight-pages.md); one-line warnings on token naming and the bottom-barnav.tabs. No gate or script logic changed (one line of the RESUME starter text inbin/lib/resume-template.sh).Impact
gate-check.shon a scaffold with Q16 answered and Q1–Q15 carrying theUnverifiedline: Stage PPASS(markers present, says nothing about the answers), Stages 0–4, 6, 7PENDING, Stage 5MANUAL; with one question unmarked Stage P isFAIL. The skill tells the user this is expected and not to waive stages to turn it green.bin/lib/intake-template.sh(Q16, appended; Q9 not renumbered),bin/init-project.sh(two PROJECT.md scaffold lines),bin/lib/resume-template.sh(step 2 of the RESUME starter),skills/conversion-runbook.md(Stage P row),skills/poc-build.md,bin/lib/skill-routing.tsv(one on-demand row) and its renders (README.md,ROUTING.md, four agent stubs),CHANGELOG.md. No ordering constraint with open PRs. Producer ofBuild type:: the kickoff answer (Q16). Fixtures grepping the changed lines (test-stage-p.sh,test-bug12-sync.shT10): read, unaffected; none greps the RESUME starter text; not run.bin/render-routing.sh --check; unchanged by the second push.poc-build.mdis on demand (about 2,020 words).Field evidence — fictional test apps only, Mendix 11.13, mxcli v0.28, 2026-11-10/12:
Follow-ups, not in this PR: (B)
assemble-prototype.jshides any class startingwf-; (C) friendlier defaults incheck-design-reaches-app.sh(token names that differ from the theme's) andcheck-page-shell.sh(a phone bottom barnav.tabsread as a top bar); (D) the page-shell/class rules ofui-preflight-pages.mdsplit from its StyleGallery steps; (E) session-start steps that assume a chat and a writable toolkit clone (thegit pullritual, "state in chat which commit"); (F) a hand-over block forbrd-generation; (G) a few mxcli facts for the learned-rules files; (H) the project wiring lists./bin/exec.sh <script>without--commit-first, and does not mentionFINDINGS.md/brain capturefor a POC build.render-routing.sh --check,check-scripts.sh,check-no-private-citations.sh,mdl-rules.sh --check,check-portability.sh) plus two driven builds, an independent grade and a blind comparisonbin/lib/skill-routing.tsv+bin/render-routing.sh)CHANGELOG.mdline appended in this PR, crediting the sourceGenerated by Claude Code