docs(playwright): de-slop instruction surfaces (0.6.3) - #3320
Conversation
Rewrite this plugin's README.md and every SKILL.md to drop em dashes under the repo's zero-tolerance house policy. Co-authored-by: Kyle Sexton <kyle-sexton@users.noreply.github.com>
|
Claude finished @kyle-sexton's task in 34s —— View job Security Review
Skip gate hit: no security-relevant surface. This PR only touches Per |
|
Last security-reviewed head: |
|
Claude finished @kyle-sexton's task in 52s —— View job Security review: no issues found
Skip gate hit: this PR has no security-relevant surface. All five changed files are documentation/metadata only:
No scripts, GitHub Actions workflows, auth/access-control code, injection surfaces, secret handling, or trust-boundary logic are touched by this diff. Nothing to flag. |
|
Claude finished @kyle-sexton's task in 38s —— View job /review:code-review — PR #3320
Result: skipped. Per If you'd like a review of the changes now that they've landed on |
|
Claude has reviewed this PR 1 time. The lane skips further automatic reviews after 5; deleting this comment resets the count. |
…right The gate shipped mode 100644 while the workflow invokes it as a bare path, `scripts/check-purged-em-dashes.sh`, exactly as it invokes every sibling scanner. Every one of those siblings is tracked 100755. The hygiene job would therefore have failed on contact with a permission error, and the repository's own unconditional exec-bit scanner would have reported the same defect a second time. Both the gate and its test script are now tracked executable. Add plugins/playwright to the allowlist. Its README and its skills were purged in #3320 and contain no em dash anywhere, so the entry meets the stricter standard the allowlist header declares for a seed entry: the whole list stays auditable by expanding it with --list and grepping the named files for the character. The plugin was clean when the list was seeded and was simply not carried over. Restructure the spaced double hyphen in the workflow comment into a sentence break. The gate's own failure message tells a contributor not to reach for a hyphen when an em dash comes out, and a comment introducing that gate should not model the substitution it warns against. Refs #2891 Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
1 participant
No linked issue
Summary
Refs #2891. Instruction-surface de-slop shard for the
playwrightplugin only. Other plugins stay on main.Fix
Rewrote
plugins/playwright/README.mdand everyplugins/playwright/**/SKILL.mdunder/ai-slop:audit fixsemantics (periods, commas, or a restructured sentence; never parentheses, en dashes, or a spaced hyphen as a stand-in). Meaning-preserving grammar pass after the mechanical replace. Version-only bump inplugin.json. New changelog heading only. Vendoredskills/playwright/vendor/SKILL.mdis left untouched as detector-excluded upstream baseline. No generated options block.docs/SKILL-CHEAT-SHEET.mdwas not edited.Verification
HOME+CLAUDE_PROJECT_DIRisolated sorule-em-dashis enabled): 0rule-em-dashfindings on rewritten instruction surfaces. VendorSKILL.mdwas not scanned.CHECK_SKILL_SKIP_MARKDOWNLINT=1 bash scripts/check-changed-skills.sh origin/main: 0 failed; quoted trigger phrases vs origin/main preservedpython3 scripts/sync-plugin-options-docs.py --checkup to datescripts/check-changelog-parity.sh --check-bump origin/mainpassesnode scripts/generate-cheatsheet.mjs --checkin syncRelated
Refs #2891