@botbotgo/better-call 0.1.40 → 0.1.42

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@botbotgo/better-call",
-  "version": "0.1.40",
+  "version": "0.1.42",
   "description": "Small-model tool-call reliability layer for LangChain and custom agent runtimes.",
   "type": "module",
   "license": "Apache-2.0",

package/scripts/demo.mjs

@@ -8,6 +8,18 @@ function errorMessage(error) {
   return error instanceof Error ? error.message : String(error);
 }
 
+function createGatewayErrorResult(originalToolName, reason, details = {}) {
+  return {
+    status: "error",
+    originalToolName,
+    toolName: null,
+    reason,
+    errorType: details.type ?? null,
+    errorMessage: details.message ?? null,
+    errorCause: details.cause ?? null,
+  };
+}
+
 function errorDetails(error) {
   if (error instanceof Error) {
     const causeMessage =
@@ -76,7 +88,7 @@ const stockQuote = {
       throw new Error("Invalid ticker: expected a non-empty string.");
     }
 
-    if (!VALID_MARKETS.includes(market)) {
+    if (typeof market !== "string" || !VALID_MARKETS.includes(market)) {
       throw new Error("Invalid market: expected one of US, HK, CN.");
     }
 
@@ -86,7 +98,7 @@ const stockQuote = {
 
 // Intentionally malformed repair input: "symbol" should be "ticker", and
 // "NASDAQ" is not one of the schema-valid markets ("US", "HK", or "CN").
-const malformedRepairArgs = { symbol: "Apple", market: "NASDAQ" };
+const intentionallyMalformedArgs = { symbol: "Apple", market: "NASDAQ" };
 
 const wrappedTools = betterTools([stockQuote], {
   userInput: "Get Apple stock in the US market.",
@@ -106,9 +118,9 @@ const wrappedStockQuote = wrappedTools[0];
 
 let wrappedOutput;
 try {
-  wrappedOutput = await wrappedStockQuote.invoke(malformedRepairArgs);
+  wrappedOutput = await wrappedStockQuote.invoke(intentionallyMalformedArgs);
 } catch (error) {
-  const context = `tool=stock_quote args=${JSON.stringify(malformedRepairArgs)}`;
+  const context = `tool=stock_quote args=${JSON.stringify(intentionallyMalformedArgs)}`;
   const message = `${errorMessage(error)} (${context})`;
   const contextualError =
     error instanceof Error ? new Error(message, { cause: error }) : new Error(message);
@@ -121,9 +133,9 @@ try {
     userInput: "Get Apple stock in the US market.",
     tools: [stockQuote],
     // Both the tool name and args are intentionally wrong to demonstrate repair:
-    // "stock_price" should be "stock_quote"; malformedRepairArgs uses
+    // "stock_price" should be "stock_quote"; intentionallyMalformedArgs uses
     // "symbol"/"NASDAQ" instead of schema-valid stock_quote args.
-    calls: [{ tool: "stock_price", args: malformedRepairArgs }],
+    calls: [{ tool: "stock_price", args: intentionallyMalformedArgs }],
     repair() {
       return [{ tool: "stock_quote", args: { ticker: "AAPL", market: "US" } }];
     },
@@ -157,39 +169,31 @@ try {
   });
 } catch (error) {
   const details = errorDetails(error);
-  gatewayResult = {
-    status: "error",
-    originalToolName: "research",
-    toolName: null,
-    reason: `repairToolCall failed: ${details.message}`,
-    errorType: details.type,
-    errorMessage: details.message,
-    errorCause: details.cause,
-  };
+  gatewayResult = createGatewayErrorResult(
+    "research",
+    `repairToolCall failed: ${details.message}`,
+    details
+  );
 }
 
 const gatewayRepair =
   gatewayResult != null && typeof gatewayResult === "object"
     ? gatewayResult
-    : {
-        status: "error",
-        originalToolName: "research",
-        toolName: null,
-        reason: "repairToolCall returned no result.",
-        errorType: null,
-        errorMessage: null,
-        errorCause: null,
-      };
-
-const repairedCall = reliableResult?.calls?.[0] ?? null;
+    : createGatewayErrorResult("research", "repairToolCall returned no result.");
+
+// Missing or empty calls are expected in some demo/error paths; normalize to null for reporting.
+const repairedCall =
+  Array.isArray(reliableResult?.calls) && reliableResult.calls.length > 0
+    ? reliableResult.calls[0]
+    : null;
 
 const report = {
   wrappedTool: {
-    before: malformedRepairArgs,
+    before: intentionallyMalformedArgs,
     after: wrappedOutput,
   },
   reliableToolCalls: {
-    before: { tool: "stock_price", args: malformedRepairArgs },
+    before: { tool: "stock_price", args: intentionallyMalformedArgs },
     repaired: repairedCall,
     diagnostics: reliableResult?.diagnostics ?? null,
   },