Skip to content

Commit 744f936

Browse files
committed
Add no mock repository runtime governance
1 parent f31fa61 commit 744f936

14 files changed

Lines changed: 286 additions & 30 deletions

docs_build/dev/ProjectInstructions/PROJECT_INSTRUCTIONS.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -58,6 +58,8 @@ Existing Project Instructions outside `docs_build/dev/ProjectInstructions/` rema
5858

5959
`docs_build/dev/ProjectInstructions/addendums/tool_mvp_stacked_pr_standard.md` defines the Creator-facing stacked PR standard for tool MVP work.
6060

61+
`docs_build/dev/ProjectInstructions/addendums/no_mock_repository_runtime_source.md` defines the Browser → API → Database product-data rule and mock repository technical debt boundary.
62+
6163
## Single Source and Main Lock Governance
6264

6365
`docs_build/dev/ProjectInstructions/addendums/project_instructions_single_source_eod_lock.md` defines the single active Project Instructions source, canonical START / WORK / END branch lifecycle, EOD main lock, next-day reset, team branch creation gate, daily synchronization baseline, and mandatory hard stops.

docs_build/dev/ProjectInstructions/README.txt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -60,3 +60,4 @@ Addendum index:
6060
- Samples2Tools Adapter Guidance: docs_build/dev/ProjectInstructions/addendums/samples2tools_adapter_guidance.md
6161
- King of the Iceberg Layout Contract: docs_build/dev/ProjectInstructions/addendums/koti_layout_contract.md
6262
- Tool MVP Stacked PR Standard: docs_build/dev/ProjectInstructions/addendums/tool_mvp_stacked_pr_standard.md
63+
- No Mock Repository Runtime Source: docs_build/dev/ProjectInstructions/addendums/no_mock_repository_runtime_source.md
Lines changed: 49 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,49 @@
1+
# No Mock Repository Runtime Source
2+
3+
Status: Approved
4+
Owner: OWNER
5+
6+
## Required Product-Data Flow
7+
8+
```text
9+
Browser → API → Database
10+
```
11+
12+
Runtime product data must flow through the API/service contract backed by the database.
13+
14+
## Rules
15+
16+
- Mock repositories are 100% technical debt.
17+
- "Mock repository ready" is not a valid Creator-testable completion state.
18+
- Page arrays are not product-data sources of truth.
19+
- JSON source files are not product-data sources of truth.
20+
- `/tmp` files are not product-data sources of truth.
21+
- Browser storage is not product-data source of truth.
22+
- Runtime tool data must come from the API/service contract backed by the database.
23+
- Seed data is allowed only if it seeds the database.
24+
- Seed execution must be owned by server/API/setup flow.
25+
- Browser pages must not seed authoritative records directly.
26+
27+
## Temporary Mock Repository Exception
28+
29+
If a temporary mock repository exists for transition, it must be documented as technical debt with:
30+
31+
- why it exists
32+
- affected files
33+
- removal PR
34+
- replacement API/DB path
35+
- Creator-testable limitation
36+
37+
Temporary mock repositories do not make a feature complete. They may only document a transition state.
38+
39+
## Game Configuration Example
40+
41+
"Game Configuration mock repository ready" does not count as complete.
42+
43+
Completion requires Game Configuration data to load through:
44+
45+
```text
46+
Browser → API → Database
47+
```
48+
49+
The Creator-testable completion state must prove that browser-visible Game Configuration data was read back through the API/service contract backed by the database.

docs_build/dev/ProjectInstructions/addendums/release_gate.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,7 @@ The release gate should confirm these files when relevant to the PR:
4545
- `docs_build/dev/ProjectInstructions/addendums/samples2tools_adapter_guidance.md`
4646
- `docs_build/dev/ProjectInstructions/addendums/koti_layout_contract.md`
4747
- `docs_build/dev/ProjectInstructions/addendums/tool_mvp_stacked_pr_standard.md`
48+
- `docs_build/dev/ProjectInstructions/addendums/no_mock_repository_runtime_source.md`
4849
- `docs_build/dev/ProjectInstructions/team_assignments/team_ownership.md`
4950
- `docs_build/dev/admin-notes/Installs required.txt` when present
5051
- `docs_build/dev/admin-notes/Table layout.txt` when present

docs_build/dev/ProjectInstructions/addendums/tool_mvp_stacked_pr_standard.md

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -48,6 +48,20 @@ Each tool MVP PR must state:
4848
- Previous PR dependency
4949
- Next PR dependency
5050

51+
## Runtime Data Completion Boundary
52+
53+
A PR outcome must not be described as complete if the visible data is coming from:
54+
55+
- mock repositories
56+
- page arrays
57+
- JSON source files
58+
- browser storage
59+
- `/tmp`
60+
61+
Seeded demo data is fine only when it is stored in the database and read back through the API.
62+
63+
Tool MVPs must follow `docs_build/dev/ProjectInstructions/addendums/no_mock_repository_runtime_source.md`.
64+
5165
## Hitboxes MVP Example Stack
5266

5367
Example Hitboxes MVP stack:

docs_build/dev/pr/BUILD_PR.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,8 @@ This `docs_build/dev/pr/BUILD_PR.md`, `docs_build/dev/pr/PLAN_PR.md`, and the us
2020
- Delete stale one-off bundle metadata from `docs_build/dev/` root.
2121
- Add Tool MVP Stacked PR Standard under `docs_build/dev/ProjectInstructions/`.
2222
- Update PR planning/template and report requirements for tool MVP PRs.
23+
- Add No Mock Repository Runtime Source governance under `docs_build/dev/ProjectInstructions/`.
24+
- Update Creator-testable stacked MVP standard to reject mock/page-array/JSON/browser-storage/tmp completion states.
2325
- Keep preserved historical ProjectInstructions material reference-only.
2426
- Keep `project-instructions/` out of this PR except for a tiny deprecated pointer.
2527
- Update active team start/governance docs to reference only `docs_build/dev/ProjectInstructions/`.
@@ -37,6 +39,7 @@ This `docs_build/dev/pr/BUILD_PR.md`, `docs_build/dev/pr/PLAN_PR.md`, and the us
3739
- `docs_build/dev/ProjectInstructions/team_assignments/TEAM_ASSIGNMENTS.md`
3840
- `docs_build/dev/ProjectInstructions/addendums/*.md`
3941
- `docs_build/dev/ProjectInstructions/addendums/tool_mvp_stacked_pr_standard.md`
42+
- `docs_build/dev/ProjectInstructions/addendums/no_mock_repository_runtime_source.md`
4043
- `docs_build/dev/reports/audits/*.md`
4144
- `project-instructions/README.md` (tiny deprecated pointer only)
4245
- `docs_build/dev/PROJECT_INSTRUCTIONS.md` (delete)
@@ -90,6 +93,7 @@ Test-Path docs_build/dev/bundle_readme.md
9093
Test-Path docs_build/dev/validation_checklist.txt
9194
rg -n 'only active Project Instructions source|docs_build/dev/ProjectInstructions/' docs_build/dev/ProjectInstructions
9295
rg -n 'Tool MVP Stacked PR Standard|Creator-testable outcome|What can Mr\\. Q test after applying this ZIP|What Playwright tests|What Mr\\. Q should manually test|Previous PR dependency|Next PR dependency' docs_build/dev/ProjectInstructions
96+
rg -n 'No Mock Repository Runtime Source|Browser → API → Database|Mock repositories are 100% technical debt|Game Configuration mock repository ready|Seeded demo data is fine only when it is stored in the database and read back through the API' docs_build/dev/ProjectInstructions
9397
rg -n 'docs_build/dev/PROJECT_INSTRUCTIONS.md.*source of truth|Codex must always read `docs_build/dev/PROJECT_INSTRUCTIONS.md`|Read `docs_build/dev/PROJECT_INSTRUCTIONS.md`' docs_build/dev/ProjectInstructions project-instructions
9498
git diff --name-status $(git merge-base HEAD origin/main) -- project-instructions
9599
git diff --name-status $(git merge-base HEAD origin/main) -- docs_build/dev/pr docs_build/dev/ProjectInstructions/addendums docs_build/dev/reports/audits

docs_build/dev/pr/PLAN_PR.md

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,8 @@ Make `docs_build/dev/ProjectInstructions/` the only active Project Instructions
1515
- Delete stale one-off bundle metadata from `docs_build/dev/` root.
1616
- Add Tool MVP Stacked PR Standard under `docs_build/dev/ProjectInstructions/`.
1717
- Update PR planning/template and report requirements for tool MVP PRs.
18+
- Add No Mock Repository Runtime Source governance under `docs_build/dev/ProjectInstructions/`.
19+
- Update Creator-testable stacked MVP standard to reject mock/page-array/JSON/browser-storage/tmp completion states.
1820
- Preserve historical ProjectInstructions-style sources only as deprecated reference material.
1921
- Keep `project-instructions/` out of the PR except for a tiny deprecated pointer.
2022
- Move active legacy addendums into `docs_build/dev/ProjectInstructions/addendums/`.
@@ -39,6 +41,8 @@ Make `docs_build/dev/ProjectInstructions/` the only active Project Instructions
3941
6. Confirm moved audit outputs are under `docs_build/dev/reports/audits/`.
4042
7. Confirm Tool MVP Stacked PR Standard appears in active ProjectInstructions.
4143
8. Confirm tool MVP PR template/report requirements include Creator-testable outcome, Playwright tests, Mr. Q manual test, stack membership, and previous/next dependency.
42-
9. Confirm EOD/Next Day and canonical START / WORK / END branch lifecycle rules appear in active governance docs.
43-
10. Confirm no product/runtime files changed.
44-
11. Run `git diff --check`.
44+
9. Confirm No Mock Repository Runtime Source governance appears in active ProjectInstructions.
45+
10. Confirm Creator-testable stacked MVP standard rejects mock/page-array/JSON/browser-storage/tmp completion states.
46+
11. Confirm EOD/Next Day and canonical START / WORK / END branch lifecycle rules appear in active governance docs.
47+
12. Confirm no product/runtime files changed.
48+
13. Run `git diff --check`.

docs_build/dev/reports/PR_26177_OWNER_007-project-instructions-single-source-eod-lock.md

Lines changed: 21 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
Date: 2026-06-26
44
Branch: PR_26177_OWNER_007-project-instructions-single-source-eod-lock
5-
Scope: Project Instructions single-source governance, EOD main lock, branch lifecycle governance, docs_build/dev root cleanup, and Tool MVP stacked PR governance
5+
Scope: Project Instructions single-source governance, EOD main lock, branch lifecycle governance, docs_build/dev root cleanup, Tool MVP stacked PR governance, and no-mock runtime source governance
66
Status: PASS
77

88
## Summary
@@ -17,13 +17,29 @@ Status: PASS
1717
- Moved audit outputs into docs_build/dev/reports/audits/.
1818
- Deleted stale one-off bundle metadata from docs_build/dev root.
1919
- Added docs_build/dev/ProjectInstructions/addendums/tool_mvp_stacked_pr_standard.md.
20+
- Added docs_build/dev/ProjectInstructions/addendums/no_mock_repository_runtime_source.md.
21+
- Updated Creator-testable stacked MVP governance so PR outcomes cannot be complete when visible data comes from mock repositories, page arrays, JSON source files, browser storage, or /tmp.
2022
- Updated PR workflow governance so tool MVP PR plans/templates include Creator-testable outcome, Playwright test scope, Mr. Q manual test scope, stacked MVP sequence membership, and previous/next PR dependencies.
2123
- Updated Codex artifact/reporting governance so every tool MVP PR report includes those same fields and answers what Mr. Q can test after applying the ZIP.
2224
- Corrected project-instructions/** scope so the PR only adds a tiny deprecated pointer README there.
2325
- Preserved project-instructions/addendums/** unchanged in the PR; unique current governance content is carried by docs_build/dev/ProjectInstructions/addendums/.
2426
- Confirmed docs_build/dev root no longer contains active loose instruction, audit, contract, or PR files.
2527
- No product/runtime, start_of_day, feature, or legacy SQLite file changes were made.
2628

29+
## No Mock Repository Runtime Source
30+
31+
Added governance:
32+
- Required product-data flow: Browser → API → Database.
33+
- Mock repositories are 100% technical debt.
34+
- "Mock repository ready" is not a valid Creator-testable completion state.
35+
- Page arrays, JSON source files, /tmp files, and browser storage are not product-data sources of truth.
36+
- Runtime tool data must come from the API/service contract backed by the database.
37+
- Seed data is allowed only if it seeds the database.
38+
- Seed execution must be owned by server/API/setup flow.
39+
- Browser pages must not seed authoritative records directly.
40+
- Temporary mock repositories must document why they exist, affected files, removal PR, replacement API/DB path, and Creator-testable limitation.
41+
- "Game Configuration mock repository ready" does not count as complete; completion requires Game Configuration data to load through Browser → API → Database.
42+
2743
## Tool MVP Stacked PR Standard
2844

2945
Added governance:
@@ -34,6 +50,8 @@ Added governance:
3450
- Each PR answers: What can Mr. Q test after applying this ZIP?
3551
- Codex continues through the stack unless blocked by branch state, failed validation, missing source files, Project Instructions conflict, or unresolved prior-PR dependency.
3652
- Visible acceptance for tool MVP planning is Creator-facing first.
53+
- A PR outcome cannot be complete if visible data comes from mock repositories, page arrays, JSON source files, browser storage, or /tmp.
54+
- Seeded demo data is fine only when it is stored in the database and read back through the API.
3755
- Hitboxes MVP example stack is included.
3856

3957
## Moved To ProjectInstructions SSoT
@@ -91,6 +109,8 @@ Deleted stale one-off root files:
91109
- PASS: project-instructions/** PR diff is limited to project-instructions/README.md.
92110
- PASS: active moved governance/contract docs are indexed in docs_build/dev/ProjectInstructions/PROJECT_INSTRUCTIONS.md and README.txt.
93111
- PASS: Tool MVP Stacked PR Standard exists and includes required governance text.
112+
- PASS: No Mock Repository Runtime Source exists and includes required governance text.
113+
- PASS: Tool MVP standard rejects mock/page-array/JSON/browser-storage/tmp completion states and allows seeded demo data only when stored in DB and read through API.
94114
- PASS: PR workflow/template requirements include Creator-testable outcome, Playwright tests, Mr. Q manual test, stack membership, and previous/next dependency.
95115
- PASS: Codex report requirements include Creator-testable outcome, Playwright tests, Mr. Q manual test, stack membership, and previous/next dependency.
96116
- PASS: targeted grep found no active duplicate ProjectInstructions source-of-truth claim outside the active source.

docs_build/dev/reports/PR_26177_OWNER_007-project-instructions-single-source-eod-lock_branch-validation.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ Status: PASS
77
- PASS: Worktree changes are scoped to governance/docs cleanup, reports, and the required delta artifact.
88
- PASS: docs_build/dev root contains no loose files after moving PR docs, active addendums, and audit outputs.
99
- PASS: Tool MVP Stacked PR Standard was added under docs_build/dev/ProjectInstructions/addendums/.
10+
- PASS: No Mock Repository Runtime Source was added under docs_build/dev/ProjectInstructions/addendums/.
1011
- PASS: project-instructions/** is limited to a tiny deprecated pointer README in the PR diff.
1112
- PASS: No product/runtime files changed.
1213
- PASS: No start_of_day files changed.

docs_build/dev/reports/PR_26177_OWNER_007-project-instructions-single-source-eod-lock_manual-validation-notes.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,8 @@ Status: PASS
1111
- Moved audit outputs into docs_build/dev/reports/audits/.
1212
- Deleted stale bundle metadata from docs_build/dev root.
1313
- Added Tool MVP Stacked PR Standard as active ProjectInstructions governance.
14+
- Added No Mock Repository Runtime Source as active ProjectInstructions governance.
15+
- Updated Creator-testable stacked MVP PR standard to reject mock repository, page array, JSON source file, browser storage, and /tmp completion states.
1416
- Updated PR workflow/template and reporting governance for tool MVP PRs.
1517
- Reviewed ambiguous old project-instructions addendum files and left them unchanged in this PR.
1618
- Confirmed active/current governance content is under docs_build/dev/ProjectInstructions/.

0 commit comments

Comments
 (0)