Skip to content

feat(lint-artifact): require a writeClipboard helper - #14

Merged
cursor[bot] merged 1 commit into
mainfrom
cursor/pr-clipboard-85a6
Oct 4, 2026
Merged

cursor[bot] merged 1 commit into
mainfrom
cursor/pr-clipboard-85a6

Conversation

@twidtwid

@twidtwid twidtwid commented Oct 4, 2026

Copy link
Copy Markdown
Owner

Why

Fifteen HTML files copy the same clipboard helper. A file that calls navigator.clipboard.writeText without writeClipboard will throw on file:// and hardened browsers.

What changed

  • The lint fails when writeText appears without a writeClipboard helper in the same file.
  • The existing unguarded-clipboard fail stays. A helper plus writeClipboard(t) passes.

Scope

This PR only adds the helper check. The skill snippet already uses writeClipboard, so the snippet did not change.

Blast Radius

Every current example already defines the helper. Default lint still passes. Skill tokens stay 12256.

Verification

  • examples/dashboard.html PASSes and contains writeClipboard.
  • The unguarded fixture fails and names the helper or the guard.
  • make check is green.
Open in Web Open in Cursor 

@cursor
cursor Bot force-pushed the cursor/pr-update-lint-85a6 branch from 71b2ac4 to 077d58b Compare October 4, 2026 21:09
@cursor
cursor Bot force-pushed the cursor/pr-clipboard-85a6 branch from 5c85046 to cb99762 Compare October 4, 2026 21:09
@cursor
cursor Bot force-pushed the cursor/pr-update-lint-85a6 branch from 077d58b to 1b73847 Compare October 4, 2026 22:19
@cursor
cursor Bot force-pushed the cursor/pr-clipboard-85a6 branch from cb99762 to 8f12b83 Compare October 4, 2026 22:21
@cursor
cursor Bot changed the base branch from cursor/pr-update-lint-85a6 to main October 4, 2026 22:22
A file that calls navigator.clipboard.writeText must define
writeClipboard in the same file. The unguarded fixture still fails.
A helper plus writeClipboard(t) passes.

Co-authored-by: Todd Dailey <twidtwid@users.noreply.github.com>
@cursor
cursor Bot force-pushed the cursor/pr-clipboard-85a6 branch from 8f12b83 to 5fc9e86 Compare October 4, 2026 22:23
@cursor
cursor Bot merged commit 2aaabb1 into main Oct 4, 2026
1 check 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.

2 participants