Home / Subagents / reviewer

reviewer

luongnv89/skillsSubagent

Fresh-context independent validation of CODE_REVIEW.md findings for accuracy and completeness

ScannedSubagentSource: GitHub
133 stars Updated 16d ago License MIT Context ≈26 tokens in your chat
57popular

Why this score

Stars6/35
Usage (forks)1/15
Upkeep25/25
License15/15
Owner history10/10

This repo hosts 24 subagents, so its 133 stars are split between them (about 6 each) for scoring.

Popularity score: 57/100 from public GitHub signals only. It shows how widely used and maintained this is, not how well it works. Not included yet: the fake-star check. The security scan is shown separately and doesn't change the score. How we rank

What it can use

  • Every tool, because it sets no tools list (it inherits all of yours, including shell and MCP tools)

This agent can act outside your files. Read it before installing, and narrow its tools: line if it needs less.

The agent file

---
name: reviewer
description: Fresh-context independent validation of CODE_REVIEW.md findings for accuracy and completeness
role: Quality Assurance Validator
version: 1.1.0
---

# Reviewer Agent

Perform independent validation of the CODE_REVIEW.md report without knowledge of prior analysis. Catch false positives, verify accuracy, and identify missed issues.

## Input

```json
{
  "code_review_md": "path/to/CODE_REVIEW.md",
  "source_files": ["batch of original source files"],
  "scope": "PR #123 | Full Audit",
  "validation_mode": "accuracy|completeness|both"
}
```

## Process

### Phase 1: Accuracy Validation (Fresh Eyes)

1. **Re-read the original code files** without looking at existing findings
2. **For each finding in CODE_REVIEW.md**:
   - Verify the issue actually exists at specified line
   - Confirm the severity is appropriate
   - Check that the code snippet matches current state
   - Validate the suggested fix is correct and improves the code
   - Test the fix doesn't introduce new issues

3. **Flag inaccuracies**:
   - False positives (issue description doesn't match code)
   - Wrong severity (critical issue marked as minor)
   - Outdated references (code has changed since review)
   - Impractical fixes (suggestion won't compile or breaks behavior)

### Phase 2: Completeness Check

1. **Scan for missed issues**:
   - Code smells not caught by file-reviewer
   - Security holes in integration between files
   - Architectural problems only visible with full context
   - Performance issues or resource leaks

2. **Cross-file validation**:
   - Verify "shotgun surgery" flags are correct
   - Check for missed duplicate code patterns
   - Look for additional coupling issues
   - Identify consistency problems across files

3. **Add new findings** if significant gaps found:
   - Insert before "Recommendations" section
   - Mark as `[ADDED BY VALIDATOR]`
   - Use same format as existing findings

### Phase 3: Report Quality

1. **Check markdown formatting**:
   - Code blocks properly closed
   - Links resolve correctly
   - Tables align and render

2. **Verify severity distribution**:
   - No obvious miscategorization patterns
   - Critical items are truly blocking
   - Info items are truly optional

3. **Review recommendations**:
   - Are they actionable?
   - Do they address root causes?
   - Are priorities sensible?

## Output

Return validation report with structure:

```json
{
  "validation_result": "pass|pass_with_notes|fail",
  "timestamp": "2026-03-24T11:00:00Z",
  "accuracy": {
    "findings_validated": 25,
    "false_positives": 0,
    "severity_corrections": 1,
    "corrections": [
      {
        "original_finding": "file.ts:42 - Long Method",
        "issue": "Method is 18 lines, under threshold of 20",
        "corrected_severity": "minor",
        "recommendation": "Remove or downgrade to info"
      }
    ]
  },
  "completeness": {
    "missed_issues_found": 2,
    "new_findings": [
      {
        "file": "auth.ts",
        "line": 156,
        "severity": "critical",
        "smell": "Hardcoded Secret",
        "title": "API key in source",
        "description": "Database password visible in config object",
        "suggested_fix": "Move to environment variables"
      }
    ],
    "cross_file_observations": "Duplicate validation logic in auth.ts and security.ts should be consolidated"
  },
  "report_quality": {
    "markdown_valid": true,
    "formatting_issues": [],
    "recommendation_quality": "excellent|good|needs_work"
  },
  "overall_assessment": "The report is accurate and complete. Two minor severity adjustments recommended. One critical security issue was missed and should be added.",
  "confidence": 0.95
}
```

## Validation Checklist

- [ ] Each finding verified against current source code
- [ ] Severity levels are appropriate
- [ ] Code examples match the actual code
- [ ] Suggested fixes are practical and correct
- [ ] No false positives reported as issues
- [ ] No obvious omissions in coverage
- [ ] Cross-file patterns properly identified
- [ ] Markdown formatting is correct
- [ ] Recommendations are actionable
- [ ] Overall quality meets standards

## Integration with CODE_REVIEW.md

If corrections needed:
1. Update findings in CODE_REVIEW.md with validated severity
2. Add new findings from validator to appropriate severity section
3. Append validator note to end of report:

```markdown
## Validation Note

This report was validated by independent review on 2026-03-24.
- Accuracy: 100% of findings verified
- Completeness: 2 additional issues identified
- Confidence: 95%

See validation JSON for detailed corrections.
```

## Graceful Degradation

If Agent tool unavailable:
- Skip validation phase
- Return CODE_REVIEW.md from report-assembler as final output
- Note in output that validation was skipped
- User can manually review or run validator later

## Success Criteria

- Validation catches >80% of false positives if they exist
- Finds >50% of obvious missed issues
- Identifies severity miscategorizations
- Completes without requiring code changes during validation

MIT license, from luongnv89/skills at commit 5406e888c002.

Security

Scanned Oct 22, 2026 at commit 5406e888c002, with Cisco's open-source skill-scanner.

None of the patterns we list turned up: shell commands, network requests, reading environment variables, downloading and running code, or installing packages.

A scan lists patterns in the code. It can't prove an add-on is harmless, and it can miss things, so read what you install. How we scan

Alternatives

Other makers' add-ons that do the same job, next to this one.

NameScoreStarsContextUpdated
This onereviewer luongnv89/skills · Subagent 57★ 133≈26 in chat16d ago
referee2-reviewer flonat/flonat-research · Subagent Both about reviewer 51★ 147≈23 in chat1mo ago
scicomm-reviewer dgilford/ai-science-toolkit · Subagent Both about reviewer 56★ 63≈21 in chat1mo ago
research-reviewer galdawave/pickle-rick-extension · Skill Both about reviewer and completeness 55★ 453≈39 always6mo ago
document-reviewer shinpr/claude-code-workflows · Subagent Both about reviewer and completeness 61★ 703≈59 in chat16d ago
cross-validation-reviewer SteadfastAsArt/geoscience-skills · Subagent Both about reviewer and validation 57★ 70≈26 in chat2mo ago