Categorize skills and add Ghostex orchestration surface#89
Conversation
There was a problem hiding this comment.
Code Review
This pull request documents the transition and trial of Ghostex as a replacement for cmux and tmux, adding detailed developer experience reports and updating skill definitions to support Ghostex execution. The review feedback suggests correcting the <zmxName> placeholder to <providerSessionName> to match the actual JSON output of gx sessions --json, and using the absolute path for the bundled zmx binary in skills/spawn/SKILL.md to prevent command-not-found errors.
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.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 07e3379eb6
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
07ad24a to
9349265
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 9349265917
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| ## Ghostex execution | ||
|
|
||
| Use when the terminal is managed by Ghostex (detect: `GHOSTEX_*` env vars set, e.g. `GHOSTEX_ZMX_BIN`; `gx state` succeeding alone only proves the server is running somewhere). The bundled `ghostex-agent-orchestration` skill in the app is the authoritative command reference; this section only covers the spawn loop. |
There was a problem hiding this comment.
Regenerate the Codex plugin skill copy
The canonical spawn, cmux, and tmux skills were changed here, but the rendered Codex plugin copies under plugins/dotagents/skills/ were not updated (I compared those three SKILL.md files, and the plugin copies still lack the Ghostex routing/section). README says that directory is the copied skill tree regenerated by dotagents render and that dotagents doctor/CI fail when it drifts, and until it is regenerated Codex plugin users will install stale instructions.
Useful? React with 👍 / 👎.
What
spawn: new## Ghostex executionsection — headless spawn/send/read/kill loop, zmx pre-start workaround for lazy PTY, known gxserver 0.1.0 gaps, markdown-viewing workaround.cmux: marked under replacement trial by Ghostex (since 2026-06-12); kept until the trial concludes.tmux: detection/routing now checks cmux → Ghostex → tmux.docs/reports/: Ghostex DX test report, tmux baseline DX report, full skill categorization analysis.Why
Trialing Ghostex (maddada/Ghostex) as the cmux replacement. Hands-on testing showed the headless orchestration loop works (with a zmx pre-start workaround), so skills now route to Ghostex's bundled
ghostex-agent-orchestrationskill instead of duplicating it. Orchestration cluster stays as separate per-surface skills rather than a merged one — Ghostex bundles its own authoritative reference and the trial has not concluded.Related: upstream CLI crash fix maddada/Ghostex#21.
🤖 Generated with Claude Code