Skip to content

chore(skills): add cmux skills - #1305

Merged
phernandez merged 1 commit into
mainfrom
chore/skills-cmux
Aug 23, 2026
Merged

chore(skills): add cmux skills#1305
phernandez merged 1 commit into
mainfrom
chore/skills-cmux

Conversation

@phernandez

Copy link
Copy Markdown
Member

Why

The local main checkout contained one unpublished change: four cmux operator skills installed from manaflow-ai/cmux. The other local-only commits were merge history or the already-landed semantic-read PR #1255. This PR preserves and publishes only the genuinely unlanded cmux package on top of current origin/main.

What Changed

  • Adds cmux topology/routing, Markdown viewer, settings, and workspace skills under .agents/skills/.
  • Adds Claude skill links for the four cmux skills.
  • Records the upstream source paths and content hashes in skills-lock.json.

Implementation Details

The original local commit was replayed without product-code changes onto current main (a9f4e085) and recommitted with the required semantic title and DCO sign-off. The prior local main tip is retained locally as backup/main-pre-sync-20260823.

Testing

Automated

  • just package-check: passed.
    • Claude and Codex package validation passed.
    • Hermes: 269 passed, 12 skipped.
    • OpenClaw typecheck, lint, build, 226 tests, and package dry-run passed.
  • git diff --cached --check: passed before commit.

Manual

  • Verified the four .claude/skills/* entries are symlinks to the corresponding .agents/skills/* packages.

Risks / Follow-ups

  • These skills track cmux CLI behavior and may need refreshing when the upstream command surface changes.
  • No Basic Memory runtime behavior changes.

Signed-off-by: phernandez <paul@basicmachines.co>
@phernandez
phernandez marked this pull request as ready for review August 23, 2026 17:21
@phernandez
phernandez merged commit 9007836 into main Aug 23, 2026
15 checks passed
@phernandez
phernandez deleted the chore/skills-cmux branch August 23, 2026 17:21

@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: 6e4418eb9d

ℹ️ 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 on lines +369 to +370
if any(full.startswith(s + ".") for s in supported):
continue

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Reject unknown shortcut action IDs during validation

When a shortcut action ID is misspelled, such as shortcuts.bindings.newTba, this prefix check accepts it because shortcuts.bindings is itself in the supported-path list. Consequently, cmux-settings validate exits successfully even though references/shortcut-actions.md defines a closed set of action IDs, leaving a nonfunctional binding undetected; validate dynamic object children according to their schema rather than accepting every descendant.

Useful? React with 👍 / 👎.

| [references/trigger-flash-and-health.md](references/trigger-flash-and-health.md) | Flash cue and surface health checks |
| [../cmux-workspace/SKILL.md](../cmux-workspace/SKILL.md) | Current caller workspace rules and non-disruptive automation |
| [../cmux-settings/SKILL.md](../cmux-settings/SKILL.md) | Safe cmux.json settings edits and validation |
| [../cmux-browser/SKILL.md](../cmux-browser/SKILL.md) | Browser automation on surface-backed webviews |

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Remove or install the referenced cmux-browser skill

When an agent follows this browser-automation reference, the target does not exist: a repo-wide search found only this link and the matching link in cmux-workspace, with no .agents/skills/cmux-browser, Claude symlink, or lock entry. Include the referenced skill in this package or remove the dead deep-dive link so browser requests do not lead agents to a missing instruction file.

Useful? React with 👍 / 👎.

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