Skip to content

Allow personal channel registrations to grant org services - #3659

Merged
louis4li merged 1 commit into
feature/integratefrom
fix/2026-09-17_channel-registration-org-services
Sep 17, 2026
Merged

louis4li merged 1 commit into
feature/integratefrom
fix/2026-09-17_channel-registration-org-services

Conversation

@louis4li

Copy link
Copy Markdown
Contributor

Summary

  • Allow personal channel registration Agent Keys to include organization-owned services when NyxID marks the credential source as allowed.
  • Validate scope-plan resource owners per selected service inventory, so org services must still resolve to their actual organization owner.
  • Keep organization-owned registrations strict: org registrations still require admin access to services from that organization.

Test plan

  • dotnet test test/Aevatar.GAgents.ChannelRuntime.Tests/Aevatar.GAgents.ChannelRuntime.Tests.csproj --nologo --filter ChannelRegistrationAuthorizationPlannerTests
  • bash tools/ci/test_stability_guards.sh
  • Local registration smoke: personal registration Agent Key accepted org-owned services and materialized updated allowed_service_ids.

🤖 Generated with Claude Code

Personal channel registration Agent Keys can now include organization-owned services that NyxID marks as allowed, while scope-plan validation still checks each service's actual resource owner.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 17, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-17T10:43:54.484860Z aa7da9f PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@louis4li
louis4li merged commit 6599600 into feature/integrate Sep 17, 2026
12 of 13 checks passed
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.

1 participant