fix(web): address PR #28 review — explicit clientTts boolean + BYO key prefix hint

Harden the BYO-mode signal at the API boundary (start/scene/insert-beat):
only clientTts === true drops server TTS, so a stray truthy non-boolean can't
silently disable it. Add a non-blocking prefix hint in TtsKeyModal that warns
when the pasted key prefix (tp-/sk-) mismatches the selected key type — a
mismatch hits the wrong endpoint and plays silently, the symptom BYO fixes.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
This commit is contained in:
yuanzonghao
2026-06-04 16:19:35 +08:00
parent b0b2e922d3
commit f6226facbd
4 changed files with 19 additions and 3 deletions
+16
View File
@@ -48,6 +48,14 @@ export function TtsKeyModal({
const [showKey, setShowKey] = useState(false);
const [shown, setShown] = useState(false);
const alreadyConfigured = initial != null;
// Soft guard: tp- keys belong to Token Plan, sk- to pay-as-you-go. A
// mismatched pairing hits the wrong endpoint → guaranteed auth failure →
// silent playback (the very symptom BYO exists to kill). Warn, but never
// block: prefix conventions could change and a hard gate would lock out an
// otherwise-valid key.
const expectedPrefix = keyType === "payg" ? "sk-" : "tp-";
const prefixMismatch =
apiKey.trim().length > 0 && !apiKey.trim().startsWith(expectedPrefix);
useEffect(() => {
const id = requestAnimationFrame(() => setShown(true));
@@ -214,6 +222,14 @@ export function TtsKeyModal({
/>
</button>
</div>
{prefixMismatch && (
<span className="flex items-start gap-1.5 text-[11px] leading-relaxed text-ember-500">
<i className="fa-solid fa-triangle-exclamation mt-0.5 text-[10px]" />
Key {expectedPrefix}
{keyType === "payg" ? "按量付费 Pay-as-you-go" : "套餐 Token Plan"}
</span>
)}
<a
href={TTS_KEY_DOC_URL}
target="_blank"