Skip to content

feat: add local ignore overrides for rule IDs + filepaths#59

Open
lelia wants to merge 17 commits intomainfrom
lelia/add-sast-ignore-overrides
Open

feat: add local ignore overrides for rule IDs + filepaths#59
lelia wants to merge 17 commits intomainfrom
lelia/add-sast-ignore-overrides

Conversation

@lelia
Copy link
Copy Markdown
Contributor

@lelia lelia commented Mar 30, 2026

Summary

  • Add a local SAST ignore mechanism so users can suppress narrowly scoped findings that are blocking PRs without disabling the entire rule globally
  • Improve GitHub PR state handling so ignored or resolved findings no longer leave behind misleading blocking status, stale severity labels, or stale scanner comments

Changes

  • Add sast_ignore_overrides parameter so Socket Basics users can ignore SAST findings by rule_id or exact rule_id:path and make ignored findings stop contributing to blocking CI results
  • Normalize CI checkout paths for exact-path overrides across GitHub Actions and other common CI environments
    • Add clear diagnostics for invalid override paths (log warning, troubleshooting docs)
    • Include actionReason metadata to explain why a finding was ignored
  • Update GitHub PR reporting to reconcile managed severity labels across reruns
    • Scope all-clear comment rewrites to the correct scanner section so one scanner going clean does not overwrite unrelated Socket Basics comments

Testing

  • Verified local/unit coverage for override parsing, CI path normalization, invalid-path diagnostics, ignore provenance metadata, and GitHub PR label/comment lifecycle behavior
  • Validated in a separate GitHub Actions testing repository using rule-only, exact rule_id:path, mixed comma-separated override strings, valid and invalid file paths, and dashboard-level rule disable interactions
  • Confirmed that ignored findings no longer block the PR, invalid override paths emit warnings without falling back to rule-only matching, labels downgrade/clear correctly across reruns, and all-clear updates only rewrite the corresponding scanner comment

lelia added 13 commits March 30, 2026 18:47
Signed-off-by: lelia <2418071+lelia@users.noreply.github.com>
Signed-off-by: lelia <2418071+lelia@users.noreply.github.com>
Signed-off-by: lelia <2418071+lelia@users.noreply.github.com>
Signed-off-by: lelia <2418071+lelia@users.noreply.github.com>
Signed-off-by: lelia <2418071+lelia@users.noreply.github.com>
Signed-off-by: lelia <2418071+lelia@users.noreply.github.com>
Signed-off-by: lelia <2418071+lelia@users.noreply.github.com>
…olved

Signed-off-by: lelia <2418071+lelia@users.noreply.github.com>
Signed-off-by: lelia <2418071+lelia@users.noreply.github.com>
Signed-off-by: lelia <2418071+lelia@users.noreply.github.com>
Signed-off-by: lelia <2418071+lelia@users.noreply.github.com>
…pabilities

Signed-off-by: lelia <2418071+lelia@users.noreply.github.com>
Signed-off-by: lelia <2418071+lelia@users.noreply.github.com>
@lelia lelia marked this pull request as ready for review April 2, 2026 02:21
@lelia lelia requested a review from a team as a code owner April 2, 2026 02:21
lelia added 4 commits April 2, 2026 22:59
Signed-off-by: lelia <2418071+lelia@users.noreply.github.com>
Signed-off-by: lelia <2418071+lelia@users.noreply.github.com>
Signed-off-by: lelia <2418071+lelia@users.noreply.github.com>
Signed-off-by: lelia <2418071+lelia@users.noreply.github.com>
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.

2 participants