fix: add validation for get_resource_data response and improve datetime formatting handling

Co-authored-by: Copilot <copilot@github.com>

Author: ivictbor

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,