yes I mentioned that in the comment in #266 too, we need to unify Repl and ScriptExecutor with a common base class