Rebuild Game Design with purpose flow capability demos and validation - PR_26155_068-076-game-design-rebuild

Author: ToolboxAid

assets/theme-v2/images/magic-panel.png

@@ -6,8 +6,11 @@ Missing changed runtime JS files are WARN, not FAIL.
 Source: Playwright/Chromium built-in V8 coverage from the active Playwright run.
 
 Changed runtime JS files considered:
-(0%) toolbox/toolRegistry.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
-(88%) toolbox/tools-page-accordions.js - executed lines 494/494; executed functions 30/34
+(0%) toolbox/project-workspace/project-workspace.js - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
+(79%) toolbox/project-workspace/project-workspace-mock-repository.js - executed lines 384/384; executed functions 26/33
+(89%) toolbox/tools-page-accordions.js - executed lines 1690/1690; executed functions 51/57
+(95%) toolbox/game-design/game-design-mock-repository.js - executed lines 310/310; executed functions 36/38
+(96%) toolbox/game-design/game-design.js - executed lines 255/255; executed functions 23/24
 
 Guardrail warnings:
-(0%) toolbox/toolRegistry.js - WARNING: changed runtime JS file missing from coverage; advisory only
+(0%) toolbox/project-workspace/project-workspace.js - WARNING: changed runtime JS file missing from coverage; advisory only

docs_build/dev/reports/coverage_changed_js_guardrail.txt

@@ -0,0 +1,22 @@
+# Game Configuration Ready Gate
+
+PR: PR_26155_068-076-game-design-rebuild
+
+Status: READY
+
+## Gate
+
+Game Configuration implementation can start after this bundle because Game Design now provides:
+- active Project Workspace project context
+- project purpose flow
+- Game Type, Genre, and Play Style fields
+- actionable validation overlay
+- mock repository save/update behavior
+- Toolbox Progress and Build Path handoff copy
+- targeted Game Design MSJ coverage
+
+## Boundary
+
+This bundle did not start Game Configuration implementation.
+
+Next implementation PR should build Game Configuration against the same mock-repository approach and consume Game Design readiness as project-scoped input only.

docs_build/dev/reports/game-configuration-ready-gate.md

@@ -0,0 +1,26 @@
+# Game Design Capability Demo Authoring
+
+PR: PR_26155_068-076-game-design-rebuild
+
+Status: PASS
+
+## Behavior
+
+Capability demos remain Project Workspace projects.
+
+Seeded capability demo projects:
+- Camera Follow Demo
+- Collision Demo
+- Gravity Demo
+
+Game Design stores capability demo authoring metadata in `game_design_capability_demos` by `projectId`; it does not create a separate demo project system.
+
+## Manual Test Notes
+
+Open `toolbox/game-design/index.html?project=gravity-demo`.
+
+Expected:
+- Gravity Demo is the active project.
+- Project Purpose is `Capability Demo`.
+- Capability demos are listed as Project Workspace projects.
+- Save Game Design keeps `capabilityDemoAuthoring` true in the output.

docs_build/dev/reports/game-design-capability-demo-authoring.md

@@ -0,0 +1,28 @@
+# Game Design Project Purpose Flow
+
+PR: PR_26155_068-076-game-design-rebuild
+
+Status: PASS
+
+## Purpose Flow
+
+Game Design reads the active project purpose from the Project Workspace mock repository and displays it in the Project Context accordion.
+
+Supported seeded purposes:
+- Game Project
+- Capability Demo
+- Learning Project
+- Template Project
+
+Game Design does not own project identity. Project identity and purpose remain owned by Project Workspace.
+
+## Manual Test Notes
+
+Open:
+- `toolbox/game-design/index.html`
+- `toolbox/game-design/index.html?project=gravity-demo`
+
+Expected:
+- default route shows Demo Project with purpose `Game Project`.
+- `?project=gravity-demo` shows Gravity Demo with purpose `Capability Demo`.
+- Project purpose is reflected in the saved Game Design output.

docs_build/dev/reports/game-design-project-purpose-flow.md

@@ -0,0 +1,46 @@
+# Game Design Repository Contract
+
+PR: PR_26155_068-076-game-design-rebuild
+
+Status: PASS
+
+## Contract
+
+Game Design now uses `toolbox/game-design/game-design-mock-repository.js` as an in-memory, SQL-shaped mock repository abstraction.
+
+Tables:
+- `game_design_documents`
+- `game_design_validation_items`
+- `game_design_capability_demos`
+
+The repository consumes Project Workspace mock project context through `createProjectWorkspaceMockRepository()` and never introduces a real database, auth, cloud, or persistence layer.
+
+## Fields
+
+`game_design_documents` stores:
+- `id`
+- `projectId`
+- `projectPurpose`
+- `gameType`
+- `genre`
+- `playStyle`
+- `designSummary`
+- `capabilityDemoAuthoring`
+- `capabilityDemoNotes`
+- `status`
+- `updatedAt`
+
+`game_design_validation_items` stores actionable missing-field rows by `projectId`.
+
+`game_design_capability_demos` stores project-owned capability demo authoring metadata by `projectId`.
+
+## Validation Notes
+
+Impacted lane: `game-design`.
+
+Skipped lanes:
+- `project-workspace`: Game Design depends on active project context, but the new Game Design lane directly exercises that dependency through the Project Workspace mock repository.
+- `workspace-contract`: skipped because shared launch/header/template wiring did not change.
+- `samples`: skipped because samples are out of scope.
+
+Theme V2 gap findings: none.

docs_build/dev/reports/game-design-repository-contract.md

@@ -0,0 +1,51 @@
+# Game Design Targeted MSJ Tests
+
+PR: PR_26155_068-076-game-design-rebuild
+
+Status: PASS
+
+## Impacted Lane
+
+Impacted lane: `game-design`.
+
+Added:
+- `npm run test:lane:game-design`
+- `tests/playwright/tools/GameDesignMockRepository.spec.mjs`
+
+Executed command:
+
+`node scripts/run-targeted-test-lanes.mjs --lane game-design ...`
+
+The lane runner was invoked directly so generated operational reports could be redirected outside the repo, except for the repo-owned Playwright structure and V8 coverage reports.
+
+## Covered Behavior
+
+Targeted Playwright tests verify:
+- missing project context shows an actionable overlay.
+- save/update works against the Game Design mock repository.
+- validation is actionable and clears when required fields are complete.
+- capability demos remain Project Workspace projects.
+- Toolbox Progress and Build Path show the Game Design handoff.
+- no console errors.
+
+Result:
+- 4 passed.
+
+## Coverage Notes
+
+Generated:
+- `docs_build/dev/reports/playwright_v8_coverage_report.txt`
+- `docs_build/dev/reports/coverage_changed_js_guardrail.txt`
+
+Changed Game Design runtime files were covered:
+- `toolbox/game-design/game-design.js`
+- `toolbox/game-design/game-design-mock-repository.js`
+
+Advisory WARN remains for previously changed Project Workspace runtime JS that was not exercised by this Game Design lane.
+
+## Skipped Lanes
+
+Skipped:
+- Project Workspace full lane: Game Design tests directly exercise the needed active-project context.
+- Workspace contract lane: no shared launch/contract wiring changed.
+- Full samples smoke: samples are out of scope.

docs_build/dev/reports/game-design-targeted-msj-tests.md

@@ -0,0 +1,31 @@
+# Game Design Toolbox Progress Integration
+
+PR: PR_26155_068-076-game-design-rebuild
+
+Status: PASS
+
+## Integration
+
+`toolbox/tools-page-accordions.js` was updated only for the Game Design progress checklist text.
+
+Game Design Progress now reports:
+- Project purpose context required
+- Game type, genre, and play style required
+- Validation overlay hands off to Game Configuration
+
+Build Path remains the existing visual path:
+- Project Workspace
+- Game Design
+- Game Configuration
+- Required Tool Path
+- Build Game
+- Game Testing
+- Publish
+
+No database, persistence, auth, or shared launch wiring was added.
+
+## Validation Notes
+
+The targeted Game Design lane checks Toolbox Progress and Build Path views because those views were touched.
+
+`npm run test:workspace-v2` was skipped because shared launch/contract wiring did not change. The command name remains legacy; user-facing language is Project Workspace.

docs_build/dev/reports/game-design-toolbox-progress-integration.md

@@ -0,0 +1,46 @@
+# Game Design Type Genre Model
+
+PR: PR_26155_068-076-game-design-rebuild
+
+Status: PASS
+
+## Model
+
+Game Design adds runtime-backed fields for:
+- Game Type
+- Genre
+- Play Style
+- Design Summary
+
+Game type values:
+- 2D Platformer
+- Arcade Action
+- Capability Demo
+- Narrative Adventure
+- Puzzle
+- RPG
+- Sandbox
+- Simulation
+- Strategy
+
+Genre values:
+- Action
+- Adventure
+- Educational
+- Fantasy
+- Sci-Fi
+- Sports
+- Strategy
+- Utility
+
+Play style values:
+- Competitive
+- Cooperative
+- Guided Tutorial
+- Sandbox
+- Single Player
+- Turn-Based
+
+## Validation
+
+The targeted Game Design Playwright lane verifies save/update behavior by selecting Game Type, Genre, and Play Style, saving, then updating Genre and confirming the output changes.

docs_build/dev/reports/game-design-type-genre-model.md

@@ -0,0 +1,28 @@
+# Game Design Validation And Overlay
+
+PR: PR_26155_068-076-game-design-rebuild
+
+Status: PASS
+
+## Validation Overlay
+
+Game Design now shows actionable validation for:
+- missing Project Workspace context
+- missing Game Type
+- missing Genre
+- missing Play Style
+- missing Design Summary
+
+The missing project context overlay appears when opened with:
+
+`toolbox/game-design/index.html?project=missing`
+
+The overlay uses existing Theme V2 cards, status text, and buttons. No new CSS was added.
+
+## Validation Notes
+
+Targeted Playwright verified:
+- missing project context shows an overlay.
+- Save Game Design without project context logs an actionable failure.
+- completing all required fields hides the validation overlay.
+- no console errors.