feat(skills): integrate the visual explainer skill#864
feat(skills): integrate the visual explainer skill#864andrewDoing wants to merge 7 commits intomicrosoft:mainfrom
Conversation
🔧 - Generated by Copilot
🔧 - Generated by Copilot
# Conflicts: # collections/hve-core-all.collection.yml # collections/hve-core.collection.md # collections/hve-core.collection.yml
… anvineyard/hve-visual-explainer # Conflicts: # .github/skills/experimental/visual-explainer/SKILL.md # .github/skills/experimental/visual-explainer/references/css-patterns.md # .github/skills/experimental/visual-explainer/references/libraries.md # .github/skills/experimental/visual-explainer/references/responsive-nav.md # .github/skills/experimental/visual-explainer/references/slide-patterns.md # collections/hve-core-all.collection.yml
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #864 +/- ##
==========================================
+ Coverage 83.41% 85.45% +2.03%
==========================================
Files 20 27 +7
Lines 3510 5123 +1613
==========================================
+ Hits 2928 4378 +1450
- Misses 582 745 +163
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
|
What I'd like to see out of this tool (and the PR) is effectively what roles (using our role based documentation), this tool supports, and I'd like to see where it could be integrated into the any of the extension workflows so there's natural inclusion in the toolchains. My recommendation would be to select one of the following scenarios to show its toolchain integration:
Demonstrating how we can weave the tool into just one of these workflows (or one fo your choosing) will show how it fits in the project. Thanks and looking forward to seeing the outcome! |
|
I think I'm going to close this one in favor of determining a better pattern for how we want to host collections of public skills that we can more easily keep synced with their remotes. |
🔧 - Generated by Copilot
Pull Request
Description
Add a lightly modified version of the visual-explainer skill.
Related Issue(s)
Closes #157
Type of Change
Select all that apply:
Code & Documentation:
Infrastructure & Configuration:
AI Artifacts:
prompt-builderagent and addressed all feedback.github/instructions/*.instructions.md).github/prompts/*.prompt.md).github/agents/*.agent.md).github/skills/*/SKILL.md)Other:
.ps1,.sh,.py)Sample Prompts (for AI Artifact Contributions)
User Request:
Execution Flow:
Output Artifacts:
Success Indicators:
For detailed contribution requirements, see:
Testing
Checklist
Required Checks
AI Artifact Contributions
/prompt-analyzeto review contributionprompt-builderreviewRequired Automated Checks
The following validation commands must pass before merging:
npm run lint:mdnpm run spell-checknpm run lint:frontmatternpm run validate:skillsnpm run lint:md-linksnpm run lint:psnpm run plugin:generateSecurity Considerations
Additional Notes