@holin-work/holin-cli 1.3.8 → 1.3.9

package/bin/holin-cli.js

@@ -268,10 +268,26 @@ function createProxyServer() {
       return;
     }
 
-    // Build upstream request
-    // MCP streamable-http has a single endpoint; forward directly to upstream path.
-    // Accio requests http://127.0.0.1:18787/mcp → upstream https://api.holin.work/icbu/mcp
-    const upstreamUrl = new URL(UPSTREAM_URL);
+    // Build upstream request based on path
+    // /mcp → https://api.holin.work/icbu/mcp (MCP protocol)
+    // /api/* → https://api.holin.work/api/* (platform API, e.g. file upload)
+    const upstreamBase = "https://api.holin.work";
+    let upstreamPath;
+    let isMcpRequest = false;
+    if (req.url.startsWith("/mcp")) {
+      upstreamPath = "/icbu/mcp";
+      isMcpRequest = true;
+    } else if (req.url.startsWith("/api/")) {
+      upstreamPath = req.url;
+    } else {
+      // Unknown path
+      const body = JSON.stringify({ error: "not_found", message: `Unknown path: ${req.url}` });
+      res.writeHead(404, { "Content-Type": "application/json", "Content-Length": Buffer.byteLength(body) });
+      res.end(body);
+      return;
+    }
+
+    const upstreamUrl = new URL(upstreamBase);
     const bodyBuffer = Buffer.concat(chunks);
     const forwardHeaders = { ...req.headers };
     delete forwardHeaders["transfer-encoding"];
@@ -282,25 +298,28 @@ function createProxyServer() {
     const options = {
       hostname: upstreamUrl.hostname,
       port: upstreamUrl.port || 443,
-      path: upstreamUrl.pathname,
+      path: upstreamPath,
       method: req.method,
       headers: forwardHeaders,
     };
 
-    // Parse request body to determine MCP method
+    // Parse request body to determine MCP method (only for MCP requests)
     let mcpMethod = "";
-    try {
-      const parsed = JSON.parse(rawBody);
-      mcpMethod = parsed.method || "";
-    } catch { /* not JSON */ }
+    if (isMcpRequest) {
+      try {
+        const parsed = JSON.parse(rawBody);
+        mcpMethod = parsed.method || "";
+      } catch { /* not JSON */ }
+    }
 
-    // Non-streaming methods should return JSON, not SSE
+    // Non-streaming methods should return JSON, not SSE (only for MCP requests)
     // Accio expects JSON for initialize and tools/list
-    const shouldConvertToJson = mcpMethod === "initialize" || mcpMethod === "tools/list";
+    const shouldConvertToJson = isMcpRequest && (mcpMethod === "initialize" || mcpMethod === "tools/list");
 
     const proxyReq = https.request(options, (proxyRes) => {
       const elapsed = Date.now() - requestStart;
-      log(`[proxy] ← ${proxyRes.statusCode} ${elapsed}ms content-type=${proxyRes.headers["content-type"]} method=${mcpMethod}`);
+      const routeLabel = isMcpRequest ? `mcp/${mcpMethod || "unknown"}` : req.url;
+      log(`[proxy] ← ${proxyRes.statusCode} ${elapsed}ms content-type=${proxyRes.headers["content-type"]} route=${routeLabel}`);
 
       if (shouldConvertToJson && proxyRes.headers["content-type"]?.includes("text/event-stream")) {
         // Collect SSE response and convert to JSON

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@holin-work/holin-cli",
-  "version": "1.3.8",
+  "version": "1.3.9",
   "description": "Holin CLI — Accio plugin authorization tool for ICBU batch listing",
   "type": "module",
   "bin": {