Fork sax (#1369)

There was a lot of PR with switching to another xml parser because sax
is not maintained for almost 4 years already.

Though switching parser to solve a few problems may introduce many new
bugs. This is why we decided to fork sax and maintain own version.

For initial release I removed node streams support which allows to get
rid from some magic in browser bundle build config and got rid from
String.fromCodePoint polyfill.

Forked package is here https://github.com/svg/sax

Author: TrySound

lib/svgo/svg2js.js

@@ -1,6 +1,6 @@
 'use strict';
 
-var SAX = require('sax'),
+var SAX = require('@trysound/sax'),
     JSAPI = require('./jsAPI.js'),
     CSSClassList = require('./css-class-list'),
     CSSStyleDeclaration = require('./css-style-declaration'),

package-lock.json

@@ -89,13 +89,13 @@
     ]
   },
   "dependencies": {
+    "@trysound/sax": "0.1.0",
     "chalk": "^4.1.0",
     "commander": "^7.1.0",
     "css-select": "^3.1.2",
     "css-select-base-adapter": "^0.1.1",
     "css-tree": "^1.1.2",
     "csso": "^4.2.0",
-    "sax": "~1.2.4",
     "stable": "^0.1.8"
   },
   "devDependencies": {

package.json

@@ -14,7 +14,7 @@ export default {
   plugins: [
     {
       resolveId(importee, importer) {
-        if (['os', 'stream', 'string_decoder'].includes(importee)) {
+        if (importee === 'os') {
           return importee;
         }
         // see https://github.com/csstree/csstree/pull/152
@@ -26,12 +26,6 @@ export default {
         if (id === 'os') {
           return `export var EOL = '\\n'`;
         }
-        if (id === 'stream') {
-          return `export function Stream(){}`;
-        }
-        if (id === 'string_decoder') {
-          return `export default null`;
-        }
       },
     },
     nodeResolve({ browser: true, preferBuiltins: false }),