Skip to content

Commit 08f0b6b

Browse files
committed
PR_26172_MASTER_008 build path tile overlays
1 parent 0ccaf0e commit 08f0b6b

6 files changed

Lines changed: 95 additions & 138 deletions
Lines changed: 36 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,36 @@
1+
# Build Path Tile Overlay Status Governance
2+
3+
## Purpose
4+
5+
This addendum defines visual overlay status rules for Build Path tiles.
6+
7+
## Overlay Mapping
8+
9+
| Backlog Status | Meaning | Tile Overlay |
10+
| --- | --- | --- |
11+
| `[ ]` Planned | Planned | black overlay 70% |
12+
| `[?]` Wireframe | Wireframe | black overlay 80% |
13+
| `[.]` Building | Building | black overlay 90% |
14+
| `[x]` Complete | Complete | 100% transparent / no visible overlay |
15+
| `[!]` Blocked | Blocked | yellow overlay 80% |
16+
| `[-]` Deprecated | Deprecated | red overlay 80% |
17+
18+
## Completion Denominator
19+
20+
Blocked items remain in the completion denominator.
21+
22+
Deprecated items are excluded from the completion denominator.
23+
24+
## Badge Text
25+
26+
Blocked and Deprecated tiles must show badge text.
27+
28+
Badge text must be visible enough for a creator to understand why the tile is blocked or deprecated without relying on color alone.
29+
30+
## Validation
31+
32+
PRs that change tile overlay behavior must document:
33+
- status mapping affected
34+
- badge text affected
35+
- phase percentage denominator effect
36+
- skipped UI validation, if any, with reason
Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
# PR_26172_MASTER_008 Instruction Compliance Checklist
2+
3+
- PASS: Read `docs_build/dev/PROJECT_INSTRUCTIONS.md`.
4+
- PASS: Read `docs_build/dev/PROJECT_MULTI_PC.txt`.
5+
- PASS: Explicit MASTER override accepted Team MASTER for this operating-system bootstrap.
6+
- PASS: Branch name mirrors PR ownership: `pr/26172-MASTER-008-build-path-tile-overlays`.
7+
- PASS: Implementation path is `docs_build/dev/ProjectInstructions/`.
8+
- PASS: Existing Project Instructions were preserved.
9+
- PASS: No direct commit to `main`.
10+
- PASS: Required review artifacts are produced.
11+
- PASS: Required ZIP path is `tmp/PR_26172_MASTER_008-build-path-tile-overlays_delta.zip`.
Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
# PR_26172_MASTER_008 Manual Validation Notes
2+
3+
- Confirmed branch was created from latest `main`.
4+
- Confirmed only new files were added.
5+
- Confirmed existing Project Instructions files were not deleted or rewritten.
6+
- Confirmed every requested overlay mapping is documented.
7+
- Confirmed Blocked remains in denominator and Deprecated is excluded.
8+
- Confirmed Blocked and Deprecated badge text requirement is documented.
9+
- Did not run Playwright because this is documentation-only.
10+
- Did not run samples.
Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
# PR_26172_MASTER_008-build-path-tile-overlays
2+
3+
## Scope
4+
5+
Create Build Path tile overlay status governance.
6+
7+
## Changes
8+
9+
- Added `addendums/tile_overlay_status.md`.
10+
- Documented overlay mapping for Planned, Wireframe, Building, Complete, Blocked, and Deprecated.
11+
- Documented blocked/deprecated denominator rules.
12+
- Documented badge text requirement for Blocked and Deprecated.
13+
14+
## Validation
15+
16+
- `git diff --check`
17+
- Markdown/text review for requested overlay mapping and denominator rules.
18+
- Playwright skipped: no runtime UI files changed.
19+
- Samples skipped: not requested.
20+
21+
## ZIP
22+
23+
`tmp/PR_26172_MASTER_008-build-path-tile-overlays_delta.zip`
Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
1-
docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-instruction-compliance-checklist.md
2-
docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-manual-validation-notes.md
3-
docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup.md
1+
docs_build/dev/ProjectInstructions/addendums/tile_overlay_status.md
42
docs_build/dev/reports/codex_changed_files.txt
53
docs_build/dev/reports/codex_review.diff
6-
src/dev-runtime/persistence/mock-db-store.js
4+
docs_build/dev/reports/PR_26172_MASTER_008-build-path-tile-overlays.md
5+
docs_build/dev/reports/PR_26172_MASTER_008-build-path-tile-overlays-instruction-compliance-checklist.md
6+
docs_build/dev/reports/PR_26172_MASTER_008-build-path-tile-overlays-manual-validation-notes.md
Lines changed: 11 additions & 134 deletions
Original file line numberDiff line numberDiff line change
@@ -1,138 +1,15 @@
1-
diff --git a/docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-instruction-compliance-checklist.md b/docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-instruction-compliance-checklist.md
2-
new file mode 100644
3-
index 000000000..a07705200
4-
--- /dev/null
5-
+++ b/docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-instruction-compliance-checklist.md
6-
@@ -0,0 +1,28 @@
7-
+# PR_26171_GAMMA_027 Instruction Compliance Checklist
8-
+
9-
+## Start Gate
10-
+
11-
+- PASS: Read `docs_build/dev/PROJECT_INSTRUCTIONS.md`.
12-
+- PASS: Read `docs_build/dev/PROJECT_MULTI_PC.txt`.
13-
+- PASS: PR name includes TEAM token `GAMMA`.
14-
+- PASS: Branch name mirrors TEAM token: `pr/26171-GAMMA-027-sqlite-doc-reference-cleanup`.
15-
+- PASS: Scope is diagnostics/cleanup and belongs to Team Gamma.
16-
+- PASS: Started from fresh `main` after PR 026 merged.
17-
+- PASS: Repository was clean before branch creation.
18-
+
19-
+## Scope Guard
20-
+
21-
+- PASS: One PR purpose only.
22-
+- PASS: Cleaned active docs/seed wording only.
23-
+- PASS: Preserved `PROJECT_INSTRUCTIONS.md` SQLite deprecation rules.
24-
+- PASS: Did not rewrite historical PR/report archives.
25-
+- PASS: Did not run samples.
26-
+
27-
+## Completion Gate
28-
+
29-
+- PASS: PR-specific report exists.
30-
+- PASS: Manual validation notes exist.
31-
+- PASS: Instruction compliance checklist exists.
32-
+- PASS: `docs_build/dev/reports/codex_review.diff` exists.
33-
+- PASS: `docs_build/dev/reports/codex_changed_files.txt` exists.
34-
+- PASS: Repo-structured ZIP path is documented: `tmp/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup_delta.zip`.
35-
diff --git a/docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-manual-validation-notes.md b/docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-manual-validation-notes.md
36-
new file mode 100644
37-
index 000000000..415d83d27
38-
--- /dev/null
39-
+++ b/docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-manual-validation-notes.md
40-
@@ -0,0 +1,17 @@
41-
+# PR_26171_GAMMA_027 Manual Validation Notes
42-
+
43-
+Manual validation focused on active docs/seed wording.
44-
+
45-
+## Notes
46-
+
47-
+- Confirmed PR 027 branch was created from fresh `main` after PR 026 merged.
48-
+- Confirmed `PROJECT_INSTRUCTIONS.md` deprecation rules were preserved.
49-
+- Confirmed historical report and PR docs were not modified.
50-
+- Confirmed the only active seed wording change was the Local DB session-mode description.
51-
+- Confirmed remaining SQLite references in active paths are governance, validation guards, negative tests, or legacy data-preservation guard text.
52-
+
53-
+## Skipped
54-
+
55-
+- Playwright was skipped because no UI behavior changed.
56-
+- Samples were skipped by request.
57-
+- Broad tests were skipped because this is a wording-only cleanup.
58-
diff --git a/docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup.md b/docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup.md
59-
new file mode 100644
60-
index 000000000..ecb1a233d
61-
--- /dev/null
62-
+++ b/docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup.md
63-
@@ -0,0 +1,43 @@
64-
+# PR_26171_GAMMA_027-sqlite-doc-reference-cleanup
65-
+
66-
+TEAM ownership: GAMMA.
67-
+
68-
+## Scope
69-
+
70-
+Clean active docs/seed wording that implied SQLite is still an active persistence target.
71-
+
72-
+## Changes
73-
+
74-
+- Updated `src/dev-runtime/persistence/mock-db-store.js` session-mode description from SQLite-backed wording to neutral Local DB adapter wording.
75-
+- Preserved `PROJECT_INSTRUCTIONS.md` SQLite deprecation rules.
76-
+- Did not rewrite historical PR/report archives.
77-
+- Did not remove governance, validation guard, negative test, or legacy data-preservation references.
78-
+
79-
+## Validation
80-
+
81-
+Passed:
82-
+- `git diff --check`
83-
+- `node --check src/dev-runtime/persistence/mock-db-store.js`
84-
+- Targeted text check confirmed no active docs/seed wording still says `SQLite-backed`, `SQLite persistence`, `SQLite service`, or `backed by server SQLite storage`.
85-
+- Scoped active-source search confirmed remaining non-archive SQLite references are limited to:
86-
+ - governance rules in `PROJECT_INSTRUCTIONS.md`
87-
+ - validation guard patterns in `scripts/validate-browser-env-agnostic.mjs`
88-
+ - negative test assertions that SQLite is not exposed
89-
+ - Game Journey legacy SQLite data-preservation guard
90-
+
91-
+Skipped:
92-
+- Playwright: no UI/runtime behavior changed.
93-
+- Samples: skipped by request.
94-
+- Historical reports and PR docs: explicitly out of scope.
95-
+
96-
+## Reports
97-
+
98-
+- `docs_build/dev/reports/codex_review.diff`
99-
+- `docs_build/dev/reports/codex_changed_files.txt`
100-
+- `docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup.md`
101-
+- `docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-manual-validation-notes.md`
102-
+- `docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-instruction-compliance-checklist.md`
103-
+
104-
+## ZIP
105-
+
106-
+`tmp/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup_delta.zip`
107-
diff --git a/docs_build/dev/reports/codex_changed_files.txt b/docs_build/dev/reports/codex_changed_files.txt
108-
index 4e23ae82e..5ebd4f6a7 100644
1+
diff --git a/docs_build/dev/reports/codex_changed_files.txt b/docs_build/dev/reports/codex_changed_files.txt
2+
index 5ebd4f6a7..6857842a2 100644
1093
--- a/docs_build/dev/reports/codex_changed_files.txt
1104
+++ b/docs_build/dev/reports/codex_changed_files.txt
111-
@@ -1,10 +1,6 @@
112-
-docs_build/dev/reports/PR_26171_GAMMA_026-sqlite-test-reference-cleanup-instruction-compliance-checklist.md
113-
-docs_build/dev/reports/PR_26171_GAMMA_026-sqlite-test-reference-cleanup-manual-validation-notes.md
114-
-docs_build/dev/reports/PR_26171_GAMMA_026-sqlite-test-reference-cleanup.md
115-
+docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-instruction-compliance-checklist.md
116-
+docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-manual-validation-notes.md
117-
+docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup.md
5+
@@ -1,6 +1,6 @@
6+
-docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-instruction-compliance-checklist.md
7+
-docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-manual-validation-notes.md
8+
-docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup.md
9+
+docs_build/dev/ProjectInstructions/addendums/tile_overlay_status.md
11810
docs_build/dev/reports/codex_changed_files.txt
11911
docs_build/dev/reports/codex_review.diff
120-
-tests/dev-runtime/DbSeedIntegrity.test.mjs
121-
-tests/playwright/tools/AdminDbViewer.spec.mjs
122-
-tests/playwright/tools/BrowserApiUrlConfig.spec.mjs
123-
-tests/playwright/tools/GameJourneyTool.spec.mjs
124-
-tests/playwright/tools/LoginSessionMode.spec.mjs
125-
+src/dev-runtime/persistence/mock-db-store.js
126-
diff --git a/src/dev-runtime/persistence/mock-db-store.js b/src/dev-runtime/persistence/mock-db-store.js
127-
index 2305910bc..c98c915b5 100644
128-
--- a/src/dev-runtime/persistence/mock-db-store.js
129-
+++ b/src/dev-runtime/persistence/mock-db-store.js
130-
@@ -23,7 +23,7 @@ export const MOCK_DB_SESSION_MODES = Object.freeze([
131-
adapterId: "local-db",
132-
adapterName: "LocalDbAdapter",
133-
configured: true,
134-
- description: "Uses LocalDbAdapter backed by server SQLite storage.",
135-
+ description: "Uses the Local DB adapter behind the server API boundary.",
136-
environment: "Local DB",
137-
id: "local-db",
138-
label: "Local DB",
12+
-src/dev-runtime/persistence/mock-db-store.js
13+
+docs_build/dev/reports/PR_26172_MASTER_008-build-path-tile-overlays.md
14+
+docs_build/dev/reports/PR_26172_MASTER_008-build-path-tile-overlays-instruction-compliance-checklist.md
15+
+docs_build/dev/reports/PR_26172_MASTER_008-build-path-tile-overlays-manual-validation-notes.md

0 commit comments

Comments
 (0)