Add Slack support via unified entry point with symmetric adapter pattern by Copilot · Pull Request #25 · circa10a/ava

Copilot · 2026-02-22T02:22:17Z

Adds Slack as a supported platform alongside Discord using a symmetric adapter pattern. Both platforms now use the same architecture — commands are fully decoupled from Discord.js and receive messages through platform-specific adapters. A single npm start auto-detects which platforms to run based on environment variables, supporting Discord only, Slack only, or both simultaneously.

Approach

A unified index.js entry point loads commands once, then starts whichever platforms have tokens configured. Platform-specific logic is encapsulated in lib/platforms/discord.js and lib/platforms/slack.js. Commands export only { commandName, execute } and have no dependency on Discord.js events.

// Unified entry point starts platforms based on env vars
const startups = [];
if (discordEnabled) startups.push(startDiscord({ commands, db }));
if (slackEnabled) startups.push(startSlack({ commands, db }));
await Promise.all(startups);

Changes

lib/adapters/discord.js — DiscordMessageAdapter class (thin wrapper for architectural consistency)
lib/adapters/slack.js — SlackMessageAdapter class + embedToBlocks converter (Discord EmbedBuilder → Slack Block Kit)
lib/platforms/discord.js — Discord platform startup logic (client setup, message listener, reminder daemon)
lib/platforms/slack.js — Slack platform startup logic (@slack/bolt Socket Mode, message listener, reminder daemon)
index.js — Unified entry point that auto-detects and starts Discord, Slack, or both simultaneously
All 25 command files — Removed Events import and name/once properties, decoupling from discord.js event system
package.json — @slack/bolt dependency
README.md / docs/getting-started.md — Updated docs showing unified usage for Discord, Slack, or both; Docker examples
6 pre-existing lint fixes — Fixed no-useless-assignment errors in commands and lib files

Config

Variable	Required	Notes
`AVA_DISCORD_TOKEN`	Yes (Discord)	Discord bot token
`AVA_SLACK_BOT_TOKEN`	Yes (Slack)	`xoxb-` bot token
`AVA_SLACK_APP_TOKEN`	Yes (Slack)	`xapp-` app-level token for Socket Mode
`AVA_SLACK_SIGNING_SECRET`	No	Only needed for HTTP mode

# Discord only
AVA_DISCORD_TOKEN=<token> npm start

# Slack only
AVA_SLACK_BOT_TOKEN=<bot-token> AVA_SLACK_APP_TOKEN=<app-token> npm start

# Both simultaneously
AVA_DISCORD_TOKEN=<token> AVA_SLACK_BOT_TOKEN=<bot-token> AVA_SLACK_APP_TOKEN=<app-token> npm start

The Docker image works for all configurations — just pass the relevant env vars:

docker run -e AVA_DISCORD_TOKEN="<token>" -e AVA_SLACK_BOT_TOKEN="<bot-token>" -e AVA_SLACK_APP_TOKEN="<app-token>" circa10a/ava

Original prompt

Make a PR addressing the one ope issue which is to also support slack in addition to discord. All existing commands should work the same way as the discord implementation and ideally use the same logic. Documentation should also be updated

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

- Add @slack/bolt dependency for Slack integration - Create SlackMessageAdapter to normalize Slack events to Discord message interface - Create slack.js entry point with Socket Mode support - Add start:slack npm script - Add Slack reminder daemon - Update README.md and getting-started.md with Slack setup instructions Co-authored-by: circa10a <21261388+circa10a@users.noreply.github.com>

Co-authored-by: circa10a <21261388+circa10a@users.noreply.github.com>

circa10a · 2026-02-22T04:41:32Z

+  }
+
+  // Listen for messages starting with the ava prefix
+  app.message(new RegExp(`^${avaPrefix}`, 'i'), async ({ message, client }) => {


Compiling regex on every message is fairly CPU intensive. This isn't great

Fixed in 61de5ec — the regex is now pre-compiled once as a module-level constant (avaPrefixRegex) in both slack.js and index.js, instead of being created on every message.

circa10a · 2026-02-22T04:43:20Z

@@ -0,0 +1,139 @@
+import logger from '../logger/logger.js';


If we have a slack adapter, we should replace the existing architecture to leverage a "discord" adapter also

Done in 61de5ec — created lib/adapters/discord.js with a DiscordMessageAdapter and refactored index.js to use the same adapter dispatch pattern as slack.js. Both entry points are now symmetric: load commands → listen for prefix → wrap with adapter → dispatch. Also removed Events import and name/once properties from all 25 command files to fully decouple them from discord.js.

circa10a · 2026-02-22T17:09:12Z

@copilot address open comments and resolve issues in CI

…le regex, fix lint errors - Create lib/adapters/discord.js with DiscordMessageAdapter - Refactor index.js to use adapter pattern matching slack.js architecture - Pre-compile regex in both slack.js and index.js (no per-message compilation) - Remove Events import from all 25 command files (decouple from discord.js) - Fix 6 pre-existing no-useless-assignment lint errors in commands and lib files Co-authored-by: circa10a <21261388+circa10a@users.noreply.github.com>

Copilot · 2026-02-22T17:18:07Z

@copilot address open comments and resolve issues in CI

Addressed both review comments and fixed CI in 61de5ec:

Pre-compiled regex (no per-message compilation)
Created Discord adapter + refactored index.js to symmetric adapter pattern
Removed discord.js Events coupling from all 25 command files
Fixed all 6 pre-existing no-useless-assignment lint errors that were failing CI

circa10a · 2026-03-03T21:42:16Z

    "lint": "eslint .",
    "fix-lint": "eslint . --fix",
    "start": "node --no-deprecation ./index.js",
+    "start:slack": "node --no-deprecation ./slack.js",


We can not run both simultaneously? What about the docker image?

Fixed in 0672c98. index.js is now a unified entry point that auto-detects platforms based on which env vars are set. A single npm start runs Discord, Slack, or both simultaneously. The Docker image needs no CMD override — just pass the relevant env vars:

# Both platforms simultaneously docker run -e AVA_DISCORD_TOKEN="<token>" -e AVA_SLACK_BOT_TOKEN="<bot-token>" -e AVA_SLACK_APP_TOKEN="<app-token>" circa10a/ava

Removed the separate slack.js entry point and start:slack script. Platform logic extracted into lib/platforms/discord.js and lib/platforms/slack.js.

…multaneously - Refactor index.js to auto-detect platforms based on which env vars are set - Extract platform logic into lib/platforms/discord.js and lib/platforms/slack.js - Remove separate slack.js entry point and start:slack script - Docker image works for all configs via env vars alone (no CMD override) - Update README and docs to show unified usage Co-authored-by: circa10a <21261388+circa10a@users.noreply.github.com>

Co-authored-by: circa10a <21261388+circa10a@users.noreply.github.com>

Copilot · 2026-03-03T21:49:48Z

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

discord.com
- Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node node index.js (dns block)
slack.com
- Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node node index.js (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Configure Actions setup steps to set up my environment, which run before the firewall is enabled
Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

Initial plan

87fcfff

Copilot AI assigned Copilot and circa10a Feb 22, 2026

Copilot started work on behalf of circa10a February 22, 2026 02:22 View session

Copilot AI and others added 2 commits February 22, 2026 02:29

Move eslint peer deps to devDependencies

7134b18

Co-authored-by: circa10a <21261388+circa10a@users.noreply.github.com>

Copilot AI changed the title ~~[WIP] Add support for Slack alongside Discord~~ Add Slack support via message adapter pattern Feb 22, 2026

Copilot AI requested a review from circa10a February 22, 2026 02:32

Copilot finished work on behalf of circa10a February 22, 2026 02:32

circa10a reviewed Feb 22, 2026

View reviewed changes

Copilot started work on behalf of circa10a February 22, 2026 17:09 View session

Copilot AI changed the title ~~Add Slack support via message adapter pattern~~ Add Slack support via symmetric message adapter pattern Feb 22, 2026

Copilot AI requested a review from circa10a February 22, 2026 17:18

Copilot finished work on behalf of circa10a February 22, 2026 17:18

circa10a reviewed Mar 3, 2026

View reviewed changes

Copilot started work on behalf of circa10a March 3, 2026 21:43 View session

Copilot AI and others added 2 commits March 3, 2026 21:47

Fix splice-while-iterating: iterate backwards in Slack reminder daemon

f6acd5d

Co-authored-by: circa10a <21261388+circa10a@users.noreply.github.com>

Copilot AI changed the title ~~Add Slack support via symmetric message adapter pattern~~ Add Slack support via unified entry point with symmetric adapter pattern Mar 3, 2026

Copilot AI requested a review from circa10a March 3, 2026 21:49

Copilot finished work on behalf of circa10a March 3, 2026 21:49

circa10a closed this Mar 4, 2026

circa10a deleted the copilot/add-slack-support branch March 4, 2026 21:40

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add Slack support via unified entry point with symmetric adapter pattern#25

Add Slack support via unified entry point with symmetric adapter pattern#25
Copilot wants to merge 6 commits intomainfrom
copilot/add-slack-support

Copilot AI commented Feb 22, 2026 •

edited

Loading

Uh oh!

circa10a Feb 22, 2026

Uh oh!

Copilot AI Feb 22, 2026

Uh oh!

circa10a Feb 22, 2026

Uh oh!

Copilot AI Feb 22, 2026

Uh oh!

circa10a commented Feb 22, 2026

Uh oh!

Copilot AI commented Feb 22, 2026

Uh oh!

circa10a Mar 3, 2026

Uh oh!

circa10a Mar 3, 2026

Uh oh!

Copilot AI Mar 3, 2026

Uh oh!

Copilot AI commented Mar 3, 2026

I tried to connect to the following addresses, but was blocked by firewall rules:

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

Copilot AI commented Feb 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Approach

Changes

Config

Uh oh!

circa10a Feb 22, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 22, 2026

Choose a reason for hiding this comment

Uh oh!

circa10a Feb 22, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 22, 2026

Choose a reason for hiding this comment

Uh oh!

circa10a commented Feb 22, 2026

Uh oh!

Copilot AI commented Feb 22, 2026

Uh oh!

circa10a Mar 3, 2026

Choose a reason for hiding this comment

Uh oh!

circa10a Mar 3, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 3, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI commented Mar 3, 2026

I tried to connect to the following addresses, but was blocked by firewall rules:

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Copilot AI commented Feb 22, 2026 •

edited

Loading