Overhaul repl module

Author: exoego

README.md

@@ -39,7 +39,7 @@ The following core Node.js modules (v8.7.0+) have been implemented:
 | [process](https://nodejs.org/api/process.html)               | :heavy_check_mark: |
 | [querystring](https://nodejs.org/api/querystring.html)       | :heavy_check_mark: |
 | [readline](https://nodejs.org/api/readline.html)             |                    |
-| [repl](https://nodejs.org/api/repl.html)                     |                    |
+| [repl](https://nodejs.org/api/repl.html)                     | :heavy_check_mark: |
 | [stream](https://nodejs.org/api/stream.html)                 |                    |
 | [string-decoder](https://nodejs.org/api/string_decoder.html) | :heavy_check_mark: |
 | [timers](https://nodejs.org/api/timers.html)                 | :heavy_check_mark: |

app/current/src/main/scala/io/scalajs/nodejs/repl/REPL.scala

@@ -1,7 +1,7 @@
 package io.scalajs.nodejs.repl
 
 import io.scalajs.nodejs.events.IEventEmitter
-import io.scalajs.nodejs.net.Socket
+import io.scalajs.nodejs.stream
 
 import scala.scalajs.js
 import scala.scalajs.js.annotation.JSImport
@@ -13,32 +13,31 @@ import scala.scalajs.js.annotation.JSImport
   */
 @js.native
 trait REPL extends IEventEmitter {
-  var REPL_MODE_SLOPPY: String = js.native
-  var REPL_MODE_STRICT: String = js.native
-  var REPL_MODE_MAGIC: String  = js.native
+  var REPL_MODE_SLOPPY: js.Symbol = js.native
+  var REPL_MODE_STRICT: js.Symbol = js.native
 
-  /**
-    * @example repl.start([options])
-    */
-  def start(options: REPLOptions): REPLServer = js.native
-
-  /**
-    * @example repl.start([options])
-    */
-  def start(prompt: String, socket: Socket): REPLServer = js.native
-
-  /**
-    * @example repl.start([options])
-    */
-  def start(prompt: String): REPLServer = js.native
-
-  /**
-    * @example repl.start([options])
-    */
-  def start(): REPLServer = js.native
+  @deprecated("Use REPL_MODE_SLOPPY instead", "Node.js v6.0.0")
+  var REPL_MODE_MAGIC: js.UndefOr[js.Symbol] = js.native
 
+  def start(options: StartOptions = js.native): REPLServer = js.native
+  def start(prompt: String): REPLServer                    = js.native
 }
 
+class StartOptions(
+    var prompt: js.UndefOr[String] = js.undefined,
+    var input: js.UndefOr[stream.Readable] = js.undefined,
+    var output: js.UndefOr[stream.Writable] = js.undefined,
+    var terminal: js.UndefOr[Boolean] = js.undefined,
+    var eval: js.UndefOr[js.Function4[String, js.Object, String, js.Function, Any]] = js.undefined,
+    var useColors: js.UndefOr[Boolean] = js.undefined,
+    var useGlobal: js.UndefOr[Boolean] = js.undefined,
+    var ignoreUndefined: js.UndefOr[Boolean] = js.undefined,
+    var writer: js.UndefOr[js.Function1[js.Any, Any]] = js.undefined,
+    var completer: js.UndefOr[js.Function] = js.undefined,
+    var replMode: js.UndefOr[js.Symbol] = js.undefined,
+    var breakEvalOnSigint: js.UndefOr[Boolean] = js.undefined
+) extends js.Object
+
 /**
   * REPL Singleton
   */

app/current/src/main/scala/io/scalajs/nodejs/repl/REPLServer.scala

@@ -1,9 +1,11 @@
 package io.scalajs.nodejs.repl
 
+import com.thoughtworks.enableIf
 import io.scalajs.nodejs.events.IEventEmitter
 import io.scalajs.nodejs.readline.Interface
 
 import scala.scalajs.js
+import scala.scalajs.js.|
 
 /**
   * REPL Server
@@ -16,6 +18,9 @@ trait REPLServer extends IEventEmitter with Interface {
     */
   val context: REPLContext = js.native
 
+  @enableIf(io.scalajs.nodejs.CompilerSwitches.gteNodeJs10)
+  def clearBufferedCommand(): Unit = js.native
+
   /**
     * The replServer.defineCommand() method is used to add new .-prefixed commands to the REPL instance.
     * Such commands are invoked by typing a period (.) followed by the keyword. The cmd is either a Function
@@ -27,7 +32,7 @@ trait REPLServer extends IEventEmitter with Interface {
     * @param keyword The command keyword (without a leading . character).
     * @param cmd     The function to invoke when the command is processed.
     */
-  def defineCommand(keyword: String, cmd: js.Function0[Any]): Unit = js.native
+  def defineCommand(keyword: String, cmd: DefinedCommand | js.Function1[String, Any]): Unit = js.native
 
   /**
     * The replServer.displayPrompt() method readies the REPL instance for input from the user, printing the
@@ -52,8 +57,16 @@ trait REPLServer extends IEventEmitter with Interface {
     */
   def displayPrompt(): Unit = js.native
 
+  @enableIf(io.scalajs.nodejs.CompilerSwitches.gteNodeJs12)
+  def setupHistory(historyPath: String, callback: js.Function2[io.scalajs.nodejs.Error, REPLServer, Any]): Unit =
+    js.native
 }
 
+class DefinedCommand(
+    var action: js.Function1[String, Any],
+    var help: js.UndefOr[String] = js.undefined
+) extends js.Object
+
 /**
   * REPL Server Companion
   */

app/current/src/main/scala/io/scalajs/nodejs/repl/package.scala

@@ -13,7 +13,7 @@ package object repl {
     * REPL Server events
     * @param server the given [[REPLServer instance]]
     */
-  final implicit class REPLServerEvents(val server: REPLServer) extends AnyVal {
+  final implicit class REPLServerEvents(private val server: REPLServer) extends AnyVal {
 
     @inline
     def contextAs[T]: T = server.context.asInstanceOf[T]
@@ -47,7 +47,7 @@ package object repl {
     * <li>NODE_REPL_MODE</li>
     * </ul>
     */
-  final implicit class EnvironmentVariableOptions(val env: process.Environment) extends AnyVal {
+  final implicit class EnvironmentVariableOptions(private val env: process.Environment) extends AnyVal {
 
     /**
       * When a valid path is given, persistent REPL history will be saved to the specified file rather