Migrate skills to canonical 'skills/' path for nvskills-ci onboarding#132
Closed
hannahli-nv wants to merge 3 commits into
Closed
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 |
hannahli-nv
force-pushed
the
nvskills-ci-migrate-skills-path
branch
from
e7f4fdf to
54960af
Compare
Collaborator
Author
|
/nvskills-ci |
1 similar comment
Collaborator
Author
|
/nvskills-ci |
hannahli-nv
force-pushed
the
nvskills-ci-migrate-skills-path
branch
from
13b7853 to
2385245
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. - 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>
Signed-off-by: Hannah Li <hanli@nvidia.com>
hannahli-nv
force-pushed
the
nvskills-ci-migrate-skills-path
branch
from
d6e4b14 to
0531fbc
Compare
Collaborator
Author
|
/nvskills-ci |
Collaborator
Author
|
/ok to test 0531fbc |
This was referenced May 28, 2026
mosheabr
pushed a commit
to NVIDIA/skills
that referenced
this pull request
May 29, 2026
TileGym PR NVIDIA/TileGym#132 migrated all 7 cuTile skill folders from `.agents/skills/` to the canonical release-facing `skills/` path. The previous `.agents/skills` was retained as a backward-compatibility symlink for tools that still expect the agentskills.io layout, but the sync workflow's git sparse-checkout + rsync does not reliably follow that symlink — it would either copy just the symlink-as-file or miss the contents. Update `path` from `.agents/skills/` → `skills/` so the sync workflow reads from the public-facing path directly. Single-entry parent-path form is retained (rather than per-skill flat layout) because all 7 cuTile skills are intended for the public catalog — there is no contributor/maintainer-only skill subset to filter out — and the parent-path form auto-discovers any new TileGym skills without requiring further nvidia/skills PRs. Also add the standard Apache SPDX header to match the convention adopted across components.d/ on 2026-05-28 (cf. #109, #113). Catalog effect: skills under `nvidia/skills/skills/TileGym/<name>/` will continue to populate via the existing per-skill sync-compliance gate (`skill.oms.sig` + `skill-card.md` + `evals.json`). The first TileGym skills landing `evals.json` are tracked in NVIDIA/TileGym#135 (adding-cutile-kernel, cutile-python); the remaining 5 will follow in subsequent PRs. Signed-off-by: Hannah Li <hanli@nvidia.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
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