refactor: update build config for @primer/mcp to rolldown#8042
refactor: update build config for @primer/mcp to rolldown#8042joshblack wants to merge 6 commits into
Conversation
|
4851756 to
7553cf9
Compare
2598b0d to
a5e067c
Compare
|
7553cf9 to
f41afb4
Compare
a5e067c to
fa4e757
Compare
f41afb4 to
aede53d
Compare
fa4e757 to
2b55b7e
Compare
aede53d to
f4cf5de
Compare
2b55b7e to
d34560d
Compare
f4cf5de to
82804ca
Compare
d34560d to
3c87865
Compare
82804ca to
52d7193
Compare
3c87865 to
263a167
Compare
263a167 to
52051d7
Compare
|
🤖 Lint issues have been automatically fixed and committed to this PR. |
There was a problem hiding this comment.
Pull request overview
This PR migrates the @primer/mcp package build tooling from Rollup to Rolldown, aiming to keep the same published JS + type declaration outputs while removing Rollup-specific config and dependencies.
Changes:
- Removed the Rollup build configuration (
rollup.config.js). - Added a Rolldown multi-step build configuration to emit JS and
.d.tsoutputs (rolldown.config.js). - Updated
packages/mcp/package.jsonscripts and devDependencies to use Rolldown instead of Rollup.
Show a summary per file
| File | Description |
|---|---|
| packages/mcp/rollup.config.js | Removes the old Rollup-based build pipeline configuration. |
| packages/mcp/rolldown.config.js | Introduces Rolldown config for JS bundling + separate declaration generation. |
| packages/mcp/package.json | Switches build/watch scripts to Rolldown and replaces Rollup/Babel build deps with Rolldown deps. |
Copilot's findings
- Files reviewed: 3/4 changed files
- Comments generated: 3
a6f870a to
d0d3e65
Compare
Closes #
N/A
Changelog
New
N/A
Changed
@primer/mcpto build JavaScript and declarations with Rolldown.Removed
Rollout strategy
Testing & Reviewing
Validated as part of the stack with
npm run build,npm run type-check,npm run lint,npm run lint:css,npm run format:diff, andnpm test -- --run.Merge checklist