Conversation
3f06adc to
ea1acc0
Compare
- Replace Yarn with PNPM - Use semantic-release as a dependency - Unify CI/CD - Pin Node.js & PNPM versions - Drop running CI on pre-push
ea1acc0 to
728271c
Compare
commit: |
Member
Author
|
@copilot check for breaking changes in the exposed API |
There was a problem hiding this comment.
Pull request overview
This PR modernizes the project's tooling infrastructure by migrating from Yarn to PNPM, replacing Jest with Vitest for testing, and switching from TSC to tsdown for bundling. The changes also unify CI/CD workflows, pin Node.js and PNPM versions, remove pre-push hooks, and include semantic-release as a direct dependency.
Changes:
- Replaced package manager from Yarn to PNPM with pinned version and workspace configuration
- Migrated test framework from Jest to Vitest with unified configuration for unit and integration tests
- Replaced TypeScript compiler with tsdown bundler for building, targeting Node 24 and CommonJS output
- Unified CI/CD workflows into a single workflow file with separate CI and CD jobs
- Updated dependencies and modernized import syntax to use type-only imports where appropriate
Reviewed changes
Copilot reviewed 32 out of 35 changed files in this pull request and generated 8 comments.
Show a summary per file
| File | Description |
|---|---|
| vitest.config.ts | New Vitest configuration with separate unit and integration test projects, coverage enabled in CI |
| tsdown.config.mts | New bundler configuration for main library and CLI generator with CommonJS output |
| tsconfig.json | Updated to ESNext module system with bundler resolution, removed decorators and downlevel iteration |
| pnpm-workspace.yaml | Workspace configuration specifying native dependencies that must be built |
| package.json | Updated scripts, dependencies, and removed Jest configuration; added PNPM package manager field |
| src/**/*.ts | Updated import statements to use type-only imports and replaced Jest mocks with Vitest |
| src/tests/sqlite.ts | Migrated from sqlite+sqlite3 to better-sqlite3 with synchronous API |
| src/tests/prismaClient.ts | Removed middleware client factory, kept only extension client |
| src/tests/migrate.ts | Updated to remove CommonJS module check and use BigInt literal for id |
| src/tests/integration.test.ts | Added async to beforeAll, updated client initialization, added type assertions |
| .github/workflows/ci-cd.yml | New unified workflow with CI and CD jobs using PNPM and Node 24 |
| .github/workflows/pkg.pr.new.yml | Updated to use PNPM actions and setup |
| .husky/commit-msg | Updated to use pnpm instead of yarn |
| .node-version | New file pinning Node.js to version 24.11.0 |
| .releaserc | New semantic-release configuration with branch strategy |
| jest.config.*.json | Removed Jest configuration files |
| .npmignore | Removed in favor of package.json files field |
| .husky/pre-push | Removed pre-push hook |
| .github/workflows/ci.yml | Removed old CI workflow |
| .github/workflows/cd.yml | Removed old CD workflow |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
🎉 This PR is included in version 1.6.1-beta.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.