chore: fix web-core coverage report

[PupilTong]

Summary by CodeRabbit

• Chores
  • Enhanced continuous integration configuration to automatically collect test coverage metrics during the automated testing pipeline.

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]

⚠️ No Changeset found

Latest commit: 914b106

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

[coderabbitai]

📝 Walkthrough

Walkthrough

The pull request adds a PNPM command to the web-core-e2e CI job in the test workflow. After Playwright tests complete and the JUnit report is generated, the workflow now additionally runs pnpm --filter @lynx-js/web-core-e2e run coverage:ci to collect code coverage metrics.

Changes

Cohort / File(s)	Summary
CI Workflow Enhancement .github/workflows/test.yml	Added coverage collection step (pnpm --filter @lynx-js/web-core-e2e run coverage:ci) to run immediately after Playwright tests in the web-core-e2e job.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• chore: remove web-core json tests #2362: Modifies the same CI workflow to adjust package-specific coverage command execution timing and placement.
• ci: split playwright tests, preempt more CPUs #1706: Updates the Playwright CI workflow with changes to post-test execution steps including reporting and environment configuration.

Suggested labels

platform:Web

Suggested reviewers

• Sherry-hue
• colinaaa

Poem

🐰 A coverage command hops into place,
After tests sprint through the race,
PNPM gathers metrics with care,
Web-core's health? Now crystal clear! ✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main change: adding a coverage collection step to the web-core E2E test workflow to fix coverage report generation.
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 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

• X
• Mastodon
• Reddit
• LinkedIn

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

[codecov]

❌ 4 Tests Failed:

Tests completed	Failed	Passed	Skipped
4685	4	4681	77

View the top 3 failed test(s) by shortest run time

packages/rspeedy/plugin-react/test/config.test.ts > Config > transform configs > minify should remove thread-specific pure funcs from built outputs

Stack Traces | 0.124s run time

AssertionError: expected main-thread and background outputs to exist
 ❯ test/config.test.ts:2141:16

tests/lynx.spec.ts > kitten-lynx testing framework > can click

Stack Traces | 5.01s run time

Error: No response found for deviceId: emulator-5554 port: 8901
 ❯ Connector.#sendMessageWithTransport ../...../devtool-connector/src/index.ts:433:11
 ❯ Connector.sendCDPMessage ../...../devtool-connector/src/index.ts:210:12
 ❯ CDPChannel.send src/CDPChannel.ts:158:12
 ❯ KittenLynxView.locator src/KittenLynxView.ts:257:24
 ❯ tests/lynx.spec.ts:88:24

tests/lynx.spec.ts > kitten-lynx testing framework > can fetch computed style map of an element

Stack Traces | 5.01s run time

Error: No response found for deviceId: emulator-5554 port: 8901
 ❯ Connector.#sendMessageWithTransport ../...../devtool-connector/src/index.ts:433:11
 ❯ Connector.sendCDPMessage ../...../devtool-connector/src/index.ts:210:12
 ❯ CDPChannel.send src/CDPChannel.ts:158:12
 ❯ KittenLynxView.locator src/KittenLynxView.ts:257:24
 ❯ tests/lynx.spec.ts:77:26

tests/lynx.spec.ts > kitten-lynx testing framework > can get attributes from an element

Stack Traces | 5.01s run time

Error: No response found for deviceId: emulator-5554 port: 8901
 ❯ Connector.#sendMessageWithTransport ../...../devtool-connector/src/index.ts:433:11
 ❯ Connector.sendCDPMessage ../...../devtool-connector/src/index.ts:210:12
 ❯ CDPChannel.send src/CDPChannel.ts:158:12
 ❯ ElementNode.getAttribute src/ElementNode.ts:125:17
 ❯ tests/lynx.spec.ts:68:25

tests/lynx.spec.ts > kitten-lynx testing framework > can take a screenshot

Stack Traces | 50s run time

Error: Test timed out in 50000ms.
If this is a long-running test, pass a timeout value as the last argument or configure it globally with "testTimeout".
 ❯ tests/lynx.spec.ts:100:3

To view more test analytics, go to the Test Analytics Dashboard
_{📋 Got 3 mins? Take this short survey to help us improve Test Analytics.}

[codspeed-hq]

Merging this PR will degrade performance by 6.31%

⚡ 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
⚡	transform 1000 view elements	44.9 ms	40.4 ms	+11.27%
❌	basic-performance-scroll-view-100	8.4 ms	8.9 ms	-6.31%

---

_{Comparing PupilTong:p/hw/fix-coverage-report (914b106) with main (d162bda)}

Footnotes

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

Web Explorer

#8429 Bundle Size — 724.73KiB (0%).

914b106(current) vs 6d71e3e main#8428(baseline)

Bundle metrics  no changes

	Current #8429	Baseline #8428
Initial JS	41.99KiB	41.99KiB
Initial CSS	1.99KiB	1.99KiB
Cache Invalidation	0%	0%
Chunks	8	8
Assets	10	10
Modules	149	149
Duplicate Modules	11	11
Duplicate Code	34.75%	34.75%
Packages	3	3
Duplicate Packages	0	0

Bundle size by type  no changes

	Current #8429	Baseline #8428
Other	382.38KiB	382.38KiB
JS	340.36KiB	340.36KiB
CSS	1.99KiB	1.99KiB

Bundle analysis report Branch PupilTong:p/hw/fix-coverage-repo... Project dashboard

---

^{Generated by RelativeCI Documentation Report issue}