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.
- package/assets/agents/developer.md +38 -32
- package/assets/agents/slopmachine-claude.md +36 -25
- package/assets/agents/slopmachine.md +61 -45
- package/assets/claude/agents/developer.md +27 -10
- package/assets/skills/claude-worker-management/SKILL.md +4 -4
- package/assets/skills/developer-session-lifecycle/SKILL.md +13 -3
- package/assets/skills/development-guidance/SKILL.md +24 -5
- package/assets/skills/evaluation-triage/SKILL.md +4 -4
- package/assets/skills/final-evaluation-orchestration/SKILL.md +29 -3
- package/assets/skills/integrated-verification/SKILL.md +24 -23
- package/assets/skills/p8-readiness-reconciliation/SKILL.md +98 -0
- package/assets/skills/planning-gate/SKILL.md +2 -2
- package/assets/skills/planning-guidance/SKILL.md +7 -4
- package/assets/skills/scaffold-guidance/SKILL.md +2 -0
- package/assets/skills/submission-packaging/SKILL.md +30 -3
- package/assets/skills/verification-gates/SKILL.md +11 -7
- package/assets/slopmachine/clarification-faithfulness-review-prompt.md +69 -45
- package/assets/slopmachine/clarifier-agent-prompt.md +46 -40
- package/assets/slopmachine/exact-readme-template.md +38 -11
- package/assets/slopmachine/owner-verification-checklist.md +2 -2
- package/assets/slopmachine/phase-1-design-prompt.md +94 -17
- package/assets/slopmachine/phase-1-design-template.md +124 -21
- package/assets/slopmachine/phase-2-execution-planning-prompt.md +155 -87
- package/assets/slopmachine/phase-2-plan-template.md +169 -81
- package/assets/slopmachine/scaffold-playbooks/selection-matrix.md +8 -1
- package/assets/slopmachine/scaffold-playbooks/tech-frontend-vue.md +2 -0
- package/assets/slopmachine/scaffold-playbooks/type-web-spa.md +1 -0
- package/assets/slopmachine/templates/AGENTS.md +18 -17
- package/assets/slopmachine/templates/CLAUDE.md +18 -17
- package/assets/slopmachine/templates/plan.md +115 -36
- package/package.json +9 -2
- package/src/constants.js +1 -0
- package/src/init.js +8 -0
- package/src/install.js +130 -0
- package/assets/slopmachine/utils/__pycache__/claude_live_hook.cpython-311.pyc +0 -0
- package/assets/slopmachine/utils/__pycache__/cleanup_delivery_artifacts.cpython-311.pyc +0 -0
- package/assets/slopmachine/utils/__pycache__/convert_ai_session.cpython-311.pyc +0 -0
- package/assets/slopmachine/utils/__pycache__/normalize_claude_session.cpython-311.pyc +0 -0
- 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
|
-
|
|
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
|
|
9
|
-
- Do not treat
|
|
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
|
|
15
|
-
- Inside the repo, `README.md` is the primary documentation file and repo-local `plan.md` is the explicit execution-plan exception
|
|
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
|
|
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
|
|
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
|
|
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
|
-
-
|
|
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
|
|
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
|
|
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
|
-
-
|
|
89
|
-
-
|
|
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
|
|
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
|
|
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
|
|
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
|
|
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
|
|
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
|
|
9
|
-
- Do not treat
|
|
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
|
|
15
|
-
- Inside the repo, `README.md` is the primary documentation file and repo-local `plan.md` is the explicit execution-plan exception
|
|
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
|
|
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
|
|
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
|
|
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
|
-
-
|
|
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
|
|
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
|
|
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
|
-
-
|
|
88
|
-
-
|
|
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
|
|
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
|
|
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
|
|
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
|
|
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
|
|
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:
|
|
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
|
|
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
|
|
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
|
|
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
|
|
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
|
|
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
|
|
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.
|
|
298
|
-
12.
|
|
299
|
-
13.
|
|
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
|
|
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 |
|
|
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 /
|
|
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.
|
|
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
|
-
|
|
|
636
|
+
| Local readiness evidence expected | |
|
|
584
637
|
|
|
585
638
|
### 19.0.1 Prompt-critical proof ledger
|
|
586
|
-
Use this ledger during development and
|
|
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
|
|
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
|
-
###
|
|
815
|
-
- Static/non-Docker checks to use before
|
|
816
|
-
-
|
|
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
|
-
-
|
|
822
|
-
- Development-completion
|
|
823
|
-
-
|
|
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
|
-
###
|
|
826
|
-
-
|
|
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
|
|
833
|
-
- Maximum
|
|
834
|
-
- Maximum
|
|
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
|
-
###
|
|
837
|
-
- When extra UI/platform sanity is actually needed before
|
|
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
|
-
###
|
|
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
|
-
###
|
|
853
|
-
- No major prompt, product, security, or runtime breakage remains obvious enough to make
|
|
854
|
-
- Development-completion
|
|
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
|
|
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
|
|