Skip to content

Tighten PR review findings and state record publishing#145

Open
fuller-stack-dev wants to merge 1 commit into
openclaw:mainfrom
fuller-stack-dev:codex/review-tuning-state-records
Open

Tighten PR review findings and state record publishing#145
fuller-stack-dev wants to merge 1 commit into
openclaw:mainfrom
fuller-stack-dev:codex/review-tuning-state-records

Conversation

@fuller-stack-dev
Copy link
Copy Markdown

Summary

  • keep missing/mock-only real behavior proof from short-circuiting the PR code/docs review pass
  • add targeted review prompt checks for task/media duplicate guards, request scoping, fallback delivery, and docs/code contract drift
  • preserve newer durable review records during state publishes, including broad records syncs and exact record-file syncs, while retaining state-only job/PR egg preservation

Investigation

  • The first ClawSweeper run on fix: directly deliver completed media fallback openclaw#84371 stopped at mock-only proof and recorded no code findings.
  • A later re-review did catch the P1/P2 findings and synced the public review comment, but a later broad state publish reverted the durable record to the older no-findings report.

Testing

  • pnpm run check on Node 24.14.0

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.

1 participant