chore(deps): bump swc_core to v39.0.3

[gaoachao]

Summary by CodeRabbit

• New Features

  • Added support for the "commonjs" module variant in the React transform API.

• Chores

  • Bumped core dependencies (including swc_core and getrandom).
  • Added a changeset to publish a patch release for @lynx-js/react.

• Build

  • Adjusted WASM build flags and configuration.
  • Removed custom randomness hook in WASM builds, using default behavior.

• Tests

  • Updated snapshots to reflect EOF-based parser error reporting.

Checklist

• Tests updated (or not required).
• Documentation updated (or not required).
• Changeset added, and when a BREAKING CHANGE occurs, it needs to be clearly marked (or not required).

[changeset-bot]

🦋 Changeset detected

Latest commit: c0c4e9d

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@lynx-js/react	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[coderabbitai]

📝 Walkthrough

Walkthrough

Adds a changeset for a patch release, bumps workspace swc_core to 39.0.3, updates wasm getrandom dependency and build flags, removes the custom wasm getrandom wiring, extends N‑API handling to accept/emit commonjs, and passes source_map_ignore_list: None to PrintArgs in transform code.

Changes

Cohort / File(s)	Summary
Release metadata ./.changeset/mighty-numbers-talk.md	Adds a changeset declaring a patch release for @lynx-js/react and documents the swc_core bump.
Workspace Rust dependency bump ./Cargo.toml	Updates workspace dependency swc_core from 23.2.0 to 39.0.3.
WASM getrandom dependency packages/react/transform/Cargo.toml	Bumps getrandom for wasm32-unknown from 0.2.15 to 0.3.3 and removes features = ["custom"].
WASM build script packages/react/transform/scripts/build_wasm.js	Appends --cfg getrandom_backend="custom" to RUSTFLAGS passed to cargo build; other build steps unchanged.
PrintArgs source map change packages/react/transform/src/bundle.rs, packages/react/transform/src/lib.rs	PrintArgs construction now includes source_map_ignore_list: None in bundle and transform code paths.
N‑API IsModuleConfig & wasm randomness wiring packages/react/transform/src/lib.rs	N‑API conversions extended to accept/emit "commonjs" for IsModule::CommonJS; custom wasm getrandom externs/registration removed or commented out.
Tests (snapshot updates) packages/react/transform/__test__/fixture.spec.js	Updated inline snapshots to reflect EOF-based parser error messages and adjusted error locations/lengths.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• refactor(react/transfrom): migrate to Cargo workspace #1714 — Modifies the same Cargo manifest areas and wasm getrandom settings; dependency/wasm configuration changes overlap.

Suggested reviewers

• colinaaa
• PupilTong

Pre-merge checks (3 passed)

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The title "chore(deps): bump swc_core to v39.0.3" is concise, correctly highlights the primary change (upgrading swc_core to 39.0.3) and contains no extraneous noise; it accurately represents the main intent of the PR even though the diff includes small ancillary adjustments.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

Poem

I nibble bytes beneath the moonlight, 🐰
bumping crates and making builds bright.
Maps ignore a few small tracks,
CommonJS finds its cozy cracks.
Hop, tag, and ship — a patch tonight! ✨

✨ Finishing touches

• 📝 Generate Docstrings

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch chore/bump-swc-core-39.0.3

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[codecov]

Codecov Report

❌ Patch coverage is 0% with 13 lines in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines	Patch %	Lines
packages/react/transform/src/lib.rs	0.00%	11 Missing ⚠️
packages/react/transform/scripts/build_wasm.js	0.00%	1 Missing ⚠️
packages/react/transform/src/bundle.rs	0.00%	1 Missing ⚠️

📢 Thoughts on this report? Let us know!

[coderabbitai]

Actionable comments posted: 2

🧹 Nitpick comments (1)

packages/react/transform/src/lib.rs (1)

224-225: Expose and accept “commonjs” consistently

You added IsModule::CommonJS in ToNapiValue, but FromNapiValue and the TS type still don’t accept it. Either remove the arm for now or complete the API so JS callers can pass "commonjs".

Apply this to accept and type it:

-  #[napi(ts_type = "boolean | 'unknown'")]
+  #[napi(ts_type = "boolean | 'unknown' | 'commonjs'")]
   pub is_module: Option<IsModuleConfig>,

-    let str_val = <&str>::from_napi_value(env, napi_val);
-    if str_val.is_ok() && str_val.unwrap() == "unknown" {
-      return Ok(IsModuleConfig(IsModule::Unknown));
-    }
+    if let Ok(s) = <&str>::from_napi_value(env, napi_val) {
+      return Ok(IsModuleConfig(match s {
+        "unknown" => IsModule::Unknown,
+        "commonjs" => IsModule::CommonJS,
+        _ => return Err(napi::bindgen_prelude::error!(
+          napi::bindgen_prelude::Status::InvalidArg,
+          "value `{}` does not match any variant of enum `{}`",
+          s, "IsModuleConfig"
+        )),
+      }));
+    }

Confirmed that IsModule::CommonJS exists in swc_core v39. (rustdoc.swc.rs)

📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 69b3ae0 and d317482.

⛔ Files ignored due to path filters (1)

• Cargo.lock is excluded by !**/*.lock

📒 Files selected for processing (6)

• .changeset/mighty-numbers-talk.md (1 hunks)
• Cargo.toml (1 hunks)
• packages/react/transform/Cargo.toml (1 hunks)
• packages/react/transform/scripts/build_wasm.js (1 hunks)
• packages/react/transform/src/bundle.rs (1 hunks)
• packages/react/transform/src/lib.rs (3 hunks)

🧰 Additional context used

📓 Path-based instructions (1)

.changeset/*.md

📄 CodeRabbit inference engine (AGENTS.md)

For contributions, always generate a changeset and commit the resulting markdown file(s)

Files:

• .changeset/mighty-numbers-talk.md

🧠 Learnings (4)

📚 Learning: 2025-09-10T11:42:36.855Z

Learnt from: gaoachao
PR: lynx-family/lynx-stack#1714
File: packages/react/transform/Cargo.toml:19-19
Timestamp: 2025-09-10T11:42:36.855Z
Learning: In packages/react/transform/Cargo.toml, the crate uses serde derive macros (#[derive(Serialize, Deserialize)]) in multiple files including src/esbuild.rs and src/swc_plugin_extract_str/mod.rs, so the "derive" feature is required when migrating to workspace dependencies.

Applied to files:

• Cargo.toml
• packages/react/transform/Cargo.toml

📚 Learning: 2025-07-22T09:23:07.797Z

Learnt from: colinaaa
PR: lynx-family/lynx-stack#1330
File: .changeset/olive-animals-attend.md:1-3
Timestamp: 2025-07-22T09:23:07.797Z
Learning: In the lynx-family/lynx-stack repository, changesets are only required for meaningful changes to end-users such as bugfixes and features. Internal/development changes like chores, refactoring, or removing debug info do not need changeset entries.

Applied to files:

• .changeset/mighty-numbers-talk.md

📚 Learning: 2025-07-22T09:26:16.722Z

Learnt from: colinaaa
PR: lynx-family/lynx-stack#1330
File: .changeset/olive-animals-attend.md:1-3
Timestamp: 2025-07-22T09:26:16.722Z
Learning: In the lynx-family/lynx-stack repository, CI checks require changesets when files matching the pattern "src/**" are modified (as configured in .changeset/config.json). For internal changes that don't need meaningful changesets, an empty changeset file is used to satisfy the CI requirement while not generating any release notes.

Applied to files:

• .changeset/mighty-numbers-talk.md

📚 Learning: 2025-08-07T04:00:59.645Z

Learnt from: colinaaa
PR: lynx-family/lynx-stack#1454
File: pnpm-workspace.yaml:46-46
Timestamp: 2025-08-07T04:00:59.645Z
Learning: In the lynx-family/lynx-stack repository, the webpack patch (patches/webpack5.101.0.patch) was created to fix issues with webpack5.99.9 but only takes effect on webpack5.100.0 and later versions. The patchedDependencies entry should use "webpack@^5.100.0" to ensure the patch applies to the correct version range.

Applied to files:

• .changeset/mighty-numbers-talk.md

🧬 Code graph analysis (1)

packages/react/transform/src/lib.rs (3)

packages/react/transform/src/swc_plugin_css_scope/mod.rs (1)

• to_napi_value (62-71)

packages/react/transform/src/swc_plugin_inject/mod.rs (1)

• to_napi_value (60-77)

packages/react/transform/src/target.rs (1)

• to_napi_value (37-47)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)

• GitHub Check: build / Build (Ubuntu)
• GitHub Check: build / Build (Windows)
• GitHub Check: test-rust / Test (Ubuntu)

🔇 Additional comments (5)

Cargo.toml (1)

23-23: Workspace bump to swc_core v39.0.3 looks good

No issues spotted here. Downstream updates to PrintArgs in consumers are present in this PR.

.changeset/mighty-numbers-talk.md (1)

1-6: Changeset present and scoped appropriately

Patch entry is fine for this chore. Nothing else needed.

packages/react/transform/Cargo.toml (1)

26-26: Migrated getrandom to 0.3.x for wasm target

Version bump is correct; feature flag removal aligns with 0.3’s cfg-based backend selection. Ensure the codebase provides the required custom backend entrypoint when building with getrandom_backend="custom". See comments in src/lib.rs and build script. (docs.rs)

packages/react/transform/src/bundle.rs (1)

154-178: Added source_map_ignore_list: None to PrintArgs — correct for swc_core v39

Field exists on PrintArgs in current swc toolchain; using None maintains previous behavior. (docs.rs)

packages/react/transform/src/lib.rs (1)

633-641: PrintArgs: source_map_ignore_list: None — aligned with swc changes

Field name and type match current API; keeping it None preserves prior sourcemap behavior. (docs.rs)

[relativeci]

Web Explorer

#5120 Bundle Size — 362.79KiB (0%).

c0c4e9d(current) vs dc28178 main#5118(baseline)

Bundle metrics   2 changes

	Current #5120	Baseline #5118
Initial JS	145.21KiB	145.21KiB
Initial CSS	31.89KiB	31.89KiB
Cache Invalidation	0%	0%
Chunks	8	8
Assets	8	8
Modules	219(-0.45%)	220
Duplicate Modules	16	16
Duplicate Code	3.34%(+0.3%)	3.33%
Packages	4	4
Duplicate Packages	0	0

Bundle size by type  no changes

	Current #5120	Baseline #5118
JS	236.88KiB	236.88KiB
Other	94.02KiB	94.02KiB
CSS	31.89KiB	31.89KiB

Bundle analysis report Branch chore/bump-swc-core-39.0.3 Project dashboard

---

^{Generated by RelativeCI Documentation Report issue}

[relativeci]

React Example

#5127 Bundle Size — 238.9KiB (0%).

c0c4e9d(current) vs dc28178 main#5125(baseline)

Bundle metrics  no changes

	Current #5127	Baseline #5125
Initial JS	0B	0B
Initial CSS	0B	0B
Cache Invalidation	0%	0%
Chunks	0	0
Assets	4	4
Modules	163	163
Duplicate Modules	67	67
Duplicate Code	46.79%	46.79%
Packages	2	2
Duplicate Packages	0	0

Bundle size by type  no changes

	Current #5127	Baseline #5125
IMG	145.76KiB	145.76KiB
Other	93.14KiB	93.14KiB

Bundle analysis report Branch chore/bump-swc-core-39.0.3 Project dashboard

---

^{Generated by RelativeCI Documentation Report issue}

[codspeed-hq]

CodSpeed Performance Report

Merging #1721 will improve performances by ×8

_{Comparing chore/bump-swc-core-39.0.3 (c0c4e9d) with main (e1cdec2)}

Summary

⚡ 4 improvements
✅ 52 untouched

Benchmarks breakdown

	Benchmark	BASE	HEAD	Change
⚡	transform 1000 effects	40.2 ms	30.4 ms	+31.99%
⚡	transform 1000 view elements	373.2 ms	46.4 ms	×8
⚡	transform 1000 view elements with Children	544.7 ms	151.6 ms	×3.6
⚡	transform 1000 view elements with event	690.5 ms	330.6 ms	×2.1

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (1)

packages/react/transform/__test__/fixture.spec.js (1)

698-712: Relax TSX error snapshot to assert only the essentials

Same rationale: pinning exact column/length and full text is brittle across parser versions.

Apply this diff:

-    expect(result.errors).toMatchInlineSnapshot(`
-      [
-        {
-          "location": {
-            "column": 31,
-            "file": "",
-            "length": 0,
-            "line": 1,
-            "lineText": "const p = <any>Promise.all([]);",
-          },
-          "pluginName": "",
-          "text": "Expected '</', got '<eof>'",
-        },
-      ]
-    `);
+    expect(result.errors).toHaveLength(1);
+    expect(result.errors[0]).toMatchObject({
+      text: expect.stringContaining("Expected '</'"),
+      location: expect.objectContaining({
+        line: 1,
+        file: '',
+        lineText: 'const p = <any>Promise.all([]);',
+      }),
+    });

📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between c511fe3 and 6eec8d6.

📒 Files selected for processing (1)

• packages/react/transform/__test__/fixture.spec.js (2 hunks)

🔇 Additional comments (1)

packages/react/transform/__test__/fixture.spec.js (1)

222-240: Make error assertions less brittle — relax inline snapshot

Snapshot pins EOF-based JSX parse details (column/length/text) that shift across swc_core; assert only stable fields.

File: packages/react/transform/test/fixture.spec.js (lines 222-240)

-    expect(result).toMatchInlineSnapshot(`
-      {
-        "code": "",
-        "errors": [
-          {
-            "location": {
-              "column": 7,
-              "file": "",
-              "length": 0,
-              "line": 1,
-              "lineText": "<view>;",
-            },
-            "pluginName": "",
-            "text": "Expected '</', got '<eof>'",
-          },
-        ],
-        "warnings": [],
-      }
-    `);
+    expect(result.code).toBe('');
+    expect(result.warnings).toEqual([]);
+    expect(result.errors).toHaveLength(1);
+    expect(result.errors[0]).toMatchObject({
+      text: expect.stringContaining("Expected '</'"),
+      location: expect.objectContaining({
+        line: 1,
+        file: '',
+        lineText: '<view>;',
+      }),
+    });

Run this locally to find other fragile snapshots:

#!/bin/bash
set -euo pipefail
# Look for error snapshots pinning exact "Expected '</'" messages or column/length pairs.
rg -nP --glob '!node_modules/**' "(Expected '</', got.*|\"column\":\s*\d+,\s*\n\s*\"length\":\s*\d+)"

[colinaaa]

Merging #1721 will improve performances by ×8.1

We would like to make more benchmarks to see if the result is reasonable. Please wait and update main branch until #1725 is merged.

[gaoachao]

Merging #1721 will improve performances by ×8.1

We would like to make more benchmarks to see if the result is reasonable. Please wait and update main branch until #1725 is merged.

LGTM! I was just thinking this might be somewhat unreasonable.