Skip to content
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
35 changes: 35 additions & 0 deletions docs_build/dev/ProjectInstructions/addendums/build_path_sync.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,35 @@
# Build Path Status Sync Governance

## Purpose

This addendum keeps the master backlog and Toolbox Build Path aligned.

## Source Of Truth

`docs_build/dev/ProjectInstructions/backlog/BACKLOG_MASTER.md` is the source of truth for tool status.

## Build Path Sync

`toolbox/index.html` Build Path tiles must match tool status from `BACKLOG_MASTER.md`.

When a master backlog tool status changes, update the matching Build Path tile.

Status and Build Path changes should be in the same PR when practical.

## Phase Percentages

Phase percentages are calculated from tools in each phase.

Rules:
- completed tools count as complete
- planned, wireframe, building, and blocked tools remain in the denominator
- deprecated tools are excluded from the denominator
- phase percentage must be recalculated when a tool status changes

## Validation

PRs that change backlog status or Build Path status must document:
- backlog item changed
- matching Build Path tile changed
- phase percentage before and after
- skipped sync, if any, with reason
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
# PR_26172_MASTER_007 Instruction Compliance Checklist

- PASS: Read `docs_build/dev/PROJECT_INSTRUCTIONS.md`.
- PASS: Read `docs_build/dev/PROJECT_MULTI_PC.txt`.
- PASS: Explicit MASTER override accepted Team MASTER for this operating-system bootstrap.
- PASS: PR006 was dropped by owner decision; sequence continues at PR007.
- PASS: Branch name mirrors PR ownership: `pr/26172-MASTER-007-build-path-status-sync`.
- PASS: Implementation path is `docs_build/dev/ProjectInstructions/`.
- PASS: Existing Project Instructions were preserved.
- PASS: No direct commit to `main`.
- PASS: Required review artifacts are produced.
- PASS: Required ZIP path is `tmp/PR_26172_MASTER_007-build-path-status-sync_delta.zip`.
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
# PR_26172_MASTER_007 Manual Validation Notes

- Confirmed branch was created from latest `main`.
- Confirmed only new files were added.
- Confirmed existing Project Instructions files were not deleted or rewritten.
- Confirmed source-of-truth, tile sync, phase percentage, and same-PR guidance are documented.
- Did not run Playwright because this is documentation-only.
- Did not run samples.
Original file line number Diff line number Diff line change
@@ -0,0 +1,24 @@
# PR_26172_MASTER_007-build-path-status-sync

## Scope

Create Build Path status synchronization governance.

## Changes

- Added `addendums/build_path_sync.md`.
- Documented `BACKLOG_MASTER.md` as tool status source of truth.
- Documented Build Path tile sync requirement.
- Documented phase percentage calculation rules.
- Documented same-PR sync preference.

## Validation

- `git diff --check`
- Markdown/text review for requested sync rules.
- Playwright skipped: no runtime UI files changed.
- Samples skipped: not requested.

## ZIP

`tmp/PR_26172_MASTER_007-build-path-status-sync_delta.zip`
8 changes: 4 additions & 4 deletions docs_build/dev/reports/codex_changed_files.txt
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-instruction-compliance-checklist.md
docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-manual-validation-notes.md
docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup.md
docs_build/dev/ProjectInstructions/addendums/build_path_sync.md
docs_build/dev/reports/codex_changed_files.txt
docs_build/dev/reports/codex_review.diff
src/dev-runtime/persistence/mock-db-store.js
docs_build/dev/reports/PR_26172_MASTER_007-build-path-status-sync.md
docs_build/dev/reports/PR_26172_MASTER_007-build-path-status-sync-instruction-compliance-checklist.md
docs_build/dev/reports/PR_26172_MASTER_007-build-path-status-sync-manual-validation-notes.md
145 changes: 11 additions & 134 deletions docs_build/dev/reports/codex_review.diff
Original file line number Diff line number Diff line change
@@ -1,138 +1,15 @@
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
new file mode 100644
index 000000000..a07705200
--- /dev/null
+++ b/docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-instruction-compliance-checklist.md
@@ -0,0 +1,28 @@
+# PR_26171_GAMMA_027 Instruction Compliance Checklist
+
+## Start Gate
+
+- PASS: Read `docs_build/dev/PROJECT_INSTRUCTIONS.md`.
+- PASS: Read `docs_build/dev/PROJECT_MULTI_PC.txt`.
+- PASS: PR name includes TEAM token `GAMMA`.
+- PASS: Branch name mirrors TEAM token: `pr/26171-GAMMA-027-sqlite-doc-reference-cleanup`.
+- PASS: Scope is diagnostics/cleanup and belongs to Team Gamma.
+- PASS: Started from fresh `main` after PR 026 merged.
+- PASS: Repository was clean before branch creation.
+
+## Scope Guard
+
+- PASS: One PR purpose only.
+- PASS: Cleaned active docs/seed wording only.
+- PASS: Preserved `PROJECT_INSTRUCTIONS.md` SQLite deprecation rules.
+- PASS: Did not rewrite historical PR/report archives.
+- PASS: Did not run samples.
+
+## Completion Gate
+
+- PASS: PR-specific report exists.
+- PASS: Manual validation notes exist.
+- PASS: Instruction compliance checklist exists.
+- PASS: `docs_build/dev/reports/codex_review.diff` exists.
+- PASS: `docs_build/dev/reports/codex_changed_files.txt` exists.
+- PASS: Repo-structured ZIP path is documented: `tmp/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup_delta.zip`.
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
new file mode 100644
index 000000000..415d83d27
--- /dev/null
+++ b/docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-manual-validation-notes.md
@@ -0,0 +1,17 @@
+# PR_26171_GAMMA_027 Manual Validation Notes
+
+Manual validation focused on active docs/seed wording.
+
+## Notes
+
+- Confirmed PR 027 branch was created from fresh `main` after PR 026 merged.
+- Confirmed `PROJECT_INSTRUCTIONS.md` deprecation rules were preserved.
+- Confirmed historical report and PR docs were not modified.
+- Confirmed the only active seed wording change was the Local DB session-mode description.
+- Confirmed remaining SQLite references in active paths are governance, validation guards, negative tests, or legacy data-preservation guard text.
+
+## Skipped
+
+- Playwright was skipped because no UI behavior changed.
+- Samples were skipped by request.
+- Broad tests were skipped because this is a wording-only cleanup.
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
new file mode 100644
index 000000000..ecb1a233d
--- /dev/null
+++ b/docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup.md
@@ -0,0 +1,43 @@
+# PR_26171_GAMMA_027-sqlite-doc-reference-cleanup
+
+TEAM ownership: GAMMA.
+
+## Scope
+
+Clean active docs/seed wording that implied SQLite is still an active persistence target.
+
+## Changes
+
+- Updated `src/dev-runtime/persistence/mock-db-store.js` session-mode description from SQLite-backed wording to neutral Local DB adapter wording.
+- Preserved `PROJECT_INSTRUCTIONS.md` SQLite deprecation rules.
+- Did not rewrite historical PR/report archives.
+- Did not remove governance, validation guard, negative test, or legacy data-preservation references.
+
+## Validation
+
+Passed:
+- `git diff --check`
+- `node --check src/dev-runtime/persistence/mock-db-store.js`
+- Targeted text check confirmed no active docs/seed wording still says `SQLite-backed`, `SQLite persistence`, `SQLite service`, or `backed by server SQLite storage`.
+- Scoped active-source search confirmed remaining non-archive SQLite references are limited to:
+ - governance rules in `PROJECT_INSTRUCTIONS.md`
+ - validation guard patterns in `scripts/validate-browser-env-agnostic.mjs`
+ - negative test assertions that SQLite is not exposed
+ - Game Journey legacy SQLite data-preservation guard
+
+Skipped:
+- Playwright: no UI/runtime behavior changed.
+- Samples: skipped by request.
+- Historical reports and PR docs: explicitly out of scope.
+
+## Reports
+
+- `docs_build/dev/reports/codex_review.diff`
+- `docs_build/dev/reports/codex_changed_files.txt`
+- `docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup.md`
+- `docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-manual-validation-notes.md`
+- `docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-instruction-compliance-checklist.md`
+
+## ZIP
+
+`tmp/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup_delta.zip`
diff --git a/docs_build/dev/reports/codex_changed_files.txt b/docs_build/dev/reports/codex_changed_files.txt
index 4e23ae82e..5ebd4f6a7 100644
diff --git a/docs_build/dev/reports/codex_changed_files.txt b/docs_build/dev/reports/codex_changed_files.txt
index 5ebd4f6a7..66160034b 100644
--- a/docs_build/dev/reports/codex_changed_files.txt
+++ b/docs_build/dev/reports/codex_changed_files.txt
@@ -1,10 +1,6 @@
-docs_build/dev/reports/PR_26171_GAMMA_026-sqlite-test-reference-cleanup-instruction-compliance-checklist.md
-docs_build/dev/reports/PR_26171_GAMMA_026-sqlite-test-reference-cleanup-manual-validation-notes.md
-docs_build/dev/reports/PR_26171_GAMMA_026-sqlite-test-reference-cleanup.md
+docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-instruction-compliance-checklist.md
+docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-manual-validation-notes.md
+docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup.md
@@ -1,6 +1,6 @@
-docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-instruction-compliance-checklist.md
-docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-manual-validation-notes.md
-docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup.md
+docs_build/dev/ProjectInstructions/addendums/build_path_sync.md
docs_build/dev/reports/codex_changed_files.txt
docs_build/dev/reports/codex_review.diff
-tests/dev-runtime/DbSeedIntegrity.test.mjs
-tests/playwright/tools/AdminDbViewer.spec.mjs
-tests/playwright/tools/BrowserApiUrlConfig.spec.mjs
-tests/playwright/tools/GameJourneyTool.spec.mjs
-tests/playwright/tools/LoginSessionMode.spec.mjs
+src/dev-runtime/persistence/mock-db-store.js
diff --git a/src/dev-runtime/persistence/mock-db-store.js b/src/dev-runtime/persistence/mock-db-store.js
index 2305910bc..c98c915b5 100644
--- a/src/dev-runtime/persistence/mock-db-store.js
+++ b/src/dev-runtime/persistence/mock-db-store.js
@@ -23,7 +23,7 @@ export const MOCK_DB_SESSION_MODES = Object.freeze([
adapterId: "local-db",
adapterName: "LocalDbAdapter",
configured: true,
- description: "Uses LocalDbAdapter backed by server SQLite storage.",
+ description: "Uses the Local DB adapter behind the server API boundary.",
environment: "Local DB",
id: "local-db",
label: "Local DB",
-src/dev-runtime/persistence/mock-db-store.js
+docs_build/dev/reports/PR_26172_MASTER_007-build-path-status-sync.md
+docs_build/dev/reports/PR_26172_MASTER_007-build-path-status-sync-instruction-compliance-checklist.md
+docs_build/dev/reports/PR_26172_MASTER_007-build-path-status-sync-manual-validation-notes.md
Loading