flowmade-one/architecture
3
0
Fork 0
Code Issues 15 Pull Requests 1 Actions Packages Projects Releases Wiki Activity

[Issue #62] Fix code-reviewer agent: heredoc bug and branch cleanup #63

Merged
HugoNijhuis merged 1 commits from issue-62-fix-code-reviewer-bugs into main 2026-01-09 23:50:13 +00:00
Conversation 1 Commits 1 Files Changed 1 +3 -1
1 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
agents/code-reviewer/AGENT.md
View File

@@ -25,7 +25,9 @@ You will receive a PR number to review. Follow this process:
- **Test Coverage**: Missing tests, untested edge cases - **Test Coverage**: Missing tests, untested edge cases
3. Generate a structured review comment 3. Generate a structured review comment
4. Post the review using `tea comment <number> "<review body>"` 4. Post the review using `tea comment <number> "<review body>"`
5. **If verdict is LGTM**: Merge with `tea pulls merge <number> --style rebase` - **WARNING**: Do NOT use heredoc syntax `$(cat <<'EOF'...)` with `tea comment` - it causes the command to be backgrounded and fail silently
- Keep comments concise or use literal newlines in quoted strings
5. **If verdict is LGTM**: Merge with `tea pulls merge <number> --style rebase`, then clean up with `tea pulls clean <number>`
6. **If verdict is NOT LGTM**: Do not merge; leave for the user to address 6. **If verdict is NOT LGTM**: Do not merge; leave for the user to address
## Review Comment Format ## Review Comment Format