This repository was archived by the owner on Jun 30, 2026. It is now read-only.
fix(rh-sre): align fleet-inventory with Lightspeed MCP tool schemas#126
Merged
dmartinol merged 4 commits intoJun 23, 2026
Merged
Conversation
…arameters - Added `inventory__list_hosts` and `inventory__find_host_by_name` to the allowed tools. - Updated documentation to clarify usage of `inventory__list_hosts` for fleet discovery and pagination. - Revised parameter references for `inventory__get_host_details` and `vulnerability__get_cve_systems` to ensure correct usage. - Enhanced examples to demonstrate new API patterns and pagination strategies. This update improves clarity and usability for users interacting with the Fleet Inventory Skill. Signed-off-by: Daniele Martinoli <dmartino@redhat.com>
- Created a new symlink in the CVE Impact references to point to the Fleet Inventory parameter reference documentation. This addition enhances the accessibility of parameter references for users working with CVE Impact. Signed-off-by: Daniele Martinoli <dmartino@redhat.com>
r2dedios
approved these changes
Jun 22, 2026
r2dedios
left a comment
r2dedios
left a comment
Contributor
There was a problem hiding this comment.
Just a couple of observations, Great job!
Signed-off-by: Daniele Martinoli <dmartino@redhat.com>
…se details - Added `inventory__get_host_system_profile` to the allowed tools for retrieving OS version and system profile. - Updated descriptions for `inventory__get_host_details` and `inventory__list_hosts` to clarify their functionalities and response structures. - Revised response fields and examples to reflect the latest API changes, including pagination and filtering strategies. - Improved clarity on how to handle RHEL version breakdowns and environment tags in the documentation. This update aims to provide users with comprehensive guidance on utilizing the Fleet Inventory Skill effectively. Signed-off-by: Daniele Martinoli <dmartino@redhat.com>
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
Fixes #124
Summary
Correct the
fleet-inventoryskill so documented MCP tools and parameters match the live Red Hat Lightspeed MCP schemas. Fleet discovery now usesinventory__list_hosts(withper_page/pagepagination) instead of parameterlessget_host_details. CVE-affected system queries usecve(notcve_id) withlimit/offset. Parameter reference, examples, andinsights-api.mdare updated accordingly.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)
N/A — existing skill update.
AGENTS.md compliance
${VAR}referencesValidation
make validatepasses locallyname,description)name,description)Test plan
./scripts/run-skill-linter.sh rh-sre/skills/fleet-inventory/passes/fleet-inventorywith "Show the managed fleet" and confirminventory__list_hosts(per_page=10, page=1)is usedvulnerability__get_cve_systems(cve="…")(notcve_id)inventory__get_host_detailsis only used with requiredhost_idsafter UUIDs are knownMade with Cursor