Skip to content

Expand conformance coverage beyond the initial core plugin #2

Description

@jonathanhefner

Track Agent Plugins 1.0.0 coverage beyond the initial Core plugin. Close this issue once the remaining candidates have been implemented or explicitly deferred. Existing coverage is documented in the suite README and the linked fixture READMEs.

Completed

Remaining

Each proposed check should identify its specification requirement, the evidence needed to judge it, and any additional setup or plugin installation it requires. Continued availability of valid components does not by itself establish rejection of invalid components. Missing evidence remains not_verified; client developers determine which results they require to pass.

Considered and skipped

  • Exclusion of the malformed Recovery skill: deliberately retain valid-neighbor availability as the measured behavior. This preserves the earlier scope decision about enforcing invalid-skill rejection, reaffirmed after considering an explicit catalog-based exclusion check.

  • Inline MCP configuration in plugin.json: the explicit prohibition records the removal of earlier configurable/inline discovery machinery to reduce implementation burden. Its wording does not establish a distinct implementation risk sufficient to justify this fixture.

Deferred

  • Remaining filesystem containment: manifest, fixed component locations, and discovered skill files need valid external targets that remain available after installation, without assumptions about neighboring layouts or extra preparation. Generic package files lack a standardized client access operation for a fixture; subprocess access does not test client containment. Windows file symlinks are covered, but no portable fixture without additional setup has been established for junctions or other reparse mechanisms. Revisit with an attributable client operation, portable target, or concrete implementation defect.

  • Unsupported or unrecognized MCP schema identifiers: untested. The malformed-MCP fixture checks recovery from invalid JSON, not schema selection. A dedicated plugin or installation state is not currently justified while 1.0.0 is the only published specification version and no concrete compatibility problem has been established. Revisit when another version is published or evidence of incorrect client behavior appears.

  • Dedicated checks for missing or wrong-kind fixed component locations: the guide has no mcp.json, but successful loading does not establish that no erroneous diagnostic was emitted. Wrong-kind mcp.json and skills locations are distinct untested inputs; retaining current fixtures would require additional package states or installations. The existing malformed-MCP fixture exercises the related recovery behavior, and the incremental coverage does not currently justify more installations. Revisit if concrete client evidence raises their priority.

  • Absolute MCP endpoint URLs using schemes other than HTTP or HTTPS: untested. No concrete compatibility risk or useful fixture distinguishing incorrect acceptance from ordinary connection failure has been established. Revisit with such evidence; the current URL/header checks provide representative rather than exhaustive coverage.

  • Cross-version mismatch between plugin.json and mcp.json: requires a dedicated fixture plugin, and 1.0.0 is currently the only published specification version (1.1.0 is a working draft). Revisit when another version is published and the additional fixture is justified.

  • Plugin data persistence across updates: update orchestration and evidence transfer across lifecycle phases are outside the work being pursued at this time.

  • Fatal invalid-manifest rejection: correct rejection can happen during installation, before the guide can collect its diagnostic.

  • Server startup-failure isolation: deferred until attempted startup can be reliably observed.

  • Non-recursive expansion with placeholder-containing installation paths: set aside after assessing its practical value and setup cost.

Native-client execution of the existing suite was tracked separately in #3. PR #38 consolidates these limits and their rationale into documentation and closes this issue on merge. The completed checklist remains here as the coverage history.

Activity

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

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions