Skip to content

fix(cli): restore consistent inspection behavior - #164

Merged
nkavian merged 1 commit into
mainfrom
fix/inspect-command-behavior
Oct 10, 2026
Merged

nkavian merged 1 commit into
mainfrom
fix/inspect-command-behavior

Conversation

@nkavian

@nkavian nkavian commented Oct 10, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Restore combined ODP, AEP, MPP, and x402 inspection and the established tables for service URLs, with identical behavior for implicit and explicit GET.
  • Preserve public document discovery, OpenAPI fallback, query strings, caching, and refresh. Recognize openapi.json rather than any path containing openapi.
  • Add regression tests, update command and skill documentation, and include a patch changeset.

Verification

Passed: pnpm build, pnpm typecheck, pnpm lint, pnpm test, pnpm coverage:changed, pnpm typedoc, pnpm format:check, and git diff --check.

Live checks confirmed that implicit and explicit GET display the established tables and produce identical JSON results. Exact ODP and OpenAPI documents remain publicly readable without vault unlocking, including OpenAPI URLs with query strings.

Dedicated OpenAPI command presentation and vault connection reuse are unchanged. No release is included.

@codecov

codecov Bot commented Oct 10, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 98.21429% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
packages/cli/src/commands/inspect/index.tsx 93.54% 1 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

@nkavian
nkavian force-pushed the fix/inspect-command-behavior branch from ab6bada to 562f515 Compare October 10, 2026 07:15
@nkavian
nkavian merged commit ba50514 into main Oct 10, 2026
2 checks passed
@nkavian
nkavian deleted the fix/inspect-command-behavior branch October 10, 2026 07:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant