Use more restrictive types

src/Bundle.ts

@@ -267,7 +267,7 @@ function getIncludedModules(modulesById: ReadonlyMap<string, Module | ExternalMo
 }
 
 function getAbsoluteEntryModulePaths(
-	includedModules: Module[],
+	includedModules: readonly Module[],
 	preserveModules: boolean
 ): string[] {
 	const absoluteEntryModulePaths: string[] = [];

src/ast/scopes/ClassBodyScope.ts

@@ -6,8 +6,8 @@ import ChildScope from './ChildScope';
 import type Scope from './Scope';
 
 export default class ClassBodyScope extends ChildScope {
-	instanceScope: ChildScope;
-	thisVariable: LocalVariable;
+	readonly instanceScope: ChildScope;
+	readonly thisVariable: LocalVariable;
 
 	constructor(parent: Scope, classNode: ClassNode, context: AstContext) {
 		super(parent);

src/ast/scopes/ModuleScope.ts

@@ -10,7 +10,7 @@ import ChildScope from './ChildScope';
 import type GlobalScope from './GlobalScope';
 
 export default class ModuleScope extends ChildScope {
-	context: AstContext;
+	readonly context: AstContext;
 	declare parent: GlobalScope;
 
 	constructor(parent: GlobalScope, context: AstContext) {

src/ast/scopes/ParameterScope.ts

@@ -10,9 +10,8 @@ import type Scope from './Scope';
 
 export default class ParameterScope extends ChildScope {
 	readonly hoistedBodyVarScope: ChildScope;
-
 	protected parameters: readonly LocalVariable[][] = [];
-	private context: AstContext;
+	private readonly context: AstContext;
 	private hasRest = false;
 
 	constructor(parent: Scope, context: AstContext) {

src/ast/scopes/ReturnValueScope.ts

@@ -4,7 +4,7 @@ import ParameterScope from './ParameterScope';
 
 export default class ReturnValueScope extends ParameterScope {
 	private returnExpression: ExpressionEntity | null = null;
-	private returnExpressions: ExpressionEntity[] = [];
+	private readonly returnExpressions: ExpressionEntity[] = [];
 
 	addReturnExpression(expression: ExpressionEntity): void {
 		this.returnExpressions.push(expression);

src/ast/scopes/Scope.ts

@@ -7,8 +7,8 @@ import type Variable from '../variables/Variable';
 import type ChildScope from './ChildScope';
 
 export default class Scope {
-	children: ChildScope[] = [];
-	variables = new Map<string, Variable>();
+	readonly children: ChildScope[] = [];
+	readonly variables = new Map<string, Variable>();
 
 	addDeclaration(
 		identifier: Identifier,

src/ast/scopes/TrackingScope.ts

@@ -5,7 +5,7 @@ import type LocalVariable from '../variables/LocalVariable';
 import BlockScope from './BlockScope';
 
 export default class TrackingScope extends BlockScope {
-	hoistedDeclarations: Identifier[] = [];
+	readonly hoistedDeclarations: Identifier[] = [];
 
 	addDeclaration(
 		identifier: Identifier,

src/ast/variables/ExportShimVariable.ts

@@ -3,7 +3,7 @@ import { MISSING_EXPORT_SHIM_VARIABLE } from '../../utils/variableNames';
 import Variable from './Variable';
 
 export default class ExportShimVariable extends Variable {
-	module: Module;
+	readonly module: Module;
 
 	constructor(module: Module) {
 		super(MISSING_EXPORT_SHIM_VARIABLE);

src/ast/variables/ExternalVariable.ts

@@ -6,8 +6,8 @@ import type { ObjectPath } from '../utils/PathTracker';
 import Variable from './Variable';
 
 export default class ExternalVariable extends Variable {
-	isNamespace: boolean;
-	module: ExternalModule;
+	readonly isNamespace: boolean;
+	readonly module: ExternalModule;
 	referenced = false;
 
 	constructor(module: ExternalModule, name: string) {

src/ast/variables/NamespaceVariable.ts

@@ -15,9 +15,9 @@ import { SymbolToStringTag, UNKNOWN_PATH } from '../utils/PathTracker';
 import Variable from './Variable';
 
 export default class NamespaceVariable extends Variable {
-	context: AstContext;
+	readonly context: AstContext;
 	declare isNamespace: true;
-	module: Module;
+	readonly module: Module;
 
 	private memberVariables: { [name: string]: Variable } | null = null;
 	private mergedNamespaces: readonly Variable[] = [];

src/ast/variables/SyntheticNamedExportVariable.ts

@@ -4,9 +4,9 @@ import ExportDefaultVariable from './ExportDefaultVariable';
 import Variable from './Variable';
 
 export default class SyntheticNamedExportVariable extends Variable {
-	context: AstContext;
-	module: Module;
-	syntheticNamespace: Variable;
+	readonly context: AstContext;
+	readonly module: Module;
+	readonly syntheticNamespace: Variable;
 
 	private baseVariable: Variable | null = null;
 

src/finalisers/cjs.ts

@@ -73,7 +73,7 @@ export default function cjs(
 }
 
 function getImportBlock(
-	dependencies: ChunkDependency[],
+	dependencies: readonly ChunkDependency[],
 	{ _, cnst, n }: GenerateCodeSnippets,
 	compact: boolean
 ): string {

src/finalisers/es.ts

@@ -32,7 +32,10 @@ export default function es(
 	magicString.trim();
 }
 
-function getImportBlock(dependencies: ChunkDependency[], { _ }: GenerateCodeSnippets): string[] {
+function getImportBlock(
+	dependencies: readonly ChunkDependency[],
+	{ _ }: GenerateCodeSnippets
+): string[] {
 	const importBlock: string[] = [];
 	for (const { importPath, reexports, imports, name, assertions } of dependencies) {
 		const assertion = assertions ? `${_}assert${_}${assertions}` : '';

src/finalisers/shared/getCompleteAmdId.ts

@@ -9,5 +9,5 @@ export default function getCompleteAmdId(
 		return `${options.basePath ? options.basePath + '/' : ''}${removeJsExtension(chunkId)}`;
 	}
 
-	return options.id || '';
+	return options.id ?? '';
 }

src/finalisers/shared/getExportBlock.ts

@@ -10,7 +10,7 @@ import {
 
 export function getExportBlock(
 	exports: ChunkExports,
-	dependencies: ChunkDependency[],
+	dependencies: readonly ChunkDependency[],
 	namedExportsMode: boolean,
 	interop: GetInterop,
 	snippets: GenerateCodeSnippets,
@@ -117,7 +117,7 @@ export function getExportBlock(
 
 function getSingleDefaultExport(
 	exports: ChunkExports,
-	dependencies: ChunkDependency[],
+	dependencies: readonly ChunkDependency[],
 	interop: GetInterop,
 	externalLiveBindings: boolean,
 	getPropertyAccess: (name: string) => string

src/utils/FileEmitter.ts

@@ -246,8 +246,11 @@ export class FileEmitter {
 		bundle: OutputBundleWithPlaceholders,
 		outputOptions: NormalizedOutputOptions
 	): void => {
-		const fileNamesBySource = new Map();
-		const output = (this.output = { bundle, fileNamesBySource, outputOptions });
+		const output = (this.output = {
+			bundle,
+			fileNamesBySource: new Map<string, string>(),
+			outputOptions
+		});
 		for (const emittedFile of this.filesByReferenceId.values()) {
 			if (emittedFile.fileName) {
 				reserveFileNameInBundle(emittedFile.fileName, output, this.options.onwarn);
@@ -265,16 +268,13 @@ export class FileEmitter {
 	}
 
 	private assignReferenceId(file: ConsumedFile, idBase: string): string {
-		let referenceId: string | undefined;
+		let referenceId = idBase;
 
 		do {
-			referenceId = createHash()
-				.update(referenceId || idBase)
-				.digest('hex')
-				.slice(0, 8);
+			referenceId = createHash().update(referenceId).digest('hex').slice(0, 8);
 		} while (
 			this.filesByReferenceId.has(referenceId) ||
-			this.outputFileEmitters.some(({ filesByReferenceId }) => filesByReferenceId.has(referenceId!))
+			this.outputFileEmitters.some(({ filesByReferenceId }) => filesByReferenceId.has(referenceId))
 		);
 		this.filesByReferenceId.set(referenceId, file);
 		for (const { filesByReferenceId } of this.outputFileEmitters) {