external files are being bundled for commonjs

[dnalborczyk]

• Rollup Plugin Name:
• Rollup Plugin Version: @rollup/plugin-commonjs 21.0.1 (possibly also @rollup/json ^4.1.0
• Rollup Version:
• Operating System (or Browser):
• Node Version:
• Link to reproduction (⚠️ read below):

Expected Behavior

foo.js

const js = require("./foo.js");

console.log(js);

foo.js

export default "foo";

Actual Behavior

import './foo.js';

var rollupCreateRequireTest = {};

var require$$0 = "foo";

const foo = require$$0;

console.log(foo);

export { rollupCreateRequireTest as default };

Additional Information

works for esm.

[shellscape]

Thanks for opening an issue. Citing the issue template:

🚨 Issues WITHOUT a valid reproduction WILL BE CLOSED!

Please provide one by:

1. Using the REPL.it plugin reproduction template at https://repl.it/@rollup/rollup-plugin-repro
2. Provide a minimal repository link (Read https://git.io/fNzHA for instructions).
   Please use NPM for installing dependencies!
   These may take more time to triage than the other options.
3. Using the Rollup REPL at https://rollupjs.org/repl/

⚠️ ZIP Files are unsafe and maintainers will NOT download them.

We cannot make this any clearer. Please add a reproduction and we'll be happy to triage further.

[dnalborczyk]

@shellscape I'm sorry, but I believe you are alienating people with auto closing issues. this is a bug which does not affect me, I noticed it while trying some things out. I thought I'd be filing it anyways as it appears to be a bug.

I understand your frustration in that regard, but I don't believe this fixes it, I think what you accomplish is rather the contrary: people will not bother filing issues.

[shellscape]

Thanks for your input. The issue policy here has been very effective.