Skip to content

feat(aireview): classify Assisted-by Ymir commits as ymir - #150

Open
mfrancisc wants to merge 2 commits into
konflux-ci:mainfrom
mfrancisc:addymir
Open

mfrancisc wants to merge 2 commits into
konflux-ci:mainfrom
mfrancisc:addymir

Conversation

@mfrancisc

Copy link
Copy Markdown

Named Ymir trailer matches so those commits are not lumped into assisted_by_unknown.

Named Ymir trailer matches so those commits are not lumped into assisted_by_unknown.

Co-Authored-By: Cursor Grok 4.6 <noreply@example.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@mfrancisc
mfrancisc requested a review from a team as a code owner September 3, 2026 12:33
@fullsend-ai-review

fullsend-ai-review Bot commented Sep 3, 2026 •

Copy link
Copy Markdown

🤖 Finished Review · ✅ Success · Started 12:35 PM UTC · Completed 12:49 PM UTC

Commit: 9ee3c25 · View workflow run →

Runtime: claude · Model: opus → claude-opus-4-6 · Effort: high · Cost: $3.94

@codecov-commenter

codecov-commenter commented Sep 3, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 66.66667% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 41.61%. Comparing base (c3c2f94) to head (05acf34).

Files with missing lines Patch % Lines
backend/plugins/aireview/tasks/detect_ai_commit.go 66.66% 1 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (c3c2f94) and HEAD (05acf34). Click for more details.

HEAD has 10 uploads less than BASE
Flag BASE (c3c2f94) HEAD (05acf34)
unit-tests-go 6 1
unit-tests-python 6 1
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #150      +/-   ##
==========================================
- Coverage   48.05%   41.61%   -6.44%     
==========================================
  Files         154      154              
  Lines       10501    10504       +3     
==========================================
- Hits         5046     4371     -675     
- Misses       5237     6020     +783     
+ Partials      218      113     -105     
Flag Coverage Δ
e2e-go 9.51% <0.00%> (-12.75%) ⬇️
unit-tests-go 35.90% <100.00%> (+0.01%) ⬆️
unit-tests-python 55.49% <ø> (ø)

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

Files with missing lines Coverage Δ
backend/plugins/aireview/models/ai_review.go 0.00% <ø> (ø)
backend/plugins/aireview/tasks/detect_ai_commit.go 69.69% <66.66%> (-0.31%) ⬇️

... and 27 files with indirect coverage changes


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c3c2f94...05acf34. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@fullsend-ai-review fullsend-ai-review Bot added the risk/moderate PR risk: moderate label Sep 3, 2026
@fullsend-ai-review

fullsend-ai-review Bot commented Sep 3, 2026 •

Copy link
Copy Markdown

Risk Assessment: low (1/5)

Details

Small, focused classification fix (46 lines, 8 files, single author, no CI or dependency changes) with clean git history and no security or protected-path exposure.

Previous run

Risk Assessment: moderate (2/5)

Details

Small feature addition (8 files, 46 lines) confined to aireview plugin, authored by established contributor; minimal churn and single-author ownership reduce complexity.

@fullsend-ai-review

fullsend-ai-review Bot commented Sep 3, 2026 •

Copy link
Copy Markdown

Looks good to me

Previous run

Looks good to me

fullsend-ai-review[bot]

This comment was marked as outdated.

@fullsend-ai-review fullsend-ai-review Bot added the ready-for-merge All reviewers approved — ready to merge label Sep 3, 2026

@ciaramulligan ciaramulligan left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

all good

@fullsend-ai-review

fullsend-ai-review Bot commented Sep 21, 2026 •

Copy link
Copy Markdown

🤖 Finished Review · ✅ Success · Started 9:36 AM UTC · Completed 9:49 AM UTC

Commit: efeae75 · View workflow run →

Runtime: claude · Model: opus → claude-opus-4-6 · Effort: high · Cost: $3.88

@fullsend-ai-review fullsend-ai-review Bot added risk/low PR risk: low and removed risk/moderate PR risk: moderate labels Sep 21, 2026
@fullsend-ai-review
fullsend-ai-review Bot dismissed their stale review September 21, 2026 09:49

Superseded by updated review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-merge All reviewers approved — ready to merge risk/low PR risk: low

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants