refactor: replace cosmiconfig with lilconfig

[TrySound]

What:

cosmiconfig is a massive package which recently got a small alternative
without dependencies lilconfig.

https://github.com/antonk52/lilconfig

https://packagephobia.com/result?p=cosmiconfig
https://packagephobia.com/result?p=lilconfig

Though still a big part of cosmiconfig is yaml package.
https://packagephobia.com/result?p=yaml

This package is important piece of relay and emotion. Would be good to make it as small as possible.

Why:

Node modules are bloated with large dependencies. I'm trying to help with this.

How:

Just switch to alternative.

Checklist:

• Documentation
• Tests
• Ready to be merged
  

[TrySound]

cc @kentcdodds

[kentcdodds]

Thanks for this! Is this a breaking change?

[TrySound]

Shouldn't be. lilconfig is mostly compatible. Yaml loaders are covered.
Not sure why tests are failing. Looks like something unrelated to change.

[kentcdodds]

Bummer, looks like this project is using old versions of deps and something broke. I'm afraid I don't have any time to get things updated in this project and get things fixed so we can get this merged :(

[TrySound]

If you want I can take care of this on weekend.

[conartist6]

It looks nonbreaking since it still requires yaml and defines a yaml loader. I really wouldn't mind seeing yaml config support dropped.

Would it work to just have yaml as an optional peer dependency?

[kentcdodds]

Whatever you two want to do. I don't use babel anymore.

[TrySound]

I don't personally use macros support but care about node_modules size so it's enough motivation. Will look tomorrow.

[conartist6]

From what you shared just changing to lilconfig and yaml would save ~275kb in node_modules (751 -> 466KB of deps for config loading). It's a moderate win, but not nearly as big as also eliminating the useless yaml dep, at which point we'd be down to 27kb of config loading which finally sounds like a number on the correct order of magnitude.

[TrySound]

I fixed build here. Though not sure as ci run need to be approved.
#184

[conartist6]

You'll also need to change

babel-plugin-macros/src/__tests__/index.js

Lines 10 to 18 in d613985

	jest.mock('cosmiconfig', () => {
	const cosmiconfigExports = jest.requireActual('cosmiconfig')
	const actualCosmiconfigSync = cosmiconfigExports.cosmiconfigSync
	function fakeCosmiconfigSync(...args) {
	fakeCosmiconfigSync.explorer = actualCosmiconfigSync(...args)
	return fakeCosmiconfigSync.explorer
	}
	return {...cosmiconfigExports, cosmiconfigSync: fakeCosmiconfigSync}
	})

What the heck is that for I wonder!?

[buaiscia]

there is just a recent security vulnerability on yaml <2.2.2, is it possible to take an upgraded version? Thanks
https://security.snyk.io/vuln/SNYK-JS-YAML-5458867