|
| 1 | +# PR_26174_BRAVO_022-use-active-tts-profiles-in-messages |
| 2 | + |
| 3 | +## Branch Validation |
| 4 | +PASS - Current branch is `team/BRAVO/messages`; main was not checked out, merged, or targeted. |
| 5 | + |
| 6 | +## Summary |
| 7 | +- Added a shared Text To Speech profile store/contract helper for active saved TTS profiles. |
| 8 | +- Text To Speech now persists its own profile/emotion model for sibling tools. |
| 9 | +- Messages now loads saved TTS profiles, syncs profile/emotion names to Local API-owned keys, filters Sentence emotions by selected parent profile, and keeps playback off Text To Speech Local Preview validation. |
| 10 | +- Removed the dev-runtime import of the browser Text To Speech UI module and removed separate Message Studio default TTS profile builders. |
| 11 | + |
| 12 | +## Requirement Checklist |
| 13 | +PASS - /toolbox/messages/ loads active saved TTS Profiles from /toolbox/text-to-speech/ via `tts-profile-store.js`. |
| 14 | +PASS - Parent Message TTS Profile dropdown is populated from saved active TTS profiles. |
| 15 | +PASS - Child Sentence Emotion dropdown is populated only from the selected parent profile's saved emotions. |
| 16 | +PASS - Parent Play remains in Message row Actions and resolves ordered Sentences through the selected parent profile. |
| 17 | +PASS - Child Play remains in Sentence row Actions and resolves only that Sentence through the parent profile. |
| 18 | +PASS - Messages playback does not call Text To Speech Local Preview validation and `toolbox/messages/messages.js` contains no `before preview` text. |
| 19 | +PASS - Browser UI module imports were removed from `src/dev-runtime/messages/messages-postgres-service.mjs`. |
| 20 | +PASS - Separate Message Studio default TTS profile builders were removed. |
| 21 | +PASS - No inline styles, inline scripts, style blocks, or inline handlers were added. |
| 22 | +PASS - Local API owns authoritative database keys; Messages syncs saved names to server-created TTS/emotion keys. |
| 23 | +PASS - Creator-safe guidance is used for Messages TTS profile load failures. |
| 24 | + |
| 25 | +## Validation Lane Report |
| 26 | +PASS - `node --check toolbox/messages/messages.js`. |
| 27 | +PASS - `node --check toolbox/text-to-speech/text2speech.js`. |
| 28 | +PASS - `node --check toolbox/text-to-speech/tts-profile-store.js`. |
| 29 | +PASS - `node --check src/dev-runtime/messages/messages-postgres-service.mjs`. |
| 30 | +PASS - `node --test tests/tools/Text2SpeechShell.test.mjs`. |
| 31 | +PASS - `node --test tests/tools/MessagesPlaybackSource.test.mjs`. |
| 32 | +PASS - `node --test tests/dev-runtime/MessagesPublishValidation.test.mjs`. |
| 33 | +PASS - `node --test --test-name-pattern "Messages Local API" tests/dev-runtime/DbSeedIntegrity.test.mjs`. |
| 34 | +BLOCKED - `npx playwright test tests/playwright/tools/MessagesTool.spec.mjs tests/playwright/tools/TextToSpeechFunctional.spec.mjs` could not launch because Chromium is not installed at `C:\Users\davidq\AppData\Local\ms-playwright\chromium-1217\chrome-win64\chrome.exe`. |
| 35 | +BLOCKED - Fallback `npm run test:workspace-v2` completed static preflight report generation, then hit the same missing Chromium executable. |
| 36 | +KNOWN - Full `tests/dev-runtime/DbSeedIntegrity.test.mjs` still has unrelated Local DB snapshot failures in non-Messages cases. |
| 37 | + |
| 38 | +## Manual Validation Notes |
| 39 | +- Browser manual validation was not completed because Playwright Chromium is missing in this environment. |
| 40 | +- Added Playwright coverage for creating/editing a TTS Profile in Text To Speech, opening Messages on the same origin, confirming the saved profile appears, confirming Sentence emotions are profile-filtered, and exercising Parent/Child Play. |
| 41 | +- Static grep confirmed Messages no longer contains `Select an available browser voice before preview` or `before preview` wording. |
| 42 | + |
| 43 | +## Branch Status |
| 44 | +- Branch: `team/BRAVO/messages` |
| 45 | +- Latest commit: `c45328678` |
| 46 | +- Ahead of main: 2 |
| 47 | +- Behind main: 0 |
| 48 | +- Existing unrelated local edit excluded from PR package: `.gitignore`. |
| 49 | + |
| 50 | +## ZIP |
| 51 | +- Expected ZIP: `tmp/PR_26174_BRAVO_022-use-active-tts-profiles-in-messages_delta.zip` |
0 commit comments