refactor(skills): reducing igniteui-blazor skills significantly#264
Open
kdinev wants to merge 3 commits into
Open
refactor(skills): reducing igniteui-blazor skills significantly#264kdinev wants to merge 3 commits into
kdinev wants to merge 3 commits into
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Refactors the skills/ shipped AI-skill documentation to be significantly thinner and more workflow-oriented, relying on the Ignite UI MCP servers for API and documentation facts instead of local reference files.
Changes:
- Consolidates the skills into concise “workflow” hubs (components, grids, theming, generate-from-image) with MCP-first guidance.
- Removes many
references/*.mdfiles, pushing detailed setup/feature guidance to MCP docs and tool lookups. - Updates the skills index/README and
AGENTS.mdto emphasize MCP-based API discovery and setup snippets.
Reviewed changes
Copilot reviewed 28 out of 28 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| skills/README.md | Updates skills catalog + MCP server/tooling + install guidance. |
| skills/igniteui-blazor-theming/SKILL.md | Replaces detailed theming reference content with a compact workflow and rules. |
| skills/igniteui-blazor-theming/references/mcp-setup.md | Removed (setup guidance moved out of local references). |
| skills/igniteui-blazor-theming/references/common-patterns.md | Removed (patterns folded into SKILL.md / MCP docs). |
| skills/igniteui-blazor-grids/SKILL.md | Replaces routing-hub + many references with a compact workflow/decision guide and gotchas. |
| skills/igniteui-blazor-grids/references/types.md | Removed (details folded into SKILL.md / MCP docs). |
| skills/igniteui-blazor-grids/references/structure.md | Removed (details folded into SKILL.md / MCP docs). |
| skills/igniteui-blazor-grids/references/state.md | Removed (details folded into SKILL.md / MCP docs). |
| skills/igniteui-blazor-grids/references/sizing.md | Removed (details folded into SKILL.md / MCP docs). |
| skills/igniteui-blazor-grids/references/paging-remote.md | Removed (details folded into SKILL.md / MCP docs). |
| skills/igniteui-blazor-grids/references/mcp-setup.md | Removed (setup guidance moved out of local references). |
| skills/igniteui-blazor-grids/references/grid-migration.md | Removed (migration guidance folded into SKILL.md / MCP docs). |
| skills/igniteui-blazor-grids/references/features.md | Removed (details folded into SKILL.md / MCP docs). |
| skills/igniteui-blazor-grids/references/editing.md | Removed (details folded into SKILL.md / MCP docs). |
| skills/igniteui-blazor-grids/references/data-operations.md | Removed (details folded into SKILL.md / MCP docs). |
| skills/igniteui-blazor-generate-from-image-design/SKILL.md | Streamlines the image→UI workflow and delegates details to theming/components skills + refs. |
| skills/igniteui-blazor-generate-from-image-design/references/component-mapping.md | Shrinks the mapping reference into a more compact “pattern → component” guide. |
| skills/igniteui-blazor-components/SKILL.md | Replaces multi-reference routing hub with an MCP-first workflow + curated Blazor gotchas. |
| skills/igniteui-blazor-components/references/setup.md | Removed (setup guidance moved to MCP docs / SKILL.md). |
| skills/igniteui-blazor-components/references/mcp-setup.md | Removed (setup guidance moved out of local references). |
| skills/igniteui-blazor-components/references/layout.md | Removed (details folded into SKILL.md / MCP docs). |
| skills/igniteui-blazor-components/references/layout-manager.md | Removed (details folded into SKILL.md / MCP docs). |
| skills/igniteui-blazor-components/references/form-controls.md | Removed (details folded into SKILL.md / MCP docs). |
| skills/igniteui-blazor-components/references/feedback.md | Removed (details folded into SKILL.md / MCP docs). |
| skills/igniteui-blazor-components/references/data-display.md | Removed (details folded into SKILL.md / MCP docs). |
| skills/igniteui-blazor-components/references/charts.md | Removed (details folded into SKILL.md / MCP docs). |
| skills/AGENTS.md | Adds an explicit requirement to use MCP tools for Ignite UI API/docs instead of memory. |
Member
Author
|
@ivanvpetrov I think we shouldn't waste time with this PR. The results I'm getting are way worse. |
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
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.
Description
Motivation / Context
Type of Change (check all that apply):
Component(s) / Area(s) Affected:
How Has This Been Tested?
Test Configuration:
Screenshots / Recordings
Checklist:
README.MDCHANGELOG.MDupdates for newly added functionalityCloses #