Skip to content

Fix: normalize labels strictly - #14

Merged
shinyay merged 2 commits into
demo-260105from
shinyay/issue11
Jan 5, 2026
Merged

shinyay merged 2 commits into
demo-260105from
shinyay/issue11

Conversation

@shinyay

@shinyay shinyay commented Jan 5, 2026

Copy link
Copy Markdown
Owner

Summary

Normalize TriageOutput.labels for stability:

  • trim whitespace
  • lowercase
  • drop empty labels
  • de-duplicate (preserve order)

Add a regression test to ensure the first occurrence is also lowercased.

Related issue

Checklist

  • Scoped to a single issue
  • Tests added/updated as needed
  • pytest -q passes
  • CLI output contract preserved (JSON on stdout)
  • Docs updated if behavior changed

Validation

Commands run:

  • pytest -q
  • ruff check .
  • ruff format --check .

@shinyay
shinyay merged commit eb27fb1 into demo-260105 Jan 5, 2026
2 checks passed
@shinyay
shinyay deleted the shinyay/issue11 branch January 5, 2026 10:22
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