Skip to content

Add PyPI wheel packaging smoke test for bundled registry skills (#182).#254

Merged
rosspeili merged 1 commit into
ARPAHLS:mainfrom
rosspeili:feat/issue-182-wheel-smoke-test
Jul 13, 2026
Merged

Add PyPI wheel packaging smoke test for bundled registry skills (#182).#254
rosspeili merged 1 commit into
ARPAHLS:mainfrom
rosspeili:feat/issue-182-wheel-smoke-test

Conversation

@rosspeili

Copy link
Copy Markdown
Contributor

Summary

Adds a wheel-smoke CI job that validates the PyPI shipping path: build a wheel, install it in a fresh venv (base deps only — no [all] or per-skill extras), and verify every bundled registry skill via scripts/wheel_smoke_test.py.

SkillLoader.load_skill() gains check_requirements=False so the smoke test can load bundles for packaging checks without installing optional skill extras.

Refs #182

What changed

  • scripts/wheel_smoke_test.py — checks bundle files, manifest/name parity, instructions/card assets; loads skills where possible; defers skills that need optional extras (expected)
  • .github/workflows/ci.yml — new wheel-smoke job (existing build job unchanged)
  • skillware/core/loader.pycheck_requirements: bool = True kwarg
  • tests/test_loader.py — coverage for check_requirements=False
  • DocsTESTING.md, CONTRIBUTING.md, ai_native_workflow.md
  • .flake8 / .gitignore — exclude local .venv-wheel-smoke/ test venv

Type of change

  • Framework FeatureSkillLoader.check_requirements
  • Packaging — wheel smoke test for bundled skills
  • RFC / meta — CI workflow

Checklist

  • Linked issue (Refs #182)
  • Scope matches the issue — no unrelated refactors
  • black --check and flake8 pass locally
  • pytest skills/ and pytest tests/ pass locally (266 passed)
  • CHANGELOG.md updated under [Unreleased]
  • examples/README.md — N/A
  • test_registry_docs.py — N/A (no skill/catalog changes)

Test plan

  • python -m black --check .
  • python -m flake8 .
  • python -m pytest skills/ tests/
  • Build wheel + scripts/wheel_smoke_test.py locally (10 loaded, 3 deferred)
  • Confirm wheel-smoke job passes on GitHub Actions

Related issues

Refs #182

…HLS#182).

CI runs a separate wheel-smoke job that builds a wheel, installs it in a fresh venv without optional extras, and verifies every bundled skill via scripts/wheel_smoke_test.py. SkillLoader gains check_requirements=False for load-only packaging checks.
@rosspeili
rosspeili merged commit 48fe17f into ARPAHLS:main Jul 13, 2026
6 checks passed
@rosspeili
rosspeili deleted the feat/issue-182-wheel-smoke-test branch July 13, 2026 11:15
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.

1 participant