skills: dedup overlapping .claude/skills#611
Merged
Merged
Conversation
- Merge check-python-style into format-code: format-code gains a "Check only" section (reproduce the CI gate without editing), the --include-local diff-scope note, and the behind-main tip; description picks up the CI-check triggers. Delete check-python-style. - Clarify division between flydsl-kernel-authoring (reference) and flydsl-tile-programming (guided wizard): add missing frontmatter to tile-programming, add a Scope note to each, and redirect the duplicated MFMA and debug-errors tables from the wizard to the reference. - Delete flydsl-internal-types-cleanup and port-to-layout-api. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
Pull request overview
This PR deduplicates and reorganizes the .claude/skills documentation set by merging overlapping skills, clarifying scope boundaries between kernel-authoring reference vs tile-programming wizard, and deleting redundant skills.
Changes:
- Merged
check-python-styleguidance intoformat-codeand added a check-only section for reproducing CI locally. - Added missing YAML frontmatter to
flydsl-tile-programmingand removed duplicated troubleshooting/MFMA reference tables in favor of linking toflydsl-kernel-authoring. - Deleted redundant/refactored skills (
check-python-style,port-to-layout-api,flydsl-internal-types-cleanup).
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| .claude/skills/format-code/SKILL.md | Expands skill scope to include check-only CI reproduction instructions and updates description accordingly. |
| .claude/skills/flydsl-tile-programming/SKILL.md | Adds YAML frontmatter + explicit scope note; replaces duplicated reference tables with links to the reference skill(s). |
| .claude/skills/flydsl-kernel-authoring/SKILL.md | Adds an explicit “reference vs wizard vs debugging” scope note near the top. |
| .claude/skills/check-python-style/SKILL.md | Deleted after being merged into format-code. |
| .claude/skills/port-to-layout-api/SKILL.md | Deleted as redundant with broader authoring/reference guidance. |
| .claude/skills/flydsl-internal-types-cleanup/SKILL.md | Deleted as redundant with broader authoring/reference guidance. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
coderfeli
approved these changes
Jun 4, 2026
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.
Summary
Removes duplication among the
.claude/skillsset (17 → 14 skills). No behavior change to the codebase; this only touches skill docs.1. Merge
check-python-style→format-codeformat-codewas already the superset (Python + C++) and already drovescripts/check_python_style.sh. It now also covers the check-only path:check_python_style.shwithout--fix,--install, and--include-localdiff-scope semantics.descriptionpicks up the CI-check triggers ("Check Python Code Style" job, before pushing, CI style failure).check-python-style/.2. Clear division:
flydsl-kernel-authoring(reference) vsflydsl-tile-programming(wizard)flydsl-tile-programming(it previously had none, so it wasn't properly discoverable), marking it the guided step-by-step wizard.debug-flydsl-kernel.3. Delete redundant refactor skills
flydsl-internal-types-cleanupandport-to-layout-api(their guidance is already covered byflydsl-kernel-authoring; no other files referenced them).Verification
prefetch-data-load's reference toflydsl-kernel-authoringremains valid.🤖 Generated with Claude Code