Streamline the managed config setup wizard - #372
Open
rohita5l wants to merge 1 commit into
Open
Conversation
This was referenced Aug 24, 2026
Collaborator
Author
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
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.
Depends on
Review this PR with base =
managed-config-local-draft. Its diff is only the interactive setup experience.Problem
The managed-config setup flow works, but it reads as a series of disconnected prompts and panels. Reviewers and admins cannot easily see where they are in the process, long model lists are hard to navigate, and setup repeats authentication/gateway checks that it already completed before discovery.
This PR changes presentation and prompt orchestration; it does not change the draft/published source rules introduced by the lower PRs.
New setup flow
ucode setupis presented as four explicit phases on one vertical progress rail:The rail stays active across prompts, status messages, validation, and completion so the output reads as one wizard rather than unrelated cards.
Prompt behavior
ucode setup; the rest of ucode remains on the existing Questionary promptsConnection-check cleanup
Setup needs authentication and admin status before it can inspect an existing published config, and model discovery needs the gateway immediately afterward. Previously the shared configure path repeated those checks and success messages.
configure_shared_state(..., connection_checks_verified=True)skips only the duplicate auth and gateway probes. It still obtains the bearer token and performs model discovery. Normal configure and launch behavior is unchanged.Implementation notes
inquirerpy_wizarduses context-local state, so nested prompt helpers automatically choose the wizard backend and cleanup does not leak into later commandsReview guide
setup_commandinmanaged_wizard.pyto see the four phases and check ordering.connection_checks_verifiedpath inconfigure_shared_state.ui.py, review the wizard context/header/step/outro helpers, then the InquirerPy branches in each prompt helper.tests/test_ui.pyfor layout, cleanup, fuzzy-list thresholds, keyboard hints, and choice-value behavior.Testing
pytest tests/test_ui.py tests/test_managed_wizard.py tests/test_cli.py -q: 499 passedStack