@forgeax/engine-intelligence 0.1.4 → 0.1.7

package/src/errors.ts

@@ -1,5 +1,14 @@
 import type { ActivityId } from './types';
 
+export type IntelligenceErrorCode =
+  | 'intelligence-invalid-request'
+  | 'intelligence-session-provider-mismatch'
+  | 'intelligence-capacity-exceeded'
+  | 'intelligence-activity-not-found'
+  | 'intelligence-provider-failed'
+  | 'intelligence-output-overflow'
+  | 'intelligence-closed';
+
 export interface IntelligenceErrorDetailMap {
   readonly 'intelligence-invalid-request': { readonly field: 'input'; readonly reason: string };
   readonly 'intelligence-session-provider-mismatch': {
@@ -17,8 +26,6 @@ export interface IntelligenceErrorDetailMap {
   readonly 'intelligence-closed': Readonly<Record<string, never>>;
 }
 
-export type IntelligenceErrorCode = keyof IntelligenceErrorDetailMap;
-
 export type IntelligenceErrorDetailFor<C extends IntelligenceErrorCode> =
   IntelligenceErrorDetailMap[C];
 

package/src/runtime.ts

@@ -162,7 +162,11 @@ export class IntelligenceRuntime implements IntelligenceService {
 
   private async performClose(): Promise<void> {
     this.closed = true;
-    await this.provider.close();
+    try {
+      await this.provider.close();
+    } catch {
+      // Disposal is terminal; provider failure must not strand the realm transport.
+    }
     this.records.clear();
   }
 
@@ -205,7 +209,7 @@ export class IntelligenceRuntime implements IntelligenceService {
   private createSink(record: ActivityRecord): ActivitySink {
     return {
       text: (text) => {
-        if (record.terminal || text.length === 0) return;
+        if (this.closed || record.terminal || text.length === 0) return;
         if (record.outputChars + text.length > this.limits.maxOutputChars) {
           this.overflow(record, 'output-chars', this.limits.maxOutputChars);
           return;
@@ -218,7 +222,7 @@ export class IntelligenceRuntime implements IntelligenceService {
         this.push(record, { type: 'text-delta', text });
       },
       complete: (output) => {
-        if (record.terminal) return;
+        if (this.closed || record.terminal) return;
         if (output.length > this.limits.maxOutputChars) {
           this.overflow(record, 'output-chars', this.limits.maxOutputChars);
           return;
@@ -227,13 +231,13 @@ export class IntelligenceRuntime implements IntelligenceService {
         this.push(record, { type: 'completed', session: record.ref.session, output });
       },
       fail: (cause) => {
-        if (record.terminal) return;
+        if (this.closed || record.terminal) return;
         record.terminal = true;
         const error = providerError(this.providerId, cause);
         this.push(record, { type: 'failed', error: intelligenceFailure(error) });
       },
       cancelled: () => {
-        if (record.terminal) return;
+        if (this.closed || record.terminal) return;
         record.terminal = true;
         this.push(record, { type: 'cancelled' });
       },

package/src/transport.ts

@@ -52,9 +52,11 @@ export function bindIntelligencePort(
   runtime: IntelligenceRuntime,
 ): IntelligencePortBinding {
   let closed = false;
+  let closeTask: Promise<void> | undefined;
   const listener = (
     event: MessageEvent<IntelligenceHostCommand | IntelligenceRealmMessage>,
   ): void => {
+    if (closed) return;
     const message = event.data;
     if (message.kind === 'intelligence-submit') {
       const accepted = runtime.accept(message.submission);
@@ -76,19 +78,18 @@ export function bindIntelligencePort(
       runtime.cancel(message.activityId);
       return;
     }
-    if (message.kind === 'intelligence-close') void close(true);
+    if (message.kind === 'intelligence-close') void close();
   };
-  const close = async (notifyRealm = false): Promise<void> => {
-    if (closed) return;
+  const close = (): Promise<void> => {
+    if (closeTask !== undefined) return closeTask;
     closed = true;
     port.removeEventListener('message', listener);
-    await runtime.close();
-    if (notifyRealm) {
+    closeTask = (async () => {
+      await runtime.close();
       port.postMessage({ kind: 'intelligence-closed' });
       setTimeout(() => port.close(), 0);
-    } else {
-      port.close();
-    }
+    })();
+    return closeTask;
   };
   port.addEventListener('message', listener);
   port.start?.();
@@ -157,6 +158,7 @@ export class IntelligencePortClient implements IntelligenceService {
     this.createSessionId = options.createSessionId ?? (() => nextPortIdentity('session'));
     this.listener = (event) => {
       const message = event.data;
+      if (this.closed && message.kind !== 'intelligence-closed') return;
       if (message.kind === 'intelligence-events') {
         this.pollPending = false;
         for (const item of message.events) {
@@ -167,19 +169,26 @@ export class IntelligencePortClient implements IntelligenceService {
         this.active.delete(message.activityId);
         this.rejected.set(message.activityId, message);
       } else if (message.kind === 'intelligence-closed') {
+        this.markClosed();
         this.port.removeEventListener('message', this.listener);
-        this.received.length = 0;
-        this.active.clear();
-        this.rejected.clear();
         this.port.close();
         this.closeResolve?.();
         this.closeResolve = undefined;
+        this.closeTask ??= Promise.resolve();
       }
     };
     port.addEventListener('message', this.listener);
     port.start?.();
   }
 
+  private markClosed(): void {
+    this.closed = true;
+    this.received.length = 0;
+    this.active.clear();
+    this.rejected.clear();
+    this.pollPending = false;
+  }
+
   submit(request: ActivityRequest): Result<ActivityRef, IntelligenceError> {
     if (this.closed) return err(new IntelligenceError({ code: 'intelligence-closed', detail: {} }));
     if (request.input.length === 0 || request.input.length > this.limits.maxInputChars) {
@@ -264,7 +273,11 @@ export class IntelligencePortClient implements IntelligenceService {
 
   close(): Promise<void> {
     if (this.closeTask !== undefined) return this.closeTask;
-    this.closed = true;
+    if (this.closed) {
+      this.closeTask = Promise.resolve();
+      return this.closeTask;
+    }
+    this.markClosed();
     this.closeTask = new Promise((resolve) => {
       this.closeResolve = resolve;
       this.port.postMessage({ kind: 'intelligence-close' });

package/dist/__tests__/intelligence-error-code-owner.test-d.d.ts

@@ -1,33 +0,0 @@
-import type { IntelligenceErrorCode, IntelligenceErrorDetailMap } from '../errors';
-import type { INTELLIGENCE_ERROR_HINTS, INTELLIGENCE_EXPECTED, IntelligenceErrorDetailFor, IntelligenceFailure, IntelligenceError as PublicIntelligenceError, IntelligenceErrorCode as PublicIntelligenceErrorCode } from '../index';
-type Equal<Left, Right> = (<Type>() => Type extends Left ? 1 : 2) extends <Type>() => Type extends Right ? 1 : 2 ? true : false;
-type Assert<Value extends true> = Value;
-type ExpectedCodes = 'intelligence-invalid-request' | 'intelligence-session-provider-mismatch' | 'intelligence-capacity-exceeded' | 'intelligence-activity-not-found' | 'intelligence-provider-failed' | 'intelligence-output-overflow' | 'intelligence-closed';
-type _CodeIsDetailMapKeys = Assert<Equal<IntelligenceErrorCode, keyof IntelligenceErrorDetailMap>>;
-type _CodeHasExactMembership = Assert<Equal<IntelligenceErrorCode, ExpectedCodes>>;
-type _PublicCodeIsOwnerCode = Assert<Equal<PublicIntelligenceErrorCode, IntelligenceErrorCode>>;
-type _ExpectedKeysAreCodes = Assert<Equal<keyof typeof INTELLIGENCE_EXPECTED, IntelligenceErrorCode>>;
-type _HintKeysAreCodes = Assert<Equal<keyof typeof INTELLIGENCE_ERROR_HINTS, IntelligenceErrorCode>>;
-type _DetailProjectionUsesTheMap = Assert<Equal<IntelligenceErrorDetailFor<'intelligence-output-overflow'>, IntelligenceErrorDetailMap['intelligence-output-overflow']>>;
-declare function readErrorDetail(error: PublicIntelligenceError): string;
-declare function readFailure(error: IntelligenceFailure): string;
-export type _IntelligenceErrorCodeOwnerChecks = {
-    /** @internal */
-    _mapKeys: _CodeIsDetailMapKeys;
-    /** @internal */
-    _membership: _CodeHasExactMembership;
-    /** @internal */
-    _publicCode: _PublicCodeIsOwnerCode;
-    /** @internal */
-    _expectedKeys: _ExpectedKeysAreCodes;
-    /** @internal */
-    _hintKeys: _HintKeysAreCodes;
-    /** @internal */
-    _detailProjection: _DetailProjectionUsesTheMap;
-    /** @internal */
-    _readErrorDetail: typeof readErrorDetail;
-    /** @internal */
-    _readFailure: typeof readFailure;
-};
-export {};
-//# sourceMappingURL=intelligence-error-code-owner.test-d.d.ts.map

package/dist/__tests__/intelligence-error-code-owner.test-d.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"intelligence-error-code-owner.test-d.d.ts","sourceRoot":"","sources":["../../src/__tests__/intelligence-error-code-owner.test-d.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,qBAAqB,EAAE,0BAA0B,EAAE,MAAM,WAAW,CAAC;AACnF,OAAO,KAAK,EACV,wBAAwB,EACxB,qBAAqB,EACrB,0BAA0B,EAC1B,mBAAmB,EACnB,iBAAiB,IAAI,uBAAuB,EAC5C,qBAAqB,IAAI,2BAA2B,EACrD,MAAM,UAAU,CAAC;AAGlB,KAAK,KAAK,CAAC,IAAI,EAAE,KAAK,IACpB,CAAC,CAAC,IAAI,OAAO,IAAI,SAAS,IAAI,GAAG,CAAC,GAAG,CAAC,CAAC,SAAS,CAAC,IAAI,OAAO,IAAI,SAAS,KAAK,GAAG,CAAC,GAAG,CAAC,GAClF,IAAI,GACJ,KAAK,CAAC;AAEZ,KAAK,MAAM,CAAC,KAAK,SAAS,IAAI,IAAI,KAAK,CAAC;AAExC,KAAK,aAAa,GACd,8BAA8B,GAC9B,wCAAwC,GACxC,gCAAgC,GAChC,iCAAiC,GACjC,8BAA8B,GAC9B,8BAA8B,GAC9B,qBAAqB,CAAC;AAE1B,KAAK,oBAAoB,GAAG,MAAM,CAAC,KAAK,CAAC,qBAAqB,EAAE,MAAM,0BAA0B,CAAC,CAAC,CAAC;AACnG,KAAK,uBAAuB,GAAG,MAAM,CAAC,KAAK,CAAC,qBAAqB,EAAE,aAAa,CAAC,CAAC,CAAC;AACnF,KAAK,sBAAsB,GAAG,MAAM,CAAC,KAAK,CAAC,2BAA2B,EAAE,qBAAqB,CAAC,CAAC,CAAC;AAChG,KAAK,qBAAqB,GAAG,MAAM,CACjC,KAAK,CAAC,MAAM,OAAO,qBAAqB,EAAE,qBAAqB,CAAC,CACjE,CAAC;AACF,KAAK,iBAAiB,GAAG,MAAM,CAC7B,KAAK,CAAC,MAAM,OAAO,wBAAwB,EAAE,qBAAqB,CAAC,CACpE,CAAC;AACF,KAAK,2BAA2B,GAAG,MAAM,CACvC,KAAK,CACH,0BAA0B,CAAC,8BAA8B,CAAC,EAC1D,0BAA0B,CAAC,8BAA8B,CAAC,CAC3D,CACF,CAAC;AAgCF,iBAAS,eAAe,CAAC,KAAK,EAAE,uBAAuB,GAAG,MAAM,CAqB/D;AAED,iBAAS,WAAW,CAAC,KAAK,EAAE,mBAAmB,GAAG,MAAM,CAmBvD;AAUD,MAAM,MAAM,iCAAiC,GAAG;IAC9C,gBAAgB;IAChB,QAAQ,EAAE,oBAAoB,CAAC;IAC/B,gBAAgB;IAChB,WAAW,EAAE,uBAAuB,CAAC;IACrC,gBAAgB;IAChB,WAAW,EAAE,sBAAsB,CAAC;IACpC,gBAAgB;IAChB,aAAa,EAAE,qBAAqB,CAAC;IACrC,gBAAgB;IAChB,SAAS,EAAE,iBAAiB,CAAC;IAC7B,gBAAgB;IAChB,iBAAiB,EAAE,2BAA2B,CAAC;IAC/C,gBAAgB;IAChB,gBAAgB,EAAE,OAAO,eAAe,CAAC;IACzC,gBAAgB;IAChB,YAAY,EAAE,OAAO,WAAW,CAAC;CAClC,CAAC"}

package/src/__tests__/intelligence-error-code-owner.test-d.ts

@@ -1,143 +0,0 @@
-import type { IntelligenceErrorCode, IntelligenceErrorDetailMap } from '../errors';
-import type {
-  INTELLIGENCE_ERROR_HINTS,
-  INTELLIGENCE_EXPECTED,
-  IntelligenceErrorDetailFor,
-  IntelligenceFailure,
-  IntelligenceError as PublicIntelligenceError,
-  IntelligenceErrorCode as PublicIntelligenceErrorCode,
-} from '../index';
-import { activityId, IntelligenceError, intelligenceFailure, providerError } from '../index';
-
-type Equal<Left, Right> =
-  (<Type>() => Type extends Left ? 1 : 2) extends <Type>() => Type extends Right ? 1 : 2
-    ? true
-    : false;
-
-type Assert<Value extends true> = Value;
-
-type ExpectedCodes =
-  | 'intelligence-invalid-request'
-  | 'intelligence-session-provider-mismatch'
-  | 'intelligence-capacity-exceeded'
-  | 'intelligence-activity-not-found'
-  | 'intelligence-provider-failed'
-  | 'intelligence-output-overflow'
-  | 'intelligence-closed';
-
-type _CodeIsDetailMapKeys = Assert<Equal<IntelligenceErrorCode, keyof IntelligenceErrorDetailMap>>;
-type _CodeHasExactMembership = Assert<Equal<IntelligenceErrorCode, ExpectedCodes>>;
-type _PublicCodeIsOwnerCode = Assert<Equal<PublicIntelligenceErrorCode, IntelligenceErrorCode>>;
-type _ExpectedKeysAreCodes = Assert<
-  Equal<keyof typeof INTELLIGENCE_EXPECTED, IntelligenceErrorCode>
->;
-type _HintKeysAreCodes = Assert<
-  Equal<keyof typeof INTELLIGENCE_ERROR_HINTS, IntelligenceErrorCode>
->;
-type _DetailProjectionUsesTheMap = Assert<
-  Equal<
-    IntelligenceErrorDetailFor<'intelligence-output-overflow'>,
-    IntelligenceErrorDetailMap['intelligence-output-overflow']
-  >
->;
-
-const acceptsCode = (code: PublicIntelligenceErrorCode): PublicIntelligenceErrorCode => code;
-acceptsCode('intelligence-closed');
-// @ts-expect-error unknown literals remain outside the closed public code union.
-acceptsCode('intelligence-not-real');
-
-const overflow = new IntelligenceError({
-  code: 'intelligence-output-overflow',
-  detail: {
-    activityId: activityId('overflow'),
-    bound: 'output-chars',
-    limit: 256,
-  },
-});
-type OverflowError = Extract<
-  PublicIntelligenceError,
-  { readonly code: 'intelligence-output-overflow' }
->;
-const typedOverflow: OverflowError = overflow;
-const expected: string = overflow.expected;
-const hint: string = overflow.hint;
-void typedOverflow;
-void expected;
-void hint;
-
-new IntelligenceError({
-  code: 'intelligence-capacity-exceeded',
-  // @ts-expect-error the detail must match the selected code.
-  detail: { activityId: activityId('wrong-detail') },
-});
-
-function readErrorDetail(error: PublicIntelligenceError): string {
-  switch (error.code) {
-    case 'intelligence-invalid-request':
-      // @ts-expect-error code-driven narrowing excludes unrelated detail fields.
-      void error.detail.limit;
-      return `${error.detail.field}:${error.detail.reason}`;
-    case 'intelligence-session-provider-mismatch':
-      return `${error.detail.expectedProviderId}:${error.detail.receivedProviderId}`;
-    case 'intelligence-capacity-exceeded':
-      return String(error.detail.limit);
-    case 'intelligence-activity-not-found':
-      return error.detail.activityId;
-    case 'intelligence-provider-failed':
-      return `${error.detail.providerId}:${String(error.detail.cause)}`;
-    case 'intelligence-output-overflow':
-      return `${error.detail.activityId}:${error.detail.bound}:${error.detail.limit}`;
-    case 'intelligence-closed':
-      return Object.keys(error.detail).join(',');
-  }
-  const exhaustive: never = error;
-  return exhaustive;
-}
-
-function readFailure(error: IntelligenceFailure): string {
-  switch (error.code) {
-    case 'intelligence-invalid-request':
-      return `${error.detail.field}:${error.detail.reason}`;
-    case 'intelligence-session-provider-mismatch':
-      return `${error.detail.expectedProviderId}:${error.detail.receivedProviderId}`;
-    case 'intelligence-capacity-exceeded':
-      return String(error.detail.limit);
-    case 'intelligence-activity-not-found':
-      return error.detail.activityId;
-    case 'intelligence-provider-failed':
-      return `${error.detail.providerId}:${error.detail.cause}`;
-    case 'intelligence-output-overflow':
-      return `${error.detail.activityId}:${error.detail.bound}:${error.detail.limit}`;
-    case 'intelligence-closed':
-      return Object.keys(error.detail).join(',');
-  }
-  const exhaustive: never = error;
-  return exhaustive;
-}
-
-const projectedFailure: IntelligenceFailure = intelligenceFailure(
-  providerError('test.provider', new Error('failed')),
-);
-void acceptsCode;
-void readErrorDetail;
-void readFailure;
-void projectedFailure;
-
-export type _IntelligenceErrorCodeOwnerChecks = {
-  /** @internal */
-  _mapKeys: _CodeIsDetailMapKeys;
-  /** @internal */
-  _membership: _CodeHasExactMembership;
-  /** @internal */
-  _publicCode: _PublicCodeIsOwnerCode;
-  /** @internal */
-  _expectedKeys: _ExpectedKeysAreCodes;
-  /** @internal */
-  _hintKeys: _HintKeysAreCodes;
-  /** @internal */
-  _detailProjection: _DetailProjectionUsesTheMap;
-  /** @internal */
-  _readErrorDetail: typeof readErrorDetail;
-  /** @internal */
-  _readFailure: typeof readFailure;
-};