Add MCP hook matchers and rewrite /review-pr for MCP #17

Merged

forgejo_admin merged 2 commits from 14-mcp-hooks-review-pr into main

2026-02-24 19:46:39 +00:00

forgejo_admin commented

2026-02-24 18:46:23 +00:00

Contributor

Summary

3 new hook scripts for MCP tool calls (block-mcp-merge, remind-mcp-review-loop, post-mcp-merge-rebase)
settings.json updated with MCP tool matchers in PreToolUse and PostToolUse
/review-pr rewritten to use forgejo-mcp tools instead of curl
PR body template referenced from pal-e-docs

Test Plan

mcp__forgejo__merge_approved_pr triggers ask permission
mcp__forgejo__submit_pr triggers review-loop reminder
/review-pr uses MCP tools for Forgejo repos
GitHub repos still use gh CLI

Closes #14

## Summary - 3 new hook scripts for MCP tool calls (block-mcp-merge, remind-mcp-review-loop, post-mcp-merge-rebase) - settings.json updated with MCP tool matchers in PreToolUse and PostToolUse - /review-pr rewritten to use forgejo-mcp tools instead of curl - PR body template referenced from pal-e-docs ## Test Plan - [ ] mcp__forgejo__merge_approved_pr triggers ask permission - [ ] mcp__forgejo__submit_pr triggers review-loop reminder - [ ] /review-pr uses MCP tools for Forgejo repos - [ ] GitHub repos still use gh CLI Closes #14

forgejo_admin added 1 commit

2026-02-24 18:46:23 +00:00

Add MCP hook matchers and rewrite /review-pr for MCP tools (#14 ) 4f82adfcc3

- 3 new hook scripts for MCP tool calls (block-mcp-merge, remind-mcp-review-loop, post-mcp-merge-rebase)
- settings.json updated with MCP tool matchers in PreToolUse and PostToolUse
- /review-pr rewritten to use forgejo-mcp tools instead of curl
- PR body template referenced from pal-e-docs

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

forgejo_admin added 1 commit

2026-02-24 19:34:47 +00:00

Fix review issues: hook output format, ff approach, review-pr completeness (#14 ) 4569c6c487

- CRITICAL: Match hookSpecificOutput JSON format from existing hooks
  (block-mcp-merge.sh, remind-mcp-review-loop.sh)
- MEDIUM: Use jq instead of grep for JSON parsing in post-mcp-merge-rebase.sh
- MEDIUM: Use git update-ref instead of git merge --ff-only (matches post-merge-rebase.sh)
- MEDIUM: Extract CWD from input JSON (matches post-merge-rebase.sh approach)
- MEDIUM: Add PR body template checklist step to review-pr.md
- MEDIUM: Add severity levels requirement to review output
- LOW: Remove misleading stdin comments from block-mcp-merge.sh
- LOW: Fix shebang to #!/bin/bash (matches existing hooks)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>