Skip to content

fix(search): use canonical severity labels - #97

Open
vsolano9 wants to merge 1 commit into
AgentPostmortem:mainfrom
vsolano9:fix-30-search-severity-labels
Open

fix(search): use canonical severity labels#97
vsolano9 wants to merge 1 commit into
AgentPostmortem:mainfrom
vsolano9:fix-30-search-severity-labels

Conversation

@vsolano9

Copy link
Copy Markdown

Closes #30

What changed

  • remove the private long-form severity label map from SearchResults
  • use the canonical shared labels and SeverityLevel type for the filter range
  • narrow adjacent range values with the existing severity guard rather than using any
  • preserve the deliberate short labels used by dense listings

Validation

  • Node 22.23.2: npx vitest run — 14 files, 121 tests passed; full suite repeated 20 times
  • Node 22.23.2: npx tsc --noEmit
  • Node 22.23.2: npm run lint — no warnings or errors
  • Node 22.23.2: npx prettier --check .
  • Node 22.23.2: production npm run build with the CI placeholder environment
  • headed browser: desktop and 390×844 production mobile visual inspection
  • headed browser: all five canonical accessible names; range transitions 1–5 → 2–5 → 2–4 → 3 → reset; zero browser errors/warnings

No dependency or lockfile changes.

Use the shared long-form severity vocabulary in the search filter and type the range as SeverityLevel values. Preserve the existing range interaction while eliminating the private label map that had drifted from case pages.

Closes AgentPostmortem#30
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.

Search filter shows different severity labels than the rest of the site

1 participant