heroku 9.2.0 → 9.2.1-beta.0

package/lib/commands/local/run.js

@@ -4,14 +4,14 @@ const completions_1 = require("@heroku-cli/command/lib/completions");
 const core_1 = require("@oclif/core");
 const color_1 = require("@heroku-cli/color");
 const fork_foreman_1 = require("../../lib/local/fork-foreman");
-const helpers_1 = require("../../lib/run/helpers");
 const fs = require("fs");
 class Run extends core_1.Command {
     async run() {
         const execArgv = ['run'];
         const { argv, flags } = await this.parse(Run);
-        const userArgvInputOrder = (0, helpers_1.revertSortedArgs)(process.argv, argv);
-        if (userArgvInputOrder.length === 0) {
+        const maybeOptionsIndex = process.argv.indexOf('--');
+        const commandArgs = (maybeOptionsIndex === -1 ? argv : process.argv.slice(maybeOptionsIndex + 1));
+        if (commandArgs.length === 0) {
             const errorMessage = 'Usage: heroku local:run [COMMAND]\nMust specify command to run';
             this.error(errorMessage, { exit: -1 });
         }
@@ -24,7 +24,7 @@ class Run extends core_1.Command {
         if (flags.port)
             execArgv.push('--port', flags.port);
         execArgv.push('--'); // disable node-foreman flag parsing
-        execArgv.push(...userArgvInputOrder); // eslint-disable-line unicorn/no-array-push-push
+        execArgv.push(...commandArgs); // eslint-disable-line unicorn/no-array-push-push
         await (0, fork_foreman_1.fork)(execArgv);
     }
 }

package/lib/commands/pg/promote.js

@@ -17,36 +17,36 @@ class Promote extends command_1.Command {
         core_1.ux.action.start(`Ensuring an alternate alias for existing ${color_1.default.green('DATABASE_URL')}`);
         const { body: attachments } = await this.heroku.get(`/apps/${app}/addon-attachments`);
         const current = attachments.find(a => a.name === 'DATABASE');
-        if (!current)
-            return;
-        // eslint-disable-next-line eqeqeq
-        if (((_a = current.addon) === null || _a === void 0 ? void 0 : _a.name) === attachment.addon.name && current.namespace == attachment.namespace) {
-            if (attachment.namespace) {
-                core_1.ux.error(`${color_1.default.cyan(attachment.name)} is already promoted on ${color_1.default.app(app)}`);
+        if (current) {
+            // eslint-disable-next-line eqeqeq
+            if (((_a = current.addon) === null || _a === void 0 ? void 0 : _a.name) === attachment.addon.name && current.namespace == attachment.namespace) {
+                if (attachment.namespace) {
+                    core_1.ux.error(`${color_1.default.cyan(attachment.name)} is already promoted on ${color_1.default.app(app)}`);
+                }
+                else {
+                    core_1.ux.error(`${color_1.default.addon(attachment.addon.name)} is already promoted on ${color_1.default.app(app)}`);
+                }
+            }
+            const existing = attachments.filter(a => { var _a, _b; return ((_a = a.addon) === null || _a === void 0 ? void 0 : _a.id) === ((_b = current.addon) === null || _b === void 0 ? void 0 : _b.id) && a.namespace === current.namespace; })
+                .find(a => a.name !== 'DATABASE');
+            if (existing) {
+                core_1.ux.action.stop(color_1.default.green(existing.name + '_URL'));
             }
             else {
-                core_1.ux.error(`${color_1.default.addon(attachment.addon.name)} is already promoted on ${color_1.default.app(app)}`);
+                // The current add-on occupying the DATABASE attachment has no
+                // other attachments. In order to promote this database without
+                // error, we can create a secondary attachment, just-in-time.
+                const { body: backup } = await this.heroku.post('/addon-attachments', {
+                    body: {
+                        app: { name: app },
+                        addon: { name: (_b = current.addon) === null || _b === void 0 ? void 0 : _b.name },
+                        namespace: current.namespace,
+                        confirm: app,
+                    },
+                });
+                core_1.ux.action.stop(color_1.default.green(backup.name + '_URL'));
             }
         }
-        const existing = attachments.filter(a => { var _a, _b; return ((_a = a.addon) === null || _a === void 0 ? void 0 : _a.id) === ((_b = current.addon) === null || _b === void 0 ? void 0 : _b.id) && a.namespace === current.namespace; })
-            .find(a => a.name !== 'DATABASE');
-        if (existing) {
-            core_1.ux.action.stop(color_1.default.green(existing.name + '_URL'));
-        }
-        else {
-            // The current add-on occupying the DATABASE attachment has no
-            // other attachments. In order to promote this database without
-            // error, we can create a secondary attachment, just-in-time.
-            const { body: backup } = await this.heroku.post('/addon-attachments', {
-                body: {
-                    app: { name: app },
-                    addon: { name: (_b = current.addon) === null || _b === void 0 ? void 0 : _b.name },
-                    namespace: current.namespace,
-                    confirm: app,
-                },
-            });
-            core_1.ux.action.stop(color_1.default.green(backup.name + '_URL'));
-        }
         if (!force) {
             const { body: status } = await this.heroku.get(`/client/v11/databases/${attachment.addon.id}/wait_status`, {
                 hostname: (0, host_1.default)(),
@@ -54,9 +54,9 @@ class Promote extends command_1.Command {
             if (status['waiting?']) {
                 core_1.ux.error((0, tsheredoc_1.default)(`
           Database cannot be promoted while in state: ${status.message}
-          
+
           Promoting this database can lead to application errors and outage. Please run ${color_1.default.cmd('heroku pg:wait')} to wait for database to become available.
-          
+
           To ignore this error, you can pass the --force flag to promote the database and risk application issues.
         `));
             }
@@ -79,7 +79,7 @@ class Promote extends command_1.Command {
             },
         });
         core_1.ux.action.stop();
-        const currentPooler = attachments.find(a => { var _a, _b; return a.namespace === 'connection-pooling:default' && ((_a = a.addon) === null || _a === void 0 ? void 0 : _a.id) === ((_b = current.addon) === null || _b === void 0 ? void 0 : _b.id) && a.name === 'DATABASE_CONNECTION_POOL'; });
+        const currentPooler = attachments.find(a => { var _a, _b; return a.namespace === 'connection-pooling:default' && ((_a = a.addon) === null || _a === void 0 ? void 0 : _a.id) === ((_b = current === null || current === void 0 ? void 0 : current.addon) === null || _b === void 0 ? void 0 : _b.id) && a.name === 'DATABASE_CONNECTION_POOL'; });
         if (currentPooler) {
             core_1.ux.action.start('Reattaching pooler to new leader');
             await this.heroku.post('/addon-attachments', {
@@ -100,9 +100,9 @@ class Promote extends command_1.Command {
             const unfollowLeaderCmd = `heroku pg:unfollow ${attachment.addon.name}`;
             core_1.ux.warn((0, tsheredoc_1.default)(`
         Your database has been promoted but it is currently a follower database in read-only mode.
-        
+
         Promoting a database with ${color_1.default.cmd('heroku pg:promote')} doesn't automatically unfollow its leader.
-        
+
         Use ${color_1.default.cmd(unfollowLeaderCmd)} to stop this follower from replicating from its leader (${color_1.default.yellow(promotedDatabaseDetails.leader)}) and convert it into a writable database.
       `));
         }
@@ -141,7 +141,7 @@ class Promote extends command_1.Command {
                         msg += 'without an attached DATABASE_URL.';
                     }
                     else {
-                        msg += `with ${(_c = current.addon) === null || _c === void 0 ? void 0 : _c.name} attached as DATABASE_URL.`;
+                        msg += `with ${(_c = current === null || current === void 0 ? void 0 : current.addon) === null || _c === void 0 ? void 0 : _c.name} attached as DATABASE_URL.`;
                     }
                     msg += ' Check your release phase logs for failure causes.';
                     core_1.ux.action.stop(msg);

package/lib/commands/run/index.js

@@ -10,13 +10,14 @@ const debug = (0, debug_1.default)('heroku:run');
 class Run extends command_1.Command {
     async run() {
         const { argv, flags } = await this.parse(Run);
-        const userArgvInputOrder = (0, helpers_1.revertSortedArgs)(process.argv, argv);
+        const maybeOptionsIndex = process.argv.indexOf('--');
+        const command = (0, helpers_1.buildCommand)((maybeOptionsIndex === -1 ? argv : process.argv.slice(maybeOptionsIndex + 1)));
         const opts = {
             'exit-code': flags['exit-code'],
             'no-tty': flags['no-tty'],
             app: flags.app,
             attach: true,
-            command: (0, helpers_1.buildCommand)(userArgvInputOrder),
+            command,
             env: flags.env,
             heroku: this.heroku,
             listen: flags.listen,

package/lib/lib/run/helpers.d.ts

@@ -1,3 +1,2 @@
-export declare function revertSortedArgs(processArgs: Array<string>, argv: Array<string>): string[];
 export declare function buildCommand(args: Array<string>): string;
 export declare function buildEnvFromFlag(flag: string): {};

package/lib/lib/run/helpers.js

@@ -1,20 +1,8 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.buildEnvFromFlag = exports.buildCommand = exports.revertSortedArgs = void 0;
+exports.buildEnvFromFlag = exports.buildCommand = void 0;
 /* eslint-disable @typescript-eslint/ban-ts-comment */
 const core_1 = require("@oclif/core");
-// this function exists because oclif sorts argv
-function revertSortedArgs(processArgs, argv) {
-    const originalInputOrder = [];
-    // this reorders the arguments in the order the user inputted
-    for (const processArg of processArgs) {
-        if (argv.includes(processArg)) {
-            originalInputOrder.push(processArg);
-        }
-    }
-    return originalInputOrder;
-}
-exports.revertSortedArgs = revertSortedArgs;
 function buildCommand(args) {
     if (args.length === 1) {
         // do not add quotes around arguments if there is only one argument

package/oclif.manifest.json

@@ -13779,5 +13779,5 @@
       ]
     }
   },
-  "version": "9.2.0"
+  "version": "9.2.1-beta.0"
 }

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "heroku",
   "description": "CLI to interact with Heroku",
-  "version": "9.2.0",
+  "version": "9.2.1-beta.0",
   "author": "Heroku",
   "bin": "./bin/run",
   "bugs": "https://github.com/heroku/cli/issues",
@@ -388,5 +388,5 @@
     "version": "oclif readme --multi && git add README.md ../../docs"
   },
   "types": "lib/index.d.ts",
-  "gitHead": "2aa043ac05addfe888b817a5d5e4f175e41a22eb"
+  "gitHead": "32de98d0165a6772f480dff37fac720ec8525d97"
 }