Skip to content

fix(comments): keep label justifications current#148

Merged
Takhoffman merged 4 commits into
mainfrom
codex/label-review-dropoff
May 20, 2026
Merged

fix(comments): keep label justifications current#148
Takhoffman merged 4 commits into
mainfrom
codex/label-review-dropoff

Conversation

@Takhoffman
Copy link
Copy Markdown
Contributor

Summary

  • record PR label syncs as ClawSweeper-owned churn so label-only updates do not look like target activity
  • force durable review comment body refresh when ClawSweeper-owned label changes alter the rendered label explanations
  • add regression coverage for PR label sync timestamps and recent stale comment refreshes

Root Cause

PR-specific label sync helpers returned only updated labels, so apply mode did not know those helpers had changed GitHub labels and did not write labels_synced_at. Separately, sync-comments-only could skip updating a recent durable comment because of comment_sync_min_age_days, leaving the latest visible comment without the newly-rendered label justifications even though labels had changed.

Validation

  • pnpm run build
  • pnpm exec node --test test/clawsweeper.test.ts
  • pnpm exec oxfmt --check src/clawsweeper.ts test/clawsweeper.test.ts
  • pnpm run check

@Takhoffman Takhoffman force-pushed the codex/label-review-dropoff branch from 3788402 to 895ed3c Compare May 20, 2026 03:30
@Takhoffman Takhoffman merged commit f607adc into main May 20, 2026
7 checks passed
@Takhoffman Takhoffman deleted the codex/label-review-dropoff branch May 20, 2026 03:32
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