zapier-platform-core 16.0.0 → 16.1.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "zapier-platform-core",
-  "version": "16.0.0",
+  "version": "16.1.0",
   "description": "The core SDK for CLI apps in the Zapier Developer Platform.",
   "repository": "zapier/zapier-platform",
   "homepage": "https://platform.zapier.com/",
@@ -42,7 +42,7 @@
   },
   "engineStrict": true,
   "dependencies": {
-    "@zapier/secret-scrubber": "^1.1.1",
+    "@zapier/secret-scrubber": "^1.1.2",
     "bluebird": "3.7.2",
     "content-disposition": "0.5.4",
     "dotenv": "16.4.6",
@@ -53,7 +53,7 @@
     "node-fetch": "2.7.0",
     "oauth-sign": "0.9.0",
     "semver": "7.6.3",
-    "zapier-platform-schema": "16.0.0"
+    "zapier-platform-schema": "16.1.0"
   },
   "devDependencies": {
     "@types/node-fetch": "^2.6.11",

package/src/checks/dynamic-fields-have-keys.js

@@ -0,0 +1,30 @@
+'use strict';
+
+const isInputOrOutputFields = (method) =>
+  method.endsWith('.operation.inputFields') ||
+  method.endsWith('.operation.outputFields');
+
+const dynamicFieldsHaveKeys = {
+  name: 'dynamicFieldsHaveKeys',
+  shouldRun: isInputOrOutputFields,
+  run: (method, results) => {
+    const lastMethodPart = method.split('.').pop();
+
+    if (!Array.isArray(results)) {
+      const type = typeof results;
+      return [`${lastMethodPart} must be an array, got ${type}`];
+    }
+
+    const errors = [];
+    for (let i = 0; i < results.length; i++) {
+      const field = results[i];
+      if (!field || !field.key) {
+        errors.push(`${lastMethodPart}[${i}] is missing a key`);
+      }
+    }
+
+    return errors;
+  },
+};
+
+module.exports = dynamicFieldsHaveKeys;

package/src/checks/index.js

@@ -7,7 +7,11 @@ module.exports = {
   triggerIsObject: require('./trigger-is-object'),
   triggerHasUniquePrimary: require('./trigger-has-unique-primary'),
   triggerHasId: require('./trigger-has-id'),
+
   firehoseSubscriptionIsArray: require('./firehose_is_array'),
   firehoseSubscriptionKeyIsString: require('./firehose_is_string'),
+
   performBufferReturnType: require('./perform-buffer-return-type'),
+
+  dynamicFieldsHaveKeys: require('./dynamic-fields-have-keys'),
 };

package/src/tools/fetch-logger.js

@@ -73,6 +73,9 @@ const stringifyResponseContent = async (response) => {
 //   global.fetch = wrapFetchWithLogger(global.fetch, logger);
 const wrapFetchWithLogger = (fetchFunc, logger) => {
   if (fetchFunc.patchedByZapier) {
+    // Important not to reuse logger between calls, because we always destroy
+    // the logger at the end of a Lambda call.
+    fetchFunc.zapierLogger = logger;
     return fetchFunc;
   }
 
@@ -82,25 +85,29 @@ const wrapFetchWithLogger = (fetchFunc, logger) => {
     if (requestInfo && !isZapierUserAgent(requestInfo.headers)) {
       const responseContentType = response.headers.get('content-type');
 
-      logger(`${response.status} ${requestInfo.method} ${requestInfo.url}`, {
-        log_type: 'http',
-        request_type: 'patched-devplatform-outbound',
-        request_url: requestInfo.url,
-        request_method: requestInfo.method,
-        request_headers: requestInfo.headers,
-        request_data: requestInfo.data,
-        request_via_client: false,
-        response_status_code: response.status,
-        response_headers: Object.fromEntries(response.headers.entries()),
-        response_content: shouldIncludeResponseContent(responseContentType)
-          ? await stringifyResponseContent(response)
-          : '<unsupported format>',
-      });
+      newFetch.zapierLogger(
+        `${response.status} ${requestInfo.method} ${requestInfo.url}`,
+        {
+          log_type: 'http',
+          request_type: 'patched-devplatform-outbound',
+          request_url: requestInfo.url,
+          request_method: requestInfo.method,
+          request_headers: requestInfo.headers,
+          request_data: requestInfo.data,
+          request_via_client: false,
+          response_status_code: response.status,
+          response_headers: Object.fromEntries(response.headers.entries()),
+          response_content: shouldIncludeResponseContent(responseContentType)
+            ? await stringifyResponseContent(response)
+            : '<unsupported format>',
+        },
+      );
     }
     return response;
   };
 
   newFetch.patchedByZapier = true;
+  newFetch.zapierLogger = logger;
   return newFetch;
 };
 

package/types/zapier.generated.d.ts

@@ -4,7 +4,7 @@
  * files, and/or the schema-to-ts tool and run its CLI to regenerate
  * these typings.
  *
- * zapier-platform-schema version: 16.0.0
+ * zapier-platform-schema version: 16.1.0
  *  schema-to-ts compiler version: 0.1.0
  */