fix: correct four argument-hint outliers - #3673
Conversation
wizard:generate was missing a hint for its free-text procedure. review:code-review and review:security-review advertised an empty hint on no-arg CI skills. bugs:setup used unspaced check|apply. computer-use:diagnose carried an em dash in the hint. Co-authored-by: ksextonmelodic <ksextonmelodic@gmail.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Claude finished @kyle-sexton's task in 1m 17s —— View job Code review complete
No findings. This is a mechanical Verified independently (not just author-claimed):
Not independently verified: I was not able to execute |
|
Claude finished @kyle-sexton's task in 27s —— View job Security review complete
Skip gate triggered: this PR touches only No security findings. |
|
Last security-reviewed head: |
|
Claude has reviewed this PR 1 time. The lane skips further automatic reviews after 5; deleting this comment resets the count. |
<!-- CURSOR_AGENT_PR_BODY_BEGIN --> No linked issue ## Summary `machine-health:audit` already accepts `weekly|on-demand|first-run` and `--dry-run`, but autocomplete had no `argument-hint`. This is the leftover concrete fix from `#3542` after `#3673`. ## Fix - Add `argument-hint: "[weekly|on-demand|first-run] [--dry-run]"` to `plugins/machine-health/skills/audit/SKILL.md`. - Bump machine-health to 0.12.9 and record the change. House-style convention docs and a `check-skill.sh` argument-hint criterion stay deferred (`#3620` post-pilot gate). This PR does not close `#3542`. ## Verification `scripts/affected-tests.sh --run --explain` selected no suites: every changed file is a recorded no-suite class. [affected-tests no-suite result](https://cursor.com/agents/bc-7ffb619a-6ebb-4d47-9c45-d68e846edf93/artifacts?path=%2Fopt%2Fcursor%2Fartifacts%2F3542-mh-argument-hint-demo.log) ## Related Refs #3542. Follows #3673 (items 2-5) and #3672 (machine-health 0.12.8 `reference/` rename). <sub>To show artifacts inline, <a href="https://cursor.com/dashboard/cloud-agents#my-pull-requests">enable</a> in settings.</sub> <!-- CURSOR_AGENT_PR_BODY_END --> <div><a href="https://cursor.com/agents/bc-7ffb619a-6ebb-4d47-9c45-d68e846edf93?cursor_ref=pr_footer&cursor_cta=open_in_web"><picture><source media="(prefers-color-scheme: dark)" srcset="https://cursor.com/assets/images/open-in-web-dark.png"><source media="(prefers-color-scheme: light)" srcset="https://cursor.com/assets/images/open-in-web-light.png"><img alt="Open in Web" width="114" height="28" src="https://cursor.com/assets/images/open-in-web-dark.png"></picture></a> <a href="https://cursor.com/background-agent?bcId=bc-7ffb619a-6ebb-4d47-9c45-d68e846edf93&cursor_ref=pr_footer&cursor_cta=open_in_cursor"><picture><source media="(prefers-color-scheme: dark)" srcset="https://cursor.com/assets/images/open-in-cursor-dark.png"><source media="(prefers-color-scheme: light)" srcset="https://cursor.com/assets/images/open-in-cursor-light.png"><img alt="Open in Cursor" width="131" height="28" src="https://cursor.com/assets/images/open-in-cursor-dark.png"></picture></a> </div> Co-authored-by: Cursor Agent <cursoragent@cursor.com> Co-authored-by: ksextonmelodic <ksextonmelodic@gmail.com>
No linked issue
Summary
Four concrete
argument-hintoutliers from the 2026-08-31 fleet skills audit. Official docs treat the field as an optional autocomplete hint ([issue-number],[filename] [format]); an empty string is not a documented spelling, and the repo's instruction-surface style rejects em dashes.Fix
wizard:generate: addargument-hint: "<procedure to wizardize>".review:code-reviewandreview:security-review: dropargument-hint: ""so they match the other no-arg skills.bugs:setup: spellcheck | applylike sibling setups.computer-use:diagnose: drop the em dash; hint is the closed enum[screenshots|failures|quirks]. Omit-for-guide stays in the Arguments body.Left for later slices of #3542:
machine-health:audit(version overlap with #3672), the proposed house-style convention doc (blocked on the #3620 post-pilot gate), and thecheck-skill.shcriterion that would point at that doc.Verification
argument-hintrow: optional autocomplete hint; examples are terse placeholders.scripts/affected-tests.sh --run --explain→ every changed file is a recorded no-suite class (SKILL.md / CHANGELOG / plugin.json). Exit 0.Related
Refs #3542
Refs #3672
Official: https://code.claude.com/docs/en/skills#frontmatter-reference