[Wrong] ESM module type does not support fields from `common-js` #7963

patricklafrance · 2023-09-17T02:47:15Z

Describe the bug

With @swc/core v1.3.85 I get the following error:

Error: unknown field `strict`, expected `resolveFully`.

If lower @swc/core version back to 1.3.84, the error goes away.

Full stack:

sample/another-remote-module dev: built modules 178 KiB [built]
sample/another-remote-module dev:   modules by path ../../node_modules/.pnpm/[email protected][email protected][email protected]...(truncated) 71.8 KiB 16 modules
sample/another-remote-module dev:   modules by path ../../node_modules/.pnpm/[email protected]_@[email protected][email protected]/node_...(truncated) 5.3 KiB
sample/another-remote-module dev:     ../../node_modules/.pnpm/[email protected]_@[email protected][email protected]/node_...(truncated) 1.94 KiB [built] [code generated]
sample/another-remote-module dev:     ../../node_modules/.pnpm/[email protected]_@[email protected][email protected]/node_...(truncated) 1.86 KiB [built] [code generated]
sample/another-remote-module dev:     + 2 modules
sample/another-remote-module dev:   modules by path ../../node_modules/.pnpm/[email protected]/node_modules/html-entities/lib/*.js 81.8 KiB
sample/another-remote-module dev:     ../../node_modules/.pnpm/[email protected]/node_modules/html-entitie...(truncated) 7.91 KiB [built] [code generated]
sample/another-remote-module dev:     ../../node_modules/.pnpm/[email protected]/node_modules/html-entities/lib/name...(truncated) 73 KiB [built] [code generated]
sample/another-remote-module dev:     + 2 modules
sample/another-remote-module dev:   container entry 42 bytes [built] [code generated]
sample/another-remote-module dev:   ./src/register.tsx 39 bytes [built] [code generated] [1 error]
sample/another-remote-module dev:   ../../node_modules/.pnpm/[email protected]/node_modules/ansi-html-comm...(truncated) 4.16 KiB [built] [code generated]
sample/another-remote-module dev:   ../../node_modules/.pnpm/[email protected]/node_modules/events/events.js 14.5 KiB [built] [code generated]
sample/another-remote-module dev: ERROR in ./src/register.tsx
sample/another-remote-module dev: Module build failed (from ../../node_modules/.pnpm/[email protected]_@[email protected][email protected]/node_modules/swc-loader/src/index.js):
sample/another-remote-module dev: Error: unknown field `strict`, expected `resolveFully` at line 1 column 575
sample/another-remote-module dev: webpack 5.88.2 compiled with 1 error in 585 ms
sample/host dev: asset main.js 579 KiB [emitted] (name: main)
sample/host dev: asset index.html 217 bytes [emitted]
sample/host dev: runtime modules 28.1 KiB 13 modules
sample/host dev: modules by path ../../node_modules/.pnpm/ 178 KiB
sample/host dev:   modules by path ../../node_modules/.pnpm/[email protected][email protected][email protected]...(truncated) 71.8 KiB 16 modules
sample/host dev:   modules by path ../../node_modules/.pnpm/[email protected]_@[email protected][email protected]/node_...(truncated) 5.3 KiB
sample/host dev:     ../../node_modules/.pnpm/[email protected]_@[email protected][email protected]/node_...(truncated) 1.94 KiB [built] [code generated]
sample/host dev:     ../../node_modules/.pnpm/[email protected]_@[email protected][email protected]/node_...(truncated) 1.86 KiB [built] [code generated]
sample/host dev:     + 2 modules
sample/host dev:   modules by path ../../node_modules/.pnpm/[email protected]/node_modules/html-entities/lib/*.js 81.8 KiB
sample/host dev:     ../../node_modules/.pnpm/[email protected]/node_modules/html-entitie...(truncated) 7.91 KiB [built] [code generated]
sample/host dev:     ../../node_modules/.pnpm/[email protected]/node_modules/html-entities/lib/name...(truncated) 73 KiB [built] [code generated]
sample/host dev:     ../../node_modules/.pnpm/[email protected]/node_modules/html-entities/lib/nume...(truncated) 339 bytes [built] [code generated]
sample/host dev:     ../../node_modules/.pnpm/[email protected]/node_modules/html-entities/lib/surr...(truncated) 537 bytes [built] [code generated]
sample/host dev:   ../../node_modules/.pnpm/[email protected]/node_modules/ansi-html-comm...(truncated) 4.16 KiB [built] [code generated]
sample/host dev:   ../../node_modules/.pnpm/[email protected]/node_modules/events/events.js 14.5 KiB [built] [code generated]
sample/host dev: ./src/index.ts 39 bytes [built] [code generated] [1 error]
sample/host dev: ERROR in ./src/index.ts
sample/host dev: Module build failed (from ../../node_modules/.pnpm/[email protected]_@[email protected][email protected]/node_modules/swc-loader/src/index.js):
sample/host dev: Error: unknown field `strict`, expected `resolveFully` at line 1 column 533
sample/host dev: webpack 5.88.2 compiled with 1 error in 733 ms
sample/remote-module dev: asset remoteEntry.js 585 KiB [emitted] (name: remote1)
sample/remote-module dev: asset main.js 580 KiB [emitted] (name: main)
sample/remote-module dev: asset src_register_tsx.js 1.04 KiB [emitted]
sample/remote-module dev: runtime modules 59.3 KiB 28 modules
sample/remote-module dev: built modules 178 KiB [built]
sample/remote-module dev:   modules by path ../../node_modules/.pnpm/[email protected][email protected][email protected]...(truncated) 71.8 KiB 16 modules
sample/remote-module dev:   modules by path ../../node_modules/.pnpm/[email protected]_@[email protected][email protected]/node_...(truncated) 5.3 KiB
sample/remote-module dev:     ../../node_modules/.pnpm/[email protected]_@[email protected][email protected]/node_...(truncated) 1.94 KiB [built] [code generated]
sample/remote-module dev:     ../../node_modules/.pnpm/[email protected]_@[email protected][email protected]/node_...(truncated) 1.86 KiB [built] [code generated]
sample/remote-module dev:     + 2 modules
sample/remote-module dev:   modules by path ../../node_modules/.pnpm/[email protected]/node_modules/html-entities/lib/*.js 81.8 KiB
sample/remote-module dev:     ../../node_modules/.pnpm/[email protected]/node_modules/html-entitie...(truncated) 7.91 KiB [built] [code generated]
sample/remote-module dev:     ../../node_modules/.pnpm/[email protected]/node_modules/html-entities/lib/name...(truncated) 73 KiB [built] [code generated]
sample/remote-module dev:     + 2 modules
sample/remote-module dev:   container entry 42 bytes [built] [code generated]
sample/remote-module dev:   ./src/register.tsx 39 bytes [built] [code generated] [1 error]
sample/remote-module dev:   ../../node_modules/.pnpm/[email protected]/node_modules/ansi-html-comm...(truncated) 4.16 KiB [built] [code generated]
sample/remote-module dev:   ../../node_modules/.pnpm/[email protected]/node_modules/events/events.js 14.5 KiB [built] [code generated]
sample/remote-module dev: ERROR in ./src/register.tsx
sample/remote-module dev: Module build failed (from ../../node_modules/.pnpm/[email protected]_@[email protected][email protected]/node_modules/swc-loader/src/index.js):
sample/remote-module dev: Error: unknown field `strict`, expected `resolveFully` at line 1 column 559
sample/remote-module dev: webpack 5.88.2 compiled with 1 error in 678 ms
sample/remote-module dev: <i> [webpack-dev-server] Gracefully shutting down. To force exit, press ^C again. Please wait...
sample/host dev: <i> [webpack-dev-server] Gracefully shutting down. To force exit, press ^C again. Please wait...
sample/another-remote-module dev: <i> [webpack-dev-server] Gracefully shutting down. To force exit, press ^C again. Please wait...
sample/another-remote-module dev: Failed

Input code

The project for which the error is happening is available here: https://github.com/gsoft-inc/wl-squide.

Bump the `@swc/core` version to `1.3.85` for the following projects:

- `sample/another-remote-module`
- `sample/host`
- `sample/remote-module`

Then:

- `pnpm install`
- `pnpm dev-sample`

The error should print in the console.

Config

No response

Playground link

No response

SWC Info output

Operating System:
        Platform: win32
        Arch: x64
        Machine Type: x86_64
        Version: Windows 10 Enterprise
        CPU: (12 cores)
            Models: Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz

    Binaries:
        Node: 20.6.1
        npm: N/A
        Yarn: N/A
        pnpm: 8.7.4

    Relevant Packages:
        @swc/core: N/A
        @swc/helpers: N/A
        @swc/types: N/A
        typescript: 5.2.2

    SWC Config:
        output: N/A
        .swcrc path: N/A

    Next.js info:
        output: N/A

Expected behavior

No error.

Actual behavior

No response

Version

1.3.85

Additional context

The code is transpiled with the following stack: webpack 5 -> swc-loader -> swc/core

The text was updated successfully, but these errors were encountered:

kdy1 · 2023-09-17T03:50:26Z

strict is not a valid option for ESM, as strict does not have any meaning.
The only option valid for ESM is resolveFull: pool

kdy1 · 2023-09-17T03:55:01Z

https://github.com/swc-project/swc/blob/e7493283120721499751f83d3753f5a3b523a1a3/crates/swc_ecma_transforms_module/src/lib.rs#L24C2-L27

#[serde(default)]
    pub allow_top_level_this: bool,
    #[serde(default)]
    pub strict: bool,
    #[serde(default = "default_strict_mode")]
    pub strict_mode: bool,
    #[serde(default)]
    pub lazy: Lazy,
    #[serde(default)]
    pub import_interop: Option<ImportInterop>,
    /// Emits `cjs-module-lexer` annotation
    /// `cjs-module-lexer` is used in Node.js core for detecting the named
    /// exports available when importing a CJS module into ESM.
    /// swc will emit `cjs-module-lexer` detectable annotation with this option
    /// enabled.
    ///
    /// Defaults to `true` if import_interop is Node, else `false`
    #[serde(default)]
    pub export_interop_annotation: Option<bool>,
    #[serde(default)]
    /// Note: deprecated
    pub no_interop: bool,
    #[serde(default)]
    pub ignore_dynamic: bool,
    #[serde(default)]
    pub preserve_import_meta: bool,

    #[serde(default)]
    pub resolve_fully: bool,

patricklafrance · 2023-09-17T03:57:57Z

Thank you @kdy1 🙏🏻

What exactly does resolveFully do? I can't find anything on the documentation website.

marviobezerra · 2023-09-19T19:21:08Z

I'm also getting this same error with version 1.3.86. What is resolveFully and why do I need it?

kdy1 · 2023-09-19T19:44:09Z

You don't need resolveFully

kdy1 · 2023-09-19T19:44:37Z

Remove wrong fields.

marviobezerra · 2023-09-19T20:02:00Z

I don't have this field and it stills complaining about it. Here my .swcrc content and it works fine with 1.3.80.

{
  "sourceMaps": true,
  "module": {
    "type": "nodenext",
    "strict": true,
    "strictMode": true,
    "noInterop": false
  },
  "jsc": {
    "externalHelpers": true,
    "target": "es2022",
    "parser": {
      "syntax": "typescript",
      "tsx": false,
      "decorators": true,
      "dynamicImport": true
    },
    "transform": {
      "legacyDecorator": true,
      "decoratorMetadata": true
    },
    "keepClassNames": true
  }
}

swc-bot · 2023-10-20T00:21:12Z

This closed issue has been automatically locked because it had no new activity for a month. If you are running into a similar issue, please create a new issue with the steps to reproduce. Thank you.

patricklafrance added the C-bug label Sep 17, 2023

kdy1 closed this as completed Sep 17, 2023

kdy1 changed the title ~~With @swc/core v1.3.85 I get the following error: Error: unknown field strict, expected resolveFully~~ [Wrong] ESM module type does not support fields from common-js Sep 17, 2023

This was referenced Sep 21, 2023

[Wrong] Es6/nodenext module only has a field named resolveFully and you should remove other fields. #7979

Closed

[WRONG] jsc.experimental.keepImportAssertions is experimental, explicitly #7923

Closed

meabed mentioned this issue Sep 24, 2023

SWC: unknown field noInterop, expected resolveFully TypeStrong/ts-node#2070

Open

swc-project locked as resolved and limited conversation to collaborators Oct 20, 2023

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Wrong] ESM module type does not support fields from `common-js` #7963

[Wrong] ESM module type does not support fields from `common-js` #7963

patricklafrance commented Sep 17, 2023 •

edited

Loading

kdy1 commented Sep 17, 2023

kdy1 commented Sep 17, 2023 •

edited

Loading

patricklafrance commented Sep 17, 2023

marviobezerra commented Sep 19, 2023

kdy1 commented Sep 19, 2023

kdy1 commented Sep 19, 2023

marviobezerra commented Sep 19, 2023 •

edited

Loading

swc-bot commented Oct 20, 2023

[Wrong] ESM module type does not support fields from common-js #7963

[Wrong] ESM module type does not support fields from common-js #7963

Comments

patricklafrance commented Sep 17, 2023 • edited Loading

Describe the bug

Input code

Config

Playground link

SWC Info output

Expected behavior

Actual behavior

Version

Additional context

kdy1 commented Sep 17, 2023

kdy1 commented Sep 17, 2023 • edited Loading

patricklafrance commented Sep 17, 2023

marviobezerra commented Sep 19, 2023

kdy1 commented Sep 19, 2023

kdy1 commented Sep 19, 2023

marviobezerra commented Sep 19, 2023 • edited Loading

swc-bot commented Oct 20, 2023

[Wrong] ESM module type does not support fields from `common-js` #7963

[Wrong] ESM module type does not support fields from `common-js` #7963

patricklafrance commented Sep 17, 2023 •

edited

Loading

kdy1 commented Sep 17, 2023 •

edited

Loading

marviobezerra commented Sep 19, 2023 •

edited

Loading