@hyperfixi/mcp-server 3.0.0 → 3.1.1

package/dist/index.js

@@ -2815,6 +2815,30 @@ try {
     "[mcp:lsp-bridge] @hyperfixi/core/multilingual not available \u2014 hover role inference limited to `set` and `go`"
   );
 }
+function parseToInterchange(code) {
+  if (!astToolkit?.fromCoreAST || !parseFunction) return null;
+  const result = parseFunction(code);
+  const errors = Array.isArray(result?.errors) ? result.errors : [];
+  const ast = result?.node ?? null;
+  if (!ast || ast.type === "identifier" && ast.name === "__ERROR__") return { nodes: [], errors };
+  try {
+    const opts = roleInferrer ? { inferRoles: roleInferrer } : void 0;
+    const statements = ast.type === "Program" && Array.isArray(ast.statements) ? ast.statements : [ast];
+    const interchange = statements.flatMap((stmt) => {
+      const converted = astToolkit.fromCoreAST(stmt, opts);
+      return Array.isArray(converted) ? converted : [converted];
+    });
+    const nodes = interchange.filter(
+      // `fromCoreAST` emits an `error` node for a kind it cannot represent
+      // (`def`, `behavior`, …) rather than throwing. That is a converter limit,
+      // not a user error: it must not become a diagnostic or a hover.
+      (n) => n?.type !== "error"
+    );
+    return { nodes, errors };
+  } catch {
+    return { nodes: [], errors };
+  }
+}
 console.error(
   `[mcp:lsp-bridge] capabilities: ast=${!!astToolkit}, parse=${!!parseFunction}, semantic=${!!semanticPackage2}, framework=${!!frameworkIR}, roles=${!!roleInferrer}`
 );
@@ -3040,12 +3064,25 @@ async function getDiagnostics(code, language, _uri) {
     } catch {
     }
   }
-  if (diagnostics.length === 0 && astToolkit && parseFunction) {
+  if (diagnostics.length === 0 && astToolkit?.interchangeToLSPDiagnostics && parseFunction) {
     try {
-      const ast = parseFunction(code);
-      if (ast && astToolkit.astToLSPDiagnostics) {
-        const astDiagnostics = astToolkit.astToLSPDiagnostics(ast);
-        diagnostics.push(...astDiagnostics);
+      const parsed = parseToInterchange(code);
+      for (const err of parsed?.errors ?? []) {
+        const line = Math.max(0, (err.line ?? 1) - 1);
+        const character = Math.max(0, err.column ?? 0);
+        diagnostics.push({
+          range: { start: { line, character }, end: { line, character: character + 10 } },
+          severity: 1,
+          // Error
+          code: "parse-error",
+          source: "hyperfixi-mcp",
+          message: err.message
+        });
+      }
+      if (parsed?.nodes.length) {
+        diagnostics.push(
+          ...astToolkit.interchangeToLSPDiagnostics(parsed.nodes, { source: "hyperfixi-mcp" })
+        );
       }
     } catch (parseError) {
       diagnostics.push({
@@ -3295,12 +3332,13 @@ function runSimpleDiagnostics(code, language = "en") {
 }
 async function getCompletions(code, line, character, language, context) {
   const completions = [];
-  if (astToolkit && parseFunction) {
+  if (astToolkit?.interchangeToLSPCompletions && parseFunction) {
     try {
-      const ast = parseFunction(code);
-      if (ast && astToolkit.astToLSPCompletions) {
-        const astCompletions = astToolkit.astToLSPCompletions(ast, { line, character });
-        completions.push(...astCompletions);
+      const parsed = parseToInterchange(code);
+      if (parsed?.nodes.length) {
+        completions.push(
+          ...astToolkit.interchangeToLSPCompletions(parsed.nodes, { line, character })
+        );
       }
     } catch {
     }
@@ -3457,13 +3495,9 @@ function inferContext(beforeCursor) {
 async function getHoverInfo(code, line, character, language) {
   if (astToolkit && parseFunction && astToolkit.fromCoreAST && astToolkit.interchangeToLSPHover) {
     try {
-      const ast = parseFunction(code);
-      if (ast) {
-        const interchange = astToolkit.fromCoreAST(
-          ast,
-          roleInferrer ? { inferRoles: roleInferrer } : void 0
-        );
-        const nodes = Array.isArray(interchange) ? interchange : [interchange];
+      const parsed = parseToInterchange(code);
+      if (parsed?.nodes.length) {
+        const nodes = parsed.nodes;
         const hoverOptions = {};
         if (frameworkIR) {
           hoverOptions.renderLSE = (node) => frameworkIR.renderExplicit(frameworkIR.fromInterchangeNode(node));
@@ -3586,12 +3620,11 @@ function getHoverDocumentation(word, language = "en") {
 }
 async function getDocumentSymbols(code, language) {
   const symbols = [];
-  if (astToolkit && parseFunction) {
+  if (astToolkit?.interchangeToLSPSymbols && parseFunction) {
     try {
-      const ast = parseFunction(code);
-      if (ast && astToolkit.astToLSPSymbols) {
-        const astSymbols = astToolkit.astToLSPSymbols(ast);
-        symbols.push(...astSymbols);
+      const parsed = parseToInterchange(code);
+      if (parsed?.nodes.length) {
+        symbols.push(...astToolkit.interchangeToLSPSymbols(parsed.nodes));
       }
     } catch {
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hyperfixi/mcp-server",
-  "version": "3.0.0",
+  "version": "3.1.1",
   "description": "Model Context Protocol server for HyperFixi hyperscript development",
   "type": "module",
   "main": "dist/index.js",
@@ -37,22 +37,22 @@
   "author": "LokaScript Contributors",
   "license": "MIT",
   "dependencies": {
-    "@hyperfixi/patterns-reference": "^3.0.0",
-    "@lokascript/compilation-service": "^3.0.0",
-    "@lokascript/domains": "^2.11.1",
-    "@lokascript/framework": "^3.0.0",
-    "@lokascript/planner": "^3.0.0",
-    "@lokascript/semantic": "^3.0.0",
+    "@hyperfixi/patterns-reference": "^3.1.1",
+    "@lokascript/compilation-service": "^3.1.1",
+    "@lokascript/domains": "^3.0.0",
+    "@lokascript/framework": "^3.1.1",
+    "@lokascript/planner": "^3.1.1",
+    "@lokascript/semantic": "^3.1.1",
     "@modelcontextprotocol/server": "^2.0.0",
     "yaml": "^2.8.3"
   },
   "peerDependencies": {
-    "@hyperfixi/core": "^3.0.0",
-    "@hyperfixi/patterns-reference": "^3.0.0",
-    "@hyperfixi/server-bridge": "^3.0.0",
-    "@lokascript/compilation-service": "^3.0.0",
-    "@lokascript/framework": "^3.0.0",
-    "@lokascript/semantic": "^3.0.0"
+    "@hyperfixi/core": "^3.1.1",
+    "@hyperfixi/patterns-reference": "^3.1.1",
+    "@hyperfixi/server-bridge": "^3.1.1",
+    "@lokascript/compilation-service": "^3.1.1",
+    "@lokascript/framework": "^3.1.1",
+    "@lokascript/semantic": "^3.1.1"
   },
   "peerDependenciesMeta": {
     "@lokascript/compilation-service": {
@@ -75,12 +75,12 @@
     }
   },
   "devDependencies": {
-    "@hyperfixi/core": "^3.0.0",
-    "@hyperfixi/patterns-reference": "^3.0.0",
-    "@hyperfixi/server-bridge": "^3.0.0",
-    "@lokascript/compilation-service": "^3.0.0",
-    "@lokascript/framework": "^3.0.0",
-    "@lokascript/semantic": "^3.0.0",
+    "@hyperfixi/core": "^3.1.1",
+    "@hyperfixi/patterns-reference": "^3.1.1",
+    "@hyperfixi/server-bridge": "^3.1.1",
+    "@lokascript/compilation-service": "^3.1.1",
+    "@lokascript/framework": "^3.1.1",
+    "@lokascript/semantic": "^3.1.1",
     "@types/node": "^26.1.2",
     "tsup": "^8.0.0",
     "tsx": "^4.23.1",