chore: clean up plans, fix license, update READMEs#8
Conversation
Remove 6 completed plan files (testing, CI, modernization, readme-examples, integration-testing, bitbit-wallet-standard). Update roadmap to reflect implemented milestones. Set LICENSE copyright to 2026 Nejc Drobnič. Update all package READMEs to match current codebase: rename satsConnect to xverse, add walletStandard connector docs, add missing token hooks/actions, fix config API (network not networks), add @mbga/kit README. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
packages/kit/README.md
Outdated
|
|
||
| function App() { | ||
| return ( | ||
| <KitProvider config={config}> |
There was a problem hiding this comment.
This quick start cannot work as written. KitProviderProps only accepts theme and customTheme, and the component docs explicitly say it must be placed inside MbgaProvider. Copy-pasting <KitProvider config={config}> will fail type-checking on config, and rendering ConnectButton without MbgaProvider will still break at runtime. The theming snippet below has the same API mismatch (theme is a preset name, not a theme object).
packages/connectors/README.md
Outdated
|
|
||
| ### `walletStandard` | ||
|
|
||
| Connects to any wallet that implements the [Wallet Standard](https://github.com/wallet-standard/wallet-standard) with `spark:*` features. Automatically discovers compatible wallets at runtime. |
There was a problem hiding this comment.
This walletStandard section is describing the discovery plugin, not the connector itself. WalletStandardParameters requires a concrete wallet, so walletStandard() with no arguments will not compile, and automatic discovery actually lives in walletStandardDiscovery(). Copy-pasting this example sends readers to a nonexistent zero-config flow.
packages/react/README.md
Outdated
| npm install mbga @mbga/connectors sats-connect @tanstack/react-query | ||
| pnpm add mbga @mbga/connectors sats-connect @tanstack/react-query | ||
| yarn add mbga @mbga/connectors sats-connect @tanstack/react-query | ||
| npm install mbga @mbga/connectors @tanstack/react-query |
There was a problem hiding this comment.
The new install command is missing a required dependency for the very next example. xverse imports sats-connect, and @mbga/connectors exposes it as a peer dependency. A fresh install following this README will fail as soon as someone imports mbga/connectors unless they also add sats-connect.
…stall - KitProvider requires MbgaProvider parent, not a config prop — fix quick start to show correct nesting - walletStandard connector requires a wallet arg; document the walletStandardDiscovery plugin for auto-discovery instead - Restore sats-connect in react install command (peer dep of xverse) Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Rename all GitHub URLs, repository fields, badge links, codecov slugs, and docs site references to the new org. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- List all required packages in install command (mbga, @mbga/core, @mbga/connectors, sats-connect, @tanstack/react-query) - Fix theming example: use theme="dark" string (not darkTheme object), show customTheme prop, remove nonexistent config prop Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 participant
Summary
plans/roadmap.mdto check off ~20 implemented milestones2026 Nejc Drobnič(was placeholder[yyyy] [name of copyright owner])satsConnect→xverse, addwalletStandardconnector docs, add missing token hooks/actions/utilities, fix config API (networknotnetworks), add@mbga/kitand@mbga/flashnetto related packages tablespackages/kit/README.mdcovering styled + headless components, theming, and utilities