Add admin-only local managed config testing - #371
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-cache-safety. Its diff is only the CLI workflow layered on that storage model.Problem
Admins can author a managed config with
ucode setup, but they need a safe way to launch an agent with unpublished edits before publishing. At the same time, ordinary developer launches must keep using the workspace-published config, and publishing should not depend on whether or when the admin tested.User-visible behavior
ucodeorucode <agent>ucode --localucode <agent> --localucode applyTesting is optional. An admin can edit the draft repeatedly, test any revision, and
ucode applyalways publishes the latest saved revision.Authorization and failure behavior
A local launch has no server-side write on which the workspace can enforce admin permissions. Therefore
--localchecks the current workspace identity through SCIM and fails closed:ucode setupguidance before making the admin checkBare
ucode --localverifies once and passes that result into the selected agent launch.Managed-config bypass removal
--skip-managed-configpreviously cleared the feature environment variable for one process and let a developer ignore the published config. That conflicts with the new source model, so the bypass is removed.The old spelling is retained only as a hidden fail-closed tombstone. This matters because agent commands forward unknown flags to Claude/Codex; deleting the parser outright would silently pass an old ucode flag to the underlying agent.
--skip-preflightremains supported and only skips redundant auth/gateway validation; it still applies the workspace config.Review guide
cli.py, review_require_local_config_adminand the local branch in_launch_tool._launch_managed_defaultfor bareucode --localand its single-verification handoff.--skip-preflightremains independent.test_managed_workflow.pyfor the setup -> local launch -> apply -> edit again -> normal published launch sequence.Testing
pytest tests/test_cli.py tests/test_managed_config.py tests/test_managed_wizard.py tests/test_managed_workflow.py -q: 481 passedStack