theslopmachine 1.0.2 → 1.0.3

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (39) hide show
  1. package/assets/agents/developer.md +38 -32
  2. package/assets/agents/slopmachine-claude.md +36 -25
  3. package/assets/agents/slopmachine.md +61 -45
  4. package/assets/claude/agents/developer.md +27 -10
  5. package/assets/skills/claude-worker-management/SKILL.md +4 -4
  6. package/assets/skills/developer-session-lifecycle/SKILL.md +13 -3
  7. package/assets/skills/development-guidance/SKILL.md +24 -5
  8. package/assets/skills/evaluation-triage/SKILL.md +4 -4
  9. package/assets/skills/final-evaluation-orchestration/SKILL.md +29 -3
  10. package/assets/skills/integrated-verification/SKILL.md +24 -23
  11. package/assets/skills/p8-readiness-reconciliation/SKILL.md +98 -0
  12. package/assets/skills/planning-gate/SKILL.md +2 -2
  13. package/assets/skills/planning-guidance/SKILL.md +7 -4
  14. package/assets/skills/scaffold-guidance/SKILL.md +2 -0
  15. package/assets/skills/submission-packaging/SKILL.md +30 -3
  16. package/assets/skills/verification-gates/SKILL.md +11 -7
  17. package/assets/slopmachine/clarification-faithfulness-review-prompt.md +69 -45
  18. package/assets/slopmachine/clarifier-agent-prompt.md +46 -40
  19. package/assets/slopmachine/exact-readme-template.md +38 -11
  20. package/assets/slopmachine/owner-verification-checklist.md +2 -2
  21. package/assets/slopmachine/phase-1-design-prompt.md +94 -17
  22. package/assets/slopmachine/phase-1-design-template.md +124 -21
  23. package/assets/slopmachine/phase-2-execution-planning-prompt.md +155 -87
  24. package/assets/slopmachine/phase-2-plan-template.md +169 -81
  25. package/assets/slopmachine/scaffold-playbooks/selection-matrix.md +8 -1
  26. package/assets/slopmachine/scaffold-playbooks/tech-frontend-vue.md +2 -0
  27. package/assets/slopmachine/scaffold-playbooks/type-web-spa.md +1 -0
  28. package/assets/slopmachine/templates/AGENTS.md +18 -17
  29. package/assets/slopmachine/templates/CLAUDE.md +18 -17
  30. package/assets/slopmachine/templates/plan.md +115 -36
  31. package/package.json +9 -2
  32. package/src/constants.js +1 -0
  33. package/src/init.js +8 -0
  34. package/src/install.js +130 -0
  35. package/assets/slopmachine/utils/__pycache__/claude_live_hook.cpython-311.pyc +0 -0
  36. package/assets/slopmachine/utils/__pycache__/cleanup_delivery_artifacts.cpython-311.pyc +0 -0
  37. package/assets/slopmachine/utils/__pycache__/convert_ai_session.cpython-311.pyc +0 -0
  38. package/assets/slopmachine/utils/__pycache__/normalize_claude_session.cpython-311.pyc +0 -0
  39. package/assets/slopmachine/utils/__pycache__/strip_session_parent.cpython-311.pyc +0 -0
@@ -17,7 +17,7 @@ This library replaces the old framework-default playbooks. It is organized by in
17
17
  - `type-mobile-android.md`
18
18
  - `type-desktop.md`
19
19
  4. Select independent tech modules for the exact stack pieces:
20
- - frontend: `tech-frontend-vue.md`, `tech-frontend-react.md`
20
+ - frontend: `tech-frontend-vue.md`, `tech-frontend-react.md`
21
21
  - backend: `tech-backend-go.md`, `tech-backend-koa.md`, `tech-backend-laravel.md`, `tech-backend-gin-templ.md`
22
22
  - database: `tech-db-mysql.md`, `tech-db-postgres.md`, `tech-db-room.md`, `tech-db-localdb.md`
23
23
  - language/workspace: `tech-rust-workspace.md`
@@ -33,6 +33,13 @@ This library replaces the old framework-default playbooks. It is organized by in
33
33
  6. If no full-stack profile matches, use `stack-generic.md` plus the selected type and tech modules.
34
34
  7. Restate the relevant directives in the developer prompt; do not ask the developer to read files outside `repo/`.
35
35
 
36
+ ## Default frontend stack policy
37
+
38
+ - If the prompt and adopted repo do not specify a frontend JavaScript framework for a web/frontend surface, default to Vue 3 with Vite and TypeScript and include `tech-frontend-vue.md`.
39
+ - If the prompt and adopted repo do not specify a styling library for a web/frontend surface, default to Tailwind CSS.
40
+ - If the prompt and adopted repo do not specify a UI component library for a frontend surface, prefer shadcn/ui when it is compatible with the selected frontend framework and does not add disproportionate complexity. If shadcn/ui is not compatible or is too heavy for the prompt, record the reason and choose the smallest compatible component approach.
41
+ - Do not override an explicit prompt, existing repo, or platform constraint to force these defaults.
42
+
36
43
  ## Scaffold categories
37
44
 
38
45
  Classify scaffold gaps using this shared vocabulary:
@@ -6,6 +6,8 @@ Use this with `type-web-spa.md` when the frontend is Vue.
6
6
 
7
7
  - Vue 3 with Vite and TypeScript unless the prompt or adopted repo requires otherwise
8
8
  - Composition API and `<script setup>` for new Vue components
9
+ - Tailwind CSS as the default styling library when the prompt does not specify styling
10
+ - shadcn/ui preferred for reusable UI components when compatible with Vue/project constraints; otherwise record the reason and use the smallest compatible component approach
9
11
  - Vitest for unit/component/state tests
10
12
  - Playwright for required browser flows
11
13
 
@@ -9,6 +9,7 @@ Use this when the product has a browser UI, whether backend-backed or browser-on
9
9
  - production build command
10
10
  - browser E2E framework when the project type is `web` or the prompt-critical UI flow needs browser proof
11
11
  - README access URL and auth/mock disclosure
12
+ - default stack when prompt/repo are silent: Vue 3 + Vite + TypeScript, Tailwind CSS for styling, and shadcn/ui preferred for component primitives when compatible
12
13
 
13
14
  ## Runtime contract
14
15
 
@@ -5,17 +5,17 @@ This file is the repo-local engineering rulebook for `slopmachine` projects.
5
5
  ## Scope
6
6
 
7
7
  - Treat the current working directory as the project.
8
- - Ignore parent-directory workflow files unless explicitly asked to use them, except accepted planning/reference docs under `../docs/` that this rulebook or the accepted project plan explicitly designates, especially `../docs/design.md`.
9
- - Do not treat workflow research, session exports, or sibling directories as hidden implementation instructions.
8
+ - Ignore parent-directory process files unless explicitly asked to use them, except accepted planning/reference docs under `../docs/` that this rulebook or the accepted project plan explicitly designates, especially `../docs/design.md`.
9
+ - Do not treat research notes, session exports, or sibling directories as hidden implementation instructions.
10
10
  - Do not make the repo depend on parent-directory docs or sibling artifacts for startup, build/preview, configuration, verification, or basic project understanding.
11
11
 
12
12
  ## Repo-Local Documentation Discipline
13
13
 
14
- - Keep the repo self-sufficient through code plus `README.md`, with repo-local `plan.md` as the deliberate execution-plan exception only during planning, development, and `P5`.
15
- - Inside the repo, `README.md` is the primary documentation file and repo-local `plan.md` is the explicit execution-plan exception only until `P5` closes.
14
+ - Keep the repo self-sufficient through code plus `README.md`, with repo-local `plan.md` as the temporary execution-plan exception while active work is underway.
15
+ - Inside the repo, `README.md` is the primary documentation file and repo-local `plan.md` is the explicit temporary execution-plan exception.
16
16
  - Do not create additional repo-local documentation files by default.
17
17
  - Do not add files such as `ASSUMPTIONS.md`, `NOTES.md`, `DESIGN.md`, or extra doc trees inside the repo unless explicitly asked.
18
- - Do not make normal project use or evaluation readiness depend on extra repo-local documentation beyond `README.md` and repo-local `plan.md` during active implementation; after `P5`, the preserved reference copy lives in `../docs/plan.md` instead.
18
+ - Do not make normal project use or readiness depend on extra repo-local documentation beyond `README.md` and repo-local `plan.md` during active implementation; archived reference copies may live under `../docs/` when the owner provides them.
19
19
 
20
20
  ## Working Style
21
21
 
@@ -23,14 +23,14 @@ This file is the repo-local engineering rulebook for `slopmachine` projects.
23
23
  - Read the code before making assumptions.
24
24
  - Work in meaningful end-to-end workstreams.
25
25
  - Do not call work complete while it is still shaky.
26
- - Treat `plan.md` as the definitive repo-local execution plan once planning is accepted and until `P5` closes.
27
- - Use `../docs/design.md` for system design and architecture, and use repo-local `plan.md` for module packet execution order, file/location ownership details where needed, and completion tracking until `P5` closes.
26
+ - Treat `plan.md` as the definitive repo-local execution plan once planning is accepted.
27
+ - Use `../docs/design.md` for system design and architecture, and use repo-local `plan.md` for module packet execution order, file/location ownership details where needed, and completion tracking while active work is underway.
28
28
  - At the start of implementation, treat section 3 of the accepted `plan.md` as binding; do not re-choose the playbook, starter, or bootstrap path unless planning is explicitly reopened.
29
29
  - During development, treat the security execution contract, `Core Semantic Path Proof`, `Prompt-Critical Rule Matrix`, `Role Surface Matrix`, `Runtime Lifecycle Checklist`, `Delivery Review Requirements`, `README Contract`, and the test coverage execution contract in `plan.md` as binding; do not leave security, release-readiness obligations, documentation, or tests as cleanup if the plan attached them to the current surface.
30
30
  - For fullstack or backend-backed frontend projects, treat the FE↔BE Integration Map in `plan.md` as binding; frontend surfaces must perform real backend-backed operations, and prompt-relevant backend features must be exposed through required frontend surfaces unless the accepted plan says they are internal/API-only.
31
31
  - Update `plan.md` in place from the main developer lane as implementation items are completed.
32
32
  - If work is interrupted, resume from the current state of `plan.md` instead of inventing a new hidden plan.
33
- - Once `P5` closes, the owner preserves the final truthful execution record in `../docs/plan.md` and removes repo-local `plan.md`; do not make later evaluation or packaging depend on a repo copy still existing.
33
+ - If the owner later archives the execution record under `../docs/plan.md` and removes repo-local `plan.md`, continue using `README.md`, code, tests, scripts, and config as the repo-local source of delivery truth.
34
34
  - Respect the module packet order, file/location ownership details, and shared-file contract from `plan.md`.
35
35
  - When `plan.md` assigns a shared security foundation, finish that foundation before broader module work begins.
36
36
  - When a module packet owns a surface, it should also own the matching tests and coverage work for that surface unless `plan.md` explicitly centralizes shared harness work first.
@@ -50,9 +50,9 @@ This file is the repo-local engineering rulebook for `slopmachine` projects.
50
50
  - Reuse and extend shared cross-cutting patterns instead of inventing incompatible local ones.
51
51
  - Before coding, identify the actors or personas touched by the change and the concrete path to success for each one.
52
52
  - Make important business rules explicit before coding: defaults, limits, allowed transitions, uniqueness, conflicts, reversals, retries, and ownership rules when they matter.
53
- - When the product has meaningful workflow state, define or confirm the relevant state machine before treating the flow as implemented.
53
+ - When the product has meaningful lifecycle state, define or confirm the relevant state machine before treating the flow as implemented.
54
54
  - Keep a concrete out-of-scope boundary in mind so you do not overbuild speculative features.
55
- - Do not introduce convenience-based `v1` scope cuts, role simplifications, or workflow omissions unless they were explicitly authorized.
55
+ - Do not introduce convenience-based `v1` scope cuts, role simplifications, or lifecycle omissions unless they were explicitly authorized.
56
56
 
57
57
  ## Phase Boundary Discipline
58
58
 
@@ -85,10 +85,10 @@ This file is the repo-local engineering rulebook for `slopmachine` projects.
85
85
  - Follow the original prompt and existing repository first for the runtime stack.
86
86
  - Prefer the fastest meaningful local verification for the changed area during ordinary iteration.
87
87
  - Do not rerun broad runtime/test commands on every small change.
88
- - From planning through the end of `P7`, never run Docker runtime commands.
89
- - Do not run `docker compose up --build` or any equivalent Docker-based runtime command under any circumstances during planning, development, or `P7`, even if the repo documents it, a playbook suggests it, `plan.md` seems to call for it, or the owner explicitly asks.
88
+ - During ordinary implementation, use the accepted local verification harness and targeted checks.
89
+ - Only run Docker-based runtime or broad dockerized test commands when the active instruction or accepted plan says this is the current verification step.
90
90
  - During ordinary implementation work, do not run browser E2E or Playwright by default.
91
- - Use targeted local tests during ordinary implementation work, and use the prepared local test harness before major readiness claims; the owner reruns that harness in `P5`, and Docker plus dockerized `./run_tests.sh` remain deferred to `P9`.
91
+ - Use targeted local tests during ordinary implementation work, and use the prepared local test harness before major readiness claims.
92
92
  - For web projects, require the runtime contract to be `docker compose up --build`.
93
93
  - For Android, mobile, desktop, and iOS-targeted projects, also require a meaningful `docker compose up --build` command even when platform-specific runtime proof differs from web semantics.
94
94
  - For non-web projects, `./run_app.sh` may exist as a helper wrapper, but it does not replace the required Docker command.
@@ -142,16 +142,17 @@ This file is the repo-local engineering rulebook for `slopmachine` projects.
142
142
  - The README must explain what the project is, what it does, how to run it, how to test it, the main repo contents, and any important information a new developer needs immediately.
143
143
  - The README must also explain the delivered architecture and major implementation structure clearly enough for review and handoff.
144
144
  - The README must include project type near the top, startup instructions, access method, verification method, and demo credentials for every role or the exact statement `No authentication required`.
145
+ - README.md must include a Configuration and Environment Model section or equivalent content explaining local configuration, Docker/Compose defaults, seeded/bootstrap data, auth/no-auth, and the absence of committed `.env` requirements.
145
146
  - The README must clearly document the required Docker command `docker compose up --build` and any additional helper runtime wrapper such as `./run_app.sh` when present.
146
147
  - The README must clearly document repo-root `./run_tests.sh` as the full-app broad test command.
147
148
  - For backend, fullstack, and web projects, the README should also contain the exact legacy compatibility string `docker-compose up` somewhere in startup guidance without replacing the canonical runtime contract.
148
149
  - The README must stand on its own for basic codebase use.
149
- - Keep `README.md` as the primary documentation file inside the repo; repo-local `plan.md` is the explicit execution-plan exception only through `P5`, after which the preserved plan record is `../docs/plan.md`.
150
+ - Keep `README.md` as the primary documentation file inside the repo; repo-local `plan.md` is the temporary execution-plan exception while active work is underway.
150
151
  - The repo should be statically reviewable by a fresh reviewer: entry points, routes, config, test commands, and major module boundaries should be traceable from repository artifacts.
151
152
  - The README should name the important actors, the main success paths, major limitations or out-of-scope boundaries, and any non-obvious business rules that affect usage.
152
- - Before reporting development complete, do one deliberate reread against the accepted `plan.md`, `../docs/design.md`, accepted `../docs/api-spec.md` when applicable, `README.md`, and the integrated repo so obvious contract drift is closed before `P5`.
153
+ - Before reporting development complete, do one deliberate reread against the accepted `plan.md`, `../docs/design.md`, accepted `../docs/api-spec.md` when applicable, `README.md`, and the integrated repo so obvious contract drift is closed before handoff.
153
154
  - Before reporting development complete, perform main-lane module-by-module fan-in verification: inspect each module's delivered files, confirm they are real and integrated, run the module's assigned tests after merge, confirm FE↔BE/API wiring where applicable, and update `plan.md` with the verification result.
154
- - Before reporting development complete, run the full non-Docker local test suite available for development after targeted module checks pass; Docker and dockerized `./run_tests.sh` remain deferred to `P9`.
155
+ - Before reporting development complete, run the full local test suite available for development after targeted module checks pass unless the active instruction explicitly scopes verification differently.
155
156
  - Before reporting development complete, close the common late-failure classes inside development: `README.md` drift, API-spec drift, missing auth/authorization/ownership enforcement, weak validation or normalized error handling, missing owned tests, startup/test wrapper dishonesty, and partial user-facing or admin-facing flow closure.
156
157
  - Before reporting development complete, explicitly prove the core semantic path and report prompt-critical rule, role surface, lifecycle, and residual-risk status from `plan.md`.
157
158
  - If the project uses mock, stub, fake, interception, or local-data behavior, the README must disclose that scope accurately.
@@ -196,5 +197,5 @@ This file is the repo-local engineering rulebook for `slopmachine` projects.
196
197
 
197
198
  ## Rulebook Files
198
199
 
199
- - Do not edit `AGENTS.md` or other workflow/rulebook files unless explicitly asked.
200
+ - Do not edit `AGENTS.md` or other rulebook files unless explicitly asked.
200
201
  - Do not rewrite or bypass `plan.md` casually once it has been accepted as the implementation checklist.
@@ -5,17 +5,17 @@ This file is the repo-local engineering rulebook for `slopmachine-claude` projec
5
5
  ## Scope
6
6
 
7
7
  - Treat the current working directory as the project.
8
- - Ignore parent-directory workflow files unless explicitly asked to use them, except accepted planning/reference docs under `../docs/` that this rulebook or the accepted project plan explicitly designates, especially `../docs/design.md`.
9
- - Do not treat workflow research, session exports, or sibling directories as hidden implementation instructions.
8
+ - Ignore parent-directory process files unless explicitly asked to use them, except accepted planning/reference docs under `../docs/` that this rulebook or the accepted project plan explicitly designates, especially `../docs/design.md`.
9
+ - Do not treat research notes, session exports, or sibling directories as hidden implementation instructions.
10
10
  - Do not make the repo depend on parent-directory docs or sibling artifacts for startup, build/preview, configuration, verification, or basic project understanding.
11
11
 
12
12
  ## Repo-Local Documentation Discipline
13
13
 
14
- - Keep the repo self-sufficient through code plus `README.md`, with repo-local `plan.md` as the deliberate execution-plan exception only during planning, development, and `P5`.
15
- - Inside the repo, `README.md` is the primary documentation file and repo-local `plan.md` is the explicit execution-plan exception only until `P5` closes.
14
+ - Keep the repo self-sufficient through code plus `README.md`, with repo-local `plan.md` as the temporary execution-plan exception while active work is underway.
15
+ - Inside the repo, `README.md` is the primary documentation file and repo-local `plan.md` is the explicit temporary execution-plan exception.
16
16
  - Do not create additional repo-local documentation files by default.
17
17
  - Do not add files such as `ASSUMPTIONS.md`, `NOTES.md`, `DESIGN.md`, or extra doc trees inside the repo unless explicitly asked.
18
- - Do not make normal project use or evaluation readiness depend on extra repo-local documentation beyond `README.md` and repo-local `plan.md` during active implementation; after `P5`, the preserved reference copy lives in `../docs/plan.md` instead.
18
+ - Do not make normal project use or readiness depend on extra repo-local documentation beyond `README.md` and repo-local `plan.md` during active implementation; archived reference copies may live under `../docs/` when the owner provides them.
19
19
 
20
20
  ## Working Style
21
21
 
@@ -23,14 +23,14 @@ This file is the repo-local engineering rulebook for `slopmachine-claude` projec
23
23
  - Read the code before making assumptions.
24
24
  - Work in meaningful end-to-end workstreams.
25
25
  - Do not call work complete while it is still shaky.
26
- - Treat `plan.md` as the definitive repo-local execution plan once planning is accepted and until `P5` closes.
27
- - Use `../docs/design.md` for system design and architecture, and use repo-local `plan.md` for module packet execution order, file/location ownership details where needed, and completion tracking until `P5` closes.
26
+ - Treat `plan.md` as the definitive repo-local execution plan once planning is accepted.
27
+ - Use `../docs/design.md` for system design and architecture, and use repo-local `plan.md` for module packet execution order, file/location ownership details where needed, and completion tracking while active work is underway.
28
28
  - At the start of implementation, treat section 3 of the accepted `plan.md` as binding; do not re-choose the playbook, starter, or bootstrap path unless planning is explicitly reopened.
29
29
  - During development, treat the security execution contract, `Core Semantic Path Proof`, `Prompt-Critical Rule Matrix`, `Role Surface Matrix`, `Runtime Lifecycle Checklist`, `Delivery Review Requirements`, `README Contract`, and the test coverage execution contract in `plan.md` as binding; do not leave security, release-readiness obligations, documentation, or tests as cleanup if the plan attached them to the current surface.
30
30
  - For fullstack or backend-backed frontend projects, treat the FE↔BE Integration Map in `plan.md` as binding; frontend surfaces must perform real backend-backed operations, and prompt-relevant backend features must be exposed through required frontend surfaces unless the accepted plan says they are internal/API-only.
31
31
  - Update `plan.md` in place from the main developer lane as implementation items are completed.
32
32
  - If work is interrupted, resume from the current state of `plan.md` instead of inventing a new hidden plan.
33
- - Once `P5` closes, the owner preserves the final truthful execution record in `../docs/plan.md` and removes repo-local `plan.md`; do not make later evaluation or packaging depend on a repo copy still existing.
33
+ - If the owner later archives the execution record under `../docs/plan.md` and removes repo-local `plan.md`, continue using `README.md`, code, tests, scripts, and config as the repo-local source of delivery truth.
34
34
  - Respect the module packet order, file/location ownership details, and shared-file contract from `plan.md`.
35
35
  - When `plan.md` assigns a shared security foundation, finish that foundation before broader module work begins.
36
36
  - When a module packet owns a surface, it should also own the matching tests and coverage work for that surface unless `plan.md` explicitly centralizes shared harness work first.
@@ -49,9 +49,9 @@ This file is the repo-local engineering rulebook for `slopmachine-claude` projec
49
49
  - Reuse and extend shared cross-cutting patterns instead of inventing incompatible local ones.
50
50
  - Before coding, identify the actors or personas touched by the change and the concrete path to success for each one.
51
51
  - Make important business rules explicit before coding: defaults, limits, allowed transitions, uniqueness, conflicts, reversals, retries, and ownership rules when they matter.
52
- - When the product has meaningful workflow state, define or confirm the relevant state machine before treating the flow as implemented.
52
+ - When the product has meaningful lifecycle state, define or confirm the relevant state machine before treating the flow as implemented.
53
53
  - Keep a concrete out-of-scope boundary in mind so you do not overbuild speculative features.
54
- - Do not introduce convenience-based `v1` scope cuts, role simplifications, or workflow omissions unless they were explicitly authorized.
54
+ - Do not introduce convenience-based `v1` scope cuts, role simplifications, or lifecycle omissions unless they were explicitly authorized.
55
55
 
56
56
  ## Phase Boundary Discipline
57
57
 
@@ -84,10 +84,10 @@ This file is the repo-local engineering rulebook for `slopmachine-claude` projec
84
84
  - Follow the original prompt and existing repository first for the runtime stack.
85
85
  - Prefer the fastest meaningful local verification for the changed area during ordinary iteration.
86
86
  - Do not rerun broad runtime/test commands on every small change.
87
- - From planning through the end of `P7`, never run Docker runtime commands.
88
- - Do not run `docker compose up --build` or any equivalent Docker-based runtime command under any circumstances during planning, development, or `P7`, even if the repo documents it, a playbook suggests it, `plan.md` seems to call for it, or the owner explicitly asks.
87
+ - During ordinary implementation, use the accepted local verification harness and targeted checks.
88
+ - Only run Docker-based runtime or broad dockerized test commands when the active instruction or accepted plan says this is the current verification step.
89
89
  - During ordinary implementation work, do not run browser E2E or Playwright by default.
90
- - Use targeted local tests during ordinary implementation work, and use the prepared local test harness before major readiness claims; the owner reruns that harness in `P5`, and Docker plus dockerized `./run_tests.sh` remain deferred to `P9`.
90
+ - Use targeted local tests during ordinary implementation work, and use the prepared local test harness before major readiness claims.
91
91
  - For web projects, require the runtime contract to be `docker compose up --build`.
92
92
  - For Android, mobile, desktop, and iOS-targeted projects, also require a meaningful `docker compose up --build` command even when platform-specific runtime proof differs from web semantics.
93
93
  - For non-web projects, `./run_app.sh` may exist as a helper wrapper, but it does not replace the required Docker command.
@@ -141,16 +141,17 @@ This file is the repo-local engineering rulebook for `slopmachine-claude` projec
141
141
  - The README must explain what the project is, what it does, how to run it, how to test it, the main repo contents, and any important information a new developer needs immediately.
142
142
  - The README must also explain the delivered architecture and major implementation structure clearly enough for review and handoff.
143
143
  - The README must include project type near the top, startup instructions, access method, verification method, and demo credentials for every role or the exact statement `No authentication required`.
144
+ - README.md must include a Configuration and Environment Model section or equivalent content explaining local configuration, Docker/Compose defaults, seeded/bootstrap data, auth/no-auth, and the absence of committed `.env` requirements.
144
145
  - The README must clearly document the required Docker command `docker compose up --build` and any additional helper runtime wrapper such as `./run_app.sh` when present.
145
146
  - The README must clearly document repo-root `./run_tests.sh` as the full-app broad test command.
146
147
  - For backend, fullstack, and web projects, the README should also contain the exact legacy compatibility string `docker-compose up` somewhere in startup guidance without replacing the canonical runtime contract.
147
148
  - The README must stand on its own for basic codebase use.
148
- - Keep `README.md` as the primary documentation file inside the repo; repo-local `plan.md` is the explicit execution-plan exception only through `P5`, after which the preserved plan record is `../docs/plan.md`.
149
+ - Keep `README.md` as the primary documentation file inside the repo; repo-local `plan.md` is the temporary execution-plan exception while active work is underway.
149
150
  - The repo should be statically reviewable by a fresh reviewer: entry points, routes, config, test commands, and major module boundaries should be traceable from repository artifacts.
150
151
  - The README should name the important actors, the main success paths, major limitations or out-of-scope boundaries, and any non-obvious business rules that affect usage.
151
- - Before reporting development complete, do one deliberate reread against the accepted `plan.md`, `../docs/design.md`, accepted `../docs/api-spec.md` when applicable, `README.md`, and the integrated repo so obvious contract drift is closed before `P5`.
152
+ - Before reporting development complete, do one deliberate reread against the accepted `plan.md`, `../docs/design.md`, accepted `../docs/api-spec.md` when applicable, `README.md`, and the integrated repo so obvious contract drift is closed before handoff.
152
153
  - Before reporting development complete, perform main-lane module-by-module fan-in verification: inspect each module's delivered files, confirm they are real and integrated, run the module's assigned tests after merge, confirm FE↔BE/API wiring where applicable, and update `plan.md` with the verification result.
153
- - Before reporting development complete, run the full non-Docker local test suite available for development after targeted module checks pass; Docker and dockerized `./run_tests.sh` remain deferred to `P9`.
154
+ - Before reporting development complete, run the full local test suite available for development after targeted module checks pass unless the active instruction explicitly scopes verification differently.
154
155
  - Before reporting development complete, close the common late-failure classes inside development: `README.md` drift, API-spec drift, missing auth/authorization/ownership enforcement, weak validation or normalized error handling, missing owned tests, startup/test wrapper dishonesty, and partial user-facing or admin-facing flow closure.
155
156
  - Before reporting development complete, explicitly prove the core semantic path and report prompt-critical rule, role surface, lifecycle, and residual-risk status from `plan.md`.
156
157
  - If the project uses mock, stub, fake, interception, or local-data behavior, the README must disclose that scope accurately.
@@ -195,5 +196,5 @@ This file is the repo-local engineering rulebook for `slopmachine-claude` projec
195
196
 
196
197
  ## Rulebook Files
197
198
 
198
- - Do not edit `CLAUDE.md` or other workflow/rulebook files unless explicitly asked.
199
+ - Do not edit `CLAUDE.md` or other rulebook files unless explicitly asked.
199
200
  - Do not rewrite or bypass `plan.md` casually once it has been accepted as the implementation checklist.
@@ -32,7 +32,8 @@ This file is the authoritative execution checklist inside the repo.
32
32
  - Tests travel with implementation
33
33
  - Shared files are main-lane controlled
34
34
  - Runtime/test must remain honest and portable
35
- - Final integrated verification and hardening is mandatory
35
+ - Final integrated local verification and hardening is mandatory
36
+ - External planning documents under `../docs/` are planning/reference inputs only. Delivery evidence must exist inside the repo through `README.md`, code structure, route/app/server registration, tests, scripts, config/manifests, Docker/runtime files, and seeded/bootstrap paths where applicable. Do not rely on parent-directory docs as the only evidence for any final delivery, runtime, README, API, frontend, security, or test obligation.
36
37
 
37
38
  ### 1.3 Handoff exceptions
38
39
  | Exception | Why it exists | Impact | Required owner decision |
@@ -49,13 +50,17 @@ This file is the authoritative execution checklist inside the repo.
49
50
  - Shared-foundation-first rule:
50
51
  - Parallelization rule:
51
52
  - Merge discipline rule:
52
- - Final convergence rule: `P5` is one minimal gate that includes the owner-run local test harness. If that passes and the repo is roughly coherent with `plan.md`, stop and ask whether to proceed to evaluation. Fix only owner-side local-harness/config/wrapper/README/docs/light-script churn there, reroute any real code or actual test-file work back to the developer, and defer Docker/runtime plus dockerized `./run_tests.sh` to final packaging confirmation.
53
+ - Final convergence rule: once implementation is complete, run the accepted local harness, reconcile the repo against `plan.md`, fix narrow docs/config/wrapper/light-script issues directly when assigned, route real code or test-file fixes through the active correction workstream, and leave Docker/runtime plus dockerized `./run_tests.sh` for the explicit final runtime confirmation step.
54
+
55
+ If your active instruction asks only for implementation readiness, report readiness after integrated local verification and a truthful handoff.
56
+
57
+ If an implementation agent is explicitly asked to complete the product end to end in one autonomous execution run, it must not stop at the rough local gate. It must continue through all implementation, assigned tests, README/runtime/script reconciliation, integrated local verification, and final truthful handoff unless the active instruction explicitly says to stop earlier.
53
58
 
54
59
  ---
55
60
 
56
61
  ## 3. Scaffold Step at the Start of Development
57
62
 
58
- This section is planned during `P2` and then executed as the first step of development. It is not a separate lifecycle phase or owner gate.
63
+ This section is planned before implementation and then executed as the first step of development. It is not a separate product feature.
59
64
 
60
65
  This scaffold step should be strict and straightforward. Its job is only to establish the minimal honest delivery baseline before real feature development starts.
61
66
 
@@ -64,10 +69,12 @@ This scaffold step should be strict and straightforward. Its job is only to esta
64
69
  - Exact bootstrap command:
65
70
  - Expected directories/files immediately after bootstrap:
66
71
  - Language/framework bootstrap notes to preserve:
72
+ - Frontend defaults applied if prompt/repo are silent: Vue 3 + Vite + TypeScript, Tailwind CSS, and shadcn/ui when compatible:
67
73
  - Minimal bootstrap proof surface to land now (for example a hello-world route/page/app shell):
68
74
  - Docker/runtime setup required in this step so the owner can later run `docker compose up --build` during final packaging confirmation:
75
+ - Seeded quick-start data strategy required in this step, or `Not Applicable` with reason:
69
76
  - `./run_tests.sh` setup required in this step as the dockerized broad-test path reserved for final packaging confirmation:
70
- - Local test harness setup required in this step so development and owner-side `P5` can use it:
77
+ - Local test harness setup required in this step so implementation and later local readiness checks can use it:
71
78
  This should name the stack-native local suite explicitly, for example Vitest, Jest, PHPUnit, pytest, go test, cargo test, or another framework-native equivalent.
72
79
  - Reliability rules the local test harness and deferred Docker/runtime plus dockerized broad-test path must satisfy later:
73
80
  Include what must be installed/configured locally if the stack-native local suite is not already available on the machine.
@@ -82,10 +89,11 @@ Include what must be installed/configured locally if the stack-native local suit
82
89
 
83
90
  - bootstrap the project for the selected language/framework/runtime with only a minimal hello-world-style proof surface
84
91
  - set up the real Docker/runtime path required by the project so the owner can run `docker compose up --build` later during final packaging confirmation
92
+ - set up deterministic, idempotent seeded quick-start data through the normal bootstrap/database/runtime path when the product is not useful from an empty state
85
93
  - set up repo-root `./run_tests.sh` as the real dockerized broad-test entrypoint for the later final packaging confirmation run
86
94
  - make the local test harness and deferred Docker/runtime plus dockerized broad-test path reviewably reliable: no hidden setup steps, no manual exports for Docker, no hidden shell state for local tests, deterministic execution, and useful failure output
87
95
  - do not run Docker or dockerized `./run_tests.sh` during the scaffold step; prepare the local harness for immediate development use
88
- - set up the local testing harness/tooling used for ordinary development and owner-side `P5`
96
+ - set up the local testing harness/tooling used for ordinary development and later local readiness checks
89
97
  - set up `README.md` to the exact required template/section shape
90
98
  - keep the scaffold free of prompt-specific business logic, deep domain implementation, or polish work beyond the baseline needed to prove startup and test wiring
91
99
  - require the scaffold to avoid local dependency setup beyond Docker and the documented host minimums
@@ -127,14 +135,17 @@ Required for any role-sensitive project. If not applicable, state `Not Applicabl
127
135
  |---|---|---|---|---|---|---|---|---|---|
128
136
  | | | | | | | | | | |
129
137
 
130
- Any contradiction between this matrix and `../docs/design.md`, `../docs/api-spec.md`, router/frontend behavior, backend checks, README, or tests is a planning defect before development and a release-alignment defect before evaluation.
138
+ Any contradiction between this matrix and `../docs/design.md`, `../docs/api-spec.md`, router/frontend behavior, backend checks, README, or tests is a planning defect before development and a release-alignment defect before handoff.
131
139
 
132
140
  ---
133
141
 
134
142
  ## 5. Test Coverage Execution Contract
135
143
 
136
144
  ### 5.1 Coverage target and floor
137
- - Coverage target: meaningful mapped tests for every evaluator-relevant endpoint, core flow, security boundary, and major failure path
145
+ - Coverage target: meaningful mapped tests for every delivery-relevant endpoint, core flow, security boundary, and major failure path
146
+ - Unit coverage floor: at least `90%` line/branch coverage for unit-testable product code where stack tooling can measure this honestly; otherwise name the exact measurement limitation and substitute a complete file/function/assertion ledger
147
+ - E2E/platform coverage floor: at least `90%` of planned prompt-critical E2E/platform flow rows complete before clean development completion; otherwise name the exact accepted exception and compensating proof
148
+ - API coverage floor when APIs exist: `100%` of documented prompt-relevant `METHOD + PATH` surfaces have true no-mock HTTP tests unless a per-endpoint narrow exception and compensating proof are recorded
138
149
  - Measurement path:
139
150
  - Confidence notes / expected weak spots if any:
140
151
 
@@ -170,7 +181,7 @@ Any contradiction between this matrix and `../docs/design.md`, `../docs/api-spec
170
181
  ### 5.5 Frontend coverage obligations
171
182
  If applicable:
172
183
  - frontend unit tests required? [yes/no]
173
- - Frontend unit tests verdict for later audit: [PRESENT | MISSING]
184
+ - Frontend unit tests status for later review: [PRESENT | MISSING]
174
185
  - frontend unit-test file-level evidence plan:
175
186
  - important frontend components/modules that must not remain untested:
176
187
  - component/state tests:
@@ -294,9 +305,10 @@ The final `README.md` must include these sections exactly:
294
305
  8. verification method
295
306
  9. testing
296
307
  10. authentication and roles
297
- 11. workflow / operational notes if applicable
298
- 12. feature-flag/mock/debug/demo disclosure if applicable
299
- 13. important limitations or non-goals if material
308
+ 11. quick-start seeded data
309
+ 12. lifecycle / operational notes if applicable
310
+ 13. feature-flag/mock/debug/demo disclosure if applicable
311
+ 14. important limitations or non-goals if material
300
312
 
301
313
  ### 6.1 Required command strings
302
314
  - `docker compose up --build`
@@ -309,6 +321,13 @@ Choose one:
309
321
  - credentials table for every relevant role
310
322
  - exact statement `No authentication required`
311
323
 
324
+ ### 6.2.1 Quick-start seeded data rule
325
+ Choose one:
326
+ - seeded accounts/sample records/fixture data table with values, roles, IDs, URLs, and steps needed to exercise the main flows quickly
327
+ - exact statement `No seeded data required; the app is useful from an empty state` with a short reason
328
+
329
+ Seeded data must be deterministic and idempotent. It must be created by the normal runtime/bootstrap/database path, not by hidden manual setup. It may provide local demo/test fixtures, but it must not replace real product behavior with fake success paths or static demo-only UI.
330
+
312
331
  ### 6.3 Ownership rule
313
332
  - Main-lane owner:
314
333
  - Parallel lanes may propose updates through:
@@ -395,12 +414,29 @@ Each module packet must carry this checklist into development. A module may not
395
414
  | | | | | | | planned |
396
415
 
397
416
  ### 9.6 Development exit module verification matrix
398
- Complete this before leaving development for owner-side integrated verification. Use the stack-native local suite and targeted module tests; Docker and dockerized `./run_tests.sh` remain deferred to final packaging confirmation.
417
+ Complete this before claiming development readiness. Use the stack-native local suite and targeted module tests; Docker and dockerized `./run_tests.sh` remain deferred to the explicit final runtime confirmation step.
399
418
 
400
419
  | Module | Files reviewed by main lane | Module tests run | FE↔BE wiring verified | API/route coverage verified | E2E/platform proof status | Integration issues found/fixed | Exit status |
401
420
  |---|---|---|---|---|---|---|---|
402
421
  | | | | | | | | pending |
403
422
 
423
+ ### 9.7 Plan-Row Execution Ledger
424
+ Every actionable plan row must appear here or in an equivalent section-specific checklist. This ledger is the source of truth for whether `plan.md` was actually executed, not merely referenced.
425
+
426
+ | Plan section / row | Required implementation evidence | Required verification evidence | Responsible module/work package | Status | Blocker / risk / exception |
427
+ |---|---|---|---|---|---|
428
+ | | | | | planned | |
429
+
430
+ Clean development completion requires every row to be `complete`, `not applicable`, or explicitly `risk accepted` with evidence. Rows left `planned`, `in progress`, `delegated without receiver`, or `unverified` block completion.
431
+
432
+ ### 9.8 Coverage Closure Ledger
433
+
434
+ | Coverage surface | Required floor | Measured / closed result | Evidence path | Exception / compensating proof | Status |
435
+ |---|---|---|---|---|---|
436
+ | API true no-mock HTTP coverage for documented prompt-relevant endpoints | 100% | | | | planned |
437
+ | Unit-testable product-code line/branch coverage | >=90% where measurable | | | | planned |
438
+ | Planned E2E/platform-critical flow coverage | >=90% closed rows | | | | planned |
439
+
404
440
  ---
405
441
 
406
442
  ## 9.7 In-Scope Domains / Modules Summary
@@ -471,10 +507,17 @@ Every accepted requirement, clarification, design no-orphan row, and API spec ro
471
507
  ### 13.1 Prompt-Critical Rule Matrix
472
508
  Every prompt-critical rule must have an owner and a proof path. Include dates, thresholds, limits, states, scheduling/timing, retries, security constraints, public/private boundaries, actor identity, and operator promises.
473
509
 
474
- | Source prompt phrase / accepted clarification | Normalized rule | Implementation owner | Test owner | README/doc implication | P5 proof expectation | Status |
510
+ | Source prompt phrase / accepted clarification | Normalized rule | Implementation owner | Test owner | README/doc implication | Local proof expectation | Status |
475
511
  |---|---|---|---|---|---|---|
476
512
  | | | | | | | planned |
477
513
 
514
+ ### 13.1.1 Prompt Verb Acceptance Matrix
515
+ Every action verb in the original prompt or accepted clarification that implies product behavior must have an acceptance row. Use this to prevent broad nouns such as `dashboard`, `lifecycle`, `approval`, `export`, or `sync` from hiding missing actor actions.
516
+
517
+ | Source phrase / verb | Actor | Starting state / fixture | Action performed | Required state change or side effect | Observable UI/API/artifact result | Negative or edge behavior | Proof path | Owning module |
518
+ |---|---|---|---|---|---|---|---|---|
519
+ | | | | | | | | | |
520
+
478
521
  ### 13.2 Scaffold Versus Product Completion Matrix
479
522
 
480
523
  Use this to prevent setup shells from being mistaken for delivered behavior. A row may move to `done` only when the product-facing path is wired to real behavior and tests.
@@ -491,7 +534,7 @@ Use this to prevent setup shells from being mistaken for delivered behavior. A r
491
534
 
492
535
  ## 14. State / Lifecycle Rules
493
536
 
494
- | Entity / workflow | States | Valid transitions | Invalid transitions | Required cleanup |
537
+ | Entity / lifecycle | States | Valid transitions | Invalid transitions | Required cleanup |
495
538
  |---|---|---|---|---|
496
539
  | | | | | |
497
540
 
@@ -563,12 +606,22 @@ Complete only when the product has background work, timed behavior, async jobs,
563
606
  - If yes, where it is called:
564
607
  - If no, why not applicable:
565
608
 
609
+ ### 18.4 Seeded quick-start data
610
+ - Seeded data applicable? [yes/no]
611
+ - If yes, seed entrypoint and when it runs:
612
+ - Idempotency rule:
613
+ - Seeded accounts / roles:
614
+ - Seeded sample records / IDs / URLs:
615
+ - Main flows the seeded data unlocks:
616
+ - README quick-start section owner:
617
+ - If no, why the app is useful from empty state:
618
+
566
619
  ---
567
620
 
568
621
  ## 19. Delivery Review Requirement Matrices
569
622
 
570
623
  ### 19.0 Core Semantic Path Proof
571
- This is the centerpiece path that proves the product is not only a shell. `P5` cannot close unless this path is explicitly proven or the user accepts a named residual risk.
624
+ This is the centerpiece path that proves the product is not only a shell. Do not claim implementation readiness unless this path is explicitly proven or the user accepts a named residual risk.
572
625
 
573
626
  | Field | Required value |
574
627
  |---|---|
@@ -580,10 +633,10 @@ This is the centerpiece path that proves the product is not only a shell. `P5` c
580
633
  | Expected failure-mode behavior | |
581
634
  | Test file(s) proving the path | |
582
635
  | Manual/runtime proof if automated proof is not feasible | |
583
- | P5 acceptance evidence expected | |
636
+ | Local readiness evidence expected | |
584
637
 
585
638
  ### 19.0.1 Prompt-critical proof ledger
586
- Use this ledger during development and `P5` to prevent planned-but-missing drift.
639
+ Use this ledger during development and readiness hardening to prevent planned-but-missing drift.
587
640
 
588
641
  | Planned claim | Required proof | Current proof location | Gap | Decision |
589
642
  |---|---|---|---|---|
@@ -606,7 +659,10 @@ Use this ledger during development and `P5` to prevent planned-but-missing drift
606
659
  | entry points visible | | | |
607
660
  | route registration visible | | | |
608
661
  | config path centralized | | | |
662
+ | README/docs/scripts/routes/config/examples/manifests/env examples statically consistent | | | |
609
663
  | module boundaries visible | | | |
664
+ | no excessive single-file implementation or fragmented snippets | | | |
665
+ | redundant/unnecessary files removed or justified | | | |
610
666
  | mock/local-data boundary visible | | | |
611
667
 
612
668
  ### 19.3 Runtime and broad-test contract obligations matrix
@@ -646,12 +702,16 @@ Use this ledger during development and `P5` to prevent planned-but-missing drift
646
702
  | Obligation | Planned repo evidence | Planned verification evidence | Owning lane |
647
703
  |---|---|---|---|
648
704
  | pages/routes/screens complete | | | |
705
+ | pages/routes/app shell connected | | | |
649
706
  | core interactions complete | | | |
707
+ | state/data flow organized and traceable | | | |
708
+ | service/adaptor/mock/storage boundaries clear | | | |
650
709
  | task closure real | | | |
651
710
  | UI states complete | | | |
652
711
  | validation/disclosure explicit | | | |
653
712
  | frontend test obligations explicit | | | |
654
713
  | static visual/interaction quality evidence planned | | | |
714
+ | pure-frontend mock/local data disclosed without implying backend integration where applicable | | | |
655
715
 
656
716
  ### 19.7 End-to-end and platform verification obligations matrix
657
717
  | Obligation | Planned repo evidence | Planned verification evidence | Owning lane |
@@ -671,9 +731,13 @@ Use this ledger during development and `P5` to prevent planned-but-missing drift
671
731
  | access method documented | | | |
672
732
  | verification method documented | | | |
673
733
  | credentials/no-auth disclosure correct | | | |
734
+ | quick-start seeded data or empty-state rationale documented | | | |
735
+ | Configuration and Environment Model explains local config, runtime defaults, Docker/Compose defaults, seeded/bootstrap data, auth/no-auth, no committed `.env` requirement, no manual package/runtime/database setup beyond documented host prerequisites, and config-sensitive verification | | static README/config check | |
674
736
  | no manual install/runtime setup dependency (`npm install`, `pip install`, `apt-get`, manual DB setup, `.env`, `.env.example`) | | static README/config check | |
675
737
  | mock/debug/demo disclosure honest | | | |
676
738
 
739
+ If no configuration is required, the README should state: `No manual environment configuration is required for local Docker startup. The delivered local runtime uses repo-controlled Docker/Compose defaults and bootstrap paths.`
740
+
677
741
  ---
678
742
 
679
743
  ## 20. Shared-File Contract
@@ -682,6 +746,13 @@ Use this ledger during development and `P5` to prevent planned-but-missing drift
682
746
  |---|---|---|---|---|
683
747
  | | | | | |
684
748
 
749
+ ### 20.0 Canonical Cross-Surface Contracts
750
+ Use this when data crosses boundaries between UI, API, jobs, imports, exports, notifications, files, reports, clipboard payloads, templates, or third-party adapters. If not applicable, state `Not Applicable`.
751
+
752
+ | Contract / payload | Surfaces using it | Canonical schema or fields | Validation / normalization rule | Producer | Consumer | Tests / proof | README or docs implication |
753
+ |---|---|---|---|---|---|---|---|
754
+ | | | | | | | | |
755
+
685
756
  ### 20.1 Shared foundation to land before module execution
686
757
  -
687
758
  -
@@ -731,6 +802,13 @@ Each module must produce this packet before the next module starts:
731
802
  - Shared integration required from main lane
732
803
  - Known risks or `none`
733
804
 
805
+ ### 21.1.1.1 Plan Section Closure Evidence
806
+ Every major completion claim must cite the accepted `plan.md` sections it closes. A development-complete report is incomplete if it only summarizes work without mapping back to section-addressable plan evidence.
807
+
808
+ | Plan section / matrix row | Closure evidence | Test or verification result | Residual risk or blocker | Decision |
809
+ |---|---|---|---|---|
810
+ | | | | | |
811
+
734
812
  ### 21.1.2 Module Integration Consumption Rule
735
813
 
736
814
  - Inspect each module completion checklist before moving to the next module
@@ -805,36 +883,37 @@ For `web` and `fullstack`, frontend unit/component/state tests are required unle
805
883
  - Platform honesty notes:
806
884
  - Rerun triggers after major merges:
807
885
 
808
- This section is about overall delivery verification planning. It must not imply that such proof runs before evaluation or before the owner-side pre-submission step.
886
+ This section is about overall delivery verification planning. It must not imply that every broad proof runs during ordinary implementation unless the active instruction asks for it.
809
887
 
810
888
  ---
811
889
 
812
890
  ## 26. Integrated Verification and Hardening Plan
813
891
 
814
- ### 27.1 Broad final proof
815
- - Static/non-Docker checks to use before evaluation:
816
- - Proceed-to-evaluation rule when those checks pass:
892
+ ### 26.1 Broad final proof
893
+ - Static/non-Docker checks to use before readiness handoff:
894
+ - Readiness handoff rule when those checks pass:
817
895
  - Major flow proof required:
818
896
  - Security/auth proof required:
819
897
  - Runtime/README contract proof required:
820
898
  - Coverage proof required:
821
- - Required internal `P5` issue-discovery loop: one evaluator subagent session; send the exact prepared evaluation packet once; run four additional same-session scans for additional prompt-fit/compliance, security, delivery, test-coverage, README/static-verifiability, mock/demo/fake-success, and frontend state/interaction issues not already reported; do not remediate between rounds; save five reports and extracted Blocker/High findings under `../.ai/p5-evaluation/`; consolidate, owner-analyze, route one developer remediation brief, verify fixes, then continue to the proceed-to-evaluation pause
822
- - Development-completion audit before `P5`: reread original prompt, approved requirements breakdown, accepted clarifications, accepted design, accepted API spec when applicable, accepted `plan.md`, `../docs/test-coverage.md`, `README.md`, code, tests, and current proof in one sweep; every module row must be marked complete, blocked, or explicitly risk-accepted before handoff
823
- - P3 Development Completion Report must include: modules completed, requirements closed, files/routes/tests changed, unit/API/integration/E2E/frontend-state commands run, FE↔BE/API/data proof, edge/failure/security proof, remaining risks, and exact handoff status for `P5`
899
+ - Independent readiness review expectation: after implementation, the repo must withstand a strict prompt-fit, security, delivery, test-coverage, README/static-verifiability, mock/demo/fake-success, and frontend state/interaction review. Fix all concrete Blocker/High-equivalent issues before handoff unless the user explicitly accepts the risk.
900
+ - Development-completion review: reread original prompt, approved requirements breakdown, accepted clarifications, accepted design, accepted API spec when applicable, accepted `plan.md`, `../docs/test-coverage.md`, `README.md`, code, tests, and current proof in one sweep; every module row must be marked complete, blocked, or explicitly risk-accepted before handoff
901
+ - Development Completion Report must include: modules completed, requirements closed, files/routes/tests changed, unit/API/integration/E2E/frontend-state commands run, FE↔BE/API/data proof, edge/failure/security proof, remaining risks, and exact readiness handoff status
902
+ - Development Completion Report must include Plan Section Closure Evidence for every major plan section or matrix row that claims completion
824
903
 
825
- ### 27.2 Narrow rerun strategy after discovered failures
826
- - Owner pass scope rule: every `P5` owner pass must reread the accepted design, accepted `../docs/api-spec.md` when applicable, `plan.md`, `README.md`, repo state, and current evidence in one full sweep rather than reducing follow-up passes to targeted sections only
904
+ ### 26.2 Narrow rerun strategy after discovered failures
905
+ - Review scope rule: each integrated readiness pass must reread the accepted design, accepted `../docs/api-spec.md` when applicable, `plan.md`, `README.md`, repo state, and current evidence in one full sweep rather than reducing follow-up passes to targeted sections only
827
906
  - Known failure class isolation rule:
828
907
  - Narrow rerun examples:
829
908
  - When to stop broad reruns:
830
909
  - Issue-batching rule for integrated verification:
831
910
  - Owner-side fixes allowed directly before reroute:
832
- - Remediation expectation: default to ordered fixes through the main developer lane; use helper work only for genuinely independent fixes where that is safer, and require per-bundle verification plus integrated proof before accepting completion
833
- - Maximum owner sweep expectation from `P5` before evaluation: 3
834
- - Maximum developer reroute expectation from `P5` before evaluation:
911
+ - Remediation expectation: default to ordered fixes through the active correction workstream, use helper work only for genuinely independent fixes where that is safer, and require per-bundle verification plus integrated proof before accepting completion
912
+ - Maximum integrated readiness sweep expectation before handoff: 3
913
+ - Maximum correction reroute expectation before handoff:
835
914
 
836
- ### 27.3 Optional UI/platform sanity checks before evaluation
837
- - When extra UI/platform sanity is actually needed before evaluation:
915
+ ### 26.3 Optional UI/platform sanity checks before handoff
916
+ - When extra UI/platform sanity is actually needed before handoff:
838
917
  - Screenshot review requirements if used:
839
918
  - Web/fullstack Playwright sanity if used:
840
919
  - Desktop Playwright Electron/Xvfb or equivalent sanity if used:
@@ -843,15 +922,15 @@ This section is about overall delivery verification planning. It must not imply
843
922
  - Visual regression notes if used:
844
923
  - Artifact storage notes if used:
845
924
 
846
- ### 27.4 Doc tightening after verification
925
+ ### 26.4 Doc tightening after verification
847
926
  - `README.md`:
848
927
  - `../docs/design.md` if applicable:
849
928
  - `../docs/api-spec.md` if applicable:
850
929
  - `../docs/test-coverage.md` if applicable:
851
930
 
852
- ### 27.5 Final close conditions
853
- - No major prompt, product, security, or runtime breakage remains obvious enough to make evaluation premature
854
- - Development-completion audit is done and the handoff to `P5` names exact passed checks, skipped/deferred checks, and residual risks
931
+ ### 26.5 Final close conditions
932
+ - No major prompt, product, security, or runtime breakage remains obvious enough to make handoff premature
933
+ - Development-completion review is done and the handoff names exact passed checks, skipped/deferred checks, and residual risks
855
934
  - Docker execution remains deferred until final packaging confirmation
856
935
 
857
936
  ---
@@ -863,7 +942,7 @@ This section is about overall delivery verification planning. It must not imply
863
942
  - Confirm mock/debug/demo disclosure:
864
943
  - Confirm shared-file coherence:
865
944
  - Confirm coverage docs and README match reality:
866
- - Confirm Docker files and dockerized `./run_tests.sh` are ready for final packaging confirmation, and confirm the separate local test harness is ready for ordinary development plus owner-side `P5`:
945
+ - Confirm Docker files and dockerized `./run_tests.sh` are ready for explicit final runtime confirmation, and confirm the separate local test harness is ready for ordinary development plus later readiness checks:
867
946
 
868
947
  ---
869
948