Rule proposal: no-dynamic-require

[axetroy]

Description

The input to require should be a static string literal.

This is beneficial to various Bundles and Analyzer. And less difficult to migrate to es module.

The dynamic require should be prohibited.

Fail

require(pathToTargetModule)
require('./icons/' + iconType + '.svg')

Pass

require('./path/to/module')
require('./icons/pass.svg')

Proposed rule name

no-dynamic-require

Additional Info

Of course, in our project, there will indeed be such a situation.

require(pathToTargetModule)

This is a typical scenario that needs to be ignored.

exports[`no-dynamic-require 1`] = `
## invalid(1): require(foo)

  > Input
  
        1 | require(foo)

  > Error 1/1: Unexpected dynamic require.
  
      > 1 | require(foo)
          | ^^^^^^^^^^^^

## invalid(2): require("foo" + "bar")

  > Input
  
        1 | require("foo" + "bar")

  > Error 1/1: Unexpected dynamic require.
  
      > 1 | require("foo" + "bar")
          | ^^^^^^^^^^^^^^^^^^^^^^

## invalid(3): require(\`\${foo}/bar\`)

  > Input
  
        1 | require(\`\${foo}/bar\`)

  > Error 1/1: Unexpected dynamic require.
  
      > 1 | require(\`\${foo}/bar\`)
          | ^^^^^^^^^^^^^^^^^^^^^
`;

[sindresorhus]

We are not adding any new CommonJS-related rules here.

And it seems more appropriate for https://github.com/un-ts/eslint-plugin-import-x anyway