Skip to content

fix(skills): namespace ClawHub skill slugs#263

Merged
davida-ps merged 2 commits into
mainfrom
davida-ps/fix-clawsec-skill-slugs
Jun 10, 2026
Merged

fix(skills): namespace ClawHub skill slugs#263
davida-ps merged 2 commits into
mainfrom
davida-ps/fix-clawsec-skill-slugs

Conversation

@davida-ps

@davida-ps davida-ps commented Jun 10, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Keep skill directories, skill.json names, release tags, GitHub release artifacts, and npx skills install commands unchanged.
  • Map only the ClawHub slug before clawhub inspect, clawhub publish, and GitHub release ClawHub install instructions.
  • Preserve existing clawsec-* ClawHub slugs unchanged and map non-clawsec-* skill names into the required clawsec-* namespace.
  • Share CI platform parsing across the slug resolver, install docs validator, and trust packet generator.

Testing

  • node scripts/test-skill-clawhub-slug.mjs
  • node scripts/test-skill-release-workflow.mjs
  • for test_file in scripts/test-skill-*.mjs; do node "$test_file"; done
  • npx eslint scripts/ci/skill_platforms.mjs scripts/ci/resolve_clawhub_slug.mjs scripts/ci/validate_skill_install_docs.mjs scripts/ci/generate_skill_release_trust_packet.mjs scripts/test-skill-clawhub-slug.mjs scripts/test-skill-release-workflow.mjs --max-warnings 0
  • npm run build

Release Note

No release tags were pushed from this PR. After merge, derive changed release tags from the merged PR diff, push one tag at a time, and wait for each Skill Release workflow before pushing the next tag.

Comment thread skills/clawsec-openclaw-soul-guardian/scripts/install_launchd_plist.py Outdated
Comment thread README.de.md Outdated
Comment thread skills/clawsec-openclaw-audit-watchdog/skill.json Outdated
Comment thread skills/clawsec-hermes-attestation-guardian/scripts/setup_advisory_check_cron.mjs Outdated
Comment thread skills/clawsec-hermes-attestation-guardian/scripts/setup_attestation_cron.mjs Outdated
@davida-ps davida-ps force-pushed the davida-ps/fix-clawsec-skill-slugs branch from 0f4fbba to 1525ed6 Compare June 10, 2026 12:39

Copy link
Copy Markdown
Collaborator Author

Replaced the original broad rename diff with the intended pipeline-only fix.

What changed now:

  • Skill directories, skill.json names, release tags, GitHub release artifacts, and npx skills commands stay unchanged from origin/main.
  • The release workflow resolves a separate ClawHub slug before clawhub inspect, clawhub publish, and the GitHub release ClawHub install line.
  • Existing clawsec-* slugs pass through unchanged; non-clawsec-* skills map to the required clawsec-* ClawHub namespace.
  • Manual ClawHub republish preserves the current slug helper before checking out older tags, so older failed release tags can still be republished with the new slug mapping.

Validation run locally:

  • node scripts/test-skill-clawhub-slug.mjs
  • node scripts/test-skill-release-workflow.mjs
  • for test_file in scripts/test-skill-*.mjs; do node "$test_file"; done
  • npx eslint scripts/ci/resolve_clawhub_slug.mjs scripts/test-skill-clawhub-slug.mjs scripts/test-skill-release-workflow.mjs --max-warnings 0
  • npm run build

No release tags were pushed.

Comment thread scripts/ci/resolve_clawhub_slug.mjs Outdated
Comment thread scripts/ci/resolve_clawhub_slug.mjs
@davida-ps davida-ps merged commit 59d54ed into main Jun 10, 2026
20 checks passed
@davida-ps davida-ps deleted the davida-ps/fix-clawsec-skill-slugs branch June 10, 2026 13:39
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.

1 participant