Skip to content

verify-skills: no negative test for the shipped skills/ validation path #2644

Description

@cliffhall

Problem

Raised by Copilot on the v2.10.0 milestone merge (#2637, round 3, scripts/verify-skills.mjs:320). The validation of the packaged skills/*/SKILL.md (the !override branch, lines 320–333) is never reached by a test. Every fixture run in verify-skills.main.test.mjs passes an override directory, so a regression that stopped rejecting a missing or malformed shipped skill would leave the guard suite green.

Expected

An orchestration test against a fixture repository holding both .claude/skills and skills, including a malformed shipped skill that must fail. Alternatively, extract the traversal so it can be tested directly.

Priority

Low (rubric total 5): Severity 2 (test gap in repo tooling; nothing ships broken), Urgency 2, +1 milestoned.

Activity

  1. added this to the v2.11.0 milestone on Oct 7, 2026
  2. added
    v2Issues and PRs for v2
    choreMaintenance: deps, build tooling, CI, cleanup — no user-facing behavior change
    on Oct 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    choreMaintenance: deps, build tooling, CI, cleanup — no user-facing behavior changev2Issues and PRs for v2

    Type

    No type

    Projects

    No projects

      Milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions