Skip to content

build: replace unbuild with Vite and add bundle analysis#7

Merged
ReenigneArcher merged 2 commits into
masterfrom
build/change-build-system
Jul 25, 2026
Merged

build: replace unbuild with Vite and add bundle analysis#7
ReenigneArcher merged 2 commits into
masterfrom
build/change-build-system

Conversation

@ReenigneArcher

@ReenigneArcher ReenigneArcher commented May 4, 2025

Copy link
Copy Markdown
Member

Description

Replace unbuild with Vite and integrate Codecov's Vite bundle-analysis plugin.

The new build preserves the existing ESM library and CLI entrypoints. Vite externalizes declared runtime dependencies and Node built-ins while continuing to bundle build-time-only helpers.

The project now uses a single TypeScript 7 dependency for both typechecking and native declaration emission. The legacy JavaScript compiler-API declaration stack has been removed, and internal relative imports use explicit .js specifiers so the emitted declaration tree works with NodeNext consumers. The package exposes those declarations through an explicit types export condition.

Screenshot

Issues Fixed or Closed

Roadmap Issues

Type of Change

  • feat: New feature (non-breaking change which adds functionality)
  • fix: Bug fix (non-breaking change which fixes an issue)
  • docs: Documentation only changes
  • style: Changes that do not affect the meaning of the code (white-space, formatting, missing semicolons, etc.)
  • refactor: Code change that neither fixes a bug nor adds a feature
  • perf: Code change that improves performance
  • test: Adding missing tests or correcting existing tests
  • build: Changes that affect the build system or external dependencies
  • ci: Changes to CI configuration files and scripts
  • chore: Other changes that don't modify src or test files
  • revert: Reverts a previous commit
  • BREAKING CHANGE: Introduces a breaking change (can be combined with any type above)

Checklist

  • Code follows the style guidelines of this project
  • Code has been self-reviewed
  • Code has been commented, particularly in hard-to-understand areas
  • Code docstring/documentation-blocks for new or existing methods/components have been added or updated
  • Unit tests have been added or updated for any new or modified functionality

AI Usage

See our AI usage policy.

  • None: No AI tools were used in creating this PR
  • Light: AI provided minor assistance (formatting, simple suggestions)
  • Moderate: AI helped with code generation or debugging specific parts
  • Heavy: AI generated most or all of the code changes

@sonarqubecloud

Copy link
Copy Markdown

@codecov

codecov Bot commented Jun 15, 2025

Copy link
Copy Markdown

Bundle Report

Changes will increase total bundle size by 260.5kB (100.0%) ⬆️⚠️, exceeding the configured threshold of 5%.

Bundle name Size Change
contribkit-esm 260.5kB 260.5kB (100%) ⬆️⚠️

@LizardByte-bot

Copy link
Copy Markdown
Member

It looks like this PR has been idle for 90 days. If it's still something you're working on or would like to pursue, please leave a comment or update your branch. Otherwise, we'll be closing this PR in 10 days to reduce our backlog. Thanks!

@codecov

codecov Bot commented Sep 13, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 0.00%. Comparing base (04e1654) to head (6bdc5e7).
✅ All tests successful. No failed tests found.

Additional details and impacted files

Impacted file tree graph

@@     Coverage Diff     @@
##   master   #7   +/-   ##
===========================
===========================

Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 04e1654...6bdc5e7. Read the comment docs.

@LizardByte-bot

Copy link
Copy Markdown
Member

It looks like this PR has been idle for 90 days. If it's still something you're working on or would like to pursue, please leave a comment or update your branch. Otherwise, we'll be closing this PR in 10 days to reduce our backlog. Thanks!

@LizardByte-bot

Copy link
Copy Markdown
Member

It looks like this PR has been idle for 90 days. If it's still something you're working on or would like to pursue, please leave a comment or update your branch. Otherwise, we'll be closing this PR in 10 days to reduce our backlog. Thanks!

@LizardByte-bot

Copy link
Copy Markdown
Member

It looks like this PR has been idle for 90 days. If it's still something you're working on or would like to pursue, please leave a comment or update your branch. Otherwise, we'll be closing this PR in 10 days to reduce our backlog. Thanks!

@ReenigneArcher
ReenigneArcher force-pushed the build/change-build-system branch from bb94aa2 to 58244b7 Compare July 24, 2026 22:00
@ReenigneArcher ReenigneArcher changed the title build: change build system to vite and add codecov bundle analysis build: replace unbuild with Vite and add bundle analysis Jul 24, 2026
@ReenigneArcher
ReenigneArcher marked this pull request as ready for review July 24, 2026 22:02
Reworked the build pipeline to run Vite for JS output and `tsc --project tsconfig.build.json` for declaration generation, replacing `vite-plugin-dts` with native TypeScript emit settings. Internal source imports/exports were updated to explicit `.js` specifiers across configs, providers, renders, and entrypoints to align ESM output and TS 7 behavior. Dev dependencies were also simplified (removing dts/api-extractor-related tooling and unused lint plugin) while upgrading TypeScript to ^7.0.0.
@sonarqubecloud

Copy link
Copy Markdown

@ReenigneArcher
ReenigneArcher merged commit 4c86345 into master Jul 25, 2026
14 checks passed
@ReenigneArcher
ReenigneArcher deleted the build/change-build-system branch July 25, 2026 00:32
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