Skip to content

arch: remove slack-broker from baudbot repo#70

Merged
benvinegar merged 1 commit into
mainfrom
benvinegar/remove-slack-broker
Feb 19, 2026
Merged

arch: remove slack-broker from baudbot repo#70
benvinegar merged 1 commit into
mainfrom
benvinegar/remove-slack-broker

Conversation

@benvinegar

@benvinegar benvinegar commented Feb 19, 2026

Copy link
Copy Markdown
Member

Summary

  • remove the in-repo slack-broker Cloudflare Worker code
  • keep Baudbot focused on agent runtime + Slack bridge only
  • update architecture/docs to reflect broker removal

The broker now lives in its own repo: https://github.com/modem-dev/baudbot-slack-broker

Validation

  • npm test (122 passed)

@greptile-apps

greptile-apps Bot commented Feb 19, 2026

Copy link
Copy Markdown

Greptile Summary

This PR removes the Cloudflare Worker-based Slack broker from the baudbot repository, moving it to a separate repository at https://github.com/modem-dev/baudbot-slack-broker. The change affects 23 files with 5,818 deletions, cleanly extracting the entire slack-broker/ directory including source code, tests, and configuration.

Key changes:

  • Deleted the entire slack-broker/ directory containing the Cloudflare Worker implementation
  • Updated AGENTS.md to remove slack-broker documentation section
  • Updated docs/architecture.md to reflect the removal from the directory structure
  • No remaining references to slack-broker found in the codebase
  • PR validates that all 122 tests still pass

This is a straightforward architectural cleanup that better separates concerns by keeping baudbot focused on the agent runtime and Slack bridge (Socket Mode), while extracting the broker functionality to its own repository.

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk
  • This is a clean directory deletion with no remaining references in the codebase. All tests pass (122/122), documentation is properly updated, and the extraction is complete. The change is purely subtractive with no modifications to existing functionality.
  • No files require special attention

Important Files Changed

Filename Overview
AGENTS.md Removed slack-broker section from documentation; correctly reflects architectural change
docs/architecture.md Updated architecture diagram to remove slack-broker directory reference
slack-broker/README.md Deleted - moved to separate repository
slack-broker/package.json Deleted - moved to separate repository
slack-broker/src/index.ts Deleted - Cloudflare Worker entry point moved to separate repository
slack-broker/wrangler.toml Deleted - Cloudflare Workers configuration moved to separate repository

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    subgraph before[Before - Monorepo Structure]
        A1[baudbot repo] --> B1[agent runtime]
        A1 --> C1[slack-bridge Socket Mode]
        A1 --> D1[slack-broker Cloudflare Worker]
    end
    
    subgraph after[After - Separated Repos]
        A2[baudbot repo] --> B2[agent runtime]
        A2 --> C2[slack-bridge Socket Mode]
        E2[baudbot-slack-broker repo] --> D2[Cloudflare Worker]
    end
    
    before -.extraction.-> after
    
    style D1 fill:#ffcccc
    style D2 fill:#ccffcc
Loading

Last reviewed commit: da7176b

@benvinegar benvinegar merged commit 8f56258 into main Feb 19, 2026
10 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