fix: Fix claude to always use tea command to review #248

Merged

jank merged 1 commit from claude/use-tea-cli into main

2025-06-01 10:30:52 +00:00

jank commented

2025-06-01 10:25:25 +00:00

Owner

No description provided.

jank added 1 commit 2025-06-01 10:25:25 +00:00

fix: Fix claude to always use tea command to review

CI / Get Changed Files (pull_request) Successful in 8s

Details

CI / eslint (pull_request) Successful in 30s

Details

CI / oxlint (pull_request) Successful in 30s

Details

CI / Docker frontend validation (pull_request) Successful in 32s

Details

CI / prettier (pull_request) Successful in 41s

Details

CI / Docker backend validation (pull_request) Successful in 16s

Details

CI / test-build (pull_request) Successful in 1m6s

Details

CI / Checkstyle Main (pull_request) Successful in 1m17s

Details

CI / Backend Tests (pull_request) Successful in 1m54s

Details

Setup Gitea Tea CLI / setup-and-login (pull_request) Successful in 1m18s

Details

5d5c27827b

Claude approved these changes 2025-06-01 10:29:13 +00:00

Claude left a comment

Collaborator

Review Summary:

The changes in .gitea/workflows/claude-comment.yml at lines 106-109 are functionally correct and achieve the stated goal of ensuring Claude uses tea CLI for PR responses.

Issues identified:

Line 107: The new guideline 'You MUST use the tea cli to comment on the PR so the user gets a response' is somewhat redundant with the existing line 106 'You MUST use the tea CLI for all Gitea interactions'
Line 110: Removing 'Execute only necessary tea commands' might encourage unnecessary command execution

Recommendation:
Consider consolidating the tea CLI guidelines to avoid redundancy while maintaining clarity. The change accomplishes its purpose but could be more concise.

Verdict: APPROVE - The changes work as intended despite minor redundancy.

**Review Summary:** The changes in .gitea/workflows/claude-comment.yml at lines 106-109 are functionally correct and achieve the stated goal of ensuring Claude uses tea CLI for PR responses. **Issues identified:** - Line 107: The new guideline 'You MUST use the tea cli to comment on the PR so the user gets a response' is somewhat redundant with the existing line 106 'You MUST use the tea CLI for all Gitea interactions' - Line 110: Removing 'Execute only necessary tea commands' might encourage unnecessary command execution **Recommendation:** Consider consolidating the tea CLI guidelines to avoid redundancy while maintaining clarity. The change accomplishes its purpose but could be more concise. **Verdict:** APPROVE - The changes work as intended despite minor redundancy.