feat(issues): add orthogonal area labels for triage - #830
Conversation
Map template Area fields and light heuristics to additive labels (provider, account-pool, catalog, gui, cli, proxy, platform, streaming, tools, install, service) beside existing kind/process labels, with open issue backfill via workflow_dispatch.
This comment was marked as outdated.
This comment was marked as outdated.
Prefer title-scoped matches for platform/gui/cli and stronger full-body tokens so Environment OS fields do not blanket-label every bug report.
[shipping-github] Security reviewPR / issue: #830 @ Findingsnone confirmed SummaryReviewed the issue-quality automation diff: Actions stay pinned to full SHAs, scripts still load only from the repository default branch, and Residual
Fixes this session
|
Omit Documentation Area→kind collision, restrict heuristics to semantic sections, feed translation text into area detection, and update the issue-quality workflow invariant test for the backfill guard.
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In @.github/scripts/issue-quality.cjs:
- Around line 495-513: Update the validate job in
.github/workflows/enforce-issue-quality.yml at lines 943-947 and 1226-1230 to
depend on translate via needs: translate, ensuring validation runs after
translation completes. No direct change is needed in detectAreaLabels within
.github/scripts/issue-quality.cjs; continue using the live issue.title and
existing heuristicBody flow without adding translatedTitle.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro Plus
Run ID: 24a4c61b-365f-4ef6-a15b-c821daff17d2
📒 Files selected for processing (5)
.github/scripts/issue-quality.cjs.github/scripts/issue-quality.test.cjs.github/workflows/enforce-issue-quality.ymldocs-site/src/content/docs/contributing.mdtests/ci-workflows.test.ts
Fix streaming truncat stem matching, only addLabels for labels ensureLabel confirmed available, and document issues:write on the backfill job.
Add needs: translate so area labeling can read the inline English translation block on the same issue event, while still running validate when translate is skipped or fails.
[shipping-github] Fixed (job ordering)Agreed — Landed in |
|
Merging this. Why it helps: open issues can now be filtered by orthogonal area labels ( Ship it. |
Summary
provider,account-pool,catalog,gui,cli,proxy,platform,streaming,tools,install,service) beside existing kind/process labels.issue-quality.cjs; wire additive apply + label-ensure intoenforce-issue-quality.yml.provider; workflow_dispatchbackfill_open_areasrelabels open issues only (no quality closure). Documented in contributing.Test plan
node --test .github/scripts/issue-quality.test.cjs(101 pass)main: create/ensure labels, run workflow_dispatch withbackfill_open_areas=true, spot-check open issues.github/automation)Summary by CodeRabbit
New Features
Documentation