feat: Add --triggers-dir flag to serve command #91
Conversation
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
Adds the ability to load Trigger resources from a directory when starting the trigger server, matching the existing --agents-dir and --flows-dir flags. Changes: - Added --triggers-dir CLI argument to serve command - Added TriggersConfig struct for config file support - Load triggers from directory using TriggerRegistry - Auto-register platforms based on trigger types (e.g., GitHub) - Log command bindings from loaded triggers Usage: aofctl serve --triggers-dir ./triggers/ --agents-dir ./agents/ 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
The GitHubPlatform requires both token and webhook_secret. Now reads GITHUB_TOKEN or GH_TOKEN from environment when registering GitHub platforms from triggers. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This fixes the /review and other slash commands not being processed when users comment on PRs or issues. Changes: - Add register_command_binding() method to TriggerHandler for dynamic command registration - Export CommandBinding from aof-triggers crate - Update serve.rs to actually register command bindings from Trigger YAML files (was only logging them, not registering) - Fix GitHub issue_comment event handling to detect slash commands: - When comment starts with /, use raw comment body as message text - Add comment_body, comment_id, comment_html_url to metadata - Add is_pr_comment flag to detect if comment is on a PR vs issue - Fix workflow_tests.rs to include new command_bindings field Now when a user comments "/review" on a PR, the system will: 1. Parse the webhook and extract the slash command 2. Match it against registered command bindings from Trigger YAML 3. Route to the configured fleet/agent/flow 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Fix channel_id to include #number suffix (e.g., owner/repo#8) - Enables send_response to post comments back to PR/issue - Add context construction from PR metadata when command text is empty - Fixes /review without arguments receiving proper PR context Fixes #92 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
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.
Summary
Adds
--triggers-dirflag toaofctl servecommand and fixes GitHub webhook response posting.Fixes #92
Changes
New Feature:
--triggers-dirFlagBug Fixes
owner/repo#8) so responses can be posted back as comments/reviewwithout arguments)/commandsdirectly in issue_comment textUsage
Example Trigger File
Testing
🤖 Generated with Claude Code