Skip to content

feat(tooling): share agent skills across Codex and Claude - #3514

Merged
danceratopz merged 3 commits into
ethereum:forks/amsterdamfrom
spencer-tb:codex/share-claude-skills
Sep 3, 2026
Merged

feat(tooling): share agent skills across Codex and Claude#3514
danceratopz merged 3 commits into
ethereum:forks/amsterdamfrom
spencer-tb:codex/share-claude-skills

Conversation

@spencer-tb

@spencer-tb spencer-tb commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Description

  • Make AGENTS.md the canonical repository guidance and replace CLAUDE.md with a compatibility symlink, following the pattern used by reth.
  • Move every existing .claude/commands/ workflow into a repository-scoped Codex skill under .agents/skills/, with the required skill metadata.
  • Expose the same canonical skill folders to Claude through symlinks under .claude/skills/.
  • Extend /audit-config to check that the canonical skills and Claude symlinks remain in sync.
  • Make review tasks strictly read-only: agents return findings in chat or a local draft and never publish GitHub reviews, comments, approvals, reactions, or change requests.

Codex discovers repository skills from .agents/skills/ as documented in the Codex skills guide. Claude loads project skills from .claude/skills/ and supports symlinked skill folders, as documented in the Claude Code skills guide.

Related Issues or PRs

N/A.

Checklist

  • Ran fast static checks to avoid CI fails, see Code Standards & Verifying Changes: just static
  • PR title has the form <type>(<area>): <title>, where <type> and <area> come from an appropriate C-<type>, respectively A-<area>, label. The title should match the target squash commit message.

Cute Animal Picture

Golden retriever

@spencer-tb
spencer-tb marked this pull request as ready for review September 3, 2026 09:29
@danceratopz
danceratopz self-requested a review September 3, 2026 09:30
@danceratopz danceratopz self-assigned this Sep 3, 2026

@danceratopz danceratopz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks! Quick question below.

Comment thread AGENTS.md
@codecov

codecov Bot commented Sep 3, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.01%. Comparing base (1855bb1) to head (cc6bcc5).
⚠️ Report is 2 commits behind head on forks/amsterdam.

Additional details and impacted files
@@               Coverage Diff                @@
##           forks/amsterdam    #3514   +/-   ##
================================================
  Coverage            94.01%   94.01%           
================================================
  Files                  624      624           
  Lines                36905    36905           
  Branches              3326     3326           
================================================
  Hits                 34695    34695           
  Misses                1533     1533           
  Partials               677      677           
Flag Coverage Δ
unittests 94.01% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@danceratopz danceratopz changed the title feat(doc): share Claude skills with Codex feat(tooling): share agent skills with Claude Sep 3, 2026
@danceratopz danceratopz added C-feat Category: an improvement or new feature A-tooling Area: Improvements or changes to auxiliary tooling such as uv, ruff, mypy, ... labels Sep 3, 2026
@danceratopz danceratopz changed the title feat(tooling): share agent skills with Claude feat(tooling): share agent skills across Codex and Claude Sep 3, 2026

@danceratopz danceratopz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks! 💕

  • Verified skills with the skill validator... then:
  • Pushed commit cc6bcc5 with two small documentation fixes:
    1. Updated enhance-ported-test to link to ../write-test/SKILL.md#transactions. The
      previous write-test.md reference became invalid when the flat .claude/commands/
      files were moved into separate .agents/skills//SKILL.md directories.
    2. Updated the fork layout guidance from state.py to state_tracker.py; every
      current fork now uses the latter.

@danceratopz
danceratopz merged commit 1d340aa into ethereum:forks/amsterdam Sep 3, 2026
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-tooling Area: Improvements or changes to auxiliary tooling such as uv, ruff, mypy, ... C-feat Category: an improvement or new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants