Skip to content

Commit 63530d3

Browse files
authored
Merge pull request #73 from cblecker/feat/pr-review-toolkit-improvements
feat(pr-review-toolkit): add specialists, improve signals, optimize collection
2 parents a595f4b + cd404ea commit 63530d3

4 files changed

Lines changed: 157 additions & 222 deletions

File tree

pr-review-toolkit/.claude-plugin/plugin.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "pr-review-toolkit",
3-
"version": "1.9.0",
3+
"version": "1.10.0",
44
"description": "Comprehensive PR review board using shared workflow context",
55
"author": {
66
"name": "cblecker",

pr-review-toolkit/README.md

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -95,6 +95,9 @@ repository files and use available read-only MCP tools to verify findings.
9595
| pr-test-analyzer | Functional code that should have corresponding tests | Analyzes test coverage completeness |
9696
| comment-analyzer | Changes add or modify comments, docstrings, or docs | Checks comment accuracy and maintainability |
9797
| type-design-analyzer | Changes introduce or modify type definitions in typed languages | Evaluates type design and invariant quality |
98+
| security-reviewer | Changes touch auth, crypto, tokens, credentials, or security-related code | Reviews for security vulnerabilities and unsafe patterns |
99+
| api-compat-reviewer | Changes touch public APIs, exports, or client-facing interfaces | Checks API compatibility and breaking changes |
100+
| concurrency-reviewer | Changes touch mutexes, locks, channels, goroutines, or parallel code | Reviews concurrency patterns for races and deadlocks |
98101

99102
Agent selection is liberal: when in doubt, the agent runs. All agents execute in
100103
parallel within a single workflow from the collected PR manifest and thread

pr-review-toolkit/skills/review-pr/SKILL.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ disable-model-invocation: true
66
arguments: [pr-url]
77
argument-hint: <github-pr-url>
88
allowed-tools:
9+
- ExitPlanMode
910
- Workflow
1011
- AskUserQuestion
1112
- Bash(git fetch origin refs/pull/*/merge)
@@ -45,6 +46,11 @@ The workflow and its agents are read-only — they must not call GitHub write to
4546
GitHub write tools may be used only after an exact preview and explicit final
4647
posting approval from the user.
4748

49+
## Exit Plan Mode
50+
51+
If plan mode is active, call `ExitPlanMode` now before proceeding. The workflow
52+
and its tool calls require manual mode to avoid unwanted permission prompts.
53+
4854
## Parse PR URL
4955

5056
Parse `$pr-url` to extract owner, repo, and PR number from:

0 commit comments

Comments
 (0)