anthropics / anthropics/claude-code-action

Not Updating Pull Request Comments

Đang mở
#602 4 bình luận 4 reaction 0 người được giao Xem trên GitHub
bug p1
Ngôn ngữ chính
TypeScript
Star
8.9k
Fork
2.1k
Merge trung bình
3 ngày 9 giờ
Pull request đã merge (30 ngày)
10

Mô tả

**Describe the bug**
The job successfully completes, but it does not update the GitHub comment on the pull request.

Image

**To Reproduce**
Steps to reproduce the behavior:

1. Install my workflow
2. Trigger the workflow by tagging the bot to review it
3. Wait for job to invoke and finish
4. Comment gets stuck

Image

**Expected behavior**
The original comment to be updated.

**Screenshots**
See above

**Workflow yml file**
```yml
name: Claude Code Review
on:
issue_comment:
types: [created]
pull_request_review_comment:
types: [created]
issues:
types: [opened, assigned]
pull_request:
types: [opened, synchronize, ready_for_review, reopened]

concurrency:
group: ${{ github.workflow }}-${{ github.ref }}
cancel-in-progress: true

jobs:
review:
timeout-minutes: 10
runs-on: ubuntu-latest
permissions:
contents: write
pull-requests: write
issues: write
id-token: write
actions: read
steps:
- uses: actions/checkout@v4
with:
fetch-depth: 1

- uses: anthropics/claude-code-action@v1
with:
anthropic_api_key: ${{ secrets.ANTHROPIC_API_KEY }}
track_progress: true
prompt: |
REPO: ${{ github.repository }}
PR NUMBER: ${{ github.event.pull_request.number }}

# Review PR #${{ github.event.pull_request.number }} in ${{ github.repository }} against our team checklist:

## Code Quality
- [ ] Code follows our style guide
- [ ] No commented-out code
- [ ] Meaningful variable names
- [ ] DRY principle followed
- [ ] SOLID principle followed
- [ ] Code is properly formatted and adheres to the editorconfig
- [ ] Doesn't introduce new warnings or errors
- [ ] Use meaningful variable and function names (avoid single letters except for loop counters)
- [ ] Remove unused code and commented-out code blocks

## Testing
- [ ] Unit tests for new functions
- [ ] Integration tests for new endpoints
- [ ] Edge cases covered
- [ ] Test coverage > 60%

## Documentation
- [ ] README updated if needed
- [ ] CLAUDE updated if needed
- [ ] API docs updated
- [ ] Inline comments for complex logic
- [ ] CHANGELOG.md updated
- [ ] Method intellisense (XML) comments
- [ ] Pull request description should have an issue attached to it that it can close after merging

## Security
- [ ] No hardcoded credentials
- [ ] Input validation implemented
- [ ] Proper error handling
- [ ] No sensitive data in logs

## Logging
- [ ] Proper logging implemented
- [ ] Logging with Sentry handled correctly
- [ ] User-friendly error messages
- [ ] Metrics/telemetry added for new features
- [ ] Health check endpoints updated

## Dependencies
- [ ] New dependencies justified
- [ ] No known vulnerabilities

## Breaking Changes
- [ ] Breaking changes documented
- [ ] Migration guide provided
- [ ] Deprecated features marked

## Git Hygiene
- [ ] Commit messages are clear and conform to conventional commits (https://www.conventionalcommits.org/en/v1.0.0/ and https://medium.com/neudesic-innovation/conventional-commits-a-better-way-78d6785c2e08)
- [ ] Commit messages should contain `[optional scope]: `
- [ ] Refrain from using emojis
- [ ] No merge commits (if using rebase workflow)
- [ ] Branch is up to date with base
- [ ] For description, use the future tense. Instead of fixed some bug, write fix some bug

## Code Review Specific
- [ ] PR description is clear and complete
- [ ] Screenshots/recordings for UI changes
- [ ] Related issues linked
- [ ] Deployment notes provided

For each item, check if it's satisfied and comment on any that need attention.
Post a summary comment with checklist results. Feel free to use emojis to make it easier to read.

claude_args: |
--mcp-config '{"mcpServers":{"sequential-thinking":{"command":"npx","args":["-y","@modelcontextprotocol/server-sequential-thinking"]},"fetch":{"command":"npx","args":["-y","@modelcontextprotocol/server-fetch"]},"memory":{"command":"npx","args":["-y","@modelcontextprotocol/server-memory"]},"github":{"command":"npx","args":["-y","@modelcontextprotocol/server-github"],"env":{"GITHUB_PERSONAL_ACCESS_TOKEN":"${{ secrets.GITHUB_TOKEN }}"}},"filesystem":{"command":"npx","args":["-y","@modelcontextprotocol/server-filesystem","${{ github.workspace }}"]}}}'
--allowedTools "mcp__github_inline_comment__create_inline_comment,Bash(gh pr comment:*),Bash(gh pr diff:*),Bash(gh pr view:*),Read,Write,Edit,Bash(git:*),mcp__sequential-thinking__sequentialthinking,mcp__fetch__fetch,mcp__memory__create_entities,mcp__memory__create_relations,mcp__github__create_or_update_file,mcp__github__search_repositories,mcp__github__create_repository,mcp__github__get_file_contents,mcp__github__push_files,mcp__github__create_issue,mcp__github__create_pull_request,mcp__github__fork_repository,mcp__github__create_branch,mcp__filesystem__read_file,mcp__filesystem__read_multiple_files,mcp__filesystem__write_file,mcp__filesystem__edit_file,mcp__filesystem__create_directory,mcp__filesystem__list_directory,mcp__filesystem__move_file,mcp__filesystem__search_files,mcp__filesystem__get_file_info,WebFetch"
--model claude-sonnet-4-5-20250929
--max-turns 15
```

**API Provider**

[X] Anthropic First-Party API (default)
[ ] AWS Bedrock
[ ] GCP Vertex

**Additional context**
This wasn't happening when I was using the `beta` version

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Đánh giá

Issue này chưa được đánh giá.

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.