Skip to content

Commit 0c022ea

Browse files
fix: add nlohmann_json dep to renderdoc-core and complete MCP init handshake in integration tests
- Link renderdoc-core against nlohmann_json (snapshot.cpp now uses it) - Send notifications/initialized in integration test SetUpTestSuite to satisfy the new pre-initialization check Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 97a3d3b commit 0c022ea

2 files changed

Lines changed: 8 additions & 0 deletions

File tree

CMakeLists.txt

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -89,6 +89,8 @@ if(RENDERDOC_DIR)
8989
)
9090
endif()
9191

92+
target_link_libraries(renderdoc-core PUBLIC nlohmann_json::nlohmann_json)
93+
9294
if(RENDERDOC_LIB)
9395
target_link_libraries(renderdoc-core PUBLIC ${RENDERDOC_LIB})
9496
message(STATUS "Found renderdoc library: ${RENDERDOC_LIB}")

tests/integration/test_protocol.cpp

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -264,6 +264,12 @@ class ProtocolTest : public ::testing::Test {
264264
}
265265

266266
s_initResponse = resp.value();
267+
268+
// Complete MCP handshake with notifications/initialized
269+
json notif;
270+
notif["jsonrpc"] = "2.0";
271+
notif["method"] = "notifications/initialized";
272+
s_runner->sendRequest(notif, 1000); // notification, no response expected
267273
}
268274

269275
static void TearDownTestSuite()

0 commit comments

Comments
 (0)