Skip to content

feat(review): rule-scoped inline suppression via cora-ignore (#554) - #593

Merged
ajianaz merged 4 commits into
developfrom
feat/inline-suppression
Oct 8, 2026
Merged

ajianaz merged 4 commits into
developfrom
feat/inline-suppression

Conversation

@ajianaz

@ajianaz ajianaz commented Oct 8, 2026

Copy link
Copy Markdown
Collaborator

What

Adds cora-ignore: <rule>[, <rule>...] inline markers. A marker in any comment style suppresses findings with that exact title (case-insensitive) on the same line, or the next line when the marker line is comment-only. One shared filter (engine::inline_suppress) runs on merged deterministic + LLM findings, right after ignore.rules. A bare cora-ignore suppresses nothing. Documented in docs/configuration.md; CHANGELOG entry added.

Why

ignore.rules hides a title project-wide and ignore.files hides whole files; there was no way to accept one false positive (e.g. bytesPerToken) without hiding the rest. Closes #554.

Testing

  • 12 unit tests: same line, next line, multiple rules, unrelated rule/other file/other line kept, bare marker ignored, comment styles, case-insensitivity, exact-title match, plus a test running real secrets/security scanner output through the filter.
  • cargo fmt, cargo clippy --all-targets -- -D warnings, cargo test all green.

🤖 Generated with Claude Code

ajianaz and others added 4 commits October 8, 2026 11:43
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@ajianaz
ajianaz merged commit e0b1234 into develop Oct 8, 2026
14 checks passed
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.

Support line-level suppression for individual findings

1 participant