Skip to content

ci(review): show the full Claude Code Review output - #325

Merged
Jamie-BitFlight merged 1 commit into
mainfrom
claude/review-show-full-output
Oct 6, 2026
Merged

Jamie-BitFlight merged 1 commit into
mainfrom
claude/review-show-full-output

Conversation

@Jamie-BitFlight

Copy link
Copy Markdown
Contributor

Follow-up to #313. That change made the review prompt pass --comment and allowed the comment tools, but no review comment has appeared since.

Observation

On #312 at 2edc05d (job 111192847760), the action ran with the fixed workflow and the comment tools allowed. It reported success, 2 turns, 4.7 s, $0.094, 0 permission denials, then No buffered inline comments. The log says: "Running Claude Code via SDK (full output hidden for security)... enable show_full_output: true in your workflow file for full output." So the run's conclusion is not visible, and the cause of the missing comments is unknown.

On #324 (stacked on #312's branch, which is behind main) the action was skipped with "Workflow validation failed ... identical content to the version on the repository's default branch". That is expected for a PR whose workflow file differs from main's, and retargeting to main resolves it.

Change

One input on the action step: show_full_output: true. The input exists in the action's input list in the job log ("show_full_output": "false").

Limits

  • This PR's own review run will be skipped by the same workflow validation, because the file differs from main. The output cannot be seen until this merges and the review runs on another PR.
  • Cause of the missing comments: unknown. This change only makes it visible. A follow-up change depends on what the log shows.
  • With the flag on, the job log shows tool output. The log is readable by anyone who can read this repository's Actions logs. The run has read-only repository access plus pull-requests: write.

Validation

  • Parsed the workflow with PyYAML: show_full_output is a sibling of claude_args, and the claude_args block is unchanged.
  • uv run prek run --files .github/workflows/claude-code-review.yml: all applicable hooks passed, including the workflow linter.

🤖 Generated with Claude Code

https://claude.ai/code/session_01K5rAHJfvZyEEaUQghV7hCQ


Generated by Claude Code

The action hides the whole run unless show_full_output is true, so a
review that ended after two turns without posting could not be diagnosed.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01K5rAHJfvZyEEaUQghV7hCQ
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@coderabbitai

coderabbitai Bot commented Oct 5, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: a88f00e6-1654-4dea-ae9f-e688c2932661
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

github-actions Bot commented Oct 5, 2026

Copy link
Copy Markdown

📊 Test Coverage Report

Coverage: 88.74%

📥 Coverage XML available as artifact: coverage-xml

@github-actions

github-actions Bot commented Oct 5, 2026

Copy link
Copy Markdown

Benchmark Results

🔴 Regression detected — threshold: 30%

scan-clean

Metric Base Compare Change
files_per_second 79.5 files/s 82.2 files/s ✅ +3.4%
scan_max_ms 13030.7 ms 12203.7 ms ✅ -6.3%
scan_mean_ms 12585.9 ms 12175.0 ms ✅ -3.3%
scan_min_ms 12280.2 ms 12139.2 ms ✅ -1.1%

scan-violations

Metric Base Compare Change
files_per_second 93.6 files/s 97.0 files/s ✅ +3.6%
scan_max_ms 2195.9 ms 2113.8 ms ✅ -3.7%
scan_mean_ms 2147.9 ms 2072.6 ms ✅ -3.5%
scan_min_ms 2097.5 ms 2029.3 ms ✅ -3.3%

fix-violations

Metric Base Compare Change
fix_files_per_second 145.2 files/s 148.8 files/s ✅ +2.5%
fix_max_ms 1395.1 ms 1354.1 ms ✅ -2.9%
fix_mean_ms 1384.7 ms 1350.7 ms ✅ -2.5%
fix_min_ms 1376.2 ms 1347.5 ms ✅ -2.1%

cpu

Metric Base Compare Change
cpu_clean_mean_ms 0.3 ms 0.3 ms ✅ -0.5%
cpu_fix_mean_ms 0.8 ms 0.8 ms ✅ -1.4%
cpu_violations_mean_ms 0.3 ms 0.3 ms ➡️ +0.6%

import

Metric Base Compare Change
import_package_max_ms 107.5 ms 227.4 ms 🔴 +111.5%
import_package_mean_ms 93.4 ms 132.9 ms 🔴 +42.3%
import_package_min_ms 83.6 ms 82.5 ms ✅ -1.3%
import_rules_max_ms 136.7 ms 248.1 ms 🔴 +81.5%
import_rules_mean_ms 135.3 ms 171.8 ms ➡️ +27.0%
import_rules_min_ms 133.3 ms 132.3 ms ✅ -0.7%

View benchmark history

Updated 2026-10-05 09:03 UTC

@Jamie-BitFlight
Jamie-BitFlight merged commit 1579f8f into main Oct 6, 2026
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants