@mobius-os/mobius 0.2.3 → 0.2.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mobius-os/mobius",
-  "version": "0.2.3",
+  "version": "0.2.4",
   "type": "module",
   "description": "Mobius terminal client. Reuses Mobius frontend TypeScript types and jsonl entry shapes.",
   "bin": {

package/src/components/Chat.tsx

@@ -35,7 +35,7 @@ interface TerminalSize {
   isTty: boolean
 }
 
-const VERSION = '0.2.1'
+const VERSION = '0.2.4'
 const WELCOME_ROWS = 12
 const CHROME_ROWS = 11
 

package/src/hooks/useChat.ts

@@ -118,13 +118,30 @@ export function useChat({ client, ready, resumeSessionId }: ChatApi): ChatContro
   const appendEntries = useCallback((newOnes: AnyEntry[]) => {
     if (!newOnes.length) return
     setEntries(prev => {
-      const stamped = newOnes.map(e => ({ ...e, __id: e.__id ?? nextId() }))
-      return [...prev, ...stamped]
+      // De-duplicate by uuid so a live jsonl_entry that also appears in a
+      // reconnect's history replay is never shown twice.
+      const seen = new Set<string>()
+      for (const e of prev) { const k = entryKey(e); if (k) seen.add(k) }
+      const stamped: AnyEntry[] = []
+      for (const e of newOnes) {
+        const k = entryKey(e)
+        if (k && seen.has(k)) continue
+        if (k) seen.add(k)
+        stamped.push({ ...e, __id: e.__id ?? nextId() })
+      }
+      return stamped.length ? [...prev, ...stamped] : prev
     })
   }, [])
 
   const setHistory = useCallback((list: AnyEntry[]) => {
-    setEntries(list.map(e => ({ ...e, __id: e.__id ?? nextId() })))
+    const seen = new Set<string>()
+    const out: AnyEntry[] = []
+    for (const e of list) {
+      const k = entryKey(e)
+      if (k) { if (seen.has(k)) continue; seen.add(k) }
+      out.push({ ...e, __id: e.__id ?? nextId() })
+    }
+    setEntries(out)
   }, [])
 
   // ── SSE connection ────────────────────────────────────────────────────────
@@ -136,6 +153,14 @@ export function useChat({ client, ready, resumeSessionId }: ChatApi): ChatContro
     const conn = new SseConnection(url, {
       onHistoryEntries: (es, _done) => {
         if (es.length) setHistory(es)
+        // A reconnect replays the session tail. If our optimistic placeholder is
+        // now backed by its real entry, retire it so the user's input isn't shown
+        // twice (once as the entry, once as the placeholder). The live jsonl_entry
+        // path already clears pendingUser, but a dropped SSE stream (reverse-proxy
+        // idle timeout) can deliver the message only via this history replay — and
+        // if the whole turn finished while disconnected, no live entry ever comes
+        // to clear it, leaving the duplication on screen until the next send.
+        setPendingUser(prev => (prev !== null && es.some(e => entryMatchesPendingUser(e, prev)) ? null : prev))
       },
       onEntry: (entry) => {
         if (process.env.MOBIUS_TUI_DEBUG) console.error('[onEntry]', entry?.type, (entry?.message?.content?.[0]?.text ?? '').slice(0, 40))
@@ -317,7 +342,10 @@ export function useChat({ client, ready, resumeSessionId }: ChatApi): ChatContro
 
   const send = useCallback(async (text: string) => {
     const body = text.trim()
-    if (!body || sending) return
+    // Guard on the ref (synchronous truth) as well as the state so a stale
+    // closure can't dispatch the same message twice (two distinct reqIds → two
+    // user entries on the server).
+    if (!body || sending || sendingRef.current) return
     setError(null)
     setPendingUser(body)
     statusEpochRef.current += 1