Skip to content

Update Managed Files - #563

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

Update Managed Files#563
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
Changes only affect Dependabot review workflow prompt text and post-processing of Cursor output; no application runtime or auth paths are modified.

Overview
This PR tightens how supply-chain malware Cursor output is formatted before it lands on Dependabot/Renovate PR comments.

The malware task prompt now requires the model to open with a standalone bold line (**Verdict: benign** or **Verdict: malicious**), a blank line, then reasoning—not a plain Verdict: prefix or an inline verdict.

After the agent runs, inline Python format_malware_review_verdict parses the malware review text, picks the “official” verdict (standalone line or sentence-glued forms), strips duplicate verdict snippets from the body (including format-example lines), and re-emits a normalized block with the bold verdict first. That normalized text is what gets merged into the combined Cursor analysis comment.

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

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

Stale Bugbot comment from a previous run.

Comment thread .github/workflows/dependency-cursor-review.yml

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

Stale Bugbot comment from a previous run.

Comment thread .github/workflows/dependency-cursor-review.yml

@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 f460905. Configure here.

bold_line = f"**Verdict: {verdict_value}**"
if cleaned:
return f"{bold_line}\n\n{cleaned}"
return bold_line

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

First verdict match can mislabel review

High Severity

_official_verdict_match keeps the first standalone or glued Verdict and format_malware_review_verdict then deletes every other matching line. FORMAT_HEADING_RE does not recognize this prompt’s or: examples, and glued text after e.g. can count as official, so an early draft or echoed template can become the posted label while a later correction disappears.

Additional Locations (2)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit f460905. 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