Self reviewer

Skill fengsihao23-eng/shen-hua/skills/self-reviewer

A Codex skill pack for long-running project work

Install

npx -y skills add fengsihao23-eng/shen-hua --skill self-reviewer

Assembled from the repository path, not quoted from the project. Check it against their README if it does not work.

One thing to look at

1 stars1 stars. Stars are a popularity signal and not a quality one, but at this level it is likely that nobody has read this closely except its author, and you would be relying on your own review.

What its author says it does

Copied from the file, not written here

Shen-Hua self-review workflow before finishing a task. Use before final responses, commits, PRs, handoffs, or readiness claims to inspect requirements coverage, diffs, risks, tests, missed edge cases, destructive changes, and unresolved work.

SKILL.md

2.3 KB, as published. Nobody here has run it

Self Reviewer

Self Reviewer is the final quality gate before saying the task is done.

Trigger Conditions

Use this skill when:

A work round is ending.
The agent is about to claim completion or readiness.
Code/config/docs changed.
A verification loop passed, failed, or was skipped.

Do Not Trigger

Do not use this skill for:

Pure Q&A with no project impact.
Early planning before implementation begins, unless the user asks for plan review.

Review Workflow

Re-read the user's latest request and project rules.
Read .agent/PLAN.md, .agent/TASKS.md, .agent/LOG.md, and .agent/HANDOFF.md.
Inspect the current diff or changed-file list when version control is available.
Check:
- requirement coverage
- accidental unrelated changes
- edge cases and error handling
- public API or contract changes
- security/privacy concerns
- missing tests or weak verification
- documentation or handoff gaps
If a serious issue is found and is in scope, fix it and rerun $verification-loop.
If not fixing, record the risk clearly.

Required `.agent/` Updates

Always update:

.agent/REVIEW_CHECKLIST.md: checklist result and findings.
.agent/LOG.md: self-review summary.
.agent/TASKS.md: any reopened or completed items.
.agent/HANDOFF.md: residual risks and next action.

Update when relevant:

.agent/DECISIONS.md: accepted risk or deliberate non-fix.

Other Skills

Use $verification-loop if no current verification exists after changes, or if the review leads to fixes.
Use $handoff-writer after self-review.

Subagents

A reviewer subagent may be useful for large diffs when permitted, but the main agent remains responsible for final judgment.

Verification Commands

Self-review itself is mostly inspection. Run commands only when review reveals missing or stale verification.

Output Format

SELF REVIEW
Requirements covered:
Changed files reviewed:
Verification freshness:
Findings:
Residual risks:
Ready status: