Plugin not replacing all uses of global

[mohd-akram]

I've noticed some instances of global are not replaced for some reason. I'm not sure what specifically is causing this but it happens with the node-fetch (2.3.0) library, for example:

rollup.config.js:

import globals from 'rollup-plugin-node-globals';
import resolve from 'rollup-plugin-node-resolve';

export default {
  output: {
    format: 'iife'
  },
  plugins: [
    resolve(),
    globals()
  ]
}

input.js:

import * as fetch from 'node-fetch';
console.log(fetch);

To test:

rollup -c -i input.js --silent | grep global

The output contains two instances of global:

Body.Promise = global.Promise;
fetch.Promise = global.Promise;

[mohd-akram]

Found the issue - it's not checking .mjs files. This patch fixes it:

-if (id.slice(-3) !== '.js') return null;
+if (!['.js', '.mjs'].includes(path.extname(id))) return null;