Skip to content

feat(cli): bootstrap rust addon - #27

Merged
wan9chi merged 1 commit into
mainfrom
07-18-feat_cli_bootstrap_rust_addon
Jul 21, 2025
Merged

feat(cli): bootstrap rust addon#27
wan9chi merged 1 commit into
mainfrom
07-18-feat_cli_bootstrap_rust_addon

Conversation

@Brooooooklyn

Copy link
Copy Markdown
Member

No description provided.

@Brooooooklyn
Brooooooklyn requested a review from wan9chi July 21, 2025 10:56

@wan9chi wan9chi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some minor comments

Comment thread crates/vite_task/src/execute.rs
Comment thread packages/cli/index.js
} else if (process.platform === 'freebsd') {
if (process.arch === 'x64') {
try {
return require('./vite-plus.freebsd-x64.node');

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see only macOS/Windows/Linux targets are listed in napi.targets in package.json.

Is it expected to see freebsd and other unlisted OSes/arches here?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The binding template contains all NAPI-RS supported platform/cpu combination

@wan9chi
wan9chi merged commit f08996f into main Jul 21, 2025
@wan9chi
wan9chi deleted the 07-18-feat_cli_bootstrap_rust_addon branch July 21, 2025 11:37
fengmk2 added a commit that referenced this pull request Jul 1, 2026
When migration keeps an opt-in browser provider (@vitest/browser-playwright /
@vitest/browser-webdriverio) and adds its runtime framework peer, it wrote
`playwright`/`webdriverio` as `*` (non-deterministic). Reuse a declared lockstep
sibling's version instead (e.g. @playwright/test for playwright), resolving a
`catalog:` sibling to its concrete value, referencing the catalog when it
already owns the peer, and falling back to `*` only when nothing resolves
(npmx.dev #27).

Also extracts a shared findDeclaredSpec helper (the four-field dependency scan)
and drops an unreachable peer branch in the REMOVE_PACKAGES loop.

Claude-Session: https://claude.ai/code/session_01DQhS6o1fyQd1yjiee6W8jR
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants