fix(web): mts && bts events can be binded both

[Sherry-hue]

Summary by CodeRabbit

• Bug Fixes

  • Fixed support for simultaneously binding main-thread and bind events to the same element.

• Tests

  • Added test coverage for concurrent main-thread and bind tap event binding and handler execution.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

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: ec544da

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

This PR includes changesets to release 11 packages

Name	Type
@lynx-js/web-mainthread-apis	Patch
@lynx-js/web-core-server	Patch
@lynx-js/web-core	Patch
@lynx-js/web-worker-runtime	Patch
upgrade-rspeedy	Patch
@lynx-js/web-rsbuild-server-middleware	Patch
@lynx-js/web-constants	Patch
@lynx-js/web-worker-rpc	Patch
@lynx-js/rspeedy	Patch
@lynx-js/web-core-wasm	Patch
create-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]

📝 Walkthrough

Walkthrough

This PR refactors event handler storage in the main thread APIs to support multiple handlers per event per phase, replacing single handler references with handler arrays. It includes a new test verifying simultaneous bindtap event binding behavior across both MTS and BTS.

Changes

Cohort / File(s)	Summary
Event handler refactoring packages/web-platform/web-mainthread-apis/ts/createMainThreadGlobalThis.ts	Refactored event handler storage from single hname path to arrays of handlerInfo objects. Updated dispatch logic to iterate all handlers, execute worklets when present, and signal stopPropagation based on handler count. Modified __AddEvent to manage handler lists (capture/bind) as arrays and normalize/merge new handlers into appropriate lists.
Test infrastructure .changeset/social-deer-brush.md	Added changeset file documenting patch release for @lynx-js/web-mainthread-apis with fix for simultaneous MTS and BTS event binding.
Test suite expansion packages/web-platform/web-tests/tests/react.spec.ts, packages/web-platform/web-tests/tests/react/basic-bindtap-simultaneous/index.jsx	Added basic-bindtap-simultaneous test case and corresponding React component to verify simultaneous bindtap binding on target elements with assertions for click state, styling, and status display.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~28 minutes

Possibly related PRs

• feat: move SSR hydrate essential info to the ssr attribute #1292: Modifies __AddEvent path to add SSR hooks for dehydration — shares event handler registration code modifications with this PR.
• fix(web): capture event listener should be called before bind event l… #1972: Adjusts capture-vs-bind wrapper selection and phase checks in event dispatch — overlaps with multi-handler dispatch logic changes.

Suggested reviewers

• PupilTong
• colinaaa

Poem

🐰 Multiple handlers, unified at last,
No more single paths to hold steadfast!
Bindtap and MTS dance together in sync,
Each event finds room, without a blink—
One rabbit's refactor makes handlers bloom!

🚥 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 directly describes the main fix: enabling both MTS (main thread) and BTS (bind thread) events to be bound simultaneously, which is accurately reflected in the code changes.

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

✨ Finishing touches

• 📝 Generate docstrings

---

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

[coderabbitai]

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In @.changeset/social-deer-brush.md:
- Line 5: Update the changeset message text "fix: mts && bts events can be
binded both" to use the correct past participle by replacing "binded" with
"bound" so it reads "fix: mts && bts events can be bound both"; locate and edit
that exact string in .changeset/social-deer-brush.md.

🧹 Nitpick comments (2)

packages/web-platform/web-mainthread-apis/ts/createMainThreadGlobalThis.ts (1)

358-364: Consider simplifying array initialization.

The array normalization at lines 362-364 handles cases where targetList might not be an array. However, since handler lists are now always stored as arrays (or undefined), this defensive normalization suggests potential data inconsistency concerns.

If this is for backward compatibility with existing stored data, consider adding a brief comment. Otherwise, the initialization could be simplified:

♻️ Suggested simplification

      let targetList = (isCapture
        ? runtimeInfo.eventHandlerMap[eventName]!.capture
        : runtimeInfo.eventHandlerMap[eventName]!.bind) as unknown as any[];

-      if (!Array.isArray(targetList)) {
-        targetList = targetList ? [targetList] : [];
-      }
+      // Initialize as empty array if undefined
+      targetList = targetList ?? [];

packages/web-platform/web-tests/tests/react/basic-bindtap-simultaneous/index.jsx (1)

11-15: Consider adding a verification assertion for MTS console output.

The handleMtsTap handler logs "MTS Clicked" to the console, but the test doesn't verify this output. While the data-mts-clicked attribute check confirms the handler ran, adding a console listener assertion (similar to existing tests like basic-mts-bindtap) would provide additional confidence that the MTS path executed correctly.

This is optional since the attribute check is sufficient proof of handler execution.

[codecov]

Codecov Report

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

Files with missing lines	Patch %	Lines
...b-mainthread-apis/ts/createMainThreadGlobalThis.ts	0.00%	92 Missing ⚠️

📢 Thoughts on this report? Let us know!

[codspeed-hq]

Merging this PR will degrade performance by 5.46%

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

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

Performance Changes

	Benchmark	BASE	HEAD	Efficiency
⚡	transform 1000 view elements	45.3 ms	42 ms	+7.82%
❌	basic-performance-scroll-view-100	10.6 ms	11.2 ms	-5.46%

---

_{Comparing Sherry-hue:feat/mts-bts-both (ec544da) with main (2bc3e7e)}

Footnotes

1. 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]

Web Explorer

#7261 Bundle Size — 383.63KiB (+0.1%).

ec544da(current) vs 2bc3e7e main#7259(baseline)

Bundle metrics   3 changes

	Current #7261	Baseline #7259
Initial JS	154.03KiB	154.03KiB
Initial CSS	35.05KiB	35.05KiB
Cache Invalidation	7.36%	10.98%
Chunks	8	8
Assets	8	8
Modules	239(+0.42%)	238
Duplicate Modules	16	16
Duplicate Code	2.99%(-0.33%)	3%
Packages	4	4
Duplicate Packages	0	0

Bundle size by type   1 change 1 regression

	Current #7261	Baseline #7259
JS	251.56KiB (+0.16%)	251.17KiB
Other	97.02KiB	97.02KiB
CSS	35.05KiB	35.05KiB

Bundle analysis report Branch Sherry-hue:feat/mts-bts-both Project dashboard

---

^{Generated by RelativeCI Documentation Report issue}