Skip to content

feat: add nuxt-wallet-kit - #1664

Open
MaximeBignolet wants to merge 4 commits into
nuxt:mainfrom
MaximeBignolet:add-wallet-kit
Open

MaximeBignolet wants to merge 4 commits into
nuxt:mainfrom
MaximeBignolet:add-wallet-kit

Conversation

@MaximeBignolet

Copy link
Copy Markdown

📚 Description

It makes it easy to connect a crypto wallet in Nuxt, and improves some things that are missing in the wagmi Nuxt module:

  1. Everything is configured in nuxt.config, no need to write the plugin, chains and transports yourself
  2. The wallet address is already there after a page reload with SSR, without flash or hydration error
  3. Ready-to-use components made with Nuxt UI: a connect button, a wallet picker modal and an account menu

There was no web3 module in the Nuxt modules list, so I made one.

(I couldn't run pnpm sync locally (pnpm failed to download Node), so I wrote the yml by hand following the sync output.)

@coderabbitai

coderabbitai Bot commented Oct 13, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 57 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used all 4 included reviews currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: f7870e55-90ae-454e-87fd-b015b211e557
📥 Commits

Reviewing files that changed from the base of the PR and between cd6ba76 and d24098a.

📒 Files selected for processing: 1
  • modules/nuxt-wallet-kit.yml

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: b6fa152f-27cb-4294-9876-2ac3a5e34f7b
📥 Commits

Reviewing files that changed from the base of the PR and between 12b1c50 and cd6ba76.

📒 Files selected for processing (1)
  • modules/wallet-kit.yml

Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 1 remain after this review.

📝 Walkthrough

Adds a wallet-kit catalog entry with its description, package and repository links, category, type, maintainer, and Nuxt compatibility range. The entry has no requirements.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Merge Risk: ⚪ Minimal · up to cd6ba

No concrete issue remains in the wallet-kit catalog entry; it is ready to merge subject to normal checks.

Pre-merge checks: Passed 5
Passed Passed checks (5 passed)
Check name Status Explanation
Title check Passed The title clearly identifies the main change: adding the Nuxt Wallet Kit module.
Description check Passed The description includes the required Description section and explains the module's purpose, key features, and the manual YAML creation. The linked issue is optional.
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

This branch has not been deployed

No deployments
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