Repository navigation
fix: Phase 1 - Stop New Label Prefix Violations - #2476
Conversation
|
Warning Review limit reachedNext included review available in 58 minutes. View limit detailsLimit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: Path: .coderabbit.yml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (5)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
🔍 Reviewer Summary for PR #2476CI Status: ✅ Recommendations
|
✅ Phase 1 Implementation CompleteStatus: Ready for review and merge Deliverables Verified
Closes
Next PhasePhase 2 (Issue #1604) can proceed once this merges: Fix existing ~100 issues with bare labels. Generated by Claude Code |
This commit implements Phase 1 of the label prefix enforcement initiative (#2283): **Changes:** - Delete defective scripts/automation/labeling-agent.js that applied bare labels without required family prefixes (e.g., 'bug' instead of 'type:bug') - Add Label Creation Rules to CLAUDE.md with explicit guidance on canonical prefixed labels and examples of what NOT to do (bare labels) - Add Label Creation Governance section to AGENTS.md with pre-creation validation checklist for programmatic issue/PR creation **Impact:** - Prevents new issues/PRs from being created with bare, non-canonical labels - Establishes clear governance rules for all future labeling operations - Unblocks Phase 2 (fix existing ~100 issue labels) **Validation:** - No scripts now apply bare labels - Canonical label set required for all programmatic creation - Governance documents establish the standard Resolves: Issue #2283 - Phase 1: Stop New Label Prefix Violations Related: Issue #1592 - Label Prefix Governance Enforcement Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01VYTowkhdDkCDA2BQiRgLu3
The labeling-agent.js script was removed in Phase 1 as it was applying bare labels without required family prefixes. This update disables the labeling step in the issue-management-orchestration workflow until a corrected implementation is available. The workflow will now skip the automated labeling step with a note about the pending corrected implementation. Manual labeling via validated channels (PR governance, labeler.yml rules) will handle issue labeling. Related: Issue #2283 - Phase 1: Stop New Label Prefix Violations Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01VYTowkhdDkCDA2BQiRgLu3
5f003a8 to
edb46d9
Compare
🔍 Reviewer Summary for PR #2476CI Status: ✅ Recommendations
|
🔍 Reviewer Summary for PR #2476CI Status: ✅ Recommendations
|
Milestone Allocation |
* fix: Stop new label prefix violations - Phase 1 This commit implements Phase 1 of the label prefix enforcement initiative (#2283): **Changes:** - Delete defective scripts/automation/labeling-agent.js that applied bare labels without required family prefixes (e.g., 'bug' instead of 'type:bug') - Add Label Creation Rules to CLAUDE.md with explicit guidance on canonical prefixed labels and examples of what NOT to do (bare labels) - Add Label Creation Governance section to AGENTS.md with pre-creation validation checklist for programmatic issue/PR creation **Impact:** - Prevents new issues/PRs from being created with bare, non-canonical labels - Establishes clear governance rules for all future labeling operations - Unblocks Phase 2 (fix existing ~100 issue labels) **Validation:** - No scripts now apply bare labels - Canonical label set required for all programmatic creation - Governance documents establish the standard Resolves: Issue #2283 - Phase 1: Stop New Label Prefix Violations Related: Issue #1592 - Label Prefix Governance Enforcement Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01VYTowkhdDkCDA2BQiRgLu3 * fix: Update issue-management-orchestration workflow for Phase 1 The labeling-agent.js script was removed in Phase 1 as it was applying bare labels without required family prefixes. This update disables the labeling step in the issue-management-orchestration workflow until a corrected implementation is available. The workflow will now skip the automated labeling step with a note about the pending corrected implementation. Manual labeling via validated channels (PR governance, labeler.yml rules) will handle issue labeling. Related: Issue #2283 - Phase 1: Stop New Label Prefix Violations Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01VYTowkhdDkCDA2BQiRgLu3 * docs: Add Phase 1 changelog entry for label prefix enforcement - Issue #2283 --------- Co-authored-by: Claude <noreply@anthropic.com>
* fix: Stop new label prefix violations - Phase 1 This commit implements Phase 1 of the label prefix enforcement initiative (#2283): **Changes:** - Delete defective scripts/automation/labeling-agent.js that applied bare labels without required family prefixes (e.g., 'bug' instead of 'type:bug') - Add Label Creation Rules to CLAUDE.md with explicit guidance on canonical prefixed labels and examples of what NOT to do (bare labels) - Add Label Creation Governance section to AGENTS.md with pre-creation validation checklist for programmatic issue/PR creation **Impact:** - Prevents new issues/PRs from being created with bare, non-canonical labels - Establishes clear governance rules for all future labeling operations - Unblocks Phase 2 (fix existing ~100 issue labels) **Validation:** - No scripts now apply bare labels - Canonical label set required for all programmatic creation - Governance documents establish the standard Resolves: Issue #2283 - Phase 1: Stop New Label Prefix Violations Related: Issue #1592 - Label Prefix Governance Enforcement Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01VYTowkhdDkCDA2BQiRgLu3 * fix: Update issue-management-orchestration workflow for Phase 1 The labeling-agent.js script was removed in Phase 1 as it was applying bare labels without required family prefixes. This update disables the labeling step in the issue-management-orchestration workflow until a corrected implementation is available. The workflow will now skip the automated labeling step with a note about the pending corrected implementation. Manual labeling via validated channels (PR governance, labeler.yml rules) will handle issue labeling. Related: Issue #2283 - Phase 1: Stop New Label Prefix Violations Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01VYTowkhdDkCDA2BQiRgLu3 * docs: Add Phase 1 changelog entry for label prefix enforcement - Issue #2283 --------- Co-authored-by: Claude <noreply@anthropic.com>
Document Phase 2 framework deliverables: - audit-bare-labels.js: 77 bare→canonical label mappings - query-bare-labels.js: GitHub API discovery of bare labels in use - remediate-bare-labels.yml: Bulk remediation workflow (dry-run/live modes) - PHASE2_PLAN.md: Comprehensive remediation planning document - bare-label-mapping.json & bare-labels-found.json: Mapping & query outputs Phase 2 enables bulk remediation of ~100 legacy bare labels in existing issues/PRs, establishing repeatable process for fixing historical label governance violations. Resolves changelog validation check. Related issues: #1604 (bulk remediation), #1592 (governance tracking) Phase 1 reference: #2476 (prevention framework, merged) Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015FEzkEH8c4YM96DUi7rrCZ
Document Phase 2 framework deliverables: - audit-bare-labels.js: 77 bare→canonical label mappings - query-bare-labels.js: GitHub API discovery of bare labels in use - remediate-bare-labels.yml: Bulk remediation workflow (dry-run/live modes) - PHASE2_PLAN.md: Comprehensive remediation planning document - bare-label-mapping.json & bare-labels-found.json: Mapping & query outputs Phase 2 enables bulk remediation of ~100 legacy bare labels in existing issues/PRs, establishing repeatable process for fixing historical label governance violations. Resolves changelog validation check. Related issues: #1604 (bulk remediation), #1592 (governance tracking) Phase 1 reference: #2476 (prevention framework, merged) Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015FEzkEH8c4YM96DUi7rrCZ
Document Phase 2 framework deliverables: - audit-bare-labels.js: 77 bare→canonical label mappings - query-bare-labels.js: GitHub API discovery of bare labels in use - remediate-bare-labels.yml: Bulk remediation workflow (dry-run/live modes) - PHASE2_PLAN.md: Comprehensive remediation planning document - bare-label-mapping.json & bare-labels-found.json: Mapping & query outputs Phase 2 enables bulk remediation of ~100 legacy bare labels in existing issues/PRs, establishing repeatable process for fixing historical label governance violations. Resolves changelog validation check. Related issues: #1604 (bulk remediation), #1592 (governance tracking) Phase 1 reference: #2476 (prevention framework, merged) Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015FEzkEH8c4YM96DUi7rrCZ
Linked issues
Closes #2283 (Phase 1: Stop New Label Prefix Violations)
Relates to #1592 (Label Prefix Governance Enforcement)
Relates to #2352 (Enforce PR labeling requirement)
Summary
Phase 1 of the label prefix enforcement initiative—stopping new violations by removing defective code and establishing explicit governance rules for all programmatic label creation.
Changes
Deleted
scripts/automation/labeling-agent.js— Defective script that applied bare labels without required family prefixes (e.g.,buginstead oftype:bug)Updated Governance
Impact
✅ Prevents new issues/PRs from being created with bare, non-canonical labels
✅ Establishes clear governance rules for all future labeling operations
✅ Unblocks Phase 2 (fix existing ~100 issue labels in #1604)
✅ Canonical label set required for all programmatic creation
Changelog
Removed
scripts/automation/labeling-agent.jsthat created bare labels without required prefixesChanged
Risk Assessment
Risk Level: Low
Potential Impact:
Mitigation Steps:
How to Test
Test Steps
Verify the defective script has been removed:
ls -la scripts/automation/labeling-agent.js # Expected: File not foundVerify governance documentation is clear:
Verify no bare labels are applied in workflows:
Expected Results
Edge Cases to Verify
Checklist (Global DoD / PR)
References
Generated by Claude Code