Conversation
🦋 Changeset detectedLatest commit: cdaa726 The changes in this PR will be included in the next version bump. This PR includes changesets to release 23 packages
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 |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (7)
📝 WalkthroughWalkthroughThis PR migrates the codebase from deprecated Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related PRs
Suggested labels
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Pull request overview
Housekeeping updates to ensnode-react (and related docs) to complete migration away from the deprecated ENSNodeClient and to reduce unnecessary query refetching for resolved identities.
Changes:
- Replaced
ENSNodeClientusages withEnsApiClientinensnode-reactand updated theensnode-sdkREADME accordingly. - Added default
staleTimeandrefetchInterval(5 minutes) foruseResolvedIdentity. - Added Changesets entries for
@ensnode/ensnode-reactand@ensnode/ensnode-sdk.
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| packages/ensnode-sdk/README.md | Updates documentation examples and headings to use EnsApiClient. |
| packages/ensnode-react/src/utils/query.ts | Migrates query option builders to instantiate EnsApiClient instead of ENSNodeClient. |
| packages/ensnode-react/src/provider.tsx | Migrates createConfig defaults to use EnsApiClient.defaultOptions(). |
| packages/ensnode-react/src/hooks/useResolvedIdentity.ts | Sets default 5-minute staleTime and refetchInterval for identity resolution queries. |
| .changeset/two-rice-flash.md | Declares a minor bump for @ensnode/ensnode-react due to client migration. |
| .changeset/stale-wombats-help.md | Declares a minor bump for @ensnode/ensnode-react due to query default changes. |
| .changeset/cool-hotels-dress.md | Declares a minor bump for @ensnode/ensnode-sdk due to README updates. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Lite PR →
ensnode-reacthousekeeping refactors (WIP)Summary
Why
ENSNodeClient#1637 (ensnode-react was the last place where the issue wasn't implemented)Testing
lint&typecheckcommands locally to make sure that the new changes didn't break anythingNotes for Reviewer (Optional)
Pre-Review Checklist (Blocking)