Skip to content

Add ai-ready skill (wrapper)#1551

Merged
aaronpowell merged 11 commits intogithub:stagedfrom
johnpapa:add-ai-ready-skill
Apr 30, 2026
Merged

Add ai-ready skill (wrapper)#1551
aaronpowell merged 11 commits intogithub:stagedfrom
johnpapa:add-ai-ready-skill

Conversation

@johnpapa
Copy link
Copy Markdown
Contributor

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:

  1. Downloads the latest SKILL.md from johnpapa/ai-ready to ~/.copilot/skills/ai-ready/
  2. Reloads skills
  3. Runs the full analysis on the current repo

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.

  • Small review surface — 30 lines for this repo to review and maintain
  • Always currentcurl pulls the latest version on every install
  • No external plugin policy concerns — it's a skill, not an external plugin
  • One-time install — once downloaded, it works across all repos

What 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 setup
  • CI workflow, issue templates, PR template, changelog, and more

Scores 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.

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>
Copilot AI review requested due to automatic review settings April 29, 2026 01:09
@johnpapa johnpapa requested a review from aaronpowell as a code owner April 29, 2026 01:09
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 29, 2026

🔍 Skill Validator Results

✅ All checks passed

Scope Checked
Skills 1
Agents 1
Total 2
Severity Count
--- ---:
❌ Errors 0
⚠️ Warnings 0
ℹ️ Advisories 0

Summary

Level Finding
ℹ️ 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))
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>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.md containing wrapper instructions to download the latest upstream SKILL.md into ~/.copilot/skills/ai-ready/.
  • Included steps to reload skills and then run the installed ai-ready skill 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

Comment thread skills/ai-ready/SKILL.md Outdated
Comment thread skills/ai-ready/SKILL.md
Comment thread skills/ai-ready/SKILL.md Outdated
Comment thread skills/ai-ready/SKILL.md Outdated
Comment thread skills/ai-ready/SKILL.md Outdated
johnpapa and others added 6 commits April 28, 2026 21:14
- 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
@aaronpowell
Copy link
Copy Markdown
Contributor

We've got a CI failure - the entry in the README is in the wrong place, running npm run build will correct that.

johnpapa and others added 2 commits April 29, 2026 07:51
# 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>
@johnpapa
Copy link
Copy Markdown
Contributor Author

We've got a CI failure - the entry in the README is in the wrong place, running npm run build will correct that.

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>
@johnpapa
Copy link
Copy Markdown
Contributor Author

Addressed all 5 Copilot code review comments on PR.

  1. Description quoting — already fixed (single quotes)
  2. README.skills.md entry — already fixed (npm run build)
  3. Single file download clarity — already fixed (says "download the latest SKILL.md")
  4. PowerShell equivalents — already included
  5. Review step before loading — added head -20 review step so users inspect the downloaded skill
    before reloading

@aaronpowell aaronpowell merged commit b171fc5 into github:staged Apr 30, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants