Add DOCA routing to NVIDIA skill finder (port of #378) - #379
Merged
Conversation
Signed-off-by: Tamir Tevet <ttevet@nvidia.com> Signed-off-by: Moshe Abramovitch <moshea@nvidia.com>
Collaborator
Author
|
/nvskills-ci |
…escription - Revert the plugins/nvidia-skills mirror copy of taxonomy-routing.md to main: the signing pipeline treats two paths carrying the same skill as a duplicate canonical content key (gate contract failure). The plugin bundle is updated through its own flow. - agents/openai.yaml: state that installs happen only with user approval, per security scan finding SQP-2 (installer behavior must be disclosed). Signed-off-by: Moshe Abramovitch <moshea@nvidia.com>
Collaborator
Author
|
/nvskills-ci |
6 similar comments
Collaborator
Author
|
/nvskills-ci |
Contributor
|
/nvskills-ci |
Collaborator
Author
|
/nvskills-ci |
Collaborator
Author
|
/nvskills-ci |
Collaborator
Author
|
/nvskills-ci |
Collaborator
Author
|
/nvskills-ci |
Signed-off-by: nvskills-svc-account <svc-nvskills-signing@nvidia.com>
…ENCHMARK.md Aggregate root benchmarks.json to match the re-signed nvidia-skill-finder BENCHMARK.md (signature commit 0ee611b). Root aggregate only — no changes under skills/**, so skill.oms.sig / content-integrity are unaffected. Signed-off-by: Moshe Abramovitch <moshea@nvidia.com> Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
3 tasks
3 tasks
jasonnvidia
added a commit
that referenced
this pull request
Jul 29, 2026
This repo's history uses minor bumps for deliberate changes to what the plugin is or ships (1.1.0 swapped the cuOpt skill, 1.2.0 consolidated it, 1.3.0 moved to discovery-first) and reserves patch bumps for the automated "chore: sync skills" commits. Renaming the display name, rewriting the listing copy, changing category, replacing the logo, and shipping #379's routing taxonomy is the former, not the latter. Signed-off-by: Jason Dudash <jdudash@nvidia.com>
mosheabr
pushed a commit
that referenced
this pull request
Jul 29, 2026
Rewrites the plugin description and store copy so the listing describes what the plugin actually does now — route users to the right skill in the live catalog — rather than the narrower framing it shipped with. Renames the display name to "NVIDIA Skills" so the picker distinguishes the plugin from NVIDIA the publisher, squares the logo to 512x512, moves the category to Developer Tools, and adds website/privacy/terms URLs required for store listings. The version bump is what delivers the DOCA routing changes from #379 to existing users. Claude Code resolves a plugin's version from plugin.json first and skips updates when the resolved version is unchanged, so the nightly SHA bumps in the Anthropic marketplace have not reached anyone installed at 1.3.0 since that skill content merged. Edits are to plugins.d/nvidia-skills.yml; the three plugin.json manifests and three marketplace.json registries are regenerated output from .github/scripts/build-plugins.sh. Signed-off-by: Jason Dudash <jdudash@nvidia.com>
mosheabr
pushed a commit
that referenced
this pull request
Jul 29, 2026
This repo's history uses minor bumps for deliberate changes to what the plugin is or ships (1.1.0 swapped the cuOpt skill, 1.2.0 consolidated it, 1.3.0 moved to discovery-first) and reserves patch bumps for the automated "chore: sync skills" commits. Renaming the display name, rewriting the listing copy, changing category, replacing the logo, and shipping #379's routing taxonomy is the former, not the latter. Signed-off-by: Jason Dudash <jdudash@nvidia.com>
melo-gonzo
pushed a commit
to melo-gonzo/NVIDIAskills
that referenced
this pull request
Aug 5, 2026
Rewrites the plugin description and store copy so the listing describes what the plugin actually does now — route users to the right skill in the live catalog — rather than the narrower framing it shipped with. Renames the display name to "NVIDIA Skills" so the picker distinguishes the plugin from NVIDIA the publisher, squares the logo to 512x512, moves the category to Developer Tools, and adds website/privacy/terms URLs required for store listings. The version bump is what delivers the DOCA routing changes from NVIDIA#379 to existing users. Claude Code resolves a plugin's version from plugin.json first and skips updates when the resolved version is unchanged, so the nightly SHA bumps in the Anthropic marketplace have not reached anyone installed at 1.3.0 since that skill content merged. Edits are to plugins.d/nvidia-skills.yml; the three plugin.json manifests and three marketplace.json registries are regenerated output from .github/scripts/build-plugins.sh. Signed-off-by: Jason Dudash <jdudash@nvidia.com> Signed-off-by: Carmelo Gonzales <43048528+melo-gonzo@users.noreply.github.com>
melo-gonzo
pushed a commit
to melo-gonzo/NVIDIAskills
that referenced
this pull request
Aug 5, 2026
This repo's history uses minor bumps for deliberate changes to what the plugin is or ships (1.1.0 swapped the cuOpt skill, 1.2.0 consolidated it, 1.3.0 moved to discovery-first) and reserves patch bumps for the automated "chore: sync skills" commits. Renaming the display name, rewriting the listing copy, changing category, replacing the logo, and shipping NVIDIA#379's routing taxonomy is the former, not the latter. Signed-off-by: Jason Dudash <jdudash@nvidia.com> Signed-off-by: Carmelo Gonzales <43048528+melo-gonzo@users.noreply.github.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
3 participants
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.
In-repo port of #378 by @tevet-tamir (authorship and DCO sign-off preserved) so
/nvskills-cican run the requirednvidia-skill-finderre-sign — the CI trigger no-ops on fork PRs.What
Adds DOCA / BlueField DPU / ConnectX / DPA / GPUNetIO / RDMA / DOCA Flow keywords to the Networking lane of
nvidia-skill-finder's routing taxonomy, plus a hint to search the live catalog for thedoca-skill family. Applied identically to both copies (skills/...and theplugins/nvidia-skills/...mirror).Why
The router's taxonomy had no DOCA coverage, leaving the catalog's 60
doca-*skills unreachable for taxonomy-only/ambiguous requests. Raised by the DOCA team.Validation
Before/after routing test (headless agent, BlueField-3 RDMA + Ethernet question): with catalog access both versions route to the correct
doca-*skills; with catalog access blocked both correctly refuse to fabricate skill names — no hallucination regression from the added hint.Editing
references/taxonomy-routing.mdchanges signed skill content, so this PR needs a/nvskills-cire-sign before merge (will trigger after CI checks pass).Closes #378
🤖 Generated with Claude Code