[PR #150] [MERGED] refactor: Extract bot logic into separate modules #153

New issue

Closed

opened 2026-10-27 21:10:04 +00:00 by bsakaguchi · 0 comments

bsakaguchi commented

2026-10-27 21:10:04 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/Comfy-Org/Comfy-PR/pull/150
Author: @comfy-pr-bot
Created: 2/3/2026
Status: ✅ Merged
Merged: 2/3/2026
Merged by: @snomiao

Base: main ← Head: pr/bot-architecture-refactor

📝 Commits (10+)

285342f chore: remove debugging files
e6a5eb6 feat: add .bot directory with bot instructions and skill definitions
9b1dab6 refactor: update .claude/skills definitions
f509df3 refactor: reorganize bot/code structure
5ba220b feat: add bot/box package configuration
6017163 refactor: simplify bot/index.ts with template loader
d9e94c6 feat: add template loader for bot instructions and skills
287947f refactor: rename pr-bot to prbot in CLI
ac00a48 fix: improve Slack message parsing and daily report
612b14b docs: update CLAUDE.md with prbot naming

📊 Changes

62 files changed (+6713 additions, -1722 deletions)

View changed files

➕ .TODO.md (+85 -0)
➕ .bot/AGENT.md (+843 -0)
➕ .bot/CLAUDE.md (+1 -0)
➕ .bot/README.md (+1 -0)
➕ .bot/SMART-READ-FEATURE.md (+144 -0)
➕ .bot/skills/code-search/SKILL.md (+22 -0)
➕ .bot/skills/github-issue-search/SKILL.md (+19 -0)
➕ .bot/skills/github-pr-bot/SKILL.md (+19 -0)
➕ .bot/skills/notion-search/SKILL.md (+19 -0)
➕ .bot/skills/registry-search/SKILL.md (+19 -0)
➕ .bot/skills/repo-reading/SKILL.md (+21 -0)
➕ .bot/skills/slack-file-sharing/SKILL.md (+31 -0)
➕ .bot/skills/slack-messaging/SKILL.md (+24 -0)
➕ .bot/skills/web-research/SKILL.md (+13 -0)
📝 .claude/skills/code-search/SKILL.md (+6 -6)
📝 .claude/skills/github-issue-search/SKILL.md (+6 -6)
📝 .claude/skills/github-pr-bot/SKILL.md (+6 -6)
📝 .claude/skills/notion-search/SKILL.md (+4 -4)
📝 .claude/skills/pr-bot-cli/SKILL.md (+17 -17)
📝 .claude/skills/registry-search/SKILL.md (+5 -5)

...and 42 more files

📄 Description

Summary

Simplify bot/index.ts to be a clean entry point
Extract Slack bot implementation to dedicated modules
Improve code organization and separation of concerns
Update comprehensive documentation for bot agents and skills

Changes

Code Structure

bot/index.ts: Simplified to minimal entry point
bot/slack-bot.ts: Main Slack bot implementation
bot/slack-bolt.ts: Slack Bolt framework integration
bot/codesearch-ai-wip.ts: Code search AI integration (WIP)
lib/ai/: New directory for AI utilities
- format-message.ts: Message formatting helpers
- index.ts: AI utility exports

Documentation

Updated .bot/AGENT.md with latest prbot CLI commands
Enhanced skill documentation for all bot capabilities
Added docs/ai-output-format.md for AI response formatting
Updated CLAUDE.md with bot architecture details
Improved docs/slack-api-for-agents.md

Cleanup

Removed deprecated bot/ai.ts

Benefits

Better code organization and maintainability
Clear separation between entry point and implementation
Easier to test individual components
Improved documentation for bot developers

🤖 Generated with Claude Code

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/Comfy-Org/Comfy-PR/pull/150 **Author:** [@comfy-pr-bot](https://github.com/comfy-pr-bot) **Created:** 2/3/2026 **Status:** ✅ Merged **Merged:** 2/3/2026 **Merged by:** [@snomiao](https://github.com/snomiao) **Base:** `main` ← **Head:** `pr/bot-architecture-refactor` --- ### 📝 Commits (10+) - [`285342f`](https://github.com/Comfy-Org/Comfy-PR/commit/285342fe27f45ec89956450f4cbfb2e91e23e565) chore: remove debugging files - [`e6a5eb6`](https://github.com/Comfy-Org/Comfy-PR/commit/e6a5eb68d60ada8455a3e5508632f75b7c67aa0b) feat: add .bot directory with bot instructions and skill definitions - [`9b1dab6`](https://github.com/Comfy-Org/Comfy-PR/commit/9b1dab6d7deaac0eb9f71bd8b691a51d6f76b22d) refactor: update .claude/skills definitions - [`f509df3`](https://github.com/Comfy-Org/Comfy-PR/commit/f509df35522f72476b37644031fa90750f3a8f8c) refactor: reorganize bot/code structure - [`5ba220b`](https://github.com/Comfy-Org/Comfy-PR/commit/5ba220bb4e8f316ec9c712a6f18340b34dc54493) feat: add bot/box package configuration - [`6017163`](https://github.com/Comfy-Org/Comfy-PR/commit/601716395a6132cb8401a0905fba9a33d68044fe) refactor: simplify bot/index.ts with template loader - [`d9e94c6`](https://github.com/Comfy-Org/Comfy-PR/commit/d9e94c6a6097519c8f839409a314e4ccc64fa37a) feat: add template loader for bot instructions and skills - [`287947f`](https://github.com/Comfy-Org/Comfy-PR/commit/287947feae3c750a9adff193f181b01c7f8bfb37) refactor: rename pr-bot to prbot in CLI - [`ac00a48`](https://github.com/Comfy-Org/Comfy-PR/commit/ac00a48504f8d698c7f130a90776e0fb346edf86) fix: improve Slack message parsing and daily report - [`612b14b`](https://github.com/Comfy-Org/Comfy-PR/commit/612b14b123d4bc3b53c23f6991b4d6255a62f804) docs: update CLAUDE.md with prbot naming ### 📊 Changes **62 files changed** (+6713 additions, -1722 deletions) <details> <summary>View changed files</summary> ➕ `.TODO.md` (+85 -0) ➕ `.bot/AGENT.md` (+843 -0) ➕ `.bot/CLAUDE.md` (+1 -0) ➕ `.bot/README.md` (+1 -0) ➕ `.bot/SMART-READ-FEATURE.md` (+144 -0) ➕ `.bot/skills/code-search/SKILL.md` (+22 -0) ➕ `.bot/skills/github-issue-search/SKILL.md` (+19 -0) ➕ `.bot/skills/github-pr-bot/SKILL.md` (+19 -0) ➕ `.bot/skills/notion-search/SKILL.md` (+19 -0) ➕ `.bot/skills/registry-search/SKILL.md` (+19 -0) ➕ `.bot/skills/repo-reading/SKILL.md` (+21 -0) ➕ `.bot/skills/slack-file-sharing/SKILL.md` (+31 -0) ➕ `.bot/skills/slack-messaging/SKILL.md` (+24 -0) ➕ `.bot/skills/web-research/SKILL.md` (+13 -0) 📝 `.claude/skills/code-search/SKILL.md` (+6 -6) 📝 `.claude/skills/github-issue-search/SKILL.md` (+6 -6) 📝 `.claude/skills/github-pr-bot/SKILL.md` (+6 -6) 📝 `.claude/skills/notion-search/SKILL.md` (+4 -4) 📝 `.claude/skills/pr-bot-cli/SKILL.md` (+17 -17) 📝 `.claude/skills/registry-search/SKILL.md` (+5 -5) _...and 42 more files_ </details> ### 📄 Description ## Summary - Simplify `bot/index.ts` to be a clean entry point - Extract Slack bot implementation to dedicated modules - Improve code organization and separation of concerns - Update comprehensive documentation for bot agents and skills ## Changes ### Code Structure - **bot/index.ts**: Simplified to minimal entry point - **bot/slack-bot.ts**: Main Slack bot implementation - **bot/slack-bolt.ts**: Slack Bolt framework integration - **bot/codesearch-ai-wip.ts**: Code search AI integration (WIP) - **lib/ai/**: New directory for AI utilities - `format-message.ts`: Message formatting helpers - `index.ts`: AI utility exports ### Documentation - Updated `.bot/AGENT.md` with latest prbot CLI commands - Enhanced skill documentation for all bot capabilities - Added `docs/ai-output-format.md` for AI response formatting - Updated `CLAUDE.md` with bot architecture details - Improved `docs/slack-api-for-agents.md` ### Cleanup - Removed deprecated `bot/ai.ts` ## Benefits - Better code organization and maintainability - Clear separation between entry point and implementation - Easier to test individual components - Improved documentation for bot developers 🤖 Generated with [Claude Code](https://claude.com/claude-code) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

bsakaguchi

2026-10-27 21:10:04 +00:00