@shko.online/dataverse-odata 0.1.3 → 0.1.5

package/.editorconfig

@@ -0,0 +1,8 @@
+root = true
+
+[*]
+end_of_line = lf
+
+[*.{js,json,ts,tsx,vue,svelte,html}]
+indent_style = space
+indent_size = 2

package/.github/workflows/publish.yml

@@ -0,0 +1,44 @@
+# This workflow will run tests using node and then publish a package to GitHub Packages when a release is created
+# For more information see: https://docs.github.com/en/actions/publishing-packages/publishing-nodejs-packages
+
+name: Semantic Release
+
+on:
+  push:
+    branches: [ main, beta, alpha ]
+
+permissions:
+  contents: read # for checkout
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+    permissions:
+      contents: write # to be able to publish a GitHub release
+      issues: write # to be able to comment on released issues
+      pull-requests: write # to be able to comment on released pull requests
+      id-token: write # to enable use of OIDC for trusted publishing and npm provenance
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v6
+        with:
+          fetch-depth: 0 # to be able to get all tags for semantic-release
+
+      - name: Setup Node.js
+        uses: actions/setup-node@v6
+        with:
+          node-version: "lts/*"
+
+      - name: Update npm
+        run: npm install -g npm@latest
+
+      - name: Install dependencies
+        run: npm ci
+
+      - name: Run tests
+        run: npm test
+
+      - name: Release
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+        run: npx semantic-release

package/.prettierrc.json

@@ -1,8 +1,8 @@
 {
-    "semi": true,
-    "trailingComma": "all",
-    "singleQuote": true,
-    "printWidth": 120,
-    "tabWidth": 4,
-    "endOfLine":"auto"    
-  }
+  "semi": true,
+  "trailingComma": "all",
+  "singleQuote": true,
+  "printWidth": 120,
+  "tabWidth": 4,
+  "endOfLine": "auto"
+}

package/.releaserc.yaml

@@ -1,23 +1,23 @@
-plugins:
-  - '@semantic-release/commit-analyzer'
-  - '@semantic-release/release-notes-generator'
-  - '@semantic-release/changelog'
-  - '@semantic-release/npm'
-  - '@semantic-release/exec'
-  - - '@semantic-release/git'
-    - assets:
-        - package.json
-        - CHANGELOG.md
-      message: |-
-        chore(release): ${nextRelease.version} [skip ci]
-
-        ${nextRelease.notes}
-branches:
-  - '+([0-9])?(.{+([0-9]),x}).x'
-  - main
-  - next
-  - next-major
-  - name: beta
-    prerelease: true
-  - name: alpha
-    prerelease: true
+plugins:
+  - '@semantic-release/commit-analyzer'
+  - '@semantic-release/release-notes-generator'
+  - '@semantic-release/changelog'
+  - '@semantic-release/npm'
+  - '@semantic-release/exec'
+  - - '@semantic-release/git'
+    - assets:
+        - package.json
+        - CHANGELOG.md
+      message: |-
+        chore(release): ${nextRelease.version} [skip ci]
+
+        ${nextRelease.notes}
+branches:
+  - '+([0-9])?(.{+([0-9]),x}).x'
+  - main
+  - next
+  - next-major
+  - name: beta
+    prerelease: true
+  - name: alpha
+    prerelease: true

package/CHANGELOG.md

@@ -1,3 +1,17 @@
+## [0.1.5](https://github.com/Shko-Online/dataverse-odata/compare/v0.1.4...v0.1.5) (2026-03-15)
+
+
+### Bug Fixes
+
+* Properly implemented $orderBy ([c693298](https://github.com/Shko-Online/dataverse-odata/commit/c693298bcc4cb2a96ebbc39939c60fb5abdd93d4))
+
+## [0.1.4](https://github.com/shko-online/dataverse-odata/compare/v0.1.3...v0.1.4) (2023-02-12)
+
+
+### Bug Fixes
+
+* added orderby support ([ea3748a](https://github.com/shko-online/dataverse-odata/commit/ea3748a5569d6ce2a086ff14ceb4a99421ecd069))
+
 ## [0.1.3](https://github.com/shko-online/dataverse-odata/compare/v0.1.2...v0.1.3) (2023-02-11)
 
 

package/LICENSE

@@ -0,0 +1,21 @@
+MIT License
+
+Copyright (c) 2022 Betim Beja and Shko Online LLC
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.

package/azure-pipelines.yml

@@ -0,0 +1,32 @@
+pool:
+  vmImage: ubuntu-latest
+
+steps:
+- task: NodeTool@0
+  inputs:
+    versionSpec: '24.x'
+  displayName: 'Install Node.js'
+
+- task: Npm@1
+  inputs:
+    command: 'install'
+  displayName: "Install NPM Dependencies"
+
+- task: Npm@1
+  inputs:
+    command: 'custom'
+    customCommand: 'test'
+  displayName: "Test"
+
+- task: Npm@1
+  inputs:
+    command: 'custom'
+    customCommand: 'run build'
+  displayName: "Build"
+
+- task: PublishCodeCoverageResults@2
+  displayName: "Publish code coverage results"
+  condition: succeededOrFailed()
+  inputs:
+    codeCoverageTool: Cobertura
+    summaryFileLocation: "coverage/cobertura-coverage.xml"

package/package.json

@@ -1,12 +1,13 @@
 {
   "name": "@shko.online/dataverse-odata",
-  "version": "0.1.3",
+  "version": "0.1.5",
   "description": "This package will help parse OData strings (only the Microsoft Dataverse subset). It can be used as a validator, or you can build some javascript library which consumes the output of this library.",
   "scripts": {
-    "build": "npm run lint && node ../scripts/build.js",
+    "build": "npm run lint && build-npm-package",
     "lint": "eslint",
     "tsc": "tsc --project tsconfig.build.json",
-    "test": "jest --coverage --maxWorkers=4",
+    "test": "jest",
+    "test:coverage": "jest --coverage",
     "semantic-release": "semantic-release"
   },
   "exports": {
@@ -26,7 +27,10 @@
       ]
     }
   },
-  "repository": "https://github.com/shko-online/dataverse-odata",
+  "repository": {
+    "type": "git",
+    "url": "https://github.com/Shko-Online/dataverse-odata"
+  },
   "keywords": [
     "odata",
     "dataverse",
@@ -55,24 +59,29 @@
     "@babel/preset-typescript": "^7.18.6",
     "@emotion/babel-plugin": "^11.10.5",
     "@jest/globals": "^29.4.2",
-    "@semantic-release/changelog": "^6.0.2",
-    "@semantic-release/commit-analyzer": "^9.0.2",
-    "@semantic-release/exec": "^6.0.3",
+    "@semantic-release/changelog": "^6.0.3",
+    "@semantic-release/commit-analyzer": "^13.0.1",
+    "@semantic-release/exec": "^7.1.0",
     "@semantic-release/git": "^10.0.1",
-    "@semantic-release/npm": "^9.0.2",
-    "@semantic-release/release-notes-generator": "^10.0.3",
+    "@semantic-release/release-notes-generator": "^14.1.0",
+    "@shko.online/build-npm-package": "^0.1.0",
     "@storybook/babel-plugin-require-context-hook": "^1.0.1",
+    "@types/node": "^24.12.0",
     "@typescript-eslint/eslint-plugin": "^5.40.1",
     "@typescript-eslint/parser": "^5.40.1",
     "babel-plugin-dynamic-import-node": "^2.3.3",
     "babel-plugin-macros": "^3.1.0",
     "copyfiles": "^2.4.1",
     "eslint": "^8.25.0",
+    "eslint-plugin-prettier": "^5.5.5",
+    "eslint-plugin-react": "^7.37.5",
     "jest": "^29.2.1",
     "jest-environment-jsdom": "^29.2.1",
-    "semantic-release": "^20.1.0",
+    "prettier": "^3.8.1",
+    "prettier-eslint": "^16.4.2",
+    "semantic-release": "^25.0.3",
     "ts-jest": "^29.0.3",
     "ts-node": "^10.9.1",
-    "typescript": "^4.8.4"
+    "typescript": "^5.9.3"
   }
 }

package/src/OData.types.d.ts

@@ -30,13 +30,15 @@ type ODataExpandQuery = ODataSelect & ODataExpand;
 
 interface ODataFilter {
     /**
-     * Use the {@link ODataFilter.$filter $filter} system query option to set criteria for which rows will be returned.
+     * Use the {@link $filter $filter} system query option to set criteria for which rows will be returned.
      *
      * * Microsoft Docs: {@link https://learn.microsoft.com/power-apps/developer/data-platform/webapi/query-data-web-api?WT.mc_id=DX-MVP-5004767#filter-results Filter results }
      */
-    $filter?: StandardOperator;
+    $filter?: FilterOperator;
 }
 
+type FilterOperator = StandardOperator | ColumnOperator | UnaryOperator | BinaryOperator | QueryFunctionOperator; 
+
 interface ODataFetch {
     /**
      * You can compose a FetchXML query for a specific table.
@@ -100,7 +102,20 @@ interface ODataUserQuery {
     userQuery?: string;
 }
 
-type StandardOperators = 'eq' | 'ne' | 'gt' | 'ge' | 'lt' | 'le';
+type StandardEqualityOperators = 'eq' | 'ne';
+
+type StandardOperators = StandardEqualityOperators | 'gt' | 'ge' | 'lt' | 'le';
+
+/**
+ * 
+ * * Microsoft Docs: {@link https://learn.microsoft.com/en-us/power-apps/developer/data-platform/webapi/query/filter-rows#column-comparison Column comparison }
+ */
+interface ColumnOperator {
+    column: string;
+    operator: StandardOperators;
+    otherColumn: string;
+}
+
 
 interface StandardOperator {
     operator: StandardOperators;
@@ -114,15 +129,29 @@ interface StandardOperator {
     right: string | number;
 }
 
+type QueryFunctionOperators = 'contains' | 'endswith' | 'startswith';
+
+interface QueryFunctionOperator {
+    operator: QueryFunctionOperators;
+    /**
+     * The left side of the 'X' operator must be a property of the entity.
+     */
+    left: string;
+    /**
+     * The right side of the 'X' operator must be a constant value.
+     */
+    right: string;
+}
+
 interface UnaryOperator {
     operator: 'not';
-    right: StandardOperator;
+    right: FilterOperator;
 }
 
 interface BinaryOperator {
     operator: 'and' | 'or';
-    left: StandardOperator;
-    right: StandardOperator;
+    left: FilterOperator;
+    right: FilterOperator;
 }
 
 type ODataQuery = ODataError &

package/src/getAliasedProperty.ts

@@ -0,0 +1,34 @@
+import { ODataQuery } from "./OData.types";
+
+/**
+ * Recursively gets the value of an aliased property. For example, if the query is `$orderby=@p1` and `@p1=name`, this function will return `name`
+ * @param parser The URLSearchParams object containing the query parameters
+ * @param result Will contain the error details in case there is any
+ * @param property The property to expand
+ * @returns The expanded property or null when there is an error
+ */
+export const getAliasedProperty = (parser: URLSearchParams, result: ODataQuery, property: string): string => {
+    let propertyName = parser.get(property);
+    if (!propertyName) {
+        result.error = {
+            code: '0x80060888',
+            message: 'Order By Property must be of type EdmProperty',
+        };
+
+        return null;
+    }
+
+    if (!/^[@a-zA-Z]\w+/gi.test(propertyName)) {
+        const position = propertyName.length;
+        result.error = {
+            code: '0x80060888',
+            message: `Syntax error at position ${position} in '${propertyName}'.`,
+        };
+        return null;
+    }
+
+    if (propertyName.startsWith('@')) {
+        return getAliasedProperty(parser, result, propertyName);
+    }
+    return propertyName;
+};

package/src/getExpandFromParser.ts

@@ -1,25 +1,33 @@
 import type { ODataError, ODataExpand, ODataExpandQuery, ODataQuery } from './OData.types';
 
 import { getSelectFromParser } from './getSelectFromParser';
+import { atMostOnce } from './validators/atMostOnce';
+
+const option = '$expand';
 
 /**
  * Parses the {@link ODataExpand.$expand $expand} query
  * @returns Returns `false` when the parse has an error
  */
 export const getExpandFromParser = (parser: URLSearchParams, result: ODataQuery): boolean => {
-    const $expand = parser.get('$expand');
-    if ($expand !== null) {
-        result.$expand = {};
+    const value = parser.getAll(option);
+    if (value.length === 0) {
+        return true;
+    }
+    if (!atMostOnce(option, value, result)) {
+        return false;
+    }
 
-        if (!extractExpand($expand, result)) {
-            return false;
-        }
+    result.$expand = {};
+    if (!extractExpand(value[0], result)) {
+        return false;
     }
+
     return true;
 };
 
 const extractExpand = (value: string, $expand: ODataExpand & ODataError) => {
-    const match = value.match(/^\s*(\w(\w|\d|_)*)\s*(,|\()?\s*/);
+    const match = value.match(/^\s*(\w(\w|\d|_)*)\s*(,|\(|\))?\s*/);
     if (
         match === null ||
         (match[0].length < value.length && match[3] === null) ||
@@ -27,7 +35,7 @@ const extractExpand = (value: string, $expand: ODataExpand & ODataError) => {
     ) {
         $expand.error = {
             code: '0x0',
-            message: 'invalid expand expression',
+            message: `Term '${value}' is not valid in a $select or $expand expression.`,
         };
         return false;
     }
@@ -59,14 +67,17 @@ const extractExpand = (value: string, $expand: ODataExpand & ODataError) => {
                 return false;
             }
             if (innerExpand.$expand === undefined && innerExpand.$select === undefined) {
-                $expand.error = { code: '0x0', message: 'Empty expand' };
+                $expand.error = {
+                    code: '0x0',
+                    message: `Missing expand option on navigation property '${match[1]}'. If a parenthesis expression follows an expanded navigation property, then at least one expand option must be provided.`,
+                };
                 return false;
             }
             $expand.$expand[match[1]] = innerExpand;
         }
 
         matchLength = matchLength + index;
-        const secondMatch = value.substring(matchLength + 1).match(/\s*(,?)\s*d/);
+        const secondMatch = new RegExp(/\s*(,?)\s*d/).exec(value.substring(matchLength + 1));
         if (secondMatch !== null) {
             matchLength = matchLength + secondMatch[0].length;
             if (secondMatch[1] !== null) {
@@ -90,7 +101,7 @@ const getClosingBracket = (value: string): { index: number; error?: string } =>
     while (depth > 0) {
         const match = value.substring(startAt).match(/\(|\)/);
         if (match === null) {
-            return { error: 'no closing bracket found', index: -1 };
+            return { error: 'Found an unbalanced bracket expression.', index: -1 };
         }
         if (match[0] === ')') {
             depth -= 1;
@@ -100,7 +111,7 @@ const getClosingBracket = (value: string): { index: number; error?: string } =>
         } else {
             depth += 1;
         }
-        startAt = (match.index || 0) + 1;
+        startAt += (match.index || 0) + 1;
     }
-    return { error: 'no closing bracket found', index: -1 };
+    return { error: 'Found an unbalanced bracket expression.', index: -1 };
 };

package/src/getFetchXmlFromParser.ts

@@ -1,56 +1,59 @@
 import type { ODataQuery, ODataFetch } from './OData.types';
+import { atMostOnce } from './validators/atMostOnce';
+import { differentFromEmptyString } from './validators/differentFromEmptyString';
+
+const option = 'fetchXml';
 
 /**
  * Parses the {@link ODataFetch.fetchXml fetchXml} query
  * @returns Returns `false` when the parse has an error
  */
 export const getFetchXmlFromParser = (parser: URLSearchParams, result: ODataQuery): boolean => {
-    const fetchXml = parser.get('fetchXml');
-    if (fetchXml !== null) {
-        if (fetchXml === '') {
-            result.error = {
-                code: '0x80040203',
-                message: 'Expected non-empty string.',
-            };
-            return false;
-        }
+    const value = parser.getAll(option);
+    if (value.length === 0) {
+        return true;
+    }
+    if (!atMostOnce(option, value, result) || !differentFromEmptyString(value, result)) {
+        return false;
+    }
 
-        const serializer = new DOMParser();
-        const fetchXmlDocument = serializer.parseFromString(fetchXml, 'text/xml');
-        if (fetchXmlDocument.documentElement.tagName === 'parsererror') {
-            result.error = {
-                code: '0x80040201',
-                message: 'Invalid XML.',
-            };
-            return false;
-        }
-        const entity = fetchXmlDocument
-            .evaluate('fetch/entity', fetchXmlDocument, null, XPathResult.ANY_TYPE, null)
-            .iterateNext() as Element;
-        if (fetchXmlDocument.documentElement.children.length != 1 || !entity || !entity.getAttribute('name')) {
-            result.error = {
-                code: '0x80041102',
-                message: 'Entity Name was not specified in FetchXml String.',
-            };
-            return false;
-        }
-        const invalidAttribute = fetchXmlDocument
-            .evaluate(
-                'fetch/entity/*[not(self::filter or self::order or self::link-entity or self::attribute or self::all-attributes or self::no-attrs)]',
-                fetchXmlDocument,
-                null,
-                XPathResult.ANY_TYPE,
-                null,
-            )
-            .iterateNext() as Element;
-        if (invalidAttribute) {
-            result.error = {
-                code: '0x8004111c',
-                message: `Invalid Child Node, valid nodes are filter, order, link-entity, attribute, all-attributes, no-attrs. NodeName = ${invalidAttribute.tagName} NodeXml = ${invalidAttribute.outerHTML}`,
-            };
-            return false;
-        }
-        result.fetchXml = fetchXmlDocument;
+    const fetchXml = value[0];
+    const serializer = new DOMParser();
+    const fetchXmlDocument = serializer.parseFromString(fetchXml, 'text/xml');
+    if (fetchXmlDocument.documentElement.tagName === 'parsererror') {
+        result.error = {
+            code: '0x80040201',
+            message: 'Invalid XML.',
+        };
+        return false;
     }
+    const entity = fetchXmlDocument
+        .evaluate('fetch/entity', fetchXmlDocument, null, XPathResult.ANY_TYPE, null)
+        .iterateNext() as Element;
+    if (fetchXmlDocument.documentElement.children.length != 1 || !entity || !entity.getAttribute('name')) {
+        result.error = {
+            code: '0x80041102',
+            message: 'Entity Name was not specified in FetchXml String.',
+        };
+        return false;
+    }
+    const invalidAttribute = fetchXmlDocument
+        .evaluate(
+            'fetch/entity/*[not(self::filter or self::order or self::link-entity or self::attribute or self::all-attributes or self::no-attrs)]',
+            fetchXmlDocument,
+            null,
+            XPathResult.ANY_TYPE,
+            null,
+        )
+        .iterateNext() as Element;
+    if (invalidAttribute) {
+        result.error = {
+            code: '0x8004111c',
+            message: `Invalid Child Node, valid nodes are filter, order, link-entity, attribute, all-attributes, no-attrs. NodeName = ${invalidAttribute.tagName} NodeXml = ${invalidAttribute.outerHTML}`,
+        };
+        return false;
+    }
+    result.fetchXml = fetchXmlDocument;
+
     return true;
 };

package/src/getFilterFromParser.ts

@@ -0,0 +1,23 @@
+import type { ODataQuery, ODataFilter } from './OData.types';
+import { atMostOnce } from './validators/atMostOnce';
+
+const option = '$filter';
+
+/**
+ * Parses the {@link ODataFilter.$filter $filter} query
+ * @returns {boolean} Returns `false` when the parse has an error
+ */
+export const getFilterFromParser = (parser: URLSearchParams, result: ODataQuery): boolean => {
+    const value = parser.getAll(option);
+    if (value.length === 0) {
+        return true;
+    }
+    if (!atMostOnce(option, value, result)) {
+        return false;
+    }
+    if (value.length > 0) {
+        
+        result.$filter = {operator: 'eq', left: '', right: ''};
+    }
+    return true;
+};