Skip to content

[Feat]: Enforce test_skill.py in test_skill_issuer.py for every registry skill #160

Description

@rosspeili

Feature Description

Parent: RFC #156.

Extend tests/test_skill_issuer.py so every directory under skills/ with a manifest.yaml must also contain test_skill.py.

Rollout

Also update

  • CONTRIBUTING.md skill checklist — bundle test required
  • templates/python_skill/README.md — same rule

Out of scope

Rationale

Docs alone won't stop registry drift. Issuer tests already gate manifest quality; adding bundle-test presence makes the #156 rule machine-enforced.

Implementation Idea

In test_skill_issuer.py, for each skill dir from _discover_skill_dirs(), assert (skill_dir / "test_skill.py").is_file().

Run pytest tests/test_skill_issuer.py locally. No CI workflow change beyond what #159 already does.

Metadata

Metadata

Assignees

No one assigned

    Labels

    core frameworkChanges to loader, env, or base classes.enhancementNew feature or request
    No fields configured for Feature.

    Projects

    Status
    Todo

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions