Add dashboard-aware presentation to ocp-admin skills#139
Draft
ikrispin wants to merge 2 commits into
Draft
Conversation
Add UI Widget Policy to AGENTS.md that directs agents to call UI-opening tools immediately when interactive widgets are available, falling back to skill workflows in text-only environments. cluster-creator: Add open_cluster_creator to allowed-tools and replace Step 1 with environment detection that calls it immediately on UI-capable clients, skipping text-based parameter gathering. cluster-inventory: Add get_cluster_hosts to allowed-tools, document dashboard auto-presentation after list_clusters, and add setup navigation tool to dependencies.
dmartinol
reviewed
Jul 8, 2026
| ### Required MCP Tools | ||
| - `list_clusters` (from both servers) - Lists clusters (auto-routes to correct API) | ||
| - `list_clusters` (from both servers) - Lists clusters and opens the Cluster Inventory dashboard on UI-capable clients. Falls back to text on others. | ||
| - `get_cluster_hosts` (from openshift-self-managed) - Opens the Cluster Setup dashboard for a specific cluster on UI-capable clients. |
Collaborator
There was a problem hiding this comment.
IIRC, this is a new tool we've just added. Shouldn't we check about its availability first, like we did in the other "Check for interactive UI" section?
dmartinol
reviewed
Jul 8, 2026
dmartinol
left a comment
dmartinol
left a comment
Collaborator
There was a problem hiding this comment.
Thank you!
/lgtm let's wait for the tools being committed upstream before merging it
Add "(optional)" and "If available" guard to get_cluster_hosts in the Dependencies section, matching the pattern used for open_cluster_creator in cluster-creator skill.
ikrispin
force-pushed
the
feat/ocp-admin-dashboard-aware
branch
from
bed132b to
3aa9471
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add UI Widget Policy to
AGENTS.mdthat directs agents to call UI-opening tools immediately when interactive widgets are available, falling back to skill workflows in text-only environments.cluster-creator: Add
open_cluster_creatorto allowed-tools and replace Step 1 with environment detection that calls it immediately on UI-capable clients, skipping text-based parameter gathering.cluster-inventory: Add
get_cluster_hoststo allowed-tools, document dashboard auto-presentation afterlist_clusters, and add setup navigation tool to dependencies.Summary
Pack(s) affected
ocp-adminrh-ai-engineerrh-automationrh-basicrh-developerrh-srerh-virtChange type
mcps.json)Contribution method
/agentic-contribution-skillmake validate+make validate-skill-design-changed)Pack-persona alignment (new skills only)
AGENTS.md compliance
${VAR}referencesValidation
make validatepasses locallyname,description)name,description)