feat: sync decentralize prompts with deploy and clarify update banner#377
Merged
UtkarshBhardwaj007 merged 1 commit intoJun 13, 2026
Merged
Conversation
Signer pickers (deploy + decentralize): list the phone signer first with the cursor defaulting to it, and move the "dev signer earns no XP" warning below the options, shown only while the dev signer is highlighted. Adds an optional onHighlight callback to the shared Select component. decentralize: add the same magenta help boxes as deploy above the signer, domain, publish, and a new tags prompt (recoloring the intro box to accent); reorder the publish prompt so "yes" is first and default-selected; add a category-tag step plus a --tag flag (whitelisted to PLAYGROUND_TAGS, requires --playground on the headless path), threaded into the published metadata and surfaced in the confirm summary. Lifts PromptInfo into the shared theme. version banner: now reads "Run pg update to update the CLI." for clarity.
Contributor
|
Dev build ready — try this branch: |
Contributor
E2E Test Pass · ✅ PASSTag:
Sentry traces: view spans for this run |
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.
Summary
Brings
playground decentralize's interactive prompts in line withplayground deploy, and tweaks the signer picker UX in both. Three user-driven changes plus a copy fix:Signer picker (deploy + decentralize)
onHighlightcallback to the sharedSelectcomponent to drive this.decentralizesynced withdeploy--tagflag (whitelisted toPLAYGROUND_TAGS, requires--playgroundon the headless path), threaded into the published app metadata and shown in the confirm summary.PromptInfolifted fromDeployScreeninto the shared theme so both screens use one component.Update banner
Run pg update to update the CLI.(was "playground update to upgrade") to avoid the upgrade/update wording mismatch.Testing
pnpm typecheck✓ ·pnpm format:check✓ ·pnpm lint:license✓ ·pnpm test✓ (887 tests)signerNotice.test.ts(option order + warning predicate),decentralize/signerPrompt.test.ts,decentralize/index.test.ts(--tagrequires--playground),decentralize/state.test.ts(tag-gating). Banner string assertions updated.Notes
pgalias per request; the rest of the CLI copy uses the fullplaygroundname.run.ts's one-linetagpassthrough is not unit-tested (its downstream metadata write is covered); turning on the publish branch inrun.test.tswould need substantial extra mocks.