Skip to content

Update Managed Files - #3064

Open
ChiaAutomation wants to merge 1 commit into
mainfrom
managed-files
Open

Update Managed Files#3064
ChiaAutomation wants to merge 1 commit into
mainfrom
managed-files

Conversation

@ChiaAutomation

@ChiaAutomation ChiaAutomation commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Note

Low Risk
CI-only post-processing and prompt wording for PR comments; no runtime app or auth changes.

Overview
Tightens how supply-chain malware Cursor output appears on Dependabot/Renovate PR comments.

The malware-review prompt now requires the model to start with a standalone bold line (**Verdict: benign** or **Verdict: malicious**), not plain Verdict: text. After the agent runs, inline Python normalizes the extracted malware text: it finds verdict mentions (including mid-sentence), keeps the last verdict value, strips duplicate verdict lines (and stray bold markers), and rewrites the section so the bold verdict is always first, then reasoning.

Reviewed by Cursor Bugbot for commit 9619573. Bugbot is set up for automated code reviews on this repo. Configure here.

@ChiaAutomation
ChiaAutomation requested a review from a team as a code owner September 3, 2026 16:08
@ChiaAutomation
ChiaAutomation requested a review from a team September 3, 2026 16:08

@cursor cursor Bot 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.

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 9619573. Configure here.


cleaned = text
for start, end in reversed(_merge_spans(spans)):
cleaned = cleaned[:start] + cleaned[end:]

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Last verdict mention overrides real verdict

High Severity

format_malware_review_verdict treats every line-start Verdict: benign or Verdict: malicious as a declaration and keeps the last one. Reasoning that restates those labels, which the prompt asks for when the scanner disagrees, can overwrite the model's real first-line verdict and strip the surrounding sentence.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 9619573. Configure here.

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.

1 participant