@xano/xanoscript-language-server 11.11.0 → 11.12.1

package/parser/tests/query/query_parser.spec.js

@@ -9,13 +9,13 @@ describe("Xanoscript Query Parser", () => {
   it("parses all the valid sources", () => {
     const errors = [];
     const files = readdirSync(
-      join(dirname("./"), "parser/tests/query/valid_sources")
+      join(dirname("./"), "parser/tests/query/valid_sources"),
     );
 
     files.forEach((file) => {
       const source = readFileSync(
         join(dirname("./"), "parser/tests/query/valid_sources", file),
-        "utf8"
+        "utf8",
       );
       const parser = xanoscriptParser(source);
 
@@ -27,7 +27,7 @@ describe("Xanoscript Query Parser", () => {
                   error.previousToken.image,
                   {
                     length: 20,
-                  }
+                  },
                 )})`
               : "N/A",
             file: error.token

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
@@ -110,15 +111,24 @@ export function toolDeclaration($) {
     });
 
     if (!hasInput) {
-      $.addMissingError(parent, "{} is missing an input clause");
+      $.addMissingError(
+        parent,
+        `{} ${nameToken.image} is missing an input clause`,
+      );
     }
 
     if (!hasStack) {
-      $.addMissingError(parent, "{} is missing a stack clause");
+      $.addMissingError(
+        parent,
+        `{} ${nameToken.image} is missing a stack clause`,
+      );
     }
 
     if (!hasResponse) {
-      $.addMissingError(parent, "{} is missing a response clause");
+      $.addMissingError(
+        parent,
+        `{} ${nameToken.image} is missing a response clause`,
+      );
     }
 
     $.MANY1(() => $.CONSUME1(NewlineToken)); // optional new line

package/parser/tool_parser.spec.js

@@ -19,15 +19,28 @@ describe("tool_parser", () => {
 
   it("should parse a basic a string tool url", () => {
     const parser = xanoscriptParser(`tool "foo/bar" {
-      input {
-      }
+  input {
+  }
 
-      stack {
-      }
+  stack {
+  }
 
-      response = null
-    }`);
+  response = null
+}`);
+
+    expect(parser.errors).to.be.empty;
+  });
+
+  it("should parse a tool with a string as a response", () => {
+    const parser = xanoscriptParser(`tool silent_exception {
+  input {
+  }
+
+  stack {
+  }
 
+  response = "something is wrong"
+}`);
     expect(parser.errors).to.be.empty;
   });
 

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