Hey thanks for the PR! Looks like that broke a few things. Give that a test run locally.

Hmmm tests are failing because I'm doing an export default and a named export now.
But that shouldn't fail, right? 🤔

@shellscape Any idea on how to make this a non-breaking change? 🤔

We might need some wrapper for the common JS output.

Before:

module.exports = nodeResolve(...)

After:

exports.default = nodeResolve(...)
exports.defaults = defaults

Ideal:

mod = nodeResolve(...)
mod.defaults = defaults
module.exports = mod

Also, since export default and export defaults are quite similar, I think defaultOptions or DEFAULT_OPTIONS would be better names.

@NotWoods Wrapping the module still fails tests

@MichaelDeBoey please merge from master again. we fixed up some CI related things today.

@shellscape Rebased, but CI is still failing tho 😕

Bummer. Next step is to run tests locally to triage from there: cd packages/node-resolve && pnpm run test

@shellscape It's still the same error

 TypeError {
    message: 'nodeResolve is not a function',
  }

This is because the plugin now exports both a default and a named export.
Wrapping it like @NotWoods suggested isn't working either.

So I don't think we can do this without a breaking change/major release. 🤔

The wrapper should be extracted to a separate file, and used as the entry point for only the CJS build. The presence of an export const and export default statement is throwing off Rollup's CJS build.

@NotWoods Could you point me in the right direction on how I could best fix this? 🤔

This behavior might be better inside an output plugin, but here's something I whipped together:

// cjs-wrapper.js
import nodeResolve, { DEFAULT_OPTIONS } from './index';

nodeResolve.DEFAULT_OPTIONS = DEFAULT_OPTIONS;

export default nodeResolve;

// rollup.config.js
import babel from 'rollup-plugin-babel';
import json from '@rollup/plugin-json';

import pkg from './package.json';

const plugins = [
    json(),
    babel({
      presets: [
        [
          '@babel/preset-env',
          {
            targets: {
              node: 6
            }
          }
        ]
      ]
    })
];
const external = Object.keys(pkg.dependencies).concat(['fs', 'path', 'os', 'util']);

export default [
	{
		input: 'src/index.js',
		plugins,
		external,
		output: [
    		{ file: pkg.module, format: 'es' }
  		]
	},
	{
		input: 'src/cjs-wrapper.js',
		plugins,
		external,
		output: [
    		{ file: pkg.main, format: 'cjs' }
  		]
	},
];

@shellscape @NotWoods All is green now 🙂

@shellscape @NotWoods All is green again now 🙂

@MichaelDeBoey I'll fix this conflict when I get the chance to circle back.

@shellscape Rebase onto upstream/master

Thanks for all the hard work that went into this one.

With pleasure! 🙂