Fix: Restore stable logic, keep newsHook {title,link} support

src/app.js

@@ -24,7 +24,6 @@ const DEFAULT_VOICE = "Sohee";
 /**
  * Fetch a SHORT hot news headline to weave into the greeting.
  * Short enough that the AI says it as ONE sentence.
- * Returns {title, link} or null.
  */
 async function getHotNewsGreeting() {
   let hotItem = null;
@@ -48,6 +47,7 @@ const DEFAULT_INSTRUCTIONS = [
   "You are a friendly voice assistant with a visible, human-like 3D avatar.",
   "CRITICAL: Always respond in the SAME LANGUAGE the user writes or speaks.",
   "If Vietnamese, respond in Vietnamese. If English, respond in English.",
+  "When Vietnamese: write dates as 'ngày 9 tháng 7 năm 2026' (NOT '9/7/2026'),",
   "Keep replies short, natural, warm. Never list-like.",
   "You can control avatar body with tools: set_mood, make_hand_gesture, make_facial_expression. Use them naturally.",
   "NEVER guess. Use search_web or search_wikipedia for ANY factual question.",
@@ -141,7 +141,7 @@ function effectiveInstructions(newsHook) {
   const extra = settings.instructions.trim();
   const introLine = "Tôi tên là Vương đến V.AI STUDIO.";
   // newsHook is either null or {title, link}
-  const newsInst = newsHook && newsHook.title
+  const newsInst = newsHook
     ? `\n\nWhen you start the conversation, greet the user naturally and briefly mention today's news hook: "${newsHook.title}"${newsHook.link ? ` (xem chi tiết: ${newsHook.link})` : ""}\nThen wait for their response. Keep your greeting to ONE short sentence.`
     : "";
   const base = `${dateLine}\n\n${introLine}\n\n${DEFAULT_INSTRUCTIONS}${newsInst}`;
@@ -305,25 +305,18 @@ async function startVoiceSession() {
   else { try { micStream = await navigator.mediaDevices.getUserMedia({ audio: { echoCancellation: true, noiseSuppression: true, autoGainControl: true } }); } catch { setCaption("MIC BLOCKED", "error"); sessionInProgress = false; return; } }
   const audioCtx = stage.audioCtx; const voiceSink = stage.voiceSink;
   if (!audioCtx || !voiceSink) { sessionInProgress = false; return; }
-  let greeting = preFetchedGreeting;
-  if (!greeting) { try { greeting = await getHotNewsGreeting(); } catch { greeting = null; } }
-  try {
-    const c = new S2sWsRealtimeClient({
-      ...(config.lb ? { sessionUrl: "api/session" } : { directUrl: settings.directUrl }),
-      voice: settings.voice, instructions: effectiveInstructions(greeting),
-      micStream, audioContext: audioCtx, outputNode: voiceSink,
-      workletBaseUrl: "/worklets/", tools: TOOL_DEFS, _textOnly: false,
-    });
-    client = c; _attachClientEvents(c);
-    setCaption("REQUESTING A SLOT…");
-    const ok = await connectSession(c);
-    if (!ok) return;
-    if (!autoGreetingSent) { c.requestResponse(); }
-  } catch (err) {
-    console.error("Session start error:", err);
-    setCaption("CONNECT ERROR, TAP TO RETRY", "error");
-    await endSession(true);
-  }
+  const greeting = preFetchedGreeting || (await getHotNewsGreeting());
+  const c = new S2sWsRealtimeClient({
+    ...(config.lb ? { sessionUrl: "api/session" } : { directUrl: settings.directUrl }),
+    voice: settings.voice, instructions: effectiveInstructions(greeting),
+    micStream, audioContext: audioCtx, outputNode: voiceSink,
+    workletBaseUrl: "/worklets/", tools: TOOL_DEFS, _textOnly: false,
+  });
+  client = c; _attachClientEvents(c);
+  setCaption("REQUESTING A SLOT…");
+  const ok = await connectSession(c);
+  if (!ok) return;
+  if (!autoGreetingSent) { c.requestResponse(); }
 }
 
 /**
@@ -338,26 +331,19 @@ async function startTextSession(initialText) {
   await stage.resume();
   const audioCtx = stage.audioCtx; const voiceSink = stage.voiceSink;
   if (!audioCtx || !voiceSink) { sessionInProgress = false; return; }
-  let newsHook = preFetchedGreeting;
-  if (!newsHook) { try { newsHook = await getHotNewsGreeting(); } catch { newsHook = null; } }
-  try {
-    const c = new S2sWsRealtimeClient({
-      ...(config.lb ? { sessionUrl: "api/session" } : { directUrl: settings.directUrl }),
-      voice: settings.voice, instructions: effectiveInstructions(newsHook),
-      audioContext: audioCtx, outputNode: voiceSink,
-      workletBaseUrl: "/worklets/", tools: TOOL_DEFS, _textOnly: true,
-    });
-    client = c; _attachClientEvents(c);
-    textMode = true; showTextChat(true); textModeBtn.classList.add("active");
-    const ok = await connectSession(c);
-    if (!ok) return;
-    if (!autoGreetingSent) { c.requestResponse(); }
-    if (initialText) { c.sendUserText(initialText); c.requestResponse(); }
-  } catch (err) {
-    console.error("Session start error:", err);
-    setCaption("CONNECT ERROR, TAP TO RETRY", "error");
-    await endSession(true);
-  }
+  const newsHook = preFetchedGreeting || (await getHotNewsGreeting());
+  const c = new S2sWsRealtimeClient({
+    ...(config.lb ? { sessionUrl: "api/session" } : { directUrl: settings.directUrl }),
+    voice: settings.voice, instructions: effectiveInstructions(newsHook),
+    audioContext: audioCtx, outputNode: voiceSink,
+    workletBaseUrl: "/worklets/", tools: TOOL_DEFS, _textOnly: true,
+  });
+  client = c; _attachClientEvents(c);
+  textMode = true; showTextChat(true); textModeBtn.classList.add("active");
+  const ok = await connectSession(c);
+  if (!ok) return;
+  if (!autoGreetingSent) { c.requestResponse(); }
+  if (initialText) { c.sendUserText(initialText); c.requestResponse(); }
 }
 
 function _attachClientEvents(c) {
@@ -404,7 +390,7 @@ settingsBtn.addEventListener("click", () => { inputVoice.value = settings.voice;
 settingsDialog.addEventListener("close", () => {
   settings = { voice: inputVoice.value || DEFAULT_VOICE, avatar: settingsAvatarSelect.value || "", instructions: inputInstructions.value, directUrl: inputDirectUrl.value.trim(), subtitles: inputSubtitles.checked };
   if (settings.avatar !== "vuong.glb") autoGreetingSent = false;
-  saveSettings(); chatAvatarSelect.value = settings.avatar; if (!settings.subtitles) subtitles.classList.remove("visible"); client?.updateSession({ voice: settings.voice, instructions: effectiveInstructions(preFetchedGreeting) });
+  saveSettings(); chatAvatarSelect.value = settings.avatar; if (!settings.subtitles) subtitles.classList.remove("visible"); client?.updateSession({ voice: settings.voice, instructions: effectiveInstructions(preFetchedGreeting || "") });
 });
 window.addEventListener("beforeunload", () => { client?.close(); });
 
@@ -422,7 +408,8 @@ async function boot() {
   try { await stage.init({ avatarUrl: settings.avatar ? `/avatars/${settings.avatar}` : undefined, onprogress: (ev) => { if (ev.lengthComputable) loading.textContent = `Loading avatar ${Math.min(100, Math.round((ev.loaded / ev.total) * 100))}%`; } }); }
   catch (err) { console.error(err); loading.textContent = "Avatar failed to load. Reload."; setCaption("AVATAR FAILED", "error"); return; }
   preFetchedGreeting = await newsPromise;
-  loading.classList.add("done"); setCaption(CAPTIONS.idle); setMainButton("start", "Start talking");
+  loading.classList.add("done");
+  setCaption(CAPTIONS.idle); setMainButton("start", "Start talking");
 
   // NO auto-greeting here. On direct hf.space domains AudioContext is
   // "suspended" before first click — any audio routed through a suspended