Skip to content

fix: declare vitest dependency#401

Open
morgan-coded wants to merge 1 commit into
mattpocock:v1from
morgan-coded:fix/evalite-398-vitest-dependency
Open

fix: declare vitest dependency#401
morgan-coded wants to merge 1 commit into
mattpocock:v1from
morgan-coded:fix/evalite-398-vitest-dependency

Conversation

@morgan-coded

Copy link
Copy Markdown

evalite imports from vitest at runtime, but the published package did not declare vitest, so strict pnpm/no-hoist installs could fail resolution. This declares vitest in the Evalite package dependencies, while leaving the existing Vitest utility packages and peer dependencies unchanged. I added a manifest assertion that failed before the dependency was declared, then ran pnpm run build:evalite, pnpm --filter evalite test -- src/command.test.ts, pnpm --filter evalite-tests test -- tests/basics.test.ts, typecheck, lint, format, and pnpm run ci.

Fixes #398

Copilot AI review requested due to automatic review settings July 8, 2026 00:39
@changeset-bot

changeset-bot Bot commented Jul 8, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 911f43e

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
evalite Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel

vercel Bot commented Jul 8, 2026

Copy link
Copy Markdown

@morgan-coded is attempting to deploy a commit to the Skill Recordings Team on Vercel.

A member of the Team first needs to authorize it.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

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