thinkpool-pair 0.7.308 → 0.7.309

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "thinkpool-pair",
-  "version": "0.7.308",
+  "version": "0.7.309",
   "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

@@ -36,21 +36,28 @@ export async function reapTerminalRow ({ id, token, supabaseUrl, anonKey, shutti
   if (!token)       return { reaped: false, reason: 'anon' }
   if (!id)          return { reaped: false, reason: 'no-id' }
   try {
-    // Mirror of the client reap: DELETE code_terminals WHERE id = <terminal id>. The
-    // 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).
-    const response = await fetchImpl(`${supabaseUrl}/rest/v1/code_terminals?id=eq.${encodeURIComponent(id)}`, {
-      method: 'DELETE',
-      headers: { apikey: anonKey, Authorization: `Bearer ${token}` },
+    // A terminal at the 50k transcript cap cannot archive + cascade-delete inside
+    // the authenticated role's ordinary 8s statement timeout. The member-authorized
+    // RPC scopes a longer timeout to this one cleanup operation; a raw REST DELETE
+    // timed out and left the row as a cross-device dormant ghost (8TMX546Q).
+    const response = await fetchImpl(`${supabaseUrl}/rest/v1/rpc/delete_code_terminal`, {
+      method: 'POST',
+      headers: {
+        apikey: anonKey,
+        Authorization: `Bearer ${token}`,
+        'Content-Type': 'application/json',
+      },
+      body: JSON.stringify({ p_terminal: id }),
     })
     // 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) {
+      let detail = ''
+      try { detail = String(await response.text()).slice(0, 240) } catch { /* no body */ }
       return fail({
         reaped: false,
         reason: 'error',
-        error: `HTTP ${response?.status ?? 'unknown'}${response?.statusText ? ` ${response.statusText}` : ''}`,
+        error: `HTTP ${response?.status ?? 'unknown'}${response?.statusText ? ` ${response.statusText}` : ''}${detail ? `: ${detail}` : ''}`,
         status: response?.status,
       })
     }

package/terminal-row-reconcile.mjs

@@ -1,3 +1,5 @@
+import { reapTerminalRow } from './reap-terminal.mjs'
+
 /* Reconcile durable terminal rows after a bridge has restored its on-disk roster.
 
    A clean close is reaped by reap-terminal.mjs. This module closes the other
@@ -48,13 +50,16 @@ export async function reconcileTerminalRows ({
 
     const reaped = []
     for (const id of stale) {
-      const del = await fetchImpl(
-        `${supabaseUrl}/rest/v1/code_terminals?id=eq.${encodeURIComponent(id)}` +
-        `&host_id=eq.${encodeURIComponent(hostId)}`,
-        { method: 'DELETE', headers: { apikey: anonKey, Authorization: `Bearer ${token}` } },
-      )
-      if (!del?.ok) {
-        return fail({ reason: 'error', error: `delete HTTP ${del?.status ?? 'unknown'}`, status: del?.status, reaped })
+      const deletion = await reapTerminalRow({
+        id, token, supabaseUrl, anonKey, fetchImpl,
+      })
+      if (!deletion.reaped) {
+        return fail({
+          reason: 'error',
+          error: `delete ${deletion.error || deletion.reason}`,
+          status: deletion.status,
+          reaped,
+        })
       }
       reaped.push(id)
     }