self-review

Featured

Self-review a changeset until merge-ready — a fresh-context reviewer checks it as a maintainer would, the author answers every finding, and a compact report for the PR proves the review happened.

Code & Development 46 stars 6 forks Updated today MIT

Install

View on GitHub

Quality Score: 89/100

Stars 20%
56
Recency 20%
100
Frontmatter 20%
70
Documentation 15%
100
Issue Health 10%
50
License 10%
100
Description 5%
100

Skill Content

# Self-review Make a changeset merge-ready before submission — no regressions, sound code, the project's conventions respected — and prove it was scrutinized: a fresh-context reviewer hunts for what would block the merge, the author answers every finding, and a compact report — scannable by a maintainer in seconds — records the outcome. The report is the review record only; the change's what/why belongs to the PR description (e.g. via /handover), never here. Only for a changeset you authored. The fresh-context reviewer exists to escape authoring blindness, so someone else's PR has nothing to escape and belongs to `maintainer-review`. ## Resolve the changeset 1. Resolve source and target. No input → current branch against the auto-detected target: the default branch of the `upstream` remote when one exists (fork workflow), else of `origin` — via `git ls-remote --symref <remote> HEAD`, never the often-absent local `refs/remotes/<remote>/HEAD`; that failing (e.g. offline), the sole existing candidate among `main`, `master`, `develop`/`development` (preferring `upstream`'s remote-tracking ref, then `origin`'s, then the local branch) — still ambiguous or none → ask. Explicit branches in the invocation win; a detached HEAD → ask which branch is under review. State the chosen target. 2. The reviewed state is the source as it stands — working tree when checked out, else tip. Uncommitted work is reviewed, not blocked, once the author sorts it: modified tracked...

Details

Author
eai-org
Repository
eai-org/agent-toolkit
Created
3 months ago
Last Updated
today
Language
Shell
License
MIT

Bundled in these plugins

Similar Skills

Semantically similar based on skill content — not just same category

Code & Development Listed

self-review

Review a completed diff or change set for correctness, security, regressions, scope, and missing tests. Use in author mode before handoff or in read-only mode when the user asks to review, inspect, check readiness, or report findings. Do not replace runtime completion evidence from verify-done, and do not fix findings during a review-only request.

1 Updated 3 weeks ago
adityaarakeri
Code & Development Listed

dev-review-changes

Reviews pending changes against the project's rules and reports findings with a severity verdict. Read-only - never edits, stages, or commits.

2 Updated today
ainova-systems
Code & Development Listed

self-review

Review your OWN in-progress change as a hostile stranger would, before any PR exists: run it mid-work or right before pushing. Gathers the contract first (Jira issue AC, commit messages, the diff's own claim, sibling conventions), then runs nine passes (contract conformance, completeness, premise verification, semantic delta, failure paths, blind spots, blast radius, convention, naming) and fans them out to parallel subagents so each angle is judged independently, then merges them into one ranked Korean finding list with file:line evidence. Its job is to counteract author tunnel vision by hunting the cases you never considered, including uncommitted and untracked work. Local analysis only: it never edits, commits, or posts. For judging other people's review comments on an existing PR, use review-triage instead. Triggers on phrases like '내 코드 셀프 리뷰', '푸시 전에 검토해줘', '작업 중인 변경 리뷰해줘', '빠뜨린 케이스 없는지 봐줘', '커밋 전에 스스로 리뷰', 'self-review my changes', 'review my own work before I push', 'what did I miss in this change'.

0 Updated 1 weeks ago
Srltas