Repository navigation
feat(lint-artifact): compare an update against the prior file - #13
Merged
Merged
Conversation
cursor
Bot
force-pushed
the
cursor/pr-registry-85a6
branch
from
October 4, 2026 21:08
a3cd5ce to
88d7348
Compare
cursor
Bot
force-pushed
the
cursor/pr-update-lint-85a6
branch
from
October 4, 2026 21:09
71b2ac4 to
077d58b
Compare
cursor
Bot
force-pushed
the
cursor/pr-registry-85a6
branch
from
October 4, 2026 22:17
88d7348 to
a8eb3c4
Compare
--against fails dropped ids, a dropped copy-as-prompt control, or a deleted inline script. --allow-dropped-id covers an intentional rename. JSON source islands stay podcast-only. Co-authored-by: Todd Dailey <twidtwid@users.noreply.github.com>
cursor
Bot
force-pushed
the
cursor/pr-update-lint-85a6
branch
from
October 4, 2026 22:19
077d58b to
1b73847
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
An update can drop
id="search"or the copy-as-prompt control and the current lint still passes. Class 9 needs a prior-file comparison.What changed
lint-artifact.mjs --against <prior.html>fails dropped ids, a dropped copy-as-prompt control, or a deleted inline script.--allow-dropped-idcovers an intentional rename.Scope
This PR only adds the update comparison. It does not require
data-shapeand does not addapplication/jsonsource islands to any shape. Podcast keeps the island it already has.Tradeoffs
JSON islands stay out of this stack. The spikes never measured a size or safety win that justified expanding the publish sensitivity scan. Class 9 is handled by
--against. Direction B inplans/README.mdstays a later experiment.The script-drop check ignores
application/jsonandld+jsonislands so the podcast package is not the only thing keeping an update green.Blast Radius
Default lint is unchanged. Skill tokens rose 12241 → 12256.
--againston the dashboard pair stays well under 2 seconds.Verification
id="search"orid="donut-svg"names that id.make checkis green.