feat: add DOM Parser

Gruntfile.cjs

@@ -39,7 +39,6 @@ module.exports = function (grunt) {
             default: {
                 files: {
                     "<%= properties.webappBuildDir %>/libs/": [
-                        "sax:main",
                         "filesaver.js-npm:main",
                         "jszip/dist/jszip.js",
                     ],
@@ -80,6 +79,4 @@ module.exports = function (grunt) {
 
     grunt.registerTask("build", ["build:debug"]);
 
-    grunt.registerTask("clean", ["clean"]);
-
 };

package.json

@@ -25,9 +25,26 @@
         "imsc"
     ],
     "author": "Pierre-Anthony Lemieux <pal@sandflow.com>",
-    "main": "dist/main/js/main.js",
-    "types": "dist/main/js/main.d.ts",
+    "main": "dist/main/main.js",
+    "types": "dist/main/main.d.ts",
     "unpkg": "dist/imsc.min.js",
+    "sideEffects": false,
+    "exports": {
+        ".": {
+            "types": "./dist/main/main.d.ts",
+            "default": "./dist/main/main.js"
+        },
+        "./*.js": {
+            "types": "./dist/main/*.d.ts",
+            "default": "./dist/main/*.js"
+        },
+        "./*": {
+            "types": "./dist/main/*.d.ts",
+            "default": "./dist/main/*.js"
+        },
+        "./dist/*": "./dist/*",
+        "./package.json": "./package.json"
+    },
     "scripts": {
         "prepublishOnly": "grunt build:release",
         "dev": "npx http-server build/public_html",
@@ -37,14 +54,13 @@
         "test": "node --test ./src/test/js/*Test.js"
     },
     "dependencies": {
-        "sax": "^1.6.1"
+        "tslib": "^2.6.2"
     },
     "devDependencies": {
         "@eslint/js": "^9.39.5",
         "@rollup/plugin-commonjs": "^29.0.3",
         "@rollup/plugin-node-resolve": "^16.0.3",
         "@rollup/plugin-terser": "^1.0.0",
-        "@types/sax": "^1.2.7",
         "eslint": "^9.39.5",
         "filesaver.js-npm": "latest",
         "globals": "^17.11.0",
@@ -56,7 +72,7 @@
         "jszip": "latest",
         "rollup": "^4.14.3",
         "rollup-plugin-polyfill-node": "^0.13.0",
-        "tslib": "^2.6.2",
+        "sax": "^1.6.1",
         "typescript": "^5.4.5"
     }
 }

properties.json

@@ -4,8 +4,6 @@
     "webappBuildDir" : "build/public_html",
     "unitTestsResourcesDir" : "src/test/resources",
     "umdBuildDir" : "dist",
-    "umdAllDebugName" : "imsc.all.debug.js",
     "umdDebugName" : "imsc.debug.js",
-    "umdAllMinName" : "imsc.all.min.js",
     "umdMinName" : "imsc.min.js"
 }

rollup.config.js

@@ -4,10 +4,8 @@ import terser from "@rollup/plugin-terser";
 import nodePolyfill from "rollup-plugin-polyfill-node";
 
 export default [
-  // debug
   {
     input: "dist/main/main.js",
-    external: ["sax"],
     plugins: [
       commonjs({
         sourceMap: true,
@@ -23,44 +21,11 @@ export default [
       format: "umd",
       name: "imsc",
       sourcemap: true,
-      globals: {
-        sax: "sax",
-      },
     }, {
       file: "dist/imsc.min.js",
       format: "umd",
       name: "imsc",
       sourcemap: true,
-      globals: {
-        sax: "sax",
-      },
-      plugins: [terser()],
-    }],
-  },
-
-  // all
-  {
-    input: "dist/main/main.js",
-    plugins: [
-      commonjs({
-        sourceMap: true,
-      }),
-      nodeResolve({
-        preferBuiltins: true,
-        browser: true,
-      }),
-      nodePolyfill(),
-    ],
-    output: [{
-      file: "dist/imsc.all.debug.js",
-      format: "umd",
-      name: "imsc",
-      sourcemap: true,
-    }, {
-      file: "dist/imsc.all.min.js",
-      format: "umd",
-      name: "imsc",
-      sourcemap: true,
       plugins: [terser()],
     }],
   },

src/main/js/doc.js

@@ -24,7 +24,6 @@
  * POSSIBILITY OF SUCH DAMAGE.
  */
 
-import sax from "sax";
 import { reportError, reportFatal, reportWarning } from "./error.js";
 import { ns_ebutts, ns_ittp, ns_itts, ns_tt, ns_ttp, ns_tts } from "./names.js";
 import { byName, byQName } from "./styles.js";
@@ -36,10 +35,8 @@ import { ComputedLength, hasOwnProperty, parseLength } from "./utils.js";
 
 /**
  * @typedef {import("./error").ErrorHandler} ErrorHandler
- */
-
-/**
- * @typedef {sax.Tag | sax.QualifiedTag} Node
+ * @typedef {import("./parser").Node} Node
+ * @typedef {import("./parser").Parser} Parser
  */
 
 /**
@@ -80,18 +77,22 @@ import { ComputedLength, hasOwnProperty, parseLength } from "./utils.js";
  * @param {string} xmlstring XML document
  * @param {ErrorHandler} errorHandler Error callback
  * @param {?MetadataHandler} metadataHandler Callback for <Metadata> elements
+ * @param {Parser} parser XML parser
  * @returns {?TT} Opaque in-memory representation of an IMSC1 document
  */
 
-export function fromXML(xmlstring, errorHandler, metadataHandler) {
-    const p = sax.parser(true, { xmlns: true });
+export function fromParser(xmlstring, errorHandler, metadataHandler, parser) {
+    if (!parser) {
+        reportFatal(errorHandler, "No parser provided");
+    }
+
     const estack = [];
     const xmllangstack = [];
     const xmlspacestack = [];
     let metadata_depth = 0;
     let doc = null;
 
-    p.onclosetag = function () {
+    parser.onclosetag = function () {
 
         if (estack[0] instanceof Region) {
 
@@ -191,7 +192,7 @@ export function fromXML(xmlstring, errorHandler, metadataHandler) {
         estack.shift();
     };
 
-    p.ontext = function (str) {
+    parser.ontext = function (str) {
 
         if (estack[0] === undefined) {
 
@@ -234,7 +235,7 @@ export function fromXML(xmlstring, errorHandler, metadataHandler) {
 
     };
 
-    p.onopentag = function (node) {
+    parser.onopentag = function (node) {
 
         // maintain the xml:space stack
 
@@ -604,7 +605,7 @@ export function fromXML(xmlstring, errorHandler, metadataHandler) {
 
     // parse the document
 
-    p.write(xmlstring).close();
+    parser.write(xmlstring).close();
 
     // all referential styling has been flatten, so delete styles