|
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 |
| 1 | +diff --git a/docs_build/dev/reports/PR_26171_GAMMA_028-final-sqlite-clean-status-report-instruction-compliance-checklist.md b/docs_build/dev/reports/PR_26171_GAMMA_028-final-sqlite-clean-status-report-instruction-compliance-checklist.md |
2 | 2 | new file mode 100644 |
3 | | -index 000000000..a07705200 |
| 3 | +index 000000000..f60e2b587 |
4 | 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 |
| 5 | ++++ b/docs_build/dev/reports/PR_26171_GAMMA_028-final-sqlite-clean-status-report-instruction-compliance-checklist.md |
| 6 | +@@ -0,0 +1,18 @@ |
| 7 | ++# PR_26171_GAMMA_028 Instruction Compliance Checklist |
10 | 8 | + |
11 | 9 | +- PASS: Read `docs_build/dev/PROJECT_INSTRUCTIONS.md`. |
12 | 10 | +- 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 |
| 11 | ++- PASS: Instruction enforcement start gate was applied. |
| 12 | ++- PASS: TEAM ownership verified as GAMMA. |
| 13 | ++- PASS: Branch name mirrors TEAM token: `pr/26171-GAMMA-028-final-sqlite-clean-status-report`. |
| 14 | ++- PASS: Started from fresh `main` after PR026 and PR027 merged. |
| 15 | ++- PASS: Scope stayed report-only and did not change runtime code. |
| 16 | ++- PASS: Inventory classified remaining SQLite references. |
| 17 | ++- PASS: Active runtime SQLite implementation count is 0. |
| 18 | ++- PASS: Local API SQLite reference count is 0. |
| 19 | ++- PASS: Historical archive/report references were not rewritten. |
| 20 | ++- PASS: `git diff --check` validation is required and recorded. |
| 21 | ++- PASS: No Playwright was run because report-only changes do not affect UI/runtime behavior. |
| 22 | ++- PASS: No samples were run. |
| 23 | ++- PASS: Required report artifacts were produced. |
| 24 | ++- PASS: Repo-structured ZIP path is documented: `tmp/PR_26171_GAMMA_028-final-sqlite-clean-status-report_delta.zip`. |
| 25 | +diff --git a/docs_build/dev/reports/PR_26171_GAMMA_028-final-sqlite-clean-status-report-manual-validation-notes.md b/docs_build/dev/reports/PR_26171_GAMMA_028-final-sqlite-clean-status-report-manual-validation-notes.md |
| 26 | +new file mode 100644 |
| 27 | +index 000000000..3f7689bc5 |
| 28 | +--- /dev/null |
| 29 | ++++ b/docs_build/dev/reports/PR_26171_GAMMA_028-final-sqlite-clean-status-report-manual-validation-notes.md |
| 30 | +@@ -0,0 +1,12 @@ |
| 31 | ++# PR_26171_GAMMA_028 Manual Validation Notes |
| 32 | ++ |
| 33 | ++- Confirmed current branch: `pr/26171-GAMMA-028-final-sqlite-clean-status-report`. |
| 34 | ++- Confirmed fresh `main` after PR026 and PR027 was used as the branch base. |
| 35 | ++- Ran full SQLite text inventory excluding `.git`, `node_modules`, `tmp`, and generated Playwright `tests/results`. |
| 36 | ++- Confirmed active runtime SQLite implementation count is 0. |
| 37 | ++- Confirmed Local API SQLite reference count is 0. |
| 38 | ++- Confirmed remaining active docs references are SQLite deprecation governance only. |
| 39 | ++- Confirmed remaining tests are negative exposure assertions or the Game Journey legacy data-preservation guard. |
| 40 | ++- Confirmed remaining allowed technical debt is limited to validation/governance guard rules and the Game Journey legacy data-preservation guard. |
| 41 | ++- Did not run Playwright because this PR changes report artifacts only. |
| 42 | ++- Did not run samples. |
| 43 | +diff --git a/docs_build/dev/reports/PR_26171_GAMMA_028-final-sqlite-clean-status-report.md b/docs_build/dev/reports/PR_26171_GAMMA_028-final-sqlite-clean-status-report.md |
36 | 44 | new file mode 100644 |
37 | | -index 000000000..415d83d27 |
| 45 | +index 000000000..d87ba0edc |
38 | 46 | --- /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 |
| 47 | ++++ b/docs_build/dev/reports/PR_26171_GAMMA_028-final-sqlite-clean-status-report.md |
| 48 | +@@ -0,0 +1,69 @@ |
| 49 | ++# PR_26171_GAMMA_028-final-sqlite-clean-status-report |
42 | 50 | + |
43 | | -+Manual validation focused on active docs/seed wording. |
| 51 | ++## Scope |
44 | 52 | + |
45 | | -+## Notes |
| 53 | ++Run final SQLite inventory after PR026 and PR027. Confirm active runtime and Local API SQLite implementation status, classify remaining references, and document any follow-up. |
46 | 54 | + |
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. |
| 55 | ++## Dependency Closeout |
52 | 56 | + |
53 | | -+## Skipped |
| 57 | ++- PR #48 / `PR_26171_GAMMA_026-sqlite-test-reference-cleanup` was merged to `main`. |
| 58 | ++- PR #49 / `PR_26171_GAMMA_027-sqlite-doc-reference-cleanup` was merged to `main`. |
| 59 | ++- Fresh branch was created from `main` at `0ccaf0e64badb3daa338a6b1862a7f457b35d280`. |
54 | 60 | + |
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 |
| 61 | ++## Inventory Commands |
65 | 62 | + |
66 | | -+TEAM ownership: GAMMA. |
| 63 | ++Counts were captured before PR028 report artifacts were written, so the report does not self-count its own SQLite summary text. |
67 | 64 | + |
68 | | -+## Scope |
| 65 | ++```text |
| 66 | ++rg -n -i --glob '!node_modules/**' --glob '!.git/**' --glob '!tmp/**' --glob '!tests/results/**' "sqlite|node:sqlite|DatabaseSync|\.sqlite" . |
| 67 | ++rg -n -i --glob '!node_modules/**' --glob '!.git/**' --glob '!tmp/**' --glob '!tests/results/**' "node:sqlite|DatabaseSync|better-sqlite|sqlite3|messages-sqlite-service|messages\.sqlite|GAMEFOUNDRY_MESSAGES_SQLITE_PATH" src src/dev-runtime/server/local-api-router.mjs |
| 68 | ++``` |
| 69 | ++ |
| 70 | ++## Summary |
| 71 | ++ |
| 72 | ++| Category | Files | Lines | Status | |
| 73 | ++| --- | ---: | ---: | --- | |
| 74 | ++| Active Runtime | 0 | 0 | PASS: no active SQLite runtime implementation remains. | |
| 75 | ++| Local API | 0 | 0 | PASS: `src/dev-runtime/server/local-api-router.mjs` has zero SQLite matches. | |
| 76 | ++| Tests | 2 | 5 | PASS: remaining references are negative exposure assertions and the legacy data-preservation guard test. | |
| 77 | ++| Docs | 1 | 7 | PASS: remaining active docs references are `PROJECT_INSTRUCTIONS.md` deprecation governance. | |
| 78 | ++| Archive/Reference | 130 | 393 | PASS: historical PR/report references retained for traceability. | |
| 79 | ++| Allowed Technical Debt | 2 | 17 | PASS: validation/governance guard rules plus the Game Journey legacy data-preservation guard. | |
| 80 | ++ |
| 81 | ++Total remaining inventory: 135 files and 422 lines. |
| 82 | ++ |
| 83 | ++## Migration Status |
| 84 | ++ |
| 85 | ++SQLite runtime migration complete. |
| 86 | ++ |
| 87 | ++Active runtime SQLite implementation count is 0. Active Local API SQLite reference count is 0. |
| 88 | ++ |
| 89 | ++## Remaining Current References |
69 | 90 | + |
70 | | -+Clean active docs/seed wording that implied SQLite is still an active persistence target. |
| 91 | ++### Tests |
71 | 92 | + |
72 | | -+## Changes |
| 93 | ++- `tests/playwright/tools/AdminHealthOperationsPage.spec.mjs` |
| 94 | ++ - Lines 214 and 221 assert the Admin Health page/runtime source does not expose `SQLite`. |
| 95 | ++- `tests/playwright/tools/GameJourneyTool.spec.mjs` |
| 96 | ++ - Lines 1458, 1459, and 1469 verify the Game Journey legacy SQLite data-preservation guard. |
73 | 97 | + |
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. |
| 98 | ++### Docs |
78 | 99 | + |
79 | | -+## Validation |
| 100 | ++- `docs_build/dev/PROJECT_INSTRUCTIONS.md` |
| 101 | ++ - Lines 379 and 385-390 document SQLite deprecation governance and prohibit new SQLite persistence. |
80 | 102 | + |
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 |
| 103 | ++### Allowed Technical Debt |
90 | 104 | + |
91 | | -+Skipped: |
92 | | -+- Playwright: no UI/runtime behavior changed. |
93 | | -+- Samples: skipped by request. |
94 | | -+- Historical reports and PR docs: explicitly out of scope. |
| 105 | ++- `src/dev-runtime/persistence/game-journey-completion-metrics-store.mjs` |
| 106 | ++ - Purpose: preserve data safety by failing visibly if an unmigrated legacy SQLite metrics file still exists. |
| 107 | ++ - This guard does not import, open, read, write, or fall back to SQLite. |
| 108 | ++- `scripts/validate-browser-env-agnostic.mjs` |
| 109 | ++ - Purpose: validation/governance rules that reject SQLite/provider/environment implementation details from browser/runtime surfaces. |
95 | 110 | + |
96 | | -+## Reports |
| 111 | ++## Archive References |
97 | 112 | + |
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` |
| 113 | ++Historical references remain in `docs_build/dev/reports/**`, `docs_build/pr/**`, and old validation logs. They are retained for audit traceability and are not active runtime ownership. |
103 | 114 | + |
104 | | -+## ZIP |
| 115 | ++## Follow-Up |
105 | 116 | + |
106 | | -+`tmp/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup_delta.zip` |
| 117 | ++No active SQLite runtime or Local API follow-up is required. A future cleanup may retire the Game Journey legacy data-preservation guard only after the owner confirms legacy file migration risk has expired. |
107 | 118 | 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 |
| 119 | +index 5ebd4f6a7..2b8d410d0 100644 |
109 | 120 | --- a/docs_build/dev/reports/codex_changed_files.txt |
110 | 121 | +++ 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 |
118 | | - docs_build/dev/reports/codex_changed_files.txt |
| 122 | +@@ -1,6 +1,5 @@ |
| 123 | +-docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-instruction-compliance-checklist.md |
| 124 | +-docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup-manual-validation-notes.md |
| 125 | +-docs_build/dev/reports/PR_26171_GAMMA_027-sqlite-doc-reference-cleanup.md |
| 126 | +-docs_build/dev/reports/codex_changed_files.txt |
| 127 | ++docs_build/dev/reports/codex_changed_files.txt |
119 | 128 | 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", |
| 129 | +-src/dev-runtime/persistence/mock-db-store.js |
| 130 | ++docs_build/dev/reports/PR_26171_GAMMA_028-final-sqlite-clean-status-report.md |
| 131 | ++docs_build/dev/reports/PR_26171_GAMMA_028-final-sqlite-clean-status-report-instruction-compliance-checklist.md |
| 132 | ++docs_build/dev/reports/PR_26171_GAMMA_028-final-sqlite-clean-status-report-manual-validation-notes.md |
0 commit comments