Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
29 changes: 14 additions & 15 deletions .github/workflows/claude-code-review.yml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
name: Claude Code Review

on:
pull_request_target:
pull_request:
types: [opened, reopened]
issue_comment:
types: [created]
Expand All @@ -14,18 +14,18 @@ concurrency:
jobs:
claude-review:
if: |
github.event_name == 'pull_request_target' ||
(github.event_name == 'issue_comment' &&
github.event.issue.pull_request &&
github.event_name == 'pull_request' ||
(github.event_name == 'issue_comment' &&
github.event.issue.pull_request &&
contains(github.event.comment.body, '@claude'))

runs-on: ubuntu-latest
permissions:
contents: read
pull-requests: write
issues: write
id-token: write

steps:
- name: Checkout repository
uses: actions/checkout@v4
Expand All @@ -39,10 +39,10 @@ jobs:
with:
anthropic_api_key: ${{ secrets.ANTHROPIC_API_KEY }}
github_token: ${{ secrets.GITHUB_TOKEN }}

# Optional: Specify model (defaults to Claude Sonnet 4, uncomment for Claude Opus 4)
# model: "claude-opus-4-20250514"

# Direct prompt for automated review (no @claude mention needed)
direct_prompt: |
Please review this pull request and provide feedback on:
Expand All @@ -51,28 +51,27 @@ jobs:
- Performance considerations
- Security concerns
- Test coverage

Be constructive and helpful in your feedback.

# Optional: Customize review based on file types
# direct_prompt: |
# Review this PR focusing on:
# - For TypeScript files: Type safety and proper interface usage
# - For API endpoints: Security, input validation, and error handling
# - For React components: Performance, accessibility, and best practices
# - For tests: Coverage, edge cases, and test quality

# Optional: Different prompts for different authors
# direct_prompt: |
# ${{ github.event.pull_request.author_association == 'FIRST_TIME_CONTRIBUTOR' &&
# ${{ github.event.pull_request.author_association == 'FIRST_TIME_CONTRIBUTOR' &&
# 'Welcome! Please review this PR from a first-time contributor. Be encouraging and provide detailed explanations for any suggestions.' ||
# 'Please provide a thorough code review focusing on our coding standards and best practices.' }}

# Optional: Add specific tools for running tests or linting
# allowed_tools: "Bash(npm run test),Bash(npm run lint),Bash(npm run typecheck)"

# Optional: Skip review for certain conditions
# if: |
# !contains(github.event.pull_request.title, '[skip-review]') &&
# !contains(github.event.pull_request.title, '[WIP]')

8 changes: 6 additions & 2 deletions .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -56,8 +56,12 @@ jobs:
push: true
tags: ${{ steps.meta.outputs.tags }}
labels: ${{ steps.meta.outputs.labels }}
cache-from: type=gha
cache-to: type=gha,mode=max
cache-from: |
type=gha
type=registry,ref=${{ env.REGISTRY }}/${{ env.IMAGE_NAME }}:buildcache
cache-to: |
type=gha,mode=max
type=registry,ref=${{ env.REGISTRY }}/${{ env.IMAGE_NAME }}:buildcache,mode=max

release:
name: Create GitHub Release
Expand Down
4 changes: 4 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
- Improved logging organization
- Updated documentation with logging capabilities and examples

### Performance
- Docker build caching in GitHub Actions (dual GHA + registry cache strategy)


## [0.1.0] - 2025-06-21

### Added
Expand Down
14 changes: 12 additions & 2 deletions CLAUDE.md
Original file line number Diff line number Diff line change
@@ -1,13 +1,23 @@

# CLAUDE.md

This file provides guidance to Claude Code (claude.ai/code) when working with code in this repository.

## We're following Issue-Based Development (IBD) workflow
1. Before running any Git/GitHub CLI `Bash` command (`git commit`, `gh issue create`, `gh pr create`, etc.), open the corresponding file in @workflows to review required steps.
2. Always apply the exact templates or conventions from the following files:
- @workflows/GITHUB-ISSUE.md → issues
- @workflows/GIT-COMMIT.md → commits
- @workflows/GITHUB-PR.md → pull requests
3. Keep one branch per issue; merging the PR must auto-close its linked issue.


## PROJECT STRUCTURE

```
claude-code-yolo/
├── CLAUDE.md # This file - project guidance for Claude
├── README.md # Main documentation
├── README.md # Main documentation
├── CHANGELOG.md # Version history
├── DEV-LOGS.md # Development journal
├── TODO.md # Task tracking
Expand All @@ -31,8 +41,8 @@ claude-code-yolo/
├── workflows/ # Development workflows
│ ├── GIT-COMMIT.md # Commit guidelines
│ ├── GITHUB-ISSUE.md # Issue creation guidelines
│ ├── GITHUB-PR.md # PR creation process
│ └── PRE-RELEASE.md # Release checklist
└── references/ # Documentation and research
├── chats/ # AI conversation logs
Expand Down
3 changes: 1 addition & 2 deletions DEV-LOGS.md
Original file line number Diff line number Diff line change
Expand Up @@ -34,14 +34,13 @@
---

## Issue Analysis: 2025-06-23

### [enhancement-completed] Claude Code Review workflow simplification

**Problem**: Overcomplicated workflow with manual duplicate detection using GitHub CLI.

**Solution**: Adopted ChatGPT pattern with critical fixes:
- `pull_request_target` → enables secret access for `ANTHROPIC_API_KEY`
- Concurrency groups → automatic duplicate prevention
- Concurrency groups → automatic duplicate prevention
- Proper checkout ref → works for comment-triggered reviews
- Removed complex GitHub CLI duplicate detection logic

Expand Down
Loading