Skip to content

feat: add severity filter support in findings dashboard#128

Open
swathiyara9-ai wants to merge 1 commit into
ionfwsrijan:mainfrom
swathiyara9-ai:feature/severity-filter
Open

feat: add severity filter support in findings dashboard#128
swathiyara9-ai wants to merge 1 commit into
ionfwsrijan:mainfrom
swathiyara9-ai:feature/severity-filter

Conversation

@swathiyara9-ai

@swathiyara9-ai swathiyara9-ai commented Jun 17, 2026

Copy link
Copy Markdown

Linked issue

Closes #39

What this PR does

Adds severity-based filtering to the Findings dashboard. Users can filter findings by Critical, High, Medium, and Low severity levels. The findings table updates dynamically based on selected filters, and filter selections are preserved using URL search parameters for improved navigation and triage workflows.

Type of change

  • Bug fix
  • New feature
  • ML model / training pipeline
  • Refactor (no behaviour change)
  • Documentation
  • Tests only

ML tier (if applicable)

  • Tier 1 — Triage
  • Tier 2 — Predictive
  • Tier 3 — Autonomous
  • Not ML-related

Stack affected

  • Backend
  • Frontend
  • Both

Changes

Backend

  • None

Frontend

  • Added Critical, High, Medium, and Low severity filter options.
  • Implemented dynamic filtering of findings based on selected severities.
  • Preserved filter state using URL search parameters.
  • Improved findings triage workflow and user experience.

New dependencies

  • None

Database / schema changes

  • None

Testing

How did you test this?

Tested locally by running the frontend application and verifying individual and multiple severity filter selections. Confirmed that findings update dynamically based on selected filters and that filter state persists during navigation and page refresh. Also verified that URL search parameters correctly preserve selected severity filters.

Checklist

  • Tested locally end-to-end (upload ZIP or GitHub URL → scan → findings returned correctly)
  • New ML model falls back gracefully when model file is absent
  • No new "console.error" or unhandled Python exceptions introduced
  • Added or updated tests where applicable
  • "requirements.txt" / "package.json" updated if new dependencies added
  • New model files (".pkl", ".pt", etc.) are gitignored, not committed

Anything reviewers should focus on

Please review the severity filtering logic, URL state persistence implementation, and interaction between severity filters and findings table updates in "frontend/src/app/pages/findings.tsx".

Screenshots (if UI changed)

  • Added severity filter chips (Critical, High, Medium, Low) to the Findings dashboard.
  • Verified dynamic filtering of findings based on selected severity levels.
  • Screenshot attached in the PR showing the filter functionality in action.

@github-actions

Copy link
Copy Markdown

⚠️ Automated Check: This PR does not strictly follow the required template. Please ensure you have not deleted any checkboxes or mandatory headings, and that you have written explanations under What this PR does and How did you test this?.

@ionfwsrijan

Copy link
Copy Markdown
Owner

@swathiyara9-ai Please fix failing checks

@github-actions github-actions Bot added feature New feature frontend Frontend issues labels Jun 18, 2026
@swathiyara9-ai

Copy link
Copy Markdown
Author

Hi! @ionfwsrijan I noticed that a PR implementing Issue #39 has already been merged after I opened my PR. My branch now shows conflicts with the updated "main" branch. Since the issue appears to be resolved, please let me know if any further changes are needed from my side. Thank you for your review.

@ionfwsrijan

Copy link
Copy Markdown
Owner

@swathiyara9-ai Fix merge conflicts and PR description.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New feature frontend Frontend issues needs-work Work needed SSoC26

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature] Add finding severity filter in frontend dashboard

2 participants