code-review

Verifies merge request completeness against requirements, mutation-tested assertions, and gate evidence before submission.

Updated Jun 4, 2026
One-click install
npx skills add https://github.com/tedeuxx/tadeumendonca-skills --skill code-review-tedeuxx
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: code-review
Source: https://github.com/tedeuxx/tadeumendonca-skills/tree/main/skills/code-review
Command: npx skills add https://github.com/tedeuxx/tadeumendonca-skills --skill code-review-tedeuxx

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve? Developers often open merge requests with unmet requirements, tautological tests that cannot fail, stale documentation claims, and unverified gate results, causing expensive review rounds and re-ratifications. This Skill runs an author-side completeness pass so a merge request arrives carrying evidence rather than claims. ## Core Features & Use Cases - Requirement and DoD Verification: Enumerates every Issue requirement and Definition of Done item, marking each met or unmet with real command output as evidence. - Mutation Testing Discipline: Instructs authors to mutate source code and confirm every new assertion can actually fail, catching tautological tests that coverage cannot see. - Drift and Production-Risk Sweeps: Greps the repository for claims the change made false, tries alternative spellings for anything parsed, and checks production-risk axes like dependencies, secrets, and deploy paths. - Use Case: Before opening a pull request on a hook or pipeline change, run this pass to catch a flag spelling bug, seven non-failing assertions, and stale README counts in minutes instead of across eight corrective commits. ## Quick Start Ask the agent to run the code-review completeness pass on your finished slice before opening the merge request.

Frequently Asked Questions about code-review

High-intent search queries and answers about installing and using this skill.

FAQPage Schema
How do I review my own code before opening a merge request?

Enumerate every requirement from the Issue and mark each met or unmet with evidence, then verify each Definition of Done item using real command output. Finish with gates run locally and their actual output pasted, never a claim that they passed.

How do I check if my tests can actually fail?

Mutate the source code, run the suite, and count the failures; a new assertion that adds no red asserts nothing. Watch for short-circuited branches and assertions comparing a literal against a component built from the same literal.

What is the difference between author-side review and gatekeeper review?

The author-side pass verifies completeness before submission so the request arrives with evidence, while the gatekeeper re-verifies independently in a fresh context. When the two disagree, the gate wins, and the author's check is never inherited as evidence.

How do I find documentation drift after a code change?

Grep the repository for the words of the old claim the change made false, not the tokens you edited, and disposition every hit before running any suite. Also check places that tell someone how to do the thing, since phrase sweeps find restatements but never omissions.

When should a rule be a hook instead of a checklist skill?

Irreversible acts like terraform destroy or git reset --hard need a mechanical hook that cannot be argued down. Rules about judgement, such as what finished means, belong in a skill because no command-string matcher can express intent.