thinkpool-pair 0.7.266 → 0.7.267

package/bridge.mjs

@@ -695,7 +695,10 @@ async function refreshOwnerPlan() {
 // unit-tested reap-terminal.mjs. Fire-and-forget from endStructured — the module never
 // throws (best-effort), so an unawaited promise can't reject.
 function reapTerminalRow(id) {
-  return _reapTerminalRow({ id, token: codeAuthToken, supabaseUrl: SUPABASE_URL, anonKey: SUPABASE_ANON, shuttingDown })
+  return _reapTerminalRow({
+    id, token: codeAuthToken, supabaseUrl: SUPABASE_URL, anonKey: SUPABASE_ANON, shuttingDown,
+    onFailure: ({ error, status }) => process.stderr.write(`\n  ⚠ terminal row reap failed (${String(id).slice(0, 8)}${status ? `, HTTP ${status}` : ''}): ${error || 'unknown error'}\n`),
+  })
 }
 // Auth: is the sender a PARTICIPANT of THIS room (owner OR granted OR joined)? Room mode
 // serves the room, so RLS returns participants; we confirm the jwt's user is one of them.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "thinkpool-pair",
-  "version": "0.7.266",
+  "version": "0.7.267",
   "description": "Share a local coding-agent CLI (Claude Code, Codex, Gemini, Aider, …) into a ThinkPool Code room, live.",
   "type": "module",
   "bin": {

package/reap-terminal.mjs

@@ -24,10 +24,14 @@
 // the feature, not the leak.
 
 /**
- * @returns {Promise<{reaped: boolean, reason: 'ok'|'shutdown'|'anon'|'no-id'|'error', error?: string}>}
+ * @returns {Promise<{reaped: boolean, reason: 'ok'|'shutdown'|'anon'|'no-id'|'error', error?: string, status?: number}>}
  *   Never throws — best-effort; the client reap (#205) is the fallback.
  */
-export async function reapTerminalRow ({ id, token, supabaseUrl, anonKey, shuttingDown = false, fetchImpl = fetch }) {
+export async function reapTerminalRow ({ id, token, supabaseUrl, anonKey, shuttingDown = false, fetchImpl = fetch, onFailure } = {}) {
+  const fail = (outcome) => {
+    try { onFailure?.(outcome) } catch { /* observability must not break a close */ }
+    return outcome
+  }
   if (shuttingDown) return { reaped: false, reason: 'shutdown' }
   if (!token)       return { reaped: false, reason: 'anon' }
   if (!id)          return { reaped: false, reason: 'no-id' }
@@ -36,12 +40,22 @@ export async function reapTerminalRow ({ id, token, supabaseUrl, anonKey, shutti
     // terminal id IS the code_terminals PK (client/bridge-minted uuid). RLS scopes the
     // delete to this bridge's sessions, so id alone is sufficient + idempotent (a fast
     // client tab may have reaped first — 204 / 0 rows is fine).
-    await fetchImpl(`${supabaseUrl}/rest/v1/code_terminals?id=eq.${encodeURIComponent(id)}`, {
+    const response = await fetchImpl(`${supabaseUrl}/rest/v1/code_terminals?id=eq.${encodeURIComponent(id)}`, {
       method: 'DELETE',
       headers: { apikey: anonKey, Authorization: `Bearer ${token}` },
     })
+    // fetch resolves for HTTP failures. A 401/500 did not reap the row and must
+    // stay visible to the fire-and-forget caller as a failed best-effort cleanup.
+    if (!response?.ok) {
+      return fail({
+        reaped: false,
+        reason: 'error',
+        error: `HTTP ${response?.status ?? 'unknown'}${response?.statusText ? ` ${response.statusText}` : ''}`,
+        status: response?.status,
+      })
+    }
     return { reaped: true, reason: 'ok' }
   } catch (e) {
-    return { reaped: false, reason: 'error', error: e?.message }
+    return fail({ reaped: false, reason: 'error', error: e?.message })
   }
 }