Skip to content

Update Managed Files - #406

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

Update Managed Files#406
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 workflow prompt text and post-processing of review comments; malware scanning and merge logic are unchanged.

Overview
Standardizes how Supply-Chain Malware Review verdicts appear in Dependabot/Renovate PR comments from the Dependency Cursor Review workflow.

The malware task prompt now requires the model to open with a standalone bold line (**Verdict: benign** or **Verdict: malicious**), not plain Verdict: text embedded in prose. After Cursor returns JSON, inline Python format_malware_review_verdict parses verdicts (including mid-sentence mentions), prefers a line-start match, strips duplicate verdict fragments, and rewrites the section so the posted comment always leads with one normalized bold verdict before the reasoning.

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

@ChiaAutomation
ChiaAutomation requested a review from a team September 3, 2026 16:09
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 66d661f. 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.

Wrong verdict can be selected

Medium Severity

The formatter promotes the first line-start Verdict: benign or Verdict: malicious token to the PR comment header, even when that token is an instruction example or the prefix of a longer sentence. The posted malware review can then show the opposite conclusion from the analysis.

Additional Locations (1)
Fix in Cursor Fix in Web

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