object.assign

codemods/globalthis/index.js

@@ -0,0 +1,37 @@
+import jscodeshift from 'jscodeshift';
+import { removeImport } from '../shared.js';
+
+/**
+ * @typedef {import('../../types.js').Codemod} Codemod
+ * @typedef {import('../../types.js').CodemodOptions} CodemodOptions
+ */
+
+/**
+ * @param {CodemodOptions} [options]
+ * @returns {Codemod}
+ */
+export default function (options) {
+	return {
+		name: 'globalthis',
+		transform: ({ file }) => {
+			const j = jscodeshift;
+			const root = j(file.source);
+			const identifier1 = removeImport('globalthis', root, j).identifier;
+			const identifier2 = removeImport(
+				'globalthis/polyfill',
+				root,
+				j,
+			).identifier;
+			const identifier3 = removeImport('globalthis/shim', root, j).identifier;
+			const identifier = identifier1 || identifier2 || identifier3;
+
+			if (identifier) {
+				root
+					.find(j.Identifier, { name: identifier })
+					.replaceWith(j.identifier('globalThis'));
+			}
+
+			return root.toSource({ quote: 'single' });
+		},
+	};
+}

codemods/object.assign/index.js

@@ -0,0 +1,46 @@
+import jscodeshift from 'jscodeshift';
+import { removeImport } from '../shared.js';
+
+/**
+ * @typedef {import('../../types.js').Codemod} Codemod
+ * @typedef {import('../../types.js').CodemodOptions} CodemodOptions
+ */
+
+/**
+ * @param {CodemodOptions} [options]
+ * @returns {Codemod}
+ */
+export default function (options) {
+	return {
+		name: 'object.assign',
+		transform: ({ file }) => {
+			const j = jscodeshift;
+			const root = j(file.source);
+			const identifierObjectAssign = removeImport(
+				'object.assign',
+				root,
+				j,
+			).identifier;
+			const identifierPolyfill = removeImport(
+				'object.assign/polyfill',
+				root,
+				j,
+			).identifier;
+			const identifierShim = removeImport(
+				'object.assign/shim',
+				root,
+				j,
+			).identifier;
+			const identifier =
+				identifierObjectAssign || identifierPolyfill || identifierShim;
+
+			if (identifier) {
+				root
+					.find(j.Identifier, { name: identifier })
+					.replaceWith(j.identifier('Object.assign'));
+			}
+
+			return root.toSource({ quote: 'single' });
+		},
+	};
+}

codemods/shared.js

@@ -33,6 +33,91 @@ export function removeImport(name, root, j) {
 		},
 	});
 
+	// Require statements with call expressions like `var globalThis = require('globalthis')()`
+	const requireCallExpression = root.find(j.VariableDeclarator, {
+		init: {
+			callee: {
+				type: 'CallExpression',
+				callee: {
+					name: 'require',
+				},
+				arguments: [
+					{
+						value: name,
+					},
+				],
+			},
+		},
+	});
+
+	// Same as above without variable declaration like `require('globalthis')()`
+	const sideEffectRequireCallExpression = root.find(j.ExpressionStatement, {
+		expression: {
+			callee: {
+				type: 'CallExpression',
+				callee: {
+					name: 'require',
+				},
+				arguments: [
+					{
+						value: name,
+					},
+				],
+			},
+		},
+	});
+
+	// Require chained call expressions like `var globalThis = require('globalthis').shim()`
+	const requireMethodCallExpression = root.find(j.VariableDeclarator, {
+		init: {
+			type: 'CallExpression',
+			callee: {
+				type: 'MemberExpression',
+				object: {
+					type: 'CallExpression',
+					callee: {
+						name: 'require',
+					},
+					arguments: [
+						{
+							value: name,
+						},
+					],
+				},
+				property: {
+					type: 'Identifier',
+				},
+			},
+		},
+	});
+
+	// Same as above without variable declaration like `require('globalthis').shim()`
+	const sideEffectRequireMethodCallExpression = root.find(
+		j.ExpressionStatement,
+		{
+			expression: {
+				type: 'CallExpression',
+				callee: {
+					type: 'MemberExpression',
+					object: {
+						type: 'CallExpression',
+						callee: {
+							name: 'require',
+						},
+						arguments: [
+							{
+								value: name,
+							},
+						],
+					},
+					property: {
+						type: 'Identifier',
+					},
+				},
+			},
+		},
+	);
+
 	// Require statements without declarations like `Object.is = require("object-is");`
 	const requireAssignment = root.find(j.AssignmentExpression, {
 		operator: '=',
@@ -64,17 +149,25 @@ export function removeImport(name, root, j) {
 
 	// Return the identifier name, e.g. 'fn' in `import { fn } from 'is-boolean-object'`
 	// or `var fn = require('is-boolean-object')`
-	const identifier =
-		importDeclaration.paths().length > 0
-			? importDeclaration.get().node.specifiers[0].local.name
-			: requireDeclaration.paths().length > 0
-				? requireDeclaration.find(j.Identifier).get().node.name
-				: requireAssignment.paths().length > 0
-					? requireAssignment.find(j.Identifier).get().node.name
-					: null;
+	let identifier = null;
+	if (importDeclaration.paths().length > 0) {
+		identifier = importDeclaration.get().node.specifiers[0].local.name;
+	} else if (requireDeclaration.paths().length > 0) {
+		identifier = requireDeclaration.find(j.Identifier).get().node.name;
+	} else if (requireCallExpression.paths().length > 0) {
+		identifier = requireCallExpression.find(j.Identifier).get().node.name;
+	} else if (requireMethodCallExpression.paths().length > 0) {
+		identifier = requireMethodCallExpression.find(j.Identifier).get().node.name;
+	} else if (requireAssignment.paths().length > 0) {
+		identifier = requireAssignment.find(j.Identifier).get().node.name;
+	}
 
 	importDeclaration.remove();
 	requireDeclaration.remove();
+	requireCallExpression.remove();
+	sideEffectRequireCallExpression.remove();
+	requireMethodCallExpression.remove();
+	sideEffectRequireMethodCallExpression.remove();
 	requireAssignment.remove();
 	sideEffectRequireExpression.remove();
 

index.js

@@ -60,6 +60,7 @@ import functionPrototypeName from './codemods/function.prototype.name/index.js';
 import functionsHaveNames from './codemods/functions-have-names/index.js';
 import getSymbolDescription from './codemods/get-symbol-description/index.js';
 import global from './codemods/global/index.js';
+import globalthis from './codemods/globalthis/index.js';
 import gopd from './codemods/gopd/index.js';
 import has from './codemods/has/index.js';
 import hasOwnProp from './codemods/has-own-prop/index.js';
@@ -109,6 +110,7 @@ import numberPrototypeToexponential from './codemods/number.prototype.toexponent
 import objectAssign from './codemods/object-assign/index.js';
 import objectIs from './codemods/object-is/index.js';
 import objectKeys from './codemods/object-keys/index.js';
+import objectAssign2 from './codemods/object.assign/index.js';
 import objectDefineproperties from './codemods/object.defineproperties/index.js';
 import objectEntries from './codemods/object.entries/index.js';
 import objectFromentries from './codemods/object.fromentries/index.js';
@@ -214,6 +216,7 @@ export const codemods = {
   "functions-have-names": functionsHaveNames,
   "get-symbol-description": getSymbolDescription,
   "global": global,
+  "globalthis": globalthis,
   "gopd": gopd,
   "has": has,
   "has-own-prop": hasOwnProp,
@@ -263,6 +266,7 @@ export const codemods = {
   "object-assign": objectAssign,
   "object-is": objectIs,
   "object-keys": objectKeys,
+  "object.assign": objectAssign2,
   "object.defineproperties": objectDefineproperties,
   "object.entries": objectEntries,
   "object.fromentries": objectFromentries,
@@ -304,4 +308,4 @@ export const codemods = {
   "typed-array-length": typedArrayLength,
   "typedarray.prototype.slice": typedarrayPrototypeSlice,
   "xtend": xtend,
-};
+};

test/fixtures/globalthis/case-1/after.js

@@ -0,0 +1 @@
+globalThis.a = 42;

test/fixtures/globalthis/case-1/before.js

@@ -0,0 +1,2 @@
+var globalThis = require('globalthis')();
+globalThis.a = 42;

test/fixtures/globalthis/case-1/result.js

@@ -0,0 +1 @@
+globalThis.a = 42;

test/fixtures/globalthis/case-2/after.js

@@ -0,0 +1 @@
+globalThis.a = 42;

test/fixtures/globalthis/case-2/before.js

@@ -0,0 +1,2 @@
+var globalThis = require('globalthis/polyfill')();
+globalThis.a = 42;

test/fixtures/globalthis/case-2/result.js

@@ -0,0 +1 @@
+globalThis.a = 42;

test/fixtures/globalthis/case-3/after.js

@@ -0,0 +1 @@
+globalThis.a = 42;

test/fixtures/globalthis/case-3/before.js

@@ -0,0 +1,2 @@
+var newGlobalThis = require('globalthis')();
+newGlobalThis.a = 42;

test/fixtures/globalthis/case-3/result.js

@@ -0,0 +1 @@
+globalThis.a = 42;

test/fixtures/globalthis/case-4/after.js

@@ -0,0 +1 @@
+globalThis.a = 42;

test/fixtures/globalthis/case-4/before.js

@@ -0,0 +1,2 @@
+var shimmedGlobal = require('globalthis').shim();
+shimmedGlobal.a = 42;

test/fixtures/globalthis/case-4/result.js

@@ -0,0 +1 @@
+globalThis.a = 42;

test/fixtures/globalthis/case-5/after.js

@@ -0,0 +1 @@
+globalThis.a = 42;

test/fixtures/globalthis/case-5/before.js

@@ -0,0 +1,2 @@
+var shimmedGlobal = require('globalthis/shim')();
+shimmedGlobal.a = 42;

test/fixtures/globalthis/case-5/result.js

@@ -0,0 +1 @@
+globalThis.a = 42;

test/fixtures/object.assign/case-1/after.js

@@ -0,0 +1,13 @@
+var assert = require('assert');
+
+var target = { a: true };
+var source1 = { b: true };
+var source2 = { c: true };
+Object.assign(target, source1, source2);
+
+var expected = {
+    a: true,
+    b: true,
+    c: true,
+};
+assert.deepEqual(target, expected);

test/fixtures/object.assign/case-1/before.js

@@ -0,0 +1,14 @@
+var assign = require('object.assign').getPolyfill();
+var assert = require('assert');
+
+var target = { a: true };
+var source1 = { b: true };
+var source2 = { c: true };
+assign(target, source1, source2);
+
+var expected = {
+    a: true,
+    b: true,
+    c: true,
+};
+assert.deepEqual(target, expected);

test/fixtures/object.assign/case-1/result.js

@@ -0,0 +1,13 @@
+var assert = require('assert');
+
+var target = { a: true };
+var source1 = { b: true };
+var source2 = { c: true };
+Object.assign(target, source1, source2);
+
+var expected = {
+    a: true,
+    b: true,
+    c: true,
+};
+assert.deepEqual(target, expected);

test/fixtures/object.assign/case-2/after.js

@@ -0,0 +1,10 @@
+var assert = require('assert');
+
+var target = { a: true };
+Object.assign(target, { b: true });
+
+var expected = {
+    a: true,
+    b: true,
+};
+assert.deepEqual(target, expected);

test/fixtures/object.assign/case-2/before.js

@@ -0,0 +1,11 @@
+var assign = require('object.assign/polyfill')();
+var assert = require('assert');
+
+var target = { a: true };
+assign(target, { b: true });
+
+var expected = {
+    a: true,
+    b: true,
+};
+assert.deepEqual(target, expected);

test/fixtures/object.assign/case-2/result.js

@@ -0,0 +1,10 @@
+var assert = require('assert');
+
+var target = { a: true };
+Object.assign(target, { b: true });
+
+var expected = {
+    a: true,
+    b: true,
+};
+assert.deepEqual(target, expected);

test/fixtures/object.assign/case-3/after.js

@@ -0,0 +1,2 @@
+var target = { a: 1 };
+Object.assign(target, { b: 2 });

test/fixtures/object.assign/case-3/before.js

@@ -0,0 +1,3 @@
+var assign = require('object.assign').shim();
+var target = { a: 1 };
+assign(target, { b: 2 });

test/fixtures/object.assign/case-3/result.js

@@ -0,0 +1,2 @@
+var target = { a: 1 };
+Object.assign(target, { b: 2 });

test/fixtures/object.assign/case-4/after.js

@@ -0,0 +1,2 @@
+var target = { a: 1 };
+Object.assign(target, { b: 2 });