Extract config processing.

[jamestalmage]

Pull out our option handling into a sub-module that can be used by some of the other tooling that we are building. It would also include some utility methods for determining if a given file was a test or helper or source, etc:

var Config = require('ava/config');

var config = new Config({
  // defaults
  args: process.argv.slice(2),
  cwd: process.cwd(),
  pkg: pkgConf.sync('ava', {cwd})
});

// config.options would simply be our merged options
config.options.serial // boolean
config.options.watch // boolean

// convenience methods
config.isTest('/some/path');
config.isSource('/some/path');
config.isHelper('/some/path');

// Maybe move some of the file watching here? I don't know that anything needs this yet. 
var watcher = config.watchChanges();

// events are not immediate (like with chokidar), but debounced like we already do in watcher.
watcher.on('source-changed', filename => {});
watcher.on('sources-changed', filenameArray => {});

watcher.on('test-changed', filename => {});
watcher.on('helper-changed', filename => {});

watcher.on('files-changed', obj => {
  obj.tests // a array
  obj.sources // a array
  obj.helpers // a array
});

[sindresorhus]

Is there anything that actually needs this? If so, I think we should limit the scope to what's actually needed, and drop anything that might-be-useful.

[jamestalmage]

we should limit the scope to what's actually needed

Yeah - drop the watcher stuff. The rest is definitely useful (I'm thinking ava-codemods and eslint-plugin-ava could both benefit).

[sindresorhus]

config.isTest('/some/path');
config.isSource('/some/path');
config.isHelper('/some/path');

This doesn't look config related though. More like util.

[jamestalmage]

This doesn't look config related though. More like util.

Except the answer depends on your config. Those are the methods we really need. Actually, I missed the important ones:

getTestFiles(), getSourceFiles(), getHelperFiles().

[sindresorhus]

👍 Alright

[novemberborn]

We should do this on a case-by-case basis, if and when it makes sense for an external module. No need to keep the issue open.