@fitchmultz/pi-subagents 0.52.7 → 0.52.8

package/CHANGELOG.md

@@ -2,6 +2,21 @@
 
 ## [Unreleased]
 
+## 0.52.8 - 2026-11-02
+
+<!-- npm-release:unreleased:start -->
+<!-- npm-release:unreleased:end -->
+
+- Merge pull request #321 from fitchmultz/chore/oxlint-v512-strict-ts ([600fcaa](https://github.com/fitchmultz/pi-subagents/commit/600fcaa9b22d59d4111179a8563332ff0420f9d0))
+
+### What's Changed
+
+- Align Oxlint toolchain and baseline config with the strict standard (1/5) by @fitchmultz in https://github.com/fitchmultz/pi-subagents/pull/319
+- Narrow the node:test registration allowance to package-origin distinct names (2/5) by @fitchmultz in https://github.com/fitchmultz/pi-subagents/pull/320
+- Adopt the strict compiler options and clear every strict-mode error (3/5) by @fitchmultz in https://github.com/fitchmultz/pi-subagents/pull/321
+
+**Full Changelog**: https://github.com/fitchmultz/pi-subagents/compare/v0.52.7...v0.52.8
+
 ## 0.52.7 - 2026-11-01
 
 <!-- npm-release:unreleased:start -->

package/dist/agent/launch.js

@@ -89,7 +89,7 @@ export async function prepareLaunch(options) {
         model: selection.model.id,
         thinking: selection.thinking,
         trusted,
-        name: input.name,
+        ...(input.name !== undefined ? { name: input.name } : {}),
         createdAt: Date.now(),
     };
     // Retain even unpublished native journals on failure, including full fork evidence.

package/dist/agent/observation-view.js

@@ -197,7 +197,7 @@ export class Viewer {
     }
     message(frame, state) {
         const live = state.message;
-        if (live === undefined) {
+        if (live === undefined || live.message === undefined) {
             return undefined;
         }
         const reset = this.key !== live.key;
@@ -213,18 +213,26 @@ export class Viewer {
         this.ended = live.ended;
         const limit = Math.min(content.length, BLOCKS);
         this.blocks.length = Math.min(this.blocks.length, limit);
-        for (let index = 0; index < limit && !frame.full; index++) {
-            const item = this.cursor(index, content[index]).next(index, frame, state);
-            if (item !== undefined) {
-                frame.blocks.push(item);
-            }
-        }
+        this.appendBlocks(frame, state, content, limit);
         if (!changed && frame.blocks.length === 0) {
             return undefined;
         }
         // An ended message stays live until its session entry is found and handed over.
         return `{"key":${live.key},"reset":${reset},"ended":${live.ended},"count":${content.length},"blocks":[${frame.blocks.join(",")}]}`;
     }
+    /** Refill the frame from live content blocks, reusing cursors by block identity. */
+    appendBlocks(frame, state, content, limit) {
+        for (let index = 0; index < limit && !frame.full; index++) {
+            const block = content[index];
+            if (block === undefined) {
+                continue;
+            }
+            const item = this.cursor(index, block).next(index, frame, state);
+            if (item !== undefined) {
+                frame.blocks.push(item);
+            }
+        }
+    }
     cursor(index, block) {
         const existing = this.blocks.at(index);
         if (existing?.ref === block) {

package/dist/agent/observe-card.js

@@ -119,7 +119,7 @@ export class Card {
         this.tool.updateResult({
             content: blocks(message.content)
                 .filter((part) => part.type === "text")
-                .map((part) => ({ type: "text", text: part.text })),
+                .map((part) => part.text === undefined ? { type: "text" } : { type: "text", text: part.text }),
             isError: message.isError === true,
         });
         this.tool.setExpanded(this.expanded);
@@ -169,7 +169,7 @@ export class Card {
             this.tool?.updateResult({
                 content: blocks(result.content)
                     .filter((part) => part.type === "text")
-                    .map((part) => ({ type: "text", text: part.text })),
+                    .map((part) => part.text === undefined ? { type: "text" } : { type: "text", text: part.text }),
                 isError,
             }, partial);
         }

package/dist/agent/observe-client.js

@@ -259,7 +259,7 @@ export class ObservationClient {
     }
     /** Retire live state only after its exact journal entries have been indexed. */
     settle(has) {
-        while (this.handovers.length > 0 && has(this.handovers[0][1])) {
+        while (this.handovers.length > 0 && has(this.handovers[0]?.[1] ?? "")) {
             const item = this.handovers.shift();
             if (item) {
                 this.retire(item[2], item[3]);

package/dist/agent/observe.js

@@ -199,7 +199,10 @@ class Observer {
         if (this.ctx !== ctx) {
             return;
         }
-        const live = new Map(claims.map(({ root }, index) => [root, verdicts[index]]));
+        const live = new Map(claims.flatMap(({ root }, index) => {
+            const verdict = verdicts[index];
+            return verdict === undefined ? [] : [[root, verdict]];
+        }));
         for (const commit of commits) {
             commit(live);
         }

package/dist/agent/process.js

@@ -70,7 +70,7 @@ export class ParentCheck {
             try {
                 const [live] = await observeAlive([attachment]);
                 if (this.current === check &&
-                    !live &&
+                    live !== true &&
                     read(file, attachmentSchema)?.generation === attachment.generation) {
                     detached();
                 }
@@ -94,10 +94,11 @@ function replaced(identity) {
 }
 export function ownIdentity(pid = process.pid) {
     // An ordinary main or managed launcher need not be a group leader. Ambiguous live PIDs fail closed.
+    const clock = readProcessClock();
     return {
         pid,
         identity: readChildProcessIdentity(pid) ?? "unverified",
-        clock: readProcessClock(),
+        ...(clock !== undefined ? { clock } : {}),
     };
 }
 export function actor(base, launch) {
@@ -177,11 +178,12 @@ async function spawnActor(base, launch, token, claim) {
         trySignalChildTree(child, "SIGKILL");
     });
     child.unref();
+    const clock = readProcessClock();
     write(path.join(dir, "actor.lock", "process.json"), {
         pid: child.pid,
         identity: readChildProcessIdentity(child.pid) ?? "unverified",
         token,
-        clock: readProcessClock(),
+        ...(clock !== undefined ? { clock } : {}),
     });
 }
 async function awaitStopping(dir, existing, signal) {

package/dist/agent/runner.js

@@ -31,6 +31,10 @@ function completionStatus(stopping, message, error) {
     }
     return "completed";
 }
+/** The recorded failure: the launch error, the model's error, or the missing terminal message. */
+const failureReason = (error, message) => error ??
+    message?.errorMessage ??
+    (message ? undefined : "Native run produced no terminal assistant message.");
 class Actor {
     active;
     stopping = false;
@@ -81,16 +85,17 @@ class Actor {
         const message = terminal?.message;
         const accounting = this.usage(launch, baseline, terminal?.id);
         const output = this.output(message);
+        const failure = failureReason(error, message);
         const result = {
             id: launch.id,
             owner: launch.owner,
             status: completionStatus(this.stopping, message, error),
             endedAt: Date.now(),
-            terminalEntryId: accounting.accountingError === undefined ? terminal?.id : undefined,
+            ...(accounting.accountingError === undefined && terminal?.id !== undefined
+                ? { terminalEntryId: terminal.id }
+                : {}),
             ...accounting,
-            error: error ??
-                message?.errorMessage ??
-                (message ? undefined : "Native run produced no terminal assistant message."),
+            ...(failure !== undefined ? { error: failure } : {}),
         };
         fs.writeFileSync(path.join(runDirectory(this.base, launch.id), "output.txt"), output, {
             mode: 0o600,

package/dist/agent/saved.js

@@ -42,9 +42,19 @@ function ownedRun(context, id) {
         throw new Error("Legacy continuation requires a published native owner.");
     }
     const journal = new NativeJournal(file, "live");
-    if (journal.records[0].value.id !== context.sessionManager.getSessionId()) {
+    const [opening] = journal.records;
+    if (opening === undefined || opening.value.id !== context.sessionManager.getSessionId()) {
         throw new Error("Native owner journal identity changed; nothing was launched.");
     }
+    const owned = findOwnedRun(journal, id);
+    if (owned?.ownerSessionId !== context.sessionManager.getSessionId()) {
+        throw new Error("Missing saved native owner or foreign legacy run; nothing was launched.");
+    }
+    Assert(singleSchema, owned);
+    return owned;
+}
+/** The most recent saved subagent-run record for this run id, if one exists. */
+function findOwnedRun(journal, id) {
     let owned;
     for (const record of journal.records) {
         if (record.value.type === "custom" && record.value.customType === "subagent-run") {
@@ -54,10 +64,6 @@ function ownedRun(context, id) {
             }
         }
     }
-    if (owned?.ownerSessionId !== context.sessionManager.getSessionId()) {
-        throw new Error("Missing saved native owner or foreign legacy run; nothing was launched.");
-    }
-    Assert(singleSchema, owned);
     return owned;
 }
 function requireExited(pid) {
@@ -95,7 +101,11 @@ function nativeSession(contract) {
         throw new Error("Legacy continuation requires the exact saved native journal.");
     }
     const journal = new NativeJournal(contract.sessionFile, "live");
-    const sessionId = journal.body(journal.records[0]).id;
+    const [opening] = journal.records;
+    if (opening === undefined) {
+        throw new Error("Not a readable native Pi session.");
+    }
+    const sessionId = journal.body(opening).id;
     if (typeof sessionId !== "string" ||
         sessionId.length === 0 ||
         (contract.nativeSessionId !== undefined && contract.nativeSessionId !== sessionId)) {

package/dist/agent/view.js

@@ -111,7 +111,10 @@ async function summaries(base, launches, ctx) {
         ...new Map(launches.map((launch) => [launch.rootId, actor(base, launch)])).values(),
     ].filter((identity) => identity !== undefined);
     const verdicts = await observeAlive(identities);
-    const observed = new Map(identities.map((identity, index) => [identity.token, verdicts[index]]));
+    const observed = new Map(identities.flatMap((identity, index) => {
+        const verdict = verdicts[index];
+        return verdict === undefined ? [] : [[identity.token, verdict]];
+    }));
     return launches.map((launch) => {
         const view = metadata(base, launch, readCanonicalResult(base, launch), {
             ctx,
@@ -125,7 +128,7 @@ async function summaries(base, launches, ctx) {
                 ? {
                     status: view.result.status,
                     endedAt: view.result.endedAt,
-                    error: view.result.error?.slice(0, 2048),
+                    ...(view.result.error !== undefined ? { error: view.result.error.slice(0, 2048) } : {}),
                 }
                 : undefined,
         };

package/dist/build-info.js

@@ -1 +1 @@
-export const EXTENSION_BUILD = Object.freeze({"version":"0.52.7","sha256":"ccb2b2cb4725bfa7f5e813494a2007dc60ed9c04e677502d08df23612cc30af8"});
+export const EXTENSION_BUILD = Object.freeze({"version":"0.52.8","sha256":"afeb6d4b7e13dfe796c199d39eddf3836e8f7ee7185f302254f4182659a0388c"});

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

@@ -16,7 +16,7 @@ export function routeFor(req) {
         Object.keys(req.headers).some((key) => key.startsWith("sec-fetch-"))) {
         throw new Fault(403, "browser_forbidden", "Browser access is not supported.");
     }
-    const route = `${req.method ?? ""} ${(req.url ?? "").split("?")[0]}`;
+    const route = `${req.method ?? ""} ${(req.url ?? "").split("?")[0] ?? ""}`;
     if (!isRoute(route)) {
         throw new Fault(404, "not_found", "Unknown method or path.");
     }

package/dist/pi-intercom/bridge.js

@@ -366,17 +366,18 @@ class Bridge {
 }
 function main() {
     const args = process.argv.slice(2);
-    if (args.length === 1 && (args[0] === "-h" || args[0] === "--help")) {
+    const [flag, configPath] = args;
+    if (args.length === 1 && (flag === "-h" || flag === "--help")) {
         process.stdout.write("Usage: node dist/pi-intercom/bridge.js --config PATH\n\nOptional loopback HTTPS/mTLS bridge for the local pi-intercom broker.\nConfig and TLS key must be owned by you with mode 0600.\nWithout stateDir, messaging never starts the broker. Durable presence safely starts\none only when none is running; it never restarts or replaces a live broker.\nSIGHUP reloads the client allowlist (invalid reload denies all clients).\nRestart this helper for TLS/listener rotation. SIGINT/SIGTERM disconnect peers.\n\nExample: node dist/pi-intercom/bridge.js --config ~/.pi/agent/intercom/bridge.json\n");
         return;
     }
-    if (args.length !== 2 || args[0] !== "--config" || args[1] === "") {
+    if (args.length !== 2 || flag !== "--config" || configPath === undefined || configPath === "") {
         process.stderr.write("Usage: node dist/pi-intercom/bridge.js --config PATH (or --help)\n");
         process.exitCode = 2;
         return;
     }
     try {
-        const path = resolve(args[1]);
+        const path = resolve(configPath);
         new Bridge(loadConfig(path), path).start();
     }
     catch (error) {

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

@@ -250,7 +250,7 @@ class IntercomBroker {
         }
         const targets = findSessions(this.sessions, msg.to, currentId);
         const target = targets[0];
-        if (targets.length === 1) {
+        if (targets.length === 1 && target !== undefined) {
             const guardError = guardedDeliveryError(this.sessions, msg.to, message);
             if (guardError !== null) {
                 writeMessage(socket, {

package/dist/pi-intercom/broker/client-last.d.ts

@@ -7,7 +7,7 @@ export declare function lastRequest(msg: UnknownRecord): {
     readonly requestId: string;
     readonly from: SessionInfo;
     readonly query: NativeReadQuery;
-    readonly parked?: SessionInfo;
+    readonly parked?: SessionInfo | undefined;
 };
 export declare class LastMessageRequests {
     /** Negotiated at registration; a failed connection supports neither. */

package/dist/pi-intercom/broker/client.d.ts

@@ -48,14 +48,14 @@ export declare class IntercomClient extends EventEmitter {
     private requireSendSocket;
     send(to: string, options: SendOptions): Promise<SendResult>;
     updatePresence(updates: {
-        readonly applicationGuard?: string | null;
-        readonly name?: string;
-        readonly status?: string;
-        readonly model?: string;
-        readonly pendingAsks?: number;
-        readonly acceptsAsks?: boolean;
-        readonly lastIntercomActivity?: number;
-        readonly subscriptions?: SessionInfo["subscriptions"];
-        readonly topics?: readonly TopicUpdate[];
+        readonly applicationGuard?: string | null | undefined;
+        readonly name?: string | undefined;
+        readonly status?: string | undefined;
+        readonly model?: string | undefined;
+        readonly pendingAsks?: number | undefined;
+        readonly acceptsAsks?: boolean | undefined;
+        readonly lastIntercomActivity?: number | undefined;
+        readonly subscriptions?: SessionInfo["subscriptions"] | undefined;
+        readonly topics?: readonly TopicUpdate[] | undefined;
     }): void;
 }

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

@@ -48,10 +48,16 @@ class MessageBuffer {
         if (size === 0) {
             return Buffer.alloc(0);
         }
+        if (first === undefined) {
+            throw new Error("Frame reader has no buffered chunk for a non-empty read.");
+        }
         const result = first.length >= size ? first.subarray(0, size) : Buffer.allocUnsafe(size);
         let written = 0;
         while (written < size) {
             const chunk = this.chunks[this.index];
+            if (chunk === undefined) {
+                throw new Error("Frame reader ran out of buffered chunks mid-read.");
+            }
             const count = Math.min(chunk.length, size - written);
             if (first.length < size) {
                 chunk.copy(result, written, 0, count);

package/dist/pi-intercom/broker/last.d.ts

@@ -14,7 +14,7 @@ interface LastPeer {
     readonly receiptReads: boolean;
     readonly parked?: {
         readonly closed: boolean;
-    };
+    } | undefined;
 }
 /** Correlation belongs to the broker, not the requester or the recipient's model queue. */
 export declare class LastMessageRelay {
@@ -23,6 +23,10 @@ export declare class LastMessageRelay {
     constructor(sessions: ReadonlyMap<string, LastPeer>);
     /** `reader` names another peer that answers for an exited target from its captured history. */
     request(fromId: string, input: UnknownRecord, reader: (id: string) => LastPeer | undefined): void;
+    /** Register the correlated wait, then forward the read to the reader or the recipient. */
+    private forward;
+    /** A sole matched target may delegate the read to another peer's captured history. */
+    private delegatedReader;
     respond(fromId: string, message: UnknownRecord): void;
     disconnected(sessionId: string): void;
     private finish;

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

@@ -75,21 +75,22 @@ function exitedRefusal(target, query, via) {
     return via === undefined ? "Parked session history is unavailable." : undefined;
 }
 function unavailable(targets, name, pending, { query, via }) {
-    if (targets.length === 0) {
+    const [target] = targets;
+    if (target === undefined) {
         return "Session not found";
     }
     if (targets.length > 1) {
         return `Multiple sessions named "${name}" are connected. Use the session ID instead.`;
     }
-    const exited = exitedRefusal(targets[0], query, via);
+    const exited = exitedRefusal(target, query, via);
     if (exited !== undefined) {
         return exited;
     }
-    if (!targets[0].lastMessages) {
+    if (!target.lastMessages) {
         return "Recipient does not support last-message reads; reload its extension.";
     }
     if ("messageId" in query &&
-        (!targets[0].receiptReads || targets[0].info.rootSessionId !== query.rootSessionId)) {
+        (!target.receiptReads || target.info.rootSessionId !== query.rootSessionId)) {
         return "publication_unavailable";
     }
     if (pending >= 128) {
@@ -135,19 +136,19 @@ export class LastMessageRelay {
     request(fromId, input, reader) {
         const message = lastRequest(input);
         const targets = findSessions(this.sessions, message.to, fromId);
-        const target = targets[0];
         const from = this.sessions.get(fromId);
         if (!from) {
             return;
         }
         const token = randomUUID();
-        const via = targets.length === 1 ? reader(target.info.id) : undefined;
+        const delegated = this.delegatedReader(targets, reader);
+        const via = delegated?.via;
         const frame = {
             type: "last_request",
             requestId: token,
             from: messageSender(from.info),
             query: message.query,
-            parked: via && messageSender(target.info),
+            parked: delegated === undefined ? undefined : messageSender(delegated.target.info),
         };
         const error = unavailable(targets, message.to, this.pending.size, { query: message.query, via }) ??
             validateIntercomMessageSize(frame)?.message;
@@ -159,22 +160,39 @@ export class LastMessageRelay {
             });
             return;
         }
-        const timer = setTimeout(() => this.finish(token, { error: "Last-message read timed out" }), 5000);
+        const [recipient] = targets;
+        if (recipient === undefined) {
+            return;
+        }
+        this.forward({ token, fromId, requestId: message.requestId, query: message.query }, recipient, via, frame);
+    }
+    /** Register the correlated wait, then forward the read to the reader or the recipient. */
+    forward(wait, recipient, via, frame) {
+        const timer = setTimeout(() => this.finish(wait.token, { error: "Last-message read timed out" }), 5000);
         timer.unref();
-        this.pending.set(token, {
-            fromId,
-            toId: target.info.id,
+        this.pending.set(wait.token, {
+            fromId: wait.fromId,
+            toId: recipient.info.id,
             via: via?.info.id,
-            requestId: message.requestId,
-            query: message.query,
+            requestId: wait.requestId,
+            query: wait.query,
             timer,
         });
         try {
-            writeMessage((via ?? target).socket, frame);
+            writeMessage((via ?? recipient).socket, frame);
         }
         catch {
-            this.finish(token, { error: "Recipient disconnected before last-message read" });
+            this.finish(wait.token, { error: "Recipient disconnected before last-message read" });
+        }
+    }
+    /** A sole matched target may delegate the read to another peer's captured history. */
+    delegatedReader(targets, reader) {
+        const [target] = targets;
+        if (targets.length !== 1 || target === undefined) {
+            return undefined;
         }
+        const via = reader(target.info.id);
+        return via === undefined ? undefined : { target, via };
     }
     respond(fromId, message) {
         if (typeof message.requestId !== "string") {

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

@@ -79,7 +79,7 @@ export class ParkedSessions {
             return { id: randomUUID() };
         }
         previous?.socket.destroy();
-        return { id, resumed: previous };
+        return { id, ...(previous !== undefined ? { resumed: previous } : {}) };
     }
     /** Flushes a restored child's input and re-sends wakes owed to a (re)registered main. */
     registered(id, previous) {

package/dist/pi-intercom/outbound.js

@@ -47,6 +47,7 @@ export class IntercomOutbound {
             case "queue":
                 return " (intentionally deferred behind active recipient work)";
             case undefined:
+            default:
                 return " (defaults to steer; wakes idle recipients and steers active recipients after the current tool call)";
         }
     }

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

@@ -44,8 +44,9 @@ function explicitReply(contexts, replyTo, to) {
 }
 function replyBySender(contexts, to) {
     const matches = matchingSender(contexts, to);
-    if (matches.length === 1) {
-        return matches[0];
+    const [onlyMatch] = matches;
+    if (matches.length === 1 && onlyMatch !== undefined) {
+        return onlyMatch;
     }
     if (matches.length > 1) {
         throw new RecoveryError("ambiguous_reply_target", `Multiple pending asks from "${to}" — use one of: ${pendingSenderOptions(matches, contexts)}.`);
@@ -113,7 +114,7 @@ export class ReplyTracker {
         const beforeQueued = this.pendingTurnContexts.length;
         for (let index = this.pendingTurnContexts.length - 1; index >= 0; index -= 1) {
             const context = this.pendingTurnContexts[index];
-            if (expires(context)) {
+            if (context !== undefined && expires(context)) {
                 this.pendingTurnContexts.splice(index, 1);
             }
         }
@@ -145,11 +146,16 @@ export class ReplyTracker {
         if (to !== "") {
             return replyBySender(contexts, 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;
         }
-        if (pending.length === 1) {
-            return pending[0];
+        const [solePending] = pending;
+        if (pending.length === 1 && solePending !== undefined) {
+            return solePending;
         }
         if (pending.length === 0) {
             throw new RecoveryError("no_pending_reply", "No active intercom context to reply to");

package/dist/pi-intercom/session-targets.d.ts

@@ -1,10 +1,10 @@
 export interface TargetIdentity {
     readonly id: string;
-    readonly name?: string;
+    readonly name?: string | undefined;
 }
 export interface ProjectSessionIdentity extends TargetIdentity {
     readonly cwd: string;
-    readonly projectId?: string;
+    readonly projectId?: string | undefined;
 }
 export declare const MIN_SESSION_TARGET_PREFIX_LENGTH = 8;
 export declare function nativePeerId(sessionId: string): string;

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

@@ -97,8 +97,9 @@ export function formatPeerAwarenessHint(sessions, currentSessionId) {
         : undefined;
 }
 function matchedTargets(matches) {
-    if (matches.length === 1) {
-        return { status: "found", target: matches[0], matches };
+    const [target] = matches;
+    if (matches.length === 1 && target !== undefined) {
+        return { status: "found", target, matches };
     }
     return matches.length > 1 ? { status: "ambiguous", matches } : { status: "none", matches: [] };
 }

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

@@ -9,6 +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) {
             const result = throwIfToolError(await tools.execute(params, signal, ctx));
             return { content: [...result.content], details: result.details };