Skip to content

Move prompts and skills under .claude and adapt to Claude Code conventions - #1207

Merged
dgee2 merged 4 commits into
mainfrom
claude/move-prompts-skills-claude-c77afc
Aug 4, 2026
Merged

Move prompts and skills under .claude and adapt to Claude Code conventions#1207
dgee2 merged 4 commits into
mainfrom
claude/move-prompts-skills-claude-c77afc

Conversation

@dgee2

@dgee2 dgee2 commented Aug 4, 2026

Copy link
Copy Markdown
Owner

Summary

Relocates the agent assets out of .github/ and into .claude/, then adapts them from GitHub Copilot conventions to Claude Code ones.

  • .github/skills/.claude/skills/ (10 skills, git-tracked as renames)
  • .github/prompts/issues-diagram.prompt.md.claude/commands/issues-diagram.md

Claude Code adaptations

  • Frontmatter — removed the Copilot-only user-invokable and context: inline fields from all ten SKILL.md files. Claude Code skills use name + description only.
  • Prompt → slash command — Claude Code discovers slash commands from .claude/commands/, not .claude/prompts/, so the prompt landed there as issues-diagram.md (making it runnable as /issues-diagram). Dropped the unused name: field, kept description:/argument-hint:, and switched the output-path override to $ARGUMENTS.
  • pr-comment-identity — the skill mandated a > 🤖 **This comment was written by GitHub Copilot.** attribution block on PR review comments. Now attributes to Claude Code.

References checked

Nothing in the repo referenced the old paths — no matches in workflows, CI path filters (docs/ci-path-filters.md), docs, or the skills themselves. .github/ now contains only workflows/. Added a line to AGENTS.md recording where skills and commands live.

gh CLI usage in github-issues-status was left alone — that's the repo's actual issue tracker, not Copilot-specific syntax.

🤖 Generated with Claude Code

dgee2 and others added 2 commits August 4, 2026 08:29
Relocate .github/prompts and .github/skills to .claude/prompts and
.claude/skills so agent assets live alongside the other Claude Code
configuration.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
- Drop the Copilot-only `user-invokable` and `context: inline` frontmatter
  fields from all ten SKILL.md files; Claude Code skills only use
  `name` and `description`.
- Relocate the issues-diagram prompt to `.claude/commands/issues-diagram.md`
  so it is discoverable as `/issues-diagram`, drop the unused `name` field,
  and use `$ARGUMENTS` for the output-path override.
- Rewrite pr-comment-identity to attribute comments to Claude Code rather
  than GitHub Copilot.
- Note the `.claude/skills` and `.claude/commands` locations in AGENTS.md.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings August 4, 2026 07:36

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Relocates agent assets from .github/ to .claude/ and updates their frontmatter/content to align with Claude Code’s skill and slash-command discovery conventions.

Changes:

  • Updated AGENTS.md to document the new canonical locations for skills and commands.
  • Removed Copilot-specific frontmatter (user-invokable, context) from all moved skill definitions.
  • Converted the old issues diagram prompt into a Claude Code slash command under .claude/commands/ and updated argument handling ($ARGUMENTS).

Reviewed changes

Copilot reviewed 12 out of 12 changed files in this pull request and generated no comments.

Show a summary per file
File Description
AGENTS.md Documents .claude/skills/ and .claude/commands/ as the locations for agent assets.
.claude/skills/write-tests/SKILL.md Removes Copilot-only frontmatter fields; keeps Claude Code-compatible metadata.
.claude/skills/pr-comment-identity/SKILL.md Updates attribution wording to “Claude Code” and removes Copilot-only frontmatter fields.
.claude/skills/openapi-sync/SKILL.md Removes Copilot-only frontmatter fields; keeps Claude Code-compatible metadata.
.claude/skills/mermaid-html-diagram/SKILL.md Removes Copilot-only frontmatter fields; keeps Claude Code-compatible metadata.
.claude/skills/github-issues-status/SKILL.md Removes Copilot-only frontmatter fields; keeps Claude Code-compatible metadata.
.claude/skills/frontend-component/SKILL.md Removes Copilot-only frontmatter fields; keeps Claude Code-compatible metadata.
.claude/skills/ef-migration/SKILL.md Removes Copilot-only frontmatter fields; keeps Claude Code-compatible metadata.
.claude/skills/code-review-checklist/SKILL.md Removes Copilot-only frontmatter fields; keeps Claude Code-compatible metadata.
.claude/skills/add-value-object/SKILL.md Removes Copilot-only frontmatter fields; keeps Claude Code-compatible metadata.
.claude/skills/add-api-endpoint/SKILL.md Removes Copilot-only frontmatter fields; keeps Claude Code-compatible metadata.
.claude/commands/issues-diagram.md Defines /issues-diagram as a Claude Code command and switches output override to $ARGUMENTS.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

dgee2 and others added 2 commits August 4, 2026 08:40
Resolves the conflict caused by main replacing `pr-comment-identity` with
the multi-agent `agent-identity` skill:

- Took main's deletion of `pr-comment-identity` (the Claude-only rewrite on
  this branch is superseded by `agent-identity`, which covers Copilot,
  Claude, and Codex).
- Relocated the new `agent-identity` skill to `.claude/skills/` in line with
  the rest of this branch, and stripped its Copilot-only `user-invokable` /
  `context: inline` frontmatter.
- Updated the AGENTS.md pointer to `.claude/skills/agent-identity/SKILL.md`.

Co-authored-by: Claude Opus 5 <noreply@anthropic.com>
The skill now lives under `.claude/skills/`, which only Claude Code reads,
so the Copilot and Codex attribution lines and co-author trailers were dead
weight. Narrowed the skill to Claude's own attribution block and trailer.

AGENTS.md keeps its multi-agent identity section, since that file is read by
every agent working in this repository.

Co-authored-by: Claude Opus 5 <noreply@anthropic.com>
@dgee2
dgee2 enabled auto-merge (squash) August 4, 2026 07:43
@sonarqubecloud

sonarqubecloud Bot commented Aug 4, 2026

Copy link
Copy Markdown

@dgee2
dgee2 merged commit f002522 into main Aug 4, 2026
13 checks passed
@dgee2
dgee2 deleted the claude/move-prompts-skills-claude-c77afc branch August 4, 2026 07:45
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.

2 participants