Silence potentially upcoming circular dependency warning (#973)

addaleax · nfischer · commit a1111ee793e0 · 2020-04-24T20:45:11.000-07:00
Node.js is currently considering printing a warning when a non-existent property of `module.exports` is accessed while in a circular `require()` dependency, in order to make it easier to catch issues with circular dependencies. In order to avoid printing these warnings for shelljs, checking for the property’s existence rather than its truthiness suffices. Refs: nodejs/node#29935
diff --git a/src/common.js b/src/common.js
@@ -450,7 +450,7 @@ function _register(name, implementation, wrapOptions) {
   // If an option isn't specified, use the default
   wrapOptions = Object.assign({}, DEFAULT_WRAP_OPTIONS, wrapOptions);
 
-  if (shell[name]) {
+  if (shell.hasOwnProperty(name)) {
     throw new Error('Command `' + name + '` already exists');
   }
 

Original file line number	Diff line number	Diff line change
`@@ -450,7 +450,7 @@ function _register(name, implementation, wrapOptions) {`
`450`	`450`	`// If an option isn't specified, use the default`
`451`	`451`	`wrapOptions = Object.assign({}, DEFAULT_WRAP_OPTIONS, wrapOptions);`
`452`	`452`
`453`		`- if (shell[name]) {`
	`453`	`+ if (shell.hasOwnProperty(name)) {`
`454`	`454`	throw new Error('Command `' + name + '` already exists');
`455`	`455`	`}`
`456`	`456`