chore: format skill.json and update claw-release to 0.0.5#266
Open
davida-ps wants to merge 2 commits into
Open
chore: format skill.json and update claw-release to 0.0.5#266davida-ps wants to merge 2 commits into
davida-ps wants to merge 2 commits into
Conversation
Comment on lines
+247
to
+253
| for f in skills/*/skill.json; do | ||
| jq -r '"\(.name)\t\(.openclaw.category // "-")\tinternal=\(.openclaw.internal // false)"' "$f" | ||
| done | ||
| ``` | ||
|
|
||
| Each skill's category and internal flag are the source of truth in its own | ||
| `skills/<name>/skill.json` (`.openclaw.category` and `.openclaw.internal`). |
Contributor
There was a problem hiding this comment.
jq hardcodes .openclaw.category / .openclaw.internal for every skills/*/skill.json, but some manifests use picoclaw/hermes/nanoclaw, so the generated list shows -/false; should we derive the platform key from .platform or enumerate per-platform manifests instead of assuming openclaw?
Want Baz to fix this for you? Activate Fixer
Other fix methods
Prompt for AI Agents
Before applying, verify this suggestion against the current code. In
skills/claw-release/SKILL.md around lines 247-253, the “Existing Skills” enumeration
command uses jq fields `.openclaw.category` and `.openclaw.internal` for every
`skills/*/skill.json`, which breaks for manifests that store metadata under other
platform sections (e.g., `picoclaw`, `hermes`, `nanoclaw`). Refactor that snippet to
derive the platform key dynamically from `.platform` (or enumerate per-platform blocks)
and then read `category` and `internal` from that derived section; if the platform key
is missing, fall back to safe defaults while still not hardcoding `openclaw`. Update the
nearby explanatory text to match the new logic so the generated list accurately reflects
each skill’s real metadata.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
User description
Opener Type
Summary
Bump claw-release to version 0.0.5, apply consistent JSON formatting to
skill.json, and replace the hardcoded skills table inSKILL.mdwith a dynamic enumeration command.Changes Made
skill.jsonandSKILL.mdfrontmatterskill.jsonwith consistent indentation and line breaks for improved readability and maintainabilitySKILL.mdwith a bash command that dynamically enumerates skills from the repository, preventing documentation driftskill.jsonrather than a centralized listRelated Issues
None
Type of Change
Testing
No testing needed. Changes are formatting and documentation updates. Version bump follows semver conventions and is reflected in both required locations (
skill.jsonandSKILL.mdfrontmatter), which is enforced by CI.Checklist
https://claude.ai/code/session_014y5bVUCYtowdjUCEG4EZPw
Generated description
Below is a concise technical summary of the changes proposed in this PR:
Update skill metadata by bumping
claw-releaseto version 0.0.5 and applying consistent formatting toskill.json. Replace the static skills listing inSKILL.mdwith a dynamic enumeration script that derives category and internal flags from each skill’s metadata.skill.jsonformatting and raiseclaw-releaseto version 0.0.5 so the skill metadata reflects the latest release.Modified files (1)
Latest Contributors(2)
SKILL.mdwith a bash loop that enumerates each repository skill, pulling category and internal flags from itsskill.jsonmetadata.Modified files (1)
Latest Contributors(2)