Skip to content

skills: rename resources/ to references/ - #1301

Merged
rapids-bot[bot] merged 2 commits into
mainfrom
skill-rename-resources-to-references-main
May 27, 2026
Merged

skills: rename resources/ to references/#1301
rapids-bot[bot] merged 2 commits into
mainfrom
skill-rename-resources-to-references-main

Conversation

@ramakrishnap-nv

@ramakrishnap-nv ramakrishnap-nv commented May 26, 2026

Copy link
Copy Markdown
Collaborator

Align skill subfolder name with the agentskills.io standard. Pure rename — no content changes. Mirrors #1242 against main.

Align skill subfolder name with the agentskills.io standard. Pure rename
across all five skills that currently have a resources/ tree
(cuopt-developer, cuopt-install, cuopt-numerical-optimization-api-c,
cuopt-numerical-optimization-api-python, cuopt-routing-api-python),
plus the path references in SKILL.md / skill-card.md / asset READMEs,
benchmark/evals.json ground-truth strings, AGENTS.md, and the
.github/workflows/pr.yaml changed-files filters. Skill signatures
(skill.oms.sig) will be refreshed by the signing workflow.

Mirrors #1242 (release/26.06) and #1283 (release/26.06, in NVIDIA/cuopt),
but redone fresh on main to cover the wider skill tree (skill-card.md
files and the evals/ -> benchmark/ rename that landed on main).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Signed-off-by: Ramakrishna Prabhu <ramakrishnap@nvidia.com>
@ramakrishnap-nv
ramakrishnap-nv requested review from a team as code owners May 26, 2026 20:14
@ramakrishnap-nv
ramakrishnap-nv requested review from Iroy30 and bdice May 26, 2026 20:14
@ramakrishnap-nv ramakrishnap-nv changed the title skills: rename resources/ to references/ (main) skills: rename resources/ to references/ May 26, 2026
@ramakrishnap-nv ramakrishnap-nv self-assigned this May 26, 2026
@ramakrishnap-nv ramakrishnap-nv added non-breaking Introduces a non-breaking change improvement Improves an existing functionality labels May 26, 2026
@coderabbitai

coderabbitai Bot commented May 26, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 75fb4ab9-554e-4b7e-8253-f699b9533547

📥 Commits

Reviewing files that changed from the base of the PR and between 7cccde4 and 7b518ac.

📒 Files selected for processing (10)
  • skills/cuopt-developer/skill-card.md
  • skills/cuopt-developer/skill.oms.sig
  • skills/cuopt-install/skill-card.md
  • skills/cuopt-install/skill.oms.sig
  • skills/cuopt-numerical-optimization-api-c/skill-card.md
  • skills/cuopt-numerical-optimization-api-c/skill.oms.sig
  • skills/cuopt-numerical-optimization-api-python/skill-card.md
  • skills/cuopt-numerical-optimization-api-python/skill.oms.sig
  • skills/cuopt-routing-api-python/skill-card.md
  • skills/cuopt-routing-api-python/skill.oms.sig
🚧 Files skipped from review as they are similar to previous changes (1)
  • skills/cuopt-install/skill-card.md

📝 Walkthrough

Walkthrough

This PR consistently migrates documentation references across the cuOpt skills directory from resources/ subdirectories to references/ subdirectories. Changes span CI workflows that detect file modifications, skill documentation and overview files, example asset READMEs, benchmark evaluation data, and updated Sigstore signature bundles.

Changes

Documentation Path Migration

Layer / File(s) Summary
CI configuration for references path detection
.github/workflows/pr.yaml
Workflow file-group patterns for test_cpp, test_python_conda, and test_python_wheels are updated to trigger on changes under skills/**/references/** instead of skills/**/resources/**.
Skill documentation and overview files
AGENTS.md, skills/cuopt-developer/SKILL.md, skills/cuopt-developer/skill-card.md, skills/cuopt-install/SKILL.md, skills/cuopt-install/skill-card.md, skills/cuopt-numerical-optimization-api-c/SKILL.md, skills/cuopt-numerical-optimization-api-c/skill-card.md, skills/cuopt-numerical-optimization-api-python/SKILL.md, skills/cuopt-numerical-optimization-api-python/assets/README.md, skills/cuopt-numerical-optimization-api-python/skill-card.md, skills/cuopt-routing-api-python/SKILL.md, skills/cuopt-routing-api-python/skill-card.md
All skill documentation files update internal references to point to documentation under the new references/ directory structure. Links to setup guides, build instructions, Python bindings, examples, and VRP dimension internals are retargeted consistently; a few descriptive lines in skill-card/use-case text were adjusted.
Example asset documentation
skills/cuopt-numerical-optimization-api-c/assets/*/README.md, skills/cuopt-routing-api-python/assets/*/README.md
Asset example READMEs across LP, MILP, MPS, PDP, and VRP examples are updated to reference the corresponding references/examples.md location in their "See also" sections.
Benchmark evaluation data migration
skills/cuopt-developer/benchmark/evals.json
Benchmark evaluation items for developer workflows and installation tasks are updated to reference documentation paths under references/ for Python bindings, first-time setup, and contribution workflows.
Sigstore DSSE bundles
skills/*/skill.oms.sig files
Multiple Sigstore DSSE signature bundle files were regenerated: the embedded DSSE payloads (in-toto statements) and signature values were replaced while certificate chain and tlogEntries structures were preserved.

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly Related PRs

  • NVIDIA/cuopt#1233: Introduces VRP-dimension developer skill documentation that this PR now correctly references under the new references/ directory structure.

Suggested Labels

doc

Suggested Reviewers

  • Iroy30
  • tmckayus
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely summarizes the main change: renaming skill subfolder from resources/ to references/ for agentskills.io alignment.
Description check ✅ Passed The description is directly related to the changeset, explaining the purpose (align with agentskills.io standard), scope (pure rename), and context (mirrors #1242).
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch skill-rename-resources-to-references-main

Comment @coderabbitai help to get the list of available commands and usage tips.

@ramakrishnap-nv

Copy link
Copy Markdown
Collaborator Author

@bdice may I get your review on this PR, this is same as #1242, I just closed that one since the CI for skill eval only runs from branch on main repo.

@ramakrishnap-nv

Copy link
Copy Markdown
Collaborator Author

/nvskills-ci

@copy-pr-bot

copy-pr-bot Bot commented May 27, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@ramakrishnap-nv

Copy link
Copy Markdown
Collaborator Author

/ok to test 7b518ac

@ramakrishnap-nv

Copy link
Copy Markdown
Collaborator Author

/merge

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Improves an existing functionality non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants