Skip to content

Conversation

@GuiBibeau
Copy link
Collaborator

@GuiBibeau GuiBibeau commented Jan 14, 2026

Summary

  • Add Connector adapter internally to @solana/client (not yet exposed publicly)
  • Factor wallet-standard session creation to reuse between connectors
  • Add Connector dependency/catalog entry
  • Dedupe react/react-dom in the Vite example to avoid invalid hook calls with workspace aliases
  • Export createWalletTransactionSigner and related utilities (fixes Docs request: WalletSession -> TransactionSigner adapter for Codama / @solana/kit #151)

New Public Exports

import {
  createWalletTransactionSigner,
  isWalletSession,
  resolveSignerMode,
  type WalletTransactionSigner,
  type WalletTransactionSignerConfig,
} from '@solana/client';

This allows users to adapt a WalletSession to a TransactionSigner for use with Codama-generated instructions.

Testing

  • pnpm test
  • pnpm size (bundle size check passes)

@vercel
Copy link

vercel bot commented Jan 14, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
framework-kit-docs Ready Ready Preview, Comment Jan 16, 2026 9:37am

Review with Vercel Agent

@GuiBibeau GuiBibeau changed the title Add ConnectorKit support and dedupe React in Vite Add ConnectorKit internal adapter and dedupe React in Vite Jan 15, 2026
Addresses #151 - exports the WalletSession to TransactionSigner adapter
so users can use Codama-generated instructions with wallet sessions.
@GuiBibeau GuiBibeau changed the title Add ConnectorKit internal adapter and dedupe React in Vite Add ConnectorKit internal adapter, export WalletSession signer utilities Jan 15, 2026
@Vishwa9011
Copy link

Thanks a lot for the quick response and for addressing this so fast — really appreciate the help and the clean solution 🙏
Just wanted to check: do you have an idea when this will be available in an npm release? Or is there a recommended way to access this in projects until it’s published?

@GuiBibeau
Copy link
Collaborator Author

@Vishwa9011

Thanks a lot for the quick response and for addressing this so fast — really appreciate the help and the clean solution 🙏 Just wanted to check: do you have an idea when this will be available in an npm release? Or is there a recommended way to access this in projects until it’s published?

Most likely today on npm (Friday Jan 16th 2026), if not Monday at the latest

…or IDs

Stable ConnectorKit entrypoint: added packages/client/src/connectorkit/index.ts, wired build + d.ts emit, and added @solana/client/connectorkit subpath export in packages/client/package.json.

Optional peer dep: moved @solana/connector to peerDependencies (optional via peerDependenciesMeta) while keeping it in devDependencies.
SSR safety + readiness semantics: connectorKit() no longer constructs a ConnectorClient when window is missing (returns [] + noop destroy()); isSupported() now respects ConnectorKit ready.

Connector-id aliasing + canonical persistence: createWalletRegistry().get() now falls back from "phantom" → wallet-standard:phantom → mwa:phantom, and connectWallet() now stores the canonical connector.id in client state.

Docs: updated canonical ID guidance + @solana/client/connectorkit usage in packages/client/README.md and apps/docs/content/docs/client.mdx (plus hook JSDoc in packages/react-hooks/src/hooks.ts).
Tests: added coverage for aliasing, SSR safety, readiness gating, canonical connectorId persistence, and createClient().destroy() cleanup.
Copy link
Contributor

@stevesarmiento stevesarmiento left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Look good, made the entry point available, and added a working example.

I think theres some more i can do on the connector kit side to ingest your patterns better, but thats a problem for another day -- this is a great start to get folks who want certain features that are available in connector to be able to use those with framework kit.

I'd wait for @amilz input if he has anything specific before merging.

@GuiBibeau GuiBibeau merged commit f3cdf94 into main Jan 16, 2026
9 checks passed
@GuiBibeau
Copy link
Collaborator Author

Confirmed with @amilz that we are good to merge.

@github-actions github-actions bot mentioned this pull request Jan 16, 2026
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.

Docs request: WalletSession -> TransactionSigner adapter for Codama / @solana/kit

4 participants