Skip to content

Publish rocm-doctor skill#87

Open
volen-silo wants to merge 1 commit into
mainfrom
rocm-doctor-publish
Open

Publish rocm-doctor skill#87
volen-silo wants to merge 1 commit into
mainfrom
rocm-doctor-publish

Conversation

@volen-silo

Copy link
Copy Markdown
Collaborator

What

Publishes the rocm-doctor skill: diagnoses ROCm / HIP / PyTorch / llama.cpp failures on AMD GPUs (Linux + Windows) against the closed failure-mode catalog now shipped natively in the rocm CLI (ROCm/rocm-cli#39: rocm examine / rocm diagnose / rocm fix). The skill is a thin driver over those commands — it fixes with consent or routes upstream, and never re-implements the probe/catalog/fixes.

Changes

  • New skill skills/rocm-doctor/ (SKILL.md, reference.md, skill-card.md), verified against the shipped rocm-core implementation (exit codes, JSON fields, the 15-mode catalog, the 4 auto-applicable fixes).
  • Phase 0 install step — the skill installs the rocm CLI (with user consent) when missing, so it's self-sufficient instead of dead-ending.
  • Register in .claude-plugin/marketplace.json; regenerate the derived Cursor/Codex/agents manifests.
  • README — mark rocm-doctor as in-repo (was _planned_) and genericize the "what is a skill?" example tree.
  • Cleanup — remove the superseded staging/rocm-doctor/ copy (old Python-scripts version) and the now-dead skillspector-allow.yml suppressions that pointed at those deleted scripts.

Notes

  • In-repo (Path A) skill — no sources.yml / federation entry needed.
  • check.sh passes: 0 errors across 7 skills, manifests in sync.

Add the rocm-doctor skill to the catalog: diagnoses ROCm/HIP/PyTorch/
llama.cpp failures on AMD GPUs against the closed catalog now shipped in
the rocm CLI (rocm examine / diagnose / fix), fixing with consent or
routing upstream. A Phase 0 step installs the rocm CLI (with consent) when
it's missing so the skill is self-sufficient. Register it in the
marketplace, regenerate the derived manifests, mark it in-repo in the
README, drop the superseded staging copy, and remove the now-dead
skillspector suppressions for the deleted staging scripts.

Signed-off-by: Eugene Volen <Eugene.Volen@amd.com>
@volen-silo volen-silo requested a review from danielholanda July 10, 2026 12:45
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.

1 participant