@matatbread/matbot-plugin-api 0.4.16 → 0.4.17

package/package.json

@@ -4,7 +4,7 @@
     "node",
     "browser"
   ],
-  "version": "0.4.16",
+  "version": "0.4.17",
   "engines": {
     "node": ">=24.0.0"
   },

package/src/hooks.ts

@@ -20,11 +20,12 @@ export class HookRegistry implements HookRegistrar {
   private readonly markedFailures = new Set<Hook>();
   private pendingFailureMarkers: { channel: HookPoint; pluginName?: string; message: string }[] = [];
 
+  // A fresh array, never an in-place push + sort, for the reason given at `removeOne`: a run loop
+  // iterating the old list across its awaits would otherwise visit a hook registered mid-run, or visit
+  // one twice after the sort moved it behind the cursor.
   register(hook: Hook): void {
-    const list = this.hooks.get(hook.on) ?? [];
-    list.push(hook);
-    list.sort((a, b) => (a.priority ?? 50) - (b.priority ?? 50));
-    this.hooks.set(hook.on, list);
+    const list = [...(this.hooks.get(hook.on) ?? []), hook];
+    this.hooks.set(hook.on, list.sort((a, b) => (a.priority ?? 50) - (b.priority ?? 50)));
   }
 
   // The hooks registered on one channel, narrowed to that channel's arm. The map is keyed BY `on`, so

package/src/host-machine.ts

@@ -2,6 +2,7 @@ import type {
   CompletionRequest, MatbotMachine, MatbotServices, Mounted, MountConsumeOptions, SingleTurnRequest,
 } from './plugin.js';
 import { RegistryChangeKind } from './notify.js';
+import { createMessage } from './session.js';
 
 /*
  * Host-side machine assembly — the wiring an *embedder* does once at boot, not API a plugin author
@@ -205,16 +206,17 @@ export function createMountTable(getMachine: () => MatbotMachine): MountTable {
 
 /**
  * Build the one-message CompletionRequest for a {@link MatbotRuntime.singleTurn} call, hiding the
- * otherwise-mandatory and meaningless Message fields (id/traceId/createdAt) an out-of-band one-shot
- * has no use for. Pure; the host invokes its own complete() with the result.
+ * otherwise-mandatory Message fields an out-of-band one-shot has no use for. The host invokes its own
+ * complete() with the result.
  */
 export function singleTurnRequest(req: SingleTurnRequest): CompletionRequest {
   return {
     provider: req.provider,
-    messages: [{
-      id: '', traceId: '', createdAt: new Date().toISOString(), role: 'user',
+    messages: [createMessage({
+      role:    'user',
+      traceId: '',
       content: typeof req.prompt === 'string' ? [{ type: 'text', text: req.prompt }] : req.prompt,
-    }],
+    })],
     ...(req.system     !== undefined ? { system: req.system } : {}),
     ...(req.parameters !== undefined ? { parameters: req.parameters } : {}),
     ...(req.signal     !== undefined ? { signal: req.signal } : {}),

package/src/notify.ts

@@ -193,6 +193,7 @@ export interface Notifier {
  */
 export function createNotifier(defaultPlugin: string): Notifier {
   const events = createBroadcaster<Notification>();
+  const warned = new Set<string>();
   return {
     notify(notification) {
       const plugin = notification.plugin ?? defaultPlugin;
@@ -200,9 +201,12 @@ export function createNotifier(defaultPlugin: string): Notifier {
       // matter most here: a plain-JS plugin, or a bridge injecting another instance's traffic. Warn, don't
       // throw — `notify` is contractually non-throwing, and a badly-named kind is a hygiene fault, not a
       // delivery failure. Only the shape is checkable; that the prefix names the *definer* is not, since
-      // the emitter legitimately differs from it.
-      if (!notification.kind.includes('#')) {
-        console.warn(`[matbot] ${plugin} published unqualified notification kind "${notification.kind}" — expected <package-name>#<InterfaceName>`);
+      // the emitter legitimately differs from it. Once per emitter and kind: the fault is in the source,
+      // so repeating it on every publish only buries the log.
+      const warning = `${plugin} published unqualified notification kind "${notification.kind}"`;
+      if (!notification.kind.includes('#') && !warned.has(warning)) {
+        warned.add(warning);
+        console.warn(`[matbot] ${warning} — expected <package-name>#<InterfaceName>`);
       }
       events.emit({ ...notification, plugin } as Notification);
     },

package/src/principal-context.ts

@@ -116,8 +116,7 @@ function isAsyncIterator(value: unknown): value is AsyncIterator<unknown> & Asyn
     && typeof (value as AsyncIterator<unknown>).next === 'function';
 }
 
-// The per-pull re-entry itself is `scopeIterator` (./scoped-iterator.ts) — the usage carrier needs the
-// identical wrap for its call site, and had its own, less faithful copy.
+// The per-pull re-entry itself is `scopeIterator` (./scoped-iterator.ts), shared with the usage carrier.
 function scopedIterator<T>(
   carrier:   PrincipalCarrier,
   principal: Principal,