zapier-platform-core 17.4.0 → 17.6.0

package/index.js

@@ -1,4 +1,6 @@
 const zapier = require('./src');
 zapier.version = require('./package.json').version;
 zapier.tools = require('./src/tools/exported');
+zapier.errors = require('./src/errors');
+zapier.console = require('./src/tools/console-singleton').consoleProxy;
 module.exports = zapier;

package/index.mjs

@@ -1,8 +1,12 @@
 import zapier from './src/index.js';
 import packageJson from './package.json' with { type: 'json' };
 import _tools from './src/tools/exported.js';
+import _errors from './src/errors.js';
+import { consoleProxy } from './src/tools/console-singleton.js';
 zapier.version = packageJson.version;
 zapier.tools = _tools;
+zapier.errors = _errors;
+zapier.console = consoleProxy;
 // Allows `import { ... } from 'zapier-platform-core'`
 export const {
   createAppHandler,
@@ -14,8 +18,10 @@ export const {
   defineSearch,
   defineTrigger,
   integrationTestHandler,
+  console,
   tools,
   version,
+  errors,
 } = zapier;
 // Allows `import zapier from 'zapier-platform-core'`
 export default zapier;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "zapier-platform-core",
-  "version": "17.4.0",
+  "version": "17.6.0",
   "description": "The core SDK for CLI apps in the Zapier Developer Platform.",
   "repository": "zapier/zapier-platform",
   "homepage": "https://platform.zapier.com/",
@@ -56,14 +56,14 @@
     "content-disposition": "0.5.4",
     "dotenv": "16.5.0",
     "fernet": "^0.3.3",
-    "form-data": "4.0.1",
+    "form-data": "4.0.4",
     "lodash": "4.17.21",
     "mime-types": "2.1.35",
     "node-abort-controller": "3.1.1",
     "node-fetch": "2.7.0",
     "oauth-sign": "0.9.0",
     "semver": "7.7.1",
-    "zapier-platform-schema": "17.4.0"
+    "zapier-platform-schema": "17.6.0"
   },
   "devDependencies": {
     "@types/node-fetch": "^2.6.11",

package/src/app-middlewares/before/z-object.js

@@ -10,7 +10,7 @@ const createJSONtool = require('../../tools/create-json-tool');
 const createStoreKeyTool = require('../../tools/create-storekey-tool');
 const createCallbackHigherOrderFunction = require('../../tools/create-callback-wrapper');
 const createLegacyScriptingRunner = require('../../tools/create-legacy-scripting-runner');
-const createLoggerConsole = require('../../tools/create-logger-console');
+const { initialize } = require('../../tools/console-singleton');
 const errors = require('../../errors');
 const hashing = require('../../tools/hashing');
 
@@ -19,9 +19,10 @@ const hashing = require('../../tools/hashing');
 */
 const injectZObject = (input) => {
   const bundle = _.get(input, '_zapier.event.bundle', {});
+
   const zRoot = {
     cache: createCache(input),
-    console: createLoggerConsole(input),
+    console: initialize(input),
     cursor: createStoreKeyTool(input),
     dehydrate: createDehydrator(input, 'method'),
     dehydrateFile: createDehydrator(input, 'file'),

package/src/index.js

@@ -2,6 +2,7 @@
 
 const createLambdaHandler = require('./tools/create-lambda-handler');
 const createAppTester = require('./tools/create-app-tester');
+const { consoleProxy } = require('./tools/console-singleton');
 
 let _integrationTestHandler;
 const integrationTestHandler = (event, context, callback) => {
@@ -15,5 +16,6 @@ module.exports = {
   createAppHandler: createLambdaHandler,
   createAppTester,
   integrationTestHandler,
+  console: consoleProxy,
   ...require('./typeHelpers'),
 };

package/src/tools/console-singleton.js

@@ -0,0 +1,49 @@
+'use strict';
+
+const createLoggerConsole = require('./create-logger-console');
+
+/**
+ * Shared console instance that can be used standalone or as z.console.
+ * Uses createLoggerConsole to create the single instance shared between both.
+ */
+
+// Shared console instance - will be initialized by middleware
+let loggerConsole = null;
+
+/**
+ * Initialize the console with the input context from middleware.
+ * This is called by the z-object middleware and uses createLoggerConsole.
+ * Creates a new logger console for each Lambda invocation to ensure proper isolation.
+ * Returns the shared console instance for z.console to use.
+ */
+const initialize = (input) => {
+  // Always create a new logger console for each invocation to ensure
+  // log context isolation between Lambda invocations
+  loggerConsole = createLoggerConsole(input);
+  return loggerConsole;
+};
+
+/**
+ * Reset the singleton for testing purposes
+ */
+const reset = () => {
+  loggerConsole = null;
+};
+
+/**
+ * Proxy that behaves like a console object.
+ * Forwards calls to loggerConsole if initialized, otherwise to global console.
+ */
+const consoleProxy = new Proxy(console, {
+  get(target, prop, receiver) {
+    // Use loggerConsole if initialized, otherwise fall back to global console
+    const actualConsole = loggerConsole || target;
+    return Reflect.get(actualConsole, prop, actualConsole);
+  },
+});
+
+module.exports = {
+  consoleProxy,
+  initialize,
+  reset,
+};

package/src/tools/create-logger.js

@@ -32,16 +32,68 @@ const LOG_STREAM_BYTES_LIMIT = 15 * 1024 * 1024;
 
 const DEFAULT_LOGGER_TIMEOUT = 200;
 
+// Will be initialized lazily
+const SAFE_AUTH_DATA_KEYS = new Set();
+
 const sleep = promisify(setTimeout);
 
-const isUrl = (url) => {
-  try {
-    // eslint-disable-next-line no-new
-    new URL(url);
-    return true;
-  } catch (e) {
+const initSafeAuthDataKeys = () => {
+  // An authData key in (safePrefixes x safeSuffixes) is considered safe to log
+  // uncensored
+  const safePrefixes = [
+    'account',
+    'bot_user',
+    'cloud',
+    'cloud_site',
+    'company',
+    'domain',
+    'email',
+    'environment',
+    'location',
+    'org',
+    'organization',
+    'project',
+    'region',
+    'scope',
+    'scopes',
+    'site',
+    'subdomain',
+    'team',
+    'token_type',
+    'user',
+    'workspace',
+  ];
+  const safeSuffixes = ['', '_id', '_name', 'id', 'name'];
+  for (const prefix of safePrefixes) {
+    for (const suffix of safeSuffixes) {
+      SAFE_AUTH_DATA_KEYS.add(prefix + suffix);
+    }
+  }
+};
+
+const isUrl = (value) => {
+  if (!value || typeof value !== 'string') {
     return false;
   }
+  const commonProtocols = [
+    'https://',
+    'http://',
+    'ftp://',
+    'ftps://',
+    'file://',
+  ];
+  for (const protocol of commonProtocols) {
+    if (value.startsWith(protocol)) {
+      try {
+        // eslint-disable-next-line no-new
+        new URL(value);
+        return true;
+      } catch (e) {
+        return false;
+      }
+    }
+  }
+  return false;
 };
 
 const MAX_LENGTH = 3500;
@@ -136,6 +188,13 @@ const attemptFindSecretsInStr = (s, isGettingNewSecret) => {
   return findSensitiveValues(parsedRespContent);
 };
 
+const isSafeAuthDataKey = (key) => {
+  if (SAFE_AUTH_DATA_KEYS.size === 0) {
+    initSafeAuthDataKeys();
+  }
+  return SAFE_AUTH_DATA_KEYS.has(key.toLowerCase());
+};
+
 const buildSensitiveValues = (event, data) => {
   const bundle = event.bundle || {};
   const authData = bundle.authData || {};
@@ -147,7 +206,9 @@ const buildSensitiveValues = (event, data) => {
     if (isSensitiveKey(key)) {
       return true;
     }
-
+    if (isSafeAuthDataKey(key)) {
+      return false;
+    }
     if (isUrl(value) && !isUrlWithSecrets(value)) {
       return false;
     }

package/types/console.test-d.ts

@@ -0,0 +1,24 @@
+import { expectType } from 'tsd';
+import { console } from '..';
+
+// Test that console is exported and has the right type
+expectType<Console>(console);
+
+// Test that console has expected methods with proper signatures
+expectType<{
+  (...data: any[]): void;
+  (message?: any, ...optionalParams: any[]): void;
+}>(console.log);
+expectType<{
+  (...data: any[]): void;
+  (message?: any, ...optionalParams: any[]): void;
+}>(console.error);
+expectType<{
+  (...data: any[]): void;
+  (message?: any, ...optionalParams: any[]): void;
+}>(console.warn);
+
+// Test that console can be used for logging
+console.log('test message');
+console.error('test error');
+console.warn('test warning');

package/types/custom.d.ts

@@ -11,6 +11,8 @@ import { Headers } from 'node-fetch';
 // The EXPORTED OBJECT
 export const version: string;
 export const tools: { env: { inject: (filename?: string) => void } };
+export const console: Console;
+export const errors: ErrorsModule;
 
 // see: https://github.com/zapier/zapier-platform-cli/issues/339#issue-336888249
 export const createAppTester: (
@@ -138,19 +140,30 @@ export interface Bundle<
   targetUrl?: string;
 }
 
-declare class AppError extends Error {
-  constructor(message: string, code?: string, status?: number);
-}
-declare class HaltedError extends Error {}
-declare class ExpiredAuthError extends Error {}
-declare class RefreshAuthError extends Error {}
-declare class ThrottledError extends Error {
-  constructor(message: string, delay?: number);
-}
-declare class ResponseError extends Error {
-  constructor(response: HttpResponse);
+// Error class types that match the runtime structure from src/errors.js
+type ErrorConstructor = new (message?: string) => Error;
+type AppErrorConstructor = new (message: string, code?: string, status?: number) => Error;
+type ThrottledErrorConstructor = new (message: string, delay?: number) => Error;
+type ResponseErrorConstructor = new (response: HttpResponse) => Error;
+
+interface ErrorsModule {
+  Error: AppErrorConstructor;
+  HaltedError: ErrorConstructor;
+  ExpiredAuthError: ErrorConstructor;
+  RefreshAuthError: ErrorConstructor;
+  ThrottledError: ThrottledErrorConstructor;
+  ResponseError: ResponseErrorConstructor;
+  CheckError: ErrorConstructor;
+  DehydrateError: ErrorConstructor;
+  MethodDoesNotExist: ErrorConstructor;
+  NotImplementedError: ErrorConstructor;
+  RequireModuleError: ErrorConstructor;
+  StashedBundleError: ErrorConstructor;
+  StopRequestError: ErrorConstructor;
+  handleError: (...args: any[]) => never;
 }
 
+
 // copied http stuff from external typings
 export interface HttpRequestOptions {
   agent?: Agent;
@@ -286,14 +299,7 @@ export interface ZObject {
     input_encoding?: string,
   ) => string;
 
-  errors: {
-    Error: typeof AppError;
-    HaltedError: typeof HaltedError;
-    ExpiredAuthError: typeof ExpiredAuthError;
-    RefreshAuthError: typeof RefreshAuthError;
-    ThrottledError: typeof ThrottledError;
-    ResponseError: typeof ResponseError;
-  };
+  errors: ErrorsModule;
 
   cache: {
     get: (key: string) => Promise<any>;

package/types/schemas.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: 17.3.1
+ * zapier-platform-schema version: 17.5.0
  *  schema-to-ts compiler version: 0.1.0
  */
 import type {
@@ -704,7 +704,15 @@ export interface AuthField {
   label?: string;
 
   /** The type of this value used to be. */
-  type?: 'string' | 'number' | 'boolean' | 'datetime' | 'copy' | 'password';
+  type?:
+    | 'string'
+    | 'number'
+    | 'boolean'
+    | 'datetime'
+    | 'copy'
+    | 'password'
+    | 'integer'
+    | 'text';
 
   /** If this value is required or not. This defaults to `true`. */
   required?: boolean;