Skip to content

test(mcp): prove tracedecay_remote_status behavior - #1678

Closed
ScriptedAlchemy wants to merge 1 commit into
masterfrom
cursor/prove-remote-status-behavior-e558
Closed

ScriptedAlchemy wants to merge 1 commit into
masterfrom
cursor/prove-remote-status-behavior-e558

Conversation

@ScriptedAlchemy

Copy link
Copy Markdown
Owner

Summary

  • Prove tracedecay_remote_status through real MCP tools/call responses, not handler-internal serialization.
  • A fresh daemon-mounted Remote Brain returns the literal unconfigured read in JSON and Markdown.
  • A server that never installed the reader returns the literal unavailable read, not an empty object or a JSON-RPC error.

Motivation

The tool's client-visible contract is a typed operational read. Existing dispatch coverage compares a fixture to its own serde output, which does not lock the JSON-RPC text a host actually receives.

Changes

  • crates/tracedecay/tests/mcp_suite/mcp_handler_test/remote_status_test.rs calls the production composition and an unmounted MCP server the way a host does, and asserts the full JSON-RPC result against literal payloads.

Test plan

  • REQUIRE_EXACT_TEST_COUNT=2 scripts/require-exact-test.sh cargo test -p tracedecay --features test-transport --test mcp_suite remote_status_test
  • cargo nextest run --workspace --no-fail-fast not run; this change is one MCP behavior test
  • cargo clippy not run workspace-wide
  • No production code change; manual proof is the test execution itself

Checklist

  • No changelog entry: test-only, no user-facing product change
  • No secrets, credentials, or .env files included
  • No breaking changes
Open in Web Open in Cursor 

Lock the client-visible reads: a fresh daemon reports unconfigured,
and a server without the reader reports unavailable.

Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com>
@changeset-bot

changeset-bot Bot commented Sep 18, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 65080f1

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@ScriptedAlchemy

Copy link
Copy Markdown
Owner Author

Closing as superseded. A sibling change proving the same tool landed first as #1816.

Evidence from the triage: git log origin/master --diff-filter=A -- mcp_handler_test/remote_status_test.rs => 391c6a4 via branch pr1645/remote-status-proof; gh pr list --state merged 'remote status in:title' => 1816 pr1645/remote-status-proof 'prove remote status across the tools/call boundary'; same path added in both, 4 conflict hunks

Reopen if the landed version misses an assertion this branch carries; name it and it can be ported as a small follow-up.

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.

2 participants