Merge branch 'main' of https://github.com/ToolboxAid/HTML-JavaScript-Gaming

# Conflicts:
#	docs_build/dev/reports/codex_changed_files.txt
#	docs_build/dev/reports/codex_review.diff

Author: ToolboxAid

docs_build/dev/PROJECT_INSTRUCTIONS.md

@@ -358,6 +358,22 @@ Prohibited product-data ownership:
 
 Toolbox and Admin tool metadata must use a shared DB-backed tool metadata source for `toolKey`, `toolName`, `group`, `path`, `order`, and `status`. Browser pages may render metadata returned by the API/service contract, but they must not own a separate runtime copy of that metadata.
 
+## DATABASE DIRECTION
+
+SQLite is deprecated.
+Postgres is authoritative.
+
+Rules:
+- New database work must target Postgres.
+- Local API -> Postgres is the required direction.
+- New PRs must not introduce SQLite persistence.
+- Do not add new SQLite services.
+- Do not add new SQLite DDL.
+- Do not add new SQLite seed data.
+- Do not add new SQLite runtime persistence.
+- Legacy SQLite references may remain only as documented technical debt when they already exist.
+- Browser code must not own product data or generate authoritative persistence keys.
+
 ## DEV RUNTIME BOUNDARY
 
 All mock/dev-only runtime implementation must live under `src/dev-runtime/`.
@@ -1874,3 +1890,46 @@ Rules:
 - Do not introduce new report/test prose that describes the current user-facing experience as `Workspace V2`.
 - Existing package scripts such as `npm run test:workspace-v2`, legacy lane identifiers, and historical test suite names may remain until renamed by a dedicated cleanup PR.
 - When a report invokes a legacy command name such as `npm run test:workspace-v2`, the report must explain that the command name is legacy and the user-facing product language is `Game Hub`.
+
+## CODEX GIT WORKFLOW OWNERSHIP
+
+Codex owns Git execution for implementation PRs.
+
+Required workflow:
+1. Verify current branch.
+2. Checkout main.
+3. Pull latest main.
+4. Verify clean repository.
+5. Create PR branch.
+6. Implement changes.
+7. Stage only scoped files.
+8. Commit.
+9. Push branch to GitHub.
+10. Create Pull Request automatically.
+11. Resolve merge conflicts if encountered.
+12. Re-run validation after conflict resolution.
+13. Merge PR.
+14. Return to main.
+15. Pull latest main.
+16. Continue to next approved PR.
+
+Rules:
+- Do not ask the user if a PR should be created.
+- Do not ask the user if a branch should be pushed.
+- Treat PR creation as required.
+- Treat branch push as required.
+- Treat merge as required after validation passes.
+- If GitHub prompts `Would you like to create a Pull Request?`, answer YES automatically.
+- If merge conflicts occur:
+  - preserve latest main
+  - preserve PR scope
+  - avoid unrelated cleanup
+  - revalidate before merge
+
+Required Git workflow report fields:
+- current branch
+- created branch
+- push result
+- PR URL
+- merge result
+- final main commit

docs_build/dev/commit_comment.txt

@@ -1 +1 @@
-Add Messages TTS profile foundation - PR_26171_008
+Add browser-local Messages speech preview - PR_26171_010

docs_build/dev/reports/PR_26171_010-message-speech-preview-manual-validation.md

@@ -0,0 +1,9 @@
+# PR_26171_010 Manual Validation Notes
+
+- Confirmed the Messages inspector includes a Speech Preview section.
+- Confirmed selected message preview uses the active TTS profile voice/language and the message emotion profile delivery values.
+- Confirmed active segment preview speaks active segments in display order.
+- Confirmed segment preview applies each segment emotion profile's delivery values.
+- Confirmed Stop Preview calls cancel and reports stopped status.
+- Confirmed missing browser speech synthesis reports an actionable diagnostic.
+- Confirmed no server TTS, audio generation, audio persistence, runtime playback API, or preview history storage was introduced.

docs_build/dev/reports/PR_26171_010-message-speech-preview-validation.txt

@@ -0,0 +1,33 @@
+PR_26171_010-message-speech-preview validation
+
+Branch:
+PASS pr/PR_26171_010-message-speech-preview
+
+Syntax:
+PASS node --check toolbox/messages/messages.js
+PASS node --check tests/playwright/tools/MessagesTool.spec.mjs
+
+Targeted Playwright:
+PASS npx playwright test tests/playwright/tools/MessagesTool.spec.mjs --project=playwright --workers=1 --reporter=list
+Result: 2 passed.
+
+Preview behavior:
+PASS selected full-message preview produced a stubbed utterance.
+PASS active ordered segment preview produced ordered stubbed utterances.
+PASS segment preview used emotion profile volume, pitch, and rate.
+PASS Stop Preview called cancel.
+PASS missing speech synthesis displayed an actionable diagnostic.
+
+Workspace lane:
+FAIL npm run test:workspace-v2
+Failure file: tests/playwright/tools/RootToolsFutureState.spec.mjs
+Failures:
+- Toolbox accordion control-card count was 0.
+- Header alphabetical expectation failed around Game Hub/Game Journey ordering.
+- Non-Messages pages reported failed requests to session, platform banner, registry, and toolbox constants APIs.
+
+Whitespace:
+PASS git diff --check on touched files. CRLF warnings only.
+
+Disposition:
+MERGE WITH RECORDED WORKSPACE FAILURE. PR_010 targeted validation passed. The required workspace-v2 lane failed outside the Messages speech preview scope and is documented for follow-up.

docs_build/dev/reports/PR_26171_010-message-speech-preview.md

@@ -0,0 +1,59 @@
+# PR_26171_010-message-speech-preview
+
+## Branch Validation
+
+- Branch: `pr/PR_26171_010-message-speech-preview`
+- Source: clean `main` after PR_26171_008 merge and push.
+- Status: PASS for branch setup and scoped implementation.
+
+## Requirement Checklist
+
+- PASS: Added a Theme V2 Speech Preview section to the Messages tool.
+- PASS: Added selected full-message preview.
+- PASS: Added selected ordered active-segment preview.
+- PASS: Preview uses selected/default active TTS Profile settings for language and voice.
+- PASS: Segment preview uses each segment's Emotion Profile settings for volume, pitch, and rate.
+- PASS: Browser-local preview uses `speechSynthesis` and `SpeechSynthesisUtterance` when available.
+- PASS: Missing browser speech support shows a visible actionable diagnostic.
+- PASS: Stop Preview calls `speechSynthesis.cancel()` and updates status.
+- PASS: Preview logic lives in external `toolbox/messages/messages.js`.
+- PASS: Playwright stubs browser speech synthesis and validates preview payloads without real audio output.
+- PASS: No server-side TTS generation, audio file generation, audio persistence, AI voices, voice adapters, runtime playback API, dialog trees, localization, translation, game runtime integration, or preview history storage was added.
+- PASS: No new API/DB persistence was added.
+- PASS: No inline styles, style blocks, inline scripts, or inline event handlers were introduced.
+- FAIL: Required `npm run test:workspace-v2` lane failed in existing `RootToolsFutureState.spec.mjs` coverage outside the Messages speech preview scope.
+
+## Validation Lane Report
+
+- PASS: `node --check toolbox/messages/messages.js`
+- PASS: `node --check tests/playwright/tools/MessagesTool.spec.mjs`
+- PASS: `npx playwright test tests/playwright/tools/MessagesTool.spec.mjs --project=playwright --workers=1 --reporter=list`
+- PASS: Validated preview selected message payload with stubbed speech synthesis.
+- PASS: Validated preview active ordered segment payloads with emotion profile settings.
+- PASS: Validated Stop Preview cancel behavior.
+- PASS: Validated unavailable speech synthesis diagnostic.
+- PASS: `git diff --check` on touched files.
+- FAIL: `npm run test:workspace-v2`
+
+## Workspace-V2 Failure Summary
+
+`npm run test:workspace-v2` failed in `tests/playwright/tools/RootToolsFutureState.spec.mjs`:
+
+- Toolbox accordion `.control-card` count was `0`.
+- Header alphabetical expectation failed around `Game Hub` and `Game Journey`.
+- Non-Messages pages reported failed requests to session, platform banner, registry, and toolbox constants APIs.
+
+These failures were already present for the batch and are outside PR_010's browser-local Messages speech preview scope.
+
+## Manual Validation Notes
+
+- Selected message preview reports a preview request and produces one stubbed utterance.
+- Active ordered segment preview reports a preview request and produces ordered stubbed utterances.
+- Segment utterances use the segment emotion profile's volume, pitch, and rate.
+- Stop Preview reports stopped status and calls cancel.
+- Missing browser speech synthesis shows an actionable diagnostic.
+- No audio is generated or persisted.
+
+## Samples Decision
+
+- Full samples smoke was not run because samples are out of scope for browser-local Messages speech preview.

docs_build/dev/reports/PR_26171_012-message-tool-postgres-table-layout-cleanup-manual-validation.md

@@ -0,0 +1,11 @@
+# Manual Validation Notes
+
+- Opened `/tools/messages/index.html` through the repo Playwright server.
+- Verified browser-rendered document title: `Message Studio - GameFoundryStudio`.
+- Verified browser-rendered H1: `Message Studio`.
+- Verified visible category/category controls are absent from the Message Studio page.
+- Verified Message Rows appears before row detail editing.
+- Verified visible row actions use `Add Message Row`, `Update Row`, and `Disable Row`.
+- Verified Emotion Profiles and TTS Profiles appear in the center work surface.
+- Verified Speech Preview remains display-only in the inspector.
+- Verified no provider calls, audio generation, playback generation, delete behavior, or new SQLite persistence was introduced.

docs_build/dev/reports/PR_26171_012-message-tool-postgres-table-layout-cleanup-validation.txt

@@ -0,0 +1,33 @@
+Validation report for PR_26171_012-message-tool-postgres-table-layout-cleanup
+
+Branch and git validation:
+- git branch --show-current: main
+- git checkout main: PASS
+- git -c http.sslBackend=schannel pull origin main: PASS, already up to date
+- git status before branch: PASS, clean
+- git checkout -b pr/26171-012-message-tool-postgres-table-layout-cleanup: PASS
+
+Static validation:
+- node --check toolbox/messages/messages.js: PASS
+- node --check src/shared/toolbox/tool-metadata-inventory.js: PASS
+- node --check src/dev-runtime/server/local-api-router.mjs: PASS
+- node --check tests/playwright/tools/MessagesTool.spec.mjs: PASS
+- node --check tests/playwright/tools/RootToolsFutureState.spec.mjs: PASS
+- git diff --check for touched files: PASS with CRLF warnings only
+
+Targeted UI/static validation:
+- Category/category visible UI removed from toolbox/messages/index.html: PASS
+- Save Message / New Message / Message Editor visible copy removed from toolbox/messages/index.html: PASS
+- Inline style/script/event handler check for toolbox/messages/index.html: PASS
+- Rendered tool title/H1 after registry override: PASS, Message Studio
+
+Playwright:
+- npx playwright test tests/playwright/tools/MessagesTool.spec.mjs --project=playwright --workers=1 --reporter=list: PASS, 2 passed
+- npm run test:workspace-v2: WARN, 3 passed / 2 failed
+  - Remaining failures are existing header ordering expectations:
+    - root tools surface links current tool pages without old_* routes
+    - common header renders primary navigation order across active pages
+  - Both failures compare expected alphabetical Toolbox menu order `Game Hub` before `Game Journey` against current UI order `Game Journey` before `Game Hub`.
+
+Samples:
+- Full samples validation: SKIP, no samples changed.

docs_build/dev/reports/PR_26171_012-message-tool-postgres-table-layout-cleanup.md

@@ -0,0 +1,59 @@
+# PR_26171_012-message-tool-postgres-table-layout-cleanup
+
+## Branch Validation
+
+- Current branch before implementation: `main`
+- Required PR branch: `pr/26171-012-message-tool-postgres-table-layout-cleanup`
+- Branch source: pulled latest `origin/main` before branch creation
+- Clean repo before branch creation: PASS
+
+## Summary
+
+Message Studio now reflects the approved Postgres direction and creator-facing row workflow. The tool no longer exposes category management or category selection in the UI, Emotion Profiles and TTS Profiles are in the center work surface, and the message editor is organized around a Message Rows table with Add/Update/Disable row language.
+
+## Requirement Checklist
+
+| Requirement | Result | Evidence |
+| --- | --- | --- |
+| Update repo guidance to state SQLite is deprecated. | PASS | Added `POSTGRES DATABASE DIRECTION` to `docs_build/dev/PROJECT_INSTRUCTIONS.md`. |
+| New database work must target Postgres / Local API direction. | PASS | Guidance now requires new persistence through Local API / Postgres direction only. |
+| Do not add new SQLite services, DDL, seed, or runtime persistence. | PASS | No SQLite service, DDL, seed, or persistence file was added. Existing legacy SQLite service remains unchanged as technical debt. |
+| Remove Category/category UI from the left column. | PASS | `toolbox/messages/index.html` no longer contains visible category controls or category tables. |
+| Do not introduce category replacement. | PASS | No new classification form was introduced; left column documents Tags as the future direction only. |
+| Move Emotion Profiles into center column. | PASS | Emotion Profiles form/table now render in the center work surface. |
+| Move TTS Profiles into center column. | PASS | TTS Profiles form/table now render in the center work surface. |
+| Keep Speech Preview display-only. | PASS | Speech Preview remains in the inspector and no provider/audio generation behavior was added. |
+| Change Message Editor to table-driven workflow. | PASS | Message Rows table now appears before row detail editing. |
+| Add new row/action at bottom of table. | PASS | Message Rows table includes `Add Message Row` below the table. |
+| Avoid Save Message / New Message language. | PASS | Visible tool copy now uses `Add Message Row`, `Update Row`, and `Disable Row`. |
+| Preferred visible label is Message Studio. | PASS | Page title/H1 and source-controlled Toolbox metadata now use `Message Studio`. |
+| Do not rename routes/folders. | PASS | Existing `messages` route/folder/id remain unchanged. |
+
+## Root Cause / Notes
+
+The tool was still presenting the earlier Messages foundation UI: category management in the left rail, Emotion/TTS profile management outside the primary work surface, and form-first `Save Message` / `New Message` actions. The server-backed toolbox registry also retained the old `Messages` label because `messages` was not included in the source-controlled metadata sync allowlist.
+
+This PR keeps the current legacy category key only as an internal compatibility value required by the existing API contract. The category UI is removed from the creator surface and the Postgres migration direction is documented.
+
+## Impacted Lane
+
+- Tool/UI lane: Message Studio Theme V2 surface
+- Server metadata lane: Toolbox registry source-controlled metadata sync
+- Documentation/governance lane: database direction guidance
+
+## Skipped Lanes
+
+- Samples: skipped, no sample runtime or sample JSON changed.
+- Provider/audio generation: skipped, explicitly out of scope.
+- Full samples smoke: skipped, out of scope.
+
+## Playwright Result
+
+- PASS: `npx playwright test tests/playwright/tools/MessagesTool.spec.mjs --project=playwright --workers=1 --reporter=list`
+- WARN: `npm run test:workspace-v2` still fails on existing header ordering expectations where the UI renders `Game Journey` before `Game Hub`. The PR-caused Toolbox count expectation was updated from `13/42` to `14/43`.
+
+## Manual Validation Notes
+
+- Confirmed rendered `/tools/messages/index.html` title and H1 resolve to `Message Studio` after the server registry override.
+- Confirmed visible category controls are absent from `toolbox/messages/index.html`.
+- Confirmed no inline style blocks, inline scripts, or inline event handlers were added.

docs_build/dev/reports/PR_26171_014-message-studio-table-governance-manual-validation.txt

@@ -0,0 +1,14 @@
+PR_26171_014-message-studio-table-governance manual validation notes
+
+Reviewed Message Studio page structure after the table governance conversion:
+- Message Rows use table columns Name, Primary Emotion, Segments, Tags, Status, Actions.
+- Add Message Row opens an inline editable row.
+- Edit opens inline row editing.
+- Disable actions are present for active rows and no Delete actions are present.
+- Selected Message Row renders a nested Segment Table.
+- Segment rows support Edit, Move Up, Move Down, and Disable.
+- Emotion Profiles and TTS Profiles use inline table rows for add/edit.
+- Speech Preview is display-only and has no playback buttons.
+
+No browser category UI is present in toolbox/messages.
+No manual provider/audio playback was attempted because it is out of scope.

docs_build/dev/reports/PR_26171_014-message-studio-table-governance-validation.txt

@@ -0,0 +1,22 @@
+PR_26171_014-message-studio-table-governance validation report
+
+Branch:
+PASS - Started from main.
+
+Static and syntax checks:
+PASS - node --check toolbox/messages/messages.js
+PASS - node --check toolbox/messages/messages-api-client.js
+PASS - node --check src/dev-runtime/messages/messages-sqlite-service.mjs
+PASS - node --check tests/playwright/tools/MessagesTool.spec.mjs
+PASS - git diff --check
+
+Targeted runtime/static assertions:
+PASS - No category references remain in toolbox/messages browser files.
+PASS - No speech synthesis or preview action hooks remain in toolbox/messages browser files.
+PASS - No Delete labels exist in Message Studio browser files or the targeted Playwright spec.
+
+Playwright:
+PASS - npx playwright test tests/playwright/tools/MessagesTool.spec.mjs --workers=1 --reporter=list
+
+Samples:
+SKIPPED - Full samples smoke was out of scope.