Skip to content

feat: add Anthropic best practice checks (STR-018, STR-019, STR-020)#1

Merged
melanie531 merged 1 commit intomainfrom
feat/best-practice-checks
Mar 17, 2026
Merged

feat: add Anthropic best practice checks (STR-018, STR-019, STR-020)#1
melanie531 merged 1 commit intomainfrom
feat/best-practice-checks

Conversation

@melanie531
Copy link
Contributor

Add three new structure checks based on agentskills.io spec and Anthropic skill authoring best practices:

  • STR-018 (WARNING): Name contains reserved words ('anthropic', 'claude') Per agentskills.io spec, names must not contain these reserved words.

  • STR-019 (WARNING): Description contains XML tags Per agentskills.io spec, description must not contain XML/HTML tags as they can interfere with system prompt injection.

  • STR-020 (INFO): Description should use third person Per Anthropic best practices, descriptions should use third person ('Processes files') instead of first person ('I can help') or second person ('You can use'). Inconsistent point-of-view causes discovery problems.

All three are pure static checks (no LLM needed).
Includes 10 new test cases covering positive and negative scenarios.

Issue #, if available:

Description of changes:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Add three new structure checks based on agentskills.io spec and
Anthropic skill authoring best practices:

- STR-018 (WARNING): Name contains reserved words ('anthropic', 'claude')
  Per agentskills.io spec, names must not contain these reserved words.

- STR-019 (WARNING): Description contains XML tags
  Per agentskills.io spec, description must not contain XML/HTML tags
  as they can interfere with system prompt injection.

- STR-020 (INFO): Description should use third person
  Per Anthropic best practices, descriptions should use third person
  ('Processes files') instead of first person ('I can help') or
  second person ('You can use'). Inconsistent point-of-view causes
  discovery problems.

All three are pure static checks (no LLM needed).
Includes 10 new test cases covering positive and negative scenarios.
@melanie531 melanie531 merged commit ddb15c6 into main Mar 17, 2026
4 checks passed
@melanie531 melanie531 deleted the feat/best-practice-checks branch March 17, 2026 12:35
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