- Linting: Add local ESLint rule to prefer no-namespace on public files, but disable rule for now for our two very large files until they may be refactored into smaller files

.eslintrc.js

@@ -174,6 +174,9 @@ module.exports = {
         ]
       },
       rules: {
+        // Should probably add this rule to ash-nazg as facilitates tree-shaking
+        'import/no-namespace': ['error'],
+
         'node/no-unsupported-features/node-builtins': 'off'
       }
     },

editor/svg-editor.js

@@ -18,6 +18,10 @@
 import './touch.js';
 import {NS} from './namespaces.js';
 import {isWebkit, isChrome, isGecko, isIE, isMac, isTouch} from './browser.js';
+
+// Until we split this into smaller files, this helps distinguish utilities
+//   from local methods
+// eslint-disable-next-line import/no-namespace
 import * as Utils from './utilities.js';
 import {getTypeMap, convertUnit, isValidUnit} from './units.js';
 import {

editor/svgcanvas.js

@@ -20,7 +20,13 @@
 import './svgpathseg.js';
 import jQueryPluginSVG from './jQuery.attr.js'; // Needed for SVG attribute setting and array form with `attr`
 import jQueryPluginDBox from './dbox.js';
+
+/* eslint-disable import/no-namespace -- Keeping namespaced to help distinguish
+  utilities from local functions until we split canvas into separate files */
+import * as pathModule from './path.js';
+import * as hstry from './history.js';
 import * as draw from './draw.js';
+/* eslint-enable import/no-namespace */
 // eslint-disable-next-line no-duplicate-imports
 import {
   identifyLayers, createLayer, cloneLayer, deleteCurrentLayer,
@@ -28,7 +34,6 @@ import {
   setLayerVisibility, moveSelectedToLayer, mergeLayer, mergeAllLayers,
   leaveContext, setContext
 } from './draw.js';
-import * as pathModule from './path.js';
 import {sanitizeSvg} from './sanitize.js';
 import {getReverseNS, NS} from './namespaces.js';
 import {importSetGlobal, importScript} from './external/dynamic-import-polyfill/importModule.js';
@@ -42,7 +47,6 @@ import {
   init as utilsInit, getBBox as utilsGetBBox, getStrokedBBoxDefaultVisible,
   isNullish
 } from './utilities.js';
-import * as hstry from './history.js';
 import {
   transformPoint, matrixMultiply, hasMatrixTransform, transformListToTransform,
   getMatrix, snapToAngle, isIdentity, rectsIntersect, transformBox