Skip to content

Complete featured icon declarations for Web Design Guidelines and Power BI - #84

Merged
aphansal123 merged 5 commits into
mainfrom
aphansal123-missing-featured-icons
Oct 14, 2026
Merged

aphansal123 merged 5 commits into
mainfrom
aphansal123-missing-featured-icons

Conversation

@aphansal123

@aphansal123 aphansal123 commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

This pull request adds icon metadata and icon entries for two skills in the catalog, improving their discoverability and visual representation. The changes ensure that both the Vercel Labs Web Design Guidelines skill and the Microsoft Power BI Modeling MCP skill have associated icons, with provenance and approval details included.

Icon metadata additions:

  • Added the com.github.agentfinder.icon.url field to the Vercel Labs Web Design Guidelines skill in ai-catalog.json, linking to its approved icon.
  • Added the com.github.agentfinder.icon.url field to the Microsoft Power BI Modeling MCP skill in ai-catalog.json, linking to its approved icon.

Catalog icon entries:

  • Added a new entry for the Vercel Labs Web Design Guidelines icon in catalog-icons.json, including provenance, approval, and SHA-256 checksum.
  • Added a new entry for the Microsoft Power BI Modeling MCP icon in catalog-icons.json, including provenance, approval, and SHA-256 checksum.

aphansal123 and others added 2 commits October 14, 2026 12:57
Rasterized only public/mcp-icons/power-bi.svg from github/github-app at 26f6444fb6602932e32f5b1ac8c587ba4e84eaf4 to a transparent 512x512 PNG, preserving the full viewBox without cropping. The contributing maintainer explicitly approved public redistribution and SVG-to-PNG conversion; no third-party license is asserted.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: dc1e2a4f-7beb-47ab-a052-3988639efad8
Add the existing approved Web Design Guidelines asset declaration and the newly approved Power BI PNG declaration. Pin both public asset URLs and record explicit redistribution approval and SHA-256 digests. Regenerate with the existing generator against the checked-in registry snapshot because the live endpoint rejects its own pagination cursor; only the two intended icon metadata fields change.

Related to github/copilot-ecosystem#1624. Service delivery remains separate in github/copilot-ecosystem#1625 and github/agentfinder#575.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: dc1e2a4f-7beb-47ab-a052-3988639efad8

This comment was marked as resolved.

Copilot AI and others added 2 commits October 14, 2026 17:57
Co-authored-by: aphansal123 <21958153+aphansal123@users.noreply.github.com>
Co-authored-by: aphansal123 <21958153+aphansal123@users.noreply.github.com>
@aphansal123
aphansal123 requested a review from a team October 14, 2026 18:01
@aphansal123
aphansal123 marked this pull request as ready for review October 14, 2026 18:02
@aphansal123
aphansal123 requested a balanced review from Copilot October 14, 2026 18:04

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🔵 Needs a closer look

The generated catalog contains substantial registry churn unrelated to the icon-focused scope.

0 open findings

1 resolved since last review

🧠 Review effort: Balanced

@aphansal123
aphansal123 merged commit 1d6fd10 into main Oct 14, 2026
6 checks passed
@aphansal123
aphansal123 deleted the aphansal123-missing-featured-icons branch October 14, 2026 19:14
@aphansal123 aphansal123 self-assigned this Oct 14, 2026
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.

4 participants