zapier-platform-cli 11.1.1 → 11.3.1

package/src/oclif/commands/migrate.js

@@ -46,10 +46,14 @@ class MigrateCommand extends BaseCommand {
     }
 
     const body = {
-      percent,
+      job: {
+        name: 'migrate',
+        from_version: fromVersion,
+        to_version: toVersion,
+        email: user,
+      },
     };
     if (user) {
-      body.user = user;
       this.startSpinner(
         `Starting migration from ${fromVersion} to ${toVersion} for ${user}`
       );
@@ -58,8 +62,12 @@ class MigrateCommand extends BaseCommand {
         `Starting migration from ${fromVersion} to ${toVersion} for ${percent}%`
       );
     }
+    if (percent) {
+      body.job.percent_human = percent;
+    }
+
+    const url = `/apps/${app.id}/migrations`;
 
-    const url = `/apps/${app.id}/versions/${fromVersion}/migrate-to/${toVersion}`;
     try {
       await callAPI(url, { method: 'POST', body });
     } finally {
@@ -110,7 +118,7 @@ Start a migration to move users between different versions of your integration.
 
 Only use this command to migrate users between non-breaking versions, use \`zapier deprecate\` if you have breaking changes!
 
-Migration time varies based on the number of affected Zaps. Be patient and check \`zapier history\` to track the status.
+Migration time varies based on the number of affected Zaps. Be patient and check \`zapier jobs\` to track the status. Or use \`zapier history\` if you want to see older jobs.
 
 Since a migration is only for non-breaking changes, users are not emailed about the update/migration. It will be a transparent process for them.
 

package/src/oclif/commands/promote.js

@@ -60,19 +60,22 @@ class PromoteCommand extends BaseCommand {
       `Preparing to promote version ${version} of your integration "${app.title}".`
     );
 
-    const body = {};
-    if (changelog) {
-      body.changelog = changelog;
-    }
+    const body = {
+      job: {
+        name: 'promote',
+        to_version: version,
+        changelog,
+      },
+    };
 
     this.startSpinner(`Verifying and promoting ${version}`);
 
-    const url = `/apps/${app.id}/versions/${version}/promote/production`;
+    const url = `/apps/${app.id}/migrations`;
     try {
       await callAPI(
         url,
         {
-          method: 'PUT',
+          method: 'POST',
           body,
         },
         true
@@ -120,7 +123,7 @@ PromoteCommand.args = [
   {
     name: 'version',
     required: true,
-    description: 'The version you want promote.',
+    description: 'The version you want to promote.',
   },
 ];
 
@@ -136,6 +139,8 @@ Promote an integration version into production (non-private) rotation, which mea
 
 Promotes are an inherently safe operation for all existing users of your integration.
 
-If your integration is private and passes our integration checks, this will give you a URL to a form where you can fill in additional information for your integration to go public. After reviewing, the Zapier team will approve to make it public if there are no issues or decline with feedback.`;
+If your integration is private and passes our integration checks, this will give you a URL to a form where you can fill in additional information for your integration to go public. After reviewing, the Zapier team will approve to make it public if there are no issues or decline with feedback.
+
+Check \`zapier jobs\` to track the status of the promotion. Or use \`zapier history\` if you want to see older jobs.`;
 
 module.exports = PromoteCommand;

package/src/oclif/commands/test.js

@@ -1,5 +1,3 @@
-const path = require('path');
-const { pathExists } = require('fs-extra');
 const { flags } = require('@oclif/command');
 const chalk = require('chalk');
 
@@ -9,6 +7,7 @@ const constants = require('../../constants');
 const { buildFlags } = require('../buildFlags');
 const { readCredentials } = require('../../utils/api');
 const { runCommand } = require('../../utils/misc');
+const { getPackageManager } = require('../../utils/package-manager');
 
 class TestCommand extends BaseCommand {
   async perform() {
@@ -35,9 +34,7 @@ class TestCommand extends BaseCommand {
 
     const env = Object.assign({}, process.env, extraEnv);
 
-    const useYarn =
-      this.flags.yarn ||
-      (await pathExists(path.join(process.cwd(), 'yarn.lock')));
+    const packageManager = await getPackageManager(this.flags);
 
     const passthroughArgs = this.argv.includes('--')
       ? this.argv.slice(this.argv.indexOf('--') + 1)
@@ -47,21 +44,22 @@ class TestCommand extends BaseCommand {
       'run',
       '--silent',
       'test',
-      useYarn ? '' : '--', // yarn gives a warning if we include `--`
+      packageManager.useDoubleHyphenBeforeArgs ? '--' : '',
       ...passthroughArgs,
     ].filter(Boolean);
-    const packageManager = useYarn ? 'yarn' : 'npm';
 
     this.log('Running test suite with the following command:');
     // some extra formatting happen w/ quotes so it's clear when they're already like that in the array,
     // but the space-joined array made that unclear
     this.log(
-      `\n  ${chalk.cyanBright.bold(packageManager)} ${chalk.cyanBright(
+      `\n  ${chalk.cyanBright.bold(
+        packageManager.executable
+      )} ${chalk.cyanBright(
         argv.map((a) => (a.includes(' ') ? `"${a}"` : a)).join(' ')
       )}\n`
     );
 
-    const output = await runCommand(packageManager, argv, {
+    const output = await runCommand(packageManager.executable, argv, {
       stdio: 'inherit',
       env,
     });
@@ -81,6 +79,10 @@ TestCommand.flags = buildFlags({
       description:
         "Use `yarn` instead of `npm`. This happens automatically if there's a `yarn.lock` file, but you can manually force `yarn` if you run tests from a sub-directory.",
     }),
+    pnpm: flags.boolean({
+      description:
+        "Use `pnpm` instead of `npm`. This happens automatically if there's a `pnpm-lock.yaml` file, but you can manually force `pnpm` if you run tests from a sub-directory.",
+    }),
   },
 });
 

package/src/oclif/hooks/renderMarkdownHelp.js

@@ -1,5 +1,5 @@
 const chalk = require('chalk');
-const marked = require('marked');
+const { marked } = require('marked');
 const TerminalRenderer = require('marked-terminal');
 const { stdtermwidth } = require('@oclif/plugin-help/lib/screen');
 

package/src/oclif/oCommands.js

@@ -15,6 +15,7 @@ module.exports = {
   'env:set': require('./commands/env/set'),
   'env:unset': require('./commands/env/unset'),
   history: require('./commands/history'),
+  jobs: require('./commands/jobs'),
   init: require('./commands/init'),
   integrations: require('./commands/integrations'),
   link: require('./commands/link'),

package/src/utils/api.js

@@ -327,6 +327,8 @@ const listLogs = (opts) => {
 const listEnv = (version) =>
   listEndpoint(`versions/${version}/environment`, 'env');
 
+const listMigrations = () => listEndpoint('migrations');
+
 // the goal of this is to call `/check` with as much info as possible
 // if the app is registered and auth is available, then we can send app id
 // otherwise, we should just send the definition and get back checks about that
@@ -411,6 +413,7 @@ module.exports = {
   listInvitees,
   listLogs,
   listVersions,
+  listMigrations,
   readCredentials,
   upload,
   validateApp,

package/src/utils/package-manager.js

@@ -0,0 +1,45 @@
+const { pathExists } = require('fs-extra');
+const path = require('path');
+
+const packageManagers = [
+  {
+    lockFile: 'package-lock.json',
+    forceFlag: undefined,
+    executable: 'npm',
+    useDoubleHyphenBeforeArgs: true,
+  },
+  {
+    lockFile: 'yarn.lock',
+    forceFlag: 'yarn',
+    executable: 'yarn',
+    useDoubleHyphenBeforeArgs: false, // yarn gives a warning if we include `--`
+  },
+  {
+    lockFile: 'pnpm-lock.yaml',
+    forceFlag: 'pnpm',
+    executable: 'pnpm',
+    useDoubleHyphenBeforeArgs: true,
+  },
+];
+
+const defaultPackageManager = packageManagers[0];
+
+const getPackageManager = async (flags) => {
+  for (const man of packageManagers) {
+    if (flags[man.forceFlag]) {
+      return man;
+    }
+  }
+
+  for (const man of packageManagers) {
+    if (await pathExists(path.join(process.cwd(), man.lockFile))) {
+      return man;
+    }
+  }
+
+  return defaultPackageManager;
+};
+
+module.exports = {
+  getPackageManager,
+};