Repository navigation
Harden shared action and agent credential responses - #1920
Merged
Paul Lizer (paullizer) merged 2 commits intoNov 11, 2026
Merged
Conversation
Separate shared group/global discovery payloads from credential-bearing configuration. Reuse native secret-preserving writes and conditional revisions for classic group editors, including explicit clears and safe masked-credential handling. Add offline API, SDK lifecycle, and browser regressions, update usage guidance, and bump the application patch version. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Retain both shared-credential and support-export SDK inventory entries. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
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.
Summary
0.261.428.Intentional API tightening: non-managing legacy readers no longer receive connector/agent configuration, and group managers cannot reveal saved credential values. Legitimate discovery, selection, and execution remain available.
Linked issue
N/A.
Release Notes & Latest Features
Is this visible to end users?
Is this admin-facing (Admin Settings, governance, deployment, config)?
Should this become a Latest Feature card?
Screenshot needed for the card?
Version bump
application/single_app/config.pyVERSIONthird segment bumped, or not needed because this is docs-onlydeployers/version.txtbumped, or not needed becausedeployers/was not changedTesting / validation
API/SDK runs used synthetic Cosmos/Key Vault fixtures with external networking blocked. Browser checks used the production local assets with intercepted synthetic APIs.
python -B .\scripts\check_cosmos_response_boundaries.py --run-contracts- Passed: 2,053 tests. This broad run preceded the final readiness-source integration guard; the final guard was then verified by the focused command below.python -B -m pytest -q --tb=short -p no:cacheprovider -p test_support.cosmos_network_guard .\functional_tests\test_shared_action_secret_responses.py .\functional_tests\test_shared_agent_secret_responses.py .\functional_tests\test_workspace_authoring_sdk_boundaries.py- Passed: 99 tests on the final source.python -B -m pytest -q --tb=short -p no:cacheprovider .\ui_tests\test_shared_workspace_secret_editing.py- Passed: 22 browser workflows.python -B .\scripts\check_cosmos_response_boundaries.py- Passed: 1,094 owners, 426 doubles, zero review gaps. Module-only changes received renewed owner-runtime SDK evidence; no source-only exemptions were added.python -B .\functional_tests\test_docs_app_surface_coverage.py && python -B .\functional_tests\test_docs_site_quality.py- Passed: 7 coverage checks and 6 site-quality checks.python -B .\functional_tests\route_tests\test_route_blueprint_policy_inventory.py && python -B .\functional_tests\route_tests\test_route_unauthenticated_policy_contract.py && python -B .\functional_tests\route_tests\test_route_policy_test_coverage.py- Passed: 28 route-policy checks.Commit-diff XSS and broken-access-control guardrails also passed for all 10 changed application files and all 5 changed Python files, respectively. All five changed JavaScript assets passed
node --check; patch whitespace checks passed.Documentation
Updated only the existing group-workspace and agent/action usage guides. Release notes and standalone feature/fix reports are intentionally not included in this scoped change.
Security checklist
@swagger_route(security=get_auth_security())sanitize_settings_for_user()No new Flask routes or raw settings-object serialization were added. Shared-resource records use canonical allowlisted projections and credential masking; existing settings-sanitization boundaries and dedicated admin settings are unchanged. Test credentials are synthetic, and session-only planning artifacts are excluded.