Merge pull request #332 from chumyin/fix/labeler-collapse-all-prefixes

ci(labeler): collapse duplicate module labels for all prefixes

.github/workflows/labeler.yml

@@ -297,7 +297,7 @@ jobs:
                 .toLowerCase()
                 .replace(/\.rs$/g, "")
                 .replace(/[^a-z0-9_-]+/g, "-")
-                .replace(/^-+|-+$/g, "")
+                .replace(/^[-_]+|[-_]+$/g, "")
                 .slice(0, 40);
             }
 
@@ -378,44 +378,36 @@ jobs:
               return refined;
             }
 
-            function compactNoisyModuleLabels(labels) {
+            function compactModuleLabels(labels) {
-              const noisyPrefixes = new Set(["tool", "provider", "channel"]);
               const groupedSegments = new Map();
-              const compacted = new Set();
+              const compactedModuleLabels = new Set();
               const forcePathPrefixes = new Set();
 
               for (const label of labels) {
                 const parsed = parseModuleLabel(label);
-                if (!parsed) continue;
+                if (!parsed) {
+                  compactedModuleLabels.add(label);
+                  continue;
+                }
                 if (!groupedSegments.has(parsed.prefix)) {
                   groupedSegments.set(parsed.prefix, new Set());
                 }
                 groupedSegments.get(parsed.prefix).add(parsed.segment);
               }
 
-              for (const label of labels) {
-                const parsed = parseModuleLabel(label);
-                if (!parsed) continue;
-                if (!noisyPrefixes.has(parsed.prefix)) {
-                  compacted.add(label);
-                }
-              }
-
               for (const [prefix, segments] of groupedSegments) {
-                if (!noisyPrefixes.has(prefix)) continue;
+                const uniqueSegments = [...new Set([...segments].filter(Boolean))];
+                if (uniqueSegments.length === 0) continue;
 
-                const specificSegments = [...segments].filter((segment) => segment !== "core");
+                if (uniqueSegments.length === 1) {
-                const uniqueSpecificSegments = [...new Set(specificSegments)];
+                  compactedModuleLabels.add(`${prefix}:${uniqueSegments[0]}`);
-
-                if (uniqueSpecificSegments.length === 1) {
-                  compacted.add(`${prefix}:${uniqueSpecificSegments[0]}`);
                 } else {
                   forcePathPrefixes.add(prefix);
                 }
               }
 
               return {
-                moduleLabels: compacted,
+                moduleLabels: compactedModuleLabels,
                 forcePathPrefixes,
               };
             }
@@ -560,7 +552,7 @@ jobs:
             }
 
             const refinedModuleLabels = refineModuleLabels(detectedModuleLabels);
-            const compactedModuleState = compactNoisyModuleLabels(refinedModuleLabels);
+            const compactedModuleState = compactModuleLabels(refinedModuleLabels);
             const selectedModuleLabels = compactedModuleState.moduleLabels;
             const forcePathPrefixes = compactedModuleState.forcePathPrefixes;
             const modulePrefixesWithLabels = new Set(

docs/ci-map.md

@@ -31,7 +31,7 @@ Merge-blocking checks should stay small and deterministic. Optional checks are u
     - Additional behavior: label descriptions are auto-managed as hover tooltips to explain each auto-judgment rule
     - Additional behavior: provider-related keywords in provider/config/onboard/integration changes are promoted to `provider:*` labels (for example `provider:kimi`, `provider:deepseek`)
     - Additional behavior: hierarchical de-duplication keeps only the most specific scope labels (for example `tool:composio` suppresses `tool:core` and `tool`)
-    - Additional behavior: noisy namespaces (`tool`, `provider`, `channel`) are compacted — one specific module keeps `prefix:component`; multiple specifics collapse to just `prefix`
+    - Additional behavior: module namespaces are compacted — one specific module keeps `prefix:component`; multiple specifics collapse to just `prefix`
     - Additional behavior: applies contributor tiers on PRs by merged PR count (`experienced` >=10, `principal` >=20, `distinguished` >=50)
     - Additional behavior: final label set is priority-sorted (`risk:*` first, then `size:*`, then contributor tier, then module/path labels)
     - Additional behavior: managed label colors follow display order to produce a smooth left-to-right gradient when many labels are present

docs/pr-workflow.md

@@ -50,7 +50,7 @@ Maintain these branch protection rules on `main`:
 
 - Contributor opens PR with full `.github/pull_request_template.md`.
 - `PR Labeler` applies scope/path labels + size labels + risk labels + module labels (for example `channel:telegram`, `provider:kimi`, `tool:shell`) and contributor tiers by merged PR count (`experienced` >=10, `principal` >=20, `distinguished` >=50), while de-duplicating less-specific scope labels when a more specific module label is present.
-- For `tool` / `provider` / `channel`, module labels are compacted to reduce noise: one specific module keeps `prefix:component`, but multiple specifics collapse to the base scope label.
+- For all module prefixes, module labels are compacted to reduce noise: one specific module keeps `prefix:component`, but multiple specifics collapse to the base scope label `prefix`.
 - Label ordering is priority-first: `risk:*` -> `size:*` -> contributor tier -> module/path labels.
 - Hovering a label in GitHub shows its auto-managed description (rule/threshold summary).
 - Managed label colors are arranged by display order to create a smooth gradient across long label rows.