feat(ts): ban cjs exports in ts file

[zanminkian]

Description

Ban cjs exports in ts file.

if (true)
  module.exports = {}

module.exports = {}

if (true)
  exports.a = 1

exports.a = 1

function f() {
  exports.a = 1
}

const a = {
  exports: {
    tmp: 'hi',
  },
}

a.exports.tmp = 'hello'

export = 'abc'

export default 'hello'

export const tmp = 'world'

Copy this content to test.

Linked Issues

Additional context

[zanminkian]

It's ready to be reviewed.

[antfu]

I am not sure if I'd like to take that complicity. Ideally, I would prefer to have it defined in a plugin and enable as a rule.

[zanminkian]

I would prefer to have it defined in a plugin and enable as a rule

It's better.

I don't have to much experience on eslint plugin. I will look into it on weekends.

[zanminkian]

Hi @antfu. Refactored to plugin rules. Please take a review.

[zanminkian]

It's ready.

[zanminkian]

Hi @antfu , can you review this PR? After merging this PR, this eslint-config preset will force user to use ESM in ts file. This is the last step, because require statement has already been blocked in ts file.

Let's make ESM greater.