build(deps): bump xmldom to 0.8.0

Switching from package `xmldom` to `@xmldom/xmldom`, which resolves the security issue present in latest xmldom version 0.6.0:
GHSA-5fg8-2547-mr8q

The reason is that the maintainers were forced to switch to a scoped package since 0.7.0:
 xmldom/xmldom#271

- I used node 12 to run `npm install`.
- I executed `npm run test` on my machine but some tests complained about missing titanium config in my home directory.
  Since I don't know what that means we will have to see what CI checks complain about.

I'm one of the xmldom maintainers. Don't hesitate to ask me questions.

Alloy/commands/compile/compilerUtils.js

@@ -9,7 +9,7 @@ var U = require('../../utils'),
 	astController = require('./ast/controller'),
 	_ = require('lodash'),
 	styler = require('./styler'),
-	XMLSerializer = require('xmldom').XMLSerializer,
+	XMLSerializer = require('@xmldom/xmldom').XMLSerializer,
 	CONST = require('../../common/constants'),
 	sourceMapper = require('./sourceMapper');
 

Alloy/commands/extract-i18n/i18nHandler.js

@@ -2,7 +2,7 @@ var U = require('../../utils'),
 	logger = require('../../logger'),
 	path = require('path'),
 	_ = require('lodash'),
-	XMLSerializer = require('xmldom').XMLSerializer,
+	XMLSerializer = require('@xmldom/xmldom').XMLSerializer,
 	fs = require('fs-extra'),
 	os = require('os');
 

Alloy/tiapp.js

@@ -1,6 +1,6 @@
 var fs = require('fs'),
 	path = require('path'),
-	XMLSerializer = require('xmldom').XMLSerializer,
+	XMLSerializer = require('@xmldom/xmldom').XMLSerializer,
 	pkg = require('../package'),
 	U = require('./utils'),
 	CONST = require('./common/constants'),

Alloy/utils.js

@@ -11,8 +11,8 @@ var path = require('path'),
 	paths = require('global-paths'),
 	logger = require('./logger'),
 	tiapp = require('./tiapp'),
-	XMLSerializer = require('xmldom').XMLSerializer,
-	DOMParser = require('xmldom').DOMParser,
+	XMLSerializer = require('@xmldom/xmldom').XMLSerializer,
+	DOMParser = require('@xmldom/xmldom').DOMParser,
 	_ = require('lodash'),
 	CONST = require('./common/constants'),
 	sourceMapper = require('./commands/compile/sourceMapper'),

package.json

@@ -36,6 +36,7 @@
     "@babel/traverse": "^7.4.5",
     "@babel/types": "^7.4.4",
     "@babel/template": "^7.4.4",
+    "@xmldom/xmldom": "^0.8.0",
     "async": "^3.2.0",
     "colors": "^1.1.2",
     "commander": "^8.0.0",
@@ -50,8 +51,7 @@
     "resolve": "^1.1.7",
     "source-map": "^0.7.3",
     "walk-sync": "^0.3.2",
-    "xml2tss": "0.0.5",
-    "xmldom": "^0.6.0"
+    "xml2tss": "0.0.5"
   },
   "license": "Apache-2.0",
   "bin": {

test/specs/compilefile.js

@@ -1,6 +1,6 @@
 var fs = require('fs-extra'),
 	path = require('path'),
-	DOMParser = require('xmldom').DOMParser,
+	DOMParser = require('@xmldom/xmldom').DOMParser,
 	TU = require('../lib/testUtils'),
 	CONST = require('../../Alloy/common/constants'),
 	platforms = require('../../platforms/index'),

test/specs/generate.js

@@ -2,7 +2,7 @@ var fs = require('fs'),
 	path = require('path'),
 	os = require('os'),
 	exec = require('child_process').exec,
-	DOMParser = require('xmldom').DOMParser,
+	DOMParser = require('@xmldom/xmldom').DOMParser,
 	TU = require('../lib/testUtils'),
 	U = require('../../Alloy/utils'),
 	CU = require('../../Alloy/commands/compile/compilerUtils'),

test/specs/new.js

@@ -1,6 +1,6 @@
 var fs = require('fs-extra'),
 	path = require('path'),
-	DOMParser = require('xmldom').DOMParser,
+	DOMParser = require('@xmldom/xmldom').DOMParser,
 	TU = require('../lib/testUtils'),
 	CONST = require('../../Alloy/common/constants'),
 	_ = require('lodash');