Skip to content

feat(contracts): compile severity tables from the pattern registry - #300

Merged
edithatogo merged 2 commits into
mainfrom
pattern-registry-contracts-pr3
Aug 11, 2026
Merged

feat(contracts): compile severity tables from the pattern registry#300
edithatogo merged 2 commits into
mainfrom
pattern-registry-contracts-pr3

Conversation

@edithatogo

Copy link
Copy Markdown
Owner

Summary

  • compile-skill.js now emits ## SEVERITY CLASSIFICATION from src/document-intelligence/patterns.json.
  • Pattern bodies stay in src/modules/SKILL_CORE_PATTERNS.md. Generated SKILL.md and references/core-patterns.md tables use registry titles.
  • Diff in the compiler is small: load the registry, render the four tiers, replace the copied module section.

Test plan

  • node --test test/skill-contracts.test.js test/patterns.test.js test/sync-determinism.test.js
  • node scripts/compile-skill.js
  • CI test, validate-skill, and portable-spec green

Refs #278

Keep pattern bodies in Markdown and emit the compiled severity lists from patterns.json so table IDs cannot drift by hand.

Refs #278

Co-authored-by: Cursor <cursoragent@cursor.com>
Copilot AI lite review requested due to automatic review settings August 11, 2026 22:27
@cursor

cursor Bot commented Aug 11, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Use the existing table labels from the registry so npm run sync keeps the prose maintainers already review.

Co-authored-by: Cursor <cursoragent@cursor.com>
@codecov

codecov Bot commented Aug 11, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@edithatogo
edithatogo merged commit af66236 into main Aug 11, 2026
10 checks passed
@edithatogo
edithatogo deleted the pattern-registry-contracts-pr3 branch August 11, 2026 22:41
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.

3 participants