Skip to content

SMOODEV-tsdown: drop unused tsup root devDependency - #9

Merged
brentrager merged 2 commits into
mainfrom
smoodev-tsdown-cleanup-root-devdep
May 18, 2026
Merged

SMOODEV-tsdown: drop unused tsup root devDependency#9
brentrager merged 2 commits into
mainfrom
smoodev-tsdown-cleanup-root-devdep

Conversation

@brentrager

Copy link
Copy Markdown
Contributor

Summary

The observability monorepo already builds with tsdown (see `packages/core/tsdown.config.ts`); the root `tsup` devDependency was a leftover from before the migration and is unreferenced by any build script. Dropping it cleans up the install graph.

Test plan

  • `pnpm install` clean
  • `pnpm build` clean
  • CI green

🤖 Generated with Claude Code

The package already builds with tsdown — tsup was a leftover root devDep
not referenced by any build script. Removing it cleans up the install
graph. No functional change.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@changeset-bot

changeset-bot Bot commented May 18, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: b9a4d5a

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

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

💥 An error occurred when fetching the changed packages and changesets in this PR
Some errors occurred when validating the changesets config:
The package or glob expression "@smooai/observability-react" specified in the `linked` option does not match any package in the project. You may have misspelled the package name or provided an invalid glob expression. Note that glob expressions must be defined according to https://www.npmjs.com/package/micromatch.
The package or glob expression "@smooai/observability-next" specified in the `linked` option does not match any package in the project. You may have misspelled the package name or provided an invalid glob expression. Note that glob expressions must be defined according to https://www.npmjs.com/package/micromatch.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@brentrager
brentrager merged commit 2984514 into main May 18, 2026
1 check passed
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.

1 participant