Add ai-ready skill (wrapper)#1551
Merged
aaronpowell merged 11 commits intogithub:stagedfrom Apr 30, 2026
Merged
Conversation
Lightweight wrapper that installs the full ai-ready skill from johnpapa/ai-ready and runs it. The full skill (~600 lines) analyzes repos and generates AGENTS.md, copilot-instructions.md, CI workflows, issue templates, and more — all customized to the repo's actual stack. Wrapper approach keeps this repo's review surface small (30 lines) while the source of truth stays in johnpapa/ai-ready. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
🔍 Skill Validator Results✅ All checks passed
Summary
Full validator output```text Found 1 skill(s) [ai-ready] 📊 ai-ready: 521 BPE tokens [chars/4: 516] (detailed ✓), 2 sections, 3 code blocks ✅ All checks passed (1 skill(s)) ``` |
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Adds a new wrapper skill that makes the upstream johnpapa/ai-ready skill discoverable in this repo while delegating the full (rapidly evolving) implementation to the upstream project.
Changes:
- Added
skills/ai-ready/SKILL.mdcontaining wrapper instructions to download the latest upstreamSKILL.mdinto~/.copilot/skills/ai-ready/. - Included steps to reload skills and then run the installed
ai-readyskill on the current repository.
Show a summary per file
| File | Description |
|---|---|
| skills/ai-ready/SKILL.md | New wrapper skill that installs the upstream ai-ready skill definition and guides users to reload and run it |
Copilot's findings
Comments suppressed due to low confidence (1)
skills/ai-ready/SKILL.md:32
- After
/skills reload, the instructions say “Then run the ai-ready skill…” but don’t specify how to invoke it. The CLI docs indicate skills can be invoked directly as a slash command (e.g.,/ai-ready) or by asking with a matching prompt; adding an explicit invocation step here would reduce user confusion.
- Files reviewed: 1/1 changed files
- Comments generated: 5
# Conflicts: # docs/README.skills.md
- Quote frontmatter description (YAML consistency) - Add PowerShell install commands for cross-platform support - Clarify that download overwrites existing copy - Add note about pinning to a specific tag/commit - Explicit invocation instructions (say 'make this repo ai-ready') Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
# Conflicts: # .github/workflows/check-plugin-structure.yml # docs/README.agents.md # docs/README.instructions.md # docs/README.skills.md
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
# Conflicts: # .github/plugin/marketplace.json # agents/address-comments.agent.md # agents/gem-browser-tester.agent.md # agents/gem-code-simplifier.agent.md # agents/gem-critic.agent.md # agents/gem-debugger.agent.md # agents/gem-designer-mobile.agent.md # agents/gem-designer.agent.md # agents/gem-devops.agent.md # agents/gem-documentation-writer.agent.md # agents/gem-implementer-mobile.agent.md # agents/gem-implementer.agent.md # agents/gem-mobile-tester.agent.md # agents/gem-orchestrator.agent.md # agents/gem-planner.agent.md # agents/gem-researcher.agent.md # agents/gem-reviewer.agent.md # agents/kusto-assistant.agent.md # agents/mentoring-juniors.agent.md # docs/README.agents.md # docs/README.skills.md # eng/generate-website-data.mjs # plugins/external.json # plugins/gem-team/.github/plugin/plugin.json # plugins/gem-team/README.md # website/data/tools.yml # website/src/content/docs/learning-hub/agents-and-subagents.md # website/src/content/docs/learning-hub/automating-with-hooks.md # website/src/content/docs/learning-hub/copilot-configuration-basics.md # website/src/content/docs/learning-hub/using-copilot-coding-agent.md
Contributor
|
We've got a CI failure - the entry in the README is in the wrong place, running |
# Conflicts: # docs/README.skills.md # eng/generate-website-data.mjs # website/astro.config.mjs # website/src/integrations/pagefind-resources.ts # website/src/pages/agents.astro # website/src/pages/hooks.astro # website/src/pages/index.astro # website/src/pages/instructions.astro # website/src/pages/plugins.astro # website/src/pages/skills.astro # website/src/pages/tools.astro # website/src/pages/workflows.astro # website/src/scripts/pages/agents-render.ts # website/src/scripts/pages/agents.ts # website/src/scripts/pages/hooks-render.ts # website/src/scripts/pages/hooks.ts # website/src/scripts/pages/index.ts # website/src/scripts/pages/instructions-render.ts # website/src/scripts/pages/instructions.ts # website/src/scripts/pages/plugins-render.ts # website/src/scripts/pages/plugins.ts # website/src/scripts/pages/skills-render.ts # website/src/scripts/pages/skills.ts # website/src/scripts/pages/tools-render.ts # website/src/scripts/pages/tools.ts # website/src/scripts/pages/workflows-render.ts # website/src/scripts/pages/workflows.ts # website/src/styles/global.css
Run npm run build to fix the entry ordering that caused CI failure. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
Author
Thanks, I addressed it. |
Address code review feedback — users should inspect the downloaded SKILL.md before reloading skills, per supply chain safety suggestion. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
Author
|
Addressed all 5 Copilot code review comments on PR.
|
aaronpowell
approved these changes
Apr 30, 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.
What this adds
A lightweight wrapper skill (~30 lines) that installs and runs the full ai-ready skill by John Papa.
How it works
When invoked, the wrapper:
SKILL.mdfromjohnpapa/ai-readyto~/.copilot/skills/ai-ready/Why a wrapper?
The full ai-ready skill is ~600 lines and evolves frequently. Rather than maintaining a copy here that goes stale, this wrapper keeps the skill discoverable in awesome-copilot while the source of truth stays in johnpapa/ai-ready — always up to date.
curlpulls the latest version on every installWhat the full skill does
Analyzes any repo and generates:
AGENTS.md— repo structure, build/test commands, how to add features.github/copilot-instructions.md— conventions mined from PR reviews + maintenance matrix.github/copilot-setup-steps.yml— cloud agent setupScores repos on a 🥉→🏆 medal system (12 tracked assets across AI Context, Dev Workflow, and Onboarding categories).
Previous PR
Replaces #1542 (external plugin approach), closed in favor of this wrapper approach per feedback about external plugin policy.