Skip to content

fix(gateway): pin secondary credentials to profiles - #5

Merged
f-trycua merged 1 commit into
mainfrom
codex/multi-bot-slack
Aug 22, 2026
Merged

f-trycua merged 1 commit into
mainfrom
codex/multi-bot-slack

Conversation

@f-trycua

Copy link
Copy Markdown

What changed

  • bind multiplexed secondary adapters to the profile that owns their credential
  • keep global profile routes scoped to the primary/shared credential
  • document and test independent peer bots sharing one Slack channel

Verification

  • uv run --frozen --extra dev ruff check gateway/platforms/base.py gateway/run.py tests/gateway/test_profile_resolution.py
  • uv run --frozen --extra dev pytest -q tests/gateway/test_profile_resolution.py tests/gateway/test_slack_peer_agent_smoke.py tests/gateway/test_slack.py tests/gateway/test_slack_mention.py tests/gateway/test_slack_channel_session_scope.py -o addopts='' (213 passed)

@github-actions

github-actions Bot commented Aug 22, 2026 •

Copy link
Copy Markdown

૮ >ﻌ< ა ci review

ran on 79696bc — fix(gateway): pin secondary credentials to profiles

⚠️ Warnings

OSV vulnerability scan · View job

5 known vulnerabilities found in pinned dependencies.

How to fix:

Review the findings in the Security tab. Update the affected dependencies if a patched version is available.


debug info

CI timings

CI timings · View report · View job

Wall time 2m59s vs 4m19s (-30.9%). 14 job(s) slower, 8 faster, 3 unchanged.

  • Python tests / Run tests slice 6/12: +46.0s
  • Python tests / Run tests slice 8/12: +36.0s
  • Python tests / Run tests slice 7/12: +21.0s
  • Python tests / Run tests slice 12/12: +16.0s
  • Python tests / Run tests slice 11/12: +10.0s

@f-trycua
f-trycua merged commit 4dafaab into main Aug 22, 2026
44 checks passed
@f-trycua
f-trycua deleted the codex/multi-bot-slack branch August 22, 2026 21:43
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