feat: remove element api calls alog by default by upupming · Pull Request #2192 · lynx-family/lynx-stack

upupming · 2026-02-04T03:55:23Z

Element API calls are too verbose, user may not need this information when enabling alog, so we provide an extra option to enable it seperately.

Summary by CodeRabbit

New Features
- Element API calls are now disabled by default and can be enabled via the REACT_ALOG_ELEMENT_API environment variable for granular control.

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 · 2026-02-04T03:55:27Z

🦋 Changeset detected

Latest commit: 0c71204

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

This PR includes changesets to release 5 packages

Name	Type
@lynx-js/testing-environment	Patch
@lynx-js/react-webpack-plugin	Patch
@lynx-js/react	Patch
@lynx-js/react-alias-rsbuild-plugin	Patch
@lynx-js/react-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-02-04T03:55:42Z

📝 Walkthrough

Walkthrough

This pull request introduces a feature-gated element API logging capability via a new __ALOG_ELEMENT_API__ flag. Element PAPICall logging initialization is relocated from the core ALog setup to a conditional block in the lynx runtime, with corresponding webpack plugin configuration and testing environment setup updates.

Changes

Cohort / File(s)	Summary
Feature flag introduction and initialization control `.changeset/tough-cases-help.md`, `packages/react/runtime/types/types.d.ts`, `packages/webpack/react-webpack-plugin/src/ReactWebpackPlugin.ts`	Adds new global declaration and webpack define for `__ALOG_ELEMENT_API__` flag, controlled via `REACT_ALOG_ELEMENT_API` environment variable. Patch version bumps documented for three packages.
ALog initialization restructuring `packages/react/runtime/src/alog/index.ts`, `packages/react/runtime/src/lynx.ts`	Removes `initElementPAPICallAlog` call from core alog initialization and relocates it to lynx.ts with conditional guard checking `__ALOG_ELEMENT_API__` flag.
Testing environment setup `packages/react/testing-library/src/vitest-global-setup.js`, `packages/testing-library/testing-environment/src/index.ts`	Sets `__ALOG_ELEMENT_API__` to `true` in testing environment globals and wraps initialization call with feature flag guard in vitest setup.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

feat(react): support alog of component rendering #1164: Modifies ALog initialization flow and feature flags similarly to this PR, relocating initialization logic between modules.

Suggested labels

framework:React

Suggested reviewers

hzy
HuJean
colinaaa

Poem

🐰 A feature flag hops through the code,
Element logs now take a gated road!
Conditional guards keep tabs so tight,
ALog's element API shines bright! ✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The PR title 'feat: remove element api calls alog by default' accurately describes the main change: element API calls alog is disabled by default and controlled via feature flags (ALOG_ELEMENT_API and REACT_ALOG_ELEMENT_API).
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

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

✨ Finishing touches

📝 Generate docstrings

🧪 Generate unit tests (beta)

Create PR with unit tests
Post copyable unit tests in a comment
Commit unit tests in branch feat/REACT_ALOG_ELEMENT_API

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-02-04T04:00:21Z

Codecov Report

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

Files with missing lines	Patch %	Lines
...s/testing-library/testing-environment/src/index.ts	0.00%	2 Missing ⚠️
packages/react/runtime/src/lynx.ts	50.00%	1 Missing ⚠️

📢 Thoughts on this report? Let us know!

codspeed-hq · 2026-02-04T04:13:41Z

CodSpeed Performance Report

Merging this PR will degrade performance by 7.79%

_{Comparing feat/REACT_ALOG_ELEMENT_API (0c71204) with main (cbb5efc)}

Summary

❌ 1 regressed benchmark
✅ 62 untouched benchmarks
⏩ 3 skipped benchmarks¹

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

Performance Changes

	Benchmark	`BASE`	`HEAD`	Efficiency
❌	`basic-performance-small-css`	7.6 ms	8.2 ms	-7.79%

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-02-04T04:16:26Z

Web Explorer

#7534 Bundle Size — 383.66KiB (0%).

0c71204(current) vs cbb5efc main#7532(baseline)

Bundle metrics

1 change

	Current #7534	Baseline #7532
Initial JS	`154.82KiB`	`154.82KiB`
Initial CSS	`35.05KiB`	`35.05KiB`
Cache Invalidation	`0%`	`0%`
Chunks	`8`	`8`
Assets	`8`	`8`
Modules	`238`(`+0.42%`)	`237`
Duplicate Modules	`16`	`16`
Duplicate Code	`2.99%`	`2.99%`
Packages	`4`	`4`
Duplicate Packages	`0`	`0`

Bundle size by type no changes

	Current #7534	Baseline #7532
JS	`252.76KiB`	`252.76KiB`
Other	`95.85KiB`	`95.85KiB`
CSS	`35.05KiB`	`35.05KiB`

Bundle analysis report Branch feat/REACT_ALOG_ELEMENT_API Project dashboard

^{Generated by RelativeCI Documentation Report issue}

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/motion@0.0.3 ### Patch Changes - Fix an issue that motion/mini will accidentally imports full version, causing compiling error ([#2204](#2204)) ## @lynx-js/react@0.116.3 ### Patch Changes - fix: remove `lynx.createSelectorQuery` deprecated warning in production ([#2195](#2195)) - Add a DEV-only guard that detects MainThread flush loops caused by re-entrant MTS handlers. ([#2159](#2159)) This typically happens when a MainThread handler (e.g. event callback or `MainThreadRef`) performs UI mutations (like `Element.setStyleProperty`, `setStyleProperties`, `setAttribute`, or `invoke`) that synchronously trigger a flush which re-enters the handler again. - Avoid DEV_ONLY_SetSnapshotEntryName on standalone lazy bundle. ([#2184](#2184)) - Add alog and trace for BTS event handlers. ([#2102](#2102)) - fix: Main thread functions cannot access properties on `this` before hydration completes. ([#2194](#2194)) This fixes the `cannot convert to object` error. - Remove element api calls alog by default, and only enable it when `__ALOG_ELEMENT_API__` is defined to `true` or environment variable `REACT_ALOG_ELEMENT_API` is set to `true`. ([#2192](#2192)) - fix: captured variables in main thread functions within class components do not update correctly ([#2197](#2197)) ## @lynx-js/rspeedy@0.13.4 ### Patch Changes - Bump ts-blank-space v0.7.0 ([#2238](#2238)) - Bump Rsbuild v1.7.3 with Rspack v1.7.5. ([#2189](#2189)) - Updated dependencies \[]: - @lynx-js/web-rsbuild-server-middleware@0.19.8 ## @lynx-js/qrcode-rsbuild-plugin@0.4.5 ### Patch Changes - Only register console shortcuts when running in TTY environments ([#2202](#2202)) ## @lynx-js/react-rsbuild-plugin@0.12.8 ### Patch Changes - Updated dependencies \[[`4240138`](4240138)]: - @lynx-js/react-webpack-plugin@0.7.4 - @lynx-js/react-alias-rsbuild-plugin@0.12.8 - @lynx-js/use-sync-external-store@1.5.0 - @lynx-js/react-refresh-webpack-plugin@0.3.4 - @lynx-js/template-webpack-plugin@0.10.3 ## @lynx-js/testing-environment@0.1.11 ### Patch Changes - Remove element api calls alog by default, and only enable it when `__ALOG_ELEMENT_API__` is defined to `true` or environment variable `REACT_ALOG_ELEMENT_API` is set to `true`. ([#2192](#2192)) ## @lynx-js/web-constants@0.19.8 ### Patch Changes - Updated dependencies \[]: - @lynx-js/web-worker-rpc@0.19.8 ## @lynx-js/web-core@0.19.8 ### Patch Changes - fix: avoid error when LynxView is removed immediately after connected ([#2182](#2182)) - Updated dependencies \[]: - @lynx-js/web-constants@0.19.8 - @lynx-js/web-mainthread-apis@0.19.8 - @lynx-js/web-worker-rpc@0.19.8 - @lynx-js/web-worker-runtime@0.19.8 ## @lynx-js/web-core-wasm@0.0.3 ### Patch Changes - Updated dependencies \[[`e0972ef`](e0972ef), [`3bc017e`](3bc017e), [`e2d349e`](e2d349e), [`d924b6a`](d924b6a)]: - @lynx-js/web-elements@0.11.2 - @lynx-js/web-worker-rpc@0.19.8 ## @lynx-js/web-elements@0.11.2 ### Patch Changes - Add scrollHeight/scrollWidth getters to XList. ([#2156](#2156)) - Inherit padding styles for x-input elements. ([#2199](#2199)) - Remove the default lazy-loading attribute from x-image elements. ([#2186](#2186)) - Fix x-input number type forwarding to the inner input element. ([#2193](#2193)) ## @lynx-js/web-mainthread-apis@0.19.8 ### Patch Changes - Updated dependencies \[]: - @lynx-js/web-constants@0.19.8 ## @lynx-js/web-worker-runtime@0.19.8 ### Patch Changes - Updated dependencies \[]: - @lynx-js/web-constants@0.19.8 - @lynx-js/web-mainthread-apis@0.19.8 - @lynx-js/web-worker-rpc@0.19.8 ## @lynx-js/react-webpack-plugin@0.7.4 ### Patch Changes - Remove element api calls alog by default, and only enable it when `__ALOG_ELEMENT_API__` is defined to `true` or environment variable `REACT_ALOG_ELEMENT_API` is set to `true`. ([#2192](#2192)) ## create-rspeedy@0.13.4 ## @lynx-js/react-alias-rsbuild-plugin@0.12.8 ## upgrade-rspeedy@0.13.4 ## @lynx-js/web-core-server@0.19.8 ## @lynx-js/web-rsbuild-server-middleware@0.19.8 ## @lynx-js/web-worker-rpc@0.19.8 Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

feat: remove element api calls alog by default

0c71204

upupming requested review from HuJean, colinaaa, hzy and luhc228 as code owners February 4, 2026 03:55

coderabbitai bot added the framework:React label Feb 4, 2026

HuJean approved these changes Feb 4, 2026

View reviewed changes

upupming enabled auto-merge (squash) February 4, 2026 04:14

luhc228 approved these changes Feb 4, 2026

View reviewed changes

upupming merged commit 4240138 into main Feb 4, 2026
46 of 49 checks passed

upupming deleted the feat/REACT_ALOG_ELEMENT_API branch February 4, 2026 04:39

github-actions bot mentioned this pull request Feb 4, 2026

chore: Release 2026-02-21 04:23:50 #2183

Merged

coderabbitai bot mentioned this pull request Mar 10, 2026

feat: allow enable __DEV__ on NODE_ENV === 'development' #2324

Merged

3 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: remove element api calls alog by default#2192

feat: remove element api calls alog by default#2192
upupming merged 1 commit intomainfrom
feat/REACT_ALOG_ELEMENT_API

upupming commented Feb 4, 2026 •

edited by coderabbitai bot

Loading

Uh oh!

changeset-bot bot commented Feb 4, 2026

Uh oh!

coderabbitai bot commented Feb 4, 2026 •

edited

Loading

Walkthrough

Changes

Estimated code review effort

Possibly related PRs

Suggested labels

Suggested reviewers

Poem

Uh oh!

codecov bot commented Feb 4, 2026 •

edited

Loading

Uh oh!

codspeed-hq bot commented Feb 4, 2026

Uh oh!

relativeci bot commented Feb 4, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

upupming commented Feb 4, 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 Feb 4, 2026

🦋 Changeset detected

Uh oh!

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

Uh oh!

Walkthrough

Changes

Estimated code review effort

Possibly related PRs

Suggested labels

Suggested reviewers

Poem

Uh oh!

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

Uh oh!

Codecov Report

Uh oh!

codspeed-hq bot commented Feb 4, 2026

CodSpeed Performance Report

Merging this PR will degrade performance by 7.79%

Summary

Performance Changes

Footnotes

Uh oh!

relativeci bot commented Feb 4, 2026

Web Explorer

#7534 Bundle Size — 383.66KiB (0%).

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

upupming commented Feb 4, 2026 •

edited by coderabbitai bot

Loading

coderabbitai bot commented Feb 4, 2026 •

edited

Loading

codecov bot commented Feb 4, 2026 •

edited

Loading