@alpic-ai/insights 0.0.0-dev.g1cd9b9b → 0.0.0-dev.g1d08792

package/dist/index.d.mts

@@ -1,6 +1,5 @@
 import { Server } from "@modelcontextprotocol/sdk/server/index.js";
 import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
-
 //#region src/intent-middleware.d.ts
 interface PromptData {
   toolName: string;
@@ -65,10 +64,6 @@ declare function feedbackMiddleware(options?: FeedbackMiddlewareOptions): McpMid
  * or the low-level `Server` and patches the `tools/list` and `tools/call`
  * request handlers to surface captured feedback via `options.handler` (or,
  * when `ALPIC_FEEDBACK_META_KEY` is set, via the response `_meta`).
- *
- * Already-registered handlers are wrapped immediately; future registrations
- * are wrapped via a `Map.set` proxy so order of calls relative to
- * `registerTool` does not matter.
  */
 declare const captureFeedback: (server: McpServer | Server, options?: FeedbackMiddlewareOptions) => void;
 //#endregion
@@ -79,10 +74,6 @@ declare const captureFeedback: (server: McpServer | Server, options?: FeedbackMi
  * low-level `Server` and patches the `tools/list` and `tools/call` request
  * handlers to surface the captured intent via `options.handler` (or, when
  * `ALPIC_INTENT_META_KEY` is set, via the response `_meta`).
- *
- * Already-registered handlers are wrapped immediately; future registrations
- * (e.g. tools added after this call) are wrapped via a `Map.set` proxy so order
- * of calls relative to `registerTool` does not matter.
  */
 declare const captureIntents: (server: McpServer | Server, options?: IntentMiddlewareOptions) => void;
 //#endregion

package/dist/index.mjs

@@ -3,6 +3,17 @@ import { CallToolRequestSchema, CallToolResultSchema, ListToolsResultSchema } fr
 const FEEDBACK_TOOL_NAME = "send_feedback";
 const FEEDBACK_TOOL_DESCRIPTION = "Send feedback about this MCP server to its operators. Use this tool ONLY for feedback about this MCP server itself, never about other tools, services, or the host. You MAY call this tool when you detect a genuine issue with this server (e.g. a tool that failed unexpectedly, an unhelpful response, a missing capability). You MAY also call it when the user explicitly asks to send feedback. Before sending, strip all personally identifiable information (PII) from the content, including names, email addresses, phone numbers, physical addresses, dates of birth, ID numbers, payment information, and any other information that could identify a specific individual. Replace stripped values with generic placeholders (e.g. \"[name]\", \"[email]\").";
 const FEEDBACK_RESPONSE_TEXT = "Feedback received. Thanks!";
+const FEEDBACK_OUTPUT_SCHEMA = {
+	type: "object",
+	properties: { status: {
+		type: "string",
+		enum: ["received"],
+		description: "Confirmation that the feedback was received by the server operators."
+	} },
+	required: ["status"],
+	additionalProperties: false
+};
+const FEEDBACK_STRUCTURED_CONTENT = { status: "received" };
 /**
 * Lets MCP server builders collect qualitative feedback from end users about their
 * tool/server. Injects a `send_feedback` tool at `tools/list` time and intercepts calls
@@ -36,10 +47,11 @@ function feedbackMiddleware(options) {
 					},
 					required: ["content", "source"]
 				},
+				outputSchema: FEEDBACK_OUTPUT_SCHEMA,
 				annotations: {
 					readOnlyHint: false,
 					destructiveHint: false,
-					openWorldHint: true,
+					openWorldHint: false,
 					idempotentHint: true
 				}
 			});
@@ -73,39 +85,39 @@ function feedbackMiddleware(options) {
 				type: "text",
 				text: FEEDBACK_RESPONSE_TEXT
 			}],
+			structuredContent: FEEDBACK_STRUCTURED_CONTENT,
 			_meta: { [metaKeyName]: feedback }
 		};
-		return { content: [{
-			type: "text",
-			text: FEEDBACK_RESPONSE_TEXT
-		}] };
+		return {
+			content: [{
+				type: "text",
+				text: FEEDBACK_RESPONSE_TEXT
+			}],
+			structuredContent: FEEDBACK_STRUCTURED_CONTENT
+		};
 	};
 }
 //#endregion
-//#region src/capture-feedback.ts
-const INSTALLED_MARKER$1 = "__alpicCaptureFeedbackInstalled";
+//#region src/install-capture-middleware.ts
 /**
-* Injects a `send_feedback` tool into a vanilla `@modelcontextprotocol/sdk`
-* server and captures feedback submissions. Accepts the high-level `McpServer`
-* or the low-level `Server` and patches the `tools/list` and `tools/call`
-* request handlers to surface captured feedback via `options.handler` (or,
-* when `ALPIC_FEEDBACK_META_KEY` is set, via the response `_meta`).
+* Patches the `tools/list` and `tools/call` request handlers of a vanilla
+* `@modelcontextprotocol/sdk` server (high-level `McpServer` or low-level
+* `Server`) to run the given middleware.
 *
 * Already-registered handlers are wrapped immediately; future registrations
-* are wrapped via a `Map.set` proxy so order of calls relative to
-* `registerTool` does not matter.
+* (e.g. tools added after this call) are wrapped via a `Map.set` proxy so order
+* of calls relative to `registerTool` does not matter.
 */
-const captureFeedback = (server, options) => {
+const installCaptureMiddleware = (server, { middleware, installedMarker, disabledWarning }) => {
 	const handlers = ("server" in server ? server.server : server)?._requestHandlers;
 	if (!(handlers instanceof Map)) {
-		console.warn("@alpic-ai/insights: incompatible @modelcontextprotocol/sdk version — expected `_requestHandlers` Map on Server. Feedback capture disabled.");
+		console.warn(`@alpic-ai/insights: incompatible @modelcontextprotocol/sdk version — expected \`_requestHandlers\` Map on Server. ${disabledWarning}`);
 		return;
 	}
 	const marked = handlers;
-	if (marked[INSTALLED_MARKER$1]) return;
-	marked[INSTALLED_MARKER$1] = true;
-	const middleware = feedbackMiddleware(options);
-	const targets = new Set(["tools/list", "tools/call"]);
+	if (marked[installedMarker]) return;
+	marked[installedMarker] = true;
+	const targets = /* @__PURE__ */ new Set(["tools/list", "tools/call"]);
 	const wrap = (method, handler) => {
 		if (!targets.has(method)) return handler;
 		return async (...args) => {
@@ -121,6 +133,22 @@ const captureFeedback = (server, options) => {
 	handlers.set = (method, handler) => originalSet(method, wrap(method, handler));
 };
 //#endregion
+//#region src/capture-feedback.ts
+/**
+* Injects a `send_feedback` tool into a vanilla `@modelcontextprotocol/sdk`
+* server and captures feedback submissions. Accepts the high-level `McpServer`
+* or the low-level `Server` and patches the `tools/list` and `tools/call`
+* request handlers to surface captured feedback via `options.handler` (or,
+* when `ALPIC_FEEDBACK_META_KEY` is set, via the response `_meta`).
+*/
+const captureFeedback = (server, options) => {
+	installCaptureMiddleware(server, {
+		middleware: feedbackMiddleware(options),
+		installedMarker: "__alpicCaptureFeedbackInstalled",
+		disabledWarning: "Feedback capture disabled."
+	});
+};
+//#endregion
 //#region src/intent-middleware.ts
 const USER_INTENT_FIELD = "user_intent";
 /**
@@ -228,42 +256,19 @@ Examples:
 }
 //#endregion
 //#region src/capture-intents.ts
-const INSTALLED_MARKER = "__alpicCaptureIntentsInstalled";
 /**
 * Captures the user's natural-language intent behind each tool call on a vanilla
 * `@modelcontextprotocol/sdk` server. Accepts the high-level `McpServer` or the
 * low-level `Server` and patches the `tools/list` and `tools/call` request
 * handlers to surface the captured intent via `options.handler` (or, when
 * `ALPIC_INTENT_META_KEY` is set, via the response `_meta`).
-*
-* Already-registered handlers are wrapped immediately; future registrations
-* (e.g. tools added after this call) are wrapped via a `Map.set` proxy so order
-* of calls relative to `registerTool` does not matter.
 */
 const captureIntents = (server, options) => {
-	const handlers = ("server" in server ? server.server : server)?._requestHandlers;
-	if (!(handlers instanceof Map)) {
-		console.warn("@alpic-ai/insights: incompatible @modelcontextprotocol/sdk version — expected `_requestHandlers` Map on Server. Prompt capture disabled.");
-		return;
-	}
-	const marked = handlers;
-	if (marked[INSTALLED_MARKER]) return;
-	marked[INSTALLED_MARKER] = true;
-	const middleware = intentMiddleware(options);
-	const targets = new Set(["tools/list", "tools/call"]);
-	const wrap = (method, handler) => {
-		if (!targets.has(method)) return handler;
-		return async (...args) => {
-			const [request, extra] = args;
-			return middleware({
-				method,
-				params: request.params ?? {}
-			}, extra, () => handler(...args));
-		};
-	};
-	for (const [method, handler] of [...handlers]) handlers.set(method, wrap(method, handler));
-	const originalSet = handlers.set.bind(handlers);
-	handlers.set = (method, handler) => originalSet(method, wrap(method, handler));
+	installCaptureMiddleware(server, {
+		middleware: intentMiddleware(options),
+		installedMarker: "__alpicCaptureIntentsInstalled",
+		disabledWarning: "Prompt capture disabled."
+	});
 };
 //#endregion
 export { captureFeedback, captureIntents, feedbackMiddleware, intentMiddleware };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@alpic-ai/insights",
-  "version": "0.0.0-dev.g1cd9b9b",
+  "version": "0.0.0-dev.g1d08792",
   "description": "User insights middlewares for Alpic-hosted MCP servers",
   "type": "module",
   "main": "./dist/index.mjs",
@@ -28,12 +28,12 @@
   "devDependencies": {
     "@modelcontextprotocol/sdk": "^1.29.0",
     "@total-typescript/tsconfig": "^1.0.4",
-    "@types/node": "^25.9.1",
+    "@types/node": "^25.9.5",
     "shx": "^0.4.0",
-    "skybridge": "^1.0.4",
-    "tsdown": "^0.22.1",
+    "skybridge": "^1.2.7",
+    "tsdown": "^0.22.5",
     "typescript": "^6.0.3",
-    "vitest": "^4.1.8",
+    "vitest": "^4.1.10",
     "zod": "^4.4.3"
   },
   "scripts": {
@@ -42,7 +42,7 @@
     "test": "pnpm run test:unit && pnpm run test:type && pnpm run test:format",
     "test:unit": "vitest run",
     "test:format": "biome check --error-on-warnings .",
-    "test:type": "tsc --noEmit",
+    "test:type": "tsgo --noEmit",
     "publish:npm": "pnpm publish --tag \"${NPM_TAG}\" --access public --no-git-checks"
   }
 }