Skip to content

Commit eaee83f

Browse files
authored
Merge PR_26171_BETA_071 tts profile emotion table
Merge PR_26171_BETA_071 tts profile emotion table
2 parents 78b642f + 9ff8afd commit eaee83f

35 files changed

Lines changed: 695 additions & 1212 deletions
Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,21 @@
1+
# PR_26171_BETA_071-tts-profile-emotion-table Manual Validation Notes
2+
3+
Generated: 2026-06-20T22:26:42.301Z
4+
Team ownership: BETA
5+
Branch: pr/26171-BETA-071-tts-profile-emotion-table
6+
7+
## Manual Review
8+
9+
- Reviewed TTS Studio table markup in `toolbox/text-to-speech/index.html`; parent table already uses the required TTS Profiles columns.
10+
- Reviewed TTS Studio rendering in `toolbox/text-to-speech/text2speech.js`; clicking a profile row opens the selected profile's Emotion Settings child table, and inline profile/emotion editors remain external-JS driven.
11+
- Reviewed Message Studio compatibility coverage in `tests/playwright/tools/MessagesTool.spec.mjs`; the default TTS Profile dropdown and Message Parts TTS Profile dropdown remain validated.
12+
13+
## Automated Browser Coverage Used As Manual Proxy
14+
15+
- TTS Playwright validation clicked the default, Man Profile 1, and Woman Profile 2 rows and verified each selected profile exposes four emotion rows.
16+
- TTS Playwright validation exercised Add/Edit Profile and Add/Edit Emotion inline paths.
17+
- Message Studio Playwright validation exercised message-level and part-level TTS Profile dropdown behavior and playback routing.
18+
19+
## Not Performed
20+
21+
- No separate unscripted browser session was run beyond the targeted Playwright browser validations.
Lines changed: 24 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,24 @@
1+
# PR_26171_BETA_071-tts-profile-emotion-table Validation Report
2+
3+
Generated: 2026-06-20T22:26:42.301Z
4+
Team ownership: BETA
5+
Branch: pr/26171-BETA-071-tts-profile-emotion-table
6+
7+
## Results
8+
9+
- PASS: `npx playwright test tests/playwright/tools/TextToSpeechFunctional.spec.mjs`
10+
- 2 passed.
11+
- PASS: `node --test tests/tools/Text2SpeechShell.test.mjs`
12+
- 4 passed.
13+
- PASS: `npx playwright test tests/playwright/tools/MessagesTool.spec.mjs --trace=off`
14+
- 2 passed.
15+
- PASS: `npm run test:workspace-v2`
16+
- 5 workspace-contract Playwright tests passed.
17+
- PASS: `git diff --check`
18+
- No whitespace errors reported.
19+
20+
## Notes
21+
22+
- The Message Studio compatibility check used `--trace=off` to avoid the known trace artifact writer issue while keeping the browser assertions unchanged.
23+
- Node emitted experimental SQLite warnings during Playwright runs; tests still passed.
24+
- Workspace validation refreshed the standard generated validation report snapshots under `docs_build/dev/reports`.
Lines changed: 31 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,31 @@
1+
# PR_26171_BETA_071-tts-profile-emotion-table
2+
3+
Generated: 2026-06-20T22:26:42.301Z
4+
Team ownership: BETA
5+
Branch: pr/26171-BETA-071-tts-profile-emotion-table
6+
Base HEAD: 78b642f70cf5698b611f2f8e1f6c6e18f9d73961
7+
origin/main: 78b642f70cf5698b611f2f8e1f6c6e18f9d73961
8+
Ahead/behind before commit: 0 0
9+
Spec source: latest user request in Codex session; no repository BUILD_PR doc for PR_26171_BETA_071 was found by targeted search.
10+
11+
## Scope
12+
13+
- Implemented the missing TTS Studio parent/child table contract detail for Emotion Settings.
14+
- Kept TTS Studio parent table as TTS Profiles.
15+
- Kept profile row click expansion for selected-profile Emotion Settings.
16+
- Kept inline Add/Edit Profile and Add/Edit Emotion behavior.
17+
- Kept Message Studio TTS Profile dropdown compatibility.
18+
- Kept Theme V2 and external JavaScript only.
19+
20+
## Code Changes
21+
22+
- Changed the TTS Studio Emotion Settings child-table column from `SSML-like Preset` to authoritative `Preset`.
23+
- Updated the targeted TTS Playwright validation expectation to require the `Preset` child column.
24+
25+
## Existing Behavior Verified
26+
27+
- TTS Profiles columns remain: Profile Name, Voice, Language, Gender, Age, Emotion Count, Status, Actions.
28+
- Emotion Settings child columns now read: Emotion, Pitch, Rate, Volume, Preset, Status, Actions.
29+
- Default profile rows include Default Balanced Profile, Man Profile 1, and Woman Profile 2.
30+
- Man Profile 1 and Woman Profile 2 each expose Neutral, Happy, Angry, and Scared emotion settings when selected.
31+
- No separate Emotion Studio was introduced.

docs_build/dev/reports/codex_changed_files.txt

Lines changed: 5 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,6 @@
1-
docs_build/dev/reports/PR_26171_BETA_069-message-tts-profile-contract-alignment-manual-validation-notes.md
2-
docs_build/dev/reports/PR_26171_BETA_069-message-tts-profile-contract-alignment-message-tts-ownership-checklist.md
3-
docs_build/dev/reports/PR_26171_BETA_069-message-tts-profile-contract-alignment-parent-child-table-checklist.md
4-
docs_build/dev/reports/PR_26171_BETA_069-message-tts-profile-contract-alignment-validation.md
5-
docs_build/dev/reports/PR_26171_BETA_069-message-tts-profile-contract-alignment.md
1+
docs_build/dev/reports/PR_26171_BETA_071-tts-profile-emotion-table-manual-validation-notes.md
2+
docs_build/dev/reports/PR_26171_BETA_071-tts-profile-emotion-table-validation.md
3+
docs_build/dev/reports/PR_26171_BETA_071-tts-profile-emotion-table.md
64
docs_build/dev/reports/codex_changed_files.txt
75
docs_build/dev/reports/codex_review.diff
86
docs_build/dev/reports/coverage_changed_js_guardrail.txt
@@ -15,12 +13,9 @@ docs_build/dev/reports/incremental_validation_report.md
1513
docs_build/dev/reports/lane_compilation_report.md
1614
docs_build/dev/reports/lane_deduplication_report.md
1715
docs_build/dev/reports/lane_input_validation_report.md
18-
docs_build/dev/reports/lane_manifests/workspace-contract.json
1916
docs_build/dev/reports/lane_runtime_optimization_report.md
2017
docs_build/dev/reports/lane_snapshot_report.md
21-
docs_build/dev/reports/lane_snapshots/workspace-contract.json
2218
docs_build/dev/reports/lane_warm_start_report.md
23-
docs_build/dev/reports/lane_warm_starts/workspace-contract.json
2419
docs_build/dev/reports/monolith_trigger_removal_report.md
2520
docs_build/dev/reports/persistent_lane_manifest_report.md
2621
docs_build/dev/reports/playwright_discovery_ownership_report.md
@@ -36,7 +31,5 @@ docs_build/dev/reports/test_cleanup_routing_report.md
3631
docs_build/dev/reports/testing_lane_execution_report.md
3732
docs_build/dev/reports/validation_cache_report.md
3833
docs_build/dev/reports/zero_browser_preflight_report.md
39-
src/dev-runtime/messages/messages-sqlite-service.mjs
40-
tests/playwright/tools/MessagesTool.spec.mjs
41-
toolbox/messages/index.html
42-
toolbox/messages/messages.js
34+
tests/playwright/tools/TextToSpeechFunctional.spec.mjs
35+
toolbox/text-to-speech/text2speech.js

0 commit comments

Comments
 (0)