Skip to content

chore: Expand cleanup_labels.py classifications#739

Merged
stranske merged 4 commits intomainfrom
docs/short-term-plan-update
Jan 10, 2026
Merged

chore: Expand cleanup_labels.py classifications#739
stranske merged 4 commits intomainfrom
docs/short-term-plan-update

Conversation

@stranske
Copy link
Copy Markdown
Owner

@stranske stranske commented Jan 10, 2026

Summary

Improves the label cleanup script to better classify labels across repos.

Changes

  • Add autofix:, integration-, agents:keepalive-nudge to FUNCTIONAL_LABELS
  • Add common component labels (app, engine, ui, backend, cli) to INFORMATIONAL_LABELS
  • Add tech labels (javascript, python, github:actions)
  • Add domain labels (metrics, modeling, schema, etc.)

Impact

  • Reduces idiosyncratic labels from 150+ to just 24
  • The remaining 24 are legitimate project-specific labels (size:, stage:, epic:*, etc.)
  • 6 of 8 repos now have 0 idiosyncratic labels

Related

Part of Phase 4 label audit work

@stranske
docs: Update SHORT_TERM_PLAN with label matcher fixes and validation …
391fc96 
…results

- Document PRs #733, #735 (deep label matcher fixes)
- Record validation test results (issues #265-267)
- Mark auto-label validation as complete
- Key win: 2FA feature request now gets only 'enhancement' (was 3 labels)
@stranske
docs: Add LONG_TERM_PLAN for Phases 4-5
8631da7 
- Phase 4: Auto-pilot workflow, user guide, conflict resolution
- Phase 5: Learning from feedback, multi-model arbitration
- Infrastructure: Performance, monitoring, cost optimization
- Risk assessment and success metrics
- Prioritized 8-week roadmap
@stranske
Expand cleanup_labels.py classifications
3ab0d45 
- Add autofix:*, integration-*, agents:keepalive-nudge to functional
- Add common component labels (app, engine, ui, backend, cli)
- Add tech labels (javascript, python, github:actions)
- Add domain labels (metrics, modeling, schema, etc.)
- Reduces idiosyncratic labels from 150+ to 24
- Remaining 24 are legitimate project-specific labels
Copilot AI review requested due to automatic review settings January 10, 2026 05:04
Copilot AI reviewed Jan 10, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR expands the label classification sets in the cleanup script to better identify and categorize labels across repositories, reducing the count of idiosyncratic (unclassified) labels from 150+ to 24.

Changes:

  • Adds multiple autofix variants, integration/CI markers, and agent labels to FUNCTIONAL_LABELS
  • Expands INFORMATIONAL_LABELS with 75 new entries covering common categories (security, performance, testing), component labels (app, engine, ui, backend), priority/risk/status labels, and tech/domain labels
  • Updates documentation files (SHORT_TERM_PLAN.md and new LONG_TERM_PLAN.md) to track Phase 3 completion and plan for Phases 4-5

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
scripts/cleanup_labels.py Expands FUNCTIONAL_LABELS with autofix variants and CI markers; adds 75+ entries to INFORMATIONAL_LABELS covering categories, components, priorities, and tech labels
docs/plans/SHORT_TERM_PLAN.md Updates progress tracking with label matcher fixes, validation results, and completed PRs
docs/plans/LONG_TERM_PLAN.md New file documenting long-term planning for Phases 4-5 including full automation, intelligence features, and infrastructure improvements

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

scripts/cleanup_labels.py
"agent:copilot",
"agent:needs-attention",
"agent:decompose",
"agent:optimize",
Copy link

Copilot AI Jan 10, 2026

Choose a reason for hiding this comment

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

Both "agent:optimize" (singular) and "agents:optimize" (plural) are present in FUNCTIONAL_LABELS. This appears to be a duplicate entry with inconsistent naming. Verify whether both labels are actually used in workflows, or if one should be removed to avoid confusion.

Copilot uses AI. Check for mistakes.
scripts/cleanup_labels.py
"agent:claude",
"agent:copilot",
"agent:needs-attention",
"agent:decompose",
Copy link

Copilot AI Jan 10, 2026

Choose a reason for hiding this comment

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

Both "agent:decompose" (singular) and "agents:decompose" (plural) are present in FUNCTIONAL_LABELS. This appears to be a duplicate entry with inconsistent naming. Verify whether both labels are actually used in workflows, or if one should be removed to avoid confusion.

Copilot uses AI. Check for mistakes.
@agents-workflows-bot
Copy link
Copy Markdown
Contributor

agents-workflows-bot bot commented Jan 10, 2026

⚠️ Action Required: Unable to determine source issue for PR #739. The PR title, branch name, or body must contain the issue number (e.g. #123, branch: issue-123, or the hidden marker ).

@github-actions github-actions bot added the autofix Opt-in automated formatting & lint remediation label Jan 10, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jan 10, 2026

Status | ✅ no new diagnostics
History points | 1
Timestamp | 2026-01-10 05:13:13 UTC
Report artifact | autofix-report-pr-739
Remaining | 0
New | 0
No additional artifacts

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jan 10, 2026

Automated Status Summary

Head SHA: 0ccce25
Latest Runs: ⏳ pending — Gate
Required contexts: Gate / gate, Health 45 Agents Guard / Enforce agents workflow protections
Required: core tests (3.11): ⏳ pending, core tests (3.12): ⏳ pending, docker smoke: ⏳ pending, gate: ⏳ pending

Workflow / Job Result Logs
(no jobs reported) ⏳ pending —

Coverage Overview

  • Coverage history entries: 1

Coverage Trend

Metric Value
Current 92.21%
Baseline 85.00%
Delta +7.21%
Minimum 70.00%
Status ✅ Pass

Top Coverage Hotspots (lowest coverage)

File Coverage Missing
scripts/workflow_health_check.py 62.6% 28
scripts/classify_test_failures.py 62.9% 37
scripts/ledger_validate.py 65.3% 63
scripts/mypy_return_autofix.py 82.6% 11
scripts/ledger_migrate_base.py 85.5% 13
scripts/fix_cosmetic_aggregate.py 92.3% 1
scripts/coverage_history_append.py 92.8% 2
scripts/workflow_validator.py 93.3% 4
scripts/update_autofix_expectations.py 93.9% 1
scripts/pr_metrics_tracker.py 95.7% 3
scripts/generate_residual_trend.py 96.6% 1
scripts/build_autofix_pr_comment.py 97.0% 2
scripts/aggregate_agent_metrics.py 97.2% 0
scripts/fix_numpy_asserts.py 98.1% 0
scripts/sync_test_dependencies.py 98.3% 1

Updated automatically; will refresh on subsequent CI/Docker completions.

Keepalive checklist

Scope

No scope information available

Tasks

  • No tasks defined

Acceptance criteria

  • No acceptance criteria defined

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jan 10, 2026

🤖 Keepalive Loop Status

PR #739 | Agent: Codex | Iteration 0/5

Current State

Metric Value
Iteration progress [----------] 0/5
Action wait (missing-agent-label)
Disposition skipped (transient)
Gate success
Tasks 0/7 complete
Keepalive ❌ disabled
Autofix ❌ disabled

🔍 Failure Classification

| Error type | infrastructure |
| Error category | resource |
| Suggested recovery | Confirm the referenced resource exists (repo, PR, branch, workflow, or file). |

@stranske stranske merged commit 2b3b9ea into main Jan 10, 2026
36 checks passed
@stranske stranske deleted the docs/short-term-plan-update branch January 10, 2026 05:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

Assignees

No one assigned

Labels

autofix Opt-in automated formatting & lint remediation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@stranske