warnOnceIfEsmOnlyPackage erroneously warned for marked

[jamesarosen]

What version of Remix are you using?

1.6.5

Steps to Reproduce

Add marked to dependencies, run remix run.

Expected Behavior

No warning message, since marked publishes CJS and ESM in the same package:

"main": "./lib/marked.cjs",
"module": "./lib/marked.esm.js",
"browser": "./lib/marked.umd.js",

markedjs/marked#2281 has some relevant history.

Actual Behavior

remix dev emits

marked is possibly an ESM only package and should be bundled with "serverDependenciesToBundle in remix.config.js.

[machour]

@jamesarosen could you try with the latest Remix version?

[jamesarosen]

Good call. I tested on 1.7.2 and the issue remains:

$ npm list @remix-run/dev
co2-frontend@ /Users/jamesrosen/Code/fe
└── @remix-run/[email protected]

$ npm run dev

> dev
> remix dev

Loading environment variables from .env
Watching Remix app in test mode...
marked is possibly an ESM only package and should be bundled with "serverDependenciesToBundle in remix.config.js.

[pcattori]

Fixed by #6916