navy 7.0.0-alpha.1 → 7.0.0-alpha.2

package/lib/cli/__tests__/program.js

@@ -27,6 +27,7 @@ describe('cli/program', function () {
     actionsByCommand = {};
     helpHandlersByCommand = {};
     fakeProgram = {
+      option: sandbox.stub().returnsThis(),
       command(spec) {
         const name = spec.split(' ')[0];
         const cmd = {
@@ -150,6 +151,10 @@ describe('cli/program', function () {
       });
       (0, _chai.expect)(consoleLogStub.called).to.equal(true);
     });
+    it('should register a global -e, --navy option on the root program', function () {
+      loadModule();
+      (0, _chai.expect)(fakeProgram.option.calledWith('-e, --navy [env]', _sinon.default.match.string, 'dev')).to.equal(true);
+    });
   });
   describe('lazyRequire wrapped action', function () {
     it('should pass through arguments to the underlying module', async function () {
@@ -332,6 +337,18 @@ describe('cli/program', function () {
       (0, _chai.expect)(navyStub.start.firstCall.args[0]).to.eql(['web', 'api']);
       (0, _chai.expect)(navyStub.ensurePluginsLoaded.calledOnce).to.equal(true);
     });
+    it('should merge optsWithGlobals so global navy is used when Commander passes the command', async function () {
+      navyStub.start = sandbox.stub().resolves();
+      const fakeCommand = {
+        optsWithGlobals: () => ({
+          navy: 'from-global'
+        })
+      };
+      await actionsByCommand.start(['web'], {
+        navy: 'subcommand-default'
+      }, fakeCommand);
+      (0, _chai.expect)(getNavyStub.firstCall.args[0]).to.equal('from-global');
+    });
     it('should pass undefined when called with an empty service list', async function () {
       navyStub.start = sandbox.stub().resolves();
       await actionsByCommand.start([], {
@@ -436,6 +453,7 @@ describe('cli/program', function () {
   function createCapturingProgram() {
     capturedActions = {};
     return {
+      option: sandbox.stub().returnsThis(),
       command(spec) {
         const name = spec.split(' ')[0];
         const cmd = {

package/lib/cli/program.js

@@ -11,6 +11,7 @@ var _errors = require("../errors");
 var _config = require("../config");
 var _driverLogging = require("../driver-logging");
 var _configProvider = require("../config-provider");
+var _mergeActionOptions = require("./util/merge-action-options");
 const loadingLabelMap = {
   destroy: 'Destroying services...',
   start: 'Starting services...',
@@ -28,6 +29,23 @@ const loadingLabelMap = {
 function removeFirstLineFromStackTrace(stack) {
   return stack?.split('\n')?.slice(1)?.join('\n');
 }
+function normaliseLazyRequireArgs(args) {
+  if (args.length === 0) return args;
+  const last = args[args.length - 1];
+  if (!last || typeof last.optsWithGlobals !== 'function') {
+    return args;
+  }
+  const command = last;
+  const rest = args.slice(0, -1);
+  if (rest.length === 0) {
+    return rest;
+  }
+  const lastOpt = rest[rest.length - 1];
+  if (lastOpt && typeof lastOpt === 'object' && !Array.isArray(lastOpt)) {
+    return [...rest.slice(0, -1), (0, _mergeActionOptions.mergeActionOptions)(lastOpt, command)];
+  }
+  return rest;
+}
 function wrapper(res) {
   if (res.catch) {
     res.catch(ex => {
@@ -62,11 +80,21 @@ function basicCliWrapper(fnName, wrapperOpts = {}) {
     } = require('../navy');
 
     // commander v12 invokes action handlers with a trailing Command instance
-    // appended after the parsed options. Strip it so the rest of this wrapper
-    // can continue to treat the last remaining `args` element as the options.
-    if (args.length > 0) args = args.slice(0, -1);
-    const opts = args.length === 0 ? maybeServices : args[args.length - 1];
+    // appended after the parsed options. Strip it and merge global options
+    // (e.g. `navy -e dev start`) into the opts object.
+    let command = null;
+    if (args.length > 0) {
+      const last = args[args.length - 1];
+      if (last && typeof last.optsWithGlobals === 'function') {
+        command = last;
+        args = args.slice(0, -1);
+      }
+    }
+    let opts = args.length === 0 ? maybeServices : args[args.length - 1];
     const otherArgs = args.slice(0, args.length - 1);
+    if (command && opts && typeof opts === 'object' && !Array.isArray(opts)) {
+      opts = (0, _mergeActionOptions.mergeActionOptions)(opts, command);
+    }
     const envName = opts.navy;
     if (wrapperOpts.serviceBasedAlias && maybeServices.length) {
       console.log(`This command should not be called with a list of services. calling '${wrapperOpts.serviceBasedAlias}' instead`);
@@ -102,10 +130,12 @@ function basicCliWrapper(fnName, wrapperOpts = {}) {
 function lazyRequire(path) {
   return function (...args) {
     const mod = require(path);
-    return wrapper((mod.default || mod)(...args));
+    const normalised = normaliseLazyRequireArgs(args);
+    return wrapper((mod.default || mod)(...normalised));
   };
 }
 const defaultNavy = process.env.NAVY_NAME || (0, _config.getConfig)().defaultNavy;
+_commander.program.option('-e, --navy [env]', `set the navy name to be used [${defaultNavy}]`, defaultNavy);
 const importCommand = _commander.program.command('import').option('-e, --navy [env]', `set the navy name to be used [${defaultNavy}]`, defaultNavy).description('Imports docker compose configuration from the current working directory and initialises a new navy').action(lazyRequire('./import'));
 (0, _configProvider.getImportCommandLineOptions)().forEach(opt => importCommand.option(...opt));
 _commander.program.command('launch [services...]').option('-e, --navy [env]', `set the navy name to be used [${defaultNavy}]`, defaultNavy).description('Launches the given services in a navy').action(lazyRequire('./launch')).on('--help', () => console.log(`

package/lib/cli/util/__tests__/merge-action-options.js

@@ -0,0 +1,47 @@
+"use strict";
+
+var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault");
+var _chai = require("chai");
+var _sinon = _interopRequireDefault(require("sinon"));
+var _mergeActionOptions = require("../merge-action-options");
+/* eslint-env mocha */
+
+describe('cli/util/merge-action-options', function () {
+  it('should return parsed opts when command is missing', function () {
+    const opts = {
+      navy: 'dev'
+    };
+    (0, _chai.expect)((0, _mergeActionOptions.mergeActionOptions)(opts, null)).to.equal(opts);
+  });
+  it('should return parsed opts when optsWithGlobals is missing', function () {
+    const opts = {
+      navy: 'dev'
+    };
+    (0, _chai.expect)((0, _mergeActionOptions.mergeActionOptions)(opts, {})).to.equal(opts);
+  });
+  it('should merge optsWithGlobals so global navy wins over subcommand defaults', function () {
+    const command = {
+      optsWithGlobals: () => ({
+        navy: 'dev',
+        json: true
+      })
+    };
+    const merged = (0, _mergeActionOptions.mergeActionOptions)({
+      navy: 'default',
+      json: false
+    }, command);
+    (0, _chai.expect)(merged.navy).to.equal('dev');
+    (0, _chai.expect)(merged.json).to.equal(true);
+  });
+  it('should call optsWithGlobals on the command', function () {
+    const spy = _sinon.default.stub().returns({
+      navy: 'x'
+    });
+    (0, _mergeActionOptions.mergeActionOptions)({
+      navy: 'y'
+    }, {
+      optsWithGlobals: spy
+    });
+    (0, _chai.expect)(spy.calledOnce).to.equal(true);
+  });
+});

package/lib/cli/util/merge-action-options.js

@@ -0,0 +1,20 @@
+"use strict";
+
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+exports.mergeActionOptions = mergeActionOptions;
+/**
+ * Merge Commander's per-command opts with inherited globals (e.g. `navy -e dev ps`).
+ * `optsWithGlobals()` is the combined view; spread it after local opts so globals
+ * override stale option defaults on the subcommand.
+ */
+function mergeActionOptions(parsedOpts, command) {
+  if (!command || typeof command.optsWithGlobals !== 'function') {
+    return parsedOpts;
+  }
+  return {
+    ...parsedOpts,
+    ...command.optsWithGlobals()
+  };
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "navy",
-  "version": "7.0.0-alpha.1",
+  "version": "7.0.0-alpha.2",
   "description": "Quick and powerful development environments using Docker and Docker Compose",
   "main": "lib/index.js",
   "types": "lib/index.d.ts",
@@ -49,6 +49,5 @@
     "resolve": "^1.20.0",
     "strip-ansi": "^6.0.1",
     "www-authenticate": "^0.6.3"
-  },
-  "gitHead": "c977d36ee1e962134aee34d5fa20071cd863a619"
+  }
 }