Conversation
Signed-off-by: Jared Scott <jared.scott@variable.team>
Signed-off-by: Jared Scott <jared.scott@variable.team>
Signed-off-by: Jared Scott <jared.scott@variable.team>
Signed-off-by: Jared Scott <jared.scott@variable.team>
Signed-off-by: Jared Scott <jared.scott@variable.team>
Signed-off-by: Jared Scott <jared.scott@variable.team>
Signed-off-by: Jared Scott <jared.scott@variable.team>
There was a problem hiding this comment.
Pull request overview
This pull request migrates the project's tooling from ESLint/Prettier/Jest to Biome/Vitest. The changes include removing legacy testing and linting dependencies, adding new configuration files, migrating existing tests to Vitest, creating comprehensive test coverage for previously untested modules, and adding a CI workflow. Additionally, copyright years across all source files are updated to 2026.
Changes:
- Replaced ESLint and Prettier with Biome for linting and formatting
- Migrated from Jest to Vitest for testing with improved test coverage
- Added GitHub Actions CI workflow for automated testing and linting
Reviewed changes
Copilot reviewed 21 out of 23 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| vitest.config.ts | New Vitest configuration file defining test environment and patterns |
| biome.json | New Biome configuration with comprehensive linting rules and formatting settings |
| package.json | Updated scripts and dependencies, replacing Jest/ESLint packages with Vitest/Biome |
| src/utils.spec.ts | Migrated test from Jest to Vitest with improved test cases |
| src/service_worker.spec.ts | New test file providing coverage for service worker functionality |
| src/main.spec.ts | New comprehensive test file for main UI bootstrap logic |
| eslint.config.mjs | Removed Jest globals and prettier plugin reference |
| jest.config.json | Deleted Jest configuration (replaced by Vitest) |
| tsconfig.build.json | Updated to exclude vitest.config.ts instead of jest.config.json |
| .github/workflows/ci.yml | New CI workflow for linting and testing on Node 22 and 24 |
| AGENTS.md | Updated documentation to reflect new tooling |
| Multiple source files | Copyright year updated from 2024 to 2026 |
| pnpm-workspace.yaml | Added core-js and esbuild to onlyBuiltDependencies |
| public/manifest.json | Formatting changes only (no functional changes) |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Jared Scott <jared.scott@variable.team>
Signed-off-by: Jared Scott <jared.scott@variable.team>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 21 out of 23 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Replace eslint and prettier with Biome. Replace jest with vitest