@fitchmultz/pi-subagents 0.52.16 → 0.52.18

package/CHANGELOG.md

@@ -2,6 +2,36 @@
 
 ## [Unreleased]
 
+## 0.52.18 - 2026-11-06
+
+<!-- npm-release:unreleased:start -->
+
+- Keep replied or retired asks out of reply attribution when their queued input reaches a later native turn.
+- Preserve unanswered queued asks through abort recovery without sending a premature recipient-failure reply.
+
+<!-- npm-release:unreleased:end -->
+
+- Keep retired asks out of native reply attribution (#341) ([27a93f9](https://github.com/fitchmultz/pi-subagents/commit/27a93f9bc52aaa9e454ffc593017b66b01461bc4))
+
+### What's Changed
+
+- Keep retired asks out of native reply attribution by @fitchmultz in https://github.com/fitchmultz/pi-subagents/pull/341
+
+**Full Changelog**: https://github.com/fitchmultz/pi-subagents/compare/v0.52.17...v0.52.18
+
+## 0.52.17 - 2026-11-05
+
+<!-- npm-release:unreleased:start -->
+<!-- npm-release:unreleased:end -->
+
+- Adopt strict Oxlint/Oxfmt v5.2 without finding baselines (#339) ([8de822c](https://github.com/fitchmultz/pi-subagents/commit/8de822c14a56fdad3e13961abb02a106dac71a4f))
+
+### What's Changed
+
+- Adopt strict Oxlint/Oxfmt v5.2 without finding baselines by @fitchmultz in https://github.com/fitchmultz/pi-subagents/pull/339
+
+**Full Changelog**: https://github.com/fitchmultz/pi-subagents/compare/v0.52.16...v0.52.17
+
 ## 0.52.16 - 2026-11-03
 
 <!-- npm-release:unreleased:start -->

package/dist/agent/host.js

@@ -75,7 +75,7 @@ export async function loadSDK(root) {
         return resolvedSDK;
     }
     // Selected host SDK, not a second package-local runtime hidden behind Jiti aliases.
-    // oxlint-disable-next-line typescript/no-unsafe-assignment
+    // oxlint-disable-next-line typescript/no-unsafe-assignment -- QX-004: selected host module must retain its runtime identity.
     const sdk = await import(__rewriteRelativeImportExtension(pathToFileURL(path.join(root, "dist/index.js")).href));
     return sdk;
 }

package/dist/agent/notifications.js

@@ -5,17 +5,19 @@ import { readOutputPage } from "../shared/output-page.js";
 import { journalStamp } from "../shared/native-journal.js";
 import { launchAt, list, runDirectory } from "./storage.js";
 import { readCanonicalResult, readResult } from "./usage.js";
+function isStringSet(value) {
+    if (!(value instanceof Set)) {
+        return false;
+    }
+    const entries = value;
+    return [...entries].every((entry) => typeof entry === "string");
+}
 function queuedAdmissions() {
     const store = globalThis;
     const existing = store.__pi_single_agent_admissions__;
-    if (existing instanceof Set) {
-        const values = existing;
-        const strings = [...values].filter((value) => typeof value === "string");
-        if (strings.length === values.size) {
-            // Keep one live queue identity across native extension reloads.
-            // oxlint-disable-next-line typescript/no-unsafe-return
-            return existing;
-        }
+    if (isStringSet(existing)) {
+        // Keep one live queue identity across native extension reloads.
+        return existing;
     }
     const queued = new Set();
     store.__pi_single_agent_admissions__ = queued;

package/dist/agent/observe-layout.js

@@ -19,11 +19,11 @@ if (manifest === undefined) {
     throw new Error("Cannot locate the native Pi TUI package.");
 }
 const root = pathToFileURL(path.join(path.dirname(manifest), "dist/index.js"));
-// oxlint-disable-next-line typescript/no-unsafe-assignment
+// oxlint-disable-next-line typescript/no-unsafe-assignment -- QX-004: resolve the selected host TUI, not a second package instance.
 const tuiPromise = import(__rewriteRelativeImportExtension(root.href));
-// oxlint-disable-next-line typescript/no-unsafe-assignment
+// oxlint-disable-next-line typescript/no-unsafe-assignment -- QX-004: layout must share the selected host TUI instance.
 const layoutPromise = import(__rewriteRelativeImportExtension(new URL("./layout.js", root).href));
-// oxlint-disable-next-line typescript/no-unsafe-assignment
+// oxlint-disable-next-line typescript/no-unsafe-assignment -- QX-004: dispatch must share the selected host TUI instance.
 const dispatchPromise = import(__rewriteRelativeImportExtension(new URL("./tui.js", root).href));
 const [tui, layout, dispatch, sdk] = await Promise.all([
     tuiPromise,

package/dist/agent/parking.js

@@ -297,7 +297,6 @@ async function awaitWake(base, launch, signal, legacy) {
                 throw new Error(failure);
             }
             // DirectoryWatch reconciles dropped hints without imposing a deadline on native startup.
-            // oxlint-disable-next-line no-await-in-loop
             await wake.sleep(200);
         }
     }

package/dist/agent/process.js

@@ -38,9 +38,7 @@ export async function observeAlive(identities) {
     for (const claim of identities) {
         // One `ps` at a time: each spawn briefly holds the event loop, so concurrent spawns add up.
         const { pid, identity, clock } = claim;
-        verdicts.push(probe(claim) ??
-            // oxlint-disable-next-line no-await-in-loop
-            !(await observeChildProcessReplaced(pid, identity, claim.published, clock)));
+        verdicts.push(probe(claim) ?? !(await observeChildProcessReplaced(pid, identity, claim.published, clock)));
     }
     return verdicts;
 }
@@ -198,7 +196,6 @@ async function awaitStopping(dir, existing, signal) {
                 throw new Error("Previous actor is still stopping; no replacement was launched.");
             }
             // The old actor must actually exit before replacing its generation.
-            // oxlint-disable-next-line no-await-in-loop
             await delay(25);
         }
     }

package/dist/agent/runner.js

@@ -310,13 +310,11 @@ class Actor {
                     this.retry = false;
                     this.pending().forEach((command) => this.accept(command));
                 }
-                // oxlint-disable-next-line no-await-in-loop
                 if (await this.park(runtime, parked)) {
                     return;
                 }
                 const polling = this.retry || this.stopping || watch === "unwatched";
                 const tick = Math.max(1, this.nextParentCheck - performance.now());
-                // oxlint-disable-next-line no-await-in-loop
                 await this.wake.sleep(polling ? 100 : tick);
             }
         }

package/dist/agent/shutdown.js

@@ -37,7 +37,6 @@ async function waitForExit(targets, timeout) {
     const deadline = Date.now() + timeout;
     while (Date.now() < deadline && targets.some(alive)) {
         // Join actual process exit while native abort persists the interrupted turn.
-        // oxlint-disable-next-line no-await-in-loop
         await delay(25);
     }
 }

package/dist/agent/typebox.js

@@ -13,11 +13,11 @@ import { isChild, packageRoot } from "./host.js";
 // Pi aliases these imports to its own instance (including bundled virtual modules).
 // Only the detached runner needs absolute imports from the selected host package.
 const require = isChild() ? createRequire(path.join(packageRoot(), "dist/index.js")) : undefined;
-// oxlint-disable-next-line typescript/no-unsafe-assignment
+// oxlint-disable-next-line typescript/no-unsafe-assignment -- QX-004: detached runners resolve the selected host schema instance.
 const typeboxPromise = require === undefined
     ? import("typebox")
     : import(__rewriteRelativeImportExtension(pathToFileURL(require.resolve("typebox")).href));
-// oxlint-disable-next-line typescript/no-unsafe-assignment
+// oxlint-disable-next-line typescript/no-unsafe-assignment -- QX-004: validation must use the selected host schema instance.
 const valuePromise = require === undefined
     ? import("typebox/value")
     : import(__rewriteRelativeImportExtension(pathToFileURL(require.resolve("typebox/value")).href));

package/dist/build-info.js

@@ -1 +1 @@
-export const EXTENSION_BUILD = Object.freeze({"version":"0.52.16","sha256":"ec0886af93d91687cfae1ed692e1e2e31127d3fbf98abadf192034a044e44d51"});
+export const EXTENSION_BUILD = Object.freeze({"version":"0.52.18","sha256":"83db07f9acbdd6ff5985c46e1f5f5411f68b6766697a2dfab019b3d00dbba1c1"});

package/dist/pi-intercom/bridge-presence.js

@@ -88,7 +88,6 @@ export class PersistentPresence {
             }
             try {
                 // Each identity has one connection attempt; preserve the native broker's spawn safeguards.
-                // oxlint-disable-next-line no-await-in-loop
                 await this.owner.connect(identity, registration.expires);
             }
             catch {

package/dist/pi-intercom/bridge-protocol.js

@@ -25,7 +25,6 @@ export function object(value, fields) {
 }
 export function label(value, field, max = 512) {
     // Labels are single-line identifiers and paths; reject C0/C1 controls and DEL.
-    // oxlint-disable-next-line no-control-regex
     const controls = /[\u0000-\u001f\u007f-\u009f]/;
     if (typeof value !== "string" ||
         value.trim() === "" ||
@@ -37,7 +36,6 @@ export function label(value, field, max = 512) {
 }
 export function text(value) {
     // Message text permits tab/newline/CR, but rejects other C0/C1 controls and DEL.
-    // oxlint-disable-next-line no-control-regex
     const controls = /[\u0000-\u0008\u000b\u000c\u000e-\u001f\u007f-\u009f]/;
     if (typeof value !== "string" || value.trim() === "" || controls.test(value)) {
         throw new Fault(400, "invalid_input", "Invalid message.");

package/dist/pi-intercom/bridge-session.js

@@ -303,7 +303,6 @@ export class Session {
             log("inbox_wait", { ...this.fields(), requestId: context.requestId, after: operation.after });
             try {
                 // A nonmatching commit must rearm the same cursor, not start parallel readers.
-                // oxlint-disable-next-line no-await-in-loop
                 await delay(remaining, undefined, { signal });
             }
             catch (error) {

package/dist/pi-intercom/bridge-wake.js

@@ -167,7 +167,6 @@ export class WakeDeliveries {
             // ponytail: two cold Macs can each admit one wake. Literal cross-Mac single admission
             // requires a neutral quota authority governing every writer, not a URL hash or HTTP 2xx.
             try {
-                // oxlint-disable-next-line no-await-in-loop
                 await this.post(candidate);
             }
             finally {

package/dist/pi-intercom/broker/client.js

@@ -148,12 +148,10 @@ export class IntercomClient extends EventEmitter {
     async restoreTopics(topics, subscriptions) {
         for (const topic of topics ?? []) {
             // Restore frames are individually acknowledged to preserve order and bound socket payloads.
-            // oxlint-disable-next-line no-await-in-loop
             await this.updateTopics({ action: "restore", topic });
         }
         for (const subscription of subscriptions ?? []) {
             // Subscriptions are committed before connect resolves and subsequent messages can arrive.
-            // oxlint-disable-next-line no-await-in-loop
             await this.updateTopics({ action: "restore", subscription });
         }
     }

package/dist/pi-intercom/broker/spawn.js

@@ -138,7 +138,6 @@ async function previousBrokerWarning() {
     const previousSockets = getPreviousBrokerSocketPaths();
     for (const socketPath of previousSockets) {
         // Migration probes are sequential and never register or send to old peers.
-        // oxlint-disable-next-line no-await-in-loop
         if (isOwnedBrokerSocket(socketPath) && (await checkOneSocket(socketPath))) {
             return "An older agent-directory Intercom broker is still running. Its peers are not part of shared same-user discovery until they reload with the updated extension. Existing sessions and accepted work are untouched; no legacy fallback is used.";
         }
@@ -240,12 +239,10 @@ async function waitForBroker(timeoutMs = 5000) {
     const start = Date.now();
     while (Date.now() - start < timeoutMs) {
         // Each probe must finish before the next to avoid parallel socket attempts.
-        // oxlint-disable-next-line no-await-in-loop
         if (await checkSocketConnectable()) {
             return;
         }
         // Polling waits between probes rather than starting an unbounded batch.
-        // oxlint-disable-next-line no-await-in-loop
         await sleep(100);
     }
     throw new Error("Broker failed to start within timeout");

package/dist/pi-intercom/inbound-delivery.js

@@ -131,7 +131,7 @@ export class InboundDeliveryOwner {
         const trigger = index === -1 ? undefined : rest.splice(index, 1)[0];
         const replyTarget = entries.find((entry) => entry.message.expectsReply === true) ?? trigger;
         if (replyTarget && this.journal.get(replyTarget.message.id) === replyTarget) {
-            this.hooks.replies.queueTurnContext({ ...replyTarget, receivedAt: Date.now() });
+            this.hooks.replies.queueTurnContext(replyTarget.message.id);
         }
         for (const entry of rest) {
             if (!this.live(generation)) {
@@ -218,11 +218,7 @@ export class InboundDeliveryOwner {
             this.journal.markNative(entry.message.id);
         }
         if (delivery !== "passive") {
-            this.hooks.replies.queueTurnContext({
-                from: entry.from,
-                message: entry.message,
-                receivedAt: Date.now(),
-            });
+            this.hooks.replies.queueTurnContext(entry.message.id);
         }
     }
     nativeMessage(entry) {

package/dist/pi-intercom/reply-tracker.js

@@ -55,9 +55,9 @@ function replyBySender(contexts, to) {
 }
 export class ReplyTracker {
     pendingAsks = new Map();
-    pendingTurnContexts = [];
-    currentTurnContext = null;
-    activeAgentContext = null;
+    pendingTurnIds = [];
+    currentTurnId = null;
+    activeAgentId = null;
     askTimeoutMs;
     constructor(askTimeoutMs = 10 * 60 * 1000) {
         this.askTimeoutMs = askTimeoutMs;
@@ -70,61 +70,58 @@ export class ReplyTracker {
         }
         return context;
     }
-    queueTurnContext(context) {
-        if (context.message.expectsReply !== true) {
+    queueTurnContext(messageId) {
+        if (!this.pendingAsks.has(messageId) || this.hasTurnContext(messageId)) {
             return;
         }
-        if (this.hasTurnContext(context.message.id)) {
-            return;
-        }
-        this.pendingTurnContexts.push(context);
+        this.pendingTurnIds.push(messageId);
     }
     beginTurn(now = Date.now()) {
         this.pruneExpired(now);
-        this.currentTurnContext = this.pendingTurnContexts.shift() ?? null;
-        if (this.currentTurnContext) {
-            this.activeAgentContext = this.currentTurnContext;
+        this.currentTurnId = this.pendingTurnIds.shift() ?? null;
+        if (this.currentTurnId !== null) {
+            this.activeAgentId = this.currentTurnId;
         }
     }
     currentTurn() {
-        return this.currentTurnContext ?? this.activeAgentContext;
+        const id = this.currentTurnId ?? this.activeAgentId;
+        return id === null ? null : (this.pendingAsks.get(id) ?? null);
     }
     endTurn() {
-        this.currentTurnContext = null;
+        this.currentTurnId = null;
     }
     endAgent() {
-        this.currentTurnContext = null;
-        this.activeAgentContext = null;
+        this.currentTurnId = null;
+        this.activeAgentId = null;
     }
     reset() {
         this.pendingAsks.clear();
-        this.pendingTurnContexts.length = 0;
-        this.currentTurnContext = null;
-        this.activeAgentContext = null;
+        this.pendingTurnIds.length = 0;
+        this.endAgent();
     }
     expireSender(sessionId) {
-        const expires = (context) => context.from.id === sessionId;
+        const expiring = new Set([...this.pendingAsks.values()]
+            .filter((context) => context.from.id === sessionId)
+            .map((context) => context.message.id));
         let expired = 0;
-        for (const [messageId, context] of this.pendingAsks) {
-            if (expires(context)) {
-                this.pendingAsks.delete(messageId);
-                expired += 1;
-            }
+        for (const messageId of expiring) {
+            this.pendingAsks.delete(messageId);
+            expired += 1;
         }
-        const beforeQueued = this.pendingTurnContexts.length;
-        for (let index = this.pendingTurnContexts.length - 1; index >= 0; index -= 1) {
-            const context = this.pendingTurnContexts[index];
-            if (context !== undefined && expires(context)) {
-                this.pendingTurnContexts.splice(index, 1);
+        const beforeQueued = this.pendingTurnIds.length;
+        for (let index = this.pendingTurnIds.length - 1; index >= 0; index -= 1) {
+            const id = this.pendingTurnIds[index];
+            if (id !== undefined && expiring.has(id)) {
+                this.pendingTurnIds.splice(index, 1);
             }
         }
-        expired += beforeQueued - this.pendingTurnContexts.length;
-        if (this.currentTurnContext && expires(this.currentTurnContext)) {
-            this.currentTurnContext = null;
+        expired += beforeQueued - this.pendingTurnIds.length;
+        if (this.currentTurnId !== null && expiring.has(this.currentTurnId)) {
+            this.currentTurnId = null;
             expired += 1;
         }
-        if (this.activeAgentContext && expires(this.activeAgentContext)) {
-            this.activeAgentContext = null;
+        if (this.activeAgentId !== null && expiring.has(this.activeAgentId)) {
+            this.activeAgentId = null;
             expired += 1;
         }
         return expired;
@@ -132,26 +129,21 @@ export class ReplyTracker {
     resolveReplyTarget(options, now = Date.now()) {
         this.pruneExpired(now);
         const pending = Array.from(this.pendingAsks.values());
-        const contexts = this.currentTurnContext
-            ? [
-                this.currentTurnContext,
-                ...pending.filter((context) => context.message.id !== this.currentTurnContext?.message.id),
-            ]
-            : pending;
         const replyTo = options.replyTo ?? "";
         const to = options.to ?? "";
         if (replyTo !== "") {
-            return explicitReply(contexts, replyTo, to);
+            return explicitReply(pending, replyTo, to);
         }
         if (to !== "") {
-            return replyBySender(contexts, to);
+            return replyBySender(pending, to);
         }
         return this.implicitReplyTarget(pending);
     }
     /** Without an explicit target, the turn context wins, then a sole pending ask. */
     implicitReplyTarget(pending) {
-        if (this.currentTurnContext) {
-            return this.currentTurnContext;
+        const current = this.currentTurnId === null ? undefined : this.pendingAsks.get(this.currentTurnId);
+        if (current) {
+            return current;
         }
         const [solePending] = pending;
         if (pending.length === 1 && solePending !== undefined) {
@@ -170,22 +162,22 @@ export class ReplyTracker {
         return Array.from(this.pendingAsks.values()).sort((a, b) => a.receivedAt - b.receivedAt);
     }
     hasTurnContext(messageId) {
-        return (this.pendingTurnContexts.some((context) => context.message.id === messageId) ||
-            this.currentTurnContext?.message.id === messageId ||
-            this.activeAgentContext?.message.id === messageId);
+        return (this.pendingTurnIds.includes(messageId) ||
+            this.currentTurnId === messageId ||
+            this.activeAgentId === messageId);
     }
     removeContext(messageId) {
         this.pendingAsks.delete(messageId);
-        for (let index = this.pendingTurnContexts.length - 1; index >= 0; index -= 1) {
-            if (this.pendingTurnContexts[index]?.message.id === messageId) {
-                this.pendingTurnContexts.splice(index, 1);
+        for (let index = this.pendingTurnIds.length - 1; index >= 0; index -= 1) {
+            if (this.pendingTurnIds[index] === messageId) {
+                this.pendingTurnIds.splice(index, 1);
             }
         }
-        if (this.currentTurnContext?.message.id === messageId) {
-            this.currentTurnContext = null;
+        if (this.currentTurnId === messageId) {
+            this.currentTurnId = null;
         }
-        if (this.activeAgentContext?.message.id === messageId) {
-            this.activeAgentContext = null;
+        if (this.activeAgentId === messageId) {
+            this.activeAgentId = null;
         }
     }
     pruneExpired(now) {

package/dist/pi-intercom/session-events.js

@@ -46,6 +46,8 @@ export class IntercomSessionEvents {
         if (client?.isConnected() !== true ||
             !context ||
             context.message.expectsReply !== true ||
+            // A native turn can start without consuming its queued input after an abort.
+            journal.get(context.message.id) !== undefined ||
             error === null ||
             error === "") {
             return;

package/dist/pi-intercom/tool-registration.js

@@ -9,8 +9,7 @@ export function registerIntercomTools(pi, tools) {
         description: "Message connected Pi sessions in your group; mains also reach other mains across projects. List shows target IDs. Ask waits for a correlated reply; reply answers the active or sole pending ask. Pending lists asks; status shows connection and inbound queue. Last pages a peer's last completed response without waking it. Broker acceptance is not recipient persistence, consumption or completion.",
         parameters: intercomSchema,
         renderResult: renderCompactResult,
-        // oxlint-disable-next-line max-params
-        async execute(_id, params, signal, _update, ctx) {
+        async execute(...[, params, signal, , ctx]) {
             const result = throwIfToolError(await tools.execute(params, signal, ctx));
             return { content: [...result.content], details: result.details };
         },

package/dist/pi-intercom/validation.js

@@ -34,10 +34,8 @@ export function asError(error) {
     return error instanceof Error ? error : new Error(errorMessage(error));
 }
 // Protocol text permits line feeds and tabs, but rejects other C0/C1 controls.
-// oxlint-disable-next-line no-control-regex
 const UNSAFE_TEXT_CONTROLS = /[\u0000-\u0008\u000b\u000c\u000e-\u001f\u007f-\u009f]/;
 // Protocol labels are single-line fields and reject every C0/C1 control.
-// oxlint-disable-next-line no-control-regex
 const UNSAFE_LABEL_CONTROLS = /[\u0000-\u001f\u007f-\u009f]/;
 export function safeText(value) {
     return typeof value === "string" && !UNSAFE_TEXT_CONTROLS.test(value);

package/dist/shared/cooperative.js

@@ -14,7 +14,6 @@ export async function runCooperatively(steps) {
         // single-record stall off-thread rather than increasing this batch size.
         if (++count === 32) {
             // Yield between ordered atomic recovery records; concurrency would reorder delivery.
-            // oxlint-disable-next-line no-await-in-loop
             await yieldToInput();
             count = 0;
         }