Conversation
- Replace Bun with Node.js as runtime - Replace bun:test with Vitest for testing - Update package.json with new dependencies (vite, vitest, tsx, oxlint) - Add vitest.config.ts configuration - Update tsconfig.json for modern bundler moduleResolution - Remove bun.lockb and bunfig.toml - Migrate all test files from bun:test to vitest - Fix spyOn usage in consoleLogPromise.test.ts to use vi.spyOn
- Replace Bun.build with Vite for JS bundling - Use rollup-plugin-dts for .d.ts generation - Create utils module for file operations - Migrate all build helpers to use Node.js fs APIs - Update coherency scripts to use Node.js - Update license script to use glob package - Add glob as dev dependency - All builds now generate proper .d.ts files
- Update all GitHub Actions to use Node.js 22 instead of Bun - Replace oven-sh/setup-bun with actions/setup-node - Change bun install to npm ci - Change bun run commands to npm run - Update cache paths from bun.lockb to package-lock.json - Update devcontainer to use Node.js 22 feature - Replace Bun VS Code extension with Vitest Explorer - Update release workflow to use npx tsx for scripts
- Update all README.md files to use npm/npx tsx commands - Update RELEASE.md and ARCHITECTURE.md documentation - Update scripts to reference npm commands instead of bun - Update pre-release-validator to check npm instead of bun - Update release.ts to use npm run commands - Update pr-validation.yml workflow jobs - Update SECURITY.md to reference npm audit - Update .editorconfig and .cspell.json for new lockfile
- Replace npm with pnpm 9.15.4 as package manager - Update all GitHub Actions to use pnpm/action-setup@v4 - Upgrade Node.js version requirement from 20 to 24 - Configure pnpm store as Docker volume in devcontainer - Add PNPM_HOME environment variable to devcontainer - Update all documentation to use pnpm commands - Replace package-lock.json with pnpm-lock.yaml
- Add parse() function to parse semantic version strings - Export ParsedVersion interface for typed results - Support core version (MAJOR.MINOR.PATCH) - Support pre-release identifiers (-alpha, -beta.1, -rc.1) - Support build metadata (+build, +sha.abc123) - Support optional 'v' prefix - Add comprehensive test suite (20 tests)
- Import parse() from ./parse instead of inline parseVersion - Full SemVer 2.0.0 compliance maintained - Comprehensive test suite (16 tests) for compare function - Tests cover: core version, prerelease, build metadata, edge cases
- Add toISO8601(): Convert to ISO 8601 format (YYYY-MM-DDTHH:mm:ss.sssZ) - Add toRFC3339(): Convert to RFC 3339 format (with optional milliseconds) - Add toRFC2822(): Convert to RFC 2822 format (email/HTTP headers) - All functions handle Date, timestamp, or date string inputs - Return null for invalid dates - Add comprehensive test suite (19 tests)
- Document project overview and tech stack - Define coding conventions and file structure - Explain testing guidelines with Vitest - Document build system (Vite + Rollup) - List standards: SemVer 2.0.0, ISO 8601, RFC 3339, RFC 2822 - Provide common tasks and commit message format - Include CI/CD and DevContainer setup details
Contributor
🔍 PR Validation Summary
|
- Configure Vitest with json-summary reporter for CI - Add coverage thresholds (current: 80%, target: 100%) - Update run-unit-tests action to support coverage input - Extract coverage metrics (lines, branches, functions, statements) - Display coverage table with progress bars in PR comment - Show overall status based on jobs + coverage results
- Deleted the `run-verification`, `setup-environment`, and `version-manager` actions as they are no longer needed. - Updated the `pr-validation` workflow to use new job workflows for build, test, lint, and verification. - Introduced new job workflows: `job-build`, `job-lint`, `job-security`, `job-tests`, `job-typecheck`, and `job-verification` to modularize the CI process. - Enhanced the `release` workflow to include a dry-run option for safer releases. - Improved Node.js version handling across workflows for consistency.
- Add .github/copilot-instructions.md for Copilot guidance - Document preference for Claude models over GPT - Include key coding conventions and guidelines - Reference AGENTS.md for detailed project context
- Remove pull_request_target trigger that caused workflow conflicts - Simplify workflow to use only pull_request for internal PRs - Remove IS_TRUSTED_PR conditions no longer needed - Delete redundant copilot-instructions.md (AGENTS.md suffices) - Update AGENTS.md with critical restrictions (no git push, no GPT models)
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
1 participant
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.
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
New CI
What is the current behavior? (You can also link to an open issue here)
bun CI
What is the new behavior (if this is a feature change)?
Vite (plus) CI
Other information:
Improve PR validation also