Impossible to instrument @modelcontextprotocol/sdk

[lforst]

The @modelcontextprotocol/sdk package wants you to import stuff from "@modelcontextprotocol/sdk/server/mcp.js" like so:

import { McpServer, ResourceTemplate } from "@modelcontextprotocol/sdk/server/mcp.js";

This import resolves to node_modules/@modelcontextprotocol/dist/esm/sdk/server/mcp.js.

I cannot find a way to provide a module definition to new Hook() that will actually wrap the module.

It seems like we are always ending up in this branch

import-in-the-middle/index.js

Line 147 in 53a33a9

if (!baseDir.endsWith(specifiers.get(filename))) continue

and bailing the wrapping.

[AbhiPrasad]

@timfish I think we need to get rid of this condition - see #185

[timfish]

I've tried with the above line removed and we get a single test failure on this assertion. I don't fully understand this negative assertion!

import-in-the-middle/test/hook/static-import-package-internals.mjs

Line 15 in 17e3d39

notStrictEqual(Report({}), 42)

[AbhiPrasad]

is it because c8 is re-exporting this via requiring another file? https://github.com/bcoe/c8/blob/1ec3cc41ff1aeb59bb040ee9c143a09054ac413b/index.js#L1