fix: normalize skill frontmatter for cost-projection, diff-cost-projection, optimize-triage#29
fix: normalize skill frontmatter for cost-projection, diff-cost-projection, optimize-triage#29klamb wants to merge 3 commits into
Conversation
…optimize-triage Bumps cost-projection and diff-cost-projection from version 0.1.0 to 1.0.0. Fixes optimize-triage: adds missing version/author/license fields, converts block scalar description to a quoted single-line string, and replaces invalid `Task` tool reference with `Agent` in allowed-tools.
| author: CloudZero <support@cloudzero.com> | ||
| version: 1.0.0 | ||
| license: Apache-2.0 | ||
| allowed-tools: Bash, Read, Glob, Grep, WebFetch, Task |
There was a problem hiding this comment.
Quick check on the PR description before merging — it says:
corrects
allowed-toolsentry from invalidTaskname (should be the short-formTasksupported by the harness)
That sentence is internally contradictory (changing Task → Task), and the diff confirms this line is unchanged — Bash, Read, Glob, Grep, WebFetch, Task both before and after. Two cases:
- The description was edited in haste and
allowed-toolssimply wasn't touched. Harmless, but worth cleaning up the PR body so future git-log greppers aren't misled. - The intent was to fix
Task→ something else (e.g.,Agent, which is the sub-agent dispatch tool in current Claude Code), and the change got dropped.
Worth a 30-second sanity check because optimize-triage is the only skill in the catalog that declares allowed-tools, and if the correct name is Agent, then Task silently fails to grant permission — which is exactly the symptom your test plan is trying to verify ("confirm it dispatches research agents"). If you run the skill locally and parallel dispatch works, then Task is right and we just need to tidy the PR description; if it doesn't dispatch, fold the rename into this PR while we're already in the file.
There was a problem hiding this comment.
Good catch — you were right on both counts. The original PR description was wrong (the rename got dropped from that commit), and Agent is the correct tool name for sub-agent dispatch. Fixed in 2a5cd79.
2 participants
No description provided.