fix(create-rspeedy): use correct source by colinaaa · Pull Request #2427 · lynx-family/lynx-stack

colinaaa · 2026-04-07T04:00:36Z

Summary by CodeRabbit

Bug Fixes
- Fixed an error occurring when installing the Lynx DevTool skill during project scaffolding.

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-04-07T04:00:43Z

🦋 Changeset detected

Latest commit: e72a4f3

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

This PR includes changesets to release 3 packages

Name	Type
create-rspeedy	Patch
@lynx-js/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-04-07T04:00:50Z

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 6e7bb1c7-d7f4-4f4f-a5ed-8d750f049f53

📥 Commits

Reviewing files that changed from the base of the PR and between 55602e2 and e72a4f3.

📒 Files selected for processing (3)

.changeset/fruity-rules-agree.md
.github/workflows/test.yml
packages/rspeedy/create-rspeedy/src/index.ts

📝 Walkthrough

Walkthrough

This PR fixes an installation error with the Lynx DevTool skill in create-rspeedy by reformatting the skill configuration from an array-based structure to a source/value pair format, adding test coverage for the skill installation, and releasing a patch version.

Changes

Cohort / File(s)	Summary
Lynx DevTool Skill Configuration `packages/rspeedy/create-rspeedy/src/index.ts`	Refactored the extraSkills configuration for "Lynx DevTool" from `skills: ['lynx-community/skills/lynx-devtool']` to `source: 'lynx-community/skills'` and `value: 'lynx-devtool'`, restructuring how the skill identifier is specified.
Test Coverage `.github/workflows/test.yml`	Extended the `test-publish` workflow to include `--skill lynx-devtool` when scaffolding the `create-rspeedy-regression` test project.
Release Metadata `.changeset/fruity-rules-agree.md`	Added changeset entry marking `create-rspeedy` for a patch release with message addressing the Lynx DevTool skill installation error fix.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

feat(create-rspeedy): add optional Lynx DevTool skill #2421 — Introduced the original Lynx DevTool skill entry to extraSkills; this PR adjusts its configuration format from an array to a source/value pair structure.

Suggested reviewers

upupming
luhc228

Poem

🐰 A skill that stumbled, now stands tall,
From lynx-community, we heed the call,
Source and value, a cleaner way,
DevTools shine bright, hip-hip-hooray! 🌟

🚥 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 accurately summarizes the main change: fixing the Lynx DevTool skill configuration by updating the source specification in create-rspeedy from an inline skills array to a source/value format.
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

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-04-07T04:03:05Z

Codecov Report

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

Files with missing lines	Patch %	Lines
packages/rspeedy/create-rspeedy/src/index.ts	0.00%	2 Missing ⚠️

📢 Thoughts on this report? Let us know!

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: e72a4f3f7f

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

chatgpt-codex-connector · 2026-04-07T04:05:24Z

.github/workflows/test.yml

        pnpm --recursive publish --no-git-checks --access public --registry=http://localhost:4873
        cd `mktemp -d`
-        npx --registry http://localhost:4873 create-rspeedy-canary@latest --template react --dir create-rspeedy-regression --tools eslint
+        npx --registry http://localhost:4873 create-rspeedy-canary@latest --template react --dir create-rspeedy-regression --tools eslint --skill lynx-devtool


Keep skill install off the mock npm registry

In .github/workflows/test.yml (test-publish job), adding --skill lynx-devtool to the npx --registry http://localhost:4873 create-rspeedy-canary@latest ... invocation causes create-rspeedy to execute its skill installer path, and create-rstack runs that via npx skills add ... without overriding registry. Because npm_config_registry from the outer npx --registry is inherited by child processes, npx skills is resolved against the mock Verdaccio registry (which only contains the published canary packages), so skill installation fails before the regression project can build/lint.

Useful? React with 👍 / 👎.

codspeed-hq · 2026-04-07T04:16:58Z

Merging this PR will not alter performance

✅ 72 untouched benchmarks
⏩ 21 skipped benchmarks¹

_{Comparing colinaaa:colin/0407/cra-skill (e72a4f3) with main (55602e2)}

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 · 2026-04-07T04:18:43Z

React Example

#7086 Bundle Size — 237.81KiB (0%).

e72a4f3(current) vs 55602e2 main#7082(baseline)

Bundle metrics no changes

	Current #7086	Baseline #7082
Initial JS	`0B`	`0B`
Initial CSS	`0B`	`0B`
Cache Invalidation	`0%`	`0%`
Chunks	`0`	`0`
Assets	`4`	`4`
Modules	`180`	`180`
Duplicate Modules	`71`	`71`
Duplicate Code	`46.39%`	`46.39%`
Packages	`2`	`2`
Duplicate Packages	`0`	`0`

Bundle size by type no changes

	Current #7086	Baseline #7082
IMG	`145.76KiB`	`145.76KiB`
Other	`92.05KiB`	`92.05KiB`

Bundle analysis report Branch colinaaa:colin/0407/cra-skill Project dashboard

^{Generated by RelativeCI Documentation Report issue}

relativeci · 2026-04-07T04:18:50Z

React MTF Example

#219 Bundle Size — 207.38KiB (0%).

e72a4f3(current) vs 55602e2 main#215(baseline)

Bundle metrics no changes

	Current #219	Baseline #215
Initial JS	`0B`	`0B`
Initial CSS	`0B`	`0B`
Cache Invalidation	`0%`	`0%`
Chunks	`0`	`0`
Assets	`3`	`3`
Modules	`174`	`174`
Duplicate Modules	`68`	`68`
Duplicate Code	`46.08%`	`46.08%`
Packages	`2`	`2`
Duplicate Packages	`0`	`0`

Bundle size by type no changes

	Current #219	Baseline #215
IMG	`111.23KiB`	`111.23KiB`
Other	`96.15KiB`	`96.15KiB`

Bundle analysis report Branch colinaaa:colin/0407/cra-skill Project dashboard

^{Generated by RelativeCI Documentation Report issue}

relativeci · 2026-04-07T04:18:53Z

Web Explorer

#8661 Bundle Size — 728.84KiB (0%).

e72a4f3(current) vs 55602e2 main#8657(baseline)

Bundle metrics

2 changes

	Current #8661	Baseline #8657
Initial JS	`43.31KiB`	`43.31KiB`
Initial CSS	`2.16KiB`	`2.16KiB`
Cache Invalidation	`0%`	`0%`
Chunks	`8`	`8`
Assets	`10`	`10`
Modules	`148`(`-0.67%`)	`149`
Duplicate Modules	`11`	`11`
Duplicate Code	`34.7%`(`+0.06%`)	`34.68%`
Packages	`3`	`3`
Duplicate Packages	`0`	`0`

Bundle size by type no changes

	Current #8661	Baseline #8657
Other	`384.62KiB`	`384.62KiB`
JS	`342.07KiB`	`342.07KiB`
CSS	`2.16KiB`	`2.16KiB`

Bundle analysis report Branch colinaaa:colin/0407/cra-skill Project dashboard

^{Generated by RelativeCI Documentation Report issue}

relativeci · 2026-04-07T04:21:53Z

React External

#205 Bundle Size — 591.44KiB (0%).

e72a4f3(current) vs 55602e2 main#201(baseline)

Bundle metrics no changes

	Current #205	Baseline #201
Initial JS	`0B`	`0B`
Initial CSS	`0B`	`0B`
Cache Invalidation	`0%`	`0%`
Chunks	`0`	`0`
Assets	`3`	`3`
Modules	`17`	`17`
Duplicate Modules	`5`	`5`
Duplicate Code	`8.59%`	`8.59%`
Packages	`0`	`0`
Duplicate Packages	`0`	`0`

Bundle size by type no changes

	Current #205	Baseline #201
Other	`591.44KiB`	`591.44KiB`

Bundle analysis report Branch colinaaa:colin/0407/cra-skill Project dashboard

^{Generated by RelativeCI Documentation Report issue}

fix(create-rspeedy): use correct source

e72a4f3

colinaaa requested review from luhc228 and upupming as code owners April 7, 2026 04:00

chatgpt-codex-connector bot reviewed Apr 7, 2026

View reviewed changes

colinaaa merged commit 8c0e483 into lynx-family:main Apr 7, 2026
48 of 50 checks passed

colinaaa deleted the colin/0407/cra-skill branch April 7, 2026 06:04

github-actions bot mentioned this pull request Apr 7, 2026

chore: Release 2026-04-10 04:17:43 #2430

Open

Conversation

colinaaa commented Apr 7, 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 Apr 7, 2026

🦋 Changeset detected

Uh oh!

coderabbitai bot commented Apr 7, 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!

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

Uh oh!

Codecov Report

Uh oh!

chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

💡 Codex Review

Uh oh!

chatgpt-codex-connector bot Apr 7, 2026

Choose a reason for hiding this comment

Uh oh!

codspeed-hq bot commented Apr 7, 2026

Merging this PR will not alter performance

Footnotes

Uh oh!

relativeci bot commented Apr 7, 2026

React Example

#7086 Bundle Size — 237.81KiB (0%).

Uh oh!

relativeci bot commented Apr 7, 2026

React MTF Example

#219 Bundle Size — 207.38KiB (0%).

Uh oh!

relativeci bot commented Apr 7, 2026

Web Explorer

#8661 Bundle Size — 728.84KiB (0%).

Uh oh!

relativeci bot commented Apr 7, 2026

React External

#205 Bundle Size — 591.44KiB (0%).

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

colinaaa commented Apr 7, 2026 •

edited by coderabbitai bot

Loading

coderabbitai bot commented Apr 7, 2026 •

edited

Loading

codecov bot commented Apr 7, 2026 •

edited

Loading