Skip to content

fix(mcp): status() silently omits the api_compatibility "unknown" case that doctor() handles #6263

Description

@JSONbored

Context

Both status() (packages/loopover-mcp/bin/loopover-mcp.js:2941-2947) and doctor() (lines 3004-3016) render the same inspectApiCompatibility() result, which can return status unavailable/unknown/incompatible/compatible (defined at line 4214). doctor()'s if/else-if chain covers all four; status()'s human-readable branch covers only incompatible/compatible/unavailable, with no unknown case and no final else — so when the API reports an unparseable minimum-version string, loopover-mcp status (non-JSON output) prints nothing about compatibility at all, while doctor explicitly warns about it.

Requirements

  • Add the missing unknown case to status()'s compatibility-rendering branch, matching doctor()'s handling and messaging style.
  • Do not change doctor() — it's already correct.

Test Coverage Requirements

99%+ Codecov patch coverage on the added branch; add a test simulating an unknown compatibility status and confirming status() now prints something about it.

Deliverables

  • status() handles the unknown compatibility case.
  • Regression test.

Expected Outcome

loopover-mcp status never silently says nothing about API compatibility when the API itself couldn't be parsed.

Links & Resources

  • packages/loopover-mcp/bin/loopover-mcp.js:2941-2947,3004-3016,4214

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

    gittensor:bugGittensor-scored bug fix — scores a 0.05x multiplier.help wantedExtra attention is needed

    Projects

    No projects

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions