Skip to content

feat(triage): make labels off by default (CL-12165) - #852

Merged
TheGreatAxios merged 1 commit into
mainfrom
cl-12165-labels-off-by-default
Nov 5, 2026
Merged

TheGreatAxios merged 1 commit into
mainfrom
cl-12165-labels-off-by-default

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Contributor

Summary

Triage now adds no labels unless the policy file sets labels: true or a map that renames them. A policy without labels:, or no policy file, means no verdict labels and no Triage: Failed to Run.

  • A missing labels key reads as off; a repository's own key still wins over extends.
  • With labels off, the first write Triage makes to a pull request takes its own old labels off, such as Triage: Ready. This uses the mirror's existing owned-label stripping, so no new code path was needed.
  • Exporting a saved policy that uses the default label names writes labels: true, so the labels survive the round trip.
  • The starter policy file and its preview carry no labels.
  • Docs: the User Manual now says Triage adds no labels unless the policy file sets labels: true, and its label section and Failed to Run text are trimmed to match. The setup skill's reference is updated too.

Verification

  • bun run check and bun run lint pass.
  • bun --conditions=intx-src test packages/triage-contracts packages/triage-workflows packages/triage-manager packages/github-tool tests/model tests/policy-corpus: 544 pass.
  • bun --conditions=intx-src test --parallel=2 tests/chain: 13 pass. The chain policies set labels: true.
  • New policy-corpus fixtures cover no key (off), labels: true (on) and a rename map (on), checked against the policy the hub runs.
  • New model test: a stack with labels off adds no label, and removes an existing "Triage: Ready" exactly once across two heads.

Closes CL-12165

@linear-code

linear-code Bot commented Nov 5, 2026

Copy link
Copy Markdown

CL-12165

@corbits-triage

corbits-triage Bot commented Nov 5, 2026 •

Copy link
Copy Markdown
Contributor

Merged before Triage finished on 3dab6d1.

@TheGreatAxios
TheGreatAxios merged commit 0a36922 into main Nov 5, 2026
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant