Skip to content

PR_26172_MASTER_009-deprecation-workflow#58

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
ToolboxAid wants to merge 1 commit into from
Closed

PR_26172_MASTER_009-deprecation-workflow #58

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
View file
Open in desktop
Original file line numberDiff line numberDiff 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
View file
Open in desktop
Original file line numberDiff line numberDiff 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.
11 changes: 11 additions & 0 deletions ...ts/PR_26172_MASTER_009-deprecation-workflow-instruction-compliance-checklist.md
View file
Open in desktop
Original file line numberDiff line numberDiff 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`.
8 changes: 8 additions & 0 deletions ...dev/reports/PR_26172_MASTER_009-deprecation-workflow-manual-validation-notes.md
View file
Open in desktop
Original file line numberDiff line numberDiff 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
View file
Open in desktop
Original file line numberDiff line numberDiff 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
View file
Open in desktop
Original file line numberDiff line numberDiff 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
View file
Open in desktop
Original file line numberDiff line numberDiff 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
, 'i'); if (__m === '*' || __re.test(location.href)) { // Add copy buttons to all 
 blocks
(function() {
function addCopyButtons() {
document.querySelectorAll('pre code').forEach(function(codeBlock) {
if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;
codeBlock.parentElement.setAttribute('data-copy-added', 'true');
var btn = document.createElement('button');
btn.textContent = 'Copy';
btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';
btn.onmouseover = function() { this.style.opacity = '1'; };
btn.onmouseout = function() { this.style.opacity = '0.7'; };
btn.onclick = function() {
navigator.clipboard.writeText(codeBlock.textContent).then(function() {
btn.textContent = 'Copied!';
setTimeout(function() { btn.textContent = 'Copy'; }, 1500);
});
};
codeBlock.parentElement.style.position = 'relative';
codeBlock.parentElement.appendChild(btn);
});
}
addCopyButtons();
// Re-run on dynamic content
var observer = new MutationObserver(addCopyButtons);
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '
PR_26172_MASTER_009-deprecation-workflow by ToolboxAid · Pull Request #58 · ToolboxAid/HTML-JavaScript-Gaming · GitHub
Skip to content
PR_26172_MASTER_009-deprecation-workflow#58

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

 Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and
privacy statement. We’ll occasionally send you account related emails.

Already on GitHub?
Sign in
to your account

 Closed

ToolboxAid
wants to merge
1
commit into

from

 Closed


PR_26172_MASTER_009-deprecation-workflow
#58
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


1 commit


Select commit


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

View file


Open in desktop

Original file line numberDiff line numberDiff 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

View file


Open in desktop

Original file line numberDiff line numberDiff 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.

11 changes: 11 additions & 0 deletions
...ts/PR_26172_MASTER_009-deprecation-workflow-instruction-compliance-checklist.md

View file


Open in desktop

Original file line numberDiff line numberDiff 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`.

8 changes: 8 additions & 0 deletions
...dev/reports/PR_26172_MASTER_009-deprecation-workflow-manual-validation-notes.md

View file


Open in desktop

Original file line numberDiff line numberDiff 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

View file


Open in desktop

Original file line numberDiff line numberDiff 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

View file


Open in desktop

Original file line numberDiff line numberDiff 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

View file


Open in desktop

Original file line numberDiff line numberDiff 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

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Force GitHub README to respect dark mode
(function() {
var style = document.createElement('style');
style.textContent = '
.markdown-body {
color-scheme: dark light;
}
.markdown-body pre { background: #161b22 !important; }
.markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }
.markdown-body table th, .markdown-body table td { border-color: #30363d !important; }
.markdown-body img { background: #0d1117; }
.markdown-body blockquote { border-left-color: #8b949e; }
.markdown-body hr { border-color: #30363d; }
';
document.head.appendChild(style);
})();
}
} catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '
PR_26172_MASTER_009-deprecation-workflow by ToolboxAid · Pull Request #58 · ToolboxAid/HTML-JavaScript-Gaming · GitHub
Skip to content
PR_26172_MASTER_009-deprecation-workflow#58

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

 Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and
privacy statement. We’ll occasionally send you account related emails.

Already on GitHub?
Sign in
to your account

 Closed

ToolboxAid
wants to merge
1
commit into

from

 Closed


PR_26172_MASTER_009-deprecation-workflow
#58
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


1 commit


Select commit


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

View file


Open in desktop

Original file line numberDiff line numberDiff 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

View file


Open in desktop

Original file line numberDiff line numberDiff 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.

11 changes: 11 additions & 0 deletions
...ts/PR_26172_MASTER_009-deprecation-workflow-instruction-compliance-checklist.md

View file


Open in desktop

Original file line numberDiff line numberDiff 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`.

8 changes: 8 additions & 0 deletions
...dev/reports/PR_26172_MASTER_009-deprecation-workflow-manual-validation-notes.md

View file


Open in desktop

Original file line numberDiff line numberDiff 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

View file


Open in desktop

Original file line numberDiff line numberDiff 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

View file


Open in desktop

Original file line numberDiff line numberDiff 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

View file


Open in desktop

Original file line numberDiff line numberDiff 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

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Highlight search terms from Google/DuckDuckGo/Bing referrer
(function() {
var ref = document.referrer;
var terms = [];
if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {
var url = new URL(ref);
var q = url.searchParams.get('q') || url.searchParams.get('p');
if (q) {
terms = q.split(/\s+/).filter(function(t) { return t.length > 2; });
}
}
if (terms.length === 0) return;
var style = document.createElement('style');
style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';
document.head.appendChild(style);
function highlight(node) {
if (node.nodeType === 3) { // text node
var text = node.textContent;
var found = false;
terms.forEach(function(term) {
var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\]\\]/g, '\\') + ')', 'gi');
if (regex.test(text)) {
found = true;
var frag = document.createDocumentFragment();
var parts = text.split(regex);
parts.forEach(function(part, i) {
if (i % 2 === 0) {
frag.appendChild(document.createTextNode(part));
} else {
var span = document.createElement('span');
span.className = 'userscript-highlight';
span.textContent = part;
frag.appendChild(span);
}
});
node.parentNode.replaceChild(frag, node);
}
});
} else if (node.nodeType === 1 && node.childNodes) { // element
var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];
if (!skipTags.includes(node.tagName)) {
Array.from(node.childNodes).forEach(highlight);
}
}
}
highlight(document.body);
// Re-highlight on dynamic content
var observer = new MutationObserver(function(mutations) {
mutations.forEach(function(m) {
m.addedNodes.forEach(function(node) {
if (node.nodeType === 1 || node.nodeType === 3) highlight(node);
});
});
});
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '
PR_26172_MASTER_009-deprecation-workflow by ToolboxAid · Pull Request #58 · ToolboxAid/HTML-JavaScript-Gaming · GitHub
Skip to content
PR_26172_MASTER_009-deprecation-workflow#58

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

 Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and
privacy statement. We’ll occasionally send you account related emails.

Already on GitHub?
Sign in
to your account

 Closed

ToolboxAid
wants to merge
1
commit into

from

 Closed


PR_26172_MASTER_009-deprecation-workflow
#58
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


1 commit


Select commit


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

View file


Open in desktop

Original file line numberDiff line numberDiff 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

View file


Open in desktop

Original file line numberDiff line numberDiff 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.

11 changes: 11 additions & 0 deletions
...ts/PR_26172_MASTER_009-deprecation-workflow-instruction-compliance-checklist.md

View file


Open in desktop

Original file line numberDiff line numberDiff 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`.

8 changes: 8 additions & 0 deletions
...dev/reports/PR_26172_MASTER_009-deprecation-workflow-manual-validation-notes.md

View file


Open in desktop

Original file line numberDiff line numberDiff 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

View file


Open in desktop

Original file line numberDiff line numberDiff 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

View file


Open in desktop

Original file line numberDiff line numberDiff 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

View file


Open in desktop

Original file line numberDiff line numberDiff 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

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Strip utm_, fbclid, gclid, etc. from all links on page
(function() {
var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',
'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',
'ref', 'ref_src', 'source', 'medium', 'campaign'];
function cleanUrl(url) {
try {
var u = new URL(url, window.location.origin);
var changed = false;
trackingParams.forEach(function(p) {
if (u.searchParams.has(p)) {
u.searchParams.delete(p);
changed = true;
}
});
return changed ? u.toString() : url;
} catch (e) {
return url;
}
}
function cleanLinks() {
document.querySelectorAll('a[href]').forEach(function(a) {
var clean = cleanUrl(a.href);
if (clean !== a.href) a.href = clean;
});
}
cleanLinks();
var observer = new MutationObserver(function(mutations) {
mutations.forEach(function(m) {
m.addedNodes.forEach(function(node) {
if (node.nodeType === 1) {
if (node.tagName === 'A') cleanLinks();
node.querySelectorAll('a[href]').forEach(function(a) {
var clean = cleanUrl(a.href);
if (clean !== a.href) a.href = clean;
});
}
});
});
});
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); }
})();
(function(){
try {
var __m = "youtube.com";
var __re = new RegExp('^' + "youtube\\.com" + '
PR_26172_MASTER_009-deprecation-workflow by ToolboxAid · Pull Request #58 · ToolboxAid/HTML-JavaScript-Gaming · GitHub
Skip to content
PR_26172_MASTER_009-deprecation-workflow#58

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

 Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and
privacy statement. We’ll occasionally send you account related emails.

Already on GitHub?
Sign in
to your account

 Closed

ToolboxAid
wants to merge
1
commit into

from

 Closed


PR_26172_MASTER_009-deprecation-workflow
#58
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


1 commit


Select commit


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

View file


Open in desktop

Original file line numberDiff line numberDiff 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

View file


Open in desktop

Original file line numberDiff line numberDiff 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.

11 changes: 11 additions & 0 deletions
...ts/PR_26172_MASTER_009-deprecation-workflow-instruction-compliance-checklist.md

View file


Open in desktop

Original file line numberDiff line numberDiff 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`.

8 changes: 8 additions & 0 deletions
...dev/reports/PR_26172_MASTER_009-deprecation-workflow-manual-validation-notes.md

View file


Open in desktop

Original file line numberDiff line numberDiff 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

View file


Open in desktop

Original file line numberDiff line numberDiff 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

View file


Open in desktop

Original file line numberDiff line numberDiff 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

View file


Open in desktop

Original file line numberDiff line numberDiff 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

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Auto-enable theater mode on YouTube
(function() {
function tryTheater() {
var btn = document.querySelector('button[aria-label="Theater mode"], ytd-player #player button[title="Theater mode"]');
if (btn && !btn.classList.contains('activated')) {
btn.click();
}
}
// Try immediately
tryTheater();
// Try after navigation (SPA)
var lastUrl = location.href;
setInterval(function() {
if (location.href !== lastUrl) {
lastUrl = location.href;
setTimeout(tryTheater, 500);
}
}, 1000);
// Also try on player load
var observer = new MutationObserver(tryTheater);
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '
PR_26172_MASTER_009-deprecation-workflow by ToolboxAid · Pull Request #58 · ToolboxAid/HTML-JavaScript-Gaming · GitHub
Skip to content
PR_26172_MASTER_009-deprecation-workflow#58

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

 Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and
privacy statement. We’ll occasionally send you account related emails.

Already on GitHub?
Sign in
to your account

 Closed

ToolboxAid
wants to merge
1
commit into

from

 Closed


PR_26172_MASTER_009-deprecation-workflow
#58
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


1 commit


Select commit


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

View file


Open in desktop

Original file line numberDiff line numberDiff 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

View file


Open in desktop

Original file line numberDiff line numberDiff 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.

11 changes: 11 additions & 0 deletions
...ts/PR_26172_MASTER_009-deprecation-workflow-instruction-compliance-checklist.md

View file


Open in desktop

Original file line numberDiff line numberDiff 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`.

8 changes: 8 additions & 0 deletions
...dev/reports/PR_26172_MASTER_009-deprecation-workflow-manual-validation-notes.md

View file


Open in desktop

Original file line numberDiff line numberDiff 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

View file


Open in desktop

Original file line numberDiff line numberDiff 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

View file


Open in desktop

Original file line numberDiff line numberDiff 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

View file


Open in desktop

Original file line numberDiff line numberDiff 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

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Remove or un-stick sticky/fixed headers that block content
(function() {
function unstick() {
document.querySelectorAll('header, nav, [role="banner"], .header, .navbar, .sticky, .fixed-top, [style*="position: fixed"], [style*="position:sticky"]').forEach(function(el) {
if (el.style.position === 'fixed' || el.style.position === 'sticky' || getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {
el.style.position = 'static';
el.style.top = 'auto';
el.style.zIndex = 'auto';
}
});
}
unstick();
var observer = new MutationObserver(unstick);
observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });
})();
}
} catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '
PR_26172_MASTER_009-deprecation-workflow by ToolboxAid · Pull Request #58 · ToolboxAid/HTML-JavaScript-Gaming · GitHub
Skip to content
PR_26172_MASTER_009-deprecation-workflow#58

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

 Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and
privacy statement. We’ll occasionally send you account related emails.

Already on GitHub?
Sign in
to your account

 Closed

ToolboxAid
wants to merge
1
commit into

from

 Closed


PR_26172_MASTER_009-deprecation-workflow
#58
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


1 commit


Select commit


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

View file


Open in desktop

Original file line numberDiff line numberDiff 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

View file


Open in desktop

Original file line numberDiff line numberDiff 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.

11 changes: 11 additions & 0 deletions
...ts/PR_26172_MASTER_009-deprecation-workflow-instruction-compliance-checklist.md

View file


Open in desktop

Original file line numberDiff line numberDiff 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`.

8 changes: 8 additions & 0 deletions
...dev/reports/PR_26172_MASTER_009-deprecation-workflow-manual-validation-notes.md

View file


Open in desktop

Original file line numberDiff line numberDiff 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

View file


Open in desktop

Original file line numberDiff line numberDiff 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

View file


Open in desktop

Original file line numberDiff line numberDiff 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

View file


Open in desktop

Original file line numberDiff line numberDiff 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

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Universal Dark Mode - works on any site
(function() {
var enabled = true;
function applyDarkMode() {
if (!enabled) return;
// Create style element if it doesn't exist
var style = document.getElementById('universal-dark-mode-style');
if (!style) {
style = document.createElement('style');
style.id = 'universal-dark-mode-style';
document.head.appendChild(style);
}
// Dark mode CSS - inverts colors but preserves images/video
style.textContent = '
/* Invert everything except media */
html {
filter: invert(1) hue-rotate(180deg) !important;
background: #1a1a2e !important;
}
/* Restore images, videos, iframes, canvas */
img, video, iframe, canvas, svg, picture, [style*="background-image"] {
filter: invert(1) hue-rotate(180deg) !important;
}
/* Preserve specific elements that should not be inverted */
.no-dark-mode, .no-dark-mode *,
[data-theme="light"], [data-theme="light"],
.ace_editor, .ace_editor *,
.CodeMirror, .CodeMirror *,
.monaco-editor, .monaco-editor *,
.markdown-body pre, .markdown-body pre *,
.highlight, .highlight *,
pre code, pre code * {
filter: none !important;
}
/* Fix common UI elements */
.modal, .popup, .dropdown-menu, .tooltip, .popover {
filter: invert(1) hue-rotate(180deg) !important;
background: #2d2d44 !important;
border-color: #444 !important;
}
/* Scrollbars */
::-webkit-scrollbar { background: #1a1a2e !important; }
::-webkit-scrollbar-thumb { background: #444 !important; }
::-webkit-scrollbar-thumb:hover { background: #555 !important; }
/* Selection */
::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }
::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }
';
}
function removeDarkMode() {
var style = document.getElementById('universal-dark-mode-style');
if (style) style.remove();
}
// Toggle with Alt+Shift+D
document.addEventListener('keydown', function(e) {
if (e.altKey && e.shiftKey && e.key === 'D') {
e.preventDefault();
enabled = !enabled;
if (enabled) {
applyDarkMode();
console.log('[Universal Dark Mode] Enabled');
} else {
removeDarkMode();
console.log('[Universal Dark Mode] Disabled');
}
}
});
// Apply on load
applyDarkMode();
// Re-apply on dynamic content
var observer = new MutationObserver(function(mutations) {
if (enabled && !document.getElementById('universal-dark-mode-style')) {
applyDarkMode();
}
});
observer.observe(document.head, { childList: true });
console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');
})();
}
} catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); }
})();
})();
PR_26172_MASTER_009-deprecation-workflow by ToolboxAid · Pull Request #58 · ToolboxAid/HTML-JavaScript-Gaming · GitHub
Skip to content
PR_26172_MASTER_009-deprecation-workflow#58

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

 Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and
privacy statement. We’ll occasionally send you account related emails.

Already on GitHub?
Sign in
to your account

 Closed

ToolboxAid
wants to merge
1
commit into

from

 Closed


PR_26172_MASTER_009-deprecation-workflow
#58
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


1 commit


Select commit


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

View file


Open in desktop

Original file line numberDiff line numberDiff 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

View file


Open in desktop

Original file line numberDiff line numberDiff 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.

11 changes: 11 additions & 0 deletions
...ts/PR_26172_MASTER_009-deprecation-workflow-instruction-compliance-checklist.md

View file


Open in desktop

Original file line numberDiff line numberDiff 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`.

8 changes: 8 additions & 0 deletions
...dev/reports/PR_26172_MASTER_009-deprecation-workflow-manual-validation-notes.md

View file


Open in desktop

Original file line numberDiff line numberDiff 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

View file


Open in desktop

Original file line numberDiff line numberDiff 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

View file


Open in desktop

Original file line numberDiff line numberDiff 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

View file


Open in desktop

Original file line numberDiff line numberDiff 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