status: show full managed-config box; abridge the launch banner - #357
Open
rohita5l wants to merge 1 commit into
Open
status: show full managed-config box; abridge the launch banner#357rohita5l wants to merge 1 commit into
rohita5l wants to merge 1 commit into
Conversation
`ucode status` now renders the workspace-managed config as one box (enabled agents, provider/model, MCPs, skills, budget tiers) whenever the feature is on and a config exists for the current workspace — reusing the same `_print_managed_summary` panel launch/configure already show, read from the local cache so status stays offline-fast. Bare `ucode` launch traded its full box for a one-line banner, since it runs every session and re-enumerating the whole config each time is noise: • Using managed config — launching Claude Code as the default agent with <model> "as the default agent" only appears when the launched agent really is the config's default; a budget-tier override drops it (the tier note in `_launch_tool` explains the swap). The model clause is omitted when none is set. The full box stays for `status` and `configure`. Co-authored-by: Isaac
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.
What
Two changes to how the workspace-managed config surfaces:
1.
ucode statusshows the full managed-config box. When the managed-config feature is on (ENABLE_MANAGED_AGENT_CONFIG) and a config exists for the current workspace, status now renders the whole setup — enabled agents, provider/model, MCPs, skills, and budget tiers — as one box. It reuses the same_print_managed_summarypanel that launch andconfigurealready show, read from the local cache so status stays offline-fast.2. Bare
ucodelaunch trades its full box for a one-line banner. Launch runs every session, so re-printing the entire config each time is noise:"as the default agent"only appears when the launched agent really is the config'sdefault_agent. A budget-tier override drops it (the existing tier note in_launch_toolexplains the swap), reading e.g.launching OpenCode with haiku.with <model>clause is omitted when the config pins no model.ucode statusanducode configure, where the reader asked to see everything.Tests
Added 5 focused tests (3 for the status box gating: present+enabled shows it, feature-off hides it and skips the cache read, no-config hides it; 2 for the launch banner: abridged content vs. the full box, and the tier-override wording). Full
test_cli.py(218) passes; lint clean.The only failing suite test,
test_e2e_user_agent, is pre-existing and unrelated (verified failing on a clean checkout with these changes stashed).This pull request and its description were written by Isaac.