Skip to content

feat: add dashboard-aware presentation tools to cve-impact and fleet-inventory skills#131

Draft
TomerFi wants to merge 1 commit into
RHEcosystemAppEng:mainfrom
TomerFi:feat/dashboard-aware-skills
Draft

feat: add dashboard-aware presentation tools to cve-impact and fleet-inventory skills#131
TomerFi wants to merge 1 commit into
RHEcosystemAppEng:mainfrom
TomerFi:feat/dashboard-aware-skills

Conversation

@TomerFi

@TomerFi TomerFi commented Jul 2, 2026

Copy link
Copy Markdown
Member

Summary

  • Add load_cve_dashboard to the cve-impact skill workflow, prerequisites, and dependencies
  • Add load_inventory_dashboard to the fleet-inventory skill workflow, prerequisites, and dependencies
  • Both skills instruct the client to call the respective dashboard presentation tool with collected data after fetching results

These presentation tools are implemented in insights-mcp #389. They detect UI support at runtime — rendering a dashboard on UI-capable clients or falling back to text on others.

For full context on the approach investigation and decision, see APPENG-5287 and the MCP UI Integration Strategy ADR.

Validation

validate_skills_tier1.py (cve-impact)          ✅ passed, 0 warnings
validate_skills_tier1.py (fleet-inventory)     ✅ passed, 0 warnings
validate_skills_tier2.py (cve-impact)          ⚠️ passed with 4 warnings (pre-existing, same on main)
validate_skills_tier2.py (fleet-inventory)     ⚠️ passed with 5 warnings (pre-existing, same on main)
validate_skill_doc_links.py (cve-impact)       ✅ 14 links checked, 0 errors
validate_skill_doc_links.py (fleet-inventory)  ✅ 6 links checked, 0 errors
validate_docs_tree_links.py (cve-impact)       ✅ 104 files scanned, 0 errors
validate_docs_tree_links.py (fleet-inventory)  ✅ 104 files scanned, 0 errors

Recordings

CVE Dashboard (cve-impact):

Fleet Inventory Dashboard (fleet-inventory):

Related tickets

…inventory skills

Signed-off-by: Tomer Figenblat <tfigenbl@redhat.com>
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