Add SkillEvaluator metadata.author and fix dead Markdown links - #5
Closed
scrimshawlife-ctrl wants to merge 1 commit into
Closed
scrimshawlife-ctrl wants to merge 1 commit into
scrimshawlife-ctrl wants to merge 1 commit into
Conversation
SkillEvaluator schema requires metadata.author in Name <email@host> form. Keep the existing top-level author. Retarget relative links that the code-integrity checker cannot resolve (query-string hrefs and the examples/quickstart directory) to files that exist. Co-authored-by: Daniel Meyer <scrimshawlife@gmail.com>
scrimshawlife-ctrl
marked this pull request as ready for review
August 21, 2026 02:20
prabu-openclaw
approved these changes
Aug 21, 2026
prabu-openclaw
left a comment
There was a problem hiding this comment.
Docs/metadata-only: adds metadata.author to SKILL.md frontmatter and fixes several dead Markdown links (points at examples/quickstart/README.md instead of the directory, strips non-functional ?family=/?term= query-string links in favor of the actual index.md target). No runtime code touched. CI failures (CodeQL, validate, build) also fail identically on main at HEAD, so they're pre-existing/unrelated to this diff, not a regression introduced here.
Contributor
Author
|
Closing: CI red since Aug; SkillEvaluator metadata/link fixes are stale vs current org main + personal product tip (mutation/installer). Re-open on a fresh branch from main if still needed after personal→org sync. |
2 of 4 tasks
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.
Summary
Fixes NVIDIA SkillEvaluator Tier 1 schema and code-integrity failures observed on this skill tree.
metadata.authortoSKILL.mdin the SkillEvaluatorName <email@host>shape. Keeps the existing top-levelauthor: Applied Alchemy Labs / Hermes.index.md?family=…and?term=…now point at the existingindex.mdfiles (docs/start/see-it-work.md,docs/index.md,docs/map/index.md, and the archive run pages that used the same pattern).QUICKSTART.md→examples/quickstart/now points atexamples/quickstart/README.md.Verified, not changed
These reported targets already exist at the linked paths, so the hrefs were left as-is:
QUICKSTART.md→./README.md,docs/start/*,docs/hermes-skill.mdSPEC.md→docs/api-v1.mdARCHITECTURE.md→./SPEC.md,./docs/api-v1.md,./docs/hermes-skill.mddocs/phase5.mdarchive links (archive/index.md,archive/runs/backfill-phase5-rizz-2026/index.md)docs/commands.mdanddocs/operator-loop.mdrelative links (commands.md,start/quickstart.md,modules/vectordb.md)No
skills/*/SKILL.mdcopies exist in this repo.Related issue or project item
SkillEvaluator 0.2.0 keyless
validate/ schema / code-integrity on this skill tree.Testing performed
PYTHONPATH=src python3 -m pytest -q: 196 passed, 5 skippedvalidate --checks schema --no-dedupon ahyperlex/skill tree: PASS (author_formataccepted; noauthor_missing)HygieneValidator._check_dead_links: all relative links valid in 98 markdown filesCollaboration checklist