Skip to content

sui-bundler: exclude packages with script.dev#1975

Merged
paulusrex merged 2 commits intomasterfrom
feat/sui-bundler-exclude-packages-dev-script
Mar 27, 2026
Merged

sui-bundler: exclude packages with script.dev#1975
paulusrex merged 2 commits intomasterfrom
feat/sui-bundler-exclude-packages-dev-script

Conversation

@paulusrex
Copy link
Copy Markdown
Contributor

Description

When using --link-package or --link-all, sui-bundler now automatically excludes packages that define a dev script in their package.json. These packages are assumed to have their own file watcher/compiler running (e.g. tsup --watch, rollup -w), so linking them via sui-bundler would be redundant and could cause conflicts.

A log message is printed to inform the user when a package is skipped for this reason.

README has been updated to document this behavior, including a note about using npm link directly for packages outside the monorepo.

Related Issue

N/A

Example

Given a monorepo with these packages:

/my-monorepo/packages/
  my-lib/          # has "dev": "tsc --watch"  → skipped by sui-bundler
  my-components/   # no dev script             → linked normally
$ sui-bundler dev --link-all ../my-monorepo/packages
# ✔ my-components from path "..."
# ℹ Package from path ".../my-lib" wouldn't be linked because it has its own watcher.

To link my-lib during development, run its watcher separately:

# in /my-monorepo/packages/my-lib
npm run dev

If the package is not part of the monorepo, use npm link directly instead.

@paulusrex paulusrex force-pushed the feat/sui-bundler-exclude-packages-dev-script branch from 428664b to c857795 Compare March 26, 2026 20:20
@paulusrex paulusrex merged commit 3e2b8c5 into master Mar 27, 2026
2 checks passed
@paulusrex paulusrex deleted the feat/sui-bundler-exclude-packages-dev-script branch March 27, 2026 07:22
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.

3 participants