Skip to content

add public channel managers endpoint - #5

Open
deployor wants to merge 1 commit into
hackclub:mainfrom
deployor:add-channel-managers-api
Open

add public channel managers endpoint#5
deployor wants to merge 1 commit into
hackclub:mainfrom
deployor:add-channel-managers-api

Conversation

@deployor

@deployor deployor commented Aug 2, 2026

Copy link
Copy Markdown

This allows any bot or person to look up channel managers without having to use a self-bot, even in private channels.

This means that bots such as at-channel, bChannel, etc. can rely on channel managers rather than the creators of the channel for private channels. This makes permission checks much easier!

Copilot AI review requested due to automatic review settings August 2, 2026 18:58

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds support for querying Slack “channel manager” role assignments via the existing proxy + pipeline service, enabling downstream bots/services to rely on channel managers (including for private channels) rather than channel creators.

Changes:

  • Extend the proxy allowlist to permit admin.roles.listAssignments calls.
  • Add the required Slack app OAuth scope (admin.roles:read) to the manifest.
  • Introduce a new GET /channel-managers endpoint in the pipeline service that pages through role assignments and returns manager user IDs for a channel.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

File Description
proxy/app.py Allows the proxy to forward admin.roles.listAssignments using the admin credential.
pipeline/slack.manifest.yaml Adds admin.roles:read so the Slack app can call role assignment APIs.
pipeline/service/app.py Adds a public /channel-managers HTTP endpoint backed by proxy calls to Slack admin role assignment APIs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread pipeline/service/app.py
Comment thread pipeline/service/app.py
Comment thread pipeline/service/app.py
Comment thread pipeline/service/app.py
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.

2 participants