Keep wallet providers in Composer and search in the navbar - #1174
Open
jasonhedman wants to merge 3 commits into
Open
Keep wallet providers in Composer and search in the navbar#1174jasonhedman wants to merge 3 commits into
jasonhedman wants to merge 3 commits into
Conversation
Contributor
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
jasonhedman
changed the title
jasonhedman
removed this pull request from stack #1176
September 9, 2026 14:39
jasonhedman
added this pull request to stack #1128
September 9, 2026 14:40
jasonhedman
marked this pull request as ready for review
September 9, 2026 14:43
jasonhedman
force-pushed
the
json/scope-composer-wallet-providers
branch
2 times, most recently
from
September 9, 2026 15:02
3828234 to
7b63cf5
Compare
jasonhedman
force-pushed
the
json/scope-composer-wallet-providers
branch
2 times, most recently
from
September 9, 2026 15:13
ea60056 to
5527bb6
Compare
jasonhedman
force-pushed
the
json/scope-composer-wallet-providers
branch
from
September 9, 2026 19:14
5527bb6 to
fca13a6
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Public pages currently sit below Coinbase’s
ssr: falsewallet provider, so their content waits for browser JavaScript. Move the session, CDP, Wagmi, and Solana providers into Composer’s existing access gate, preserving their order. Use the URL as the sole public network filter state: a supported?chain=value selects that network, and an absent or invalid value means all chains. Remove the chain cookie helpers, context and provider; both pages and selector use the same parser.Global search is colocated in the navbar:
GlobalSearchowns its trigger, dialog, queries and keyboard shortcut. Remove the search context/provider/hook and the app-wide wrapper. Search behavior and result navigation are preserved.Validation: Production build passed. Raw registration HTTP response includes visible page content without JavaScript. Browser checks passed at 390px in light/dark mode; the selector and pages now share URL-derived chain state without a cookie fallback. Composer provider placement and access gate passed independent source review; wallet sign-in was not exercised. App TypeScript, 245 tests, UI compliance, source integrity, and repository boundaries pass. Full
pnpm checknow passes with zero lint warnings or errors, including source integrity and repository boundaries.Stack base: #1173.
CI compatibility: align React types to 19.2.17 and React DOM types to 19.2.3, use the specific submit event in the invite form, and mirror Foundation’s composition lint exceptions for the exact installed AI Elements. Runtime React versions and protected source stay unchanged.
Search validation: browser checked the button, Command-K, query results, closing and server navigation. Full
pnpm checkpasses on the updated first layer.