Skip to content

Conversation

@ovitrif
Copy link
Collaborator

@ovitrif ovitrif commented Nov 28, 2025

Description

  • Enable use_sticky_comment in Claude review workflow to update the same comment instead of creating new ones on each push
  • Update permissions from pull-requests: read to write for comment editing
  • Update permissions from issues: read to write for sticky comment feature
  • Remove manual gh pr comment instruction and tool from workflow (handled automatically by sticky comment)
  • Add additional_permissions: "actions:read" input to let Claude read CI results
  • Update claude.yml to allow Claude to create commits on implementation requests:
    • Change contents, pull-requests, issues permissions to write
    • Change fetch-depth to 0 for full git history
  • Add "AI Code Review with Claude" documentation section to README.md with usage examples

Preview

N/A - workflow and docs changes.

QA Notes

  • Verify the workflow action succeeds by confirming Claude still reviews.
  • On next PR with multiple pushes, confirm Claude updates the existing review comment instead of adding new ones.
  • Test @claude mention to verify it can create commits when requested.
  • Check README renders correctly on GitHub with the new Contributing section.

@ovitrif ovitrif self-assigned this Nov 28, 2025
@ovitrif ovitrif force-pushed the feat/update-claude-review branch from 710dae6 to eeb8890 Compare November 28, 2025 21:02
@ovitrif ovitrif enabled auto-merge November 28, 2025 21:04
@ovitrif ovitrif marked this pull request as draft November 28, 2025 21:04
auto-merge was automatically disabled November 28, 2025 21:04

Pull request was converted to draft

@ovitrif ovitrif force-pushed the feat/update-claude-review branch from eeb8890 to de68d6e Compare November 28, 2025 21:08
@ovitrif ovitrif marked this pull request as ready for review November 28, 2025 21:08
@ovitrif ovitrif enabled auto-merge November 28, 2025 21:09
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR modernizes the Claude Code review workflow to use sticky comments (updating a single comment instead of creating new ones on each push) and enables Claude to create commits for implementation requests. The changes update both automated PR reviews and on-demand Claude assistance, while adding comprehensive documentation to the README.

Key changes:

  • Enable sticky comment behavior in automated code reviews to reduce PR comment clutter
  • Grant write permissions and full git history access to allow Claude to create commits when requested
  • Add detailed "AI Code Review with Claude" documentation section with usage examples

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
.github/workflows/claude-code-review.yml Enables sticky comments, adds actions:read permission, removes manual comment tool, and upgrades permissions from read to write for PR/issue commenting
.github/workflows/claude.yml Upgrades permissions to write for contents/PRs/issues, changes fetch-depth to 0 for full git history, and aligns comments with permission changes
README.md Adds comprehensive "Contributing" section with Claude Code integration documentation, usage examples, and command reference table

ovitrif and others added 5 commits November 29, 2025 01:41
Add author_association checks to prevent untrusted users from
triggering @claude with write permissions. Only OWNER, MEMBER,
and COLLABORATOR can now invoke the on-demand Claude workflow.
@ovitrif ovitrif force-pushed the feat/update-claude-review branch from 496906d to 6507ef6 Compare November 29, 2025 00:42
@ovitrif ovitrif merged commit cb882a1 into master Nov 29, 2025
7 of 10 checks passed
@ovitrif ovitrif deleted the feat/update-claude-review branch November 29, 2025 14:09
BitcoinErrorLog pushed a commit to BitcoinErrorLog/bitkit-ios that referenced this pull request Dec 15, 2025
…review

chore: Make claude review update a single comment
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.

3 participants