Skip to content

Categorize skills and add Ghostex orchestration surface#89

Merged
yourconscience merged 1 commit into
mainfrom
ghostex-skill-categorization
Jun 12, 2026
Merged

Categorize skills and add Ghostex orchestration surface#89
yourconscience merged 1 commit into
mainfrom
ghostex-skill-categorization

Conversation

@yourconscience

Copy link
Copy Markdown
Owner

What

  • README: skills list regrouped by category with the execution surface each skill drives.
  • spawn: new ## Ghostex execution section — 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-orchestration skill 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

@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 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.

Comment thread skills/spawn/SKILL.md Outdated
Comment thread skills/spawn/SKILL.md Outdated

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 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".

Comment thread docs/reports/tmux-baseline-dx.md Outdated
Comment thread skills/tmux/SKILL.md Outdated
@yourconscience yourconscience force-pushed the ghostex-skill-categorization branch from 07ad24a to 9349265 Compare June 12, 2026 14:00

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 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".

Comment thread skills/spawn/SKILL.md
Comment on lines +224 to +226
## 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.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge 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 👍 / 👎.

@yourconscience yourconscience merged commit 2ab8c07 into main Jun 12, 2026
5 checks passed
@yourconscience yourconscience deleted the ghostex-skill-categorization branch June 12, 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.

1 participant