#240: fixed regression causing empty tooling options to throw an error

CHANGELOG.md

@@ -5,6 +5,7 @@
 * Fixed bug causing https proxy routes to be assigned when they shouldnt be
 * Fixed bug causing cache from repopulating old proxy addresses that have been removed [#209](https://github.com/lando/core/issues/209)
 * Fixed bug in `v4` auto `entrypoint` and `command` population
+* Fixed regression causing empty tooling `options` to throw an error [#240](https://github.com/lando/core/issues/240)
 
 ## v3.23.0-beta.4 - [October 22, 2024](https://github.com/lando/core/releases/tag/v3.23.0-beta.4)
 

examples/tooling/.lando.yml

@@ -201,6 +201,10 @@ tooling:
     cmd: pwd
     service: web
     dir: /tmp
+  emptyopter:
+    cmd: pwd
+    service: web
+    options:
   pwd:
     cmd: pwd
     service: :service

examples/tooling/README.md

@@ -158,6 +158,9 @@ lando everything --help | grep "lando everything \[arg1\] \[arg2\] MORETHINGS"
 
 # Should allow for example pasthru in task definition
 lando everything --help | grep "lando this is just for testing"
+
+# Should be able to run even if options are empty
+lando emptyopter
 ```
 
 ## Destroy tests

lib/cli.js

@@ -460,6 +460,9 @@ module.exports = class Cli {
 
     // Return our yarg command
     return {command, describe, handler, builder: yargs => {
+      // ensure options is an object if it isnt
+      if (!_.isPlainObject(options)) options = {};
+
       // if we have options then let is sort and add them
       if (Object.keys(options).length > 0) {
         for (const [option, config] of Object.entries(formatters.sortOptions(options))) {