Skip to content

fix: add missing path-posix workspace dependency#1382

Closed
Han5991 wants to merge 1 commit intovoidzero-dev:mainfrom
Han5991:fix/missing-path-posix-workspace-dep
Closed

fix: add missing path-posix workspace dependency#1382
Han5991 wants to merge 1 commit intovoidzero-dev:mainfrom
Han5991:fix/missing-path-posix-workspace-dep

Conversation

@Han5991
Copy link
Copy Markdown
Contributor

@Han5991 Han5991 commented Apr 14, 2026

Summary

  • rolldown_plugin_vite_import_glob in the rolldown upgrade (feat(deps): upgrade upstream dependencies #1375) added path-posix as a workspace dependency
  • It was not added to the root Cargo.toml, causing cargo build to fail after just init
  • Adds path-posix = "0.0.1" to [workspace.dependencies] in root Cargo.toml

Fixes #1384

Test plan

  • Run just init then cargo check — should pass without errors

🤖 Generated with Claude Code

rolldown upgrade in voidzero-dev#1375 introduced \`path-posix\` in
\`rolldown_plugin_vite_import_glob\` but it was not added to the root
workspace dependencies, causing \`cargo build\` to fail after \`just init\`.

Fixes voidzero-dev#1381

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@netlify
Copy link
Copy Markdown

netlify bot commented Apr 14, 2026

Deploy Preview for viteplus-preview canceled.

Name Link
🔨 Latest commit c509e31
🔍 Latest deploy log https://app.netlify.com/projects/viteplus-preview/deploys/69de5e8fa05d6400081b21f8

Copy link
Copy Markdown

Rechecked this after aligning the local upstream state with CI.

The path-posix change was only needed when rolldown/ had drifted to a newer commit that was not the pinned upstream revision. On the pinned/tagged upstream revision, rolldown no longer requires path-posix, cargo build --locked succeeds without it, and cargo-shear correctly reports it as unused when added.

So this PR is fixing a transient checkout drift rather than a real issue on the current pinned upstream state. Tag-based pinning looks stable here; following newer upstream commits is what reintroduced the breakage.

Closing this PR.

@Han5991 Han5991 closed this Apr 15, 2026
@Han5991 Han5991 deleted the fix/missing-path-posix-workspace-dep branch April 15, 2026 16:49
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.

fix: add missing path-posix workspace dependency after rolldown upgrade (#1375)

2 participants