Skip to content

fix: repair broken vendored references, add missing shared skills, add validation CI - #20

Open
keyneszeng wants to merge 2 commits into
internet-court:mainfrom
keyneszeng:fix/upstream-fixes
Open

keyneszeng wants to merge 2 commits into
internet-court:mainfrom
keyneszeng:fix/upstream-fixes

Conversation

@keyneszeng

Copy link
Copy Markdown

Summary

Fixes several integrity issues in the vendored skill tree and adds CI to prevent regressions. All changes verified locally: 91/91 skills hash-match skills-lock.json, all skill files now have complete YAML frontmatter, and 3 of 11 broken relative links are repaired (the remaining 8 are upstream bugs in OKX/Starknet skills, reported separately).

Changes

New CI workflow (.github/workflows/validate.yml) — runs on push/PR:

  • JSON validity check for skills-lock.json, plugin.json, marketplace.json, openclaw.plugin.json
  • Verifies every vendored skill's computedHash matches its file on disk
  • Checks every skill.md has complete name/description frontmatter
  • Scans for broken relative markdown links

Repaired broken vendored references:

  • vendored/altlayer/_shared/ — 7 altlayer skills reference ../_shared/* but these files were never vendored. Added the 4 upstream files (preflight, session-and-target, cloud-claw-api-surface, cloud-claw-preflight).
  • vendored/altlayer/altllm-x402/ — referenced by altllm-portal-cli/references/skill-map.md but missing; added SKILL.md + references + LICENSE, registered in skills-lock.json.
  • vendored/kleros/feedback/SKILL.md — referenced by kleros skills but missing.

Content fixes:

  • vendored/near/near-ai-cloud/SKILL.md — copy-paste bug: link text says private-vs-anonymised.md but URL points to nonexistent model-list.md.
  • vendored/humanode/humanode-agentlink/SKILL.md — the only skill file missing YAML frontmatter (agents cannot discover/trigger it).

Docs fixes (vendored/README.md):

  • OKX skill count corrected: 23 → 9 (actual vendored set).
  • Removed references to nonexistent refreshCommand field and ../CLAUDE.md workflow doc.

skills-lock.json: recomputed computedHash for the two modified skill files; added altllm-x402 entry.

Not fixed here (upstream bugs, reported separately)

  • OKX okx-ai references nonexistent entry-points.md / payment-modes.md
  • Starknet skills reference repo-level ../README.md / packages/starknet-mcp-server/README.md

keyneszeng added 2 commits August 17, 2026 16:27
…d add validation CI

- Add .github/workflows/validate.yml: JSON schema check, skills-lock hash
  verification, frontmatter check, broken-link scan (prevents regressions)
- Add missing vendored/altlayer/_shared/ (preflight, session-and-target,
  cloud-claw-*) referenced by all 7 altlayer skills but never copied
- Add missing vendored/altlayer/altllm-x402/ skill referenced by
  altllm-portal-cli's skill-map (registered in skills-lock.json)
- Add missing vendored/kleros/feedback/SKILL.md referenced by kleros skills
- Fix vendored/near/near-ai-cloud/SKILL.md copy-paste bug: link pointed to
  references/model-list.md (404) but text said private-vs-anonymised.md
- Add YAML frontmatter to vendored/humanode/humanode-agentlink/SKILL.md
  (the only skill file missing name/description frontmatter)
- Fix vendored/README.md: OKX count 23->9, remove references to nonexistent
  refreshCommand field and ../CLAUDE.md workflow doc
- Update computedHash in skills-lock.json for modified files
…ile casing, harden CI hash check

- NOTICE.md: add altllm-x402 to the altlayer row (license compliance)
- README.md: correct vendored skill count 90 -> 91 (altllm-x402 added in
  the previous commit); remove references to nonexistent 'refresh command'
  field in skills-lock.json
- skills-lock.json: fix smart-accounts-kit entryFile to match disk
  (skill.md is lowercase upstream); lock now resolves on Linux
- validate.yml: resolve entryFile case-insensitively so upstream-faithful
  lowercase filenames (metamask) don't fail the hash check on Linux
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