@gallopsystems/agent-skills 1.10.0 → 1.11.0

package/README.md

@@ -44,10 +44,17 @@ yarn add -D @gallopsystems/agent-skills
 ```
 
 On install, a `postinstall` script symlinks the package's content into the
-project's `.claude/` directory:
+project, for both Claude Code and Codex:
 
 - **skills** — each directory containing a `SKILL.md` → `.claude/skills/<name>`
-- **commands** — each `.md` file under any `commands/` directory → `.claude/commands/<name>.md`
+  (Claude Code) **and** `.agents/skills/<name>` (Codex). The `SKILL.md` format is
+  identical for both agents, so each skill dir is linked verbatim. Everything ships
+  as a skill: in current Claude Code a skill is both slash-invocable (`/<name>`) and
+  model-invocable, so `contribute-skill`, for example, gives you `/contribute-skill`
+  *and* gets auto-offered when relevant — no separate command needed.
+- **commands** — each `.md` file under any `commands/` directory →
+  `.claude/commands/<name>.md` (Claude only; Codex has no project command dir). Kept
+  for legacy/Claude-only commands; this package ships none today.
 
 Updating is just a version bump:
 
@@ -61,12 +68,13 @@ recreated on every install), so ignore them in `.gitignore`:
 ```
 .claude/skills
 .claude/commands
+.agents/skills
 ```
 
 Notes:
-- The script never clobbers a real `.claude/skills/<name>` or `.claude/commands/<name>`
-  you authored, and only removes symlinks it created. Run `yarn unlink-skills` to
-  remove all managed links.
+- The script never clobbers a real `.claude/skills/<name>`, `.agents/skills/<name>`,
+  or `.claude/commands/<name>` you authored, and only removes symlinks it created.
+  Run `yarn unlink-skills` to remove all managed links.
 - Works out of the box with Yarn (Classic, or Berry with `nodeLinker: node-modules`)
   and npm. **pnpm** (v10+) blocks dependency build scripts by default — add the
   package to `pnpm.onlyBuiltDependencies` for the `postinstall` to run.
@@ -163,8 +171,8 @@ Covers:
 
 Every skill ends with a **Contributing Back** section: when Claude works through
 something the skill didn't cover, it offers to contribute the lesson upstream. The
-`/contribute-skill` command (shipped in this package and symlinked into
-`.claude/commands/` on install) automates the flow: distill the generic lesson,
+`contribute-skill` skill (run `/contribute-skill` on Claude Code, or let either agent
+invoke it when relevant) automates the flow: distill the generic lesson,
 privacy-sweep it, clone or fork this repo, and open a PR against the right skill
 file. PRs from forks are welcome — content must be generic (placeholders only, no
 project-specific names, IDs, or domains).

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@gallopsystems/agent-skills",
-  "version": "1.10.0",
-  "description": "Gallop Systems Claude Code skills, symlinked into .claude/skills on install.",
+  "version": "1.11.0",
+  "description": "Gallop Systems agent skills, symlinked into .claude/skills (Claude Code) and .agents/skills (Codex) on install.",
   "license": "UNLICENSED",
   "repository": {
     "type": "git",

package/plugins/contribute-skill/.claude-plugin/plugin.json

@@ -0,0 +1,8 @@
+{
+  "name": "contribute-skill",
+  "description": "Contribute a lesson learned this session back to the gallop-systems/agent-skills repo as a PR: distill the generic rule, privacy-sweep it, and open a PR against the right skill.",
+  "version": "1.0.0",
+  "author": {
+    "name": "yeedle"
+  }
+}

package/{commands/contribute-skill.md → plugins/contribute-skill/skills/contribute-skill/SKILL.md}

@@ -1,5 +1,6 @@
 ---
-description: Contribute a lesson learned this session back to the gallop-systems/agent-skills repo as a PR
+name: contribute-skill
+description: Contribute a lesson learned this session back to the gallop-systems/agent-skills repo as a PR. Use when capturing an error→fix, a behavior that contradicted a skill, or a workflow gap the skill didn't cover, and turning it into a generic, public-safe addition to an installed skill.
 argument-hint: [which skill and/or what lesson]
 ---
 

package/scripts/link-skills.mjs

@@ -1,10 +1,19 @@
 #!/usr/bin/env node
-// Postinstall: symlink this package's skills and commands into the consuming
-// project's .claude directory, so a `yarn add` / `yarn install` keeps the
-// project in sync with this package's version:
+// Postinstall: symlink this package's skills (and any commands) into the consuming
+// project, so a `yarn add` / `yarn install` keeps the project in sync with this
+// package's version:
 //
-//   skills   -> .claude/skills/<name>        (a dir containing SKILL.md)
-//   commands -> .claude/commands/<rel>.md    (a .md file under any commands/ dir)
+//   skills   -> .claude/skills/<name>        (Claude Code; a dir with SKILL.md)
+//   skills   -> .agents/skills/<name>        (Codex; same SKILL.md format)
+//   commands -> .claude/commands/<rel>.md    (Claude Code only; a .md under commands/)
+//
+// The SKILL.md format (name + description frontmatter, no nesting) is identical
+// across Claude and Codex, so skill dirs are linked verbatim into both roots. In
+// current Claude Code, custom commands were merged into skills: a skill is itself
+// slash-invocable (`/<name>`) AND model-invocable, so it fully covers what a command
+// did. We therefore ship workflows as skills, not commands. The commands/ path is
+// still honored for any legacy Claude-only command, but Codex has no project command
+// dir, so commands are never linked there.
 //
 // This runs during `yarn install`, so it MUST NOT throw — a thrown error aborts
 // the whole install. Every failure path here degrades to a warning + exit 0.
@@ -229,20 +238,27 @@ function main() {
     return
   }
 
-  const skillsRoot = path.join(projectRoot, '.claude', 'skills')
-  const commandsRoot = path.join(projectRoot, '.claude', 'commands')
+  const claudeSkillsRoot = path.join(projectRoot, '.claude', 'skills') // Claude Code
+  const codexSkillsRoot = path.join(projectRoot, '.agents', 'skills') // Codex
+  const commandsRoot = path.join(projectRoot, '.claude', 'commands') // Claude Code only
 
   if (UNLINK) {
-    for (const root of [skillsRoot, commandsRoot]) {
+    for (const root of [claudeSkillsRoot, codexSkillsRoot, commandsRoot]) {
       if (fs.existsSync(root)) removeStaleLinks(realOrSelf(root), new Set())
     }
-    log('removed managed symlinks from .claude/skills and .claude/commands.')
+    log('removed managed symlinks from .claude/ and .agents/.')
     return
   }
 
-  const skills = linkInto(skillsRoot, collectSkills(PKG_DIR), 'skills')
+  const skillSources = collectSkills(PKG_DIR)
+  const skills = linkInto(claudeSkillsRoot, skillSources, 'skills')
+  linkInto(codexSkillsRoot, skillSources, 'skills')
   const commands = linkInto(commandsRoot, collectCommands(PKG_DIR), 'commands')
-  log(`linked ${skills} skill${skills === 1 ? '' : 's'} and ${commands} command${commands === 1 ? '' : 's'} into .claude/`)
+  log(
+    `linked ${skills} skill${skills === 1 ? '' : 's'} (.claude + .agents)` +
+      (commands ? ` and ${commands} command${commands === 1 ? '' : 's'} (.claude)` : '') +
+      '.',
+  )
 }
 
 try {