thinkpool-pair 0.7.342 → 0.7.343

package/bridge.mjs

@@ -1283,6 +1283,15 @@ const replayPump = createLatestReplayPump({
 // every announce carries them so a late-joining or second device sees them too.
 const termNames = loadNames(room)
 const manualNameTouched = new Set()
+const acknowledgeTermOpen = (id, kind, entry = null) => {
+  bcast('term-opened', {
+    id,
+    kind,
+    sideParent: entry?.sideParent || undefined,
+    sideTask: entry?.sideTask || undefined,
+    name: termNames[id] || undefined,
+  })
+}
 const autoNameAttempts = new Set()
 const autoNames = new Map()
 // The SDK's supported-model LIST (value/displayName/description), captured from
@@ -1726,7 +1735,6 @@ function pumpDesign(term) {
   }
   const active = { ...next, resultOk: false, proof: null, timer: null }
   designActive.set(term, active)
-  designStatus({ previewId: next.record.previewId, requestId: next.request.cid, state: 'applying' })
   // Complete room-visible instructions, without host paths or locator packets.
   const visible = {
     kind: 'you',
@@ -1747,6 +1755,10 @@ function pumpDesign(term) {
   } catch { accepted = false }
   if (accepted) beginStructuredTurn(lane)
   stampEvent(visible); pushLog(lane, visible); bcast('code-event', { term, evt: visible })
+  // Browser clients use this as the positive dispatch acknowledgment. Keep it
+  // after the accepted turn and its visible prompt so "applying" never navigates
+  // a submitter away from an actionable Design failure.
+  if (accepted) designStatus({ previewId: next.record.previewId, requestId: next.request.cid, state: 'applying' })
   if (!accepted) {
     syncStructuredTurn(lane)
     const failed = { kind: 'error', message: 'The producing lane could not start the edit.', recoverable: true }
@@ -4111,6 +4123,13 @@ channel
     // Multi-bridge rooms: a targeted open is for ONE machine. Untargeted
     // opens (older web) are taken by whoever hears them — the solo case.
     if (payload.host && payload.host !== name) return
+    // A client retries an open when the first broadcast or this acknowledgement is
+    // lost. PTY creation was already idempotent, but acknowledge the existing lane
+    // explicitly so the watchdog can settle without waiting for a full roster.
+    if (terms.has(payload.id)) {
+      acknowledgeTermOpen(payload.id, 'pty', terms.get(payload.id))
+      return
+    }
     // resume is a FLAG, never argv: the channel must not pass arbitrary
     // args even though the room is shell-trust by design. The args come
     // from our own KNOWN_AGENTS table, probe-gated so `--continue` with
@@ -4121,6 +4140,15 @@ channel
       if (agent?.resume) { try { if (agent.resume.probe()) args = [...agent.resume.args] } catch { /* fresh */ } }
     }
     if (wantStructured(payload.cmd)) {
+      // The open id is the idempotency key for the ENTIRE operation, including a
+      // side lane's initial task. openStructured() already no-ops for an existing
+      // runtime, but the old handler continued below and published side-started +
+      // dispatched the task again. A retry must only replay the acceptance ack.
+      const existing = sessions.get(payload.id)
+      if (existing) {
+        acknowledgeTermOpen(payload.id, 'structured', existing)
+        return
+      }
       // model: the web passes the model to open on — a NEW terminal inherits the
       // previous terminal's model (and the first uses the provider default). Falls
       // through to the SDK default when absent. Stamped on the entry so the announce
@@ -4179,13 +4207,17 @@ channel
           return
         }
         child.flush?.(); parent.flush?.(); announce()
+        acknowledgeTermOpen(payload.id, 'structured', child)
         process.stderr.write(`\n  ◆ ${by} opened side lane ${String(payload.id).slice(0, 8)} under ${String(sideParent).slice(0, 8)}.\n`)
         return
       }
+      const opened = sessions.get(payload.id)
+      if (opened) acknowledgeTermOpen(payload.id, 'structured', opened)
       process.stderr.write(`\n  ◆ web opened a structured "${payload.cmd}" session (${payload.mode || 'default'}${payload.model ? `, model ${payload.model}` : ''}).\n`)
       return
     }
     openTerm({ id: payload.id, cmd: payload.cmd, args })
+    if (terms.has(payload.id)) acknowledgeTermOpen(payload.id, 'pty', terms.get(payload.id))
     process.stderr.write(`\n  ◆ web opened a "${payload.cmd}"${args.length ? ' (continue)' : ''} terminal (headless).\n`)
   })
   .on('broadcast', { event: 'term-close' }, ({ payload }) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "thinkpool-pair",
-  "version": "0.7.342",
+  "version": "0.7.343",
   "description": "Connect Claude Code, Codex, or Hermes on your computer to a Thinkpool Code room.",
   "type": "module",
   "bin": {