Skip to content
This repository was archived by the owner on Jun 30, 2026. It is now read-only.

Fix plugin install instructions on generated website#32

Merged
dmartinol merged 3 commits into
RHEcosystemAppEng:mainfrom
acmenezes:fix_internal_plugin_name
Mar 12, 2026
Merged

Fix plugin install instructions on generated website#32
dmartinol merged 3 commits into
RHEcosystemAppEng:mainfrom
acmenezes:fix_internal_plugin_name

Conversation

@acmenezes

@acmenezes acmenezes commented Mar 11, 2026

Copy link
Copy Markdown
Contributor

Summary

Fix plugin name mismatches and exclude incomplete pack from docs

  • Fix rh-virt plugin.json name to match marketplace.json (openshift-virtualization)
  • Fix rh-sre plugin.json name to match marketplace.json (sre-agents)
  • Exclude rh-support-engineer from docs generation (not in marketplace, no skills yet)

Error sample from terminal install

claude plugin install rh-sre
Installing plugin "rh-sre"...
✘ Failed to install plugin "rh-sre": Plugin "rh-sre" not found in any configured marketplace

Impact

  • Website now shows correct claude plugin install commands
  • Local installation with claude plugin install openshift-virtualization and sre-agents now matches documentation
  • Removes empty rh-support-engineer pack from generated site until ready

Pack(s) affected

  • rh-sre
  • rh-developer
  • ocp-admin
  • rh-support-engineer
  • rh-virt
  • Other / repo-wide

Change type

  • New skill
  • New agent
  • New pack
  • Update existing skill / agent
  • MCP server config (.mcp.json)
  • Docs / README
  • CI / tooling

Other: generated docs

CLAUDE.md compliance

  • Agents orchestrate skills; no direct MCP/tool calls in agents
  • Skills are single-purpose task executors
  • Skills encapsulate all tool access (MCP tools invoked only inside skills)
  • Document consultation: file is read with the Read tool, then declared to the user
  • No credentials hardcoded; env vars used via ${VAR} references
  • Human-in-the-loop confirmation added for any destructive or critical operations

Validation

  • make validate passes locally
  • New/changed skills have valid YAML frontmatter (name, description)
  • New/changed agents have valid YAML frontmatter (name, description)

@dmartinol

dmartinol commented Mar 12, 2026

Copy link
Copy Markdown
Collaborator

Hey @acmenezes thanks for your contribution!

claude plugin install rh-sre
I hit the same issue and was thinking to fix the other way, changing to marketplace.json file to expose rh-sre instead of sre-agents.

claude plugin install openshift-virtualization
good catch!

Removes empty rh-support-engineer pack from generated site until ready
Agree.

Since we're discussing how to introduce a package manager to handle these steps, things may change in the next days.
Anyway, to unblock new installations, we can merge the PR, but please revert the rh-sre plugin name and update the marketplace.json accordingly. We recently removed the only agent in the collection, so sre-agents would be misleading.

See acmenezes#1

@acmenezes

Copy link
Copy Markdown
Contributor Author

Hi @dmartinol I reverted the names as requested. I understand the reasons. And also tested locally here. Let me know if that's what you wanted or if you need anything else.

@dmartinol dmartinol left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you for your contribution!

@dmartinol dmartinol merged commit baf785f into RHEcosystemAppEng:main Mar 12, 2026
2 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants