esm: fix misleading error when import empty package.json

[himself65]

Fixes: #49674

---

➜  node  /Users/himself65/Code/node/out/Debug/node -e 'import("whatever")'

node:internal/process/promises:394
    triggerUncaughtException(err, true /* fromPromise */);
    ^
Error: No package entry point defined for package 'file:///Users/himself65/Code/xxx/node_modules/whatever/package.json' imported from xxx/src/index.ts
    at legacyMainResolve (node:internal/modules/esm/resolve:207:26)
    at packageResolve (node:internal/modules/esm/resolve:826:14)
    at moduleResolve (node:internal/modules/esm/resolve:908:18)
    at defaultResolve (node:internal/modules/esm/resolve:1038:11)
    at nextResolve (node:internal/modules/esm/hooks:748:28)
    at resolve (file:///Users/himself65/Applications/WebStorm.app/Contents/plugins/nodeJS/js/ts-file-loader/node_modules/tsx/dist/esm/index.mjs?1726363031016:2:3893)
    at nextResolve (node:internal/modules/esm/hooks:748:28)
    at Hooks.resolve (node:internal/modules/esm/hooks:240:30)
    at handleMessage (node:internal/modules/esm/worker:199:24)
    at Immediate.checkForMessages (node:internal/modules/esm/worker:141:28) {
  code: 'ERR_MODULE_NOT_FOUND'
}

Node.js v23.0.0-pre

Process finished with exit code 1

[nodejs-github-bot]

Review requested:

• @nodejs/loaders

[GeoffreyBooth]

The problem isn't really that main isn't a string; it's that the package.json is invalid. We also discourage the use of main nowadays since exports replaces it.

[himself65]

The problem isn't really that main isn't a string; it's that the package.json is invalid. We also discourage the use of main nowadays since exports replaces it.

The question is how to determine if a package.json is correct? Do we have such schema

[himself65]

Updated

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/54075/

[RedYetiDev]

@himself65 are you still interested in continueing this? There are outstanding comments, and failing tests.

[RedYetiDev]

Oops! Sorry... closed the wrong PR tab 😅

[himself65]

@himself65 are you still interested in continueing this? There are outstanding comments, and failing tests.

Let me fix this

[himself65]

working on it

[himself65]

fixed!

[himself65]

idk why format-cpp action will delete half of the file. any idea? @RedYetiDev

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.04%. Comparing base (6dfa3e4) to head (ce27df1).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #49728      +/-   ##
==========================================
+ Coverage   88.03%   88.04%   +0.01%     
==========================================
  Files         652      652              
  Lines      183765   183764       -1     
  Branches    35858    35856       -2     
==========================================
+ Hits       161771   161791      +20     
+ Misses      15239    15227      -12     
+ Partials     6755     6746       -9     

Files with missing lines	Coverage Δ
src/node_file.cc	77.06% <100.00%> (+<0.01%)	⬆️

... and 26 files with indirect coverage changes

[RedYetiDev]

idk why format-cpp action will delete half of the file. any idea? @RedYetiDev

Rebase with the latest changes to main to resolve the issue. #54994

[aduh95]

There are some relevant test failures, PTAL

[aduh95]

IIUC, there is an entry point defined, './index.node', but the resolution still fails because the file doesn't exist. This change would likely be confusing.

[himself65]

yeah make sense, so I'm gonna check wheher user request a file or a module then give a differnt error