@volter/supercode-teams 0.3.129 → 0.3.130

package/mail/relay.mjs

@@ -373,8 +373,8 @@ export class Relays {
    */
   /** Whether `sender`'s relay still holds a send queued (its queue file, written within SEND_TIMEOUT_MS twice over): a
    *  relay runtime outlives a daemon that ended mid-send and may still make it. */
-  sending(sender) {
-    try { return Date.now() - statSync(this.spec(sender).queue).mtimeMs < 2 * SEND_TIMEOUT_MS; } catch { return false; }
+  sending(sender, senderName) {
+    try { return Date.now() - statSync(this.spec(sender, senderName).queue).mtimeMs < 2 * SEND_TIMEOUT_MS; } catch { return false; }
   }
 
   /** Whether message `messageId` from `sender` landed in Claude session `sessionId`'s transcript, as a relay send reads

package/mail/send.mjs

@@ -330,13 +330,29 @@ export class MailSend {
       // at its first tool call finds it, and a daemon that ends before the relay answers (an upgrade's restart, a crash)
       // leaves it for the carrier, which hands it over again (t_b0f9a2a0: filed read before, such a send was lost). It is
       // read once the relay has made it; a relay that fails takes back the copy this delivery filed
+      // never beside another hand-over of the same message: one running here, one a relay that outlived its daemon may
+      // still be making, or one an earlier daemon made that landed (every path: a wake's handOver, an agent's re-plan)
+      const key = `${to}\n${handed.id}`;
+      const unanswered = (detail) => Object.assign(new Error(detail), { unanswered: true });
+      if (this.relaying.has(key)) throw unanswered('its relay send is still running here');
+      const earlier = this.store.hasWake(to, handed.id) ? this.store.wakeState(to, handed.id).handover_at_ms : null;
+      if (earlier != null) {
+        if (this.relays.sending?.(handed.from, handed.from_name)) throw unanswered('its relay may still be making it');
+        let landed = false;
+        try { landed = await this.relays.landed(door.session.address.split(':').slice(3).join(':'), handed.id, handed.from); } catch { /* unread: hand it over */ }
+        if (landed) {
+          try { this.store.claimFor(to, handed.id, { pid: process.pid, via: 'door', caller: null, recipient: true, at_ms: Date.now() }); } catch { /* best kept */ }
+          const now = this.store.findIn(to, handed.id);
+          if (now?.state === 'unread') { try { this.store.markRead(to, now.path); } catch { /* moved meanwhile */ } }
+          return 'already';
+        }
+      }
       const filedHere = this.store.findIn(to, handed.id) == null;
       const filed = this.store.deliver(to, handed);
       if (wake && !this.store.hasWake(to, handed.id)) {
         // its hand-over begins now: a daemon that restarts reads the receiver's transcript before sending it again
         try { this.store.requestWake(to, handed.id); this.store.setWakeState(to, handed.id, { attempts: 0, next_at_ms: Date.now() + RELAY_GRACE_MS, handover_at_ms: Date.now() }); } catch { /* the relay's own outcome stands */ }
       }
-      const key = `${to}\n${handed.id}`;
       this.relaying.add(key);
       let relayed;
       try { relayed = await this.relays.send(handed.from, handed.from_name, { sessionId: door.session.address.split(':').slice(3).join(':'), name: door.session.name.split('@')[0], socket: door.session.socket }, text, handed.id); }
@@ -783,19 +799,6 @@ export class MailSend {
     // tried again after its backoff, settled by that hand-over's own outcome
     const key = `${address}\n${id}`;
     if (this.handing.has(key) || this.relaying.has(key)) return { outcome: 'later', detail: 'its hand-over is still in progress' };
-    // an earlier hand-over never settled (the daemon ended mid-way): one its receiver's transcript holds landed, and is
-    // read; any other is handed over again, never given up (t_b0f9a2a0)
-    if (state.handover_at_ms != null && door.kind === 'native') {
-      // a relay that outlived its daemon may still be making that send (its queue still there, fresh): waited for, never
-      // sent beside it (#1264 review P2a)
-      if (this.relays.sending?.(sent.from, id)) return { outcome: 'later', detail: 'its relay may still be making it' };
-      let landed = false;
-      try { landed = await this.relays.landed(door.session.address.split(':').slice(3).join(':'), id, sent.from); } catch { /* unread: hand it over */ }
-      if (landed) {
-        try { this.store.claimFor(address, id, { pid: process.pid, via: 'door', caller: null, recipient: true, at_ms: Date.now() }); this.store.markRead(address, found.path); } catch { /* moved meanwhile */ }
-        return { outcome: 'filed', detail: 'an earlier hand-over landed' };
-      }
-    }
     this.store.setWakeState(address, id, { ...state, handover_at_ms: Date.now() });
     this.handing.add(key);
     try {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@volter/supercode-teams",
-  "version": "0.3.129",
+  "version": "0.3.130",
   "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",