Skip to content

feat(lint-artifact): require html data-shape from the registry - #12

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

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

Conversation

@twidtwid

@twidtwid twidtwid commented Oct 4, 2026

Copy link
Copy Markdown
Owner

Why

Wrong shape still passes today's lint. The omega-3 draft was an editorial page on tabular data. A declared data-shape on <html> lets the lint name the missing feature.

What changed

  • Generated examples set <html data-shape> to a contracts/shapes.json id.
  • lint-artifact.mjs reads that attribute on the html element only, never on gallery cards.
  • Editorial requires thesis, so a dashboard tagged editorial fails.

Scope

This PR only adds the declared-shape gate. It does not add --against and does not thin SKILL.md.

Blast Radius

Every generated example gained one attribute. Interactive behavior did not change. Skill tokens rose 12241 → 12257.

Verification

  • make check is green.
  • A dashboard copy with data-shape="editorial" fails and names thesis.
  • data-shape="not-a-shape" names contracts/shapes.json.
Open in Web Open in Cursor 

@cursor
cursor Bot force-pushed the cursor/pr-registry-85a6 branch from a3cd5ce to 88d7348 Compare October 4, 2026 21:08
@cursor
cursor Bot force-pushed the cursor/pr-shape-lint-85a6 branch from c826ca6 to 6a1b47b Compare October 4, 2026 21:09
@cursor
cursor Bot force-pushed the cursor/pr-registry-85a6 branch from 88d7348 to a8eb3c4 Compare October 4, 2026 22:17
@cursor
cursor Bot force-pushed the cursor/pr-shape-lint-85a6 branch from 6a1b47b to d752814 Compare October 4, 2026 22:19
@cursor
cursor Bot changed the base branch from cursor/pr-registry-85a6 to main October 4, 2026 22:19
The lint reads data-shape on the html element only. A value that is
not a contracts/shapes.json id fails. Editorial requires thesis so a
dashboard tagged editorial still fails.

Co-authored-by: Todd Dailey <twidtwid@users.noreply.github.com>
@cursor
cursor Bot force-pushed the cursor/pr-shape-lint-85a6 branch from d752814 to a7fc509 Compare October 4, 2026 22:20
@cursor
cursor Bot merged commit 5f957c3 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