Skip to content

fix: add ponytail-audit and ponytail-debt to help skill table#441

Closed
nanaubusiness wants to merge 2 commits into
DietrichGebert:mainfrom
nanaubusiness:fix/help-skill-table
Closed

fix: add ponytail-audit and ponytail-debt to help skill table#441
nanaubusiness wants to merge 2 commits into
DietrichGebert:mainfrom
nanaubusiness:fix/help-skill-table

Conversation

@nanaubusiness

Copy link
Copy Markdown
Contributor

The skills table in ponytail-help/SKILL.md only listed 4 of 6 skills. ponytail-audit and ponytail-debt are now added to the table.

Closes #437

nanaubusiness added 2 commits June 28, 2026 06:23
JSON.parse throws SyntaxError which has no .code property,
so it was re-thrown and crashed the script mid-cleanup.
Now only re-throw if code is ENOENT; SyntaxError is harmless
(missing/invalid config file — just use defaults).

Closes DietrichGebert#434
@DietrichGebert

Copy link
Copy Markdown
Owner

Thanks @nanaubusiness, closing without merging. This PR bundles two unrelated changes: the skills-table fix (#437) and an uninstall.js change (#434). The uninstall.js hunk has a syntax error (missing closing paren in !(e instanceof SyntaxError)) so it would not parse, and the ponytail-audit row describes it as reviewing diffs, which is ponytail-review job — audit is whole-repo. #437 and #438 are now fixed in #480 with accurate descriptions and the regenerated .openclaw copy. If you want to pursue the malformed-settings.json handling, that belongs on its own PR against #434. Appreciate the report.

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.

skills/ponytail-help/SKILL.md: table only lists 4 of 6 skills — ponytail-audit and ponytail-debt missing

2 participants