Make skills aggregator-compatible: co-located evals + self-contain complexa-sweep - #57
Open
trvachov wants to merge 1 commit into
Open
Make skills aggregator-compatible: co-located evals + self-contain complexa-sweep#57trvachov wants to merge 1 commit into
trvachov wants to merge 1 commit into
Conversation
…mplexa-sweep - Add evals/evals.json for all 5 skills (were catalog-only; sourced skills are overwritten from this repo on sync, so evals belong here — SRC-10). - complexa-sweep referenced ../../../docs/SWEEP.md (outside .claude/skills/, breaks when aggregated); copied it into complexa-sweep/reference/SWEEP.md and repointed the link (SRC-4). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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
1 participant
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.
What
Makes the Proteina-Complexa skills cleanly aggregatable by
bionemo-agent-toolkit(and
NVIDIA/skills), fixing two concrete compatibility issues:Co-located evals (SRC-10) — adds
evals/evals.jsonfor all 5 skills(
complexa-design,complexa-evaluate-pdbs,complexa-setup,complexa-sweep,complexa-target). These currently live only in theaggregator catalog; since these are sourced skills, the sync overwrites the
catalog from this repo, so the evals belong here or they're lost on sync.
(Copied from
bionemo-agent-toolkit@main; please verify they match currentskill behavior.)
Self-contained
complexa-sweep(SRC-4) —complexa-sweep/SKILL.mdreferenced
../../../docs/SWEEP.md, which is outside the vendored.claude/skills/subtree, so the link breaks once aggregated. Copieddocs/SWEEP.mdintocomplexa-sweep/reference/SWEEP.mdand repointed thelink. (The other two refs — to
../complexa-design/SKILL.mdand../_shared/reference/hardware.md— stay inside the subtree and are fine.)Not done here — needs owner decisions
complexa-slurm: this skill exists in the aggregator catalog but isabsent on
dev. I did not add it back — please confirm whether it wasintentionally removed or should be restored to this repo.
.claude/skills/README.md: gets vendored into the catalog as a straynon-skill file (SRC-9). Minor; remove or keep as you prefer.
.gitignoreignores.claude/: the skills here are force-tracked, sonew skill files (like these evals) are silently ignored unless
git add -f.Consider un-ignoring
.claude/skills/so future skill additions aren't dropped.🤖 Generated with Claude Code