You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
SkillSpector reports P2 prompt-injection on benign standalone header
comments: license headers (<!-- Copyright (c) 2026 Example Corp. SPDX-License-Identifier: Apache-2.0 ... -->), get-started pointers,
and dependency declarations in complete top-of-file comments that
carry no override or exfiltration signal. These are distinct from #37's reported spans (matches running past --> into <style>
blocks), which come from the unbounded .*? in the P2 comment
regex and are out of scope here.
Proposed scope
Exempt a P2 comment match only when it is one complete comment near
the top of the file (or directly after closed frontmatter), at most
300 characters, danger-free, and every fragment fully matches an
anchored license-line form or a per-key metadata grammar. Spanning,
partial, and multi-comment matches keep firing (fail-closed).
Acceptance
The three repro cases above (plus reference-comment form) scan
clean; the reviewer's space-joined and imperative-value payloads
still fire, pinned as regressions.
The scoped implementation is in the open PR #622. It provides the narrowly bounded, complete, danger-free header-comment carve-out described here; it is not a general comment exemption. This issue remains open until that PR merges, and the broader spanning-comment/root-cause work in #37 remains separate.
Activity
rng1995 commentedon Oct 4, 2026
The scoped implementation is in the open PR #622. It provides the narrowly bounded, complete, danger-free header-comment carve-out described here; it is not a general comment exemption. This issue remains open until that PR merges, and the broader spanning-comment/root-cause work in #37 remains separate.
PR-state snapshot checked on 2026-10-04:
Keeping this issue open: the relevant implementation is not merged. Check/review status is a point-in-time snapshot, not a claim of merge readiness.