Skip to content

feat: add rollup build pipeline with minification#73

Merged
gaureshpai merged 15 commits intomainfrom
feat/add-build-pipeline
Mar 21, 2026
Merged

feat: add rollup build pipeline with minification#73
gaureshpai merged 15 commits intomainfrom
feat/add-build-pipeline

Conversation

@gaureshpai
Copy link
Copy Markdown
Owner

@gaureshpai gaureshpai commented Mar 21, 2026

  • Integrate rollup with terser for minified builds
  • Update CI to build before tests
  • Update publish workflow to build before release
  • Exclude dist/ from git, publish only built output
  • Configure biome to ignore generated dist folder

Summary by CodeRabbit

  • Tests

    • Added a CI smoke test to verify the built CLI artifact
    • Added tests validating packaging/build configuration and updated test entrypoints to run the bundled artifact
  • Documentation

    • Cleaned and normalized formatting across docs and issue templates
  • Chores

    • Switched CLI to publish and run bundled distribution artifacts and added a bundling build step
    • CI/CD now runs a build step before testing and publishing
  • Repository

    • Ignored build artifacts (dist) from source control

- Integrate rollup with terser for minified builds
- Update CI to build before tests
- Update publish workflow to build before release
- Exclude dist/ from git, publish only built output
- Configure biome to ignore generated dist folder
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 21, 2026

Warning

Rate limit exceeded

@gaureshpai has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 11 minutes and 43 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: bf544bc7-ea17-47ac-b532-0aa453dc4227

📥 Commits

Reviewing files that changed from the base of the PR and between 1a43665 and dcdbbab.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (18)
  • .github/ISSUE_TEMPLATE/bug_report.yml
  • .github/ISSUE_TEMPLATE/feature_request.yml
  • .github/workflows/ci.yml
  • .github/workflows/post-test-results.yml
  • .github/workflows/publish.yml
  • .github/workflows/welcome.yml
  • .gitignore
  • CODE_OF_CONDUCT.md
  • CONTRIBUTING.md
  • README.md
  • SECURITY.md
  • docs/index.html
  • package.json
  • rollup.config.mjs
  • src/create-next-quick.js
  • src/test/build-config.test.js
  • src/test/interactive.test.js
  • src/test/test.js

Note

Reviews paused

It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:

  • @coderabbitai resume to resume automatic reviews.
  • @coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

  • ▶️ Resume reviews
  • ✅ Review completed - (🔄 Check again to review again)
📝 Walkthrough

Walkthrough

Adds a Rollup bundling pipeline and switches CLI entrypoints to use dist artifacts; CI and publish workflows now build packages and run a smoke test; tests updated to validate and use built output; .gitignore ignores dist/; many docs and templates normalized for whitespace/newlines.

Changes

Cohort / File(s) Summary
Build & Bundle
rollup.config.mjs, package.json
Added Rollup config and rollup-related devDependencies; package.json now targets dist/create-next-quick.js for main/bin, publishes dist, adds build and pretest scripts, updates start/test scripts.
CI/CD Workflows
.github/workflows/ci.yml, .github/workflows/publish.yml, .github/workflows/post-test-results.yml, .github/workflows/welcome.yml
CI workflow now runs run build after deps and a smoke test that captures logs and fails on non-zero exit; publish workflow runs pnpm build before publish; other workflows received quoting/whitespace/newline normalizations.
Tests
src/test/...
src/test/test.js, src/test/interactive.test.js, src/test/build-config.test.js
Tests updated to invoke the bundled CLI in dist; added build-config.test.js asserting rollup config, package.json wiring, .gitignore contains dist/, and presence/order of CI/publish build & smoke-test steps.
Repo Ignore
.gitignore
Added dist/ entry to ignore built artifacts.
Docs & Templates
README.md, CONTRIBUTING.md, CODE_OF_CONDUCT.md, SECURITY.md, .github/ISSUE_TEMPLATE/*, docs/index.html
Formatting, spacing, and newline normalization across docs and issue templates; large reformat of docs/index.html (doctype/attribute/indentation normalization).

Sequence Diagram(s)

sequenceDiagram
  participant Dev as Developer
  participant CI as CI Runner
  participant Build as Rollup / Package Builder
  participant Smoke as Smoke Test
  participant Store as Artifact Store

  Dev->>CI: push / open PR
  CI->>CI: install dependencies
  CI->>Build: run build (matrix package-manager run build / pnpm build)
  Build-->>CI: produce `dist/` artifacts
  CI->>Smoke: run smoke test `node dist/create-next-quick.js --version`
  Smoke-->>CI: write `test-logs/smoke-test.txt` (stdout/stderr + exit code)
  alt smoke test success
    CI->>Store: upload logs & artifacts
  else smoke test failure
    CI-->>Dev: fail job
  end
Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Poem

🐇 I rolled a bundle, snug and light,

Dist now hops into the night,
CI sniffs smoke, then gives a cheer,
Logs in paw, I nibble a gear,
A tiny hop — the build takes flight.

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description is incomplete relative to the template. It lacks a formal description section, related issue links, type of change selection, testing details, and checklist items. Only bullet points addressing specific implementation details are provided. Fill out the description template with: a detailed description section, linked issue(s), selected type of change, testing methodology, and completed checklist items.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main change: adding a rollup build pipeline with minification. This aligns directly with the core objective of integrating rollup with terser for minified builds.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/add-build-pipeline

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 21, 2026

✅ CI/CD Test Results

Overall Status: success


View full workflow run

@coderabbitai

This comment was marked as outdated.

@coderabbitai

This comment was marked as outdated.

@gaureshpai gaureshpai added the enhancement New feature or request label Mar 21, 2026
@gaureshpai gaureshpai self-assigned this Mar 21, 2026
coderabbitai[bot]

This comment was marked as outdated.

@coderabbitai

This comment was marked as outdated.

- Add rollup with terser for minified production builds
- Rename entry point to create-next-quick.js
- Update tests to run against built dist artifact
- Add CI smoke test step to validate built artifact
- Remove unused @rollup/plugin-alias dependency
- Fix biome config (remove invalid files.ignore key)
coderabbitai[bot]

This comment was marked as outdated.

Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
@coderabbitai

This comment was marked as outdated.

coderabbitai bot and others added 3 commits March 21, 2026 15:11
Fixed 1 file(s) based on 1 unresolved review comment.

Co-authored-by: CodeRabbit <noreply@coderabbit.ai>
coderabbitai[bot]

This comment was marked as outdated.

@gaureshpai

This comment was marked as outdated.

@coderabbitai

This comment was marked as outdated.

@gaureshpai gaureshpai merged commit 83babcb into main Mar 21, 2026
8 checks passed
@gaureshpai gaureshpai deleted the feat/add-build-pipeline branch March 21, 2026 17:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant