Skip to content

Feat: Reflect deduplicated findings in the evidence pack#93

Open
Meeran-Dev wants to merge 1 commit into
ionfwsrijan:mainfrom
Meeran-Dev:feature/deduplicatedfindings
Open

Feat: Reflect deduplicated findings in the evidence pack#93
Meeran-Dev wants to merge 1 commit into
ionfwsrijan:mainfrom
Meeran-Dev:feature/deduplicatedfindings

Conversation

@Meeran-Dev

@Meeran-Dev Meeran-Dev commented Jun 11, 2026

Copy link
Copy Markdown

Before opening: make sure there is an issue tracking this work, and link it below. PRs without a linked issue may be closed without review.

Linked issue

Closes #86

What this PR does

This PR updates the evidence pack generation to reflect deduplicated findings and surface deduplication metrics.

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

  • Updated evidence_pack.py to build evidence packs from deduplicated findings
  • Added fallback to use in-memory scan results when DB results are unavailable
  • Added summary section line, e.g. Findings after deduplication: 12 (from 47 raw)
  • Included related_files for each deduplicated finding so auditors can see the full file scope

Testing

How did you test this?

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

@github-actions github-actions Bot added backend Backend issues feature New feature SSoC26 needs-work Work needed labels Jun 11, 2026
@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?.

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

Labels

backend Backend issues feature New feature needs-work Work needed SSoC26

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Reflect deduplicated findings in the evidence pack

1 participant