skills/cuopt-user-rules: update with benchmark details

[ramakrishnap-nv]

update with benchmark details

[copy-pr-bot]

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

[ramakrishnap-nv]

/nvskills-ci

[coderabbitai]

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: 9c5d3c9c-a8a8-450f-957c-7f8b278033a9

📥 Commits

Reviewing files that changed from the base of the PR and between d7bc6aa and 25188a8.

📒 Files selected for processing (3)

• skills/cuopt-user-rules/BENCHMARK.md
• skills/cuopt-user-rules/skill-card.md
• skills/cuopt-user-rules/skill.oms.sig

✅ Files skipped from review due to trivial changes (1)

• skills/cuopt-user-rules/BENCHMARK.md

---

📝 Walkthrough

Walkthrough

Adds a BENCHMARK report, enriches the skill card metadata and outputs, inserts one blank line in SKILL.md front-matter, and updates the DSSE signature bundle to reflect those file changes.

Changes

Docs and signature update

Layer / File(s)	Summary
Skill docs and benchmark files skills/cuopt-user-rules/SKILL.md, skills/cuopt-user-rules/BENCHMARK.md, skills/cuopt-user-rules/skill-card.md	Inserted a blank line after SKILL.md front-matter, added BENCHMARK.md, and reformatted/expanded skill-card.md metadata, outputs, and evaluation sections.
DSSE signature bundle update skills/cuopt-user-rules/skill.oms.sig	Updated the DSSE envelope payload digests and replaced the signature to include the new/modified documentation resources.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• NVIDIA/cuopt#1308: Updates skill.oms.sig for cuopt-user-rules evaluation artifacts.
• NVIDIA/cuopt#1287: Touches cuopt-user-rules skill-card and regenerates skill.oms.sig; related content changes.

Suggested reviewers

• Iroy30
• tmckayus

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'skills/cuopt-user-rules: update with benchmark details' accurately reflects the main changes in the PR, which include adding a BENCHMARK.md file and updating related skill documentation.
Description check	✅ Passed	The description 'update with benchmark details' is related to the changeset, though minimal in detail. It references the benchmark updates that are central to the PR.
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 skills-cuopt-user-rules

---

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

[coderabbitai]

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

skills/cuopt-user-rules/SKILL.md (1)

1-231: ⚠️ Potential issue | 🟠 Major | ⚡ Quick win

PR title and description do not match the actual changes.

The PR title claims "update with benchmark details" but the only change is adding a blank line for formatting. No benchmark-related content is present in this diff. This suggests either:

• The PR title/description is incorrect
• Benchmark changes are in missing commits
• This is an incomplete submission

Additionally, as per coding guidelines, since this PR changes content under the skills/ directory, you should request NVSkills CI validation by commenting /nvskills-ci and ensure the signature commit remains in the PR.

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@skills/cuopt-user-rules/SKILL.md` around lines 1 - 231, PR metadata doesn't
match the diff: SKILL.md only has a formatting blank-line but the
title/description claim "update with benchmark details"; either update the PR
title/description to reflect the actual change, add the missing benchmark
commits, or split into a new PR containing the benchmark content; also, because
this modifies content under skills/, post a comment `/nvskills-ci` to request
NVSkills CI validation and ensure the signature commit (the required NVSkills
signature) remains in the PR so CI can validate the change.

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Outside diff comments:
In `@skills/cuopt-user-rules/SKILL.md`:
- Around line 1-231: PR metadata doesn't match the diff: SKILL.md only has a
formatting blank-line but the title/description claim "update with benchmark
details"; either update the PR title/description to reflect the actual change,
add the missing benchmark commits, or split into a new PR containing the
benchmark content; also, because this modifies content under skills/, post a
comment `/nvskills-ci` to request NVSkills CI validation and ensure the
signature commit (the required NVSkills signature) remains in the PR so CI can
validate the change.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: f99c1185-1062-4eea-ba0c-4b0122d02c29

📥 Commits

Reviewing files that changed from the base of the PR and between 6c61fcb and d7bc6aa.

📒 Files selected for processing (1)

• skills/cuopt-user-rules/SKILL.md

[copy-pr-bot]

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.