|
1 | | -# Codex Changed Files - PR_26171_065-message-studio-parent-child-table-foundation |
| 1 | +PR_26171_044-idea-board-game-hub-project-flow |
2 | 2 |
|
3 | | -## Git Status Short |
4 | | -```text |
5 | | -M docs_build/dev/reports/coverage_changed_js_guardrail.txt |
6 | | - M docs_build/dev/reports/playwright_v8_coverage_report.txt |
7 | | - M tests/playwright/tools/MessagesTool.spec.mjs |
8 | | - M toolbox/messages/index.html |
9 | | - M toolbox/messages/message-tts-service-registry.js |
10 | | - M toolbox/messages/messages.js |
11 | | -?? docs_build/dev/reports/PR_26171_065-instruction-compliance-checklist.md |
12 | | -?? docs_build/dev/reports/PR_26171_065-manual-validation-notes.md |
13 | | -?? docs_build/dev/reports/PR_26171_065-message-studio-parent-child-table-foundation.md |
14 | | -?? docs_build/dev/reports/PR_26171_065-parent-child-table-checklist.md |
15 | | -?? docs_build/dev/reports/PR_26171_065-validation.md |
16 | | -``` |
17 | | - |
18 | | -## Scoped Diff Stat |
19 | | -```text |
20 | | -.../dev/reports/coverage_changed_js_guardrail.txt | 2 + |
21 | | - .../dev/reports/playwright_v8_coverage_report.txt | 43 +- |
22 | | - tests/playwright/tools/MessagesTool.spec.mjs | 317 +++++--------- |
23 | | - toolbox/messages/index.html | 29 +- |
24 | | - toolbox/messages/message-tts-service-registry.js | 8 +- |
25 | | - toolbox/messages/messages.js | 486 ++++++++++++--------- |
26 | | - 6 files changed, 415 insertions(+), 470 deletions(-) |
27 | | -``` |
28 | | - |
29 | | -## Changed Files |
30 | | -- toolbox/messages/index.html |
31 | | -- toolbox/messages/messages.js |
32 | | -- toolbox/messages/message-tts-service-registry.js |
33 | | -- tests/playwright/tools/MessagesTool.spec.mjs |
34 | | -- docs_build/dev/reports/coverage_changed_js_guardrail.txt |
35 | | -- docs_build/dev/reports/playwright_v8_coverage_report.txt |
36 | | -- docs_build/dev/reports/PR_26171_065-message-studio-parent-child-table-foundation.md |
37 | | -- docs_build/dev/reports/PR_26171_065-parent-child-table-checklist.md |
38 | | -- docs_build/dev/reports/PR_26171_065-validation.md |
39 | | -- docs_build/dev/reports/PR_26171_065-manual-validation-notes.md |
40 | | -- docs_build/dev/reports/PR_26171_065-instruction-compliance-checklist.md |
| 3 | +Changed files: |
| 4 | +- docs_build/pr/PR_26171_044-idea-board-game-hub-project-flow/PLAN_PR.md |
| 5 | +- docs_build/pr/PR_26171_044-idea-board-game-hub-project-flow/BUILD_PR.md |
| 6 | +- docs_build/pr/PR_26171_044-idea-board-game-hub-project-flow/APPLY_PR.md |
41 | 7 | - docs_build/dev/reports/codex_review.diff |
42 | 8 | - docs_build/dev/reports/codex_changed_files.txt |
| 9 | +- src/dev-runtime/persistence/tool-repositories/game-workspace-mock-repository.js |
| 10 | +- tests/playwright/tools/GameWorkspaceMockRepository.spec.mjs |
| 11 | +- tests/playwright/tools/IdeaBoardTableNotes.spec.mjs |
| 12 | +- tests/playwright/tools/ToolboxRoutePages.spec.mjs |
| 13 | +- toolbox/game-workspace/game-workspace.js |
| 14 | +- toolbox/game-workspace/index.html |
| 15 | +- toolbox/idea-board/index.html |
| 16 | +- toolbox/idea-board/index.js |
| 17 | + |
| 18 | +Requirement PASS evidence: |
| 19 | +- PASS: Create Project appears only for Ready ideas. Evidence: Idea Board actions remain Edit/Create Project/Delete for Ready rows and switch to Open in Game Hub/Archive after creation in targeted Playwright. |
| 20 | +- PASS: Create Project uses the shared Game Hub project contract. Evidence: Idea Board calls the existing game-workspace server repository client createGame method; Playwright asserts the createGame repository request occurs. |
| 21 | +- PASS: Create Project sets Idea status to Project. Evidence: targeted Playwright asserts the row status becomes Project. |
| 22 | +- PASS: Project ideas and notes are read-only. Evidence: targeted Playwright asserts no Edit/Delete/Add Note/note actions remain for the Project row. |
| 23 | +- PASS: Project row actions are Open in Game Hub and Archive. Evidence: targeted Playwright asserts exact action labels. |
| 24 | +- PASS: Archived ideas remain hidden by default and show Restore/Delete when visible. Evidence: targeted Playwright archives, shows Archived through the filter, and asserts Restore/Delete. |
| 25 | +- PASS: Game Hub shows creator-facing Project Information. Evidence: Game Hub HTML/JS replaces internal record display with Project Information and targeted Game Hub Playwright passes. |
| 26 | +- PASS: Game Hub shows read-only Source Idea fields. Evidence: Idea Board-to-Game Hub Playwright asserts Idea, Pitch, and Notes from the source idea render in Game Hub. |
| 27 | +- PASS: Game Hub avoids internal IDs, DB/API/mock/debug/seed wording in creator-facing display. Evidence: touched runtime files pass creator-visible text scan and targeted Playwright checks the navigated Game Hub main content. |
| 28 | +- PASS: Open in Game Hub navigates to the related project. Evidence: targeted Playwright waits for /toolbox/game-workspace/index.html?game=lantern-reef-* and asserts the linked project renders. |
| 29 | +- PASS: No Game Journey expansion beyond existing link/reference. Evidence: implementation does not add Game Journey creation or new journey wiring. |
43 | 30 |
|
44 | | -## Validation |
45 | | -- PASS: `node --check toolbox\messages\messages.js`. |
46 | | -- PASS: `node --check toolbox\messages\message-tts-service-registry.js`. |
47 | | -- PASS: `node --check tests\playwright\tools\MessagesTool.spec.mjs`. |
48 | | -- PASS: HTML inline style/script/event scan for `toolbox/messages/index.html`. |
49 | | -- PASS: `npx playwright test tests/playwright/tools/MessagesTool.spec.mjs --project=playwright --workers=1 --reporter=list`. |
50 | | -- PASS: `npm run test:workspace-v2` (legacy command name; user-facing language is Project Workspace). |
| 31 | +Validation evidence: |
| 32 | +- PASS: node --check toolbox/idea-board/index.js |
| 33 | +- PASS: node --check toolbox/game-workspace/game-workspace.js |
| 34 | +- PASS: node --check toolbox/game-workspace/game-workspace-api-client.js |
| 35 | +- PASS: node --check src/dev-runtime/persistence/tool-repositories/game-workspace-mock-repository.js |
| 36 | +- PASS: node --check tests/playwright/tools/IdeaBoardTableNotes.spec.mjs |
| 37 | +- PASS: node --check tests/playwright/tools/ToolboxRoutePages.spec.mjs |
| 38 | +- PASS: node --check tests/playwright/tools/GameWorkspaceMockRepository.spec.mjs |
| 39 | +- PASS: npx playwright test tests/playwright/tools/IdeaBoardTableNotes.spec.mjs --project=playwright --workers=1 --reporter=line |
| 40 | +- PASS: npx playwright test tests/playwright/tools/ToolboxRoutePages.spec.mjs --project=playwright --workers=1 --reporter=line --grep "Idea Board launches" |
| 41 | +- PASS: npx playwright test tests/playwright/tools/GameWorkspaceMockRepository.spec.mjs --project=playwright --workers=1 --reporter=line --grep "Game Hub" |
| 42 | +- PASS: npm run test:workspace-v2 |
| 43 | +- PASS: git diff --check |
51 | 44 |
|
52 | | -## ZIP |
53 | | -- Path: `tmp/PR_26171_065-message-studio-parent-child-table-foundation_delta.zip`. |
| 45 | +Git workflow: |
| 46 | +- Current branch: codex/pr-26171-044-idea-board-game-hub-project-flow |
| 47 | +- Created branch: codex/pr-26171-044-idea-board-game-hub-project-flow |
| 48 | +- Push result: to be reported after push |
| 49 | +- PR URL: to be reported after PR creation |
| 50 | +- Merge result: to be reported after merge |
| 51 | +- Final main commit: to be reported after returning to main and pulling latest |
0 commit comments