fix: ci reusable workflow name by Dugyu · Pull Request #2290 · lynx-family/lynx-stack

Dugyu · 2026-03-02T18:42:07Z

Summary by CodeRabbit

Chores
- Updated GitHub Actions workflows for improved CI/CD automation and testing processes.

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

Add name: Benchmark, Build, and Reusable Test to workflow-bench.yml, workflow-build.yml, and workflow-test.yml respectively, so they are clearly identifiable in the GitHub Actions UI. https://claude.ai/code/session_01GUDHcoJCmuFtj9HAxot7Pf

Add name: Benchmark, Build, and Reusable Test to `workflow-bench.yml`, `workflow-build.yml`, and `workflow-test.yml` respectively, so they are clearly identifiable in the GitHub Actions UI.

changeset-bot · 2026-03-02T18:42:17Z

⚠️ No Changeset found

Latest commit: 2c3e6bf

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 · 2026-03-02T18:42:24Z

📝 Walkthrough

Walkthrough

Adds workflow names to three GitHub Actions workflow files: "Benchmark" to workflow-bench.yml, "Build" to workflow-build.yml with CODECOV_TOKEN secret, and introduces a new comprehensive reusable workflow in workflow-test.yml with secrets, inputs, permissions, environment variables, and build/test steps.

Changes

Cohort / File(s)	Summary
GitHub Actions Workflows `.github/workflows/workflow-bench.yml`, `.github/workflows/workflow-build.yml`, `.github/workflows/workflow-test.yml`	Adds top-level workflow names and introduces reusable test workflow with CODECOV_TOKEN and LHCI_GITHUB_APP_TOKEN secrets, configurable inputs (runs-on, run, is-web, web-report-name, web-report-path, codecov-flags), environment variables, container configuration, and complete build/test/upload pipeline steps.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

fix(benchx_cli): resolve upstream conflict #2221: Modifies workflow-bench.yml by changing the runs-on runner image, directly related to the workflow configuration changes in this PR.

Suggested reviewers

colinaaa
HuJean

Poem

🐰 GitHub Actions now run clean with names so bright,
Workflows orchestrate our tests throughout the night,
Codecov tokens flow, and secrets stay secure,
Our CI/CD pipeline, steady and pure! ✨
Reusable, composable, our builds take flight!

🚥 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 'fix: ci reusable workflow name' accurately describes the main change—adding explicit name fields to reusable GitHub Actions workflows for better UI identification.
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
Post copyable unit tests in a comment

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

cla-assistant · 2026-03-02T18:42:26Z

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ Dugyu
❌ claude
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

coderabbitai

🧹 Nitpick comments (1)

.github/workflows/workflow-test.yml (1)
102-102: Minor: Inconsistent spacing in expression.

The expression ${{ inputs.web-report-path}} is missing a space before the closing braces, unlike other expressions in this file (e.g., line 101 has ${{ inputs.web-report-name }}).
✏️ Suggested fix
-          path: ${{ inputs.web-report-path}}
+          path: ${{ inputs.web-report-path }}
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In @.github/workflows/workflow-test.yml at line 102, The expression for the
workflow input path is missing a space before the closing braces which makes it
inconsistent with other expressions; update the expression `${{
inputs.web-report-path}}` to include a space before the closing braces so it
reads like `${{ inputs.web-report-path }}` to match the style used for `${{
inputs.web-report-name }}` and other placeholders.

🤖 Prompt for all review comments with AI agents

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

Nitpick comments:
In @.github/workflows/workflow-test.yml:
- Line 102: The expression for the workflow input path is missing a space before
the closing braces which makes it inconsistent with other expressions; update
the expression `${{ inputs.web-report-path}}` to include a space before the
closing braces so it reads like `${{ inputs.web-report-path }}` to match the
style used for `${{ inputs.web-report-name }}` and other placeholders.

ℹ️ Review info

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 8ca97fe and 2c3e6bf.

📒 Files selected for processing (3)

.github/workflows/workflow-bench.yml
.github/workflows/workflow-build.yml
.github/workflows/workflow-test.yml

codecov · 2026-03-02T18:44:52Z

❌ 1 Tests Failed:

Tests completed	Failed	Passed	Skipped
4789	1	4788	273

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

packages/rspeedy/plugin-qrcode/test/shortcuts.test.ts > PluginQRCode - CLI Shortcuts > open page

Stack Traces | 1.82s run time

Error: Matcher did not succeed in time.
 ❯ test/shortcuts.test.ts:163:40

Caused by: Caused by: Caused by: AssertionError: expected "Xe" to be called 2 times, but got 0 times

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 · 2026-03-02T18:56:53Z

Merging this PR will improve performance by 9.41%

⚡ 3 improved benchmarks
✅ 69 untouched benchmarks
⏩ 3 skipped benchmarks¹

Performance Changes

	Benchmark	`BASE`	`HEAD`	Efficiency
⚡	`basic-performance-div-10000`	485.4 ms	457.4 ms	+6.11%
⚡	`basic-performance-div-100`	7.4 ms	6.9 ms	+6%
⚡	`transform 1000 view elements`	47.4 ms	43.3 ms	+9.41%

_{Comparing Dugyu:fix/ci-workflow-name (2c3e6bf) with main (8ca97fe)}

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-02T18:58:52Z

Web Explorer

#7889 Bundle Size — 383.64KiB (0%).

2c3e6bf(current) vs 8ca97fe main#7876(baseline)

Bundle metrics

1 change

	Current #7889	Baseline #7876
Initial JS	`154.78KiB`	`154.78KiB`
Initial CSS	`35.06KiB`	`35.06KiB`
Cache Invalidation	`0%`	`0%`
Chunks	`8`	`8`
Assets	`8`	`8`
Modules	`238`(`-0.42%`)	`239`
Duplicate Modules	`16`	`16`
Duplicate Code	`2.99%`	`2.99%`
Packages	`4`	`4`
Duplicate Packages	`0`	`0`

Bundle size by type no changes

	Current #7889	Baseline #7876
JS	`252.72KiB`	`252.72KiB`
Other	`95.85KiB`	`95.85KiB`
CSS	`35.06KiB`	`35.06KiB`

Bundle analysis report Branch Dugyu:fix/ci-workflow-name Project dashboard

^{Generated by RelativeCI Documentation Report issue}

claude and others added 2 commits March 2, 2026 18:34

ci: add name fields to reusable workflow files

07eb767

Add name: Benchmark, Build, and Reusable Test to workflow-bench.yml, workflow-build.yml, and workflow-test.yml respectively, so they are clearly identifiable in the GitHub Actions UI. https://claude.ai/code/session_01GUDHcoJCmuFtj9HAxot7Pf

ci: add name fields to reusable workflow files

2c3e6bf

Add name: Benchmark, Build, and Reusable Test to `workflow-bench.yml`, `workflow-build.yml`, and `workflow-test.yml` respectively, so they are clearly identifiable in the GitHub Actions UI.

coderabbitai bot reviewed Mar 2, 2026

View reviewed changes

Dugyu closed this Mar 3, 2026

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix: ci reusable workflow name#2290

fix: ci reusable workflow name#2290
Dugyu wants to merge 2 commits intolynx-family:mainfrom
Dugyu:fix/ci-workflow-name

Dugyu commented Mar 2, 2026 •

edited by coderabbitai bot

Loading

Uh oh!

changeset-bot bot commented Mar 2, 2026

Uh oh!

coderabbitai bot commented Mar 2, 2026 •

edited

Loading

Walkthrough

Changes

Estimated code review effort

Possibly related PRs

Suggested reviewers

Poem

Uh oh!

cla-assistant bot commented Mar 2, 2026

Uh oh!

coderabbitai bot left a comment

Uh oh!

codecov bot commented Mar 2, 2026 •

edited

Loading

Uh oh!

codspeed-hq bot commented Mar 2, 2026

Uh oh!

relativeci bot commented Mar 2, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

Dugyu commented Mar 2, 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 2, 2026

⚠️ No Changeset found

Uh oh!

coderabbitai bot commented Mar 2, 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 reviewers

Poem

Uh oh!

cla-assistant bot commented Mar 2, 2026

Uh oh!

coderabbitai bot left a comment

Choose a reason for hiding this comment

Uh oh!

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

Uh oh!

❌ 1 Tests Failed:

Uh oh!

codspeed-hq bot commented Mar 2, 2026

Merging this PR will improve performance by 9.41%

Performance Changes

Footnotes

Uh oh!

relativeci bot commented Mar 2, 2026

Web Explorer

#7889 Bundle Size — 383.64KiB (0%).

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Dugyu commented Mar 2, 2026 •

edited by coderabbitai bot

Loading

coderabbitai bot commented Mar 2, 2026 •

edited

Loading

codecov bot commented Mar 2, 2026 •

edited

Loading