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
32 changes: 32 additions & 0 deletions docs_build/dev/ProjectInstructions/addendums/deprecation.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,32 @@
# Deprecation Governance

## Purpose

This addendum defines how Project Instructions backlog and Build Path items become deprecated while staying traceable.

## Deprecation Group

Deprecation is its own group and must not be hidden inside normal completion work.

## Traceability

Deprecated items must preserve:
- original item text
- deprecated status
- owner approval reference
- replacement path, if any
- archive/removal decision, if any

## Owner Approval

Owner approval is required before archive or removal.

No team may remove a deprecated item solely because it is deprecated.

## Build Path Display

Deprecated Build Path items use red overlay if visible.

Deprecated items do not count against current MVP completion.

If a deprecated item remains visible, it must communicate Deprecated status to creators without relying on color alone.
20 changes: 20 additions & 0 deletions docs_build/dev/ProjectInstructions/deprecation/README.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
# Deprecation Workflow

Deprecation is its own governance group.

Deprecated items remain traceable from backlog status through any visible Build Path representation and any later archive/removal decision.

## Rules

- Deprecation uses status `[-]`.
- Deprecated items are traceable.
- Owner approval is required before archive or removal.
- Deprecated Build Path items use red overlay if visible.
- Deprecated items do not count against current MVP completion.
- Deprecated items should include a note explaining replacement path, retention reason, or removal blocker.

## Archive And Removal

Do not archive or remove deprecated material until the owner explicitly approves.

When approved, archive before removal whenever traceability is needed.
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
# PR_26172_MASTER_009 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-009-deprecation-workflow`.
- 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_009-deprecation-workflow_delta.zip`.
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
# PR_26172_MASTER_009 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 deprecation group, traceability, owner approval, red overlay, and MVP completion rules are documented.
- Did not run Playwright because this is documentation-only.
- Did not run samples.
23 changes: 23 additions & 0 deletions docs_build/dev/reports/PR_26172_MASTER_009-deprecation-workflow.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,23 @@
# PR_26172_MASTER_009-deprecation-workflow

## Scope

Create deprecation workflow governance.

## Changes

- Added `deprecation/README.md`.
- Added `addendums/deprecation.md`.
- Documented deprecation as its own group.
- Documented traceability, owner approval before archive/removal, red overlay behavior, and MVP completion exclusion.

## Validation

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

## ZIP

`tmp/PR_26172_MASTER_009-deprecation-workflow_delta.zip`
9 changes: 5 additions & 4 deletions docs_build/dev/reports/codex_changed_files.txt
Original file line number Diff line number Diff line change
@@ -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/deprecation.md
docs_build/dev/ProjectInstructions/deprecation/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_009-deprecation-workflow.md
docs_build/dev/reports/PR_26172_MASTER_009-deprecation-workflow-instruction-compliance-checklist.md
docs_build/dev/reports/PR_26172_MASTER_009-deprecation-workflow-manual-validation-notes.md
146 changes: 12 additions & 134 deletions docs_build/dev/reports/codex_review.diff
Original file line number Diff line number Diff line change
@@ -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..c01b77076 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/deprecation.md
+docs_build/dev/ProjectInstructions/deprecation/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_009-deprecation-workflow.md
+docs_build/dev/reports/PR_26172_MASTER_009-deprecation-workflow-instruction-compliance-checklist.md
+docs_build/dev/reports/PR_26172_MASTER_009-deprecation-workflow-manual-validation-notes.md
Loading