@zapier/zapier-sdk-cli 0.13.9 → 0.13.11

package/dist/index.cjs

@@ -107,28 +107,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") {
@@ -161,7 +161,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,
+  authBaseUrl,
+  authClientId
+}) => {
+  const authOptions = { baseUrl, authBaseUrl };
+  const tokenUrl = zapierSdkCliLogin.getAuthTokenUrl(authOptions);
+  const authorizeUrl = zapierSdkCliLogin.getAuthAuthorizeUrl(authOptions);
   zapierSdkCliLogin.logout();
   const availablePort = await findAvailablePort();
   const redirectUri = `http://localhost:${availablePort}/oauth`;
@@ -187,9 +195,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: authClientId || zapierSdkCliLogin.ZAPIER_AUTH_CLIENT_ID,
     redirect_uri: redirectUri,
     scope: "internal offline_access",
     state: generateRandomString(),
@@ -218,26 +226,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: authClientId || zapierSdkCliLogin.ZAPIER_AUTH_CLIENT_ID,
       code_verifier: codeVerifier
     },
     {
@@ -258,7 +266,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 = {}) {
@@ -316,23 +324,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");
+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);
   }
-  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 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;
@@ -649,14 +669,6 @@ var addPlugin = ({ sdk }) => {
     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,
@@ -1276,17 +1288,12 @@ var generateAppTypesPlugin = ({ sdk }) => {
       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 = sdk.listApps({ appKeys }).items();
@@ -1314,8 +1321,8 @@ var generateAppTypesPlugin = ({ sdk }) => {
         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 = {};
@@ -1328,17 +1335,18 @@ var generateAppTypesPlugin = ({ sdk }) => {
       });
       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) {
@@ -1361,56 +1369,47 @@ var generateAppTypesPlugin = ({ sdk }) => {
           }
         }
         const manifestKey = getManifestKey(app);
-        try {
-          const generator = new AstTypeGenerator();
-          const typeDefinitionString = await generator.generateTypes({
-            app,
-            authenticationId,
-            sdk
-          });
-          result.typeDefinitions[manifestKey] = typeDefinitionString;
+        const generator = new AstTypeGenerator();
+        const typeDefinitionString = await generator.generateTypes({
+          app,
+          authenticationId,
+          sdk
+        });
+        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;
@@ -1445,9 +1444,6 @@ var BuildManifestSchema = zod.z.object({
 var buildManifestPlugin = ({ sdk, 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 = sdk.listApps({ appKeys }).items();
     const apps = [];
@@ -1457,7 +1453,7 @@ var buildManifestPlugin = ({ sdk, context }) => {
     }
     onProgress?.({ type: "apps_lookup_complete", count: apps.length });
     if (apps.length === 0) {
-      return result;
+      return {};
     }
     let updatedManifest;
     for (const app of apps) {
@@ -1490,20 +1486,23 @@ var buildManifestPlugin = ({ sdk, 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,

package/dist/index.d.mts

@@ -50,10 +50,6 @@ type ManifestBuildProgressEvent = {
 };
 interface BuildManifestResult {
     manifest?: Manifest;
-    errors: Array<{
-        appKey: string;
-        error: string;
-    }>;
 }
 
 interface BuildManifestPluginProvides {
@@ -132,10 +128,6 @@ type AppTypesProgressEvent = {
 interface GenerateAppTypesResult {
     typeDefinitions: Record<string, string>;
     writtenFiles?: Record<string, string>;
-    errors: Array<{
-        appKey: string;
-        error: string;
-    }>;
 }
 
 interface GenerateAppTypesPluginProvides {

package/dist/index.d.ts

@@ -50,10 +50,6 @@ type ManifestBuildProgressEvent = {
 };
 interface BuildManifestResult {
     manifest?: Manifest;
-    errors: Array<{
-        appKey: string;
-        error: string;
-    }>;
 }
 
 interface BuildManifestPluginProvides {
@@ -132,10 +128,6 @@ type AppTypesProgressEvent = {
 interface GenerateAppTypesResult {
     typeDefinitions: Record<string, string>;
     writtenFiles?: Record<string, string>;
-    errors: Array<{
-        appKey: string;
-        error: string;
-    }>;
 }
 
 interface GenerateAppTypesPluginProvides {