Skip to content

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

Merged
royalpinto007 merged 1 commit into
AgentPostmortem:mainfrom
vsolano9:fix-30-search-severity-labels
Sep 4, 2026
Merged

fix(search): use canonical severity labels#97
royalpinto007 merged 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
@royalpinto007
royalpinto007 merged commit f36c2bc into AgentPostmortem:main Sep 4, 2026
11 checks passed
@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown

Nice work @vsolano9, this is merged.

Genuinely helpful change for agentpostmortem.

If this project helps you out, a star helps others find it too.

Happy to see more from you whenever you have the time.

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

2 participants