Skip to content
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 9 additions & 0 deletions extension/src/rules/attribute-injection-sanitize.ts
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,15 @@
// would use to plant a "pre-confirmed" instruction that the agent
// treats as load-bearing while the user has no chance to clear it.
//
// Accepted limitation: an enabled `<input value="…">` sitting inside a
// CSS-hidden wrapper (`visibility:hidden`, off-left, opacity:0) is not
// scrubbed. The same asymmetry as the disabled case applies — the user
// can't see or edit the value — but matching it would require a
// computed-style check at scrub time, which conflicts with this rule's
// lightweight attribute-driven watcher. Pre-#176 these were caught when
// `hidden-text-strip` detached the wrapper; the regression is accepted
// because the trigger surface is narrow (enabled input + hidden wrapper).
//
// On a match we remove the whole attribute rather than blanking its
// value. An empty `aria-label` actively hides an element from
// accessibility-tree consumers (which is worse than no aria-label,
Expand Down
Loading