@adminforth/agent 1.26.8 → 1.26.10

package/apiBasedTools.ts

@@ -110,11 +110,17 @@ const TOOL_OVERRIDES: Record<string, ToolOverride> = {
         return output;
       }
 
+      if (!hasGetResourceDataRows(output)) {
+        logger.warn(
+          `Skipping datetime formatting for get_resource_data because response.data is not an array for resource ${getInputString(inputs, 'resourceId') ?? 'unknown'}`,
+        );
+        return output;
+      }
+
       const localizedTimeZone = userTimeZone ?? DEFAULT_USER_TIME_ZONE;
-      const response = output as GetResourceDataToolResponse;
-      formatDateTimeColumns(response.data, dateTimeColumnNames, localizedTimeZone);
+      formatDateTimeColumns(output.data, dateTimeColumnNames, localizedTimeZone);
 
-      return response;
+      return output;
     },
   },
   aggregate: {
@@ -288,6 +294,14 @@ function hasToolError(output: unknown): output is { error: unknown } {
   return typeof output === 'object' && output !== null && 'error' in output;
 }
 
+function hasGetResourceDataRows(output: unknown): output is GetResourceDataToolResponse {
+  if (typeof output !== 'object' || output === null || !('data' in output)) {
+    return false;
+  }
+
+  return Array.isArray((output as { data?: unknown }).data);
+}
+
 function getDateTimeColumnNames(
   adminforth: IAdminForth,
   inputs: Record<string, unknown> | undefined,
@@ -726,11 +740,13 @@ async function callOpenApiSchema(params: {
     toolName,
   });
   const hasRequestBody = !METHODS_WITHOUT_REQUEST_BODY.has(method);
+  logger.info(`Calling OpenAPI tool "${toolName}" with method ${method} at URL ${requestUrl}`);
   const response = await fetch(hasRequestBody ? requestUrl : appendInputsToQueryString(requestUrl, body), {
     method,
     headers: createToolRequestHeaders(httpExtra, userTimeZone),
     body: hasRequestBody ? JSON.stringify(body) : undefined,
   });
+  logger.info(`Received response with status ${response.status} from OpenAPI tool "${toolName}"`);
 
   return parseOpenApiToolResponse(response);
 }

package/build.log

@@ -38,5 +38,5 @@ custom/skills/fetch_data/SKILL.md
 custom/skills/mutate_data/
 custom/skills/mutate_data/SKILL.md
 
-sent 207,935 bytes  received 562 bytes  416,994.00 bytes/sec
+sent 207,931 bytes  received 562 bytes  416,986.00 bytes/sec
 total size is 205,621  speedup is 0.99

package/dist/apiBasedTools.js

@@ -55,6 +55,7 @@ const TOOL_OVERRIDES = {
     get_resource_data: {
         format_tool: ({ inputs, resourceLabel }) => (`Get ${getDataPrefix(inputs)}${resourceLabel}`),
         post_process_response: (_a) => __awaiter(void 0, [_a], void 0, function* ({ adminforth, output, inputs, userTimeZone }) {
+            var _b;
             if (hasToolError(output)) {
                 return output;
             }
@@ -62,10 +63,13 @@ const TOOL_OVERRIDES = {
             if (dateTimeColumnNames.length === 0) {
                 return output;
             }
+            if (!hasGetResourceDataRows(output)) {
+                logger.warn(`Skipping datetime formatting for get_resource_data because response.data is not an array for resource ${(_b = getInputString(inputs, 'resourceId')) !== null && _b !== void 0 ? _b : 'unknown'}`);
+                return output;
+            }
             const localizedTimeZone = userTimeZone !== null && userTimeZone !== void 0 ? userTimeZone : DEFAULT_USER_TIME_ZONE;
-            const response = output;
-            formatDateTimeColumns(response.data, dateTimeColumnNames, localizedTimeZone);
-            return response;
+            formatDateTimeColumns(output.data, dateTimeColumnNames, localizedTimeZone);
+            return output;
         }),
     },
     aggregate: {
@@ -192,6 +196,12 @@ function wipePath(target, pathParts) {
 function hasToolError(output) {
     return typeof output === 'object' && output !== null && 'error' in output;
 }
+function hasGetResourceDataRows(output) {
+    if (typeof output !== 'object' || output === null || !('data' in output)) {
+        return false;
+    }
+    return Array.isArray(output.data);
+}
 function getDateTimeColumnNames(adminforth, inputs) {
     const resourceId = getInputString(inputs, 'resourceId');
     const resource = adminforth.config.resources.find((res) => res.resourceId === resourceId);
@@ -493,11 +503,13 @@ function callOpenApiSchema(params) {
             toolName,
         });
         const hasRequestBody = !METHODS_WITHOUT_REQUEST_BODY.has(method);
+        logger.info(`Calling OpenAPI tool "${toolName}" with method ${method} at URL ${requestUrl}`);
         const response = yield fetch(hasRequestBody ? requestUrl : appendInputsToQueryString(requestUrl, body), {
             method,
             headers: createToolRequestHeaders(httpExtra, userTimeZone),
             body: hasRequestBody ? JSON.stringify(body) : undefined,
         });
+        logger.info(`Received response with status ${response.status} from OpenAPI tool "${toolName}"`);
         return parseOpenApiToolResponse(response);
     });
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adminforth/agent",
-  "version": "1.26.8",
+  "version": "1.26.10",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
   "type": "module",