Skip to content

fix(analyzers): avoid nominal MP3 and P6 matches - #453

Merged
rng1995 merged 1 commit into
mainfrom
naren/fix-mp3-p6-noun-phrases
Aug 28, 2026
Merged

fix(analyzers): avoid nominal MP3 and P6 matches#453
rng1995 merged 1 commit into
mainfrom
naren/fix-mp3-p6-noun-phrases

Conversation

@rng1995

@rng1995 rng1995 commented Aug 28, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • recognize the complete state-coverage enumeration around initial/reset state as a nominal MP3 context
  • recognize the declarative selector taxonomy around descendant/compound/print rules as a nominal P6 context, including its single logical-line wrap
  • keep suppression bounded and fail closed around directive framing, incomplete windows, and referential continuations
  • add exact-source, malicious near-miss, mixed-content, logical-line, and boundary regressions

The guards are analyzer-local and positive-shape based. They do not whitelist documentation, code, paths, or file types, so direct and framed extraction/state-manipulation instructions remain findings.

Validation

  • ruff check src/ tests/
  • ruff format --check src/ tests/
  • pytest -q tests/unit/test_patterns_new.py — 473 passed
  • pytest -q -m "not integration and not provider" tests/ — 3,138 passed, 14 skipped, 4 xfailed
  • full skillspector scan --no-llm of the reported skill — zero MP3/P6 findings from the two benign passages

Signed-off-by: Narendran Raghavan nraghavan@nvidia.com

Signed-off-by: Narendran Raghavan <nraghavan@nvidia.com>
@rng1995
rng1995 merged commit 2d1dd38 into main Aug 28, 2026
5 checks passed
@rng1995
rng1995 deleted the naren/fix-mp3-p6-noun-phrases branch August 28, 2026 06:43
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.

1 participant