Commit Graph

11 Commits

Author SHA1 Message Date
David Cramer 9c0e34277f fix(skills): Standardize allowed-tools format and remove model attribute (#43)
* fix(skills): Standardize allowed-tools to comma-separated format

Claude Code docs specify comma-separated format for allowed-tools.
Update all skill frontmatters, documentation, and the scan_skill.py
parser to use and support comma-separated tool lists.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* fix(skills): Remove model attribute from skill documentation

Skills should not hardcode model overrides — let users control
which model runs their skills.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
2026-02-10 14:57:28 -08:00
David Cramer d4d8757434 fix(iterate-pr): Handle workflow field as string from gh CLI (#35)
The gh pr checks --json workflow field returns a string, not an object.
The previous code assumed it was a dict with a "name" key, causing an
AttributeError when accessing .get("name") on a string.

Also update SKILL.md to use uv run instead of python directly, and add
skill design guideline to AGENTS.md for consistent script invocation.

Co-authored-by: Claude <noreply@anthropic.com>
2026-02-03 10:05:57 -08:00
David Cramer 70e209a3ba fix(security-review): Move frontmatter to start of file
The skills CLI parser expects YAML frontmatter to be at the very
beginning of the file. The HTML attribution comment was preventing
the skill from being detected.

Also updated docs to clarify frontmatter positioning requirements.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2026-01-29 12:58:18 -08:00
David Cramer 6ef251d038 feat(skills): Add prerequisite checks to commit and create-pr skills
Add workflow prerequisites to ensure proper git hygiene:
- commit skill: Check if on main/master and prompt to create feature branch
- create-pr skill: Check for uncommitted changes and invoke commit skill first
- AGENTS.md: Add step to include new skills in .claude/settings.json allowlist

These changes help enforce the expected workflow where changes are made on
feature branches and properly committed before creating PRs.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2026-01-06 10:59:23 -08:00
David Cramer 79adde273c fix(claude-settings-audit): Add skills allowlist for Sentry projects
- Add Skill() permissions for all sentry-skills to the recommendation
- Update AGENTS.md to remind contributors to update the skills list
  when adding new skills

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2026-01-06 10:19:18 -08:00
David Cramer 762c8a3a42 Update README and AGENTS.md documentation
- Clarify repo is primarily for Claude Code but follows open Agent Skills format
- Add instruction to update README when creating new skills
- Remove duplicated skills list from AGENTS.md
- Add missing deslop skill to README

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
2026-01-05 16:16:08 -08:00
David Cramer 6358fc4701 Fix information on updates 2026-01-05 16:01:20 -08:00
David Cramer b8cab86eac add find-bugs and update docs 2026-01-05 15:55:29 -08:00
David Cramer 622bafc03b correct source path 2026-01-05 14:46:49 -08:00
David Cramer 5ae6cbb122 reorganize 2026-01-05 14:44:41 -08:00
David Cramer da8f1cd882 first commit 2026-01-05 14:43:16 -08:00