docs: distinguish complete reviews from clean reviews

[BenCodez]

Summary

Address the review comment on the already-merged #307: #307 (comment)

The previous Otherwise branch incorrectly required Review incomplete. whenever a review found a defect, even when coverage and validation were complete. This follow-up separates completeness from findings and publishing readiness:

Review state	Required output
Complete, with findings	Keep coverage complete and report prioritized findings; do not invent limitations.
Complete, without findings	Use No findings. in the findings section.
Incomplete, with or without findings	Use Review incomplete., explain actual missing/unresolved/stale evidence, and still report verified findings.

A completed, investigated failing check is evidence of a defect, not automatically a coverage gap. Unresolved failures still block publishing; the existing pre-publish gate and static-only review caveat are preserved.

Scope

Only .github/skills/code-review/SKILL.md changes: 19 insertions, 5 deletions. No additional files, application code, workflow changes, or changes to SimpleAPI/VotingPlugin.

Branched from master at c3bdd207c46824eee15358e62c6b7c28fd9bdfea, which includes #307. The existing open PRs and matching branch names were checked first.

Validation

• PASS: YAML frontmatter parsing; unchanged metadata, earlier skill sections, publishing gate, and shell examples.
• PASS: UTF-8/LF/final newline, whitespace, balanced Markdown fences, and bash -n on the unchanged shell examples (syntax only; no builds executed).
• PASS: Static text assertions for the three explicit output cases, removal of the erroneous fallback, and the failed-check clarification. These are document checks, not live agent behavioral tests.
• PASS: git diff --check and single-file scope in a disposable local fixture containing the exact old/new skill bytes. This was not a full repository checkout.
• PASS: GitHub comparison confirms one commit and only the requested file changed; uploaded blob 403efbb0f2bd3c43b3213e37874c384e6f6ca4fb matches the locally validated bytes.
• NOT RUN: Maven/JAR build (mvn is unavailable), live agent execution, or an independent reviewer. No build or independent clean-review result is claimed.

Kept as a draft. No external review was manually requested.

AI disclosure: This pull request was prepared with assistance from ChatGPT.

[chatgpt-codex-connector]

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review	Status	Commit	Review trigger
📝 Code Review	✅ Completed 2026-09-06T00:15:42.561158Z	9129d31	Draft marked ready
🔒 Security Review	✅ Completed 2026-09-06T00:17:34.734508Z	9129d31	Draft marked ready

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.