docs/archive/2026-07-ai-artifacts/claude-skills/feature-pipeline-preview/17-opus-pr-review/SKILL.md
> **ARCHIVAL KNOWLEDGE — DO NOT USE OR REFERENCE.** > Superseded by `codemap/` + root `AGENTS.md`. Scheduled for deletion once the pipeline is verified (Phase 6). > Manifest: `docs/archive/2026-07-ai-artifacts/MANIFEST.md` --- name: 17-opus-pr-review description: Opus reviews the open PR locally, proposes fixes, pauses for maintainer approval; then applies the accepted fixes. kind: process executor: opus model: claude-opus-4-7 --- # Opus PR Review After the PR is opened (phase 16), an Opus re
npx skillsauth add neohaskell/neohaskell docs/archive/2026-07-ai-artifacts/claude-skills/feature-pipeline-preview/17-opus-pr-reviewInstall this skill globally with one command. Works with Claude Code, Cursor, and Windsurf.
3 of 9 scanners reported clean
Some scanners were skipped, did not run, or reported a non-clean status. Review each row below.
ARCHIVAL KNOWLEDGE — DO NOT USE OR REFERENCE. Superseded by
codemap/+ rootAGENTS.md. Scheduled for deletion once the pipeline is verified (Phase 6). Manifest:docs/archive/2026-07-ai-artifacts/MANIFEST.md
After the PR is opened (phase 16), an Opus reviewer reads the diff against a strict NeoHaskell quality rubric and proposes fixes locally — NOT as inline GitHub comments. The phase PAUSES for the maintainer to accept/reject individual proposed fixes by editing .pipeline/findings-17.json. On pipeline.py approve 17, the orchestrator applies the accepted fixes, commits each one, and pushes.
./01-review/SKILL.md and follow it. Verify: .pipeline/findings-17.json exists with an array of findings; a markdown digest is surfaced in the orchestrator's conversation. The leaf calls pipeline.py complete 17 and stops. The orchestrator PAUSES.pipeline.py approve 17. Spawn an Agent (model: sonnet) and instruct it to read ./02-apply/SKILL.md and follow it. Verify: each maintainer_decision == "accept" entry produced a corresponding commit on the branch; cabal build all + cabal test all are still green; git push updated the PR.maintainer_decision != "accept"..pipeline/findings-17.json (typically to flip pending → accept/reject/defer per entry).cabal build all + cabal test all AFTER each fix and refuses to commit if either breaks.development
> **ARCHIVAL KNOWLEDGE — DO NOT USE OR REFERENCE.** > Superseded by `codemap/` + root `AGENTS.md`. Scheduled for deletion once the pipeline is verified (Phase 6). > Manifest: `docs/archive/2026-07-ai-artifacts/MANIFEST.md` --- name: neohaskell-style-guide description: NeoHaskell coding style reference and enforcement rules. Load when writing, reviewing, or modifying any Haskell code in the NeoHaskell project. Triggers on 'NeoHaskell style', 'NeoHaskell conventions', 'how to write NeoHaskell', '
development
> **ARCHIVAL KNOWLEDGE — DO NOT USE OR REFERENCE.** > Superseded by `codemap/` + root `AGENTS.md`. Scheduled for deletion once the pipeline is verified (Phase 6). > Manifest: `docs/archive/2026-07-ai-artifacts/MANIFEST.md` --- name: neohaskell-security-review description: Security & Code Quality review for NeoHaskell. Use when reviewing code changes, PRs, or architectural decisions for security implications. Evaluates OWASP, NIST, EU compliance. Handles pipeline phases 2 (ADR review) and 10 (im
development
> **ARCHIVAL KNOWLEDGE — DO NOT USE OR REFERENCE.** > Superseded by `codemap/` + root `AGENTS.md`. Scheduled for deletion once the pipeline is verified (Phase 6). > Manifest: `docs/archive/2026-07-ai-artifacts/MANIFEST.md` --- name: neohaskell-qa-designer description: Testing QA Designer for NeoHaskell. Designs comprehensive test specifications with exhaustive edge cases, boundary conditions, and happy paths BEFORE implementation. Handles pipeline phase 6 (Test Spec Design). Outside-in TDD meth
development
> **ARCHIVAL KNOWLEDGE — DO NOT USE OR REFERENCE.** > Superseded by `codemap/` + root `AGENTS.md`. Scheduled for deletion once the pipeline is verified (Phase 6). > Manifest: `docs/archive/2026-07-ai-artifacts/MANIFEST.md` --- name: neohaskell-performance-review description: Performance review for NeoHaskell targeting 50k req/s throughput. Use when reviewing code for performance implications, INLINE pragmas, strictness, and allocation patterns. Handles pipeline phases 3 (ADR review) and 11 (imp