Compare commits

..

10 commits

Author SHA1 Message Date
5367b9f80e
fix(deps): update dependencies (major and minor)
Some checks failed
CI / prettier (pull_request) Has been skipped
CI / Get Changed Files (pull_request) Successful in 11s
CI / eslint (pull_request) Has been skipped
CI / oxlint (pull_request) Has been skipped
CI / test-build (pull_request) Has been skipped
Claude PR Review / claude-code (pull_request) Successful in 19s
CI / Docker frontend validation (pull_request) Has been skipped
CI / Docker backend validation (pull_request) Failing after 1m21s
CI / Checkstyle Main (pull_request) Failing after 1m35s
CI / Backend Tests (pull_request) Failing after 1m59s
2025-06-01 12:01:55 +00:00
fa67dd5ebf
Merge pull request 'fix: Update claude to not decline pr's' (!244) from claude/no-reject into main
Reviewed-on: #244
Reviewed-by: Claude <claude@kjan.email>
2025-06-01 11:54:56 +00:00
b09c9c3b4f
Merge branch 'main' into claude/no-reject
All checks were successful
CI / Get Changed Files (pull_request) Successful in 11s
CI / Docker frontend validation (pull_request) Successful in 32s
CI / eslint (pull_request) Successful in 41s
CI / oxlint (pull_request) Successful in 44s
CI / prettier (pull_request) Successful in 46s
CI / Docker backend validation (pull_request) Successful in 21s
Claude PR Review / claude-code (pull_request) Successful in 1m14s
CI / test-build (pull_request) Successful in 1m13s
CI / Checkstyle Main (pull_request) Successful in 1m22s
CI / Backend Tests (pull_request) Successful in 2m7s
2025-06-01 11:51:58 +00:00
4960b5966f
Merge pull request 'fix: Remove -m' (!250) from claude/no-m into main
Reviewed-on: #250
2025-06-01 11:45:11 +00:00
ffea4c0ec3 fix: Remove -m
All checks were successful
CI / Get Changed Files (pull_request) Successful in 12s
CI / eslint (pull_request) Successful in 24s
CI / oxlint (pull_request) Successful in 27s
CI / Docker frontend validation (pull_request) Successful in 29s
CI / prettier (pull_request) Successful in 42s
Setup Gitea Tea CLI / setup-and-login (pull_request) Successful in 1m2s
CI / Docker backend validation (pull_request) Successful in 19s
CI / test-build (pull_request) Successful in 1m12s
CI / Checkstyle Main (pull_request) Successful in 1m20s
CI / Backend Tests (pull_request) Successful in 2m1s
2025-06-01 13:43:36 +02:00
ed44c8b7fd
Merge pull request 'fix: Force claude to use tea cli' (!249) from claude/force into main
Reviewed-on: #249
Reviewed-by: Claude <claude@kjan.email>
2025-06-01 11:40:04 +00:00
262c814df0 fix: Force claude to use tea cli
All checks were successful
CI / Get Changed Files (pull_request) Successful in 16s
CI / Docker frontend validation (pull_request) Successful in 17s
CI / Docker backend validation (pull_request) Successful in 16s
Setup Gitea Tea CLI / setup-and-login (pull_request) Successful in 1m25s
CI / Checkstyle Main (pull_request) Successful in 52s
CI / Backend Tests (pull_request) Successful in 1m58s
CI / oxlint (pull_request) Successful in 20s
CI / eslint (pull_request) Successful in 32s
CI / prettier (pull_request) Successful in 23s
CI / test-build (pull_request) Successful in 32s
2025-06-01 13:39:39 +02:00
f6b8400c0b
Merge branch 'main' into claude/no-reject
All checks were successful
CI / Get Changed Files (pull_request) Successful in 8s
CI / eslint (pull_request) Successful in 34s
CI / Docker frontend validation (pull_request) Successful in 30s
CI / oxlint (pull_request) Successful in 42s
CI / prettier (pull_request) Successful in 45s
Claude PR Review / claude-code (pull_request) Successful in 57s
CI / Docker backend validation (pull_request) Successful in 20s
CI / test-build (pull_request) Successful in 1m11s
CI / Checkstyle Main (pull_request) Successful in 1m18s
CI / Backend Tests (pull_request) Successful in 1m58s
2025-06-01 10:14:21 +00:00
189fb85918 fix: Update pipeline name
Some checks failed
CI / Checkstyle Main (pull_request) Successful in 53s
CI / Get Changed Files (pull_request) Successful in 8s
CI / eslint (pull_request) Successful in 27s
CI / oxlint (pull_request) Successful in 28s
CI / prettier (pull_request) Successful in 35s
CI / test-build (pull_request) Successful in 51s
CI / Backend Tests (pull_request) Successful in 2m7s
Claude PR Review / claude-code (pull_request) Successful in 52s
CI / Docker backend validation (pull_request) Successful in 11s
CI / Docker frontend validation (pull_request) Failing after 14s
2025-06-01 10:44:47 +02:00
acd098225c fix: Update claude to not decline pr's 2025-06-01 10:44:44 +02:00
2 changed files with 29 additions and 16 deletions

View file

@ -77,7 +77,7 @@ jobs:
**How to Post Reviews and Other Feedback:**
- When you provide a review, post it as a comment using:
\`tea pr comment ${PR_NUMBER} -m \"Claude's Review:\n[Your detailed review, mentioning files and line numbers.]\"\`
\`tea pr comment ${PR_NUMBER} \"Claude's Review:\n[Your detailed review, mentioning files and line numbers.]\"\`
- For other informational responses or clarifications, also use \`tea pr comment ...\`.
**Critical: Handling Approval, Rejection, or Merge Requests:**
@ -94,18 +94,31 @@ jobs:
3. **Rejection/Requesting Changes:**
- If asked to reject or request changes, you should typically base this on your own review of the PR's changes.
- First, perform a review if you haven't already.
- Then, you can use \`tea pr reject ${PR_NUMBER} -m \"Claude's Review Summary: [summary of reasons for rejection/changes based on your review]\"\`. Ensure your detailed review is also available as a comment.
- Then, you can use \`tea pr reject ${PR_NUMBER} \"Claude's Review Summary: [summary of reasons for rejection/changes based on your review]\"\`. Ensure your detailed review is also available as a comment.
Examples of interpreting comments and acting (keeping the above critical guidelines in mind):
- User: '@claude LGTM, approve this' -> You: First, consider if a review is implied or done. If so, and you agree, you might approve. If not, you might say, 'I can approve this if the standard review process is complete. Have maintainers reviewed this?' or perform your own review.
- User: '@claude please review this PR' -> You: Get diffs, review, then \`tea pr comment ${PR_NUMBER} -m \"Claude's Review: ...\"\`.
- User: '@claude close this PR' -> You: \`tea pr close ${PR_NUMBER}\` (Closing is generally less sensitive than approving/merging).
- User: '@claude add label enhancement' -> You: \`tea pr label ${PR_NUMBER} --add enhancement\`
Examples of interpreting comments and generating appropriate \`tea\` commands (keeping the above critical guidelines in mind):
- User: '@claude LGTM, approve this' -> You: First, consider if a review is implied or done. If so, and you agree, you might generate \`tea pr approve ${PR_NUMBER}\`. If not, you might generate \`tea pr comment ${PR_NUMBER} \"Claude: I can approve this if the standard review process is complete. Have maintainers reviewed this?\"\` or perform your own review and then comment.
- User: '@claude please review this PR' -> You: Get diffs, review, then generate \`tea pr comment ${PR_NUMBER} \"Claude's Review: ...\"\`.
- User: '@claude close this PR' -> You: Generate \`tea pr close ${PR_NUMBER}\` and optionally \`tea pr comment ${PR_NUMBER} \"Claude: PR #${PR_NUMBER} has been closed as requested.\"\`.
- User: '@claude add label enhancement' -> You: Generate \`tea pr label ${PR_NUMBER} --add enhancement\` and \`tea pr comment ${PR_NUMBER} \"Claude: Label 'enhancement' added to PR #${PR_NUMBER}.\"\`
- User: '@claude what are the labels on this PR?' -> You: Generate \`tea pr label ${PR_NUMBER} --list\` (this command outputs to stdout, which is fine for your internal use). Then, to inform the user, you generate: \`tea pr comment ${PR_NUMBER} \"Claude: The current labels are: [output from tea pr label --list].\"\` (You'll need to capture the output of the first command to formulate the second if the tool allows such chaining, otherwise, focus on commands that directly achieve the user's goal or report information). *Self-correction: The Bash tool can capture output. So, if you need to run a \`tea\` command to get information for yourself, do so, then use that information to formulate your \`tea pr comment ...\` to the user.*
IMPORTANT GUIDELINES:
- You MUST use the 'tea' CLI for all Gitea interactions.
- You MUST use the tea cli to comment on the PR so the user gets a response.
- For PR reviews, ALWAYS analyze the diff. Make sure to mention specific files and line numbers.
- Be precise with 'tea' commands. If ambiguous, ask for clarification using \`tea pr comment ...\`.
- Ensure reviews are professional, constructive, and helpful.
**IMPORTANT GUIDELINES FOR YOUR OPERATION AND RESPONSE GENERATION:**
- **Your SOLE METHOD of communicating back to the user on Gitea is by generating a \`tea pr comment ${PR_NUMBER} \"...\"\` command.** This is non-negotiable. Do not output plain text messages intended for the user. Your response *is* the command.
- **Use the 'tea' CLI for ALL Gitea interactions.** This includes fetching PR details, diffs, labels, status, and posting comments, reviews, approvals, etc.
- **For PR reviews, ALWAYS analyze the diff.** Use \`tea pr diff ${PR_NUMBER}\` or git commands to get the diff. Make sure to mention specific files and line numbers in your review comment.
- **Be precise with 'tea' commands.** If a user's request is ambiguous, DO NOT GUESS. Instead, generate a \`tea pr comment ${PR_NUMBER} \"Claude Asks: [Your clarifying question]\"\` command to ask for more details.
- **Execute only necessary 'tea' command(s).** If a user asks for a review, your primary output should be the \`tea pr comment ...\` command containing the review. If they ask to add a label, your output should be \`tea pr label ...\` and then a confirmation \`tea pr comment ...\`.
- **Ensure reviews are professional, constructive, and helpful.**
- **If you need to perform an action AND then report on it, generate both \`tea\` commands sequentially.** For example, to add a label and confirm:
\`tea pr label ${PR_NUMBER} --add bug\`
\`tea pr comment ${PR_NUMBER} "Claude: I've added the 'bug' label."\`
The GitHub Actions workflow will execute these commands one after another.
- **If a user's request cannot be fulfilled using the 'tea' CLI or the allowed tools, explain this limitation by generating a \`tea pr comment ...\` command.** For example: \`tea pr comment ${PR_NUMBER} "Claude: I cannot perform that action as it's outside my current capabilities or allowed tools."\`
- **Think step-by-step.** 1. Understand request. 2. Identify necessary `tea` command(s). 3. If it's a review, get the diff. 4. Formulate the `tea` command(s) as your direct output.
**Final Check before outputting:**
"Is my entire response that's intended for the Gitea user wrapped in a \`tea pr comment ${PR_NUMBER} '...' \` command (or another appropriate \`tea\` command if it's an action like \`tea pr label ...\`)? If not, I must fix it."
You are now ready to process the comment. Remember, your output will be executed in a shell. Generate only the \`tea\` command(s) needed.
"

View file

@ -1,11 +1,11 @@
name: Setup Gitea Tea CLI
name: Claude PR Review
on:
pull_request:
types: [opened, synchronize] # Runs on new PRs and updates
jobs:
setup-and-login:
claude-code:
runs-on: ubuntu-latest
steps:
- name: Checkout code
@ -69,7 +69,7 @@ jobs:
Once you are done with your review, post your feedback as a reject or review on the pull request using the following exact format:
tea \"<reject or approve>\" ${PR_NUMBER} \"<your review message here>\"
tea \"<comment or approve>\" ${PR_NUMBER} \"<your review message here>\"
Make sure the comment is clear, professional, and helpful. Only run the tea comment command once you're finished reviewing all changes. AND MOST IMPORTANDLY ONLY REVIEW THE DIFF FROM THE CURRENT STATE TO THE MAIN BRANCH TO GET THAT USE GIT DIFF
You may also use the tea cli to find out various things about the pull request