@zapier/zapier-sdk-cli 0.13.9 → 0.13.11

package/CHANGELOG.md

@@ -1,5 +1,21 @@
 # @zapier/zapier-sdk-cli
 
+## 0.13.11
+
+### Patch Changes
+
+- 6de89a9: Allow overriding ZAPIER_BASE_URL with env vars, and also add new ZAPIER_TRACKING_BASE_URL, ZAPIER_AUTH_BASE_URL, and ZAPIER_AUTH_CLIENT_ID env vars and parameters.
+- Updated dependencies [6de89a9]
+  - @zapier/zapier-sdk-cli-login@0.3.4
+  - @zapier/zapier-sdk@0.13.8
+  - @zapier/zapier-sdk-mcp@0.3.21
+
+## 0.13.10
+
+### Patch Changes
+
+- f56c6a0: Refactor buildManifest and generateAppTypes to throw proper errors, instead of returning an error property
+
 ## 0.13.9
 
 ### Patch Changes

package/dist/cli.cjs

@@ -1251,28 +1251,28 @@ var client_default = api;
 
 // src/utils/getCallablePromise.ts
 var getCallablePromise = () => {
-  let resolve3 = () => {
+  let resolve4 = () => {
   };
   let reject = () => {
   };
   const promise = new Promise((_resolve, _reject) => {
-    resolve3 = _resolve;
+    resolve4 = _resolve;
     reject = _reject;
   });
   return {
     promise,
-    resolve: resolve3,
+    resolve: resolve4,
     reject
   };
 };
 var getCallablePromise_default = getCallablePromise;
 var findAvailablePort = () => {
-  return new Promise((resolve3, reject) => {
+  return new Promise((resolve4, reject) => {
     let portIndex = 0;
     const tryPort = (port) => {
       const server = express__default.default().listen(port, () => {
         server.close();
-        resolve3(port);
+        resolve4(port);
       });
       server.on("error", (err) => {
         if (err.code === "EADDRINUSE") {
@@ -1305,7 +1305,15 @@ var generateRandomString = () => {
     (dec) => ("0" + dec.toString(16)).substring(-2)
   ).join("");
 };
-var login = async (timeoutMs = LOGIN_TIMEOUT_MS) => {
+var login = async ({
+  timeoutMs = LOGIN_TIMEOUT_MS,
+  baseUrl: baseUrl2,
+  authBaseUrl: authBaseUrl2,
+  authClientId: authClientId2
+}) => {
+  const authOptions = { baseUrl: baseUrl2, authBaseUrl: authBaseUrl2 };
+  const tokenUrl = zapierSdkCliLogin.getAuthTokenUrl(authOptions);
+  const authorizeUrl = zapierSdkCliLogin.getAuthAuthorizeUrl(authOptions);
   zapierSdkCliLogin.logout();
   const availablePort = await findAvailablePort();
   const redirectUri = `http://localhost:${availablePort}/oauth`;
@@ -1331,9 +1339,9 @@ var login = async (timeoutMs = LOGIN_TIMEOUT_MS) => {
   process.on("SIGINT", cleanup);
   process.on("SIGTERM", cleanup);
   const { code_verifier: codeVerifier, code_challenge: codeChallenge } = await pkceChallenge__default.default();
-  const authUrl = `${zapierSdkCliLogin.ZAPIER_BASE}/oauth/authorize/?${new URLSearchParams({
+  const authUrl = `${authorizeUrl}?${new URLSearchParams({
     response_type: "code",
-    client_id: zapierSdkCliLogin.LOGIN_CLIENT_ID,
+    client_id: authClientId2 || zapierSdkCliLogin.ZAPIER_AUTH_CLIENT_ID,
     redirect_uri: redirectUri,
     scope: "internal offline_access",
     state: generateRandomString(),
@@ -1362,26 +1370,26 @@ var login = async (timeoutMs = LOGIN_TIMEOUT_MS) => {
   } finally {
     process.off("SIGINT", cleanup);
     process.off("SIGTERM", cleanup);
-    await new Promise((resolve3) => {
+    await new Promise((resolve4) => {
       const timeout = setTimeout(() => {
         log_default.info("Server close timed out, forcing connection shutdown...");
         connections.forEach((conn) => conn.destroy());
-        resolve3();
+        resolve4();
       }, 1e3);
       server.close(() => {
         clearTimeout(timeout);
-        resolve3();
+        resolve4();
       });
     });
   }
   log_default.info("Exchanging authorization code for tokens...");
   const { data } = await client_default.post(
-    `${zapierSdkCliLogin.ZAPIER_BASE}/oauth/token/`,
+    tokenUrl,
     {
       grant_type: "authorization_code",
       code: await promisedCode,
       redirect_uri: redirectUri,
-      client_id: zapierSdkCliLogin.LOGIN_CLIENT_ID,
+      client_id: authClientId2 || zapierSdkCliLogin.ZAPIER_AUTH_CLIENT_ID,
       code_verifier: codeVerifier
     },
     {
@@ -1402,7 +1410,7 @@ var LoginSchema = zod.z.object({
 
 // package.json
 var package_default = {
-  version: "0.13.9"};
+  version: "0.13.11"};
 
 // src/telemetry/builders.ts
 function createCliBaseEvent(context = {}) {
@@ -1460,23 +1468,35 @@ function buildCliCommandExecutedEvent({
 
 // src/plugins/login/index.ts
 var CLI_COMMAND_EXECUTED_EVENT_SUBJECT = "platform.sdk.CliCommandExecutedEvent";
-var loginWithSdk = zapierSdk.createFunction(async function loginWithSdk2(options) {
-  const timeoutSeconds = options.timeout ? parseInt(options.timeout, 10) : 300;
-  if (isNaN(timeoutSeconds) || timeoutSeconds <= 0) {
-    throw new Error("Timeout must be a positive number");
-  }
-  await login_default(timeoutSeconds * 1e3);
-  const user = await zapierSdkCliLogin.getLoggedInUser();
-  console.log(`\u2705 Successfully logged in as ${user.email}`);
-  setTimeout(() => process.exit(0), 100);
-}, LoginSchema);
+var loginWithSdk = zapierSdk.createFunction(
+  async (options) => {
+    const timeoutSeconds = options.timeout ? parseInt(options.timeout, 10) : 300;
+    if (isNaN(timeoutSeconds) || timeoutSeconds <= 0) {
+      throw new Error("Timeout must be a positive number");
+    }
+    await login_default({
+      timeoutMs: timeoutSeconds * 1e3,
+      baseUrl: options.baseUrl,
+      authBaseUrl: options.authBaseUrl,
+      authClientId: options.authClientId
+    });
+    const user = await zapierSdkCliLogin.getLoggedInUser();
+    console.log(`\u2705 Successfully logged in as ${user.email}`);
+    setTimeout(() => process.exit(0), 100);
+  }
+);
 var loginPlugin = ({ context }) => {
   const loginWithTelemetry = async (options) => {
     const startTime = Date.now();
     let success = false;
     let errorMessage = null;
     try {
-      await loginWithSdk(options);
+      await loginWithSdk({
+        ...options,
+        baseUrl: context.options?.baseUrl,
+        authBaseUrl: context.options?.authBaseUrl,
+        authClientId: context.options?.authClientId
+      });
       success = true;
     } catch (error) {
       success = false;
@@ -1793,14 +1813,6 @@ var addPlugin = ({ sdk: sdk2 }) => {
     if (successfulApps.length > 0) {
       console.log(`\u2705 Added ${successfulApps.length} app(s) to manifest`);
     }
-    const allErrors = [...manifestResult.errors, ...typesResult.errors];
-    if (allErrors.length > 0) {
-      console.warn(`
-\u26A0\uFE0F ${allErrors.length} error(s) occurred:`);
-      allErrors.forEach(({ appKey, error }) => {
-        console.warn(`  - ${appKey}: ${error}`);
-      });
-    }
   }, AddSchema);
   return {
     add,
@@ -2420,17 +2432,12 @@ var generateAppTypesPlugin = ({ sdk: sdk2 }) => {
       appKeys,
       authenticationIds,
       skipWrite = false,
-      typesOutputDirectory,
+      typesOutputDirectory = await detectTypesOutputDirectory(),
       onProgress
     } = options;
-    if (!skipWrite && !typesOutputDirectory) {
-      throw new Error(
-        "typesOutputDirectory is required when skipWrite is false"
-      );
-    }
+    const resolvedTypesOutput = path.resolve(typesOutputDirectory);
     const result = {
-      typeDefinitions: {},
-      errors: []
+      typeDefinitions: {}
     };
     onProgress?.({ type: "apps_lookup_start", count: appKeys.length });
     const appsIterator = sdk2.listApps({ appKeys }).items();
@@ -2458,8 +2465,8 @@ var generateAppTypesPlugin = ({ sdk: sdk2 }) => {
         count: authentications.length
       });
     }
-    if (!skipWrite && typesOutputDirectory) {
-      await promises.mkdir(typesOutputDirectory, { recursive: true });
+    if (!skipWrite && resolvedTypesOutput) {
+      await promises.mkdir(resolvedTypesOutput, { recursive: true });
     }
     if (!skipWrite) {
       result.writtenFiles = {};
@@ -2472,17 +2479,18 @@ var generateAppTypesPlugin = ({ sdk: sdk2 }) => {
       });
       try {
         if (!app.version) {
-          const error = `Invalid implementation ID format: ${app.implementation_id}. Expected format: <implementationName>@<version>`;
-          result.errors.push({
-            appKey: app.key,
-            error
-          });
+          const errorMessage = `Invalid implementation ID format: ${app.implementation_id}. Expected format: <implementationName>@<version>`;
           onProgress?.({
             type: "app_processing_error",
             appKey: app.key,
-            error
+            error: errorMessage
+          });
+          throw new zapierSdk.ZapierValidationError(errorMessage, {
+            details: {
+              appKey: app.key,
+              implementationId: app.implementation_id
+            }
           });
-          continue;
         }
         let authenticationId;
         if (authentications.length > 0) {
@@ -2505,56 +2513,47 @@ var generateAppTypesPlugin = ({ sdk: sdk2 }) => {
           }
         }
         const manifestKey = getManifestKey(app);
-        try {
-          const generator = new AstTypeGenerator();
-          const typeDefinitionString = await generator.generateTypes({
-            app,
-            authenticationId,
-            sdk: sdk2
-          });
-          result.typeDefinitions[manifestKey] = typeDefinitionString;
+        const generator = new AstTypeGenerator();
+        const typeDefinitionString = await generator.generateTypes({
+          app,
+          authenticationId,
+          sdk: sdk2
+        });
+        result.typeDefinitions[manifestKey] = typeDefinitionString;
+        onProgress?.({
+          type: "type_generated",
+          manifestKey,
+          sizeBytes: typeDefinitionString.length
+        });
+        if (!skipWrite && resolvedTypesOutput && result.writtenFiles) {
+          const filePath = path.join(resolvedTypesOutput, `${manifestKey}.d.ts`);
+          await promises.writeFile(filePath, typeDefinitionString, "utf8");
+          result.writtenFiles[manifestKey] = filePath;
           onProgress?.({
-            type: "type_generated",
+            type: "file_written",
             manifestKey,
-            sizeBytes: typeDefinitionString.length
-          });
-          if (!skipWrite && typesOutputDirectory && result.writtenFiles) {
-            const filePath = path.join(typesOutputDirectory, `${manifestKey}.d.ts`);
-            await promises.writeFile(filePath, typeDefinitionString, "utf8");
-            result.writtenFiles[manifestKey] = filePath;
-            onProgress?.({
-              type: "file_written",
-              manifestKey,
-              filePath
-            });
-          }
-          onProgress?.({
-            type: "app_processing_complete",
-            appKey: app.key
-          });
-        } catch (error) {
-          const errorMessage = `Failed to generate types: ${error}`;
-          result.errors.push({
-            appKey: app.key,
-            error: errorMessage
-          });
-          onProgress?.({
-            type: "app_processing_error",
-            appKey: app.key,
-            error: errorMessage
+            filePath
           });
         }
-      } catch (error) {
-        const errorMessage = `Failed to process app: ${error}`;
-        result.errors.push({
-          appKey: app.key,
-          error: errorMessage
+        onProgress?.({
+          type: "app_processing_complete",
+          appKey: app.key
         });
+      } catch (error) {
+        const errorMessage = `Failed to process app ${app.key}: ${error instanceof Error ? error.message : String(error)}`;
         onProgress?.({
           type: "app_processing_error",
           appKey: app.key,
           error: errorMessage
         });
+        if (error instanceof zapierSdk.ZapierValidationError) {
+          throw error;
+        } else {
+          throw new zapierSdk.ZapierUnknownError(errorMessage, {
+            cause: error
+            // Works for both Error and non-Error
+          });
+        }
       }
     }
     return result;
@@ -2589,9 +2588,6 @@ var BuildManifestSchema = zod.z.object({
 var buildManifestPlugin = ({ sdk: sdk2, context }) => {
   const buildManifest = zapierSdk.createFunction(async function buildManifest2(options) {
     const { appKeys, skipWrite = false, configPath, onProgress } = options;
-    const result = {
-      errors: []
-    };
     onProgress?.({ type: "apps_lookup_start", count: appKeys.length });
     const appsIterator = sdk2.listApps({ appKeys }).items();
     const apps = [];
@@ -2601,7 +2597,7 @@ var buildManifestPlugin = ({ sdk: sdk2, context }) => {
     }
     onProgress?.({ type: "apps_lookup_complete", count: apps.length });
     if (apps.length === 0) {
-      return result;
+      return {};
     }
     let updatedManifest;
     for (const app of apps) {
@@ -2634,20 +2630,23 @@ var buildManifestPlugin = ({ sdk: sdk2, context }) => {
         });
         onProgress?.({ type: "app_processing_complete", appKey: app.key });
       } catch (error) {
-        const errorMessage = `Failed to process app: ${error}`;
-        result.errors.push({
-          appKey: app.key,
-          error: errorMessage
-        });
+        const errorMessage = `Failed to process app ${app.key}: ${error instanceof Error ? error.message : String(error)}`;
         onProgress?.({
           type: "app_processing_error",
           appKey: app.key,
           error: errorMessage
         });
+        if (error instanceof zapierSdk.ZapierValidationError) {
+          throw error;
+        } else {
+          throw new zapierSdk.ZapierUnknownError(errorMessage, {
+            cause: error
+            // Works for both Error and non-Error
+          });
+        }
       }
     }
-    result.manifest = updatedManifest;
-    return result;
+    return { manifest: updatedManifest };
   }, BuildManifestSchema);
   return {
     buildManifest,
@@ -2671,14 +2670,32 @@ function createZapierCliSdk(options = {}) {
 
 // package.json with { type: 'json' }
 var package_default2 = {
-  version: "0.13.9"};
+  version: "0.13.11"};
 
 // src/cli.ts
 var program = new commander.Command();
-program.name("zapier-sdk").description("CLI for Zapier SDK").version(package_default2.version, "-v, --version", "display version number").option("--debug", "Enable debug logging");
+program.name("zapier-sdk").description("CLI for Zapier SDK").version(package_default2.version, "-v, --version", "display version number").option("--debug", "Enable debug logging").option("--base-url <url>", "Base URL for Zapier API endpoints").option(
+  "--auth-base-url <url>",
+  "Base URL for Zapier authentication endpoints"
+).option("--auth-client-id <id>", "OAuth client ID for Zapier authentication").option(
+  "--tracking-base-url <url>",
+  "Base URL for Zapier tracking endpoints"
+);
 var isDebugMode = process.env.DEBUG === "true" || process.argv.includes("--debug");
+var baseUrlIndex = process.argv.indexOf("--base-url");
+var authBaseUrlIndex = process.argv.indexOf("--auth-base-url");
+var authClientIdIndex = process.argv.indexOf("--auth-client-id");
+var trackingBaseUrlIndex = process.argv.indexOf("--tracking-base-url");
+var baseUrl = baseUrlIndex !== -1 ? process.argv[baseUrlIndex + 1] : void 0;
+var authBaseUrl = authBaseUrlIndex !== -1 ? process.argv[authBaseUrlIndex + 1] : void 0;
+var authClientId = authClientIdIndex !== -1 ? process.argv[authClientIdIndex + 1] : void 0;
+var trackingBaseUrl = trackingBaseUrlIndex !== -1 ? process.argv[trackingBaseUrlIndex + 1] : void 0;
 var sdk = createZapierCliSdk({
-  debug: isDebugMode
+  debug: isDebugMode,
+  baseUrl,
+  authBaseUrl,
+  authClientId,
+  trackingBaseUrl
 });
 generateCliCommands(program, sdk);
 program.parse();