Skip to content

Upgrade all dependencies and migrate to Vitest 4 - #23

Open
llambeau wants to merge 1 commit into
masterfrom
chore/upgrade-dependencies
Open

Upgrade all dependencies and migrate to Vitest 4#23
llambeau wants to merge 1 commit into
masterfrom
chore/upgrade-dependencies

Conversation

@llambeau

Copy link
Copy Markdown
Contributor
  • Upgrade all npm dependencies to their latest versions
  • Migrate vitest.workspace.ts to vitest.config.ts using the new test.projects API (Vitest 4 requirement)
  • Replace deprecated poolOptions with top-level pool options
  • Update mock function types for Vitest 4 compatibility
  • Fix import sorting in test files (lint auto-fix)

Notable major upgrades:

  • vitest: 3.x → 4.x
  • @inquirer/prompts: 7.x → 8.x
  • listr2: 9.x → 10.x
  • glob: 11.x → 13.x
  • protobufjs: 7.x → 8.x

- Upgrade all npm dependencies to their latest versions
- Migrate vitest.workspace.ts to vitest.config.ts using the new
  test.projects API (Vitest 4 requirement)
- Replace deprecated poolOptions with top-level pool options
- Update mock function types for Vitest 4 compatibility
- Fix import sorting in test files (lint auto-fix)

Notable major upgrades:
- vitest: 3.x → 4.x
- @inquirer/prompts: 7.x → 8.x
- listr2: 9.x → 10.x
- glob: 11.x → 13.x
- protobufjs: 7.x → 8.x

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
llambeau added a commit that referenced this pull request Jul 17, 2026
`validateUserConfig` and `validateEmbfile` guarded the read with
`if (await stat(path))`. stat() resolves to a (truthy) Stats object or
rejects — it is never falsy — so the `else { throw 'Could not find
file' }` branch was dead code, and a missing file surfaced a raw
`ENOENT: no such file or directory, stat '...'` instead.

Wrap stat() in try/catch so the intended friendly message actually
fires when the file is absent.

Addresses REPORT.md finding #23.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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.

1 participant