Skip to content

feat(claude_skill): add CSKILL-087 placeholder description rule - #116

Open
kureen-cyber wants to merge 1 commit into
trustabl:mainfrom
kureen-cyber:feat/cskill-087-placeholder-description
Open

feat(claude_skill): add CSKILL-087 placeholder description rule#116
kureen-cyber wants to merge 1 commit into
trustabl:mainfrom
kureen-cyber:feat/cskill-087-placeholder-description

Conversation

@kureen-cyber

@kureen-cyber kureen-cyber commented Aug 24, 2026

Copy link
Copy Markdown

Summary

  • Adds CSKILL-087 (low): skill description: is a placeholder stub (TODO / TBD / FIXME / placeholder / no description / does stuff).
  • Complements CSKILL-070, which only flags an empty description. Claude Code still loads a stub into context and uses it as the model-invocation signal, so "TODO: describe this skill" is functionally the same as no description.
  • Reuses the existing skill_description_has_text predicate and the same needle list as CSDK-017. No schema_version bump.

Companion PRs (same branch name feat/cskill-087-placeholder-description):

Test plan

  • trustabl rules validate . on this pack
  • Engine TestPolicyRules_AllRulesCovered includes CSKILL-087 fire + silent
  • Fire: description: "TODO: describe this skill."
  • Silent: a real description with no placeholder needles

CSKILL-070 only flags an empty description. A stub like TODO still
gives Claude no model-invocation signal.
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