docs(issues): [#2233] add review process tuning spec

[josecelano]

Summary

• Formalize the evidence-backed Tune the unified PR review process from first-use evidence #2233 specification for tuning the unified PR-review workflow.
• Archive completed Unify and make deterministic the PR review-processing workflow #2219 in the closed-issue buffer and update active metadata/navigation.
• Keep the review-workflow contract test aligned with the archived Unify and make deterministic the PR review-processing workflow #2219 specification.

Validation

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

Related to #2233

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.39%. Comparing base (a628b24) to head (eb7c41d).

Additional details and impacted files

@@           Coverage Diff            @@
##           develop    #2235   +/-   ##
========================================
  Coverage    85.39%   85.39%           
========================================
  Files          353      353           
  Lines        31315    31315           
  Branches     31315    31315           
========================================
+ Hits         26740    26741    +1     
+ Misses        4193     4190    -3     
- Partials       382      384    +2     

☔ 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]

🟡 Changes recommended

Address the outstanding metadata, archived-history, and verifier findings.

Get a fresh assessment by requesting another Copilot review.

Pull request overview

Formalizes the #2233 review-process tuning specification, archives #2219, and updates related workflow references and validation.

Changes:

• Adds the #2233 tuning specification.
• Archives #2219 and updates metadata and supporting documents.
• Aligns contract-test and skill references with the archived specification.

File summaries

File	Reviewed change	Findings
docs/issues/open/2233-2003-tune-unified-pr-review-process/ISSUE.md	Adds the tuning specification and implementation plan.	moderate (3): use draft status; strengthen the exact zero-context diff verification.
docs/issues/open/2003-overhaul-guardrails-and-automation/EPIC.md	Marks #2219 complete and archived.	nit (3): retain canonical timestamp format.
docs/issues/closed/2219-2003-unify-pr-review-processing/manual-verification-evidence.md	Updates archived metadata and evidence references.	nit (3): retain canonical timestamp format.
docs/issues/closed/2219-2003-unify-pr-review-processing/ISSUE.md	Archives #2219 and updates closure metadata.	nit (3): retain canonical timestamp format; moderate (1): set related-pr to 2232.
docs/issues/closed/2219-2003-unify-pr-review-processing/implementation-retrospective.md	Updates archived artifact references.	No final comments.
docs/issues/closed/2219-2003-unify-pr-review-processing/agent-review-reports.md	Preserves archived review history.	nit (2): append the later passing report.
contrib/dev-tools/checks/tests/test-agent-review-report-contract.sh	Points contract validation to the archived specification.	No final comments.
.github/skills/dev/pr-reviews/process-pr-review/SKILL.md	Updates the canonical archived-issue reference.	No final comments.

Review details

Suppressed comments (4)

docs/issues/closed/2219-2003-unify-pr-review-processing/ISSUE.md:11

• The archived issue's frontmatter must retain the canonical YYYY-MM-DD HH:MM UTC format (docs/skills/semantic-skill-link-convention.md:139-151). This changed value is ISO-8601 with T, seconds, and Z; rewrite it as minute precision.
  docs/issues/closed/2219-2003-unify-pr-review-processing/ISSUE.md:11
• PR #2232 is the merged PR that closes #2219, but the archived spec still leaves related-pr: null. The cleanup metadata should retain that implementation link; set related-pr to 2232 while archiving the issue.
  docs/issues/closed/2219-2003-unify-pr-review-processing/manual-verification-evidence.md:4
• Supplementary issue frontmatter uses the same canonical last-updated-utc: YYYY-MM-DD HH:MM format (docs/skills/semantic-skill-link-convention.md:139-151). This changed ISO-8601 value should be rewritten without T, seconds, or Z.
  docs/issues/open/2233-2003-tune-unified-pr-review-process/ISSUE.md:11
• Issue frontmatter requires last-updated-utc in YYYY-MM-DD HH:MM UTC form (docs/skills/semantic-skill-link-convention.md:150). This ISO-8601 value with T, seconds, and Z does not match the repository contract; use minute precision without those separators.

last-updated-utc: 2026-09-16T08:15:31Z

• Files reviewed: 7/8 changed files
• Comments generated: 3
• Review effort level: Lite

---

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[josecelano]

ACK eb7c41d