Skip to content

Update ai-ready skill wrapper for v1.0.0#1739

Open
johnpapa wants to merge 4 commits into
github:stagedfrom
johnpapa:ai-ready-v1-update
Open

Update ai-ready skill wrapper for v1.0.0#1739
johnpapa wants to merge 4 commits into
github:stagedfrom
johnpapa:ai-ready-v1-update

Conversation

@johnpapa
Copy link
Copy Markdown
Contributor

What

Updates the ai-ready wrapper skill (2 files, minimal diff) for the v1.0.0 release.

Changes

File Change
skills/ai-ready/SKILL.md Description updated with WHEN/INVOKES/FOR metadata; "~600 lines" → "12-step procedure"; noted v1.0.0
docs/README.skills.md Regenerated via npm start to reflect the new description

Review feedback addressed proactively

  • Targets staged (not main)
  • Single-quoted description (per repo convention)
  • docs/README.skills.md regenerated via npm start — clean diff

Why

The ai-ready skill shipped v1.0.0 on 2026-05-15. This wrapper should reflect it's stable and spec-compliant.

johnpapa and others added 2 commits May 16, 2026 11:54
- Update description with WHEN/INVOKES/FOR trigger metadata (single-quoted per repo convention)
- Fix outdated ~600 lines reference → 12-step procedure with progressive disclosure
- Note current version v1.0.0

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings May 16, 2026 15:55
@johnpapa johnpapa requested a review from aaronpowell as a code owner May 16, 2026 15:55
@github-actions github-actions Bot added the skills PR touches skills label May 16, 2026
@github-actions
Copy link
Copy Markdown
Contributor

🟡 Contributor Reputation Check: MEDIUM risk

Check Risk
Profile MEDIUM
Credential audit NONE

Maintainers: please review this contributor before merging.
See the workflow run for full details.
Automated check powered by AGT.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 16, 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: 351 BPE tokens [chars/4: 366] (compact ✓), 2 sections, 2 code blocks
ℹ️ ✅ All checks passed (1 skill(s))
Full validator output ```text Found 1 skill(s) [ai-ready] 📊 ai-ready: 351 BPE tokens [chars/4: 366] (compact ✓), 2 sections, 2 code blocks ✅ All checks passed (1 skill(s)) ```

@github-actions github-actions Bot added the needs-review:MEDIUM Contributor reputation check flagged MEDIUM risk label May 16, 2026
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

Updates the ai-ready skill wrapper’s frontmatter description (and the generated skills index) to include more explicit routing metadata and refreshed wording aligned with the upstream ai-ready v1.0.0 release.

Changes:

  • Expanded skills/ai-ready/SKILL.md frontmatter description with WHEN/INVOKES/FOR guidance.
  • Updated the wrapper rationale text (“~600 lines” → “12-step procedure”, and mentions v1.0.0).
  • Regenerated docs/README.skills.md to reflect the new description.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
skills/ai-ready/SKILL.md Updates the skill’s frontmatter description and wrapper rationale text.
docs/README.skills.md Regenerates the skills index row for ai-ready to reflect the updated description.

Comment thread skills/ai-ready/SKILL.md Outdated
Comment thread skills/ai-ready/SKILL.md Outdated
- Remove WHEN/INVOKES/FOR metadata from description (keep user-facing summary only)
- Remove hardcoded v1.0.0 reference that would go stale
- Description now matches upstream staged (no docs regeneration needed)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown
Contributor

🟡 Contributor Reputation Check: MEDIUM risk

Check Risk
Profile MEDIUM
Credential audit NONE

Maintainers: please review this contributor before merging.
See the workflow run for full details.
Automated check powered by AGT.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown
Contributor

🟡 Contributor Reputation Check: MEDIUM risk

Check Risk
Profile MEDIUM
Credential audit NONE

Maintainers: please review this contributor before merging.
See the workflow run for full details.
Automated check powered by AGT.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-review:MEDIUM Contributor reputation check flagged MEDIUM risk skills PR touches skills

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants