pi-sessions 0.14.2 → 0.14.3

package/extensions/pi-sessions.ts

@@ -24,7 +24,8 @@ import { installSubagents } from "./subagents/install.ts";
  * The single advertised entrypoint. It loads settings once, constructs each feature in
  * dependency order, and wires them by constructor parameters.
  * It also owns the single `session_start`/`session_shutdown` subscription so lifecycle order
- * is deterministic: broker registration (messaging) resolves before any other feature hook.
+ * is deterministic: broker registration (messaging) resolves before any other feature hook,
+ * and incoming messages start turns only once every feature hook has resolved.
  */
 export default function piSessions(pi: ExtensionAPI): void {
   const settings = loadSettings();
@@ -141,6 +142,9 @@ export default function piSessions(pi: ExtensionAPI): void {
     for (const lifecycle of lifecycles) {
       await lifecycle.onSessionStart?.(event, ctx);
     }
+    for (const lifecycle of lifecycles) {
+      lifecycle.onSessionReady?.(ctx);
+    }
   });
 
   pi.on("session_shutdown", async (event, ctx) => {

package/extensions/session-messaging/install.ts

@@ -65,11 +65,13 @@ export function installMessaging(
     onIncomingSubagentReport: (handler) => service.onIncomingSubagentReport(handler),
     async onSessionStart(_event, ctx) {
       incomingRuntime.bindContext(ctx);
-      incomingRuntime.replayPending(ctx);
       try {
         await service.start(ctx);
       } catch {}
     },
+    onSessionReady() {
+      incomingRuntime.open();
+    },
     onSessionShutdown() {
       incomingRuntime.clearContext();
       service.stop();

package/extensions/session-messaging/pi/incoming-runtime.ts

@@ -21,6 +21,7 @@ interface IncomingMessageActions {
 
 export class IncomingSessionMessageRuntime {
   private context: ExtensionContext | undefined;
+  private ready = false;
   private readonly actions: IncomingMessageActions;
 
   constructor(pi: ExtensionAPI) {
@@ -32,10 +33,17 @@ export class IncomingSessionMessageRuntime {
 
   bindContext(ctx: ExtensionContext): void {
     this.context = ctx;
+    this.ready = false;
+  }
+
+  open(): void {
+    this.replayPending(this.requireContext());
+    this.ready = true;
   }
 
   clearContext(): void {
     this.context = undefined;
+    this.ready = false;
   }
 
   deliver(received: ReceivedMessageEntry): void {
@@ -47,14 +55,18 @@ export class IncomingSessionMessageRuntime {
       throw new Error(SESSION_STARTING_MESSAGE);
     }
     this.actions.appendEntry(MESSAGE_RECEIVED_CUSTOM_TYPE, received);
-    this.inject(ctx, received);
+    // The broker connects before other features finish session_start, and a turn started now
+    // would run without the tools they activate. The receipt is durable; open() replays it.
+    if (this.ready) {
+      this.inject(ctx, received);
+    }
   }
 
   cancel(): void {
     this.requireContext().abort();
   }
 
-  replayPending(ctx: ExtensionContext): void {
+  private replayPending(ctx: ExtensionContext): void {
     const entries = ctx.sessionManager.getEntries();
     const deliveredMessageIds = new Set<string>();
     const pendingReceipts: ReceivedMessageEntry[] = [];

package/extensions/shared/composition.ts

@@ -11,6 +11,8 @@ import type {
  */
 export interface SessionLifecycle {
   onSessionStart?(event: SessionStartEvent, ctx: ExtensionContext): Promise<void> | void;
+  /** Runs once every feature's onSessionStart has resolved. */
+  onSessionReady?(ctx: ExtensionContext): void;
   onSessionShutdown?(event: SessionShutdownEvent, ctx: ExtensionContext): Promise<void> | void;
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-sessions",
-  "version": "0.14.2",
+  "version": "0.14.3",
   "type": "module",
   "description": "Pi session search, ask, handoff, messaging, auto-titling, and indexing tools",
   "license": "MIT",