refactor: clean up context creation

Signed-off-by: Snehil Shah <[email protected]>

---
type: pre_commit_static_analysis_report
description: Results of running static analysis checks when committing changes.
report:
  - task: lint_filenames
    status: passed
  - task: lint_editorconfig
    status: passed
  - task: lint_markdown
    status: na
  - task: lint_package_json
    status: na
  - task: lint_repl_help
    status: na
  - task: lint_javascript_src
    status: passed
  - task: lint_javascript_cli
    status: na
  - task: lint_javascript_examples
    status: na
  - task: lint_javascript_tests
    status: na
  - task: lint_javascript_benchmarks
    status: na
  - task: lint_python
    status: na
  - task: lint_r
    status: na
  - task: lint_c_src
    status: na
  - task: lint_c_examples
    status: na
  - task: lint_c_benchmarks
    status: na
  - task: lint_c_tests_fixtures
    status: na
  - task: lint_shell
    status: na
  - task: lint_typescript_declarations
    status: na
  - task: lint_typescript_tests
    status: na
  - task: lint_license_headers
    status: passed
---

Author: Snehil-Shah

lib/node_modules/@stdlib/repl/base/ctor/lib/create_evaluation_context.js

@@ -43,21 +43,21 @@ var resolveLookupPaths = Module._resolveLookupPaths; // eslint-disable-line no-u
 * Creates a REPL evaluation context.
 *
 * @private
+* @param {Object} context - context object
 * @param {Array} out - strided output array for storing variables added to the evaluation context
 * @param {Stream} ostream - output stream
 * @param {Stream} estream - error stream
 * @param {boolean} sandbox - boolean indicating whether the evaluation context should be "sandboxed"
 * @returns {Object} context
 */
-function createContext( out, ostream, estream, sandbox ) {
-	var context;
+function createContext( context, out, ostream, estream, sandbox ) {
 	var keys;
 	var i;
 
 	// Create the REPL context...
 	if ( sandbox ) {
 		// Create a sandboxed context:
-		context = vm.createContext();
+		context = vm.createContext( context );
 
 		// Assign globals from the current global context to the sandboxed context (note: shallow copy!)...
 		keys = objectKeys( GLOBALS );

lib/node_modules/@stdlib/repl/base/ctor/lib/main.js

@@ -31,6 +31,7 @@ var hasOwnProp = require( '@stdlib/assert/has-own-property' );
 var objectKeys = require( '@stdlib/utils/keys' );
 var properties = require( '@stdlib/utils/properties' );
 var typeOf = require( '@stdlib/utils/type-of' );
+var append = require( '@stdlib/utils/append' );
 var setNonEnumerable = require( '@stdlib/utils/define-nonenumerable-property' );
 var setNonEnumerableReadOnly = require( '@stdlib/utils/define-nonenumerable-read-only-property' );
 var setReadOnlyAccessor = require( '@stdlib/utils/define-nonenumerable-read-only-accessor' );
@@ -50,6 +51,8 @@ var validate = require( './validate.js' );
 var defaults = require( './defaults.js' );
 var setAliases = require( './set_aliases.js' );
 var setAliasesGlobal = require( './set_aliases_global.js' );
+var setCommands = require( './set_commands.js' );
+var setCommandsGlobal = require( './set_commands_global.js' );
 var createEvalContext = require( './create_evaluation_context.js' );
 var completerFactory = require( './completer.js' );
 var tokenizerFactory = require( './tokenizer.js' );
@@ -448,25 +451,31 @@ setNonEnumerableReadOnly( REPL.prototype, 'tokenizer', function tokenizer( line
 * @name createContext
 * @memberof REPL.prototype
 * @type {Function}
+* @param {Object} context - context object
 * @returns {Object} REPL context
 *
 * @example
 * // Create a new REPL:
 * var repl = new REPL();
 *
 * // Return a new REPL context:
-* var ctx = repl.createContext();
+* var ctx = repl.createContext({});
 *
 * // Close the REPL:
 * repl.close();
 */
-setNonEnumerableReadOnly( REPL.prototype, 'createContext', function createContext() {
-	var context;
-
+setNonEnumerableReadOnly( REPL.prototype, 'createContext', function createContext( context ) {
+	if ( arguments.length ) {
+		if ( !isPlainObject( context ) ) {
+			throw new TypeError( format( 'invalid argument. First argument must be a plain object. Value: `%s`.', context ) );
+		}
+	} else {
+		context = {};
+	}
 	debug( 'Creating REPL execution context...' );
 
 	// Create an evaluation context:
-	context = createEvalContext( this._contextVars, this._output, this._error, this._sandbox ); // eslint-disable-line max-len
+	context = createEvalContext( context, this._contextVars, this._output, this._error, this._sandbox ); // eslint-disable-line max-len
 
 	// Add project APIs...
 	if ( this._sandbox ) {
@@ -475,6 +484,15 @@ setNonEnumerableReadOnly( REPL.prototype, 'createContext', function createContex
 		setAliasesGlobal( this._globalVars, this._aliases, context, ALIAS_OVERRIDES ); // eslint-disable-line max-len
 	}
 
+	// NOTE: the context should not be augmented **after** this point, except as done by the user when declaring variables and functions!
+
+	// Sort the list of global variables:
+	if ( this._sandbox === false ) {
+		this._globalVars.sort( propertyComparator );
+	}
+	// Capture a snapshot of the current global workspace:
+	append( this._workspace, properties( context ).sort( propertyComparator ) );
+
 	return context;
 });
 
@@ -550,6 +568,23 @@ setNonEnumerableReadOnly( REPL.prototype, 'resetContext', function resetContext(
 	return this;
 });
 
+/**
+* Sets commands on a context object.
+*
+* @name setCommands
+* @memberof REPL.prototype
+* @type {Function}
+* @param {Object} context - context object
+* @param {ArrayArray} commands - commands
+* @returns {Object} context object
+*/
+setNonEnumerableReadOnly( REPL.prototype, 'setCommands', function set( context, commands ) {
+	if ( this._sandbox ) {
+		return setCommands( context, commands );
+	}
+	return setCommandsGlobal( this._globalVars, context, commands );
+});
+
 /**
 * Creates a new workspace.
 *

lib/node_modules/@stdlib/repl/lib/set_commands.js renamed to lib/node_modules/@stdlib/repl/base/ctor/lib/set_commands.js

@@ -41,8 +41,6 @@ var readFileSync = require( '@stdlib/fs/read-file' ).sync;
 var RE_EOL = require( '@stdlib/regexp/eol' ).REGEXP;
 var setNonEnumerable = require( '@stdlib/utils/define-nonenumerable-property' );
 var setNonEnumerableReadOnly = require( '@stdlib/utils/define-nonenumerable-read-only-property' );
-var properties = require( '@stdlib/utils/properties' );
-var append = require( '@stdlib/utils/append' );
 var nextTick = require( '@stdlib/utils/next-tick' );
 var Boolean = require( '@stdlib/boolean/ctor' );
 var assign = require( '@stdlib/object/assign' );
@@ -51,9 +49,6 @@ var validate = require( './validate.js' );
 var parseKey = require( './parse_key.js' );
 var defaults = require( './defaults.js' );
 var isSettingName = require( './is_setting_name.js' );
-var setCommands = require( './set_commands.js' );
-var setCommandsGlobal = require( './set_commands_global.js' );
-var propertyComparator = require( './property_comparator.js' );
 var commands = require( './commands.js' );
 var displayPrompt = require( './display_prompt.js' );
 var inputPrompt = require( './input_prompt.js' );
@@ -685,28 +680,17 @@ setNonEnumerableReadOnly( REPL.prototype, 'createContext', function createContex
 	var context;
 	var cmds;
 
-	context = BaseREPL.prototype.createContext.call( this );
+	// Initialize context object:
+	context = {};
 
 	// Get the list of REPL-specific commands:
 	cmds = commands( this );
 
 	// Add commands requiring privileged access to internal instance variables...
-	if ( this._sandbox ) {
-		setCommands( context, cmds );
-	} else {
-		setCommandsGlobal( this._globalVars, context, cmds );
-	}
-
-	// NOTE: the context should not be augmented **after** this point, except as done by the user when declaring variables and functions!
+	context = this.setCommands( context, cmds );
 
-	// Sort the list of global variables:
-	if ( this._sandbox === false ) {
-		this._globalVars.sort( propertyComparator );
-	}
-	// Capture a snapshot of the current global workspace:
-	append( this._workspace, properties( context ).sort( propertyComparator ) );
-
-	return context;
+	// Call the parent method:
+	return BaseREPL.prototype.createContext.call( this, context );
 });
 
 /**