Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 3 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ Claude Code and Copilot CLI can use this repository's native marketplace manifes
| Plugin | Contents |
| --- | --- |
| [deepwiki](plugins/deepwiki) | Public repository documentation tools and guidance |
| [engineering](plugins/engineering) | Interactive PR review skill |
| [engineering](plugins/engineering) | Interactive PR review and supplemental change impact review |
| [productivity](plugins/productivity) | Direct technical writing and behavioral evals |
| [finance](plugins/finance) | Financial modeling guidance and an original fictional XLSX example |
| [tradingview](plugins/tradingview) | TradingView market data MCP configuration |
Expand All @@ -32,6 +32,8 @@ The CLI-specific guidance plugin remains in [allagents/plugins/allagents](https:

Some plugins connect to external services; their availability and authentication are governed by those services. The catalogue contains reusable guidance and configurations, not personal credentials or records.

After a primary code review completes, invoke `change-impact-review` with its receipt, exact base/head revisions, and a writable report location. The [skill](plugins/engineering/skills/change-impact-review/SKILL.md) traces effects through unchanged code and reports supported additional findings without changing the primary review.

## Workspace templates

Workspace templates remain under [allagents/examples/workspaces](https://github.com/allagentsdev/allagents/tree/main/examples/workspaces). They compose plugins from this catalogue and other sources:
Expand Down
4 changes: 2 additions & 2 deletions plugins/engineering/.claude-plugin/plugin.json
Original file line number Diff line number Diff line change
@@ -1,9 +1,9 @@
{
"name": "engineering",
"description": "Reusable engineering workflow skills for repository maintenance, delivery, and interactive pull request review.",
"description": "Reusable engineering workflow skills for repository maintenance, delivery, interactive pull request review, and supplemental change impact review.",
"author": {
"name": "AllAgents"
},
"version": "1.0.1",
"version": "1.1.0",
"homepage": "https://github.com/allagentsdev/agent-plugins/tree/main/plugins/engineering"
}
38 changes: 38 additions & 0 deletions plugins/engineering/skills/change-impact-review/SKILL.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,38 @@
---
name: change-impact-review
description: Supplement a completed primary code review by tracing changed behavior through unchanged code and later effects. Use for an additional impact review after CE or another primary review.
---

# Change Impact Review

Report consequential defects the primary review missed. Prefer a fresh context when the host supports it; use the existing primary receipt rather than starting another primary review pipeline.

## Establish the scope

Read the primary review receipt and its findings. Verify that it completed and that its repository, exact fixed base/head revisions, and any recorded worktree changes match the source being inspected. Keep that scope fixed. If the receipt is missing, failed, stale, or cannot establish the match, return `blocked` with the concrete missing precondition. Claim CE completion only when its receipt supports it.

Inspect source read-only. Use existing authorized evidence; running code or tests, or changing source, Git, or GitHub requires separate caller authority. Save reports only to the caller's supplied writable artifact location, or return them inline when no location was supplied. Keep the primary receipt immutable.

## Follow the behavior

Trace each changed behavior through its reachable callers, unchanged helpers, shared state, and deferred work until its relevant effects are accounted for or an evidence gap is identified. Use symbol/reference tools or bounded text searches to inspect the execution path and cited code at the reviewed revision.

Consider success, failure, and repeated calls; cache identity and invalidation; trust boundaries; and interactive or control state where the change makes them relevant. These are reasoning prompts, not finding quotas.

For each suspected defect:

- Establish how the change introduces or newly exposes it, the concrete trigger and reachable path, and the supported consequence. Check suspected missing guards or fields against declarations and callers.
- Inspect whether relevant regression fixtures actually reach the behavior. Distinguish code-derived evidence from observed execution; preserve missing evidence and claim test runs or red/green results only when recorded.
- Compare the causal defect with the primary findings. Suppress duplicates even when wording, affected lines, or downstream symptoms differ.

Retain only additional consequential defects supported by that evidence. Keep unsupported concerns as explicitly separate questions or coverage gaps. Zero additional findings is valid.

## Return the supplemental report

Prefer the caller's structured format and existing finding schema. Otherwise return a concise, self-contained report with:

- `complete` or `blocked`, exact reviewed scope, and primary receipt reference.
- Additional findings, each with file/line, trigger and path evidence, cause tied to the change, and observed or supported consequence.
- Paths covered by this pass, unresolved paths/questions, and actual checks or execution limits.

Mark complete when the relevant effects have been traced or explicitly recorded as unresolved and every retained finding meets the evidence and deduplication criteria. State only this pass's coverage. Completion leaves the primary verdict unchanged and grants no authority to ship or apply fixes.
Loading