Migrate skills to canonical 'skills/' path for nvskills-ci onboarding#132
Open
hannahli-nv wants to merge 3 commits into
Open
Migrate skills to canonical 'skills/' path for nvskills-ci onboarding#132hannahli-nv wants to merge 3 commits into
hannahli-nv wants to merge 3 commits into
Conversation
Collaborator
Author
|
/nvskills-ci |
Collaborator
Author
|
/ok to test a885136 |
hannahli-nv
force-pushed
the
nvskills-ci-migrate-skills-path
branch
from
a885136 to
d7d6643
Compare
hannahli-nv
added a commit
that referenced
this pull request
May 26, 2026
Add the trigger workflow used by nvskills-ci. Matches the official template at NVIDIA/nvskills-ci/templates/team-request-workflow.yml (verbatim; SPDX header added to satisfy this repo's SPDX checker). The trigger fires when: - A maintainer/admin comments /nvskills-ci on a PR. - nv-skills-ci[bot] pushes a signature commit titled "Attach NVSkills validation signatures". Companion to PR #132 which migrates the skills/ canonical path. Signed-off-by: Hannah Li <hanli@nvidia.com>
hannahli-nv
added a commit
that referenced
this pull request
May 26, 2026
Companion to PR #132 which migrates the skills/ canonical path. Signed-off-by: Hannah Li <hanli@nvidia.com>
hannahli-nv
added a commit
that referenced
this pull request
May 26, 2026
Companion to PR #132 which migrates the skills/ canonical path. Signed-off-by: Hannah Li <hanli@nvidia.com>
hannahli-nv
force-pushed
the
nvskills-ci-migrate-skills-path
branch
from
d7d6643 to
579a467
Compare
Collaborator
Author
|
/nvskills-ci |
hannahli-nv
force-pushed
the
nvskills-ci-migrate-skills-path
branch
4 times, most recently
from
7cf4154 to
e7f4fdf
Compare
Collaborator
Author
|
/nvskills-ci |
Changes: - Move 7 cuTile skill folders from .agents/skills/ to skills/. - Add .agents/skills and .claude/skills symlinks pointing to ../skills for backward compatibility. - Update LICENSE, CONTRIBUTING.md, and .github/scripts/check_spdx_headers.py to reference the new skills/ path. - Update SKILL.md frontmatter license field to 'CC-BY-4.0 OR Apache-2.0'. - Split skills/cutile-autotuning/SKILL.md: move API Reference, Step-by-Step Workflow, and Pitfall Checklist into new files under references/ to keep SKILL.md concise. Signed-off-by: Hannah Li <hanli@nvidia.com>
hannahli-nv
force-pushed
the
nvskills-ci-migrate-skills-path
branch
from
e7f4fdf to
54960af
Compare
Collaborator
Author
|
/nvskills-ci |
These links pointed to `references/X.md`, which was correct when this content lived in the parent SKILL.md but resolves to `references/references/X.md` (404) now that workflow.md itself lives inside the `references/` directory. Drop the `references/` prefix so the links resolve to siblings.
Collaborator
Author
|
/nvskills-ci |
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
2 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.
Summary
Move the 7 cuTile skill folders from
.agents/skills/toskills/so thatnvskills-ci(added in companion PR #133) can pick up skill changes and produce signature commits.Why
The nvskills-ci validation gatekeeper only watches the following content path prefixes:
skills/team-skills/rules/team-rules/plugins/TileGym's skills lived at
.agents/skills/, which is not in this list. As a result,/nvskills-ciwould be skipped with "Request is skipped with no supported changes" and no signature commit would be produced.Layout after migration
The symlinks preserve compatibility with Claude Code's skill loader and any agent-tool wrappers that hard-code the older paths.
Changes
.agents/skills/<name>/toskills/<name>/(git rename detected — full history preserved)..agents/skillsand.claude/skillsconverted to symlinks pointing at../skills.LICENSE: Sections A and B updated to includeskills/in the dual-license scope alongside the existing.agents//.claude/mentions. Content unchanged — OSRB approval of the dual license travels with the files per Section B's travel clause.CONTRIBUTING.md: DCO section now referencesskills/as canonical with the symlinks called out as backward-compat aliases..github/scripts/check_spdx_headers.py: walker functions targetskills/directly;skillsadded toSKIP_DIRSso the default MIT walker does not try to put MIT headers on dual-licensed skill files.The
.github/workflows/request-nvskills-ci.ymltrigger workflow is added in companion PR #133 (kept separate so it can land onmainfirst, which is required forissue_comment-triggered workflows to fire).Verification
python3 .github/scripts/check_spdx_headers.py --action check→ All files have SPDX headersskills/,.agents/skills/,.claude/skills/) resolve to the same SKILL.md contentOrder of operations
mainso/nvskills-cicomments can be processed by GitHub Actions.NVSKILLS_CI_DISPATCH_TOKENsecret and TileGym inonboarded-repositories.json./nvskills-ci— once merged into main, thenv-nvskill-ci[bot]signature commit can be pushed and reviewed here.CI Configuration