fix(template-webpack-plugin): use path.posix.format for consistent path separators by luhc228 · Pull Request #2359 · lynx-family/lynx-stack

luhc228 · 2026-03-19T12:03:02Z

The tasm.json path on Windows platform has \ backslash. We should always use / slash and webpack/rspack use slash too.

Summary by CodeRabbit

Bug Fixes
- Development and debug asset paths now use consistent POSIX-style separators across Windows, macOS, and Linux, preventing mixed backslash/forward-slash issues and reducing cross-platform asset loading problems.
Chores
- Prepared and declared a patch release for the template webpack plugin to deliver this cross-platform path consistency fix.

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).

…th separators

changeset-bot · 2026-03-19T12:03:07Z

🦋 Changeset detected

Latest commit: a7eca69

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

This PR includes changesets to release 3 packages

Name	Type
@lynx-js/template-webpack-plugin	Patch
@lynx-js/react-rsbuild-plugin	Patch
@lynx-js/react-alias-rsbuild-plugin	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 · 2026-03-19T12:03:27Z

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: d467732d-b785-4590-b7a1-a0bf2bc84f4d

📥 Commits

Reviewing files that changed from the base of the PR and between 8a10bc0 and a7eca69.

📒 Files selected for processing (1)

packages/webpack/template-webpack-plugin/src/LynxTemplatePlugin.ts

🚧 Files skipped from review as they are similar to previous changes (1)

packages/webpack/template-webpack-plugin/src/LynxTemplatePlugin.ts

📝 Walkthrough

Walkthrough

Updated the webpack template plugin to normalize intermediate paths to POSIX separators and switched uses of path.format to path.posix.format. Added a changeset declaring a patch release for @lynx-js/template-webpack-plugin.

Changes

Cohort / File(s)	Summary
Changeset Documentation `.changeset/fix-path-posix-format.md`	Added a patch release changeset noting the migration from `path.format` to `path.posix.format` for consistent path separators.
Path Formatting Fix `packages/webpack/template-webpack-plugin/src/LynxTemplatePlugin.ts`	Normalized `intermediate` paths by replacing backslashes with `/` and switched `path.format()` usages to `path.posix.format()` for debug/dev emitted assets and per-chunk filenames.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested reviewers

colinaaa
upupming

Poem

🐰 I hopped through slashes, swapped backslash for day,
POSIX paths now lead the filename way.
A tiny patch, tidy and bright,
Debug assets marching left to right —
I nibble carrots and celebrate the play!

🚥 Pre-merge checks | ✅ 3

✅ 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 clearly and specifically describes the main change: using path.posix.format for consistent path separators across platforms.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests

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

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

codecov · 2026-03-19T12:05:45Z

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

Copilot

Pull request overview

This PR addresses Windows-specific path separator issues for emitted intermediate debug assets (e.g., tasm.json) by switching to POSIX path formatting so emitted asset names use /, aligning with webpack/rspack conventions.

Changes:

Use path.posix.format when emitting tasm.json and lepus chunk debug assets.
Add a changeset to publish a patch release of @lynx-js/template-webpack-plugin.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
packages/webpack/template-webpack-plugin/src/LynxTemplatePlugin.ts	Switches debug asset emission paths to POSIX formatting to avoid `\\` separators on Windows.
.changeset/fix-path-posix-format.md	Adds a patch changeset documenting the path separator fix.

Comments suppressed due to low confidence (1)

packages/webpack/template-webpack-plugin/src/LynxTemplatePlugin.ts:884

Same issue here: path.posix.format does not convert existing Windows separators inside dir, so asset names can still contain \\. Normalize intermediate (or construct this path via POSIX-safe logic) before emitting the .js debug assets to guarantee consistent / separators across platforms.

            path.posix.format({
              dir: intermediate,
              name,
              ext: '.js',
            }),

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

coderabbitai

🧹 Nitpick comments (1)

packages/webpack/template-webpack-plugin/src/LynxTemplatePlugin.ts (1)

867-888: Add a regression test for Windows-style path separators.

Please add a fixture/assertion that emitted asset keys use / (e.g., .../tasm.json), so this behavior doesn’t regress in future path-related refactors.

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@packages/webpack/template-webpack-plugin/src/LynxTemplatePlugin.ts` around
lines 867 - 888, Add a regression test that ensures emitted asset keys always
use POSIX separators: instantiate LynxTemplatePlugin (or run the plugin's emit
phase) with intermediate set to a Windows-style path (or construct with
path.join/backslashes) and assert that compilation.emitAsset calls produce keys
with '/' separators for the tasm.json asset and for each
resolvedEncodeOptions.lepusCode.lepusChunk entry; target the code paths around
compilation.emitAsset, path.posix.format, RawSource, and
resolvedEncodeOptions.lepusCode.lepusChunk (the isDebug/isDev branch where
tasm.json and chunk .js assets are emitted) and fail the test if any emitted
asset key contains backslashes.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@packages/webpack/template-webpack-plugin/src/LynxTemplatePlugin.ts`:
- Around line 867-888: Add a regression test that ensures emitted asset keys
always use POSIX separators: instantiate LynxTemplatePlugin (or run the plugin's
emit phase) with intermediate set to a Windows-style path (or construct with
path.join/backslashes) and assert that compilation.emitAsset calls produce keys
with '/' separators for the tasm.json asset and for each
resolvedEncodeOptions.lepusCode.lepusChunk entry; target the code paths around
compilation.emitAsset, path.posix.format, RawSource, and
resolvedEncodeOptions.lepusCode.lepusChunk (the isDebug/isDev branch where
tasm.json and chunk .js assets are emitted) and fail the test if any emitted
asset key contains backslashes.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 2a4ba415-dc43-4105-a627-62fd66dd5378

📥 Commits

Reviewing files that changed from the base of the PR and between 726f600 and 58db055.

📒 Files selected for processing (2)

.changeset/fix-path-posix-format.md
packages/webpack/template-webpack-plugin/src/LynxTemplatePlugin.ts

codspeed-hq · 2026-03-19T12:20:29Z

Merging this PR will degrade performance by 14.92%

⚡ 1 improved benchmark
❌ 1 regressed benchmark
✅ 61 untouched benchmarks
⏩ 21 skipped benchmarks¹

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

	Benchmark	`BASE`	`HEAD`	Efficiency
⚡	`002-hello-reactLynx-destroyBackground`	861.5 µs	679.2 µs	+26.83%
❌	`transform 1000 view elements`	40.3 ms	47.4 ms	-14.92%

_{Comparing luhc228:fix/path-format (a7eca69) with main (5177f13)}

21 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩

relativeci · 2026-03-19T12:21:59Z

Web Explorer

#8475 Bundle Size — 727.89KiB (0%).

a7eca69(current) vs 5177f13 main#8473(baseline)

Bundle metrics

1 change

	Current #8475	Baseline #8473
Initial JS	`42.62KiB`	`42.62KiB`
Initial CSS	`2.16KiB`	`2.16KiB`
Cache Invalidation	`0%`	`0%`
Chunks	`8`	`8`
Assets	`10`	`10`
Modules	`147`(`-0.68%`)	`148`
Duplicate Modules	`11`	`11`
Duplicate Code	`34.74%`	`34.74%`
Packages	`3`	`3`
Duplicate Packages	`0`	`0`

Bundle size by type no changes

	Current #8475	Baseline #8473
Other	`384.4KiB`	`384.4KiB`
JS	`341.33KiB`	`341.33KiB`
CSS	`2.16KiB`	`2.16KiB`

Bundle analysis report Branch luhc228:fix/path-format Project dashboard

^{Generated by RelativeCI Documentation Report issue}

Copilot

Pull request overview

This PR addresses cross-platform consistency for emitted intermediate asset paths in @lynx-js/template-webpack-plugin, ensuring Windows path separators don’t leak into webpack/rspack asset names (notably tasm.json).

Changes:

Emit tasm.json and lepus chunk debug assets using path.posix.format to enforce / separators.
Add a changeset to release the fix as a patch.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
packages/webpack/template-webpack-plugin/src/LynxTemplatePlugin.ts	Switches debug-only emitted asset path formatting to POSIX-style formatting (with manual `\\` → `/` normalization).
.changeset/fix-path-posix-format.md	Adds a patch changeset describing the path separator consistency fix.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

relativeci · 2026-03-27T08:26:05Z

React Example

#6897 Bundle Size — 237.89KiB (0%).

a7eca69(current) vs 5177f13 main#6895(baseline)

Bundle metrics no changes

	Current #6897	Baseline #6895
Initial JS	`0B`	`0B`
Initial CSS	`0B`	`0B`
Cache Invalidation	`0%`	`0%`
Chunks	`0`	`0`
Assets	`4`	`4`
Modules	`180`	`180`
Duplicate Modules	`71`	`71`
Duplicate Code	`46.4%`	`46.4%`
Packages	`2`	`2`
Duplicate Packages	`0`	`0`

Bundle size by type no changes

	Current #6897	Baseline #6895
IMG	`145.76KiB`	`145.76KiB`
Other	`92.13KiB`	`92.13KiB`

Bundle analysis report Branch luhc228:fix/path-format Project dashboard

^{Generated by RelativeCI Documentation Report issue}

relativeci · 2026-03-27T08:26:07Z

React MTF Example

#31 Bundle Size — 207.47KiB (0%).

a7eca69(current) vs 5177f13 main#29(baseline)

Bundle metrics no changes

	Current #31	Baseline #29
Initial JS	`0B`	`0B`
Initial CSS	`0B`	`0B`
Cache Invalidation	`0%`	`0%`
Chunks	`0`	`0`
Assets	`3`	`3`
Modules	`174`	`174`
Duplicate Modules	`68`	`68`
Duplicate Code	`46.09%`	`46.09%`
Packages	`2`	`2`
Duplicate Packages	`0`	`0`

Bundle size by type no changes

	Current #31	Baseline #29
IMG	`111.23KiB`	`111.23KiB`
Other	`96.24KiB`	`96.24KiB`

Bundle analysis report Branch luhc228:fix/path-format Project dashboard

^{Generated by RelativeCI Documentation Report issue}

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Signed-off-by: Hengchang Lu <luhengchang228@126.com>

Copilot

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

This PR was opened by the [Changesets release](https://github.com/changesets/action) GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated. # Releases ## @lynx-js/rspeedy@0.14.0 ### Minor Changes - feat: add `Minify.mainThreadOptions` and `Minify.backgroundOptions` for thread-specific minifier. ([#2336](#2336)) ### Patch Changes - Bump Rsbuild v1.7.4 with Rspack v1.7.10. ([#2384](#2384)) - Updated dependencies \[]: - @lynx-js/web-rsbuild-server-middleware@0.20.0 ## @lynx-js/lynx-bundle-rslib-config@0.3.0 ### Minor Changes - **BREAKING CHANGE**: ([#2370](#2370)) Simplify the API for external bundle builds by `externalsPresets` and `externalsPresetDefinitions`. ### Patch Changes - Preserve the default external-bundle `output.minify.jsOptions` when users set `output.minify: true` in `defineExternalBundleRslibConfig`, so required minifier options are not lost. ([#2390](#2390)) ## @lynx-js/external-bundle-rsbuild-plugin@0.1.0 ### Minor Changes - **BREAKING CHANGE**: ([#2370](#2370)) Simplify the API for external bundle builds by `externalsPresets` and `externalsPresetDefinitions`. ### Patch Changes - Updated dependencies \[[`7b7a0c6`](7b7a0c6)]: - @lynx-js/externals-loading-webpack-plugin@0.1.0 ## @lynx-js/react-rsbuild-plugin@0.14.0 ### Minor Changes - feat: support `optimizeBundleSize` option to remove unused code for main-thread and background. ([#2336](#2336)) - If `optimizeBundleSize` is `true` or `optimizeBundleSize.background` is `true`, `lynx.registerDataProcessors` calls will be marked as pure for the background thread output. - If `optimizeBundleSize` is `true` or `optimizeBundleSize.mainThread` is `true`, `NativeModules.call` and `lynx.getJSModule` calls will be marked as pure for the main-thread output. ### Patch Changes - refactor: remove `modifyWebpackChain` since Rsbuild 2.0 dropped webpack support ([#2397](#2397)) - Updated dependencies \[[`9193711`](9193711)]: - @lynx-js/template-webpack-plugin@0.10.7 - @lynx-js/css-extract-webpack-plugin@0.7.0 - @lynx-js/react-webpack-plugin@0.8.0 - @lynx-js/react-alias-rsbuild-plugin@0.14.0 - @lynx-js/use-sync-external-store@1.5.0 - @lynx-js/react-refresh-webpack-plugin@0.3.5 ## @lynx-js/web-core@0.20.0 ### Minor Changes - **This is a breaking change** ([#2322](#2322)) ## Architectural Upgrade: `web-core-wasm` replaces `web-core` This release marks a major architectural upgrade for the web platform. The experimental, WASM-powered engine formerly known as `web-core-wasm` has been fully stabilized and merged into the main branch, completely replacing the previous pure JS/TS based `web-core` implementation. This consolidation massively improves execution performance and aligns the API boundaries of the Web platform directly with other native Lynx implementations. ### 🎉 Added Features - **Core API Enhancements**: Successfully exposed and supported `__QuerySelector` and `__InvokeUIMethod` methods. - **Security & CSP Compliance**: Added a `nonce` attribute to the iframe's `srcdoc` script execution, strengthening Content Security Policy (CSP) compliance. - **`<lynx-view>` Parameter Enhancements**: - Added the `browser-config` attribute and property to `<lynx-view>`. Development environments can now supply a `BrowserConfig` object (e.g., configuring `pixelRatio`, `pixelWidth`, `pixelHeight`) allowing the `systemInfo` payload to be dynamically configured at the instance level. ### 🔄 Changed Features - **Legacy JSON Backwards Compatibility**: Delivered comprehensive fixes and optimizations to deeply support legacy JSON output templates: - Added support for lazy loading execution mode (`lazy usage`). - Implemented the correct decoding and handling of `@keyframe` animation rules. - Rectified rule scoping matching including scoped CSS, root selectors, and type selectors. - **Ecosystem Migration**: Updated testing and ecosystem applications (such as `web-explorer` and `shell-project`) to migrate away from obsolete fragmented dependencies. The new WASM architecture seamlessly integrates Element APIs and CSS directly inside the core client module, requiring a much simpler initialization footprint. **Before (Legacy `web-core` + `web-elements`):** ```typescript // Required multiple imports to assemble the environment import "@lynx-js/web-core/client"; import type { LynxViewElement as LynxView } from "@lynx-js/web-core"; // Had to manually import separate elements and their CSS import "@lynx-js/web-elements/index.css"; import "@lynx-js/web-elements/all"; const lynxView = document.createElement("lynx-view") as LynxView; // ... ``` **After (New `web-core` unified architecture):** ```typescript // The new engine natively registers Web Components and injects fundamental CSS import "@lynx-js/web-core/client"; import type { LynxViewElement as LynxView } from "@lynx-js/web-core/client"; const lynxView = document.createElement("lynx-view") as LynxView; // ... ``` _(Applications can now drop `@lynx-js/web-elements` entirely from their `package.json` dependencies)._ - **Dependency & Boot Sequence Improvements**: Re-architected module loading pathways. Promoted `wasm-feature-detect` directly to a core dependency, and hardened the web worker count initialization assertions. - **Initialization Optimizations**: Converted `SERVER_IN_SHADOW_CSS` initialization bounds to use compilation-time constant expressions for better optimization. ### 🗑️ Deleted Features & Structural Deprecations - **`<lynx-view>` Parameter Removals**: - Removed the `thread-strategy` property and attribute. Historically, this permitted consumers to toggle between `'multi-thread'` and `'all-on-ui'` modes depending on how they wanted the background logic to be executed. The WASM-driven architecture enforces a consolidated concurrency model, deprecating this `<lynx-view>` attribute entirely. - Removed the `overrideLynxTagToHTMLTagMap` property/attribute. HTML tag overriding mechanism has been deprecated in the new engine. - Removed the `customTemplateLoader` property handler from `<lynx-view>`. - Removed the `inject-head-links` property and attribute (`injectHeadLinks`), which previously was used to automatically inject `<link rel="stylesheet">` tags from the document head into the `lynx-view` shadow root. - **Fragmented Packages Removal**: The new cohesive WASM architecture native to `@lynx-js/web-core` handles cross-thread communication, worker boundaries, and rendering loops uniformly. Consequently, multiple obsolete packages have been completely removed from the workspace: - `@lynx-js/web-mainthread-apis` - `@lynx-js/web-worker-runtime` - `@lynx-js/web-core-server` - `@lynx-js/web-core-wasm-e2e` (transitioned into standard test suites) - Added support for `rpx` unit ([#2377](#2377)) **This is a breaking change** The following Styles has been added to `web-core` ```css lynx-view { width: 100%; container-name: lynx-view; container-type: inline-size; --rpx-unit: 1cqw; } ``` Check MDN for the details about these styles: - <https://developer.mozilla.org/en-US/docs/Web/CSS/Reference/Properties/container-name> - <https://developer.mozilla.org/en-US/docs/Web/CSS/Reference/Properties/container-type> - <https://developer.mozilla.org/en-US/docs/Web/CSS/Guides/Containment/Container_queries> ### how it works? For the following code ```html <view style="height:1rpx"></view> ``` it will be transformed to ```html <view style="height:calc(1 * var(--rpx-unit))"></view> ``` Therefore you could use any `<length>` value to replace the unit, for example: ```html <lynx-view style="--rpx-unit:1px"></lynx-view> ``` By default, the --rpx-unit value is `1cqw` - Added support for transform `vw` and `vh` unit ([#2377](#2377)) Add `transform-vw` and `transform-vh` attributes and properties on `<lynx-view>`. For the following code ```html <view style="height:1vw"></view> ``` If the `transform-vw` is enabled `<lynx-view transform-vw="true">`, it will be transformed to ```html <view style="height:calc(1 * var(--vw-unit))"></view> ``` Therefore you could use any `<length>` value to replace the unit, for example: ```html <lynx-view style="--vw-unit:1px"></lynx-view> ``` ### Patch Changes - feat(web-core): add `is_bubble` parameter to `common_event_handler` to properly handle non-bubbling events like `window.Event('click', { bubbles: false })`. ([#2399](#2399)) - chore: update readme ([#2380](#2380)) - fix: the output format should be module ([#2388](#2388)) - opt: use opt-level 3 to compile wasm ([#2371](#2371)) - fix(web-core): avoid partial bundle loading and double fetching when fetchBundle is called concurrently for the same url. ([#2386](#2386)) - fix(web-core): fallback to the original export chunk when `processEvalResult` is absent during `queryComponent` execution ([#2399](#2399)) - fix: tokenizing inline style values correctly to support rpx and ppx unit conversion ([#2381](#2381)) This fixes an issue where the `transform_inline_style_key_value_vec` API bypassed the CSS tokenizer, preventing dimension units like `rpx` or `ppx` from being successfully transformed into `calc` strings when specified via inline styles. - feat: add mts lynx.querySelectorAll API ([#2382](#2382)) - fix: mts in lazy component ([#2375](#2375)) - fix: enableJSDataProcessor not work ([#2372](#2372)) - feat: add `ppx` unit support for CSS, transforming to `calc(... * var(--ppx-unit))` directly. ([#2381](#2381)) - Updated dependencies \[]: - @lynx-js/web-worker-rpc@0.20.0 ## @lynx-js/externals-loading-webpack-plugin@0.1.0 ### Minor Changes - **BREAKING CHANGE**: ([#2370](#2370)) Simplify the API for external bundle builds by `externalsPresets` and `externalsPresetDefinitions`. ## @lynx-js/devtool-connector@0.1.1 ### Patch Changes - fix: align GlobalKeys with Android DevToolSettings keys and filter global switch responses ([#2392](#2392)) ## @lynx-js/devtool-mcp-server@0.5.1 ### Patch Changes - Updated dependencies \[[`95fff27`](95fff27)]: - @lynx-js/devtool-connector@0.1.1 ## @lynx-js/react@0.117.1 ### Patch Changes - Update preact version to simplify `setProperty` implementation ([#2367](#2367)) ## @lynx-js/react-umd@0.117.1 ### Patch Changes - Add a new `entry` export to `@lynx-js/react-umd` for reuse by wrapper libraries of `@lynx-js/react`. ([#2370](#2370)) ## create-rspeedy@0.14.0 ### Patch Changes - Add optional Lynx DevTool skill. ([#2421](#2421)) - move Vitest integration to create-rstack extraTools and merge the Vitest templates into a single incremental overlay ([#2408](#2408)) ## @lynx-js/kitten-lynx-test-infra@0.1.2 ### Patch Changes - Updated dependencies \[[`95fff27`](95fff27)]: - @lynx-js/devtool-connector@0.1.1 ## @lynx-js/template-webpack-plugin@0.10.7 ### Patch Changes - use path.posix.format instead of path.format to ensure consistent path separators across platforms ([#2359](#2359)) - Updated dependencies \[[`75960cd`](75960cd), [`518c310`](518c310), [`863469e`](863469e), [`dc18c5c`](dc18c5c), [`7d242f3`](7d242f3), [`62bebcf`](62bebcf), [`75960cd`](75960cd), [`182f568`](182f568), [`1aa051d`](1aa051d), [`6b46f7e`](6b46f7e), [`fcda36a`](fcda36a), [`182f568`](182f568), [`138f727`](138f727), [`138f727`](138f727)]: - @lynx-js/web-core@0.20.0 ## @lynx-js/react-alias-rsbuild-plugin@0.14.0 ## upgrade-rspeedy@0.14.0 ## @lynx-js/web-rsbuild-server-middleware@0.20.0 ## @lynx-js/web-worker-rpc@0.20.0 Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

fix(template-webpack-plugin): use path.posix.format for consistent pa…

58db055

…th separators

luhc228 requested review from colinaaa and upupming as code owners March 19, 2026 12:03

Copilot AI review requested due to automatic review settings March 19, 2026 12:03

coderabbitai bot added the platform:Web label Mar 19, 2026

Copilot AI reviewed Mar 19, 2026

View reviewed changes

Comment thread packages/webpack/template-webpack-plugin/src/LynxTemplatePlugin.ts

Comment thread .changeset/fix-path-posix-format.md

coderabbitai bot reviewed Mar 19, 2026

View reviewed changes

luhc228 requested a review from Copilot March 27, 2026 08:09

Copilot started reviewing on behalf of luhc228 March 27, 2026 08:09 View session

Copilot AI reviewed Mar 27, 2026

View reviewed changes

Comment thread packages/webpack/template-webpack-plugin/src/LynxTemplatePlugin.ts Outdated

Comment thread packages/webpack/template-webpack-plugin/src/LynxTemplatePlugin.ts

Potential fix for pull request finding

a7eca69

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Signed-off-by: Hengchang Lu <luhengchang228@126.com>

luhc228 force-pushed the fix/path-format branch from 8a10bc0 to a7eca69 Compare March 27, 2026 08:46

luhc228 requested a review from Copilot March 27, 2026 09:45

Copilot started reviewing on behalf of luhc228 March 27, 2026 09:45 View session

Copilot AI reviewed Mar 27, 2026

View reviewed changes

Comment thread packages/webpack/template-webpack-plugin/src/LynxTemplatePlugin.ts

Comment thread .changeset/fix-path-posix-format.md

upupming approved these changes Mar 27, 2026

View reviewed changes

colinaaa approved these changes Mar 28, 2026

View reviewed changes

colinaaa merged commit 9193711 into lynx-family:main Mar 28, 2026
52 of 53 checks passed

github-actions bot mentioned this pull request Mar 28, 2026

chore: Release 2026-04-03 09:43:07 #2368

Merged

Conversation

luhc228 commented Mar 19, 2026 • edited by coderabbitai bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary by CodeRabbit

Checklist

Uh oh!

changeset-bot bot commented Mar 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🦋 Changeset detected

Uh oh!

coderabbitai bot commented Mar 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Estimated code review effort

Suggested reviewers

Poem

Uh oh!

codecov bot commented Mar 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

coderabbitai bot left a comment

Choose a reason for hiding this comment

Uh oh!

codspeed-hq bot commented Mar 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Merging this PR will degrade performance by 14.92%

Performance Changes

Footnotes

Uh oh!

relativeci bot commented Mar 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Web Explorer

#8475 Bundle Size — 727.89KiB (0%).

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

relativeci bot commented Mar 27, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

React Example

#6897 Bundle Size — 237.89KiB (0%).

Uh oh!

relativeci bot commented Mar 27, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

React MTF Example

#31 Bundle Size — 207.47KiB (0%).

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

luhc228 commented Mar 19, 2026 •

edited by coderabbitai bot

Loading

changeset-bot bot commented Mar 19, 2026 •

edited

Loading

coderabbitai bot commented Mar 19, 2026 •

edited

Loading

codecov bot commented Mar 19, 2026 •

edited

Loading

codspeed-hq bot commented Mar 19, 2026 •

edited

Loading

relativeci bot commented Mar 19, 2026 •

edited

Loading

relativeci bot commented Mar 27, 2026 •

edited

Loading

relativeci bot commented Mar 27, 2026 •

edited

Loading