Release 1.0.9

.github/workflows/npm-publish.yml

@@ -10,15 +10,15 @@ jobs:
 
     steps:
       - name: Checkout repository
-        uses: actions/checkout@v2
+        uses: actions/checkout@v4
 
       - name: Use Node.js 20.x
-        uses: actions/setup-node@v3
+        uses: actions/setup-node@v4
         with:
           node-version: 20.x
 
       - name: Cache node modules
-        uses: actions/cache@v3
+        uses: actions/cache@v4
         env:
           cache-name: cache-node-modules
         with:

.github/workflows/unit_test_coverage.yml

@@ -21,15 +21,15 @@ jobs:
 
     steps:
       - name: Checkout repository
-        uses: actions/checkout@v2
+        uses: actions/checkout@v4
 
       - name: Use Node.js 20.x
-        uses: actions/setup-node@v3
+        uses: actions/setup-node@v4
         with:
           node-version: 20.x
 
       - name: Cache node modules
-        uses: actions/cache@v3
+        uses: actions/cache@v4
         env:
           cache-name: cache-node-modules
         with:
@@ -56,7 +56,7 @@ jobs:
         run: npm run-script coverage
 
       - name: Store the coverage report as an artifact
-        uses: actions/upload-artifact@v3
+        uses: actions/upload-artifact@v4
         with:
           name: coverage
           path: coverage/lcov.info
@@ -67,7 +67,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Download coverage artifact
-        uses: actions/download-artifact@v3
+        uses: actions/download-artifact@v4
         with:
           name: coverage
 
@@ -83,14 +83,15 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout repository
-        uses: actions/checkout@v3
+        uses: actions/checkout@v4
       - name: Download coverage artifact
-        uses: actions/download-artifact@v3
+        uses: actions/download-artifact@v4
         with:
           name: coverage
 
       - name: Upload code coverage to Codecov
-        uses: codecov/codecov-action@v3
+        uses: codecov/codecov-action@v4
         with:
           file: lcov.info
           fail_ci_if_error: true
+          token: ${{ secrets.CODECOV_TOKEN }} # required

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@madie/cql-antlr-parser",
-  "version": "1.0.7",
+  "version": "1.0.11",
   "description": "Antlr Parsing of CQL in typescript",
   "publishConfig": {
     "access": "public"
@@ -28,7 +28,7 @@
     "Fhir"
   ],
   "author": "SemanticBits [email protected]",
-  "license": "CREATIVE COMMONS ATTRIBUTION 4.0 INTERNATIONAL LICENSE",
+  "license": "ISC",
   "bugs": {
     "url": "https://github.com/MeasureAuthoringTool/cql-antlr-parser/issues"
   },

src/AntlrUtils.ts

@@ -18,29 +18,12 @@ export default class AntlrUtils {
   }
 
   static findChildName(children: ParseTree[] | undefined): string | undefined {
-
-    if (children?.length != 4) {
-      console.error(
-        "########### Entering.. children length is  ",
-        children?.length
-      );
-      throw new Error("Definition might be malformed.");
-
-    }
     return children ? children[1].text : undefined;
   }
 
   static findChildExpression(
     children: ParseTree[] | undefined
   ): string | undefined {
-
-    if (children?.length != 4) {
-      console.error(
-        "########### Entering.. children length is  ",
-        children?.length
-      )
-      throw new Error("Definition might be malformed.");
-    }
     return children ? children[3].text : undefined;
   }
 
@@ -85,11 +68,12 @@ export default class AntlrUtils {
    * @param comment -> a comment with comment characters
    */
   static formatComment(comment: string): string {
-    return comment.replace(AntlrUtils.SINGLE_LINE_COMMENT_REGEX, "")
+    return comment
+      .replace(AntlrUtils.SINGLE_LINE_COMMENT_REGEX, "")
       .replace(AntlrUtils.MULTI_LINE_COMMENT_REGEX, "")
       .split("\n")
       .map((line) => line.trim())
-      .filter(line=> line !== "")
+      .filter((line) => line !== "")
       .join("\n");
   }
 

src/CqlAntlr.ts

@@ -1,4 +1,4 @@
-import {BufferedTokenStream, CharStreams, CommonTokenStream} from "antlr4ts";
+import { BufferedTokenStream, CharStreams, CommonTokenStream } from "antlr4ts";
 import { CodePointCharStream } from "antlr4ts/CodePointCharStream";
 import { ParseTreeWalker } from "antlr4ts/tree";
 import { cqlLexer, cqlParser, LibraryContext, cqlListener } from "../generated";
@@ -18,9 +18,12 @@ class CqlAntlr {
     const charStream: CodePointCharStream = CharStreams.fromString(this.cql);
     const lexer: cqlLexer = new cqlLexer(charStream);
     const bufferedTokenStream = new BufferedTokenStream(lexer);
-    bufferedTokenStream.fill()
+    bufferedTokenStream.fill();
 
-    const listener: cqlListener = new CqlAntlrListener(result, bufferedTokenStream);
+    const listener: cqlListener = new CqlAntlrListener(
+      result,
+      bufferedTokenStream
+    );
     ParseTreeWalker.DEFAULT.walk(listener, tree);
     /**
      * Disabled. Only partially implemented and would be

src/CqlAntlrListener.ts

@@ -6,6 +6,7 @@ import {
   cqlLexer,
   cqlListener,
   ExpressionDefinitionContext,
+  FunctionDefinitionContext,
   IncludeDefinitionContext,
   LibraryDefinitionContext,
   ParameterDefinitionContext,
@@ -27,13 +28,14 @@ import CqlParameter from "./dto/CqlParameter";
 import CqlContextCreator from "./CqlContextCreator";
 import CqlExpressionDefinition from "./dto/CqlExpressionDefinition";
 import CqlExpressionDefinitionCreator from "./CqlExpressionDefinitionCreator";
-import {CqlCode} from "./dto";
+import { CqlCode } from "./dto";
 import CqlIdentifier from "./dto/CqlIdentifier";
 import CqlIdentifierCreator from "./CqlIdentifierCreator";
 import CqlRetrieve from "./dto/CqlRetrieve";
 import CqlRetrieveCreator from "./CqlRetrieveCreator";
-import {BufferedTokenStream} from "antlr4ts";
+import { BufferedTokenStream } from "antlr4ts";
 import AntlrUtils from "./AntlrUtils";
+import { ParserRuleContext } from "antlr4ts/ParserRuleContext";
 
 export default class CqlAntlrListener implements cqlListener {
   // save bufferedTokenStream from lexer
@@ -107,30 +109,48 @@ export default class CqlAntlrListener implements cqlListener {
     this.cqlResult.context = new CqlContextCreator(ctx).buildDao();
   }
 
-  enterExpressionDefinition(ctx: ExpressionDefinitionContext): void {
-    const cqlExpressionCreator = new CqlExpressionDefinitionCreator(ctx);
-    const expressionDefinition: CqlExpressionDefinition | undefined = cqlExpressionCreator.buildDao();
-
-    if (expressionDefinition) {
-      if (ctx.start.inputStream) {
-        const hiddenTokens = this.bufferedTokenStream.getHiddenTokensToLeft(ctx.start.tokenIndex, cqlLexer.HIDDEN)
-        let comment = "";
-        hiddenTokens.forEach((token) => {
-          if (token.text){
-            comment += token.text;
-          }
-        })
-        comment = comment.trim();
-        if (comment){
-          // if expression has comment, start needs to be adjusted to consider comments as comment is part of definition
-          expressionDefinition.start = cqlExpressionCreator.buildLineInfo(hiddenTokens[1]);
+  private processDefinitionWithComments(
+    ctx: ParserRuleContext,
+    buildDao: () => CqlExpressionDefinition | undefined
+  ): void {
+    const expressionDefinition: CqlExpressionDefinition | undefined =
+      buildDao();
+
+    if (expressionDefinition && ctx.start.inputStream) {
+      const hiddenTokens = this.bufferedTokenStream.getHiddenTokensToLeft(
+        ctx.start.tokenIndex,
+        cqlLexer.HIDDEN
+      );
+
+      if (hiddenTokens && hiddenTokens.length > 0) {
+        const comment = hiddenTokens
+          .map((token) => token.text?.trim())
+          .filter(Boolean)
+          .join(" ");
+
+        if (comment) {
+          expressionDefinition.start = new CqlExpressionDefinitionCreator(
+            ctx
+          ).buildLineInfo(hiddenTokens[1]);
           expressionDefinition.comment = AntlrUtils.formatComment(comment);
         }
       }
       this.cqlResult.expressionDefinitions.push(expressionDefinition);
     }
   }
 
+  enterExpressionDefinition(ctx: ExpressionDefinitionContext): void {
+    this.processDefinitionWithComments(ctx, () => {
+      return new CqlExpressionDefinitionCreator(ctx).buildDao();
+    });
+  }
+
+  enterFunctionDefinition(ctx: FunctionDefinitionContext): void {
+    this.processDefinitionWithComments(ctx, () => {
+      return new CqlExpressionDefinitionCreator(ctx).buildDao();
+    });
+  }
+
   enterAggregateClause(ctx: AggregateClauseContext): void {
     const identifier: CqlIdentifier | undefined = new CqlIdentifierCreator(
       ctx

src/CustomErrorListener.ts

@@ -3,6 +3,7 @@ import { Recognizer } from "antlr4ts/Recognizer";
 import { Token } from "antlr4ts/Token";
 import { ParserATNSimulator } from "antlr4ts/atn/ParserATNSimulator";
 import CqlResult from "./dto/CqlResult";
+import convertCustomError from "./util/CustomeErrorConverter";
 
 /**
  * Fires on grammar errors.
@@ -32,7 +33,7 @@ export default class CustomErrorListener implements ANTLRErrorListener<Token> {
             1, // plus 1 to ensure full text is included in Ace Editor highlight
         },
         name: offendingSymbol.text,
-        message: msg,
+        message: convertCustomError(msg),
       });
     }
   }