@qooxdoo/framework 7.0.0-beta.9 → 7.1.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@qooxdoo/framework",
-  "version": "7.0.0-beta.9",
+  "version": "7.1.1",
   "description": "The JS Framework for Coders",
   "author": "The qooxdoo project",
   "keywords": [
@@ -36,7 +36,7 @@
   "scripts": {
     "devtools": "node source/resource/qx/tool/bin/build-devtools",
     "website": "node source/resource/qx/tool/bin/build-website",
-    "pretest": "node bootstrap-compiler --target build",
+    "pretest": "node ./bootstrap-compiler",
     "test": "node ./bin/build/qx test --colorize=false --quiet",
     "prepack": "node ./bootstrap/qx deploy --clean"
   },
@@ -51,7 +51,7 @@
     "bin/deploy"
   ],
   "dependencies": {
-    "@babel/core": "^7.12.10",
+    "@babel/core": "^7.17.10",
     "@babel/eslint-parser": "^7.16.3",
     "@babel/generator": "^7.12.11",
     "@babel/parser": "^7.12.11",
@@ -60,7 +60,7 @@
     "@babel/plugin-syntax-jsx": "^7.16.7",
     "@babel/plugin-transform-block-scoping": "^7.12.12",
     "@babel/plugin-transform-react-jsx": "^7.12.12",
-    "@babel/preset-env": "^7.12.11",
+    "@babel/preset-env": "^7.17.10",
     "@babel/preset-react": "^7.12.10",
     "@babel/preset-typescript": "^7.12.7",
     "@babel/traverse": "^7.12.12",
@@ -73,7 +73,9 @@
     "ajv": "^8.8.1",
     "app-module-path": "^2.2.0",
     "async": "^2.6.3",
+    "babelify": "^10.0.0",
     "better-ajv-errors": "^1.1.1",
+    "browserify": "^17.0.0",
     "chokidar": "^3.5.1",
     "cldr": "^7.1.1",
     "columnify": "^1.5.4",

package/source/class/qx/Mixin.js

@@ -370,6 +370,12 @@ qx.Bootstrap.define("qx.Mixin", {
           // Try looking in the class itself
           if (!fn && mixedInAt.prototype[methodName]) {
             fn = mixedInAt.prototype[methodName].base;
+            // if fn.self is set fn is an overloaded mixin method from
+            // another mixin. In this case fn.base contains the original
+            // class method.
+            if (fn && fn.self) {
+              fn = fn.base;
+            }
           }
           // Try looking in the superclass
           if (!fn && mixedInAt.superclass) {

package/source/class/qx/event/handler/Focus.js

@@ -88,7 +88,7 @@ qx.Class.define("qx.event.handler.Focus", {
       nullable: true
     },
 
-    /** The focussed DOM element */
+    /** The focused DOM element */
     focus: {
       apply: "_applyFocus",
       nullable: true

package/source/class/qx/test/Mixin.js

@@ -574,8 +574,39 @@ qx.Class.define("qx.test.Mixin", {
       this.assertEquals("A MA B MB C MC", o.sayJuhu());
       o.dispose();
 
-    }
+    },
+
+    testDoubleMixin() {
+      qx.Class.define("qx.D", {
+        extend: qx.core.Object,
+        members: {
+          sayJuhu() {
+            return "Double";
+          }
+        }
+      });
+      qx.Mixin.define("qx.MDA", {
+        members: {
+          sayJuhu() {
+            return super.sayJuhu() + " MA";
+          },
+        }
+      });
+      qx.Mixin.define("qx.MDB", {
+        members: {
+          sayJuhu() {
+            return super.sayJuhu() + " MB";
+          },
+        }
+      });
+      qx.Class.patch(qx.D, qx.MDA);
+      qx.Class.patch(qx.D, qx.MDB);
 
+      var o = new qx.D();
+      this.assertEquals("Double MA MB", o.sayJuhu());
+      o.dispose();
+
+    }
 
    }
 });

package/source/class/qx/theme/iconfont/LoadMaterialIcons.js

@@ -20,9 +20,9 @@
  * A dummy class to trigger the compiler to copy the MaterialIcons font files
  */
 /**
- * @asset(qx/iconfont/MaterialIcons/materialicons-v70.ttf)
- * @asset(qx/iconfont/MaterialIcons/materialicons-v70.woff2)
- * @asset(qx/iconfont/MaterialIcons/materialicons-v70.woff)
- * @asset(qx/iconfont/MaterialIcons/materialicons-v70.eot)
+ * @asset(qx/iconfont/MaterialIcons/materialicons-v126.ttf)
+ * @asset(qx/iconfont/MaterialIcons/materialicons-v126.woff2)
+ * @asset(qx/iconfont/MaterialIcons/materialicons-v126.woff)
+ * @asset(qx/iconfont/MaterialIcons/materialicons-v126.eot)
  */
 qx.Class.define("qx.theme.iconfont.LoadMaterialIcons", {});

package/source/class/qx/theme/iconfont/LoadMaterialIconsOutlined.js

@@ -20,9 +20,9 @@
  * A dummy class to trigger the compiler to copy the MaterialIconsOutlined font files
  */
 /**
- * @asset(qx/iconfont/MaterialIcons/materialiconsoutlined-v38.otf)
- * @asset(qx/iconfont/MaterialIcons/materialiconsoutlined-v38.woff2)
- * @asset(qx/iconfont/MaterialIcons/materialiconsoutlined-v38.woff)
- * @asset(qx/iconfont/MaterialIcons/materialiconsoutlined-v38.eot)
+ * @asset(qx/iconfont/MaterialIcons/materialiconsoutlined-v101.otf)
+ * @asset(qx/iconfont/MaterialIcons/materialiconsoutlined-v101.woff2)
+ * @asset(qx/iconfont/MaterialIcons/materialiconsoutlined-v101.woff)
+ * @asset(qx/iconfont/MaterialIcons/materialiconsoutlined-v101.eot)
  */
 qx.Class.define("qx.theme.iconfont.LoadMaterialIconsOutlined", {});

package/source/class/qx/theme/iconfont/LoadMaterialIconsRound.js

@@ -20,9 +20,9 @@
  * A dummy class to trigger the compiler to copy the MaterialIconsRound font files
  */
 /**
- * @asset(qx/iconfont/MaterialIcons/materialiconsround-v37.otf)
- * @asset(qx/iconfont/MaterialIcons/materialiconsround-v37.woff2)
- * @asset(qx/iconfont/MaterialIcons/materialiconsround-v37.woff)
- * @asset(qx/iconfont/MaterialIcons/materialiconsround-v37.eot)
+ * @asset(qx/iconfont/MaterialIcons/materialiconsround-v100.otf)
+ * @asset(qx/iconfont/MaterialIcons/materialiconsround-v100.woff2)
+ * @asset(qx/iconfont/MaterialIcons/materialiconsround-v100.woff)
+ * @asset(qx/iconfont/MaterialIcons/materialiconsround-v100.eot)
  */
 qx.Class.define("qx.theme.iconfont.LoadMaterialIconsRound", {});

package/source/class/qx/theme/iconfont/LoadMaterialIconsSharp.js

@@ -20,9 +20,9 @@
  * A dummy class to trigger the compiler to copy the MaterialIconsSharpe font files
  */
 /**
- * @asset(qx/iconfont/MaterialIcons/materialiconssharp-v38.otf)
- * @asset(qx/iconfont/MaterialIcons/materialiconssharp-v38.woff2)
- * @asset(qx/iconfont/MaterialIcons/materialiconssharp-v38.woff)
- * @asset(qx/iconfont/MaterialIcons/materialiconssharp-v38.eot)
+ * @asset(qx/iconfont/MaterialIcons/materialiconssharp-v101.otf)
+ * @asset(qx/iconfont/MaterialIcons/materialiconssharp-v101.woff2)
+ * @asset(qx/iconfont/MaterialIcons/materialiconssharp-v101.woff)
+ * @asset(qx/iconfont/MaterialIcons/materialiconssharp-v101.eot)
  */
 qx.Class.define("qx.theme.iconfont.LoadMaterialIconsSharp", {});

package/source/class/qx/theme/iconfont/LoadMaterialIconsTwoTone.js

@@ -20,9 +20,9 @@
  * A dummy class to trigger the compiler to copy the MaterialIconsTwoTone font files
  */
 /**
- * @asset(qx/iconfont/MaterialIcons/materialiconstwotone-v36.otf)
- * @asset(qx/iconfont/MaterialIcons/materialiconstwotone-v36.woff2)
- * @asset(qx/iconfont/MaterialIcons/materialiconstwotone-v36.woff)
- * @asset(qx/iconfont/MaterialIcons/materialiconstwotone-v36.eot)
+ * @asset(qx/iconfont/MaterialIcons/materialiconstwotone-v104.otf)
+ * @asset(qx/iconfont/MaterialIcons/materialiconstwotone-v104.woff2)
+ * @asset(qx/iconfont/MaterialIcons/materialiconstwotone-v104.woff)
+ * @asset(qx/iconfont/MaterialIcons/materialiconstwotone-v104.eot)
  */
 qx.Class.define("qx.theme.iconfont.LoadMaterialIconsTwoTone", {});

package/source/class/qx/theme/tangible/ColorEngine.js

@@ -87,7 +87,7 @@ var helper = {
     var color = d[0];
     var state = d[1];
     switch (state) {
-      case "focussed":
+      case "focused":
         return qx.util.ColorUtil.scale(color, {
           lightness: 10,
           saturation: 10
@@ -159,10 +159,10 @@ qx.Theme.define("qx.theme.tangible.ColorEngine", {
     "text-on-error": helper.onX,
     "primary-hovered": helper.xState,
     "primary-disabled": helper.xState,
-    "primary-focussed": helper.xState,
+    "primary-focused": helper.xState,
     "primary-selected": helper.xState,
     "primary-selected_disabled": helper.xState,
-    "error-focussed": helper.xState,
+    "error-focused": helper.xState,
 
     // alpha colors
     "primary-alpha-5": helper.setAlpha,

package/source/class/qx/theme/tangible/Decoration.js

@@ -40,7 +40,7 @@ qx.Theme.define("qx.theme.tangible.Decoration", {
       include: "material-textfield",
       style: {
         widthBottom: 2,
-        colorBottom: "primary-focussed"
+        colorBottom: "primary-focused"
       }
     },
 
@@ -56,7 +56,7 @@ qx.Theme.define("qx.theme.tangible.Decoration", {
       include: "material-textfield",
       style: {
         widthBottom: 2,
-        colorBottom: "error-focussed"
+        colorBottom: "error-focused"
       }
     },
 
@@ -287,7 +287,7 @@ qx.Theme.define("qx.theme.tangible.Decoration", {
     "button-box-pressed": {
       include: "button-box",
 
-      style: { backgroundColor: "primary-focussed" }
+      style: { backgroundColor: "primary-focused" }
     },
 
     "button-box-pressed-hovered": {

package/source/class/qx/tool/cli/api/AbstractApi.js

@@ -86,7 +86,10 @@ qx.Class.define("qx.tool.cli.api.AbstractApi", {
      */
     loadNpmModule(module) {
       const { execSync } = require("child_process");
-      let s = `npm install --no-save --no-package-lock ${module}`;
+      // since npm 7 --no-save deletes the node_modules folder
+      // see https://github.com/npm/cli/pull/3907
+//       let s = `npm install --no-save --no-package-lock ${module}`;
+      let s = `npm install --no-package-lock ${module}`;
       qx.tool.compiler.Console.info(s);
       execSync(s, {
         stdio: "inherit"

package/source/class/qx/tool/cli/commands/Compile.js

@@ -359,7 +359,10 @@ qx.Class.define("qx.tool.cli.commands.Compile", {
         this.argv["feedback"] = configDb.db("qx.default.feedback", true);
       }
 
-      if (this.argv.verbose) {
+     // Validate compile.json against the schema
+     await qx.tool.config.Compile.getInstance().load();
+
+     if (this.argv.verbose) {
         console.log(`
 Compiler:  v${this.getCompilerVersion()} in ${require.main.filename}
 Framework: v${await this.getQxVersion()} in ${await this.getQxPath()}`);
@@ -915,6 +918,7 @@ Framework: v${await this.getQxVersion()} in ${await this.getQxPath()}`);
        */
       let targetOutputPaths = {};
       let makers = [];
+
       targetConfigs.forEach(targetConfig => {
         if (!targetConfig.appConfigs) {
           qx.tool.compiler.Console.print(
@@ -1203,7 +1207,8 @@ Framework: v${await this.getQxVersion()} in ${await this.getQxPath()}`);
             "loaderTemplate",
             "publish",
             "deploy",
-            "standalone"
+            "standalone",
+            "localModules"
           ].forEach(name => {
             if (appConfig[name] !== undefined) {
               var fname = "set" + qx.lang.String.firstUp(name);
@@ -1225,6 +1230,10 @@ Framework: v${await this.getQxVersion()} in ${await this.getQxPath()}`);
             app.setDescription(appConfig.description);
           }
 
+          if (appConfig.localModules) {
+            app.setLocalModules(appConfig.localModules);
+          }
+
           var parts = appConfig.parts || targetConfig.parts || data.parts;
           if (parts) {
             if (!parts.boot) {

package/source/class/qx/tool/cli/commands/Lint.js

@@ -95,7 +95,16 @@ qx.Class.define("qx.tool.cli.commands.Lint", {
 
   members: {
     async process() {
-      await this.__applyFixes();
+
+      let files = this.argv.files || [];
+      if (files.length === 0) {
+        files.push("source/class/**/*.js");
+      }
+      for (let i = 0; i < files.length; i++) {
+        files[i] = path.join(process.cwd(), files[i]);
+      }
+
+      await this.__applyFixes(files);
 
       let helperFilePath = require.main.path;
       while (true) {
@@ -134,13 +143,6 @@ qx.Class.define("qx.tool.cli.commands.Lint", {
         fix: this.argv.fix
       });
 
-      let files = this.argv.files || [];
-      if (files.length === 0) {
-        files.push("source/class/**/*.js");
-      }
-      for (let i = 0; i < files.length; i++) {
-        files[i] = path.join(process.cwd(), files[i]);
-      }
       if (this.argv.printConfig) {
         const fileConfig = await linter.calculateConfigForFile(files[0]);
         qx.tool.compiler.Console.info(JSON.stringify(fileConfig, null, "  "));
@@ -157,13 +159,23 @@ qx.Class.define("qx.tool.cli.commands.Lint", {
         }
         if (report.errorCount > 0 || report.warningCount > 0) {
           let outputFormat = this.argv.format || "codeframe";
-
-          // If there are too many errors, the pretty formatter is appallingly slow
-          if (report.errorCount + report.warningCount > 150) {
-            outputFormat = "compact";
-          }
           const formatter = await linter.loadFormatter(outputFormat);
           const s = formatter.format(report);
+          // If there are too many errors, the pretty formatter is appallingly slow so if the
+          // user has not specified a format, change to compact mode
+          const maxDefaultFormatErrorCount = 150;
+          if (report.errorCount + report.warningCount > maxDefaultFormatErrorCount) {
+            if (!this.argv.format) {
+              qx.tool.compiler.Console.info(
+                `Total errors and warnings exceed ${maxDefaultFormatErrorCount}, switching to "compact" style report`
+              );
+              outputFormat = "compact";
+            } else {
+              qx.tool.compiler.Console.info(
+                `Total errors and warnings exceed ${maxDefaultFormatErrorCount}, the report may take some time to generate.`
+              );
+            }
+          }
           if (this.argv.outputFile) {
             if (this.argv.verbose) {
               qx.tool.compiler.Console.info(
@@ -220,19 +232,18 @@ qx.Class.define("qx.tool.cli.commands.Lint", {
      * @return {Promise<void>}
      * @private
      */
-    async __applyFixes() {
+    async __applyFixes(files) {
       const fixParams = this.argv.fixJsdocParams;
       if (fixParams && fixParams !== "off") {
-        let replaceInFiles = [];
         const regex =
           fixParams === "type-first"
             ? /@param\s+([\w$]+)\s+({[\w|[\]{}<>?. ]+})/g
             : /@param\s+({[\w|[\]{}<>?. ]+})\s+([\w$]+)/g;
-        replaceInFiles.push({
-          files: "source/class/**/*.js",
+        let replaceInFiles = {
+          files: files,
           from: regex,
           to: "@param $2 $1"
-        });
+        };
 
         await replaceInFile(replaceInFiles);
       }

package/source/class/qx/tool/compiler/ClassFile.js

@@ -223,6 +223,7 @@ qx.Class.define("qx.tool.compiler.ClassFile", {
     };
 
     this.__externals = [];
+    this.__commonjsModules = {};
 
     this.__taskQueueDrains = [];
     this.__taskQueue = async.queue(function (task, cb) {
@@ -281,6 +282,7 @@ qx.Class.define("qx.tool.compiler.ClassFile", {
     __privates: null,
     __blockedPrivates: null,
     __externals: null,
+    __commonjsModules: null,
 
     _onTaskQueueDrain() {
       var cbs = this.__taskQueueDrain;
@@ -496,6 +498,7 @@ qx.Class.define("qx.tool.compiler.ClassFile", {
       delete dbClassInfo.translations;
       delete dbClassInfo.markers;
       delete dbClassInfo.fatalCompileError;
+      delete dbClassInfo.commonjsModules;
       for (var key in this.__dbClassInfo) {
         dbClassInfo[key] = this.__dbClassInfo[key];
       }
@@ -667,6 +670,14 @@ qx.Class.define("qx.tool.compiler.ClassFile", {
         dbClassInfo.fatalCompileError = true;
       }
 
+      // CommonJS modules
+      if (Object.keys(this.__commonjsModules).length > 0) {
+        dbClassInfo.commonjsModules = {};
+        for (let moduleName in this.__commonjsModules) {
+          dbClassInfo.commonjsModules[moduleName] = [ ...this.__commonjsModules[moduleName] ];
+        }
+      }
+
       return dbClassInfo;
     },
 
@@ -1648,6 +1659,52 @@ qx.Class.define("qx.tool.compiler.ClassFile", {
               }
             }
 
+            // Are we looking at the Identifier `require`, and is it a
+            // function call (identified by having
+            // `path.node.arguments`? If so, we'll add the discovered
+            // module to the list of modules that must be browserified
+            // if the application is destined for the browser.
+            let scope;
+            let applicationTypes = t.__analyser.getApplicationTypes();
+
+            if (path.node.callee.type == "Identifier" &&
+                path.node?.callee?.name == "require" &&
+                path.node.arguments?.length == 1 &&
+                applicationTypes.includes("browser")) {
+
+              // See if this is a reference to global `require` or
+              // something in the scope chain
+              for (scope = t.__scope; scope; scope = scope.parent) {
+                if (scope.vars["require"]) {
+                  // It's in the scope chain. Ignore it.
+                  break;
+                }
+              }
+              // Did we reach top level without finding it in a local scope?
+              if (! scope) {
+                // Yup. It's the global one we're looking for. Ensure the argument is valid.
+                let arg = path.node.arguments[0];
+                if (types.isLiteral(arg)) {
+                  if (typeof arg.value != "string") {
+                    log.error(
+                      `${t.__className}: ` +
+                        "Only literal string arguments to require() are supported: " +
+                        arg.value
+                    );
+                  } else {
+                    qx.tool.compiler.Console.log(
+                      `${t.__className}:${path.node.loc.start.line}:` +
+                        ` automatically detected \'require(${arg.value})\``);
+                    t.addCommonjsModule(arg.value, t.__className, path.node.loc.start.line);
+
+                    // Don't show "unresolved" error for `require` since the
+                    // browserified code defines it as a global
+                    t.addIgnore("require");
+                  }
+                }
+              }
+            }
+
             if (
               types.isMemberExpression(path.node.callee) ||
               (es6ClassDeclarations == 0 &&
@@ -2506,6 +2563,19 @@ qx.Class.define("qx.tool.compiler.ClassFile", {
       }
     },
 
+    /**
+     * Adds a CommonJS module to be browserified
+     *
+     * @param name {String} name of the module
+     */
+    addCommonjsModule(moduleName, className, linenum) {
+      if (! this.__commonjsModules[moduleName]) {
+        this.__commonjsModules[moduleName] = new Set();
+      }
+
+      this.__commonjsModules[moduleName].add(`${className}:${linenum}`);
+    },
+
     /**
      * Adds an ignored symbol
      * @param name {String} name of the symbol

package/source/class/qx/tool/compiler/Es6ify.js

@@ -106,6 +106,7 @@ qx.Class.define("qx.tool.compiler.Es6ify", {
   construct(filename) {
     super();
     this.__filename = filename;
+    this.__knownApiFunctions = ["addListener","addListenerOnce"];
   },
 
   properties: {
@@ -125,6 +126,7 @@ qx.Class.define("qx.tool.compiler.Es6ify", {
 
   members: {
     __filename: null,
+    __knownApiFunctions: null,
 
     async transform() {
       let src = await fs.promises.readFile(this.__filename, "utf8");
@@ -275,6 +277,7 @@ qx.Class.define("qx.tool.compiler.Es6ify", {
       let t = this;
       const isTest = this.__filename.indexOf("/test/") > -1;
       let arrowFunctions = this.getArrowFunctions();
+      let knownApiFunctions = this.__knownApiFunctions;
 
       return {
         visitor: {
@@ -282,13 +285,13 @@ qx.Class.define("qx.tool.compiler.Es6ify", {
             if (path.node.callee.type == "MemberExpression") {
               let callee = collapseMemberExpression(path.node.callee);
               if (arrowFunctions == "careful") {
-                if (!callee.endsWith(".addListener")) {
+                if (!knownApiFunctions.some(fName => callee.endsWith("."+fName))) {
                   return;
                 }
                 if (
                   path.node.arguments.length != 3 ||
                   path.node.arguments[0].type != "StringLiteral" ||
-                  path.node.arguments[1].type != "ArrowFunctionExpression" ||
+                  path.node.arguments[1].type != "FunctionExpression" ||
                   path.node.arguments[2].type != "ThisExpression"
                 ) {
                   return;
@@ -324,14 +327,14 @@ qx.Class.define("qx.tool.compiler.Es6ify", {
      * @returns
      */
     __pluginRemoveUnnecessaryThis() {
+      let knownApiFunctions = this.__knownApiFunctions;
       return {
         visitor: {
           CallExpression(path) {
             if (
               path.node.callee.type == "MemberExpression" &&
-              path.node.callee.object.type == "ThisExpression" &&
               path.node.callee.property.type == "Identifier" &&
-              path.node.callee.property.name == "addListener" &&
+              knownApiFunctions.includes(path.node.callee.property.name) &&
               path.node.arguments.length == 3 &&
               path.node.arguments[0].type == "StringLiteral" &&
               path.node.arguments[1].type == "ArrowFunctionExpression" &&

package/source/class/qx/tool/compiler/TargetError.js

@@ -0,0 +1,27 @@
+/* ************************************************************************
+ *
+ *    qooxdoo-compiler - node.js based replacement for the Qooxdoo python
+ *    toolchain
+ *
+ *    https://github.com/qooxdoo/qooxdoo-compiler
+ *
+ *    Copyright:
+ *      3033 Derrell Lipman
+ *
+ *    License:
+ *      MIT: https://opensource.org/licenses/MIT
+ *
+ *      This software is provided under the same licensing terms as Qooxdoo,
+ *      please see the LICENSE file in the Qooxdoo project's top-level directory
+ *      for details.
+ *
+ *    Authors:
+ *      * Derrell Lipman (@derrell)
+ *
+ * ************************************************************************/
+
+qx.Class.define("qx.tool.compiler.TargetError",
+{
+//  extend : qx.type.BaseError
+  extend : Error
+});

package/source/class/qx/tool/compiler/app/Application.js

@@ -217,6 +217,20 @@ qx.Class.define("qx.tool.compiler.app.Application", {
     writeIndexHtmlToRoot: {
       init: false,
       check: "Boolean"
+    },
+
+    /**
+     * Map specifying local modules to include in the build. Local modules are
+     * always included in the compiled output, regardless of application type
+     * (node, browser, etc.). Each member of the map has a key that is the
+     * name of the module to be `require`d in code. The value is the path to
+     * the module to be included. The path must begin with "./", and is
+     * relative to the directory containing compile.json. The module may be
+     * either a CommonJS module or an ES6 module.
+     */
+    localModules : {
+      init : null,
+      check : "Object"
     }
   },
 

package/source/class/qx/tool/compiler/targets/Target.js

@@ -432,6 +432,13 @@ qx.Class.define("qx.tool.compiler.targets.Target", {
         new qx.tool.compiler.targets.meta.PolyfillJs(appMeta)
       );
 
+      // Add browserified CommonJS modules, if any. The Browserify
+      // class will always bundle local modules specified for an
+      // application in compile.json, but will not bundle `require()`d
+      // modules that are Node modules.
+      bootPackage.addJavascriptMeta(
+        new qx.tool.compiler.targets.meta.Browserify(appMeta));
+
       /*
        * Assemble the Parts
        */

package/source/class/qx/tool/compiler/targets/meta/AbstractJavascriptMeta.js

@@ -26,7 +26,7 @@ const sourceMap = require("source-map");
 
 /**
  * An AbstractJavascriptMeta provides an abstraction of some source code, and might be
- * compromised of a number of input files which are merged together as required.
+ * comprised of a number of input files which are merged together as required.
  *
  * This object could represent a file which already exists on disk (eg a transpiled
  * source file), or something that is generated on the fly (such as a index.js), or