Skip to content

feat(claude): make the /do skill discoverable by codex too - #415

Merged
defangdevs merged 1 commit into
masterfrom
feat/do-skill
Aug 28, 2026
Merged

feat(claude): make the /do skill discoverable by codex too#415
defangdevs merged 1 commit into
masterfrom
feat/do-skill

Conversation

@defangdevs

Copy link
Copy Markdown
Owner

Summary

Follow-up to #414 (already merged). Codex CLI has converged on the
same open SKILL.md format (name + description frontmatter,
agentskills.io) and reads project-level skills from
.codex/skills/<name>/SKILL.md -- same shape as Claude Code's
.claude/skills/, just a different discovery path.

  • Adds .codex/skills/do as a symlink to .claude/skills/do, one
    file read by both harnesses -- mirrors how CLAUDE.md already
    symlinks to AGENTS.md for the identical reason.
  • Adds the name: do frontmatter field the shared format requires
    (Claude Code alone doesn't need it; Codex's discovery does).
  • Genericized the two Claude-Code-specific spots in the skill body:
    the in-session task list now names TodoWrite as the Claude Code
    case, not the only case, and the review phase treats /code-review
    / /security-review as the Claude Code shortcut for an otherwise
    harness-agnostic review/security pass.

User-visible and security effects

None -- a new symlink and wording changes inside a Markdown skill
file; no runtime code, no new surface.

Checks run

Docs-only change (Markdown + one symlink); none of the .claude/**
or .codex/** paths trigger ci.yml or aws-ci.yml. Verified the
symlink resolves (readlink, cat through it) and read the rendered
Markdown by hand.

Codex CLI has converged on the same open SKILL.md format (name +
description frontmatter, agentskills.io) and reads project skills
from .codex/skills/<name>/SKILL.md -- same shape as Claude Code's
.claude/skills/, different path. Add .codex/skills/do as a symlink
to the canonical .claude/skills/do, mirroring how CLAUDE.md already
symlinks to AGENTS.md for the same reason (one file, two harnesses).

Also genericized the two Claude-Code-specific spots in the skill
body: the in-session task list now names TodoWrite as the Claude
Code case rather than the only case, and the review phase treats
/code-review and /security-review as the Claude Code shortcut for
an otherwise harness-agnostic review/security pass.
@coderabbitai

coderabbitai Bot commented Aug 28, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: b8934ba8-aaac-48bd-a88e-414f3687bbc9

📥 Commits

Reviewing files that changed from the base of the PR and between 6161501 and f6310e9.

📒 Files selected for processing (2)
  • .claude/skills/do/SKILL.md
  • .codex/skills/do

Included review availability: Your plan provides up to 10 included reviews per hour; 7 remain after this review.


📝 Walkthrough

Walkthrough

The PR updates the do skill metadata and workflow guidance. It also adds a symbolic link for Codex to use the shared skill definition.

Changes

Do skill workflow

Layer / File(s) Summary
Skill metadata and workflow guidance
.claude/skills/do/SKILL.md, .codex/skills/do
The skill adds name: do, requires an in-session task list with an equivalent task tool when needed, and permits manual equivalent review. The Codex path links to the shared skill.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Merge Risk: ⚪ Minimal · up to f6310

This localized documentation and skill-discovery change adds Codex access without changing runtime code or production behavior; no actionable merge-blocking risk remains.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the primary change: making the /do skill discoverable by Codex through the added skill discovery path.
Description check ✅ Passed The description directly explains the symlink, frontmatter, wording updates, expected effects, and verification steps in the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (2 skipped: 2 unsupported.)

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/do-skill

Comment @coderabbitai help to get the list of available commands.

@defangdevs
defangdevs merged commit 5975ec1 into master Aug 28, 2026
1 check passed
@defangdevs
defangdevs deleted the feat/do-skill branch August 28, 2026 03:14
@github-project-automation github-project-automation Bot moved this from Backlog to Done in Agent-Box Aug 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant