Skip to content

Reject incomplete automatically discovered skill directories #441

Closed

Description

@devantler

🤖 Generated by the Agentic Engineer

Problem

When a plugin has one valid skill and an additional incomplete skills/ child, adding that child's name to the catalogue makes the package validator pass even though the child has no own SKILL.md. Catalogue agreement currently makes an undiscoverable resource look complete.

Proposed direction

Validate each direct automatically discovered skill directory's canonical regular SKILL.md before accepting it as a resource. Preserve valid empty optional roots and existing component layout rules.

Acceptance criteria

  • An incomplete sibling skill cannot be cleared by listing it in the catalogue.
  • A nested SKILL.md cannot substitute for the direct skill entrypoint.
  • Complete regular skill directories and explicit canonical paths still validate.
  • The production gate and its offline regression suite cover the whole direct inventory.

Size: S. Part of #38.

Activity

  1. added theissue type on Oct 3, 2026
  2. moved this to 🏃🏻‍♂️ In Progress in 🌊 Project Boardon Oct 3, 2026
  3. added a commit that references this issue on Oct 3, 2026
  4. devantler commented on Oct 4, 2026

    @devantler
    ContributorAuthor

    🤖 Generated by the Agentic Engineer

    Implemented by #443, now merged. Read back on main 2ca8f56: all 24 package-discovery regressions and complete package validation pass. The corresponding negative fixture refuses and valid consumer resources retain their identities and selections.

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

    No labels
    No labels

    Type

    Fields

    Priority

    None yet

    Effort

    None yet

    Projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions