Skip to content

Lots of claude refactoring on this one#26

Merged
circa10a merged 7 commits intomainfrom
fixes
Mar 4, 2026
Merged

Lots of claude refactoring on this one#26
circa10a merged 7 commits intomainfrom
fixes

Conversation

@circa10a
Copy link
Copy Markdown
Owner

@circa10a circa10a commented Mar 4, 2026

No description provided.

Copilot AI and others added 7 commits February 22, 2026 02:22
- 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>
…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>
…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>
@circa10a circa10a merged commit 37be012 into main Mar 4, 2026
3 checks passed
@circa10a circa10a deleted the fixes branch March 4, 2026 04:04
circa10a added a commit that referenced this pull request Mar 4, 2026
* Initial plan

* Add Slack support with message adapter pattern (resolves #10)

- 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>

* Move eslint peer deps to devDependencies

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

* Add Discord adapter, refactor to symmetric adapter pattern, pre-compile 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>

* Unify entry point: single `npm start` runs Discord, Slack, or both simultaneously

- 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>

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

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

* rewrite the whole fuckin thing

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Signed-off-by: circa10a <caleblemoine@gmail.com>
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