@volter/supercode-teams 0.3.196 → 0.3.198

package/README.md

@@ -61,8 +61,8 @@ Before the first SQLite start, `supercode teams mail migration-check --home /abs
 reads that guard locally without a daemon connection or migration. Admission is not proof that the import will finish.
 The [source boundary and pending qualification](../../docs/evidence/legacy-wrapper-migration-2026-10-08.md)
 name the trusted legacy ABI, loss of later legacy diagnostics and existing retirement recovery.
-Settled read bodies expire after 30 days, with explicit placeholders and retained thread headers; pending mail keeps
-its originals. Expired read headers are capped at 1,000,000 and carrier/terminal event log tables at 10,000 rows each. Read the
+Mail bodies are kept like their headers. Read headers whose body is not kept show an explicit placeholder and are
+capped at 1,000,000; carrier/terminal event log tables keep 10,000 rows each. Read the
 [dated qualification scope](../../docs/evidence/sqlite-mail-store-2026-10-08.md) before inferring platform support.
 
 Native mail-history reads process the opened file's initial extent in 64 KiB blocks,

package/machine.mjs

@@ -84,7 +84,7 @@ import { WorkspaceContexts, fromContext } from '@volter/teams/context';
 import { TeamsClient } from '@volter/teams/http-client';
 import { supercodeHome } from '@volter/teams/home';
 import { readMailHistory, machineName } from './mail-history.mjs';
-import { randomBytes, createHash, randomUUID } from 'node:crypto';
+import { randomBytes, createHash } from 'node:crypto';
 import { WebSocket } from 'ws';
 import { TmuxTerminalHost, classifyAttentionSignal, composerHoldsDraft, composerReadyForInput } from '@volter/supercode-terminal';
 // The ConPTY host is read from the namespace, where a terminal package too old to have it gives undefined rather
@@ -909,30 +909,16 @@ export class TeamsMachine extends MachineCore {
       const reading = await this.#serveRequest('harness.v1.sessions.activity_under', { pids: [session.rootPid] }, 4_000);
       const native = reading?.activities?.find(row => row.pid === session.rootPid)?.activity;
       if (native?.harness !== expected.harness || native.session_id !== expected.session) return { delivered: false, reason: 'the pane no longer runs the addressed native session; no input submitted' };
-      // A lifecycle wake belongs to the board's owned session, not the /btw
-      // composer. Dismiss only that identified panel; never answer a question,
-      // approve a permission, clear a draft or interrupt the underlying turn.
+      // A board wake is never typed into a Claude /btw panel, and the daemon sends
+      // no keys to close it: the wake waits unread and the board reports the
+      // session's `dialog open` turn to the manager.
       const boardWake = messageIds?.length && messageIds.every(id => {
         const stored = this.mailStore.findIn(address, id);
         const envelope = stored ? this.mailStore.body(stored) : null;
         return envelope?.kind === 'notice' && parseAddress(envelope.from)?.harness === 'board';
       });
-      if (boardWake && claudeSideQuestionPanel(await this.host.capture(session.id, { lines: 200, preserveEscapes: true }), native)) {
-        const owner = randomUUID();
-        if (await this.host.acquireInput?.(session.id, owner)) {
-          try {
-            await new Promise(resolve => setTimeout(resolve, COMPOSER_RECHECK_MS));
-            const latest = await this.#serveRequest('harness.v1.sessions.activity_under', { pids: [session.rootPid] }, 4_000);
-            const current = latest?.activities?.find(row => row.pid === session.rootPid)?.activity;
-            const screen = await this.host.capture(session.id, { lines: 200, preserveEscapes: true });
-            if (!this.stopping && current?.session_id === expected.session && claudeSideQuestionPanel(screen, current)) {
-              await this.host.sendInput(session.id, '\u001b', { submitMode: 'single', inputOwner: owner, deadlineMs: Date.now() + 5_000 });
-              this.log.append({ principal: null, cap: 'control', target: `pane:${pane}`, outcome: 'board-side-question-dismissed', detail: JSON.stringify({ address, message_ids: messageIds }) });
-              await new Promise(resolve => setTimeout(resolve, COMPOSER_RECHECK_MS));
-            }
-          } finally { await this.host.releaseInput?.(session.id, owner); }
-        }
-      }
+      if (boardWake && claudeSideQuestionPanel(await this.host.capture(session.id, { lines: 200, preserveEscapes: true }), native))
+        return { delivered: false, reason: 'a Claude /btw panel is open on the pane; the board wake waits unread and no keys were sent' };
       const result = await this.#paneTurn(session, pane, text, { messageIds, harness });
       if (result.refused) throw new Error(result.refused.reason);
       return result.waits ?? result;

package/mail/send.mjs

@@ -427,7 +427,8 @@ export class MailSend {
     const [code, what] = {
       steered: [0, 'steered into its running turn'],
       started: [0, 'it was idle, so the message started a turn'],
-      native_waits: [EXIT_STORED, 'Stored: its native session is not known to be idle, so the message waits unread until it is idle. Do not resend'],
+      native_busy: [0, 'it will read it at its next tool call'],
+      native_waits: [EXIT_STORED, 'Stored: its relay could not deliver it and its session is busy, so its pane fallback waits unread until it is idle. Do not resend'],
       native_idle: [0, 'it was idle, so the message starts its next turn'],
       hooked: [0, 'queued in its mailbox; terminal delivery waits for native transcript confirmation'],
       hook_woken: [0, 'it was idle, so its pane was told to read its mailbox; its hook shows it the message in that turn'],
@@ -497,15 +498,8 @@ export class MailSend {
       if (held?.queued) return 'pane_held';
       const busy = door.session.status !== 'idle' || door.session.panes_current === false;
       if (!wake && !busy) throw new Refused('queue');
-      if (busy) {
-        attempt?.begin('native-mailbox');
-        this.store.deliver(to, sent);
-        this.store.requestWake(to, sent.id, { keep: true });
-        const state = this.store.wakeState(to, sent.id);
-        this.store.setWakeState(to, sent.id, { ...state, waiting_for_idle: true, next_at_ms: Date.now() + 300_000 });
-        if (notifyWhenIdle && !cameThroughLink(sent)) this.store.subscribeIdle(to, { message_id: sent.id, subscriber: sent.from, notice: true, final_reply: false });
-        return 'native_waits';
-      }
+      // a busy session is handed it at once, as an idle one is: Claude Code's own cross-session message surfaces at the
+      // receiver's next tool call (D391). Only the pane fallback below waits for idle, since a busy pane is never typed into
       // its reply line names the door this session really has: its tools when it runs supercode's MCP server
       const handed = sent.reply_via?.mode === 'command' && door.session.tools ? { ...sent, reply_via: { mode: 'tool' } } : sent;
       const text = render(handed);
@@ -555,7 +549,15 @@ export class MailSend {
       const received = this.store.claimOf(to, handed.id)?.recipient === true;
       // the relay produced no receipt and cannot send it late (known not made, or not in the receiver's transcript after
       // its bound, its queue gone): a session with a daemon pane has the same copy typed there instead
-      if (!relayed.delivered && !received && wake && door.pane && (!relayed.unknown || relayed.landed === false)) {
+      if (!relayed.delivered && !received && wake && door.pane && (!relayed.unknown || relayed.landed === false) && busy) {
+        // its pane is typed into only once its turn is idle: the copy waits unread, its wake resumed when it goes idle
+        // (carrier.mjs), which hands it to the relay again first
+        this.log(`relay could not deliver ${shortId(handed.id)} to busy ${to} (${relayed.detail}); its pane fallback waits for idle`);
+        try { this.store.requestWake(to, handed.id, { keep: true }); } catch { /* the wake filed above stands */ }
+        const state = this.store.wakeState(to, handed.id);
+        this.store.setWakeState(to, handed.id, { ...state, waiting_for_idle: true, next_at_ms: Date.now() + 300_000 });
+        delivered = 'native_waits';
+      } else if (!relayed.delivered && !received && wake && door.pane && (!relayed.unknown || relayed.landed === false)) {
         this.log(`relay could not deliver ${shortId(handed.id)} to ${to} (${relayed.detail}); typing it into its pane ${door.pane}`);
         this.relaying.add(key);
         try { delivered = await this.#typeIntoPane(handed, to, door.pane, wake, attempt); }
@@ -573,7 +575,7 @@ export class MailSend {
           const now = this.store.findIn(to, handed.id);
           if (now?.state === 'unread') { try { this.store.markRead(to, now.path); } catch { /* moved meanwhile */ } }
         }
-        delivered = 'native_idle';
+        delivered = busy ? 'native_busy' : 'native_idle';
       }
     } else if (door.kind === 'hook') {
       attempt?.begin('hook-mailbox');
@@ -745,7 +747,7 @@ export class MailSend {
       if (error instanceof Refused) throw new Error(`${error.bytes} bytes; the limit is ${MAX_RELAYED_BYTES}`);
       throw error;
     }
-    return { how, text: { steered: 'steered into its running turn', started: 'started a turn', native_waits: 'waits unread until its native turn is idle', native_idle: 'starts its next turn', hooked: 'in its mailbox, its hook shows it', queued: 'waits in its mailbox', operator: 'filed', already: 'it already had it', stored: 'stored; it has no delivery door', typed: 'typed into its pane', pane_held: 'held for its pane until its composer is free', pane_waits: 'waits in its mailbox (its composer was not empty)' }[how] };
+    return { how, text: { steered: 'steered into its running turn', started: 'started a turn', native_busy: 'read at its next tool call', native_waits: 'waits unread until its native turn is idle', native_idle: 'starts its next turn', hooked: 'in its mailbox, its hook shows it', queued: 'waits in its mailbox', operator: 'filed', already: 'it already had it', stored: 'stored; it has no delivery door', typed: 'typed into its pane', pane_held: 'held for its pane until its composer is free', pane_waits: 'waits in its mailbox (its composer was not empty)' }[how] };
   }
 
   // ---- a filing --------------------------------------------------------------------------------------------------
@@ -848,7 +850,7 @@ export class MailSend {
     return {
       delivered_to_bus: door.kind !== 'stored' && !['pane_waits', 'pane_held', 'native_waits'].includes(how), message_id: sent.id, reply_to: sender,
       target: { harness, session_id: sessionId, name: door.kind === 'native' ? door.session.name.split('@')[0] : null },
-      delivery: { door: door.kind, how: { steered: 'steered', started: 'started', native_waits: 'waiting_for_idle', native_idle: 'started', hooked: 'hook', queued: 'queued', stored: 'stored', operator: 'filed', already: 'already', typed: 'typed', pane_held: 'held', pane_waits: 'stored' }[how] },
+      delivery: { door: door.kind, how: { steered: 'steered', started: 'started', native_busy: 'next_tool_call', native_waits: 'waiting_for_idle', native_idle: 'started', hooked: 'hook', queued: 'queued', stored: 'stored', operator: 'filed', already: 'already', typed: 'typed', pane_held: 'held', pane_waits: 'stored' }[how] },
     };
   }
 

package/mail/sqlite.mjs

@@ -10,12 +10,12 @@ import { RECORD_SCHEMA } from '../record-state.mjs';
 
 export const MAIL_SCHEMA_VERSION = 3;
 export const MAIL_RETENTION = Object.freeze({
-  bodyDays: 30,
+  relayTextDays: 30,
   maxReadHeaders: 1_000_000,
   maxCarrierRows: 10_000,
   maxTerminalLogRows: 10_000,
 });
-export const RETAINED_BODY = `[this message's body was let go after ${MAIL_RETENTION.bodyDays} days (mail retention); its header and thread are kept]`;
+export const RETAINED_BODY = `[this message's body was not kept; its header and thread are kept]`;
 
 const SCHEMA = `
 CREATE TABLE IF NOT EXISTS mail_metadata (
@@ -331,51 +331,24 @@ export class MailDatabase extends EventEmitter {
     });
   }
 
-  /** Only settled read mail is eligible. Pending deliveries, subscriptions and
-   * claim holders protect their originals; an unresolved message never loses its body. */
+  /** Mail bodies are kept. Retention removes old relay scratch copies (their
+   * originals stay in the mailbox) and caps headers whose body is already gone. */
   retain(now = Date.now()) {
-    const cutoff = now - this.retention.bodyDays * 86_400_000;
+    const cutoff = now - this.retention.relayTextDays * 86_400_000;
     return this.transaction(() => {
-      const expired = this.statement(`UPDATE messages SET body_digest = coalesce(body_digest, supercode_sha256(body)), body = NULL, first_line = '', body_expired_at_ms = ?
-        WHERE state = 'cur' AND created_at_ms < ? AND body IS NOT NULL
-        AND NOT EXISTS (SELECT 1 FROM deliveries d WHERE d.mailbox = messages.mailbox AND d.message = messages.id)
-        AND NOT EXISTS (SELECT 1 FROM subscriptions s WHERE s.mailbox = messages.mailbox AND s.message = messages.id)`)
-        .run(now, cutoff).changes;
-      // Completed native questions duplicate an envelope and its answer input.
-      // Keep the claim/binding/header so a repeat cannot submit another answer,
-      // but let go of those text copies after the mailbox copies have expired.
-      const questions = this.statement(`UPDATE mail_records AS r SET record = json_set(
-        json_remove(record, '$.native', '$.answers'), '$.envelope.body', ?, '$.body_expired_at_ms', ?)
-        WHERE collection = 'native_questions' AND json_type(record, '$.envelope') = 'object'
-        AND json_type(record, '$.body_expired_at_ms') IS NULL
-        AND json_extract(record, '$.envelope.created_at_ms') < ?
-        AND EXISTS (SELECT 1 FROM mail_records d WHERE d.collection = 'native_questions'
-          AND d.key = substr(r.key, 1, instr(r.key, '.') - 1) || '.delivered' AND CAST(d.record AS INTEGER) < ?)
-        AND NOT EXISTS (SELECT 1 FROM mail_records w WHERE w.collection = 'native_questions'
-          AND w.key = substr(r.key, 1, instr(r.key, '.') - 1) || '.waiting')
-        AND NOT EXISTS (SELECT 1 FROM messages m WHERE m.id = json_extract(r.record, '$.envelope.id') AND m.body IS NOT NULL)`)
-        .run(RETAINED_BODY, now, cutoff, cutoff).changes;
       const relayRecords = this.statement(`DELETE FROM mail_records AS r WHERE collection = 'relays'
         AND (key LIKE '%/turn.txt' OR key LIKE '%/receipt.json') AND updated_at_ms < ?
         AND NOT EXISTS (SELECT 1 FROM mail_records q WHERE q.collection = 'relays'
           AND q.key = substr(r.key, 1, instr(r.key, '/')) || 'queue.json')`).run(cutoff).changes;
-      const launchInputs = this.statement(`UPDATE mail_records SET record = json_set(
-        json_remove(record, '$.initialInput', '$.claudeInput.text'), '$.input_body_expired_at_ms', ?)
-        WHERE collection = 'native_launches' AND json_type(record, '$.input_body_expired_at_ms') IS NULL
-        AND json_type(record, '$.input_settled_at_ms') = 'integer'
-        AND json_extract(record, '$.input_settled_at_ms') < ?
-        AND (json_type(record, '$.initialInput') = 'text' OR json_type(record, '$.claudeInput.text') = 'text')`)
-        .run(now, cutoff).changes;
       const removed = this.statement(`DELETE FROM messages WHERE (mailbox, id) IN (
         SELECT mailbox, id FROM messages m WHERE state = 'cur' AND body_expired_at_ms IS NOT NULL
         AND NOT EXISTS (SELECT 1 FROM deliveries d WHERE d.mailbox = m.mailbox AND d.message = m.id)
         AND NOT EXISTS (SELECT 1 FROM subscriptions s WHERE s.mailbox = m.mailbox AND s.message = m.id)
         ORDER BY created_at_ms DESC, id DESC LIMIT -1 OFFSET ?)`)
         .run(this.retention.maxReadHeaders).changes;
-      const result = { at_ms: now, expired_bodies: expired, expired_question_records: questions,
-        removed_relay_text_records: relayRecords, expired_launch_inputs: launchInputs, removed_headers: removed, policy: this.retention };
+      const result = { at_ms: now, removed_relay_text_records: relayRecords, removed_headers: removed, policy: this.retention };
       this.setMetadata('retention', result);
-      if (expired || questions || relayRecords || launchInputs || removed) this.changed('retention', result);
+      if (relayRecords || removed) this.changed('retention', result);
       return result;
     });
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@volter/supercode-teams",
-  "version": "0.3.196",
+  "version": "0.3.198",
   "type": "module",
   "description": "A machine daemon serving this machine's doors, and a local or remote Teams server for shared session discovery, OpenTelemetry ingestion and access to enrolled machines.",
   "license": "MIT",