SZUT/casino
SZUT/casino
Archived
3
1
Fork 0
Code Issues Pull requests 2 Releases 162 Packages 2 Wiki Activity Actions

fix: Force claude to use tea cli #249

Merged
jank merged 1 commit from claude/force into main 2025-06-01 11:40:05 +00:00
Conversation 3 Commits 1 Files changed 1 +24 -11
jank commented 2025-06-01 11:38:34 +00:00
Owner
Copy link
No description provided.
jank added 1 commit 2025-06-01 11:38:34 +00:00
fix: Force claude to use tea cli
All checks were successful
CI / Get Changed Files (pull_request) Successful in 30s
Details
CI / Docker frontend validation (pull_request) Successful in 50s
Details
CI / Checkstyle Main (pull_request) Successful in 55s
Details
Setup Gitea Tea CLI / setup-and-login (pull_request) Successful in 1m37s
Details
CI / Docker backend validation (pull_request) Successful in 19s
Details
CI / Backend Tests (pull_request) Successful in 1m55s
Details
CI / prettier (pull_request) Successful in 3m9s
Details
CI / oxlint (pull_request) Successful in 3m16s
Details
CI / eslint (pull_request) Successful in 3m25s
Details
CI / test-build (pull_request) Successful in 3m44s
Details
14eca84545
jank force-pushed claude/force from 14eca84545 to 262c814df0 2025-06-01 11:39:47 +00:00 Compare
Claude approved these changes 2025-06-01 11:39:56 +00:00
Dismissed
Claude left a comment
Collaborator

-m Code Review Summary:

This PR successfully improves the Claude-Gitea integration by making the instructions more explicit and comprehensive.

Key Improvements:

  • Clear command examples with concrete \ CLI usage patterns (lines 99-107)
  • Explicit communication guidelines requiring \ usage (lines 108-109)
  • Better error handling for unsupported requests (lines 119-121)
  • Structured step-by-step approach for processing requests (lines 122-123)

Minor Notes:

  • Line 105: The self-correction note about output capturing is slightly verbose
  • Lines 115-117: Command examples could use more realistic scenarios
  • Line 102: Minor spacing inconsistency in comment formatting

Overall: The changes address important gaps in the previous version and make the Claude integration more robust and predictable. The explicit requirements for \ CLI usage and structured examples will help ensure consistent behavior.

Recommendation: APPROVED ✅

-m **Code Review Summary:** This PR successfully improves the Claude-Gitea integration by making the instructions more explicit and comprehensive. **Key Improvements:** - Clear command examples with concrete \ CLI usage patterns (lines 99-107) - Explicit communication guidelines requiring \ usage (lines 108-109) - Better error handling for unsupported requests (lines 119-121) - Structured step-by-step approach for processing requests (lines 122-123) **Minor Notes:** - Line 105: The self-correction note about output capturing is slightly verbose - Lines 115-117: Command examples could use more realistic scenarios - Line 102: Minor spacing inconsistency in comment formatting **Overall:** The changes address important gaps in the previous version and make the Claude integration more robust and predictable. The explicit requirements for \ CLI usage and structured examples will help ensure consistent behavior. **Recommendation:** APPROVED ✅
jank merged commit ed44c8b7fd into main 2025-06-01 11:40:05 +00:00
jank deleted branch claude/force 2025-06-01 11:40:05 +00:00
Claude approved these changes 2025-06-01 11:40:09 +00:00
Dismissed
Claude left a comment
Collaborator

-m Code Review Summary: This PR successfully improves the Claude-Gitea integration by making the instructions more explicit and comprehensive. Key improvements include clear command examples with concrete tea CLI usage patterns (lines 99-107), explicit communication guidelines requiring tea pr comment usage (lines 108-109), better error handling for unsupported requests (lines 119-121), and structured step-by-step approach for processing requests (lines 122-123). Minor notes: Line 105 self-correction note is slightly verbose, lines 115-117 could use more realistic scenarios, line 102 has minor spacing inconsistency. Overall: The changes address important gaps and make the Claude integration more robust and predictable. APPROVED

-m Code Review Summary: This PR successfully improves the Claude-Gitea integration by making the instructions more explicit and comprehensive. Key improvements include clear command examples with concrete tea CLI usage patterns (lines 99-107), explicit communication guidelines requiring tea pr comment usage (lines 108-109), better error handling for unsupported requests (lines 119-121), and structured step-by-step approach for processing requests (lines 122-123). Minor notes: Line 105 self-correction note is slightly verbose, lines 115-117 could use more realistic scenarios, line 102 has minor spacing inconsistency. Overall: The changes address important gaps and make the Claude integration more robust and predictable. APPROVED
Claude approved these changes 2025-06-01 11:41:35 +00:00
Claude left a comment
Collaborator

-m This PR successfully improves Claude's integration with Gitea by enforcing tea CLI usage and providing comprehensive interaction guidelines. The changes in .gitea/workflows/claude-comment.yml:96-124 establish clear communication protocols and proper tool enforcement. While there are minor considerations around command escaping and line length for readability, the overall implementation is solid and addresses the core requirements effectively. The enhanced examples and step-by-step guidelines will improve Claude's PR interaction capabilities.

-m This PR successfully improves Claude's integration with Gitea by enforcing tea CLI usage and providing comprehensive interaction guidelines. The changes in .gitea/workflows/claude-comment.yml:96-124 establish clear communication protocols and proper tool enforcement. While there are minor considerations around command escaping and line length for readability, the overall implementation is solid and addresses the core requirements effectively. The enhanced examples and step-by-step guidelines will improve Claude's PR interaction capabilities.
Commenting is not possible because the repository is archived.
Reviewers
No reviewers
Claude
Labels
Clear labels   
Compat/Breaking

Breaking change that won't be backward compatible

  
Kind/Bug

Something is not working

  
Kind/Documentation

Documentation changes

  
Kind/Enhancement

Improve existing functionality

  
Kind/Feature

New functionality

  
Kind/Security

This is security issue

  
Kind/Testing

Issue or pull request related to testing

  
Priority
Critical
The priority is critical

  
Priority
High
The priority is high

  
Priority
Low
The priority is low

  
Priority
Medium
The priority is medium

  
Reviewed
Confirmed
Issue has been confirmed

  
Reviewed
Duplicate
This issue or pull request already exists

  
Reviewed
Invalid
Invalid issue

  
Reviewed
Won't Fix
This issue won't be fixed

  
Status
Abandoned
Somebody has started to work on this but abandoned work

  
Status
Blocked
Something is blocking this issue or pull request

  
Status
Need More Info
Feedback is required to reproduce issue or to continue work

  
backend

   
ci

   
docs

   
frontend
  
lgtm
blocked

  
lgtm
done

  
lgtm
need 1

  
size
huge

  
size
large

  
size
medium

  
size
small

No labels
Compat/Breaking
Kind/Bug
Kind/Documentation
Kind/Enhancement
Kind/Feature
Kind/Security
Kind/Testing
Priority
Critical
Priority
High
Priority
Low
Priority
Medium
Reviewed
Confirmed
Reviewed
Duplicate
Reviewed
Invalid
Reviewed
Won't Fix
Status
Abandoned
Status
Blocked
Status
Need More Info
backend
ci
docs
frontend
lgtm
blocked
lgtm
done
lgtm
need 1
size
huge
size
large
size
medium
size
small
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: SZUT/casino#249
No description provided.
Delete branch "claude/force"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?