Skip to content

fix(review): suppress changelog-only PR findings#57

Merged
steipete merged 2 commits intoopenclaw:mainfrom
rubencu:codex/changelog-not-review-finding
May 9, 2026
Merged

fix(review): suppress changelog-only PR findings#57
steipete merged 2 commits intoopenclaw:mainfrom
rubencu:codex/changelog-not-review-finding

Conversation

@rubencu
Copy link
Copy Markdown
Contributor

@rubencu rubencu commented May 9, 2026

Summary

  • Stop ordinary OpenClaw PR review from treating a missing CHANGELOG.md entry as a contributor-facing review finding.
  • Normalize model decisions for OpenClaw PRs before report rendering so changelog-only findings cannot produce review findings, repair work plans, or fix-required markers.
  • Keep maintainer-owned landing/automerge changelog handling separate from patch-correctness review.

Context

ClawSweeper flagged openclaw/openclaw#77533 with a P3 changelog finding, but contributors are not expected to add changelog entries. This keeps the review verdict focused on code/test/docs correctness, proof, security, checks, and mergeability while preserving the existing maintainer-side changelog gate.

Validation

  • pnpm run check
  • git diff --check

@rubencu rubencu force-pushed the codex/changelog-not-review-finding branch 5 times, most recently from 53d4b99 to 406bda9 Compare May 9, 2026 11:54
@rubencu rubencu force-pushed the codex/changelog-not-review-finding branch from 406bda9 to 14f9008 Compare May 9, 2026 11:57
@steipete steipete merged commit 1d17854 into openclaw:main May 9, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants