docs: update comparison for Agent Skills open standard and fair MCP framing#141
docs: update comparison for Agent Skills open standard and fair MCP framing#141ReifStanfield wants to merge 2 commits into
Conversation
|
@ReifStanfield, thanks for #123 / #141, the COMPARISON and README updates look good and match the issue scope. Before merge, two small things:
Push after rebase and we’ll merge. 🙏 |
…raming - Retitle §2 Anthropic Skills → Agent Skills (SKILL.md); reflect multi-host open standard (Cursor, Copilot, Claude Code, Gemini CLI) with links to agentskills.io and anthropics/skills - Add convergence note in intro clarifying Skillware is not pivoting to SKILL.md - Soften §1 MCP bullets: acknowledge local stdio option; add complementary- layers sentence (protocol vs in-process capability) - Update Executive Summary Matrix: column label, Primary Goal cell, and Model Compatibility cell for the Agent Skills column - Align README Comparison blurb with §2 wording
3b6eb71 to
40f7e16
Compare
|
@ReifStanfield Thanks again for taking the dive into this. The updates on Two tiny additions and we can merge:
Push to the same branch (you don't have to force push, unless you don't want your old commits to be part of the history) and we will merge once CI is green. Nice work overall <3 |
Description
Type of Change (Matches Issue Templates)
manifest.yaml,skill.py, andinstructions.md)base_skill.py,loader.py, etc.)Checklist (all PRs)
python -m flake8 .andpytest tests/locally (or the subset relevant to this change).CHANGELOG.mdupdated under[Unreleased]if this PR changes user-visible behavior.examples/README.mdis updated if this PR adds, renames, or removes a runnable script underexamples/.New or updated skill (complete only if this PR adds or changes a skill under
skills/)Skip this section for framework-only, documentation-only, or other PRs that do not touch the skill registry.
Bundle & metadata
skills/<category>/<skill_name>/(copied fromtemplates/python_skill/or equivalent).manifest.yamlhasname,version,description, validparameters, andconstitution.manifest.yamlincludesissuerwith realnameandemail(not template placeholders).short_descriptionin manifest (~80 chars) forskillware list.issuer.githubandissuer.orgset when applicable.requirementsandenv_varsare documented when the skill needs them.Logic, cognition, and UI
skill.pyis deterministic Python (no arbitrary LLM-generated code paths).instructions.mdexplains when and how to use the skill.card.jsonis present and itsissuermatchesmanifest.yaml(nameandemailat minimum).Tests & loader
test_skill.pycovers execution and schema expectations.SkillLoader.load_skill("<category>/<skill_name>")succeeds (or missing deps are documented).Documentation & catalog
docs/skills/<skill_name>.mdexists or is updated (ID, Issuer, usage).docs/skills/README.mdlists the skill with ID and Issuer.Constitution & Safety (if adding or modifying a skill)
Related Issues
Fixes #123