@xano/xanoscript-language-server 11.11.0 → 11.12.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@xano/xanoscript-language-server",
-  "version": "11.11.0",
+  "version": "11.12.0",
   "description": "Language Server Protocol implementation for XanoScript",
   "type": "module",
   "main": "server.js",

package/parser/addon_parser.js

@@ -26,10 +26,11 @@ export function addonDeclaration($) {
     // Allow leading comments and newlines before the addon declaration
     $.SUBRULE($.optionalCommentBlockFn);
     const parent = $.CONSUME(AddonToken);
-    $.OR([
+    const nameToken = $.OR([
       { ALT: () => $.CONSUME(StringLiteral) },
       { ALT: () => $.CONSUME(Identifier) },
     ]);
+    $.validateDeclarationName(nameToken, "addon");
     $.CONSUME(LCurly); // "{"
     $.MANY(() => {
       $.AT_LEAST_ONE(() => $.CONSUME(NewlineToken)); // at least one new line

package/parser/agent_parser.js

@@ -21,10 +21,11 @@ export function agentDeclaration($) {
     // Allow leading comments and newlines before the agent declaration
     $.SUBRULE($.optionalCommentBlockFn);
     const parent = $.CONSUME(AgentToken); // agent
-    $.OR([
+    const nameToken = $.OR([
       { ALT: () => $.CONSUME(StringLiteral) },
       { ALT: () => $.CONSUME(Identifier) },
     ]);
+    $.validateDeclarationName(nameToken, "agent");
     $.CONSUME(LCurly); // "{"
     $.MANY(() => {
       $.AT_LEAST_ONE(() => $.CONSUME(NewlineToken)); // at least one new line

package/parser/agent_trigger_parser.js

@@ -26,10 +26,11 @@ export function agentTriggerDeclaration($) {
     // Allow leading comments and newlines before the agent_trigger declaration
     $.SUBRULE($.optionalCommentBlockFn);
     const parent = $.CONSUME(AgentTriggerToken); // agent_trigger
-    $.OR([
+    const nameToken = $.OR([
       { ALT: () => $.CONSUME(StringLiteral) },
       { ALT: () => $.CONSUME(Identifier) },
     ]);
+    $.validateDeclarationName(nameToken, "agent_trigger");
     $.CONSUME(LCurly); // "{"
     $.MANY(() => {
       $.AT_LEAST_ONE(() => $.CONSUME(NewlineToken)); // at least one new line

package/parser/api_group_parser.js

@@ -27,10 +27,11 @@ export function apiGroupDeclaration($) {
     // Allow leading comments and newlines before the api_group declaration
     $.SUBRULE($.optionalCommentBlockFn);
     $.CONSUME(ApiGroupToken); // api_group
-    $.OR([
+    const nameToken = $.OR([
       { ALT: () => $.CONSUME(StringLiteral) },
       { ALT: () => $.CONSUME(Identifier) },
     ]);
+    $.validateDeclarationName(nameToken, "api_group");
     $.CONSUME(LCurly); // "{"
     $.MANY(() => {
       $.AT_LEAST_ONE(() => $.CONSUME(NewlineToken));

package/parser/base_parser.js

@@ -270,6 +270,17 @@ export class XanoBaseParser extends CstParser {
     this.SAVE_ERROR(new MismatchedTokenException(message, token));
   }
 
+  /**
+   * Validate that a declaration name token is not an empty string
+   * @param {import('chevrotain').IToken} nameToken
+   * @param {string} typeName the declaration type (e.g. "query", "function")
+   */
+  validateDeclarationName(nameToken, typeName) {
+    if (nameToken?.image === '""') {
+      this.addInvalidValueError(nameToken, `${typeName} name must not be empty`);
+    }
+  }
+
   /**
    * Setup all the required rules for the parser but without creating duplicates
    *

package/parser/branch_parser.js

@@ -9,10 +9,11 @@ export function branchDeclaration($) {
     $.SUBRULE($.optionalCommentBlockFn);
 
     const parent = $.CONSUME(BranchToken); // branch
-    $.OR([
+    const nameToken = $.OR([
       { ALT: () => $.CONSUME(StringLiteral) },
       { ALT: () => $.CONSUME(Identifier) },
     ]);
+    $.validateDeclarationName(nameToken, "branch");
 
     const middlewareSchema = {
       pre: [{ name: "[string]" }],

package/parser/function_parser.js

@@ -29,6 +29,7 @@ export function functionDeclaration($) {
       { ALT: () => $.CONSUME(StringLiteral) },
     ]);
 
+    $.validateDeclarationName(nameToken, "function");
     if (nameToken?.image && getVarName(nameToken).startsWith("/")) {
       $.addInvalidValueError(
         nameToken,

package/parser/function_parser.spec.js

@@ -16,6 +16,20 @@ describe("function_parser", () => {
     expect(parser.errors).to.be.empty;
   });
 
+  it("should raise an error when a function name is empty", () => {
+    const parser = xanoscriptParser(`function "" {
+      input {
+      }
+
+      stack {
+      }
+
+      response = null
+    }`);
+    expect(parser.errors).to.not.be.empty;
+    expect(parser.errors[0].message).to.include("must not be empty");
+  });
+
   it("should raise an error when a function name starts with /", () => {
     const parser = xanoscriptParser(`function "/foo" {
       input {

package/parser/mcp_server_parser.js

@@ -24,10 +24,11 @@ export function mcpServerDeclaration($) {
     // Allow leading comments and newlines before the mcp_server declaration
     $.SUBRULE($.optionalCommentBlockFn);
     const parent = $.CONSUME(McpServerToken); // mcp_server
-    $.OR([
+    const nameToken = $.OR([
       { ALT: () => $.CONSUME(StringLiteral) },
       { ALT: () => $.CONSUME(Identifier) },
     ]);
+    $.validateDeclarationName(nameToken, "mcp_server");
     $.CONSUME(LCurly); // "{"
     $.MANY(() => {
       $.AT_LEAST_ONE(() => $.CONSUME(NewlineToken)); // at least one new line

package/parser/mcp_server_trigger_parser.js

@@ -25,10 +25,11 @@ export function mcpServerTriggerDeclaration($) {
     $.SUBRULE($.optionalCommentBlockFn);
 
     const parent = $.CONSUME(McpServerTriggerToken); // mcp_server_trigger
-    $.OR([
+    const nameToken = $.OR([
       { ALT: () => $.CONSUME(StringLiteral) },
       { ALT: () => $.CONSUME(Identifier) },
     ]);
+    $.validateDeclarationName(nameToken, "mcp_server_trigger");
     $.CONSUME(LCurly); // "{"
     $.MANY(() => {
       $.AT_LEAST_ONE(() => $.CONSUME(NewlineToken)); // at least one new line

package/parser/middleware_parser.js

@@ -28,10 +28,11 @@ export function middlewareDeclaration($) {
     $.SUBRULE($.optionalCommentBlockFn);
 
     const parent = $.CONSUME(MiddlewareToken);
-    $.OR([
+    const nameToken = $.OR([
       { ALT: () => $.CONSUME(Identifier) },
       { ALT: () => $.CONSUME(StringLiteral) },
     ]);
+    $.validateDeclarationName(nameToken, "middleware");
     $.CONSUME(LCurly); // "{"
     $.MANY(() => {
       $.AT_LEAST_ONE(() => $.CONSUME(NewlineToken)); // at least one new line

package/parser/query_parser.js

@@ -43,6 +43,7 @@ export function queryDeclaration($) {
       { ALT: () => $.CONSUME(Identifier) }, // foo
     ]);
 
+    $.validateDeclarationName(nameToken, "query");
     if (nameToken?.image && getVarName(nameToken).startsWith("/")) {
       $.addInvalidValueError(nameToken, "query name must not start with '/'");
     }

package/parser/query_parser.spec.js

@@ -16,6 +16,20 @@ describe("query_parser", () => {
     expect(parser.errors).to.be.empty;
   });
 
+  it("should raise an error when a query name is empty", () => {
+    const parser = xanoscriptParser(`query "" verb=GET {
+      input {
+      }
+
+      stack {
+      }
+
+      response = null
+    }`);
+    expect(parser.errors).to.not.be.empty;
+    expect(parser.errors[0].message).to.include("must not be empty");
+  });
+
   it("should raise an error when a query starts with /", () => {
     const parser = xanoscriptParser(`query "/foo" verb=GET {
       input {

package/parser/realtime_channel_parser.js

@@ -9,10 +9,11 @@ export function realtimeChannelDeclaration($) {
     $.SUBRULE($.optionalCommentBlockFn);
 
     const parent = $.CONSUME(RealtimeChannelToken); // realtime_channel
-    $.OR([
+    const nameToken = $.OR([
       { ALT: () => $.CONSUME(StringLiteral) },
       { ALT: () => $.CONSUME(Identifier) },
     ]);
+    $.validateDeclarationName(nameToken, "realtime_channel");
 
     $.SUBRULE($.schemaParseAttributeFn, {
       ARGS: [

package/parser/realtime_trigger_parser.js

@@ -25,10 +25,11 @@ export function realtimeTriggerDeclaration($) {
     $.SUBRULE($.optionalCommentBlockFn);
 
     const parent = $.CONSUME(RealtimeTriggerToken); // realtime_trigger
-    $.OR([
+    const nameToken = $.OR([
       { ALT: () => $.CONSUME(StringLiteral) },
       { ALT: () => $.CONSUME(Identifier) },
     ]);
+    $.validateDeclarationName(nameToken, "realtime_trigger");
     $.CONSUME(LCurly); // "{"
     $.MANY(() => {
       $.AT_LEAST_ONE(() => $.CONSUME(NewlineToken)); // at least one new line

package/parser/run_parser.js

@@ -25,10 +25,11 @@ export function runJobClause($) {
   return () => {
     const parent = $.CONSUME(JobToken); // "job"
 
-    $.OR([
+    const nameToken = $.OR([
       { ALT: () => $.CONSUME(StringLiteral) },
       { ALT: () => $.CONSUME(Identifier) },
     ]);
+    $.validateDeclarationName(nameToken, "job");
 
     $.SUBRULE($.schemaParseAttributeFn, {
       ARGS: [
@@ -49,10 +50,11 @@ export function runServiceClause($) {
   return () => {
     const parent = $.CONSUME(ServiceToken); // "service"
 
-    $.OR([
+    const nameToken = $.OR([
       { ALT: () => $.CONSUME(StringLiteral) },
       { ALT: () => $.CONSUME(Identifier) },
     ]);
+    $.validateDeclarationName(nameToken, "service");
 
     $.SUBRULE($.schemaParseAttributeFn, {
       ARGS: [

package/parser/table_parser.js

@@ -25,7 +25,7 @@ export function tableDeclaration($) {
     $.SUBRULE($.optionalCommentBlockFn);
 
     $.CONSUME(TableToken); // "table"
-    $.OR1([
+    const nameToken = $.OR1([
       {
         ALT: () =>
           $.CONSUME(StringLiteral, {
@@ -39,6 +39,7 @@ export function tableDeclaration($) {
           }),
       },
     ]);
+    $.validateDeclarationName(nameToken, "table");
 
     let hasSchema = false;
     let hasView = false;

package/parser/table_trigger_parser.js

@@ -30,10 +30,11 @@ export function tableTriggerDeclaration($) {
     $.SUBRULE($.optionalCommentBlockFn);
 
     const parent = $.CONSUME(TableTriggerToken); // table_trigger
-    $.OR([
+    const nameToken = $.OR([
       { ALT: () => $.CONSUME(StringLiteral) },
       { ALT: () => $.CONSUME(Identifier) },
     ]);
+    $.validateDeclarationName(nameToken, "table_trigger");
     $.CONSUME(LCurly); // "{"
     $.MANY(() => {
       $.AT_LEAST_ONE(() => $.CONSUME(NewlineToken)); // at least one new line

package/parser/task_parser.js

@@ -25,10 +25,11 @@ export function taskDeclaration($) {
     $.SUBRULE($.optionalCommentBlockFn);
 
     $.CONSUME(TaskToken);
-    $.OR([
+    const nameToken = $.OR([
       { ALT: () => $.CONSUME(StringLiteral) },
       { ALT: () => $.CONSUME(Identifier) },
     ]);
+    $.validateDeclarationName(nameToken, "task");
     $.CONSUME(LCurly); // "{"
     $.MANY(() => {
       $.AT_LEAST_ONE(() => $.CONSUME(NewlineToken)); // at least one new line

package/parser/task_parser.spec.js

@@ -13,6 +13,17 @@ describe("task_parser", () => {
     expect(parser.errors).to.be.empty;
   });
 
+  it("should raise an error when a task name is empty", () => {
+    const parser = xanoscriptParser(`task "" {
+  stack {
+  }
+
+  schedule = []
+}`);
+    expect(parser.errors).to.not.be.empty;
+    expect(parser.errors[0].message).to.include("must not be empty");
+  });
+
   it("should parse an active task", () => {
     const parser = xanoscriptParser(`task task_for_listing {
   active = true

package/parser/tool_parser.js

@@ -21,10 +21,11 @@ export function toolDeclaration($) {
     $.SUBRULE($.optionalCommentBlockFn);
 
     const parent = $.CONSUME(ToolToken);
-    $.OR([
+    const nameToken = $.OR([
       { ALT: () => $.CONSUME(StringLiteral) },
       { ALT: () => $.CONSUME(Identifier) },
     ]);
+    $.validateDeclarationName(nameToken, "tool");
     $.CONSUME(LCurly); // "{"
     $.MANY(() => {
       $.AT_LEAST_ONE(() => $.CONSUME(NewlineToken)); // at least one new line

package/parser/workflow_test_parser.js

@@ -16,10 +16,11 @@ export function workflowTestDeclaration($) {
     $.SUBRULE($.optionalCommentBlockFn);
 
     const parent = $.CONSUME(WorkflowTestToken);
-    $.OR([
+    const nameToken = $.OR([
       { ALT: () => $.CONSUME(StringLiteral) },
       { ALT: () => $.CONSUME(Identifier) },
     ]);
+    $.validateDeclarationName(nameToken, "workflow_test");
     $.CONSUME(LCurly); // "{"
     $.MANY(() => {
       $.AT_LEAST_ONE(() => $.CONSUME(NewlineToken)); // at least one new line

package/parser/workspace_parser.js

@@ -9,10 +9,11 @@ export function workspaceDeclaration($) {
     $.SUBRULE($.optionalCommentBlockFn);
 
     const parent = $.CONSUME(WorkspaceToken); // workspace
-    $.OR([
+    const nameToken = $.OR([
       { ALT: () => $.CONSUME(StringLiteral) },
       { ALT: () => $.CONSUME(Identifier) },
     ]);
+    $.validateDeclarationName(nameToken, "workspace");
 
     $.SUBRULE($.schemaParseAttributeFn, {
       ARGS: [

package/parser/workspace_trigger_parser.js

@@ -23,10 +23,11 @@ export function workspaceTriggerDeclaration($) {
     $.SUBRULE($.optionalCommentBlockFn);
 
     const parent = $.CONSUME(WorkspaceTriggerToken); // workspace_trigger
-    $.OR([
+    const nameToken = $.OR([
       { ALT: () => $.CONSUME(StringLiteral) },
       { ALT: () => $.CONSUME(Identifier) },
     ]);
+    $.validateDeclarationName(nameToken, "workspace_trigger");
     $.CONSUME(LCurly); // "{"
     $.MANY(() => {
       $.AT_LEAST_ONE(() => $.CONSUME(NewlineToken)); // at least one new line