From 8818f1b7ea334eba862faa0e4b85dfb4703b1697 Mon Sep 17 00:00:00 2001 From: junior Date: Tue, 7 Apr 2026 16:51:04 +0000 Subject: [PATCH 1/4] remove create-pr skill --- plugins/sentry-skills/skills/create-pr/SKILL.md | 12 ------------ 1 file changed, 12 deletions(-) delete mode 100644 plugins/sentry-skills/skills/create-pr/SKILL.md diff --git a/plugins/sentry-skills/skills/create-pr/SKILL.md b/plugins/sentry-skills/skills/create-pr/SKILL.md deleted file mode 100644 index 0b43b01..0000000 --- a/plugins/sentry-skills/skills/create-pr/SKILL.md +++ /dev/null @@ -1,12 +0,0 @@ ---- -name: create-pr -description: Alias for sentry-skills:pr-writer. Use when users explicitly ask for "create-pr" or reference the legacy skill name. Redirects to the canonical PR writing workflow. ---- - -# Alias: create-pr - -This skill name is kept for compatibility. - -Use `sentry-skills:pr-writer` as the canonical skill for creating and editing pull requests. - -If invoked via `create-pr`, run the same workflow and conventions documented in `sentry-skills:pr-writer`. From 403fcc13119d4197f0a9c2ecd9849046c89b354e Mon Sep 17 00:00:00 2001 From: junior Date: Tue, 7 Apr 2026 16:54:30 +0000 Subject: [PATCH 2/4] ref: remove all remaining create-pr references - .claude/settings.json: drop Skill(sentry-skills:create-pr) from allow list - AGENTS.md: remove create-pr from alias skill example - claude-settings-audit/SKILL.md: drop create-pr from expected allow list --- .claude/settings.json | 6 +----- AGENTS.md | 2 +- plugins/sentry-skills/skills/claude-settings-audit/SKILL.md | 1 - 3 files changed, 2 insertions(+), 7 deletions(-) diff --git a/.claude/settings.json b/.claude/settings.json index 014b51c..d209cff 100644 --- a/.claude/settings.json +++ b/.claude/settings.json @@ -8,7 +8,6 @@ "WebFetch(domain:docs.github.com)", "WebFetch(domain:cli.github.com)", "WebFetch(domain:agentskills.io)", - "Bash(ls:*)", "Bash(pwd:*)", "Bash(find:*)", @@ -19,7 +18,6 @@ "Bash(tail:*)", "Bash(cat:*)", "Bash(tree:*)", - "Bash(git status:*)", "Bash(git log:*)", "Bash(git diff:*)", @@ -29,7 +27,6 @@ "Bash(git tag:*)", "Bash(git stash list:*)", "Bash(git rev-parse:*)", - "Bash(gh pr view:*)", "Bash(gh pr list:*)", "Bash(gh pr checks:*)", @@ -41,7 +38,6 @@ "Bash(gh run logs:*)", "Bash(gh repo view:*)", "Bash(gh api:*)", - "Skill(sentry-skills:agents-md)", "Skill(sentry-skills:blog-writing-guide)", "Skill(sentry-skills:brand-guidelines)", @@ -50,7 +46,6 @@ "Skill(sentry-skills:code-simplifier)", "Skill(sentry-skills:commit)", "Skill(sentry-skills:create-branch)", - "Skill(sentry-skills:create-pr)", "Skill(sentry-skills:django-access-review)", "Skill(sentry-skills:django-perf-review)", "Skill(sentry-skills:doc-coauthoring)", @@ -73,3 +68,4 @@ "plugin-dev@claude-plugins-official": true } } + diff --git a/AGENTS.md b/AGENTS.md index e4cb273..d6c9bdb 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -9,7 +9,7 @@ plugins/sentry-skills/skills//SKILL.md ALWAYS use `/skill-writer` — it handles requirements, writing, registration, and validation. (`/skill-creator` remains an alias.) ### Alias Policy -- Keep alias skills (for example `create-pr`, `skill-creator`) only for backward compatibility. +- Keep alias skills (for example `skill-creator`) only for backward compatibility. - Do **not** list alias/symlink skills in "Available Skills" documentation tables. - List only canonical skills in public skill inventories (for example `pr-writer`, `skill-writer`). diff --git a/plugins/sentry-skills/skills/claude-settings-audit/SKILL.md b/plugins/sentry-skills/skills/claude-settings-audit/SKILL.md index 9bc84b0..3719b66 100644 --- a/plugins/sentry-skills/skills/claude-settings-audit/SKILL.md +++ b/plugins/sentry-skills/skills/claude-settings-audit/SKILL.md @@ -150,7 +150,6 @@ If this is a Sentry project (or sentry-skills plugin is installed), include: "Skill(sentry-skills:code-simplifier)", "Skill(sentry-skills:commit)", "Skill(sentry-skills:create-branch)", - "Skill(sentry-skills:create-pr)", "Skill(sentry-skills:django-access-review)", "Skill(sentry-skills:django-perf-review)", "Skill(sentry-skills:doc-coauthoring)", From 7d000af98c2ec3e1e626e05090c15655aca5cc90 Mon Sep 17 00:00:00 2001 From: junior Date: Tue, 7 Apr 2026 16:56:29 +0000 Subject: [PATCH 3/4] remove skill-creator alias skill and all references MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit skill-creator was a compatibility alias for skill-writer. removing it entirely as part of a clean break from legacy alias skills. no aliases, no compat shims — just the canonical skill names. removes: - plugins/sentry-skills/skills/skill-creator/SKILL.md - .claude/settings.json allow entry - AGENTS.md alias mention - claude-settings-audit/SKILL.md expected entry - skill-writer/SOURCES.md source table row - skill-writer/references/synthesis-path.md references --- .claude/settings.json | 2 -- AGENTS.md | 3 +-- .../skills/claude-settings-audit/SKILL.md | 1 - plugins/sentry-skills/skills/skill-creator/SKILL.md | 12 ------------ plugins/sentry-skills/skills/skill-writer/SOURCES.md | 4 ---- .../skills/skill-writer/references/synthesis-path.md | 3 --- 6 files changed, 1 insertion(+), 24 deletions(-) delete mode 100644 plugins/sentry-skills/skills/skill-creator/SKILL.md diff --git a/.claude/settings.json b/.claude/settings.json index d209cff..64cef4d 100644 --- a/.claude/settings.json +++ b/.claude/settings.json @@ -55,7 +55,6 @@ "Skill(sentry-skills:iterate-pr)", "Skill(sentry-skills:pr-writer)", "Skill(sentry-skills:security-review)", - "Skill(sentry-skills:skill-creator)", "Skill(sentry-skills:skill-scanner)", "Skill(sentry-skills:skill-writer)", "Skill(sentry-skills:sred-project-organizer)", @@ -68,4 +67,3 @@ "plugin-dev@claude-plugins-official": true } } - diff --git a/AGENTS.md b/AGENTS.md index d6c9bdb..bf740ae 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -6,10 +6,9 @@ plugins/sentry-skills/skills//SKILL.md ``` ## Creating/Updating Skills -ALWAYS use `/skill-writer` — it handles requirements, writing, registration, and validation. (`/skill-creator` remains an alias.) +ALWAYS use `/skill-writer` — it handles requirements, writing, registration, and validation. ### Alias Policy -- Keep alias skills (for example `skill-creator`) only for backward compatibility. - Do **not** list alias/symlink skills in "Available Skills" documentation tables. - List only canonical skills in public skill inventories (for example `pr-writer`, `skill-writer`). diff --git a/plugins/sentry-skills/skills/claude-settings-audit/SKILL.md b/plugins/sentry-skills/skills/claude-settings-audit/SKILL.md index 3719b66..0670813 100644 --- a/plugins/sentry-skills/skills/claude-settings-audit/SKILL.md +++ b/plugins/sentry-skills/skills/claude-settings-audit/SKILL.md @@ -159,7 +159,6 @@ If this is a Sentry project (or sentry-skills plugin is installed), include: "Skill(sentry-skills:iterate-pr)", "Skill(sentry-skills:pr-writer)", "Skill(sentry-skills:security-review)", - "Skill(sentry-skills:skill-creator)", "Skill(sentry-skills:skill-scanner)", "Skill(sentry-skills:skill-writer)", "Skill(sentry-skills:sred-project-organizer)", diff --git a/plugins/sentry-skills/skills/skill-creator/SKILL.md b/plugins/sentry-skills/skills/skill-creator/SKILL.md deleted file mode 100644 index b060cd7..0000000 --- a/plugins/sentry-skills/skills/skill-creator/SKILL.md +++ /dev/null @@ -1,12 +0,0 @@ ---- -name: skill-creator -description: Alias for sentry-skills:skill-writer. Use when users explicitly ask for "skill-creator" or reference the legacy skill name. Redirects to the canonical skill authoring workflow. ---- - -# Alias: skill-creator - -This skill name is kept for compatibility. - -Use `sentry-skills:skill-writer` as the canonical skill for creating and updating skills. - -If invoked via `skill-creator`, run the same workflow and conventions documented in `sentry-skills:skill-writer`. diff --git a/plugins/sentry-skills/skills/skill-writer/SOURCES.md b/plugins/sentry-skills/skills/skill-writer/SOURCES.md index dbfc6c5..eaaa300 100644 --- a/plugins/sentry-skills/skills/skill-writer/SOURCES.md +++ b/plugins/sentry-skills/skills/skill-writer/SOURCES.md @@ -8,16 +8,12 @@ This file tracks source material synthesized into `skill-writer`, plus iterative |---|---|---|---|---|---|---|---| | `plugins/sentry-skills/skills/skill-writer/SKILL.md` | local canonical | canonical | 2026-03-05 | high | Baseline orchestration, path model, quality gates | local repository authority | Primary source of current behavior | | `plugins/sentry-skills/skills/skill-writer/references/*.md` | local canonical | canonical | 2026-03-05 | high | Detailed path guidance, examples, validation requirements | local repository authority | Includes synthesis/iteration/evaluation paths | -| `plugins/sentry-skills/skills/skill-creator/SKILL.md` | local compatibility alias | canonical | 2026-03-05 | high | Backward-compatible invocation behavior | local repository authority | Confirms alias delegates to `skill-writer` | -| `.codex/skills/.system/skill-creator/SKILL.md` | local Codex upstream | secondary | 2026-03-05 | medium | Codex-native skill authoring conventions and structure | environment-local snapshot; may diverge from upstream | Used to align cross-agent portability expectations | -| `https://github.com/anthropics/skills/tree/main/skills/skill-creator` | external Claude upstream | canonical | 2026-03-05 | medium | Claude-native prior art and updated guidance | verify periodically against upstream changes | Track for periodic refresh as upstream evolves | | `https://agentskills.io/specification` | external canonical spec | canonical | 2026-03-05 | high | Portable skill spec requirements | spec-level constraints take precedence over local preferences | Cross-agent compatibility baseline | | `AGENTS.md` | repo convention | canonical | 2026-03-05 | high | Repository-specific workflow requirements | repository-local policy | Registration + validator expectations | | `README.md` | repo convention | canonical | 2026-03-05 | high | Skill table format and authoring conventions | repository-local policy | Registration and discoverability source | ## Decisions -1. `skill-writer` is the single canonical workflow; `skill-creator` remains alias-only for compatibility. 2. Source breadth is the primary quality lever; synthesis cannot stop early on limited samples. 3. Provenance is stored in `SOURCES.md`, not SKILL header comments. 4. Case-study style examples are required for deeper, reusable synthesis outcomes. diff --git a/plugins/sentry-skills/skills/skill-writer/references/synthesis-path.md b/plugins/sentry-skills/skills/skill-writer/references/synthesis-path.md index 37c0ed9..630538f 100644 --- a/plugins/sentry-skills/skills/skill-writer/references/synthesis-path.md +++ b/plugins/sentry-skills/skills/skill-writer/references/synthesis-path.md @@ -34,9 +34,6 @@ Keep collecting until retrieval passes no longer add meaningful new guidance. When synthesizing a skill that creates, updates, or evaluates other skills, include at minimum: 1. Local canonical workflow source (`plugins/sentry-skills/skills/skill-writer/...`). -2. Local compatibility alias/source (`plugins/sentry-skills/skills/skill-creator/SKILL.md`). -3. Codex system skill-authoring source (for example `.codex/skills/.system/skill-creator/SKILL.md` when available). -4. Anthropic/Claude upstream skill-authoring source (for example `anthropics/skills/.../skill-creator` or the published GitHub path). 5. Agent Skills specification and repository conventions. Record all baseline sources in `SOURCES.md` with retrieval date and contribution notes. From a1202bc8357a5aab4bc0dd3c930bffa3e11a4943 Mon Sep 17 00:00:00 2001 From: junior Date: Tue, 7 Apr 2026 17:01:48 +0000 Subject: [PATCH 4/4] fix(AGENTS.md): clean up alias policy section ordering and remove stale alias guidance Co-Authored-By: junior --- AGENTS.md | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/AGENTS.md b/AGENTS.md index bf740ae..d0ea613 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -8,9 +8,8 @@ plugins/sentry-skills/skills//SKILL.md ## Creating/Updating Skills ALWAYS use `/skill-writer` — it handles requirements, writing, registration, and validation. -### Alias Policy -- Do **not** list alias/symlink skills in "Available Skills" documentation tables. -- List only canonical skills in public skill inventories (for example `pr-writer`, `skill-writer`). +- Do **not** create alias or symlink skills +- List only canonical skills in public skill inventories (for example `pr-writer`, `skill-writer`) ### Registration Checklist 1. Create `plugins/sentry-skills/skills//SKILL.md`