Skip to content

Allow proof nudges to comment on PRs#244

Merged
Takhoffman merged 1 commit into
mainfrom
codex/proof-nudge-pr-comment-token
Jun 2, 2026
Merged

Allow proof nudges to comment on PRs#244
Takhoffman merged 1 commit into
mainfrom
codex/proof-nudge-pr-comment-token

Conversation

@Takhoffman

Copy link
Copy Markdown
Contributor

Summary

  • request pull-requests write permission for the proof-nudge target App token
  • assert the workflow keeps that permission in the proof-nudge workflow guard test

Why

The targeted execute run for the planned proof nudges failed before posting the first comment with HTTP 403: Resource not accessible by integration on POST /repos/openclaw/openclaw/issues/71563/comments. The existing proof-nudge token had issues:write but only pull-requests:read, while the active comment-capable repair lanes request pull-requests:write for PR targets.

Failed execute run: https://github.com/openclaw/clawsweeper/actions/runs/26827458777

Validation

  • node --test --test-name-pattern "proof nudge workflow" test/clawsweeper.test.ts
  • pnpm run format:check

@Takhoffman Takhoffman requested a review from a team as a code owner June 2, 2026 14:59
@clawsweeper

clawsweeper Bot commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

ClawSweeper status: review started.

I am starting a fresh review of this pull request: Allow proof nudges to comment on PRs This is item 1/1 in the current shard. Shard 0/1.

This placeholder means the worker is alive and reading the current context. I will edit this same comment with the actual review when the claws are done clicking.

Crustacean status: shell secured, claws on keyboard, evidence pebbles being sorted.

@Takhoffman Takhoffman merged commit b76b104 into main Jun 2, 2026
5 checks passed
@Takhoffman Takhoffman deleted the codex/proof-nudge-pr-comment-token branch June 2, 2026 15:03
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