arethetypeswrong / arethetypeswrong/arethetypeswrong.github.io

`require(esm)` support

Open
#252 3 comments 7 reactions 0 assignees View on GitHub
Dominant language
TypeScript
Stars
1.6k
Forks
65
PR merge metrics
No merged PRs in 30d

Description

i may be missing something, but it seems we don't support `require(esm)`.

this messaging suggests it is an error:

https://github.com/arethetypeswrong/arethetypeswrong.github.io/blob/161725ad2e8957f109e44fb26b13c9d70f415c2f/docs/problems/CJSResolvesToESM.md?plain=1#L3

but in node 20 and above, it is no longer an error and should work fine

not sure how we should determine this in ATTW, though. since we'd need to know what node versions you're targeting?

Contributor guide

No contributing guide indexed for this repository

Research direction

Start with the linked docs/problems/CJSResolvesToESM.md guidance and compare its claim with Node 20 and later behavior. Determine which Node versions ATTW targets before deciding how require(esm) should be classified; done means the analysis and messaging match those supported targets.

Written by the indexing model from the issue text.

Assessment

Tech stack
node.js, typescript
Domain
tooling
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.