Skip to content

refactor(skill): unify Appium readiness and Apple guidance - #142

Merged
houlianpi merged 3 commits into
mainfrom
feat/unified-readiness-contract
Nov 6, 2026
Merged

houlianpi merged 3 commits into
mainfrom
feat/unified-readiness-contract

Conversation

@houlianpi

Copy link
Copy Markdown
Contributor

Summary

  • add native macOS setup and outcome-first iOS/macOS execution guidance
  • replace duplicated platform readiness prose with one structured readiness/authorization contract
  • shrink the main Skill to routing and lifecycle invariants; move platform-specific applicability into adapters
  • add focused iOS Simulator guidance and reject every Windows host/target without execution or fallback
  • expose a stable JSON verifier result while preserving fail-closed official Skill integrity checks

Verification

  • uv run ruff check src/amkernel/distribution.py src/amkernel/skill/scripts/verify_official_skills.py tests/e2e/skill_behavior.py tests/e2e/test_skill_behavior_evaluator.py tests/skills/test_amkernel_skill.py tests/test_distribution.py
  • uv run pytest tests/e2e/test_skill_behavior_evaluator.py tests/e2e/test_observation_policy.py -q — 139 passed
  • uv run pytest tests/skills/test_amkernel_skill.py tests/test_distribution.py -q — 62 passed
  • independent architecture review completed; all findings fixed and re-review found no actionable issue

Refs #98

@houlianpi
houlianpi force-pushed the feat/unified-readiness-contract branch from e3b9ef1 to 34410b6 Compare November 6, 2026 08:17
@houlianpi
houlianpi merged commit 9f4279e into main Nov 6, 2026
14 checks passed
@houlianpi
houlianpi deleted the feat/unified-readiness-contract branch November 6, 2026 08:29
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