Skip to content

[feat] Add comprehensive Claude PR review with inline comments #4453

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 31 commits into from
Jul 29, 2025
Merged

[feat] Add comprehensive Claude PR review with inline comments #4453

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
31 commits
Select commit Hold shift + click to select a range
0694a96
[test] Add test service with intentional violations
christian-byrne Jul 14, 2025
0f871b5
[fix] Fix TypeScript errors to allow CI to pass
christian-byrne Jul 14, 2025
2271607
[test] Add browser test for TestReviewService
christian-byrne Jul 15, 2025
f8cbd15
[test] Add comment to browser test
christian-byrne Jul 15, 2025
39dc780
[fix] Update CI check names to match actual workflow names
christian-byrne Jul 15, 2025
4e70864
[fix] Fix Claude action configuration
christian-byrne Jul 15, 2025
13b826f
[fix] Fix Claude review execution limits
christian-byrne Jul 15, 2025
931d772
[fix] Allow all bash commands for PR review
christian-byrne Jul 15, 2025
b800104
[fix] Add actual analysis code to review command
christian-byrne Jul 16, 2025
8a806dd
[fix] Clarify Claude should analyze files, not just run bash
christian-byrne Jul 16, 2025
f3e4bb7
[fix] Add explicit instructions for Claude's analysis phase
christian-byrne Jul 16, 2025
187fe67
[fix] Update Claude PR review workflow to fix bash permission issues
christian-byrne Jul 24, 2025
79fe98e
[fix] Update bash permissions configuration to use specific command p…
christian-byrne Jul 24, 2025
19f45c0
Update locales [skip ci]
invalid-email-address Jul 24, 2025
51622eb
[refactor] Simplify Claude PR review workflow to use direct step-by-s…
christian-byrne Jul 24, 2025
4d70abb
[fix] Add inline comment posting to Claude PR review workflow
christian-byrne Jul 24, 2025
a60e7c0
[fix] Update Claude PR review workflow with comprehensive inline prom…
christian-byrne Jul 26, 2025
59b412f
[fix] Use prompt_file and fix inline comment syntax in comprehensive …
christian-byrne Jul 26, 2025
25cd97c
[fix] Fix CI check names in wait-for-ci step to match actual check names
christian-byrne Jul 26, 2025
d4280f9
[fix] Add id-token write permission for OIDC authentication
christian-byrne Jul 26, 2025
976130a
[fix] Use valid action parameters: label_trigger and direct_prompt
christian-byrne Jul 26, 2025
2acc16e
[fix] Make prompt more explicit about using Read tool to access revie…
christian-byrne Jul 26, 2025
a6b7ccb
[fix] Correct inline comment posting in comprehensive PR review
christian-byrne Jul 26, 2025
d9f4df3
[fix] Make bash execution requirement explicit in PR review workflow
christian-byrne Jul 26, 2025
d0deb4c
[refactor] Convert comprehensive review to natural language instructions
christian-byrne Jul 26, 2025
5e2ed16
[fix] Add explicit permissions instructions to comprehensive review
christian-byrne Jul 27, 2025
1df08d0
[fix] Convert gh api commands to single-line format
christian-byrne Jul 27, 2025
1677513
[feat] Add comprehensive PR review workflow with inline comments
christian-byrne Jul 27, 2025
ee42add
[feat] Update comprehensive PR review with natural language instructions
christian-byrne Jul 28, 2025
34dee8f
remove logs file
christian-byrne Jul 29, 2025
b82e0d7
remove test code [skip ci]
christian-byrne Jul 29, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Loading