Skip to content

fix(react): narrow generated simulate hook types - #5077

Open
fuyua9 wants to merge 2 commits into
wevm:mainfrom
fuyua9:devb/wagmi-simulate-contract-fuyua9
Open

fuyua9 wants to merge 2 commits into
wevm:mainfrom
fuyua9:devb/wagmi-simulate-contract-fuyua9

Conversation

@fuyua9

@fuyua9 fuyua9 commented Apr 23, 2026

Copy link
Copy Markdown

Summary

  • narrow generated simulate hook call-site types to omit generated fields such as abi, address, functionName, and args
  • add a dts regression for functions with matching argument shapes but different return types
  • keep generated simulate hook result narrowing aligned with the recent generated read hook fix

Testing

  • pnpm bench:types packages/react/src/hooks/codegen/createUseSimulateContract.test-d.ts

@vercel

vercel Bot commented Apr 23, 2026

Copy link
Copy Markdown

@fuyua9 is attempting to deploy a commit to the Wevm Team on Vercel.

A member of the Team first needs to authorize it.

@changeset-bot

changeset-bot Bot commented Apr 23, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 965f93e

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

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

@fuyua9

fuyua9 commented Apr 25, 2026

Copy link
Copy Markdown
Author

Added a patch changeset for @wagmi/react in 965f93e so the changeset bot can classify this generated simulate hook typing fix for release.

Local validation: git diff --check passed. I did not rerun the full type suite here because this follow-up only adds a changeset file and the worktree did not have dependencies installed.

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