@fitchmultz/pi-subagents 0.52.15 → 0.52.17

package/src/pi-intercom/bridge-presence.ts

@@ -118,7 +118,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 {
         log("presence_retry", { name: identity.name, result: "broker_unavailable" });

package/src/pi-intercom/bridge-protocol.ts

@@ -33,7 +33,6 @@ export function object(value: unknown, fields: readonly string[]): UnknownRecord
 
 export function label(value: unknown, field: string, max = 512): string {
   // 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" ||
@@ -48,7 +47,6 @@ export function label(value: unknown, field: string, max = 512): string {
 
 export function text(value: unknown): string {
   // 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/src/pi-intercom/bridge-session.ts

@@ -359,7 +359,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) {
         if (!signal.aborted) {

package/src/pi-intercom/bridge-wake.ts

@@ -199,7 +199,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 {
         // Expiry starts after response/error/abort, never before COMMIT or connect/TLS.

package/src/pi-intercom/broker/client.ts

@@ -202,12 +202,10 @@ export class IntercomClient extends EventEmitter {
   ): Promise<void> {
     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/src/pi-intercom/broker/spawn.ts

@@ -179,7 +179,6 @@ async function previousBrokerWarning(): Promise<string | undefined> {
   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.";
     }
@@ -289,12 +288,10 @@ async function waitForBroker(timeoutMs = 5000): Promise<void> {
   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/src/pi-intercom/tool-registration.ts

@@ -16,8 +16,7 @@ export function registerIntercomTools(
       "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/src/pi-intercom/validation.ts

@@ -38,10 +38,8 @@ export function asError(error: unknown): 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: unknown): boolean {

package/src/shared/cooperative.ts

@@ -16,7 +16,6 @@ export async function runCooperatively<T>(steps: Generator<void, T>): Promise<T>
     // 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;
     }