Add skills validation harness and multi-harness packaging - #4
Closed
edmundmiller wants to merge 3 commits into
Closed
Add skills validation harness and multi-harness packaging#4edmundmiller wants to merge 3 commits into
edmundmiller wants to merge 3 commits into
Conversation
prek.toml config, markdownlint rules, prek-hook scripts (nf-core lint guardrails + manifest validators), agent_hooks package, harness tests, and the prek CI workflow. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Marketplace manifests for Claude/Codex/Cursor (marketplace name nf-core-agents), Claude settings.json hook wiring, and the nf-core-tools plugin (manifests, MCP config, skill, hooks). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
AGENT_HOOKS.md overview plus per-provider hook docs for Claude, Codex, Cursor, and the generic runner. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Author
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.
PR 2 of 2 porting content from
edmundmiller/nf-core-skillsinto this repo. Companion to #3 (#3), which brings the skills,AGENTS.md, docs, and README.What this PR adds
prek.toml,.markdownlint.json, thescripts/prek-hooks/validators (nf-core lint guardrails, manifest checks), thescripts/agent_hooks/provider package, harness tests undertests/, and the.github/workflows/prek.ymlworkflow..claude-plugin/), Codex (.agents/plugins/), and Cursor (.cursor-plugin/),.claude/settings.jsonhook wiring, and theplugins/nf-core-tools/plugin (manifests, MCP config, skill, hooks).docs/AGENT_HOOKS.mdplus per-provider docs underdocs/agent-hooks/.Reference updates
https://github.com/edmundmiller/nf-core-skillsreferences rewritten tohttps://github.com/nf-core/agents.namechanged fromnf-core-skillstonf-core-agentsin all three marketplace manifests, matching PR1’s install command/plugin install nf-core@nf-core-agents.Validation
prek run --all-filespasses for every harness/manifest check. The only failure is a pre-existingmarkdownlintissue in the existingREADME.md(heading/list spacing), unrelated to this move.🤖 Generated with Claude Code