Skip to content

fix(pr-review): resolve job-token review threads - #89

Merged
Troublor merged 1 commit into
mainfrom
williamaaroncheung/fix/pr-review-thread-resolution
Jul 27, 2026
Merged

fix(pr-review): resolve job-token review threads#89
Troublor merged 1 commit into
mainfrom
williamaaroncheung/fix/pr-review-thread-resolution

Conversation

@Troublor

Copy link
Copy Markdown
Collaborator

Summary

  • recognize the GitHub job-token publisher while requiring state markers before adopting generic Actions-authored review threads
  • bind inline findings to exact review and comment IDs with bounded API propagation retries
  • verify thread-resolution mutations before publishing a clean status, and preserve body-only fallback semantics
  • add regression coverage for the PR #230 failure sequence and related recovery paths

Test plan

  • PYTHONDONTWRITEBYTECODE=1 python3 -m unittest discover -s .github/actions/claude-pr-review -p "test_*.py" -v
  • git diff origin/main...HEAD --check
  • Prettier check on .github/actions/README.md
  • markdownlint on .github/actions/README.md

This PR was generated by an automated agent.

@Troublor Troublor added the agent label Jul 27, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Claude review status

✅ Review clean

Last reviewed: head d7653b52

New this round: 0 · Resolved this round: 0 · Open questions: 0

@Troublor
Troublor merged commit 2b175b3 into main Jul 27, 2026
5 checks passed
@Troublor
Troublor deleted the williamaaroncheung/fix/pr-review-thread-resolution branch July 27, 2026 08:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant