Commit Graph

16 Commits

Author SHA1 Message Date
David Cramer 360070a6ed docs(skills): Align attribution placeholder with agent identity (#85)
Align commit attribution wording to require the agent's name and
attribution byline everywhere this guidance appears.

This updates AGENTS.md, the agents-md skill, and the commit skill so
they consistently use the same placeholder format and avoid the older
'agent model' phrasing.

It also strengthens the commit skill with explicit guidance to avoid
literal \n sequences in commit messages and to use properly formatted
multiline commit bodies.

Co-authored-by: GPT-5 Codex <noreply@openai.com>
2026-03-06 10:09:56 -08:00
David Cramer 3b816eae84 feat(skills): Merge synthesis workflow into skill-writer (#84)
Merge skill synthesis into `skill-writer` so there is one canonical flow
for creating and improving skills.

**Canonical workflow consolidation**
`skill-writer` now owns synthesis, authoring, iteration, evaluation, and
registration in one path. This removes split ownership between creation
and synthesis and makes outcomes more consistent.

**`skill-creator` compatibility behavior**
`skill-creator` is now an alias entrypoint rather than a separate
framework. Existing prompts still work, but execution routes to the
canonical `skill-writer` behavior.

**Input-depth and coverage gates**
The workflow now explicitly prioritizes broad source collection before
authoring. It adds coverage/depth gates to reduce shallow skills and
force clearer gap handling.

**Provenance and iteration tracking**
Skill synthesis now records sources and change history in `SOURCES.md`
instead of adding heavy attribution blocks in `SKILL.md`. This improves
traceability and makes iterative updates easier to audit.

**Case-study guidance quality**
Reference examples were upgraded from generic guidance to case-study
style patterns with concrete source-to-decision framing. This is
intended to improve synthesis quality for future skills.

**Cross-agent execution consistency**
Guidance was normalized to be generic across Claude and Codex, with
script execution clarified around `uv run` and skill-directory script
paths. This keeps the workflow portable while staying operationally
concrete.

---------

Co-authored-by: GPT-5 Codex <noreply@openai.com>
2026-03-05 14:26:17 -08:00
David Cramer fe1dd4f1d4 docs(skills): List all skills and align registration/allowlists (#80)
* docs: list all available skills

* docs(skills): Align registration guidance and skill allowlists

Require alphabetical ordering when updating the Available Skills table\nin contributor guidance and skill creation workflow docs.\n\nSynchronize skill allowlists with the actual sentry-skills directory so\nautomatic skill injection guidance reflects all available skills.\n\nCo-Authored-By: GPT-5 Codex <noreply@openai.com>

---------

Co-authored-by: shuofengzhang <sofuncheung16@gmail.com>
2026-03-04 10:44:54 -08:00
David Cramer 942b2c1421 ref(skills): Promote pr-writer and skill-writer as canonical names (#79)
* ref(skills): Rename PR and skill authoring workflows

Promote pr-writer and skill-writer as canonical skill names while\nkeeping create-pr and skill-creator as compatibility aliases.\n\nUpdate skill registration, settings allowlists, and documentation\nreferences to prefer canonical names.\n\nExtend skill-writer guidance with capability-first naming practices\nand examples for clearer skill discovery.

Co-Authored-By: GPT-5 Codex <noreply@openai.com>

* fix(skills): Address review bot feedback on skill rename

Restrict skill-writer with an explicit allowed-tools frontmatter field,\nremove dead code in quick_validate, and delete orphaned duplicate\nreferences/scripts from the skill-creator alias directory to avoid drift.

Co-Authored-By: GPT-5 Codex <noreply@openai.com>

* fix(skills): Drop allowed-tools from skill-writer

Keep skill-writer frontmatter compatible across clients by avoiding\nthe optional allowed-tools field while retaining the other review\nfixes in this PR.

Co-Authored-By: GPT-5 Codex <noreply@openai.com>

* docs(skill-writer): Default to omitting allowed-tools

Clarify that skill authoring should avoid allowed-tools unless a user\nexplicitly requests strict tool scoping or repo policy requires it,\nsince allowlist syntax differs across clients.

Co-Authored-By: GPT-5 Codex <noreply@openai.com>

* docs(pr-writer): Include update flows in skill discovery text

Expand pr-writer trigger language to explicitly cover updating PR artifacts\nlike title and description, not only creating a new PR.\n\nAlso align the README skill table description so repository docs\nmatch the canonical skill wording.

Co-Authored-By: GPT-5 Codex <noreply@openai.com>

---------

Co-authored-by: GPT-5 Codex <noreply@openai.com>
2026-03-04 10:03:08 -08:00
David Cramer ef33bb5032 ref(agents-md): Improve skill with research-backed best practices (#77)
Replace skill-discovery focus with project-analysis approach. Add line
budget guidance (target 60, max 100), file-scoped commands section,
and anti-patterns for linter duplication. Remove Local Skills required
section since agents discover skills automatically. Rewrite AGENTS.md
for this repo to follow the updated guidance.

Co-authored-by: Claude <noreply@anthropic.com>
2026-03-03 15:04:21 -08:00
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