I think we should proceed with the removal. The only references to src/interpreter/interpreter.ts are as follows:

• js-slang/src/interpreter/closure.ts

  Lines 15 to 20 in 9b48944

  	import { apply } from './interpreter'
  	const closureToJS = (value: Closure, context: Context, klass: string) => {
  	function DummyClass(this: Closure) {
  	const args: Value[] = Array.prototype.slice.call(arguments)
  	const gen = apply(context, value, args, callExpression(identifier(klass), args), this)

• js-slang/src/runner/sourceRunner.ts

  Lines 115 to 127 in 9b48944

  	function runInterpreter(program: es.Program, context: Context, options: IOptions): Promise<Result> {
  	let it = evaluate(program, context, true, true)
  	let scheduler: Scheduler
  	if (context.variant === Variant.NON_DET) {
  	it = nonDetEvaluate(program, context)
  	scheduler = new NonDetScheduler()
  	} else if (options.scheduler === 'async') {
  	scheduler = new AsyncScheduler()
  	} else {
  	scheduler = new PreemptiveScheduler(options.steps)
  	}
  	return scheduler.run(it, context)
  	}

• js-slang/src/mocks/context.ts

  Line 5 in 9b48944

  import { createBlockEnvironment } from '../interpreter/interpreter'

  Perhaps we can just replace this with

  import { createBlockEnvironment } from '../ec-evaluator/utils'

How are would it be to completely remove it?

I think we should completely remove it. It's a confusing legacy component at this point. Where are these "mocks" used?

I think we should completely remove it. It's a confusing legacy component at this point. Where are these "mocks" used?

The mocks in the last bullet point are used in test files. I'll make a PR with the necessary changes.