tea comment: Heredoc syntax causes command to fail silently #10

New Issue

2025-12-31T16:30:00Z

HugoNijhuis commented

2025-12-31 16:30:00 +00:00

Context

Discovered during auto-review of PR #9. The code-reviewer agent's comment was never posted.

Problem

When using heredoc syntax with tea comment, the command gets automatically backgrounded and fails silently:

# Fails - gets backgrounded, no output, comment not posted
tea comment 9 "$(cat <<'EOF'
## Review
Content here
EOF
)"

# Works - runs synchronously, comment posts successfully
tea comment 9 "## Review

Content here"

Root Cause

The heredoc $(cat <<'EOF'...EOF) syntax triggers the Bash tool to run in background mode. When backgrounded:

Shell substitution $(...) doesn't execute properly
Command produces no output
Comment is never posted to Gitea

Suggested Fix

Update skills/gitea/SKILL.md Comments section to:

Remove the heredoc example
Add a warning about avoiding heredocs with tea comment
Show multiline comments using simple quoted strings with literal newlines

Affected Files

skills/gitea/SKILL.md (Comments section, lines 117-129)

## Context Discovered during auto-review of PR #9. The code-reviewer agent's comment was never posted. ## Problem When using heredoc syntax with `tea comment`, the command gets automatically backgrounded and fails silently: ```bash # Fails - gets backgrounded, no output, comment not posted tea comment 9 "$(cat <<'EOF' ## Review Content here EOF )" ``` ```bash # Works - runs synchronously, comment posts successfully tea comment 9 "## Review Content here" ``` ## Root Cause The heredoc `$(cat <<'EOF'...EOF)` syntax triggers the Bash tool to run in background mode. When backgrounded: 1. Shell substitution `$(...)` doesn't execute properly 2. Command produces no output 3. Comment is never posted to Gitea ## Suggested Fix Update `skills/gitea/SKILL.md` Comments section to: 1. Remove the heredoc example 2. Add a warning about avoiding heredocs with `tea comment` 3. Show multiline comments using simple quoted strings with literal newlines ## Affected Files - skills/gitea/SKILL.md (Comments section, lines 117-129)

HugoNijhuis referenced this issue from a commit

2025-12-31 16:46:43 +00:00

Fix tea comment docs: replace heredoc with quoted strings

HugoNijhuis referenced a pull request that will close this issue

2025-12-31 16:46:50 +00:00

[Issue #10] Fix tea comment docs: replace heredoc with quoted strings #11

HugoNijhuis closed this issue

2025-12-31 17:59:54 +00:00

HugoNijhuis referenced this issue from a commit

2025-12-31 17:59:55 +00:00

Fix tea comment docs: replace heredoc with quoted strings

HugoNijhuis referenced this issue from a commit