feat(sweep): Add master tracking issue for sweep PRs#210
Merged
Conversation
Create a GitHub issue before patching that summarizes sweep results and gives reviewers a single coordination point. Each PR references the tracking issue, and the organize phase posts a completion comment with a task list of all created PRs. New script create_issue.py handles issue creation with idempotency (skips if issueUrl already in manifest). Also extracts shared utilities (read_json, write_json, severity_badge, pr_number_from_url) into _utils.py, improves find_reviewers.py to exclude the current user, and separates timeout vs error tracking in scan output. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Make update_tracking_issue() idempotent by checking for an existing "Sweep Complete" comment before posting a new one. This preserves the organize phase's re-run safety guarantee. Restore --fail-on off flag in scan_file() that was inadvertently removed. Without it, warden exits non-zero on findings and scan_file misclassifies those files as errors, silently dropping findings. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
Restore --min-confidence off in scan_file() that was inadvertently removed. Without it, warden applies its default confidence threshold and silently drops low-confidence findings. Extract ensure_github_label() into _utils.py to eliminate three duplicate implementations across scan.py, create_issue.py, and organize.py. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Sweep PRs currently go up as isolated draft PRs with no central coordination
point. Reviewers seeing individual
fix: ...PRs labeledwardenhave no wayto understand the full sweep scope, which files were checked, what was found,
or how the PRs relate to each other.
This adds a tracking issue created between the verify and patch phases. Every
PR references the issue via
Ref #N, and the organize phase posts a completioncomment with a task list reviewers can check off as they work through PRs.
New script:
create_issue.pyissueUrlalready exists in manifestwardenlabel exists before creating the issueissueUrl,issueNumber, andphases.issueto manifestUpdated:
organize.pyupdate_tracking_issue()posts a completion comment on the tracking issuewith a summary table (created/skipped/failed/security) and a PR task list
Updated:
SKILL.mdRef #${ISSUE_NUMBER}Collateral improvements (from code review/simplifier pass):
read_json,write_json,severity_badge,pr_number_from_urlinto_utils.pyfind_reviewers.pynow excludes the current GitHub user from reviewer suggestionsscan.pyandgenerate_report.pynow separate timeouts from errors in output