Skip to content

docs(agents): remove references to skills that were never created#244

Open
mesutoezdil wants to merge 1 commit into
NVIDIA:mainfrom
mesutoezdil:mesutoezdil/docs/fix-stale-skill-refs
Open

docs(agents): remove references to skills that were never created#244
mesutoezdil wants to merge 1 commit into
NVIDIA:mainfrom
mesutoezdil:mesutoezdil/docs/fix-stale-skill-refs

Conversation

@mesutoezdil

@mesutoezdil mesutoezdil commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Three files referenced two skills, update-otel-collector-version and nvcf-self-hosted-local-dev, that were never created. Removed the dead references and pointed at existing docs instead.

Tested: version-bump script already guards each file with an existence check, so no behavior change. bash -n passes, no remaining references anywhere in the repo.

Refs: NO-REF

update-otel-collector-version and nvcf-self-hosted-local-dev were
referenced from three AGENTS.md/SKILL.md files and the version-bump
script's file list, but neither skill exists anywhere in the repo.
Drop the dead references and point at the existing docs instead.

JIRA: NO-REF
NVBug: none
Signed-off-by: mesutoezdil <mesudozdil@gmail.com>
@mesutoezdil
mesutoezdil requested a review from a team as a code owner July 17, 2026 16:45
@coderabbitai

coderabbitai Bot commented Jul 17, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

Local development guidance now references docs/dev/local-development.md. OTEL collector version-update documentation and the update script no longer reference or modify the removed version-update skill file.

Changes

Local development guidance

Layer / File(s) Summary
Update local development references
ai-tooling/dev/skills/nvcf-explore-stack/SKILL.md, tools/ncp-local-cluster/AGENTS.md
Local k3d and cluster handoff guidance now points to docs/dev/local-development.md instead of the previous local-development skill.

OTEL version-update guidance

Layer / File(s) Summary
Align collector version-update targets
src/compute-plane-services/byoo-otel-collector/AGENTS.md, src/compute-plane-services/byoo-otel-collector/scripts/update-collector-version.sh
Removed the version-update skill instructions and excluded its file from the documented and executable plain-form version-update targets.

Estimated code review effort: 2 (Simple) | ~10 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
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.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: removing references to nonexistent skills from docs and related script guidance.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants