Skip to content

Commit 4953530

Browse files
committed
Remove Game Crew mock-db expansion from PR059
1 parent b4b97a9 commit 4953530

10 files changed

Lines changed: 121 additions & 294 deletions

‎docs_build/dev/reports/PR_26177_ALFA_059-game-crew-foundation_branch-validation.md‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,4 +5,5 @@
55
- Game Crew API/DB validation: PASS
66
- Retired Alfa mock repository files absent: PASS
77
- Flat Tags schema/provider alignment: PASS
8+
- Mock-db/Admin viewer expansion removed from branch delta: PASS
89
- Worktree validation before package: PASS pending final status check

‎docs_build/dev/reports/PR_26177_ALFA_059-game-crew-foundation_manual-validation-notes.md‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,3 +5,4 @@
55
- Guest member actions redirect to `account/sign-in.html`; direct API writes return 401.
66
- Tags support add, assign to current game, remove from current game, refresh/reload persistence, edit, delete, and guest redirect on this branch.
77
- Active runtime no longer imports retired Tags/Game Design/Game Configuration mock repository files.
8+
- The PR no longer changes `mock-db-store.js` or Admin DB viewer table wiring for the new Alfa tools.

‎docs_build/dev/reports/PR_26177_ALFA_059-game-crew-foundation_report.md‎

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,13 +5,14 @@
55
- Reworked the shared Alfa tool runtime in this branch so Tags, Game Design, and Game Configuration no longer route through retired mock repositories.
66
- Restored flat DB-backed Tags contract (`project_tags`, `project_tag_assignments`) and removed old `workspace_tag_records` metadata from active code paths.
77
- Added/kept guardrail coverage that fails when retired Alfa mock repository files exist or are imported.
8+
- Removed the accidental `mock-db-store.js` and `AdminDbViewer.spec.mjs` expansion from this PR; active Game Crew and Tags behavior is validated through API/database services.
89

910
## Validation
1011
- PASS - `node --check src/dev-runtime/toolbox-api/alfa-tool-services.mjs`
1112
- PASS - `node --check src/dev-runtime/server/local-api-router.mjs`
1213
- PASS - `node --test tests/dev-runtime/DevRuntimeBoundary.test.mjs`
13-
- PASS - `npx playwright test tests/playwright/tools/GameCrewFoundation.spec.mjs --project=playwright`
14-
- PASS - `npx playwright test tests/playwright/tools/TagsTool.spec.mjs --project=playwright --workers=1`
14+
- PASS - `npx playwright test tests/playwright/tools/GameCrewFoundation.spec.mjs --project=playwright --workers=1 --reporter=line` (5 passed)
15+
- PASS - `npx playwright test tests/playwright/tools/TagsTool.spec.mjs --project=playwright --workers=1 --reporter=line` (4 passed)
1516

1617
## Removed Mock Repository Files
1718
- `src/dev-runtime/persistence/tool-repositories/tags-mock-repository.js`

‎docs_build/dev/reports/PR_26177_ALFA_059-game-crew-foundation_requirement-checklist.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,6 @@
99
| Remove member persists through API/DB | PASS | API and page coverage passed. |
1010
| Guest add/remove redirects or 401s | PASS | Guest UI/API coverage passed. |
1111
| No mock repository source of truth for Tags/Design/Configuration | PASS | Files removed; guardrail passed. |
12-
| No mock-db source of truth for Tags | PASS | Active metadata uses `project_tags` and `project_tag_assignments`. |
12+
| No mock-db source of truth for Tags/Game Crew | PASS | Accidental `mock-db-store.js` and Admin DB viewer expansion was removed; active behavior uses API/database services. |
1313
| Browser does not own product data | PASS | Writes route through API/database services. |
1414
| No JSON source of truth, SQLite, or tmp runtime dependency | PASS | None added. |

‎docs_build/dev/reports/PR_26177_ALFA_059-game-crew-foundation_validation-lane.md‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
- PASS - `node --check src/dev-runtime/toolbox-api/alfa-tool-services.mjs`
44
- PASS - `node --check src/dev-runtime/server/local-api-router.mjs`
55
- PASS - `node --test tests/dev-runtime/DevRuntimeBoundary.test.mjs`
6-
- PASS - `npx playwright test tests/playwright/tools/GameCrewFoundation.spec.mjs --project=playwright`
7-
- PASS - `npx playwright test tests/playwright/tools/TagsTool.spec.mjs --project=playwright --workers=1`
6+
- PASS - `npx playwright test tests/playwright/tools/GameCrewFoundation.spec.mjs --project=playwright --workers=1 --reporter=line`
7+
- PASS - `npx playwright test tests/playwright/tools/TagsTool.spec.mjs --project=playwright --workers=1 --reporter=line`
88

9-
Notes: Initial combined Game Crew/Tags run failed before flat Tags provider/schema alignment was restored on this branch. After correction, focused reruns passed.
9+
Notes: A parallel Tags run hit a Playwright artifact cleanup race after Game Crew passed; rerunning Tags by itself passed.
Lines changed: 8 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,8 @@
1-
docs_build/dev/reports/codex_review.diff
2-
docs_build/dev/reports/coverage_changed_js_guardrail.txt
3-
docs_build/dev/reports/playwright_v8_coverage_report.txt
4-
docs_build/dev/reports/PR_26177_ALFA_059-game-crew-foundation_branch-validation.md
5-
docs_build/dev/reports/PR_26177_ALFA_059-game-crew-foundation_manual-validation-notes.md
6-
docs_build/dev/reports/PR_26177_ALFA_059-game-crew-foundation_report.md
7-
docs_build/dev/reports/PR_26177_ALFA_059-game-crew-foundation_requirement-checklist.md
8-
docs_build/dev/reports/PR_26177_ALFA_059-game-crew-foundation_validation-lane.md
9-
src/dev-runtime/auth/provider-contract-stubs.mjs
10-
src/dev-runtime/persistence/mock-db-store.js
11-
tests/playwright/tools/AdminDbViewer.spec.mjs
12-
tests/playwright/tools/TagsTool.spec.mjs
1+
M docs_build/dev/reports/PR_26177_ALFA_059-game-crew-foundation_branch-validation.md
2+
M docs_build/dev/reports/PR_26177_ALFA_059-game-crew-foundation_manual-validation-notes.md
3+
M docs_build/dev/reports/PR_26177_ALFA_059-game-crew-foundation_report.md
4+
M docs_build/dev/reports/PR_26177_ALFA_059-game-crew-foundation_requirement-checklist.md
5+
M docs_build/dev/reports/PR_26177_ALFA_059-game-crew-foundation_validation-lane.md
6+
M docs_build/dev/reports/playwright_v8_coverage_report.txt
7+
M src/dev-runtime/persistence/mock-db-store.js
8+
M tests/playwright/tools/AdminDbViewer.spec.mjs

0 commit comments

Comments
 (0)