Skip to content

feat: add standard testing scaffolding to python_skill template (#19)#38

Merged
rosspeili merged 3 commits intoARPAHLS:mainfrom
Polimixanos:feat/testing-scaffolding-issue-19
Apr 21, 2026
Merged

feat: add standard testing scaffolding to python_skill template (#19)#38
rosspeili merged 3 commits intoARPAHLS:mainfrom
Polimixanos:feat/testing-scaffolding-issue-19

Conversation

@Polimixanos
Copy link
Copy Markdown
Contributor

@Polimixanos Polimixanos commented Apr 21, 2026

[Feat]: Add standard testing scaffolding to the python_skill template

Description

This PR implements the standard testing scaffolding for the python_skill template as requested in Issue #19. By providing a pre-configured test_skill.py in the boilerplate, we encourage Test-Driven Development (TDD) for all future community skill contributions.

Key Changes:

  • Testing Scaffolding: Added templates/python_skill/test_skill.py. This provides a pytest stub that validates skill execution against the output schema defined in manifest.yaml.
  • Manifest Standardization: Updated the manifest.yaml template to include parameters (JSON Schema), constitution, and requirements fields, aligning it with loader.py and current framework standards.
  • Boilerplate Completion: Added a boilerplate instructions.md to the template and refined skill.py to ensure all mandatory files are present and follow best practices.
  • Documentation & Process:
    • Updated CONTRIBUTING.md to mandate test_skill.py for all new skills.
    • Updated README.md architecture diagram.
    • Updated docs/TESTING.md with instructions for running skill-local tests.

Type of Change

  • 🧠 Framework Feature / RFC Updates: Core Framework Update (Templates and Contribution Standards)

Checklist

  • My code follows the Agent Code of Conduct.
  • I have included a properly formatted manifest.yaml in the template.
  • The skill logic operates purely in Python and does not rely on arbitrary LLM code generation.
  • Requirements and env_vars are explicitly documented in the manifest.
  • I have written unit tests proving deterministic execution and schema compliance.
  • I have verified that SkillLoader successfully loads this module without missing dependency errors.

Related Issues

Fixes #19

…HLS#19)

This update introduces a pytest-based test_skill.py stub, aligns manifest.yaml with core parameters/outputs standards, and ensures all mandatory boilerplate files (including instructions.md) are present in the template.
@rosspeili rosspeili merged commit 3b34d10 into ARPAHLS:main Apr 21, 2026
5 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.

[Feat]: Add standard testing scaffolding to the python_skill template

2 participants