fix(labels): reconcile canonical label families#418
Conversation
|
Caution Review failedThe pull request is closed. Note
|
| Layer / File(s) | Summary |
|---|---|
New Issue Types and Template Configuration .github/issue-types.yml, .github/ISSUE_TEMPLATE/25-help.md |
Adds Question and Support as new issue types with colors and type:* labels; updates help template to use prefixed labels (type:question, type:support). |
Canonical Labels Definition with Aliases .github/labels.yml |
Establishes canonical label registry with new type:question, type:support, and area:labels; adds backward-compatible aliases to existing type/area/language labels so legacy unprefixed names are recognized. |
Automated Labeling Rules for Branches and Files .github/labeler.yml |
Extends automated labeling with branch patterns for status:needs-review, introduces type:build for CI branches, consolidates type:chore handling, expands area mappings for labels/dependencies/security/a11y, and adds language detection for markdown/json/yaml. |
Workflow Enhancements for Validation and Link Checking .github/workflows/labeling.yml, .github/workflows/changelog-validate.yml, .github/workflows/meta.yml |
Disables actions/labeler due to schema incompatibility and delegates to agent-based labeling; gates changelog validation on meta:no-changelog label; refactors link checking to scan only changed markdown files instead of all files. |
Migration Guide and Updated Documentation docs/MIGRATION.md, docs/ISSUE_TYPES.md, docs/ISSUE_LABELS.md, docs/README.md |
Introduces comprehensive migration guide with legacy-to-canonical mappings and governance rules; updates issue type and label guidance docs; indexes migration resources in documentation. |
Orphan Labels Audit Report .github/reports/audits/issue-95-orphan-labels-audit-2026-05-27.md |
Documents label audit findings including reconciliation metrics, completed migrations, pending decisions, and recommended cleanup paths. |
Estimated code review effort
🎯 3 (Moderate) | ⏱️ ~20 minutes
Possibly related issues
#419: Directly related—disablesactions/labelerin this PR (citing schema incompatibility) while issue#419aims to make the labeler schema-compatible to re-enable it.#95: Both changes address the same orphan-labels governance issue—the PR implements the label structure and audit referenced in#95.
Suggested labels
documentation
✨ Finishing Touches
🧪 Generate unit tests (beta)
- Create PR with unit tests
- Commit unit tests in branch
codex/issue-95-label-orphan-audit
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 @coderabbitai help to get the list of available commands and usage tips.
There was a problem hiding this comment.
Code Review
This pull request adds a new audit report documenting the reconciliation of repository labels against the canonical configuration. The reviewer noted that the proposed legacy-to-canonical label mappings should be documented in a central /docs/MIGRATION.md file to align with repository standards.
| 2. Migrate actively used legacy labels to canonical equivalents where the mapping | ||
| is clear, for example `bug` to `type:bug`, `documentation` to | ||
| `type:documentation`, `security` to `type:security`, and `ci` to `area:ci` | ||
| or `type:build` depending on context. |
There was a problem hiding this comment.
The proposed migration path lists several legacy-to-canonical label mappings (e.g., 'bug' to 'type:bug', 'documentation' to 'type:documentation'). According to the repository's general rules, all migration maps and notes should be documented in a central '/docs/MIGRATION.md' file to ensure contributors can easily follow them. Please add these mappings to '/docs/MIGRATION.md' or reference it here.
References
- Document migration maps and notes in a central
/docs/MIGRATION.mdfile to ensure contributors can follow migration rules mentioned in README files across the repository.
🔍 Reviewer Summary for PR #418CI Status: ✅ Recommendations
|
🔍 Reviewer Summary for PR #418CI Status: ✅ Recommendations
|
Signed-off-by: Ash Shaw <ashley@lightspeedwp.agency>
🔍 Reviewer Summary for PR #418CI Status: ✅ Recommendations
|
1 similar comment
🔍 Reviewer Summary for PR #418CI Status: ✅ Recommendations
|
🔍 Reviewer Summary for PR #418CI Status: ✅ Recommendations
|
🔍 Reviewer Summary for PR #418CI Status: ✅ Recommendations
|
🔍 Reviewer Summary for PR #418CI Status: ✅ Recommendations
|
🔍 Reviewer Summary for PR #418CI Status: ✅ Recommendations
|
🔍 Reviewer Summary for PR #418CI Status: ✅ Recommendations
|
🔍 Reviewer Summary for PR #418CI Status: ✅ Recommendations
|
This reverts commit d10995d.
🔍 Reviewer Summary for PR #418CI Status: ✅ Recommendations
|
Summary
.github/reports/audits/.docs/MIGRATION.mdas the central contributor-facing migration map for legacy label mappings and maintainer-decision labels..github/labels.ymluse approved family prefixes.questionandsupportwithtype:questionandtype:support, and adds both to.github/issue-types.yml.area:labelsplus useful labeler rules for label governance, dependencies, security, accessibility, and language routing.Live GitHub Label Sync
lightspeedwp/.github.Current live state after sync/migration:
.github/labels.yml: 149Remaining Maintainer Decisions
The remaining active ambiguous labels are
bats,checklist,cross-reference,governance,onboarding,path-resolution,quickstart, andstandards. These need maintainer approval before migration, canonicalisation, archival, or deletion. The decision list now lives indocs/MIGRATION.md.Validation
node scripts/agents/includes/check-template-labels.jsnpx markdownlint-cli2 .github/ISSUE_TEMPLATE/25-help.md docs/ISSUE_LABELS.md docs/ISSUE_TYPES.md .github/reports/audits/issue-95-orphan-labels-audit-2026-05-27.mdnpx markdownlint-cli2 docs/MIGRATION.md docs/README.md .github/reports/audits/issue-95-orphan-labels-audit-2026-05-27.mdgit diff --checkNotes
Refs #95. This remains intentionally non-destructive for ambiguous/historical labels.
Summary by CodeRabbit
New Features
Documentation
Chores