Skip to content

ci: always post Claude review summary#30

Merged
MapleEve merged 1 commit into
mainfrom
chore/claude-review-summary-comments
May 7, 2026
Merged

ci: always post Claude review summary#30
MapleEve merged 1 commit into
mainfrom
chore/claude-review-summary-comments

Conversation

@MapleEve
Copy link
Copy Markdown
Owner

@MapleEve MapleEve commented May 7, 2026

Summary

  • make automated Claude Code Review always post or update a PR-level summary comment
  • keep the Claude check failing when the review action fails
  • grant explicit issue comment write permission for the summary comment

Verification

  • YAML parsed locally
  • git diff --check passed

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 7, 2026

👍 @MapleEve

Thank you for raising your pull request and contributing to voscript.
Please make sure you have followed our contributing guidelines. We will review it as soon as possible.
If you encounter any problems, please feel free to connect with us.
非常感谢您提出拉取请求并为 voscript 做出贡献,请确保您已经遵循了我们的贡献指南,我们会尽快审查它。
如果您遇到任何问题,请随时与我们联系。

@codecov
Copy link
Copy Markdown

codecov Bot commented May 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.18%. Comparing base (6d5242a) to head (6d1936c).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #30   +/-   ##
=======================================
  Coverage   91.18%   91.18%           
=======================================
  Files          76       76           
  Lines        3016     3016           
=======================================
  Hits         2750     2750           
  Misses        266      266           
Flag Coverage Δ
unit 91.18% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@MapleEve
Copy link
Copy Markdown
Owner Author

MapleEve commented May 7, 2026

Claude Code Review

Claude Code Review completed for 6d1936c.

This summary is posted even when Claude has no line-level findings. If no separate Claude inline comments are visible, there were no actionable line-level findings for this run.

Run: https://github.com/MapleEve/VoScript/actions/runs/25478422844

@MapleEve MapleEve merged commit 3a9155e into main May 7, 2026
9 checks passed
@MapleEve MapleEve deleted the chore/claude-review-summary-comments branch May 7, 2026 06:02
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 7, 2026

❤️ Great PR @MapleEve ❤️

The growth of project is inseparable from user feedback and contribution, thanks for your contribution!
项目的成长离不开用户反馈和贡献,感谢您的贡献!

Copilot AI review requested due to automatic review settings May 7, 2026 06:09
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the Claude Code Review GitHub Actions workflow to always post/update a single PR-level “Claude Code Review” summary comment, while keeping the overall check failing when the Claude review step fails.

Changes:

  • Adds an always-run step that posts or updates a PR comment containing a run summary (keyed by an HTML marker).
  • Makes the Claude review step continue-on-error and adds a final explicit failure step based on steps.claude-review.outcome.
  • Expands workflow permissions to allow writing issue comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

id-token: write
pull-requests: write
issues: read
issues: write

comment_id="$(
gh api "repos/$GITHUB_REPOSITORY/issues/$PR_NUMBER/comments" --paginate \
--jq ".[] | select(.body | contains(\"$MARKER\")) | .id" | tail -n 1
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