Skip to content

fix(pr-review): preserve addressed findings - #91

Merged
Troublor merged 1 commit into
mainfrom
williamaaroncheung/fix/review-thread-state
Jul 28, 2026
Merged

fix(pr-review): preserve addressed findings#91
Troublor merged 1 commit into
mainfrom
williamaaroncheung/fix/review-thread-state

Conversation

@Troublor

Copy link
Copy Markdown
Collaborator

Summary

  • track code-level finding status separately from GitHub thread resolution status
  • preserve addressed findings when thread resolution was intentionally skipped, while still detecting manually reopened confirmed threads
  • keep omitted prior findings conservatively open and reject unknown or duplicate finding IDs
  • load Claude actions from the trusted main branch instead of the PR checkout

Test plan

  • PYTHONDONTWRITEBYTECODE=1 python3 -m unittest discover -s .github/actions/claude-pr-review -p 'test_*.py' -v
  • actionlint -shellcheck='' .github/workflows/claude.yml .github/workflows/build-check.yml
  • git diff --check

This PR was generated by an automated agent.

@Troublor Troublor added the agent label Jul 27, 2026
@Troublor
Troublor merged commit 9bc6ebc into main Jul 28, 2026
3 checks passed
@Troublor
Troublor deleted the williamaaroncheung/fix/review-thread-state branch July 28, 2026 01:13
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