@outputai/llm 0.10.1-dev.b7b2fbe.0 → 0.10.1-next.1070949.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@outputai/llm",
-  "version": "0.10.1-dev.b7b2fbe.0",
+  "version": "0.10.1-next.1070949.0",
   "description": "Framework abstraction to interact with LLM models",
   "type": "module",
   "main": "src/index.js",
@@ -14,12 +14,12 @@
   "dependencies": {
     "entities": "8.0.0",
     "gray-matter": "4.0.3",
-    "liquidjs": "10.25.7",
-    "undici": "8.5.0",
-    "@outputai/core": "0.10.1-dev.b7b2fbe.0"
+    "liquidjs": "10.27.2",
+    "undici": "8.9.0",
+    "@outputai/core": "0.10.1-next.1070949.0"
   },
   "devDependencies": {
-    "@ai-sdk/amazon-bedrock": "4.0.111",
+    "@ai-sdk/amazon-bedrock": "4.0.138",
     "@ai-sdk/anthropic": "3.0.81",
     "@ai-sdk/azure": "3.0.68",
     "@ai-sdk/google-vertex": "4.0.140",

package/src/prompt/loader.js

@@ -5,7 +5,11 @@ import { validatePrompt } from './validations.js';
 import { FatalError } from '@outputai/core';
 import { escape, decode, setupLiquidEncodeFilter } from './escape.js';
 
-const liquid = new Liquid();
+const liquid = new Liquid( {
+  strictFilters: true,
+  strictVariables: true,
+  lenientIf: true
+} );
 setupLiquidEncodeFilter( liquid );
 
 /** Uses LiquidJS to interpolate variables in the prompt file content. */

package/src/utils/error_handler.js

@@ -1,56 +1,26 @@
-import {
-  APICallError,
-  InvalidArgumentError,
-  InvalidDataContentError,
-  InvalidPromptError,
-  LoadAPIKeyError,
-  LoadSettingError,
-  NoImageGeneratedError,
-  NoObjectGeneratedError,
-  NoSuchModelError,
-  NoSuchProviderError,
-  UnsupportedFunctionalityError
-} from 'ai';
-import { FatalError } from '@outputai/core';
-
-// AI SDK does not expose a dedicated schema-mismatch discriminator for NoObjectGeneratedError.
-const NO_OBJECT_SCHEMA_MISMATCH_MESSAGE = 'No object generated: response did not match schema.';
-
-/**
- * Recursively search an error cause chain until finds an error which is instance of given prototype.
- *
- * @param {object} error - Error instance.
- * @param {Function|string} _class - Target constructor or constructor name.
- * @param {number} depth - Current depth, search up to 10 causes deep.
- * @returns {object|null} - Error or null if not found.
- */
-export const findInstanceInCauseChain = ( error, _class, depth = 0 ) => {
-  if ( !error || typeof error !== 'object' ) {
-    return null;
-  }
-  if ( typeof _class === 'string' && error.constructor?.name === _class ) {
-    return error;
-  }
-  if ( typeof _class === 'function' && error instanceof _class ) {
-    return error;
-  }
-  if ( depth >= 10 ) {
-    return null;
-  }
-  return error.cause ? findInstanceInCauseChain( error.cause, _class, depth + 1 ) : null;
-};
-
-const toFatalError = ( error, extraMessage = '' ) => new FatalError(
-  `AI-SDK fatal error${extraMessage ? ` (${extraMessage})` : ''}: ${error.message}`,
-  { cause: error }
-);
+import * as ai from 'ai';
+import { FatalError, TransparentFatalError } from '@outputai/core';
+
+const nonRetryableAiSdkErrorTypes = [
+  ai.InvalidArgumentError, // Invalid call settings are deterministic caller bugs, so retrying the same activity cannot fix them.
+  ai.InvalidDataContentError, // Invalid media content has the wrong local shape/encoding and will fail again with the same input.
+  ai.InvalidPromptError, // Invalid prompt structure is a deterministic request-construction error.
+  ai.LoadAPIKeyError, // Missing or invalid API key configuration will not change during an activity retry.
+  ai.LoadSettingError, // Missing or invalid provider settings are deployment/configuration problems.
+  ai.NoImageGeneratedError, // Image generation completed provider calls but collected zero images; repeating identical input is not useful.
+  ai.NoSuchProviderError, // A missing provider id is a deterministic provider registry/configuration error.
+  ai.NoSuchModelError, // A missing model id is a deterministic provider/model configuration error.
+  ai.UnsupportedFunctionalityError // The selected model/output mode does not support the requested feature.
+];
 
 /**
- * Map an AI SDK error to a framework specific error:
+ * Maps an AI SDK error to a framework error:
+ * - AI SDK Unrecoverable error types become TransparentFatalErrors;
+ * - AI SDK API error with isRetryable=false become TransparentFatalErrors;
+ * Some errors are not mapped:
+ * - Grammar which technically are isRetryable=false, will be rethrown, because they are actually transient;
+ * - Other errors are rethrown as well;
  *
- * - AI SDK Unrecoverable errors become FatalErrors, check code to see options.
- * - NoObjectGeneratedError from invalid schema are reinitialized with a better message.
- * - Other errors are preserved.
  * @param {object} error - Original Error
  * @returns {object} A new Error
  */
@@ -59,25 +29,7 @@ export const mapAiError = error => {
     return error;
   }
 
-  // NoObjectGeneratedError can be thrown when the response doesn't match the schema.
-  // This re-creates the error with a better message, making it easier to debug.
-  if ( NoObjectGeneratedError.isInstance( error ) && error.message.includes( NO_OBJECT_SCHEMA_MISMATCH_MESSAGE ) ) {
-    const zodError = findInstanceInCauseChain( error, 'ZodError' );
-    if ( zodError && zodError.issues?.length > 0 ) {
-      const [ { path, message } ] = zodError.issues;
-      return new NoObjectGeneratedError( {
-        message: `${error.message} First issue is "${message}" at path [${path.join( ', ' )}].`,
-        cause: error.cause,
-        text: error.text,
-        response: error.response,
-        usage: error.usage,
-        finishReason: error.finishReason
-      } );
-    }
-    return error;
-  }
-
-  const isApiError = APICallError.isInstance( error );
+  const isApiError = ai.APICallError.isInstance( error );
   const isGrammarCompilationError = error.message === 'Grammar compilation timed out.';
 
   // This error is actually transient, so instead of FatalError, return it
@@ -85,45 +37,13 @@ export const mapAiError = error => {
     return error;
   }
 
+  // Non-retryable API failures are already classified by AI SDK as permanent provider failures.
   if ( isApiError && !error.isRetryable ) {
-    // Non-retryable API failures are already classified by AI SDK as permanent provider failures.
-    return toFatalError( error, error.statusCode ? `HTTP ${error.statusCode}` : '' );
-  }
-  if ( InvalidArgumentError.isInstance( error ) ) {
-    // Invalid call settings are deterministic caller bugs, so retrying the same activity cannot fix them.
-    return toFatalError( error );
-  }
-  if ( InvalidDataContentError.isInstance( error ) ) {
-    // Invalid media content has the wrong local shape/encoding and will fail again with the same input.
-    return toFatalError( error );
-  }
-  if ( InvalidPromptError.isInstance( error ) ) {
-    // Invalid prompt structure is a deterministic request-construction error.
-    return toFatalError( error );
+    return new TransparentFatalError( error );
   }
-  if ( LoadAPIKeyError.isInstance( error ) ) {
-    // Missing or invalid API key configuration will not change during an activity retry.
-    return toFatalError( error );
-  }
-  if ( LoadSettingError.isInstance( error ) ) {
-    // Missing or invalid provider settings are deployment/configuration problems.
-    return toFatalError( error );
-  }
-  if ( NoImageGeneratedError.isInstance( error ) ) {
-    // Image generation completed provider calls but collected zero images; repeating identical input is not useful.
-    return toFatalError( error );
-  }
-  if ( NoSuchProviderError.isInstance( error ) ) {
-    // A missing provider id is a deterministic provider registry/configuration error.
-    return toFatalError( error );
-  }
-  if ( NoSuchModelError.isInstance( error ) ) {
-    // A missing model id is a deterministic provider/model configuration error.
-    return toFatalError( error );
-  }
-  if ( UnsupportedFunctionalityError.isInstance( error ) ) {
-    // The selected model/output mode does not support the requested feature.
-    return toFatalError( error );
+
+  if ( nonRetryableAiSdkErrorTypes.some( E => E.isInstance( error ) ) ) {
+    return new TransparentFatalError( error );
   }
   return error;
 };

package/src/utils/error_handler.spec.js

@@ -19,8 +19,8 @@ import {
   ToolCallRepairError,
   UnsupportedFunctionalityError
 } from 'ai';
-import { FatalError } from '@outputai/core';
-import { findInstanceInCauseChain, mapAiError } from './error_handler.js';
+import { FatalError, TransparentFatalError } from '@outputai/core';
+import { mapAiError } from './error_handler.js';
 
 const makeApiCallError = ( input = {} ) => new APICallError( {
   message: 'Provider rejected the request',
@@ -130,63 +130,6 @@ const preservedAiSdkErrors = [
   ]
 ];
 
-describe( 'findInstanceInCauseChain', () => {
-  class FirstCustomError extends Error {}
-  class SecondCustomError extends Error {}
-
-  it( 'returns the input error when it matches the target constructor', () => {
-    const error = new FirstCustomError( 'first' );
-
-    expect( findInstanceInCauseChain( error, FirstCustomError ) ).toBe( error );
-  } );
-
-  it( 'returns the input error when it matches the target constructor name', () => {
-    const error = new FirstCustomError( 'first' );
-
-    expect( findInstanceInCauseChain( error, 'FirstCustomError' ) ).toBe( error );
-  } );
-
-  it( 'walks the cause chain to find an error by constructor', () => {
-    const target = new SecondCustomError( 'second' );
-    const wrapper = new FirstCustomError( 'first', { cause: target } );
-
-    expect( findInstanceInCauseChain( wrapper, SecondCustomError ) ).toBe( target );
-  } );
-
-  it( 'walks the cause chain to find an error by constructor name', () => {
-    const target = new SecondCustomError( 'second' );
-    const wrapper = new FirstCustomError( 'first', { cause: target } );
-
-    expect( findInstanceInCauseChain( wrapper, 'SecondCustomError' ) ).toBe( target );
-  } );
-
-  it( 'returns null when the target is not found', () => {
-    const error = new FirstCustomError( 'first', { cause: new Error( 'root' ) } );
-
-    expect( findInstanceInCauseChain( error, SecondCustomError ) ).toBeNull();
-  } );
-
-  it( 'returns null for empty or non-object inputs', () => {
-    expect( findInstanceInCauseChain( null, Error ) ).toBeNull();
-    expect( findInstanceInCauseChain( 'not an error', Error ) ).toBeNull();
-  } );
-
-  it( 'returns null for object causes without constructors', () => {
-    const cause = Object.create( null );
-    const error = new FirstCustomError( 'first', { cause } );
-
-    expect( findInstanceInCauseChain( error, 'SecondCustomError' ) ).toBeNull();
-  } );
-
-  it( 'stops searching after the depth limit', () => {
-    const makeErrorChain = depth => depth === 0 ?
-      new SecondCustomError( 'target' ) :
-      new FirstCustomError( `level ${depth}`, { cause: makeErrorChain( depth - 1 ) } );
-
-    expect( findInstanceInCauseChain( makeErrorChain( 11 ), SecondCustomError ) ).toBeNull();
-  } );
-} );
-
 describe( 'mapAiError', () => {
   it( 'preserves existing FatalError instances', () => {
     const error = new FatalError( 'Already fatal' );
@@ -194,7 +137,7 @@ describe( 'mapAiError', () => {
     expect( mapAiError( error ) ).toBe( error );
   } );
 
-  it( 'adds first schema issue details to NoObjectGeneratedError schema mismatches', () => {
+  it( 'preserves NoObjectGeneratedError with its complete schema issue details', () => {
     class ZodError extends Error {
       constructor( issues ) {
         super( 'schema failed' );
@@ -217,32 +160,12 @@ describe( 'mapAiError', () => {
       cause: validationError
     } );
 
-    const result = mapAiError( error );
-
-    expect( result ).not.toBe( error );
-    expect( NoObjectGeneratedError.isInstance( result ) ).toBe( true );
-    expect( result.name ).toBe( 'AI_NoObjectGeneratedError' );
-    expect( result.message ).toBe(
-      'No object generated: response did not match schema. First issue is "Expected string" at path [items, 0, title].'
-    );
-    expect( result.cause ).toBe( validationError );
-    expect( result.text ).toBe( error.text );
-    expect( result.response ).toBe( error.response );
-    expect( result.usage ).toBe( error.usage );
-    expect( result.finishReason ).toBe( error.finishReason );
-  } );
-
-  it( 'preserves NoObjectGeneratedError schema mismatches when no schema issue is available', () => {
-    const error = new NoObjectGeneratedError( {
-      message: 'No object generated: response did not match schema.',
-      text: '{"items":[{}]}',
-      cause: new Error( 'validation failed' )
-    } );
-
     expect( mapAiError( error ) ).toBe( error );
+    expect( error.cause.cause ).toBe( zodError );
+    expect( error.cause.cause.issues ).toEqual( zodError.issues );
   } );
 
-  it( 'maps non-retryable APICallError instances to FatalError', () => {
+  it( 'maps non-retryable APICallError instances to TransparentFatalError', () => {
     const error = makeApiCallError( {
       statusCode: 400,
       isRetryable: false
@@ -250,20 +173,18 @@ describe( 'mapAiError', () => {
 
     const result = mapAiError( error );
 
-    expect( result ).toBeInstanceOf( FatalError );
-    expect( result.message ).toBe( 'AI-SDK fatal error (HTTP 400): Provider rejected the request' );
+    expect( result ).toBeInstanceOf( TransparentFatalError );
     expect( result.cause ).toBe( error );
   } );
 
-  it( 'maps non-retryable APICallError instances without status codes to FatalError', () => {
+  it( 'maps non-retryable APICallError instances without status codes to TransparentFatalError', () => {
     const error = makeApiCallError( {
       isRetryable: false
     } );
 
     const result = mapAiError( error );
 
-    expect( result ).toBeInstanceOf( FatalError );
-    expect( result.message ).toBe( 'AI-SDK fatal error: Provider rejected the request' );
+    expect( result ).toBeInstanceOf( TransparentFatalError );
     expect( result.cause ).toBe( error );
   } );
 
@@ -286,13 +207,12 @@ describe( 'mapAiError', () => {
     expect( mapAiError( error ) ).toBe( error );
   } );
 
-  it.each( fatalAiSdkErrors )( 'maps %s to FatalError', ( _name, makeError ) => {
+  it.each( fatalAiSdkErrors )( 'maps %s to TransparentFatalError', ( _name, makeError ) => {
     const error = makeError();
 
     const result = mapAiError( error );
 
-    expect( result ).toBeInstanceOf( FatalError );
-    expect( result.message ).toBe( `AI-SDK fatal error: ${error.message}` );
+    expect( result ).toBeInstanceOf( TransparentFatalError );
     expect( result.cause ).toBe( error );
   } );