@alpic-ai/insights 0.0.0-dev.g279a914 → 0.0.0-dev.g27cc698

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

@@ -49,9 +49,9 @@ function feedbackMiddleware(options) {
 				},
 				outputSchema: FEEDBACK_OUTPUT_SCHEMA,
 				annotations: {
-					readOnlyHint: false,
+					readOnlyHint: true,
 					destructiveHint: false,
-					openWorldHint: true,
+					openWorldHint: false,
 					idempotentHint: true
 				}
 			});
@@ -98,30 +98,26 @@ function feedbackMiddleware(options) {
 	};
 }
 //#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) => {
@@ -137,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";
 /**
@@ -244,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.g279a914",
+  "version": "0.0.0-dev.g27cc698",
   "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.3",
+    "@types/node": "^25.9.5",
     "shx": "^0.4.0",
-    "skybridge": "^1.1.0",
-    "tsdown": "^0.22.2",
+    "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"
   }
 }