Skip to content

fix: name the lines to change in the PR description instead of repeating it - #326

Merged
shenxianpeng merged 1 commit into
mainfrom
fix/pr-description-fix-lines
Nov 9, 2026
Merged

shenxianpeng merged 1 commit into
mainfrom
fix/pr-description-fix-lines

Conversation

@shenxianpeng

@shenxianpeng shenxianpeng commented Nov 9, 2026 •

Copy link
Copy Markdown
Member

What

With pr-body-ai-attribution: true, a failing description printed the whole corrected description as "Fix" in the log, the annotation, the job summary and the PR comment. A long description was repeated in full, and the reader still had to diff it by eye.

Now the Action names only the lines to change.

Before / after

Description with Link to Devin run: https://app.devin.ai/sessions/…, ai_attribution = "forbid":

Before:

CC013 ai-attribution
  value: Devin
  AI attribution is forbidden in this project — detected: Devin
  Suggest: This project does not accept AI attribution. Remove the AI lines.
  Fix: ## Summary
  Adds a parser.

After:

CC013 ai-attribution
  value: Devin
  AI attribution is forbidden in this project — detected: Devin
  Suggest: In the description, remove "Link to Devin run: https://app.devin.ai/sessions/…"

Description with Co-Authored-By: Claude <noreply@anthropic.com>, ai_attribution = "disclose":

Before: CC014 and CC015 each printed the full description again, with the trailer replaced.

After:

CC015 ai-co-author
  Suggest: In the description, remove "Co-Authored-By: Claude <noreply@anthropic.com>", and add "Assisted-by: Claude"

Checks

  • ruff check, ruff format --check, mypy main.py: clean
  • pytest: 280 passed, 100% line and branch coverage
  • Ran main.py from main and from this branch against a local PR event with commit-check 2.20.2 under both policies; the output above is from those runs.

Commit message checks are unchanged: they still print the full corrected message, which is what git commit --amend needs.

@shenxianpeng
shenxianpeng requested a review from a team as a code owner November 9, 2026 06:58
@coderabbitai

coderabbitai Bot commented Nov 9, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 1 minute.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 46f2b147-b711-4919-80e9-842881a05704
Commits

Reviewing files that changed from the base of the PR and between 7e971e9 and 9da6b2c.

Files selected for processing: 2

  • main.py
  • main_test.py
  • Autofix · 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

@github-actions

github-actions Bot commented Nov 9, 2026

Copy link
Copy Markdown
Contributor

Commit Check

✅ All 5 checks passed

Show all 5 checks
Commit message
  ✔ PR title (fix: name the lines to change in the PR description inste...)
  ✔ Commit 1/1 (9da6b2c) (fix: name the lines to change in the PR description inste...)
Branch
  ✔ Branch (fix/pr-description-fix-lines)
Author
  ✔ Author name (Xianpeng Shen)
  ✔ Author email (3353385+shenxianpeng@users.noreply.github.com)

commit-check 2.20.2 · Rules reference

@codecov

codecov Bot commented Nov 9, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (7e971e9) to head (9da6b2c).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #326   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            1         1           
  Lines          764       791   +27     
=========================================
+ Hits           764       791   +27     
Flag Coverage Δ
unittests 100.00% <100.00%> (ø)

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

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@shenxianpeng shenxianpeng added the bug Something isn't working label Nov 9, 2026
@shenxianpeng
shenxianpeng merged commit 5aa913a into main Nov 9, 2026
25 checks passed
@shenxianpeng
shenxianpeng deleted the fix/pr-description-fix-lines branch November 9, 2026 07:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant