Skip to content

fix(vest): support Vest 6 - #874

Merged
bluebill1049 merged 1 commit into
react-hook-form:masterfrom
Fasping:fix-vest-6-support
Aug 13, 2026
Merged

fix(vest): support Vest 6#874
bluebill1049 merged 1 commit into
react-hook-form:masterfrom
Fasping:fix-vest-6-support

Conversation

@Fasping

@Fasping Fasping commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Closes #872.

Vest 6 removed the vest/promisify subpath and create() now returns a Suite object instead of a callable, so the resolver breaks at module resolution and then at call time.

  • use schema.run(...) with a plain awaitrun() returns a thenable in v6, so promisify is no longer needed
  • type ICreateResult off the Suite type that v6 exports
  • update the fixtures: create() no longer takes a suite name as its first argument
  • spy on run instead of wrapping the suite in vi.fn, since the suite is an object now
  • drop the vest/promisify UMD global from the build script

Peer range bumped to vest: ">=6.0.0": the removed subpath fails at module resolution, so a single build can't serve both v5 and v6. Happy to switch to a lazy import if you'd rather keep v5 working.

bun run test vest passes (11/11), plus lint, lint:types and build.

@bluebill1049
bluebill1049 merged commit 28ec539 into react-hook-form:master Aug 13, 2026
3 checks passed
@github-actions

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 5.8.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

issue: Vest version 6 incompatibility

2 participants