Closed general

Skill cleanup for the next ab release: 5 invalid-YAML descriptions, retired adompkg/gallia refs, name≠dir (patch attached)

Colby Knox · 19d ago ·closed by John Lauer

ab 2.1.184 · adom-wiki 1.7.77 · checked 2026-10-22

Skill cleanup for the next ab release (patch attached; not published, on purpose).

Part of today's skill cleanup across adom/* (approved by John and Kyle). Every other package was fixed and published directly. adom-bridge was not, because each package version is an ab release: a skill-only 2.1.185 would become latest with no Windows installer behind it, and it would re-upload dist/. Also, .gitignore says skills/ is staged by scripts/release-publish.sh and is not the source of truth.

What's broken in the installed 2.1.184 skills today:

  • 5 descriptions are invalid YAML (an unquoted : ): adom-bridge-embedded, -fonts, -notify, -web-control-test, -web-integration. Claude Code shows the H1 heading instead of the description for embedded and notify.
  • adom-bridge-recording's description is cut to 74 chars by #143, which YAML reads as a comment.
  • adom-bridge-discovery's description is 1323 chars and it teaches the retired adompkg. ~/.claude/CLAUDE.md routes "do something on the user's PC" to this skill.
  • The companion list names skills that don't exist (adom-bridge-kicad/-fusion/pup, gallia paths).
  • The name doesn't match the directory for bridges-guide, cli-guide, multi-machine (adom-bridge-multi-ad) and ownership.

The patch (37 files, git apply --check passes on a clean page-repo clone; staged layout, so map it onto the ab source):

  • All descriptions are valid >- YAML of 300 chars or less. The sdk-* sub-skills are under 140 chars and name adom-bridge-sdk as their hub.
  • Hidden from the model (disable-model-invocation: true): sdk-audit, web-control-test, installer, direct-api.
  • Each skill's name now matches its directory.
  • adompkg is replaced with adom-wiki pkg.
  • gallia references are removed, and so are the gallia build notes in multi-machine.
  • The title rule now matches the live pages: <Tool> - the <Tool> Bridge.
  • Windows-native and Linux skills now say who they're for, and keep adom-bridge-cli.exe.

Also seen, not in the patch (CLI text): adom-bridge --help still calls the hd namespace "Hydrogen Desktop proxy", and the commands note still recommends browser_open_window. The puppeteer bridge 2.0.733 only has pup_* verbs.

adom-bridge.patch

2 Replies

Colby Knox · 19d ago

One more item for the same release. The attached patch does NOT cover it: the "Docker container" wording in the hub SKILL.md.

Claude always runs in a Linux container. In Hydrogen (ah), though, that container is a WSL2 distro on the user's own PC, not a remote Docker box. These lines are wrong there:

  • :3 (description): "bridging the Docker container to the user's local machine"
  • :8: "Claude Code (running in an Adom Docker container)"
  • :36: "The relay server runs in the Docker container."
  • :69: "You are running on a Docker container. You have NO access to the user's desktop." This is the one that matters. On ah, Claude reaches the PC's files at /mnt/c and reaches ab over loopback, so the setup-troubleshooting flow sends the user off to do steps Claude could check itself.

Suggested wording: "Claude runs in a Linux container: on ah, the Adom-Workspace WSL2 distro on the user's own PC (/mnt/c is its C: drive); on hc, a cloud container. Either way it reaches the desktop through ab. Run adom-context to see which." For :69: "You cannot click or type on the user's desktop yourself; ab is your hands. On ah you can still read and write the PC's files at /mnt/c." The notify_user "Hello from Docker" examples at :160 and :198 could say "from your workspace".

John Lauer · 13d ago

Applied in 2.1.185 (insiders), as part of a real ab release rather than a skill-only pkg version, as you suggested.

  • Your patch, mapped from the staged layout back onto the skills/*.md sources (the root SKILL.md and skills/adom-bridge/SKILL.md diffs were identical, so applied once). Applied strictly after normalising its CRLF line endings; nothing was hand-edited.
  • Your follow-up on the hub skill: the "Docker container" wording now says Claude runs in a Linux container (the Adom-Workspace WSL2 distro on ah, /mnt/c is C:; a cloud container on hc), the :69 line says ab is Claude's hands but on ah it can still read and write /mnt/c, and the notify examples say "from your workspace". The hub description stays under 300 chars (264).
  • The CLI text: --help now says "Hydrogen (ah) control API proxy", and the commands note recommends pup_open_window, not browser_open_window.

The release check confirmed all 36 staged skills are byte-identical to their sources. Thanks for the patch.

John Lauer closed this · 13d ago

Log in to reply.