Repository navigation
refactor(channels): remove the old Slack, Discord and Telegram bots (foreman-7aak) - #58
Merged
Merged
Conversation
…foreman-7aak) Slack, Discord and Telegram now run on the AgentController's channels, so the direct agent.generate() bots, their webhook handlers, the Telegram polling script and the FOREMAN_NATIVE_CHANNELS Agent.channels scaffolding for Slack are unused. Removes them and their tests, and updates the repo and site docs: the new webhook URLs, Slack Interactivity for approval buttons, Discord's self-reconnecting Gateway, and what the :4112 webhook server still serves. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01T3R2sPwBzA2bpj3HT2BDv2
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This was referenced Oct 20, 2026
Merged
hamchowderr
force-pushed
the
claude/channels-controller
branch
from
October 20, 2026 14:07
d440f0e to
c121273
Compare
hamchowderr
force-pushed
the
claude/channels-cleanup
branch
from
October 20, 2026 14:07
1988bd9 to
e331b62
Compare
hamchowderr
changed the title
hamchowderr
marked this pull request as ready for review
October 22, 2026 17:39
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
This branch was successfully deployed
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.
Requested by Chowderr · project thread
Part 3 of 3 of the channels split that replaces #54. This PR is stacked on #57.
Before: after #57, the old direct-
generate()bots for Slack, Discord and Telegram are still in the tree but unused. That covers their webhook handlers, the Telegram polling script and theFOREMAN_NATIVE_CHANNELSscaffolding for Slack. The docs still describe the old setup.After: that code and its tests are gone. The repo and site docs now cover:
:4112webhook server still serves.How: delete the files, the env flag and the
channels:option inforeman.ts. Narrowcross-channel-memory.test.tsto the six bots that remain. Update the docs inCLAUDE.md,README.md,docs/channels/*andpackages/web/content/docs/*. The diff is mostly deletion: about 100 lines added and 1,200 removed.With all three PRs merged, the code is identical to #54's last green commit.
Beads: foreman-7aak
🤖 Generated with Claude Code
https://claude.ai/code/session_01T3R2sPwBzA2bpj3HT2BDv2