From 46d3b5c541f15fe5d1bc5141ab7343b4d9caa113 Mon Sep 17 00:00:00 2001 From: "Mr. Q" <97194984+ToolboxAid@users.noreply.github.com> Date: Sun, 21 Jun 2026 13:27:49 -0400 Subject: [PATCH] PR_26172_MASTER_002 preservation and archive governance --- .../addendums/preservation.md | 42 +++++ .../dev/ProjectInstructions/archive/README.md | 12 ++ ...rnance-instruction-compliance-checklist.md | 11 ++ ...hive-governance-manual-validation-notes.md | 8 + ...002-preservation-and-archive-governance.md | 22 +++ .../dev/reports/codex_changed_files.txt | 9 +- docs_build/dev/reports/codex_review.diff | 146 ++---------------- 7 files changed, 112 insertions(+), 138 deletions(-) create mode 100644 docs_build/dev/ProjectInstructions/addendums/preservation.md create mode 100644 docs_build/dev/ProjectInstructions/archive/README.md create mode 100644 docs_build/dev/reports/PR_26172_MASTER_002-preservation-and-archive-governance-instruction-compliance-checklist.md create mode 100644 docs_build/dev/reports/PR_26172_MASTER_002-preservation-and-archive-governance-manual-validation-notes.md create mode 100644 docs_build/dev/reports/PR_26172_MASTER_002-preservation-and-archive-governance.md diff --git a/docs_build/dev/ProjectInstructions/addendums/preservation.md b/docs_build/dev/ProjectInstructions/addendums/preservation.md new file mode 100644 index 000000000..9344992c0 --- /dev/null +++ b/docs_build/dev/ProjectInstructions/addendums/preservation.md @@ -0,0 +1,42 @@ +# Preservation And Archive Governance + +## Purpose + +This addendum protects Project Instructions material from accidental deletion, rewrite, or drift. + +## Append-First Documentation + +Project Instructions documentation is append-first. + +Allowed default actions: +- add new files +- append dated notes +- append references +- add new governance addendums +- add archive snapshots + +## Owner Approval Required + +Do not delete Project Instructions material unless the owner explicitly approves deletion. + +Do not rewrite Project Instructions material unless the owner explicitly approves rewrite. + +If a requested change conflicts with existing Project Instructions, stop before editing. Explain: +- the conflicting files +- the conflicting rule text +- the requested change +- the recommended action + +Then request owner approval. + +## Archive Instead Of Delete + +When material is no longer active, archive it instead of deleting it. Archive entries must preserve enough context to explain why the material moved out of active use. + +## History Snapshots + +History snapshots go to: + +`docs_build/dev/ProjectInstructions/archive/history/CCYYMMDD_HHMMSS.md` + +Use 24-hour local time for `HHMMSS` unless the owner specifies another time basis. diff --git a/docs_build/dev/ProjectInstructions/archive/README.md b/docs_build/dev/ProjectInstructions/archive/README.md new file mode 100644 index 000000000..dbc07a1d2 --- /dev/null +++ b/docs_build/dev/ProjectInstructions/archive/README.md @@ -0,0 +1,12 @@ +# Project Instructions Archive + +This archive retains Project Instructions reference material that is no longer active but must remain traceable. + +## Rules + +- Archive instead of delete. +- Keep archived material readable. +- Preserve original meaning and context. +- Do not rewrite archive entries unless the owner explicitly approves. +- Use `history/CCYYMMDD_HHMMSS.md` for history snapshots. +- If archive placement is unclear, stop and request owner approval. diff --git a/docs_build/dev/reports/PR_26172_MASTER_002-preservation-and-archive-governance-instruction-compliance-checklist.md b/docs_build/dev/reports/PR_26172_MASTER_002-preservation-and-archive-governance-instruction-compliance-checklist.md new file mode 100644 index 000000000..e21f663f2 --- /dev/null +++ b/docs_build/dev/reports/PR_26172_MASTER_002-preservation-and-archive-governance-instruction-compliance-checklist.md @@ -0,0 +1,11 @@ +# PR_26172_MASTER_002 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: Branch name mirrors PR ownership: `pr/26172-MASTER-002-preservation-and-archive-governance`. +- 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_002-preservation-and-archive-governance_delta.zip`. diff --git a/docs_build/dev/reports/PR_26172_MASTER_002-preservation-and-archive-governance-manual-validation-notes.md b/docs_build/dev/reports/PR_26172_MASTER_002-preservation-and-archive-governance-manual-validation-notes.md new file mode 100644 index 000000000..906e227d7 --- /dev/null +++ b/docs_build/dev/reports/PR_26172_MASTER_002-preservation-and-archive-governance-manual-validation-notes.md @@ -0,0 +1,8 @@ +# PR_26172_MASTER_002 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 preservation addendum includes append-first, owner approval, conflict stop, archive, and history snapshot rules. +- Did not run Playwright because this is documentation-only. +- Did not run samples. diff --git a/docs_build/dev/reports/PR_26172_MASTER_002-preservation-and-archive-governance.md b/docs_build/dev/reports/PR_26172_MASTER_002-preservation-and-archive-governance.md new file mode 100644 index 000000000..79b0361f4 --- /dev/null +++ b/docs_build/dev/reports/PR_26172_MASTER_002-preservation-and-archive-governance.md @@ -0,0 +1,22 @@ +# PR_26172_MASTER_002-preservation-and-archive-governance + +## Scope + +Create preservation and archive governance under `docs_build/dev/ProjectInstructions/`. + +## Changes + +- Added `addendums/preservation.md`. +- Added `archive/README.md`. +- Documented append-first rules, owner approval for delete/rewrite, conflict stops, archive-instead-of-delete, and history snapshot path. + +## Validation + +- `git diff --check` +- Markdown/text review for requested governance rules. +- Playwright skipped: no runtime UI files changed. +- Samples skipped: not requested. + +## ZIP + +`tmp/PR_26172_MASTER_002-preservation-and-archive-governance_delta.zip` diff --git a/docs_build/dev/reports/codex_changed_files.txt b/docs_build/dev/reports/codex_changed_files.txt index 5ebd4f6a7..9cc360da5 100644 --- a/docs_build/dev/reports/codex_changed_files.txt +++ b/docs_build/dev/reports/codex_changed_files.txt @@ -1,6 +1,7 @@ -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/preservation.md +docs_build/dev/ProjectInstructions/archive/README.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_002-preservation-and-archive-governance.md +docs_build/dev/reports/PR_26172_MASTER_002-preservation-and-archive-governance-instruction-compliance-checklist.md +docs_build/dev/reports/PR_26172_MASTER_002-preservation-and-archive-governance-manual-validation-notes.md diff --git a/docs_build/dev/reports/codex_review.diff b/docs_build/dev/reports/codex_review.diff index 88d1a78eb..c4178ba50 100644 --- a/docs_build/dev/reports/codex_review.diff +++ b/docs_build/dev/reports/codex_review.diff @@ -1,138 +1,16 @@ -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..9cc360da5 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,7 @@ +-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/preservation.md ++docs_build/dev/ProjectInstructions/archive/README.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_002-preservation-and-archive-governance.md ++docs_build/dev/reports/PR_26172_MASTER_002-preservation-and-archive-governance-instruction-compliance-checklist.md ++docs_build/dev/reports/PR_26172_MASTER_002-preservation-and-archive-governance-manual-validation-notes.md