fix: rename Web Preview label to fix URL alignment by Huxpro · Pull Request #2355 · lynx-family/lynx-stack

Huxpro · 2026-03-18T23:20:56Z

Summary

Rsbuild uses label.padEnd(10) internally to align labels with URLs. "Web Preview" (11 chars) exceeded this limit, causing the label and URL to run together without spacing
Rename to "∟ Preview" (9 chars) which fits within the 10-char pad and visually conveys it as a sub-item of Web

Before:

  ➜  Web       http://172.20.10.2:3003/main.web.bundle
  ➜  Web Previewhttp://172.20.10.2:3003/__web_preview?casename=main.web.bundle
  ➜  Lynx      http://172.20.10.2:3003/main.lynx.bundle

After:

  ➜  Web       http://172.20.10.2:3003/main.web.bundle
  ➜  ∟ Preview http://172.20.10.2:3003/__web_preview?casename=main.web.bundle
  ➜  Lynx      http://172.20.10.2:3003/main.lynx.bundle

Test plan

Verify pnpm dev output alignment with environments.web enabled

Summary by CodeRabbit

Style
- Renamed the web environment preview label from "Web Preview" to "∟ Preview" for improved alignment in dev output.
Tests
- Updated tests to expect the new preview label.
Chores
- Added a changeset entry to mark a patch release reflecting the label update.

Rsbuild uses `label.padEnd(10)` internally to align labels with URLs. "Web Preview" (11 chars) exceeded this limit, causing the label and URL to run together without spacing. Rename to "∟ Preview" (9 chars) which fits within the 10-char pad and visually conveys it as a sub-item of Web.

changeset-bot · 2026-03-18T23:21:04Z

🦋 Changeset detected

Latest commit: 9cb4114

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/rspeedy	Patch
create-rspeedy	Patch
upgrade-rspeedy	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-18T23:21:21Z

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 926bc051-c216-4fb2-824f-f0e14e0e9d73

📥 Commits

Reviewing files that changed from the base of the PR and between 565119d and 9cb4114.

📒 Files selected for processing (1)

.changeset/poor-colts-peel.md

📝 Walkthrough

Walkthrough

Updated the web environment preview label in the dev plugin from "Web Preview" to "∟ Preview" and adjusted the corresponding test and a changeset entry.

Changes

Cohort / File(s)	Summary
Dev Plugin `packages/rspeedy/core/src/plugins/dev.plugin.ts`, `packages/rspeedy/core/test/plugins/dev.plugin.test.ts`	Renamed printed web preview label from `"Web Preview"` to `"∟ Preview"` and updated the test expectation accordingly.
Release metadata `.changeset/poor-colts-peel.md`	Added a changeset marking a patch release noting the label rename.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

fix: rename Web Preview label to fix URL alignment #2355: Same code-level change renaming the web preview label and updating tests.
feat: support web preview in rspeedy dev #1891: Edits to dev.plugin.ts URL-printing/label logic closely related to this label rename.

Suggested labels

platform:Web

Suggested reviewers

upupming
luhc228

Poem

A rabbit hops and twitches its nose, 🐇
A tiny glyph the preview chose,
From "Web Preview" to a snug little sign,
"∟ Preview" now aligns just fine. ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly summarizes the main change: renaming the 'Web Preview' label to '∟ Preview' to fix URL alignment issues caused by padding overflow.

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

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Commit unit tests in branch Huxpro/fix-web-preview-align

📝 Coding Plan

Generate coding plan for human review comments

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

Copilot

Pull request overview

This PR adjusts the “web preview” URL label emitted by rspeedy’s dev plugin so the label stays within Rsbuild’s fixed-width label padding and doesn’t visually run into the URL in terminal output.

Changes:

Rename the printed label from Web Preview to ∟ Preview for the /__web_preview URL entry.
Update the corresponding dev plugin test expectation to match the new label.

Reviewed changes

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

File	Description
packages/rspeedy/core/src/plugins/dev.plugin.ts	Changes the generated `printUrls` label for the web preview entry to a shorter value.
packages/rspeedy/core/test/plugins/dev.plugin.test.ts	Updates test expectation to reflect the new label in `printUrls` output.

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

codecov · 2026-03-18T23:25:14Z

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!

codspeed-hq · 2026-03-18T23:35:54Z

Merging this PR will improve performance by 9.82%

⚡ 1 improved benchmark
✅ 71 untouched benchmarks
⏩ 3 skipped benchmarks¹

Performance Changes

	Benchmark	`BASE`	`HEAD`	Efficiency
⚡	`basic-performance-nest-level-100`	7.6 ms	6.9 ms	+9.82%

_{Comparing Huxpro/fix-web-preview-align (565119d) with main (799fda8)}

3 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-18T23:38:17Z

Web Explorer

#8234 Bundle Size — 385.21KiB (0%).

565119d(current) vs 799fda8 main#8222(baseline)

Bundle metrics no changes

	Current #8234	Baseline #8222
Initial JS	`155.59KiB`	`155.59KiB`
Initial CSS	`35.1KiB`	`35.1KiB`
Cache Invalidation	`0%`	`0%`
Chunks	`8`	`8`
Assets	`8`	`8`
Modules	`238`	`238`
Duplicate Modules	`16`	`16`
Duplicate Code	`2.97%`	`2.97%`
Packages	`4`	`4`
Duplicate Packages	`0`	`0`

Bundle size by type no changes

	Current #8234	Baseline #8222
JS	`254.26KiB`	`254.26KiB`
Other	`95.85KiB`	`95.85KiB`
CSS	`35.1KiB`	`35.1KiB`

Bundle analysis report Branch Huxpro/fix-web-preview-align Project dashboard

^{Generated by RelativeCI Documentation Report issue}

Signed-off-by: Qingyu Wang <40660121+colinaaa@users.noreply.github.com>

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/react@0.117.0 ### Minor Changes - feat: export `GlobalPropsProvider`, `GlobalPropsConsumer`, `useGlobalProps` and `useGlobalPropsChanged` for `__globalProps` ([#2346](#2346)) - `GlobalPropsProvider`: A Provider component that accepts `children`. It is used to provide the `lynx.__globalProps` context. - `GlobalPropsConsumer`: A Consumer component that accepts a function as a child. It is used to consume the `lynx.__globalProps` context. - `useGlobalProps`: A hook that returns the `lynx.__globalProps` object. It triggers a re-render when `lynx.__globalProps` changes. - `useGlobalPropsChanged`: A hook that accepts a callback function. The callback is invoked when `lynx.__globalProps` changes. Note: When `globalPropsMode` is not set to `'event'` (default is `'reactive'`), these APIs will be ineffective (pass-through) and will log a warning in development mode, as updates are triggered automatically by full re-render. - **BREAKING CHANGE**: ([#2319](#2319)) Change preact package from `@hongzhiyuan/preact` to `@lynx-js/internal-preact`. Upgrade preact from [f7693b72](preactjs/preact@f7693b7) to [55254ef7](preactjs/preact@55254ef), see diffs at [f7693b72...55254ef7](https://github.com/preactjs/preact/compare/f7693b72ecb4a40c66e6e47f54e2d4edc374c9f0...preactjs:preact:55254ef7021e563cc1a86fb816058964a1b6a29a?expand=1). - feat: add `globalPropsMode` option to `PluginReactLynxOptions` ([#2346](#2346)) - When configured to `"event"`, `updateGlobalProps` will only trigger a global event and skip the `runWithForce` flow. - Defaults to `"reactive"`, which means `updateGlobalProps` will trigger re-render automatically. ### Patch Changes - Add `__BACKGROUND__` guard on `onBackgroundSnapshotInstanceUpdateId` event to prevent bundling to main-thread on dev environment. ([#2332](#2332)) - refactor: extract static string in template literal ([#2334](#2334)) - fix: avoid crash when spread undefined ref ([#2333](#2333)) - Avoid registering lifecycle refs for main-thread functions (MTF) that have not received an `execId` during `renderPage()` first-screen binding. ([#2320](#2320)) ## @lynx-js/react-umd@0.117.0 ### Minor Changes - Add standalone UMD build of the ReactLynx runtime. ([#2331](#2331)) ## @lynx-js/react-rsbuild-plugin@0.13.0 ### Minor Changes - **BREAKING CHANGE**: ([#2319](#2319)) Change preact package from `@hongzhiyuan/preact` to `@lynx-js/internal-preact`. Upgrade preact from [f7693b72](preactjs/preact@f7693b7) to [55254ef7](preactjs/preact@55254ef), see diffs at [f7693b72...55254ef7](https://github.com/preactjs/preact/compare/f7693b72ecb4a40c66e6e47f54e2d4edc374c9f0...preactjs:preact:55254ef7021e563cc1a86fb816058964a1b6a29a?expand=1). - feat: add `globalPropsMode` option to `PluginReactLynxOptions` ([#2346](#2346)) - When configured to `"event"`, `updateGlobalProps` will only trigger a global event and skip the `runWithForce` flow. - Defaults to `"reactive"`, which means `updateGlobalProps` will trigger re-render automatically. ### Patch Changes - Updated dependencies \[[`f1129ea`](f1129ea), [`27f1cff`](27f1cff), [`ed566f0`](ed566f0), [`402ec2b`](402ec2b)]: - @lynx-js/react-webpack-plugin@0.8.0 - @lynx-js/react-refresh-webpack-plugin@0.3.5 - @lynx-js/react-alias-rsbuild-plugin@0.13.0 - @lynx-js/use-sync-external-store@1.5.0 - @lynx-js/template-webpack-plugin@0.10.6 - @lynx-js/css-extract-webpack-plugin@0.7.0 ## @lynx-js/react-webpack-plugin@0.8.0 ### Minor Changes - feat: add `globalPropsMode` option to `PluginReactLynxOptions` ([#2346](#2346)) - When configured to `"event"`, `updateGlobalProps` will only trigger a global event and skip the `runWithForce` flow. - Defaults to `"reactive"`, which means `updateGlobalProps` will trigger re-render automatically. ### Patch Changes - Fix sourcemap misalignment when wrapping lazy bundle main-thread chunks. ([#2361](#2361)) The lazy bundle IIFE wrapper is now injected in `processAssets` at `PROCESS_ASSETS_STAGE_OPTIMIZE_SIZE + 1` by walking chunk groups instead of patching assets in `beforeEncode`. - With `experimental_isLazyBundle: true`, the wrapper is applied to lazy-bundle chunk groups. - Without lazy bundle mode, the wrapper is applied to async main-thread chunk groups generated by dynamic import. Injecting the wrapper in this stage keeps the emitted JS stable after optimization while still running before `DEV_TOOLING` sourcemap finalization, so the generated `.js` and `.js.map` stay aligned. - Set `__DEV__` and `__PROFILE__` to `true` on `NODE_ENV === 'development'`. ([#2324](#2324)) ## @lynx-js/rspeedy@0.13.6 ### Patch Changes - Rename Web Preview label to fix URL alignment ([#2355](#2355)) - Updated dependencies \[[`799fda8`](799fda8)]: - @lynx-js/cache-events-webpack-plugin@0.0.3 - @lynx-js/web-rsbuild-server-middleware@0.19.9 ## @lynx-js/lynx-bundle-rslib-config@0.2.3 ### Patch Changes - Fix snapshot not found error when dev with external bundle ([#2316](#2316)) ## @lynx-js/external-bundle-rsbuild-plugin@0.0.4 ### Patch Changes - Updated dependencies \[[`ed566f0`](ed566f0)]: - @lynx-js/externals-loading-webpack-plugin@0.0.5 ## @lynx-js/kitten-lynx-test-infra@0.1.1 ### Patch Changes - feat: support page.screenshot() ([#2364](#2364)) - feat: initial commit ([#2272](#2272)) ## @lynx-js/testing-environment@0.1.12 ### Patch Changes - Implement `__ElementAnimate` PAPI for web platform animation lifecycle ([#2329](#2329)) ## @lynx-js/web-constants@0.19.9 ### Patch Changes - Implement `__ElementAnimate` PAPI for web platform animation lifecycle ([#2329](#2329)) - Updated dependencies \[]: - @lynx-js/web-worker-rpc@0.19.9 ## @lynx-js/web-core@0.19.9 ### Patch Changes - Updated dependencies \[[`2efecc2`](2efecc2)]: - @lynx-js/web-constants@0.19.9 - @lynx-js/web-mainthread-apis@0.19.9 - @lynx-js/web-worker-runtime@0.19.9 - @lynx-js/web-worker-rpc@0.19.9 ## @lynx-js/web-core-wasm@0.0.6 ### Patch Changes - reexports essential utils & types in @lynx-js/web-elements from @lynx-js/web-core-wasm/client ([#2321](#2321)) - Updated dependencies \[]: - @lynx-js/web-worker-rpc@0.19.9 ## @lynx-js/web-mainthread-apis@0.19.9 ### Patch Changes - Updated dependencies \[[`2efecc2`](2efecc2)]: - @lynx-js/web-constants@0.19.9 ## @lynx-js/web-worker-runtime@0.19.9 ### Patch Changes - Updated dependencies \[[`2efecc2`](2efecc2)]: - @lynx-js/web-constants@0.19.9 - @lynx-js/web-mainthread-apis@0.19.9 - @lynx-js/web-worker-rpc@0.19.9 ## @lynx-js/cache-events-webpack-plugin@0.0.3 ### Patch Changes - Cache `globalThis.loadDynamicComponent` in the cache events runtime and add tests covering tt methods, performance events, and globalThis replay behavior. ([#2343](#2343)) ## @lynx-js/externals-loading-webpack-plugin@0.0.5 ### Patch Changes - Fix snapshot not found error when dev with external bundle ([#2316](#2316)) ## @lynx-js/react-refresh-webpack-plugin@0.3.5 ### Patch Changes - Fix snapshot not found error when dev with external bundle ([#2316](#2316)) ## @lynx-js/template-webpack-plugin@0.10.6 ### Patch Changes - Updated dependencies \[[`d034dae`](d034dae)]: - @lynx-js/web-core-wasm@0.0.6 ## create-rspeedy@0.13.6 ## @lynx-js/react-alias-rsbuild-plugin@0.13.0 ## upgrade-rspeedy@0.13.6 ## @lynx-js/web-core-server@0.19.9 ## @lynx-js/web-rsbuild-server-middleware@0.19.9 ## @lynx-js/web-worker-rpc@0.19.9 Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

Huxpro requested review from colinaaa, luhc228 and upupming as code owners March 18, 2026 23:20

Copilot AI review requested due to automatic review settings March 18, 2026 23:20

Copilot AI reviewed Mar 18, 2026

View reviewed changes

Huxpro enabled auto-merge (squash) March 19, 2026 00:15

colinaaa previously approved these changes Mar 19, 2026

View reviewed changes

Create poor-colts-peel.md

9cb4114

Signed-off-by: Qingyu Wang <40660121+colinaaa@users.noreply.github.com>

colinaaa dismissed their stale review via 9cb4114 March 19, 2026 02:17

colinaaa disabled auto-merge March 19, 2026 02:18

colinaaa merged commit 726f600 into main Mar 19, 2026
6 of 8 checks passed

colinaaa deleted the Huxpro/fix-web-preview-align branch March 19, 2026 02:18

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

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

chore: Release 2026-03-24 02:07:53 #2323

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix: rename Web Preview label to fix URL alignment#2355

fix: rename Web Preview label to fix URL alignment#2355
colinaaa merged 2 commits intomainfrom
Huxpro/fix-web-preview-align

Huxpro commented Mar 18, 2026 •

edited by coderabbitai bot

Loading

Uh oh!

changeset-bot bot commented Mar 18, 2026 •

edited

Loading

Uh oh!

coderabbitai bot commented Mar 18, 2026 •

edited

Loading

Review failed

Walkthrough

Changes

Estimated code review effort

Possibly related PRs

Suggested labels

Suggested reviewers

Poem

❌ Failed checks (1 warning)

Uh oh!

Copilot AI left a comment

Uh oh!

codecov bot commented Mar 18, 2026

Uh oh!

codspeed-hq bot commented Mar 18, 2026

Uh oh!

relativeci bot commented Mar 18, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

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

Uh oh!

Summary

Test plan

Summary by CodeRabbit

Uh oh!

changeset-bot bot commented Mar 18, 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 18, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Review failed

Walkthrough

Changes

Estimated code review effort

Possibly related PRs

Suggested labels

Suggested reviewers

Poem

❌ Failed checks (1 warning)

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

codecov bot commented Mar 18, 2026

Codecov Report

Uh oh!

codspeed-hq bot commented Mar 18, 2026

Merging this PR will improve performance by 9.82%

Performance Changes

Footnotes

Uh oh!

relativeci bot commented Mar 18, 2026

Web Explorer

#8234 Bundle Size — 385.21KiB (0%).

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Huxpro commented Mar 18, 2026 •

edited by coderabbitai bot

Loading

changeset-bot bot commented Mar 18, 2026 •

edited

Loading

coderabbitai bot commented Mar 18, 2026 •

edited

Loading