docs(pr-reviews): [#2233] tune review process guidance

[josecelano]

This PR tunes the unified PR-review process from the first-use evidence gathered in #2219/#2232.

Closes #2233.

Summary

• Aligns reviewer-facing PR review guidance and the advisory finding template with the unified author-side review workflow.
• Records code-span path evidence and defers strict path-reference enforcement to a broader semantic-link/frontmatter conventions EPIC.
• Adds migration safeguards to process-pr-review for retiring workflow documents and verifying rename purity.
• Adds a design-only tiered model routing note without implementing automation.
• Records manual verification, implementation retrospective, and independent task-review evidence.

Validation

• linter markdown
• linter cspell
• linter lychee
• git diff --check
• bash contrib/dev-tools/checks/tests/test-agent-review-report-contract.sh
• TORRUST_GIT_HOOKS_LOG_DIR=.tmp ./contrib/dev-tools/git/hooks/pre-commit.sh --format=text

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.39%. Comparing base (23a68e0) to head (177b8c4).

Additional details and impacted files

@@           Coverage Diff            @@
##           develop    #2237   +/-   ##
========================================
  Coverage    85.39%   85.39%           
========================================
  Files          353      353           
  Lines        31315    31315           
  Branches     31315    31315           
========================================
+ Hits         26740    26743    +3     
+ Misses        4193     4190    -3     
  Partials       382      382           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[Copilot]

Note

Copilot was unable to run its full agentic suite in this review.

Pull request overview

This PR updates the unified PR review process documentation for issue #2233 by tightening reviewer guidance (re-push scoping, re-raise rules, and checklist N/A semantics) and adding supporting evidence/design artifacts for deferred automation and path-reference conventions.

Changes:

• Extend reviewer finding-template guidance to cover re-pushed heads, re-raised finding IDs, and checklist N/A usage.
• Add issue-local evidence artifacts (manual verification, retrospective, agent review report) plus design notes (tiered routing, semantic-link conventions EPIC).
• Record and classify non-resolving Markdown code-span “path-like” references and explicitly defer strict enforcement to a broader conventions EPIC.

Reviewed changes

Copilot reviewed 10 out of 11 changed files in this pull request and generated 5 comments.

Show a summary per file

File	Description
docs/templates/REVIEW-FINDINGS.md	Adds guidance for handling re-pushes and checklist N/A semantics in review findings.
.github/skills/dev/pr-reviews/review-pr/SKILL.md	Updates reviewer skill guidance and formatting conventions; adds re-push scope and checklist rules.
.github/skills/dev/pr-reviews/process-pr-review/SKILL.md	Adds workflow guidance for “retiring workflow docs” inventories and “rename purity” mechanical verification.
docs/issues/open/2233-2003-tune-unified-pr-review-process/ISSUE.md	Updates #2233 spec to reflect delivered work, evidence, and deferral from checker implementation to conventions EPIC.
docs/issues/open/2233-2003-tune-unified-pr-review-process/tiered-model-routing-design.md	Introduces design-only model-routing handoff roles and gates to preserve review independence.
docs/issues/open/2233-2003-tune-unified-pr-review-process/manual-verification-evidence.md	Records human verification steps and observed results for the new guidance and design note.
docs/issues/open/2233-2003-tune-unified-pr-review-process/implementation-retrospective.md	Captures implementation learnings and scope change rationale (deferral of strict path enforcement).
docs/issues/open/2233-2003-tune-unified-pr-review-process/code-span-path-case-analysis.md	Summarizes taxonomy and recommended approach for path-like code spans in Markdown.
docs/issues/open/2233-2003-tune-unified-pr-review-process/code-span-path-case-inventory.tsv	Provides the full inventory backing the T2 scope/deferral decision.
docs/issues/open/2233-2003-tune-unified-pr-review-process/agent-review-reports.md	Persists independent review reports and follow-up items for the issue’s documentation changes.
docs/issues/drafts/refactor-semantic-link-conventions/EPIC.md	Defines a draft EPIC to formalize future frontmatter/semantic-link/path-reference conventions and validation strategy.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[josecelano]

ACK 177b8c4