feat(skills): reducing the size of skills and updating for correctness - #192
Open
kdinev wants to merge 1 commit into
Open
feat(skills): reducing the size of skills and updating for correctness#192kdinev wants to merge 1 commit into
kdinev wants to merge 1 commit into
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR streamlines the Ignite UI for React “skills” documentation to be smaller and more correctness-focused, emphasizing framework-specific rules, MCP-backed lookups, and updated guidance for theming, bundle size, and Grid Lite → premium grid migration.
Changes:
- Condenses/rewrites multiple React skills to remove duplicated setup content and keep only non-inferable, React-specific guidance.
- Consolidates component usage guidance into fewer reference docs (e.g.,
COMPONENTS.md,USAGE.md,DATAVIZ.md,MCP.md,TROUBLESHOOTING.md) and removes older, more verbose references. - Updates the skills index/README and changelog to reflect the new structure and corrected guidance (palette overrides, dark mode, registration, exporter imports, etc.).
Reviewed changes
Copilot reviewed 24 out of 24 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| skills/README.md | Rewrites the skills landing page: skill list, MCP server notes, and installation guidance. |
| skills/igniteui-react-optimize-bundle-size/SKILL.md | Replaces the long bundle-size guide with a shorter Ignite-UI-specific set of rules and examples. |
| skills/igniteui-react-generate-from-image-design/SKILL.md | Refactors the “from image” workflow into a shorter, ordered process emphasizing MCP-first lookups and token generation. |
| skills/igniteui-react-generate-from-image-design/reference/gotchas.md | Condenses gotchas into a more targeted “what doesn’t exist / common pitfalls” list plus focused examples. |
| skills/igniteui-react-generate-from-image-design/reference/component-mapping.md | Removed; component selection guidance is folded into the main skill text and shared component references. |
| skills/igniteui-react-customize-theme/SKILL.md | Reframes theming around the real palette/token model, correct dark-mode behavior, and MCP tool usage. |
| skills/igniteui-react-customize-theme/reference/TROUBLESHOOTING.md | Removed; troubleshooting guidance is folded into the main theming doc set. |
| skills/igniteui-react-customize-theme/reference/MCP-SERVER.md | Removed; MCP setup guidance is consolidated/embedded elsewhere. |
| skills/igniteui-react-customize-theme/reference/CSS-THEMING.md | Condenses CSS theming patterns, focusing on scoping, switching, and ::part() usage. |
| skills/igniteui-react-components/SKILL.md | Restructures into “non-negotiables,” package routing, quick start, and a small set of canonical references. |
| skills/igniteui-react-components/reference/USAGE.md | New consolidated doc for JSX/slots/events/refs/forms/TypeScript patterns. |
| skills/igniteui-react-components/reference/TROUBLESHOOTING.md | Replaces the long narrative troubleshooting doc with a compact symptom/cause/fix table. |
| skills/igniteui-react-components/reference/REFS-FORMS.md | Removed; content consolidated into USAGE.md. |
| skills/igniteui-react-components/reference/MCP.md | New consolidated MCP usage + setup doc, including correct named-argument schemas. |
| skills/igniteui-react-components/reference/MCP-SERVER.md | Removed; setup guidance consolidated into MCP.md. |
| skills/igniteui-react-components/reference/JSX-PATTERNS.md | Removed; content consolidated into USAGE.md and COMPONENTS.md. |
| skills/igniteui-react-components/reference/INSTALLATION.md | Removed; content consolidated into SKILL.md and references. |
| skills/igniteui-react-components/reference/EVENT-HANDLING.md | Removed; content consolidated into USAGE.md. |
| skills/igniteui-react-components/reference/DATAVIZ.md | New consolidated data viz + grids reference (registration, sizing, Grid Lite vs premium). |
| skills/igniteui-react-components/reference/COMPONENTS.md | New consolidated “component picking” reference + IgrTabs navigation vs content guidance. |
| skills/igniteui-react-components/reference/COMPONENT-CATALOGUE.md | Removed; content consolidated into COMPONENTS.md and MCP-first guidance. |
| skills/igniteui-react-components/reference/CHARTS-GRIDS.md | Removed; content consolidated into DATAVIZ.md. |
| skills/grid-lite-to-igr-grid-migration/SKILL.md | Refactors the migration guide: corrects deprecated registration guidance, clarifies API renames, and adds remote-data patterns. |
| CHANGELOG.md | Documents the refactor and multiple correctness fixes to the skills guidance. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Comment on lines
+55
to
+59
| manualChunks: { | ||
| 'ig-core': ['igniteui-react'], | ||
| 'ig-grids': ['igniteui-react-grids'], // only if actually installed | ||
| 'ig-charts': ['igniteui-react-charts'], | ||
| }, |
| ### GitHub Copilot | ||
|
|
||
| Copy the skill files into your project's `.agents/skills/` directory: | ||
| Keep each skill's `SKILL.md` alongside its `reference/` folder — the reference files are loaded on demand and the relative links between skills assume the layout is preserved. |
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.
Closes #
Additional information (check all that apply):
Checklist:
README.MDCHANGELOG.MDupdates for newly added functionality