Tracker: Set defaults for all options using a config file

[kumar303]

If a user has a magically named config file in the current working directory (e.g. web-ext-config.json) then web-ext should use this to set default option values (and web-ext should log a message about using it). Here are a few other examples of how it should work:

The config can also be specified explicitly:

web-ext --config=/path/to/web-ext-config.json build ...

The config file should apply to global option values as well as sub-command option values. Here is an example of a config that is the equivalent of web-ext --verbose sign --api-url-prefix https://addons-dev.allizom.org/api/v3:

{
  "verbose": true,
  "sign": {
    "api-url-prefix": "https://addons-dev.allizom.org/api/v3"
  }
}

The camel case variants of each option should also be supported, such as:

{
  "sign": {
    "apiUrlPrefix": "https://addons-dev.allizom.org/api/v3"
  }
}

Some additional rules:

• A command line option will take precedence over any config value
• An environment variable will take precedence over any config value
• A config can exist as ~/.web-ext-config.json as well as ./web-ext-config.json (these would override each other in the order listed here).

[kumar303]

I have a patch started but it depends on yargs/yargs#464

[kumar303]

FWIW, a yaml config might be nicer than JSON because you could put comments in it

[MarZab]

I would suggest making a .js file that exports a settings object instead.

How about using a grunt task ie. Gruntfile.js like we can with jpm? I feel we are needlessly reinventing the wheel here.

[kumar303]

Sure, a .js file would be nice because you can add comments. I think yargs might need a patch to support that but maybe not.

[MarZab]

Looks like something like this should work:

var argv = require('yargs')
  .config('settings', function (configPath) {
    return require(configPath)
  }).argv

web-ext.config.js

module.exports = {
  "sign": {
    "apiUrlPrefix": "https://addons-dev.allizom.org/api/v3"
  }
}

[kumar303]

oh cool, that looks like it will work. I like using .js config files, let's do it.

[pdehaan]

You could use something like rc to use JSON or INI config files.
Or I think ESLint supports config files as:

• .js (with module.exports)
• .json (with strip-json-comments to allow for commented JSON)
• .yml (using js-yaml)

I believe their specific config loader logic is in eslint/eslint /lib/config/config-file.js.

[kumar303]

It looks like rc will parse command line args and env vars but yargs already does that. It would be nice to just fix yargs/yargs#464 but I don't know how difficult that is.