TODO comments [nocode]

lib/instrument/modify.js

@@ -273,6 +273,7 @@ function insertImportStatement(programNode, state) {
 	//   = require('/path/to/app/node_modules/livepack/lib/init/index.js')
 	//     ('/path/to/app/file.js', module, require, 100, 0);
 	// ```
+	// TODO: Make this just a direct call if no `livepack_tracker` and `livepack_getScopeId` vars required
 	const statementNode = t.variableDeclaration(
 		'const', [
 			t.variableDeclarator(

lib/instrument/visitors/eval.js

@@ -110,6 +110,9 @@ function instrumentEvalCall(
 			// This is relevant as it affects whether var is flagged as mutable or not.
 			const isConst = binding.isConst || (isStrict && (varName === 'arguments' || varName === 'eval'));
 			const varDefNodes = [t.stringLiteral(varName)];
+			// TODO: Can `binding.isSilentConst` be true when `isConst` is false?
+			// If so, this is a bug.
+			// If not, then `if (binding.isSilentConst)` could go inside `if (isConst)` block.
 			if (isConst) varDefNodes.push(t.numericLiteral(1));
 			if (binding.isSilentConst) varDefNodes.push(t.numericLiteral(1));
 			if (binding.argNames) {
@@ -281,6 +284,7 @@ function activateSuperIfIsUsable(fn, superBlock, thisBlock, superIsProto, trail,
 			createExternalVarForThis(fn, thisBlock, state);
 		} else if (fnType === 'ClassDeclaration' || fnType === 'ClassExpression') {
 			if (trail.length > 6) {
+				// TODO: I'm not sure this is correct - I think doesn't apply in `ClassMethod`s?
 				const maybeConstructorNode = getProp(fn.node, trail, 3);
 				if (maybeConstructorNode.type === 'ClassMethod' && maybeConstructorNode.kind === 'constructor') {
 					// In class constructor - need internal var for `this` as `eval()` code might include `super()`