Fix TTS seed profiles and guest save routing

Author: ToolboxAid

assets/toolbox/text-to-speech/js/index.js

@@ -16,6 +16,7 @@ import {
   listTtsProfiles,
   updateTtsProfile,
 } from "../../../../toolbox/messages/messages-api-client.js";
+import { getSessionCurrent } from "../../../../src/api/session-api-client.js";
 
 const TTS_OWNERSHIP = Object.freeze({
   DESIGN: "Design",
@@ -78,6 +79,7 @@ const TTS_PROFILE_CONTRACT_VERSION = "tts-profile-emotion-v1";
 const NEW_ROW_KEY = "__new__";
 const DEFAULT_TTS_PROFILE_ID = "default-balanced-profile";
 const DEFAULT_TTS_EMOTION_ID = "neutral";
+const SIGN_IN_ROUTE = "account/sign-in.html";
 
 const TTS_PROFILE_GENDER_OPTIONS = Object.freeze([
   Object.freeze({ label: "Neutral", value: "neutral" }),
@@ -95,7 +97,6 @@ const TTS_PROFILE_EMOTION_OPTIONS = Object.freeze([
   Object.freeze({ label: "Urgent", value: "urgent" }),
   Object.freeze({ label: "Whisper", value: "whisper" }),
   Object.freeze({ label: "Excited", value: "excited" }),
-  Object.freeze({ label: "Robot", value: "robot" })
 ]);
 
 function boundedNumber(value, { fallback, max, min, value: defaultValue }) {
@@ -306,16 +307,13 @@ function createDefaultEmotionSettings({ markNeutralInUse = false } = {}) {
       emotion: "neutral",
       messagePartsUsageCount: markNeutralInUse ? 1 : 0,
     }),
-    createTextToSpeechProfileEmotion({ emotion: "happy", pitch: 1.08, rate: 1.04 }),
-    createTextToSpeechProfileEmotion({ emotion: "angry", pitch: 0.96, rate: 1.08, volume: 1 }),
-    createTextToSpeechProfileEmotion({ emotion: "scared", pitch: 1.12, rate: 1.12, volume: 0.9 }),
+    createTextToSpeechProfileEmotion({ emotion: "calm" }),
+    createTextToSpeechProfileEmotion({ emotion: "urgent", pitch: 1.08, rate: 1.15, volume: 1 }),
   ];
 }
 
 function createDefaultTextToSpeechProfiles(voiceOptions = []) {
   const balancedVoice = defaultVoiceForProfile(voiceOptions);
-  const manVoice = defaultVoiceForProfile(voiceOptions, "male") || balancedVoice;
-  const womanVoice = defaultVoiceForProfile(voiceOptions, "female") || voiceOptions[1] || balancedVoice;
   return [
     createTextToSpeechProfile({
       emotions: createDefaultEmotionSettings({ markNeutralInUse: true }),
@@ -326,27 +324,16 @@ function createDefaultTextToSpeechProfiles(voiceOptions = []) {
       voice: balancedVoice?.value || "",
       voiceName: balancedVoice?.name || balancedVoice?.label || "Default browser voice"
     }),
-    createTextToSpeechProfile({
-      emotions: createDefaultEmotionSettings(),
-      gender: "male",
-      id: "man-profile-1",
-      language: manVoice?.language || TEXT_TO_SPEECH_DEFAULTS.language,
-      name: "Man Profile 1",
-      voice: manVoice?.value || "",
-      voiceName: manVoice?.name || manVoice?.label || "Default browser voice"
-    }),
-    createTextToSpeechProfile({
-      emotions: createDefaultEmotionSettings(),
-      gender: "female",
-      id: "woman-profile-2",
-      language: womanVoice?.language || TEXT_TO_SPEECH_DEFAULTS.language,
-      name: "Woman Profile 2",
-      voice: womanVoice?.value || "",
-      voiceName: womanVoice?.name || womanVoice?.label || "Default browser voice"
-    })
   ];
 }
 
+function signInUrl() {
+  if (typeof document === "undefined" || typeof window === "undefined") {
+    return SIGN_IN_ROUTE;
+  }
+  return new URL(SIGN_IN_ROUTE, document.baseURI || window.location.href).href;
+}
+
 function isDefaultBrowserVoice(value) {
   return ["browser default", "default", "default browser voice"].includes(String(value || "").trim().toLowerCase());
 }
@@ -851,6 +838,38 @@ function initializeTextToSpeechTool(root = document, { engine = new TextToSpeech
     return errors;
   }
 
+  function currentSessionState() {
+    try {
+      const session = getSessionCurrent();
+      return {
+        apiAvailable: true,
+        authenticated: Boolean(session?.authenticated && session.userKey),
+        session,
+      };
+    } catch (error) {
+      console.warn("Text To Speech could not verify the current session.", error instanceof Error ? error.message : String(error || ""));
+      return {
+        apiAvailable: false,
+        authenticated: false,
+        session: null,
+      };
+    }
+  }
+
+  function requireAuthenticatedWrite(action) {
+    const sessionState = currentSessionState();
+    if (!sessionState.apiAvailable) {
+      writeStatus("Session status could not be verified. Try again shortly.", "FAIL");
+      return false;
+    }
+    if (!sessionState.authenticated) {
+      writeStatus(`Sign in before ${action}.`, "FAIL");
+      window.location.href = signInUrl();
+      return false;
+    }
+    return true;
+  }
+
   function profileApiPayload(profile) {
     return {
       active: profile.active !== false,
@@ -947,6 +966,9 @@ function initializeTextToSpeechTool(root = document, { engine = new TextToSpeech
   }
 
   function commitProfile(key) {
+    if (!requireAuthenticatedWrite("saving Text To Speech profiles")) {
+      return;
+    }
     const profile = profileValues(key);
     const errors = validateProfile(profile);
     if (errors.length) {
@@ -975,6 +997,9 @@ function initializeTextToSpeechTool(root = document, { engine = new TextToSpeech
   }
 
   function deleteProfile(key) {
+    if (!requireAuthenticatedWrite("deleting Text To Speech profiles")) {
+      return;
+    }
     const profile = state.profiles.find((candidate) => candidate.id === key);
     if (!profile) return;
     if (profileInUseByMessageStudio(profile)) {
@@ -1004,6 +1029,9 @@ function initializeTextToSpeechTool(root = document, { engine = new TextToSpeech
   }
 
   function commitEmotion(key) {
+    if (!requireAuthenticatedWrite("saving Text To Speech emotion settings")) {
+      return;
+    }
     const emotion = emotionValues(key);
     const errors = validateEmotion(emotion, key === NEW_ROW_KEY ? "" : key);
     if (errors.length) {
@@ -1046,6 +1074,9 @@ function initializeTextToSpeechTool(root = document, { engine = new TextToSpeech
   }
 
   function deleteEmotion(key) {
+    if (!requireAuthenticatedWrite("deleting Text To Speech emotion settings")) {
+      return;
+    }
     const profile = selectedProfile();
     const emotion = profile?.emotions.find((candidate) => candidate.id === key);
     if (!profile || !emotion) return;

docs_build/database/seed/messages.json

@@ -16,6 +16,7 @@
       "Notification"
     ],
     "messages_emotion_profiles": [
+      "Neutral",
       "Calm",
       "Urgent",
       "Whisper",
@@ -25,8 +26,7 @@
       "Mysterious"
     ],
     "messages_tts_profiles": [
-      "Browser Speech Default",
-      "Narration Preview"
+      "Default Balanced Profile"
     ],
     "messages_records": [],
     "messages_segments": []

docs_build/dev/reports/PR_26177_BRAVO_005-fix-tts-seed-profiles-and-guest-save-routing.md

@@ -0,0 +1,71 @@
+# PR_26177_BRAVO_005 Fix TTS Seed Profiles And Guest Save Routing
+
+## Branch Validation
+
+| Check | Result | Notes |
+| --- | --- | --- |
+| Active branch | PASS | `bravo/26177-text-to-speech` |
+| Did not switch to main | PASS | Work stayed on the active Bravo branch. |
+| Scope | PASS | Changed only Team Bravo Text To Speech seed/profile usability, guest write routing, and targeted impacted tests. |
+| No governance changes | PASS | No governance files were edited. |
+| No `start_of_day` changes | PASS | `git status --short -- docs_build/dev/start_of_day start_of_day` returned no changed files. |
+
+## Removed Invalid Profiles/Emotions
+
+| Record | Result | Notes |
+| --- | --- | --- |
+| `Default Balanced Profile` | KEPT | Default-safe browser profile using `Default browser voice`, `en-US`, and Neutral/Calm/Urgent emotion settings. |
+| `Hero` | NOT PRESENT | No active runtime/static TTS seed row was present. No browser-owned cleanup was added. |
+| `Merchant` | NOT PRESENT | No active runtime/static TTS seed row was present. No browser-owned cleanup was added. |
+| `Neutral` | KEPT | Safe default emotion setting. |
+| `Robot` | REMOVED | Removed from runtime Emotion Profile seed, TTS profile emotion settings, and TTS emotion authoring options because it was sample/stylized seed data rather than default-safe TTS data. |
+| `Man Profile 1` | REMOVED | Removed unsupported starter profile seed; it did not persist a real gender/voice-filter distinction from Default Balanced. |
+| `Woman Profile 2` | REMOVED | Removed unsupported starter profile seed and its Robot setting. |
+| `Browser Speech Default` / `Narration Preview` | REMOVED | Removed stale static seed inventory names and aligned static seed inventory to the runtime `Default Balanced Profile`. |
+
+## Guest Save Routing Checklist
+
+| Requirement | Result | Notes |
+| --- | --- | --- |
+| Guest browsing allowed | PASS | GET/list routes for TTS profiles/emotions remain open. |
+| Guest saving blocked in UI | PASS | TTS Profile save/delete and Emotion setting save/delete now check the current session before mutating state. |
+| Guest write routes to sign-in | PASS | Unauthenticated TTS write actions redirect to `account/sign-in.html`. |
+| Guest saving blocked in API | PASS | Local API rejects unauthenticated POST writes to `tts-profiles` and `emotion-profiles`. |
+| No localStorage/product-data fallback | PASS | Text To Speech still uses Local API profile contracts; no browser-owned product data was added. |
+| No silent fallback profiles | PASS | Runtime and helper seeds now expose only the explicit safe default profile. |
+
+## Validation Lane Report
+
+| Command | Result | Notes |
+| --- | --- | --- |
+| `node --check assets/toolbox/text-to-speech/js/index.js` | PASS | Syntax check passed. |
+| `node --check src/dev-runtime/messages/messages-postgres-service.mjs` | PASS | Syntax check passed. |
+| `node --check src/dev-runtime/server/local-api-router.mjs` | PASS | Syntax check passed. |
+| `node --check tests/playwright/tools/TextToSpeechFunctional.spec.mjs` | PASS | Syntax check passed. |
+| `node --check tests/playwright/tools/MessagesTool.spec.mjs` | PASS | Syntax check passed after impacted seed-reference updates. |
+| `node --check tests/playwright/tools/EventsTool.spec.mjs` | PASS | Syntax check passed. |
+| `node --test tests/tools/Text2SpeechShell.test.mjs` | PASS | 6/6 tests passed. |
+| `node --test tests/dev-runtime/MessagesPublishValidation.test.mjs` | PASS | 6/6 tests passed, including guest TTS write rejection. |
+| `node --test --test-name-pattern "Messages Local API seeds" tests/dev-runtime/DbSeedIntegrity.test.mjs` | PASS | Targeted Messages seed-integrity case passed. |
+| `node --test tests/dev-runtime/DbSeedIntegrity.test.mjs` | PARTIAL | Targeted Messages seed test passed; two unrelated Local DB snapshot tests still fail on `/api/local-db/snapshot`. |
+| `npx playwright test tests/playwright/tools/TextToSpeechFunctional.spec.mjs tests/playwright/tools/MessagesTool.spec.mjs tests/playwright/tools/EventsTool.spec.mjs --project=playwright` | BLOCKED/FAIL | Browser launch failed before page runtime because Chromium is missing at `C:\Users\davidq\AppData\Local\ms-playwright\chromium-1217\chrome-win64\chrome.exe`. |
+| `git diff --check` | PASS | No whitespace errors. Git warned touched files may be normalized from LF to CRLF when Git writes them. |
+
+## Manual Validation Notes
+
+- Source inspection confirmed `Default browser voice` remains accepted by the Text To Speech preview resolver when a playable browser voice exists.
+- Source inspection confirmed TTS write actions call session verification before save/delete state mutation.
+- Source inspection confirmed no browser-owned product data, local storage SSoT, governance edits, or `start_of_day` changes were introduced.
+- Playwright browser validation could not complete locally because the configured Chromium binary is missing.
+
+## Known Issues
+
+- Local Playwright validation remains blocked until Chromium is installed for the configured Playwright version.
+- The broader `DbSeedIntegrity.test.mjs` file still has unrelated Local DB snapshot failures outside the Messages/TTS seed case.
+
+## Output Files
+
+- `docs_build/dev/reports/codex_review.diff`
+- `docs_build/dev/reports/codex_changed_files.txt`
+- `docs_build/dev/reports/PR_26177_BRAVO_005-fix-tts-seed-profiles-and-guest-save-routing.md`
+- `tmp/PR_26177_BRAVO_005-fix-tts-seed-profiles-and-guest-save-routing_delta.zip`

docs_build/dev/reports/codex_changed_files.txt

@@ -1,6 +1,13 @@
 assets/toolbox/text-to-speech/js/index.js
+docs_build/database/seed/messages.json
 docs_build/dev/reports/codex_changed_files.txt
 docs_build/dev/reports/codex_review.diff
-docs_build/dev/reports/PR_26177_BRAVO_004-fix-emotion-preview-parent-voice.md
+docs_build/dev/reports/PR_26177_BRAVO_005-fix-tts-seed-profiles-and-guest-save-routing.md
+src/dev-runtime/messages/messages-postgres-service.mjs
+src/dev-runtime/server/local-api-router.mjs
+tests/dev-runtime/DbSeedIntegrity.test.mjs
+tests/dev-runtime/MessagesPublishValidation.test.mjs
+tests/playwright/tools/EventsTool.spec.mjs
+tests/playwright/tools/MessagesTool.spec.mjs
 tests/playwright/tools/TextToSpeechFunctional.spec.mjs
 tests/tools/Text2SpeechShell.test.mjs