1718a57495
The sweep script was closing issues based solely on when a lifecycle label was applied, ignoring any human comments posted after the label. This caused active issues (like #11792) to be closed even when users responded to the stale warning. Three changes: 1. Teach the triage bot about `stale` and `autoclose` labels so it removes them when a human comments on the issue. 2. Add a safety net in `closeExpired()` that checks for non-bot comments posted after the lifecycle label was applied — if any exist, skip closing. 3. Extend the 10-upvote protection (which previously only applied to enhancements) to all issue types, in both `markStale()` and `closeExpired()`. Fixes #16497 ## Test plan Trace through scenarios manually: - Issue with stale label + human comment after → triage removes label; sweep skips even if triage hasn't run yet (safety net) - Issue with stale label + no human comment → closes as before - Issue with 10+ upvotes of any type → never marked stale or closed Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com> |
||
|---|---|---|
| .. | ||
| auto-close-duplicates.yml | ||
| backfill-duplicate-comments.yml | ||
| claude-dedupe-issues.yml | ||
| claude-issue-triage.yml | ||
| claude.yml | ||
| issue-lifecycle-comment.yml | ||
| issue-opened-dispatch.yml | ||
| lock-closed-issues.yml | ||
| log-issue-events.yml | ||
| oncall-triage.yml | ||
| remove-autoclose-label.yml | ||
| sweep.yml | ||