Skip to content

chore(deps): tighten bundler peer/dependency declarations - #159

Merged
chihab merged 2 commits into
mainfrom
claude/deps-hygiene
Aug 13, 2026
Merged

chihab merged 2 commits into
mainfrom
claude/deps-hygiene

Conversation

@chihab

@chihab chihab commented Aug 9, 2026

Copy link
Copy Markdown
Owner

Dependency-hygiene pass across the bundler integrations. Each import was checked against source before reclassifying.

Changes

  • @dotenv-run/vite — move vite from a runtime dependency to a peerDependency (+ keep it as a devDependency). A bundler plugin must not ship its own copy of the bundler: the previous setup risked a duplicate Vite instance and version conflicts with the host project. src/index.ts only imports the Plugin type from Vite, which the consumer always provides.
    • peer range: ^5.0.0 || ^6.0.0 || ^7.0.0
  • @dotenv-run/rollup — widen the rollup peer range from ^3.0.0 to ^3.0.0 || ^4.0.0 so Rollup 4 consumers are supported (the package imports only the Plugin type).
  • @dotenv-run/esbuild — align the @dotenv-run/core range to ^1.3.7 (was ~1.3.7) for consistency with every other package.

Not changed (checked, no action needed)

  • @dotenv-run/jest — jest-env.mjs is a self-contained transformer (import.meta.env = process.env); it does not import @dotenv-run/core, so no dependency is missing.

Verification

  • pnpm install + pnpm build for the three affected packages — all green.
  • @dotenv-run/esbuild unit tests pass.

A changeset (patch for all three) is included so the release automation picks these up.

Split out of the Angular 22 work in #158 to keep that PR focused; this touches unrelated packages and triggers their own releases.


Generated by Claude Code

claude added 2 commits August 9, 2026 00:01
- @dotenv-run/vite: move `vite` from dependencies to peerDependencies (+ dev).
  A bundler plugin must not bundle its own bundler; the plugin only imports the
  `Plugin` type, which the host always provides. Prevents duplicate Vite
  instances / version conflicts.
- @dotenv-run/rollup: widen the `rollup` peer range to `^3.0.0 || ^4.0.0` to
  support Rollup 4 consumers.
- @dotenv-run/esbuild: align `@dotenv-run/core` to `^1.3.7` for consistency with
  the other packages.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@chihab
chihab merged commit a95e7d7 into main Aug 13, 2026
3 checks 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.

2 participants