Skip to content

feat(claude_skill): add 7 skill text-match quality rules (batch 2)#38

Draft
ivanpaghubasan wants to merge 1 commit into
trustabl:mainfrom
ivanpaghubasan:feat/skill-text-match-rules-batch2
Draft

feat(claude_skill): add 7 skill text-match quality rules (batch 2)#38
ivanpaghubasan wants to merge 1 commit into
trustabl:mainfrom
ivanpaghubasan:feat/skill-text-match-rules-batch2

Conversation

@ivanpaghubasan

@ivanpaghubasan ivanpaghubasan commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

What this does

Adds 7 new skill-scope quality rules using the new text-match
predicate family added in the engine PR:
feat/skill-text-match-predicates

Rules added

  • CSKILL-080: Skill Claims Cryptographic Operations
  • CSKILL-081: Skill Processes Sensitive Data
  • CSKILL-082: Over-Privileged Security Skill
  • CSKILL-083: Missing Error Handling Guidance
  • CSKILL-084: Data Minimization Opportunity
  • CSKILL-085: Lacks Data Purpose Declaration
  • CSKILL-086: Data Retention or Logging Implied

Deferred

  • SKILL-034 (Inconsistent Tool Naming) — requires cross-scope
    predicate (Bucket 3), deferred to next batch

Changes

  • claude_skill/skill_quality_text.yaml — 7 new rules
  • manifest.yaml — schema_version bumped 13 → 14

⚠️ DRAFT — Do Not Merge Yet

Depends on engine PR feat/skill-text-match-predicates
merging first. Will undraft once engine predicates land in main.

Related

  • Engine PR: feat/skill-text-match-predicates

…to CSKILL-074)

Adds 7 new skill-scope detection rules using the new text-match
predicate family (skill_body_has_text, skill_name_has_text,
skill_description_has_text). SKILL-034 deferred — requires
cross-scope predicate not yet in engine.

Bumps manifest.yaml schema_version 13 → 14 to match engine.
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.

1 participant