Skip to content

Add missing ponytail-audit & ponytail-debt to the help card#164

Closed
abmorrill4 wants to merge 1 commit into
DietrichGebert:mainfrom
abmorrill4:fix/help-card-missing-skills
Closed

Add missing ponytail-audit & ponytail-debt to the help card#164
abmorrill4 wants to merge 1 commit into
DietrichGebert:mainfrom
abmorrill4:fix/help-card-missing-skills

Conversation

@abmorrill4

Copy link
Copy Markdown
Contributor

Problem

ponytail-help bills itself as a "quick-reference card for all ponytail modes, skills, and commands," but its Skills table only lists three of the five skills:

  • ponytail
  • ponytail-review
  • ponytail-help
  • ponytail-audit — missing
  • ponytail-debt — missing

The Codex @-command line had the same gap. A user who runs /ponytail-help to discover what's available never learns that two of the five skills exist.

Fix

  • Add a Skills-table row for ponytail-audit and ponytail-debt.
  • Extend the Codex @-command list to include both.

No format change — same table/prose style as before. Applied to both skills/ and the .openclaw/ mirror so the two trees stay in sync.

Filed separately from the Boundaries punctuation fix (#163) so each can be reviewed on its own.

🤖 Generated with Claude Code

The ponytail-help Skills table listed only ponytail, ponytail-review, and
ponytail-help, omitting ponytail-audit and ponytail-debt — two skills that
ship in the repo. A user running /ponytail-help would never learn they exist.

Add a row for each, and extend the Codex @-command list to match. Applied to
both skills/ and the .openclaw/ mirror.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@abmorrill4 abmorrill4 force-pushed the fix/help-card-missing-skills branch from 6ccefb6 to 7c8f53e Compare June 18, 2026 22:33
@abmorrill4

Copy link
Copy Markdown
Contributor Author

Rebased onto latest main to resolve the conflict with #108 (ponytail-gain). Kept the new ponytail-gain row and slotted ponytail-audit / ponytail-debt in alongside it. Also added @ponytail-gain to the Codex @-command line so it matches the table (it was missing there too). No em dashes introduced.

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

Thanks for filling this gap. Adding the missing ponytail-audit and ponytail-debt entries makes the help card accurately reflect the available skills, and it's nice to see the Codex command list updated to stay in sync with the table. Keeping both documentation trees aligned is also appreciated. Looks good to me! LGTM.

@DietrichGebert

Copy link
Copy Markdown
Owner

Thanks for catching the missing help-card entries. The test job is red on this branch, so I am closing for now. Happy to take it again once CI is green.

@abmorrill4

Copy link
Copy Markdown
Contributor Author

CI fix pushed (a26d1c2).

Root cause: the .openclaw/skills/* files are generated by scripts/build-openclaw-skills.js and tests/openclaw-skills.test.js asserts they match the generator. I'd hand-edited them — wrong approach. I re-ran the generator so they're now byte-identical to its output.

While doing so the generator also picked up a pre-existing drift on main: .openclaw/skills/ponytail/SKILL.md was missing the "Web tasks: rung 3 lookup" section added to skills/ponytail in #82 but never regenerated. That stale file fails the sync test on any branch, so the regeneration is bundled here. Happy to split it into its own PR if you'd rather keep this one scoped to the help card.

Verified locally: node --test tests/openclaw-skills.test.js → 18/18 pass. (The one local failure I see, csv: correct pandas one-liner passes, shells out to python+pandas, which isn't installed here; it passed in the CI log.)

The re-run looks gated on maintainer workflow approval for the fork PR.

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.

3 participants