[material_ui] Add Material 3 Expressive migration skill - #12818
[material_ui] Add Material 3 Expressive migration skill#12818QuncCccccc wants to merge 1 commit into
Conversation
There was a problem hiding this comment.
Code Review
This pull request introduces a new agent skill documentation file for Material 3 Expressive migration and links it in the skills README. The review feedback correctly points out that the documentation references a non-existent packages/material_ui package instead of packages/flutter where the Material library resides, and provides suggestions to correct these references.
| ## Packages specific guidance | ||
|
|
||
| When there is specific guidance for packages it will go here. | ||
| - [Material 3 Expressive migration](material-3-expressive-migration/SKILL.md): guidance for migrating `packages/material_ui` components to Material 3 Expressive. |
There was a problem hiding this comment.
The Flutter Material library is located in packages/flutter (specifically under packages/flutter/lib/src/material), and there is no packages/material_ui package. Consider updating this reference to packages/flutter or the Material library to ensure accuracy.
| - [Material 3 Expressive migration](material-3-expressive-migration/SKILL.md): guidance for migrating `packages/material_ui` components to Material 3 Expressive. | |
| - [Material 3 Expressive migration](material-3-expressive-migration/SKILL.md): guidance for migrating packages/flutter Material components to Material 3 Expressive. |
| @@ -0,0 +1,20 @@ | |||
| --- | |||
| name: material-3-expressive-migration | |||
| description: Use when migrating packages/material_ui components to Material 3 Expressive, including generated defaults, component theme opt-ins, tests, examples, and release notes. | |||
There was a problem hiding this comment.
Update the description to refer to packages/flutter instead of packages/material_ui to reflect the correct package name.
| description: Use when migrating packages/material_ui components to Material 3 Expressive, including generated defaults, component theme opt-ins, tests, examples, and release notes. | |
| description: Use when migrating packages/flutter Material components to Material 3 Expressive, including generated defaults, component theme opt-ins, tests, examples, and release notes. |
|
|
||
| # Material 3 Expressive Migration | ||
|
|
||
| Use this skill for Material 3 Expressive component migration work in `packages/material_ui`. |
There was a problem hiding this comment.
The Material library is part of the packages/flutter package. Consider updating packages/material_ui to packages/flutter or the Material library to match the actual package structure.
| Use this skill for Material 3 Expressive component migration work in `packages/material_ui`. | |
| Use this skill for Material 3 Expressive component migration work in packages/flutter. |
This PR is to add an agent skill for Material 3 Expressive component migration work in
packages/material_ui.Related to Bring Material 3 Expressive to Flutter
Pre-Review Checklist
[shared_preferences]///).