Skip to content

Commit 6ae9bc7

Browse files
authored
Merge pull request #330 from worldfnd/px/fix-review-format
Fix: format /claude-review output with proper markdown
2 parents 9ddba04 + 04ebc81 commit 6ae9bc7

1 file changed

Lines changed: 15 additions & 10 deletions

File tree

.github/workflows/claude-code-review.yml

Lines changed: 15 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -66,34 +66,39 @@ jobs:
6666
- Post inline comments for any violations found.
6767
6868
**C. PR Review Summary**
69-
After completing A and B, post a single PR comment using `gh pr comment ${{ github.event.issue.number }}` with this format:
69+
After completing A and B, you MUST post the review as a PR comment using `gh pr comment` with a heredoc.
70+
Do NOT use the MCP comment tool for the summary — it strips markdown formatting.
71+
Use this exact command pattern:
7072
71-
---
73+
```
74+
gh pr comment ${{ github.event.issue.number }} --body "$(cat <<'REVIEW_EOF'
7275
## Code Review
7376
7477
### Summary
75-
Brief description of what this PR does and why.
78+
<Brief description of what this PR does and why.>
7679
7780
### Review
7881
7982
**Correctness & Soundness**
80-
Any issues with logic, missing constraints, or soundness concerns. If none, say so.
83+
<Any issues with logic, missing constraints, or soundness concerns. If none, say so.>
8184
8285
**Code Quality**
83-
Error handling, naming, readability, idiomatic patterns, CLAUDE.md compliance.
86+
<Error handling, naming, readability, idiomatic patterns, CLAUDE.md compliance.>
8487
8588
**Testing**
86-
Is there adequate test coverage? Missing edge cases or regression tests?
89+
<Is there adequate test coverage? Missing edge cases or regression tests?>
8790
8891
**Suggestions**
89-
Concrete improvements, alternative approaches, or nitpicks (clearly labeled as non-blocking).
92+
<Concrete improvements, alternative approaches, or nitpicks (clearly labeled as non-blocking).>
9093
9194
### Verdict
92-
One of: ✅ Approve | ⚠️ Approve with suggestions | 🔄 Request changes
93-
---
95+
<One of: ✅ Approve | ⚠️ Approve with suggestions | 🔄 Request changes>
96+
REVIEW_EOF
97+
)"
98+
```
9499
95100
## Guidelines
96101
- Be specific — reference file paths and line numbers.
97102
- Distinguish blocking issues from non-blocking suggestions.
98103
- If the PR is trivial or obviously correct, keep the review brief but still post the summary.
99-
- You MUST always post the PR review summary comment. Never finish silently.
104+
- You MUST always post the PR review summary comment using `gh pr comment` with a heredoc. Never finish silently. Never use the MCP comment tool for the summary.

0 commit comments

Comments
 (0)