Skip to content

Add --skip-managed-config and decouple managed config from --skip-pre… - #321

Merged
AarushiShah-db merged 6 commits into
mainfrom
AarushiShah-db/skip-managed-config
Aug 14, 2026
Merged

Add --skip-managed-config and decouple managed config from --skip-pre…#321
AarushiShah-db merged 6 commits into
mainfrom
AarushiShah-db/skip-managed-config

Conversation

@AarushiShah-db

Copy link
Copy Markdown
Collaborator

…flight

--skip-managed-config (on ucode configure and every launcher) ignores the workspace's managed coding-agent config for that run, behaving as though ENABLE_MANAGED_AGENT_CONFIG were unset — it clears the env var for the process, so the single managed_agent_config_enabled() gate short-circuits every managed path (fetch/apply, budget read, MCP registration, the configure reject flow, and the bare-ucode agent picker). Per-invocation only.

--skip-preflight no longer touches managed config: it now means only "skip the per-launch auth + AI Gateway re-validation." Dropped its cache-instead-of-refresh branch, its budget-read and MCP-registration gating, and the bare-ucode "no config to pick an agent" error.

Co-authored-by: Isaac

…flight

--skip-managed-config (on `ucode configure` and every launcher) ignores the
workspace's managed coding-agent config for that run, behaving as though
ENABLE_MANAGED_AGENT_CONFIG were unset — it clears the env var for the process,
so the single managed_agent_config_enabled() gate short-circuits every managed
path (fetch/apply, budget read, MCP registration, the configure reject flow, and
the bare-`ucode` agent picker). Per-invocation only.

--skip-preflight no longer touches managed config: it now means only "skip the
per-launch auth + AI Gateway re-validation." Dropped its cache-instead-of-refresh
branch, its budget-read and MCP-registration gating, and the bare-`ucode`
"no config to pick an agent" error.

Co-authored-by: Isaac
…anaged-config

# Conflicts:
#	src/ucode/cli.py
#	tests/test_cli.py
The provider-launch e2e tests only prove routing reaches the Model Provider
Service; a real "Credit balance is too low" from the provider account is an
environmental condition, not a ucode bug, so skip it like the existing
no-permission case rather than failing CI. Generalize _skip_if_no_permission ->
_skip_if_provider_unusable to cover both.

Co-authored-by: Isaac
Comment thread src/ucode/cli.py Outdated
@lilly-luo

Copy link
Copy Markdown
Collaborator

i don't understand why we need this - doesn't this undercut the purpose of the managed config by giving userse a way around it?

@AarushiShah-db

AarushiShah-db commented Aug 14, 2026

Copy link
Copy Markdown
Collaborator Author

don't understand why we need this - doesn't this undercut the purpose of the managed config by giving userse a way around it?
@lilly-luo we need a way to disable the feature for omniagent. omniagent requires ucode configure to be a non-interactive command, but once we roll this feature out 'ucode configure' will start to ask admins to run "ucode setup" instead (with an interactive prompt of yes or no)

@lilly-luo

Copy link
Copy Markdown
Collaborator

@lilly-luo we need a way to disable the feature for omniagent. omniagent requires ucode configure to be a non-interactive command, but once we roll this feature out 'ucode configure' will start to ask admins to run "ucode setup" instead (with an interactive prompt of yes or no)

i see. can we hide this command from the --help section then? ideally a normal ucode user never knows about it.

@AarushiShah-db

Copy link
Copy Markdown
Collaborator Author

@lilly-luo updated so it doesnt show up in --help

@AarushiShah-db
AarushiShah-db enabled auto-merge (squash) August 14, 2026 17:02
@AarushiShah-db
AarushiShah-db merged commit 95bdf0f into main Aug 14, 2026
2 checks passed
@AarushiShah-db
AarushiShah-db deleted the AarushiShah-db/skip-managed-config branch August 14, 2026 17:07
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.

3 participants