Fix code-reviewer agent: heredoc comment bug and missing branch cleanup #62

New Issue

2026-01-09T23:44:40Z

HugoNijhuis commented

2026-01-09 23:44:40 +00:00

Summary

The code-reviewer agent has two bugs discovered during spawn-issues orchestration:

Uses heredoc syntax with tea comment which causes the command to be backgrounded and fail silently
Does not clean up branches after merging PRs

Acceptance Criteria

code-reviewer agent avoids heredoc syntax $(cat <<'EOF'...) with tea comment
code-reviewer agent calls tea pulls clean <number> after merging a PR
Agent prompt explicitly references the gitea skill warning about heredoc

Context

Discovered while running /spawn-issues 56. The reviewer:

Posted a detailed review comment using heredoc - it was backgrounded and never actually posted
Merged PR #61 successfully but left the branch on both local and remote

The gitea skill already documents this limitation:

Warning

: Do not use heredoc syntax $(cat <<'EOF'...EOF) with tea comment - it causes the command to be backgrounded and fail silently.

Technical Notes

Options for multi-line comments without heredoc:

Use simple inline strings with \n for newlines
Keep comments concise (single paragraph)
Use multiple tea comment calls if needed

## Summary The code-reviewer agent has two bugs discovered during spawn-issues orchestration: 1. Uses heredoc syntax with `tea comment` which causes the command to be backgrounded and fail silently 2. Does not clean up branches after merging PRs ## Acceptance Criteria - [ ] code-reviewer agent avoids heredoc syntax `$(cat <<'EOF'...)` with `tea comment` - [ ] code-reviewer agent calls `tea pulls clean <number>` after merging a PR - [ ] Agent prompt explicitly references the gitea skill warning about heredoc ## Context Discovered while running `/spawn-issues 56`. The reviewer: - Posted a detailed review comment using heredoc - it was backgrounded and never actually posted - Merged PR #61 successfully but left the branch on both local and remote The gitea skill already documents this limitation: > **Warning**: Do not use heredoc syntax `$(cat <<'EOF'...EOF)` with `tea comment` - it causes the command to be backgrounded and fail silently. ## Technical Notes Options for multi-line comments without heredoc: - Use simple inline strings with `\n` for newlines - Keep comments concise (single paragraph) - Use multiple `tea comment` calls if needed

HugoNijhuis referenced this issue from a commit

2026-01-09 23:49:16 +00:00

Fix code-reviewer agent: heredoc bug and branch cleanup

HugoNijhuis referenced a pull request that will close this issue

2026-01-09 23:49:27 +00:00

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

HugoNijhuis closed this issue

2026-01-09 23:50:14 +00:00

HugoNijhuis referenced this issue from a commit

2026-01-09 23:50:15 +00:00

Fix code-reviewer agent: heredoc bug and branch cleanup

HugoNijhuis referenced this issue from a commit