Skip to content

Adopt Claude Code agent conventions#40

Merged
GordonBeeming merged 2 commits into
mainfrom
gb/claude-agent-conventions
Jul 1, 2026
Merged

Adopt Claude Code agent conventions#40
GordonBeeming merged 2 commits into
mainfrom
gb/claude-agent-conventions

Conversation

@GordonBeeming

Copy link
Copy Markdown
Owner

Summary

  • Renamed the Codex-authored agent config to Claude Code equivalents: AGENTS.mdCLAUDE.md, .agents/.claude/, and deleted the Codex-only openai.yaml manifest (SKILL.md frontmatter already carries name/description/invocation).
  • Switched skill invocation to slash syntax (/new-feature, /feature-flags, /create-release) and pointed the new-feature artifact instructions at CLAUDE.md.
  • Rewrote the create-release, feature-flags, and new-feature skills against the Opus 4.8 prompting guidance — literal-scope wording, positive framing, and the reasoning behind each non-obvious rule — preserving every command, path, and fact.

Product-level Codex references in README.md and docs/ are genuine MCP integration features (the read-only endpoint Codex consumes, codex app-server research) and stay untouched.

Test plan

  • find .claude/skills -type f shows the three SKILL.md files, no agents//openai.yaml.
  • test -f CLAUDE.md && ! test -e AGENTS.md && ! test -e .agents passes.
  • No leftover Codex-isms: grep -rnE '\$new-feature|\$feature-flags|AGENTS\.md|openai|\.agents' CLAUDE.md .claude/skills returns nothing.
  • README.md/docs/ unchanged in the diff.

Rename the Codex-authored agent config to its Claude Code equivalents and
refresh the skills against the Opus 4.8 prompting guidance.

- Rename AGENTS.md to CLAUDE.md and .agents/ to .claude/.
- Delete the Codex-only openai.yaml manifest; SKILL.md frontmatter already
  carries name, description, and invocation.
- Switch skill invocation to slash syntax (/new-feature, /feature-flags) and
  point the new-feature artifact instructions at CLAUDE.md.
- Rewrite the create-release, feature-flags, and new-feature skills for
  literal-scope wording, positive framing, and the reasoning behind each
  non-obvious rule, preserving every command and fact.

Product-level Codex references in README.md and docs/ are real MCP
integration features and stay untouched.

Co-authored-by: Claude <noreply@anthropic.com>
Co-authored-by: GitButler <gitbutler@gitbutler.com>
@GordonBeeming GordonBeeming marked this pull request as ready for review July 1, 2026 13:54
Copilot AI review requested due to automatic review settings July 1, 2026 13:54

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request reorganizes and updates the project's agent skills and guidance files for Claude Code, moving the release skill to .claude/skills/create-release/SKILL.md, changing skill triggers from $ to /, and introducing a central CLAUDE.md file. Feedback on these changes identifies two issues in the new release skill file: a nested markdown code block that breaks rendering, and an indented here-document (EOF) that causes a bash syntax error and formatting issues.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread .claude/skills/create-release/SKILL.md Outdated
Comment thread .claude/skills/create-release/SKILL.md Outdated

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR migrates the repository’s agent/skill configuration and conventions from the prior Codex-style layout to Claude Code conventions, updating skill invocation syntax and relocating the skill definitions accordingly.

Changes:

  • Introduces CLAUDE.md project guidance and removes the legacy AGENTS.md.
  • Moves skills from .agents/ to .claude/ and removes the Codex-only openai.yaml manifest.
  • Updates skill docs to use slash invocation syntax (e.g. /new-feature, /feature-flags) and aligns artifact guidance to reference CLAUDE.md.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
CLAUDE.md Adds Claude Code project guidance and skill invocation conventions.
AGENTS.md Removes legacy agent guidance file superseded by CLAUDE.md.
.claude/skills/new-feature/SKILL.md Updates invocation wording and artifact instruction references to CLAUDE.md.
.claude/skills/feature-flags/SKILL.md Updates invocation wording and refines guidance language.
.claude/skills/create-release/SKILL.md Adds the Claude Code release skill documentation (needs a Markdown fence fix per comment).
.agents/skills/new-feature/agents/openai.yaml Removes Codex-only manifest.
.agents/skills/create-release/SKILL.md Removes legacy skill location in favor of .claude/ equivalent.

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

Comment thread .claude/skills/create-release/SKILL.md Outdated
Address reviewer feedback (gemini-code-assist, copilot):
- Use a four-backtick outer fence so the nested brew code block renders.
- Dedent the release-notes heredoc to column 0 so the <<'EOF' terminates
  correctly and the notes render without stray indentation.

Co-authored-by: Claude <noreply@anthropic.com>
Co-authored-by: GitButler <gitbutler@gitbutler.com>
@GordonBeeming GordonBeeming merged commit fcecae3 into main Jul 1, 2026
2 of 4 checks passed
@GordonBeeming GordonBeeming deleted the gb/claude-agent-conventions branch July 1, 2026 14:11
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