Minor code tweaks (after code review)

src/Declaration.js

@@ -1,5 +1,5 @@
 import { blank, forOwn, keys } from './utils/object.js';
-import { makeLegal, reservedWords } from './utils/identifier-helpers.js';
+import { makeLegal, reservedWords } from './utils/identifierHelpers.js';
 import { UNKNOWN } from './ast/values.js';
 
 export default class Declaration {

src/ExternalModule.js

@@ -1,5 +1,5 @@
 import { blank } from './utils/object.js';
-import { makeLegal } from './utils/identifier-helpers.js';
+import { makeLegal } from './utils/identifierHelpers.js';
 import { ExternalDeclaration } from './Declaration.js';
 
 export default class ExternalModule {

src/Module.js

@@ -4,7 +4,7 @@ import { locate } from 'locate-character';
 import { timeStart, timeEnd } from './utils/flushTime.js';
 import { assign, blank, keys } from './utils/object.js';
 import { basename, extname } from './utils/path.js';
-import { makeLegal } from './utils/identifier-helpers.js';
+import { makeLegal } from './utils/identifierHelpers.js';
 import getCodeFrame from './utils/getCodeFrame.js';
 import { SOURCEMAPPING_URL_RE } from './utils/sourceMappingURL.js';
 import error from './utils/error.js';

src/finalisers/amd.js

@@ -1,4 +1,4 @@
-import { getName, quotePath } from '../utils/map-helpers.js';
+import { getName, quotePath } from '../utils/mapHelpers.js';
 import getInteropBlock from './shared/getInteropBlock.js';
 import getExportBlock from './shared/getExportBlock.js';
 import esModuleExport from './shared/esModuleExport.js';

src/finalisers/iife.js

@@ -1,13 +1,13 @@
 import { blank } from '../utils/object.js';
-import { getName } from '../utils/map-helpers.js';
+import { getName } from '../utils/mapHelpers.js';
 import error from '../utils/error.js';
 import getInteropBlock from './shared/getInteropBlock.js';
 import getExportBlock from './shared/getExportBlock.js';
 import getGlobalNameMaker from './shared/getGlobalNameMaker.js';
 import { property, keypath } from './shared/sanitize.js';
 import warnOnBuiltins from './shared/warnOnBuiltins.js';
 import trimEmptyImports from './shared/trimEmptyImports.js';
-import { isLegal } from '../utils/identifier-helpers.js';
+import { isLegal } from '../utils/identifierHelpers.js';
 
 function setupNamespace ( keypath ) {
 	const parts = keypath.split( '.' );
@@ -27,10 +27,10 @@ export default function iife ( bundle, magicString, { exportMode, indentString,
 	const globalNameMaker = getGlobalNameMaker( options.globals || blank(), bundle, 'null' );
 
 	const { extend, moduleName: name } = options;
-	const isNamespaced = name && ~name.indexOf( '.' );
-	const justVariable = !extend && !isNamespaced;
+	const isNamespaced = name && name.indexOf( '.' ) !== -1;
+	const possibleVariableAssignment = !extend && !isNamespaced;
 
-	if ( name && justVariable && !isLegal(name) ) {
+	if ( name && possibleVariableAssignment && !isLegal(name) ) {
 		error({
 			code: 'ILLEGAL_IDENTIFIER_AS_NAME',
 			message: `Given moduleName (${name}) is not legal JS identifier. If you need this you can try --extend option`
@@ -72,7 +72,7 @@ export default function iife ( bundle, magicString, { exportMode, indentString,
 
 	let wrapperOutro = `\n\n}(${dependencies}));`;
 
-	if (justVariable && exportMode === 'named') {
+	if (possibleVariableAssignment && exportMode === 'named') {
 		wrapperOutro = `\n\n${indentString}return exports;${wrapperOutro}`;
 	}
 

src/finalisers/umd.js

@@ -1,5 +1,5 @@
 import { blank } from '../utils/object.js';
-import { getName, quotePath, req } from '../utils/map-helpers.js';
+import { getName, quotePath, req } from '../utils/mapHelpers.js';
 import error from '../utils/error.js';
 import getInteropBlock from './shared/getInteropBlock.js';
 import getExportBlock from './shared/getExportBlock.js';