1 Commits

This Branch

This Branch

All Branches

Author	SHA1	Message	Date
4 people	4faae6696c	Add adversarial tests for verify-semantic-motion.py and verify-sequence-oauth.py (#69) ... * Add adversarial tests for verify-semantic-motion.py and verify-sequence-oauth.py Every other verify-*.py in scripts/ has a matching test-verify-*.py (verify-docs-sync, verify-geometry, verify-motion, verify-plugin-package), and CONTRIBUTING.md lists both verify-semantic-motion.py and verify-sequence-oauth.py as required PR gates, but neither had a dedicated test file exercising its own pass/fail behavior. - test-verify-semantic-motion.py: asserts the shipped SKILL.md / semantic-patterns.md / animation.md and the policy-trace animated example pass verify_markdown()/verify_example(), plus adversarial cases for a missing semantic-pattern router link, a missing pattern name, and a duplicate HTML/SVG id in the example. - test-verify-sequence-oauth.py: CLI smoke-tests the shipped repo (expects ALL GATES PASSED) and an adversarial case that removes the dashed-return marker from the OAuth 401 response in a temporary copy of the repo tree (verify-sequence-oauth.py resolves paths from its own __file__, so the adversarial run executes the copied script against the copied tree). Also wires both new tests into CONTRIBUTING.md's gate table and combined validation command, following the exact pattern used for the existing test-verify-*.py gates. (CI workflow wiring left out of this PR since it touches .github/workflows/ci.yml — happy to add in a follow-up if useful.) * Address review feedback on PR #69 - Wire test-verify-semantic-motion.py and test-verify-sequence-oauth.py into .github/workflows/ci.yml so CONTRIBUTING's new gates actually run in CI, with matching step-summary rows. - Bump .claude-plugin/plugin.json and .codex-plugin/plugin.json in sync to 2.5.7 so scripts/verify-plugin-package.py origin/main passes again. - Simplify test-verify-sequence-oauth.py's adversarial fixture to copy only the files verify-sequence-oauth.py's main() reads before the dashed-return check (type-sequence.md + cold-cache/oauth example trios), instead of shutil.copytree of the whole repository. * Re-bump plugin manifests above current main The prior 2.5.7 bump was superseded by subsequent merges of main into this branch (main itself advanced past it to 2.6.5). Re-run bump-plugin-version.py so the branch is strictly above upstream main again (2.6.6), per @philyuchkoff's review: the diff had settled back to no version delta at all. --------- Co-authored-by: alvinhui <alvinhui@users.noreply.github.com> Co-authored-by: Cathryn Lavery <50469282+cathrynlavery@users.noreply.github.com>	2026-08-25 01:57:59 -05:00