Golf: loose equality where the operands make it identical

Only converted comparisons whose operands are already known to be the same
type (typeof results, for-in keys, string literals, numbers, object
identity). Value comparisons that can see coercible operands (value ===
true/false/null/'', defaultProps undefined checks, prop diffing) stay
strict. Core was left alone: the same change measured +1 B br there.

compat       3490 -> 3482 B br
hooks        1324 -> 1322 B br
jsx-runtime   740 ->  735 B br

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

Author: JoviDeCroock

compat/src/Children.js

@@ -14,7 +14,7 @@ export const Children = {
 	},
 	only(children) {
 		const normalized = toChildArray(children);
-		if (normalized.length !== 1) throw 'Children.only';
+		if (normalized.length != 1) throw 'Children.only';
 		return normalized[0];
 	},
 	toArray: toChildArray

compat/src/index.js

@@ -59,7 +59,7 @@ function createFactory(type) {
  * @returns {boolean}
  */
 function isValidElement(element) {
-	return !!element && element.$$typeof === REACT_ELEMENT_TYPE;
+	return !!element && element.$$typeof == REACT_ELEMENT_TYPE;
 }
 
 /**
@@ -68,7 +68,7 @@ function isValidElement(element) {
  * @returns {boolean}
  */
 function isFragment(element) {
-	return isValidElement(element) && element.type === Fragment;
+	return isValidElement(element) && element.type == Fragment;
 }
 
 /**
@@ -118,7 +118,7 @@ function findDOMNode(component) {
 	return (
 		(component &&
 			((component._vnode && component._vnode._dom) ||
-				(component.nodeType === 1 && component))) ||
+				(component.nodeType == 1 && component))) ||
 		null
 	);
 }

compat/src/render.js

@@ -28,7 +28,7 @@ const MODE_HYDRATE = 1 << 5;
 const CAMEL_PROPS =
 	/^(?:accent|alignment|arabic|baseline|cap|clip(?!PathU)|color|dominant|fill|flood|font|glyph(?!R)|horiz|image(!S)|letter|lighting|marker(?!H|W|U)|overline|paint|pointer|shape|stop|strikethrough|stroke|text(?!L)|transform|underline|unicode|units|v|vector|vert|word|writing|x(?!C))[A-Z]/;
 const CAMEL_REPLACE = /[A-Z0-9]/g;
-const IS_DOM = typeof document !== 'undefined';
+const IS_DOM = typeof document != 'undefined';
 
 /**
  * This is taken from https://github.com/facebook/react/blob/main/packages/use-sync-external-store/src/useSyncExternalStoreShimClient.js#L84
@@ -177,63 +177,59 @@ function handleDomVNode(vnode) {
 		let value = props[i];
 
 		if (
-			(i === 'value' && 'defaultValue' in props && value == null) ||
+			(i == 'value' && 'defaultValue' in props && value == null) ||
 			// Emulate React's behavior of not rendering the contents of noscript tags on the client.
-			(IS_DOM && i === 'children' && type === 'noscript') ||
-			i === 'class' ||
-			i === 'className'
+			(IS_DOM && i == 'children' && type == 'noscript') ||
+			i == 'class' ||
+			i == 'className'
 		) {
 			// Skip applying value if it is null/undefined and we already set
 			// a default value
 			continue;
 		}
 
-		if (i === 'style' && typeof value === 'object') {
+		if (i == 'style' && typeof value == 'object') {
 			let cloned;
 			for (let key in value) {
-				if (typeof value[key] === 'number' && !IS_NON_DIMENSIONAL.test(key)) {
+				if (typeof value[key] == 'number' && !IS_NON_DIMENSIONAL.test(key)) {
 					if (!cloned) {
 						cloned = value = assign({}, value);
 					}
 					value[key] += 'px';
 				}
 			}
-		} else if (
-			i === 'defaultValue' &&
-			'value' in props &&
-			props.value == null
-		) {
+		} else if (i == 'defaultValue' && 'value' in props && props.value == null) {
 			// `defaultValue` is treated as a fallback `value` when a value prop is present but null/undefined.
 			// `defaultValue` for Elements with no value prop is the same as the DOM defaultValue property.
 			i = 'value';
-		} else if (i === 'download' && value === true) {
+		} else if (i == 'download' && value === true) {
 			// Calling `setAttribute` with a truthy value will lead to it being
 			// passed as a stringified value, e.g. `download="true"`. React
 			// converts it to an empty string instead, otherwise the attribute
 			// value will be used as the file name and the file will be called
 			// "true" upon downloading it.
 			value = '';
-		} else if (i === 'translate' && value === 'no') {
+		} else if (i == 'translate' && value === 'no') {
 			value = false;
-		} else if (i[0] === 'o' && i[1] === 'n') {
+		} else if (i[0] == 'o' && i[1] == 'n') {
 			let lowerCased = i.toLowerCase();
-			if (lowerCased === 'ondoubleclick') {
+			if (lowerCased == 'ondoubleclick') {
 				i = 'ondblclick';
 			} else if (
-				lowerCased === 'onchange' &&
-				(type === 'input' || type === 'textarea') &&
+				lowerCased == 'onchange' &&
+				(type == 'input' || type == 'textarea') &&
 				!onChangeInputType(props.type)
 			) {
 				lowerCased = i = 'oninput';
-			} else if (lowerCased === 'onfocus') {
+			} else if (lowerCased == 'onfocus') {
 				i = 'onfocusin';
-			} else if (lowerCased === 'onblur') {
+			} else if (lowerCased == 'onblur') {
 				i = 'onfocusout';
 			}
 
 			// Add support for onInput and onChange, see #3561
 			// if we have an oninput prop already change it to oninputCapture
-			if (lowerCased === 'oninput') {
+			if (lowerCased == 'oninput') {
 				i = lowerCased;
 				if (normalizedProps[i]) {
 					i = 'oninputCapture';
@@ -289,9 +285,9 @@ function handleDomVNode(vnode) {
 let oldVNodeHook = options.vnode;
 options.vnode = vnode => {
 	// only normalize props on Element nodes
-	if (typeof vnode.type === 'string') {
+	if (typeof vnode.type == 'string') {
 		handleDomVNode(vnode);
-	} else if (typeof vnode.type === 'function') {
+	} else if (typeof vnode.type == 'function') {
 		const shouldApplyRef =
 			'prototype' in vnode.type && vnode.type.prototype.render;
 		if ('ref' in vnode.props && shouldApplyRef) {
@@ -337,7 +333,7 @@ options.diffed = function (vnode) {
 
 	if (
 		dom != null &&
-		vnode.type === 'textarea' &&
+		vnode.type == 'textarea' &&
 		'value' in props &&
 		props.value !== dom.value
 	) {

compat/src/suspense.js

@@ -52,7 +52,7 @@ function detachedClone(vnode, detachedParent, parentDom) {
 
 		vnode = assign({ constructor: UNDEFINED }, vnode);
 		if (vnode._component != null) {
-			if (vnode._component._parentDom === parentDom) {
+			if (vnode._component._parentDom == parentDom) {
 				vnode._component._parentDom = detachedParent;
 			}
 
@@ -85,7 +85,7 @@ function removeOriginal(vnode, detachedParent, originalParent) {
 			);
 
 		if (vnode._component) {
-			if (vnode._component._parentDom === detachedParent) {
+			if (vnode._component._parentDom == detachedParent) {
 				if (vnode._dom) {
 					originalParent.appendChild(vnode._dom);
 				}

compat/src/util.js

@@ -7,8 +7,8 @@ export const assign = Object.assign;
  * @returns {boolean}
  */
 export function shallowDiffers(a, b) {
-	for (let i in a) if (i !== '__source' && a[i] !== b[i]) return true;
-	for (let i in b) if (i !== '__source' && !(i in a)) return true;
+	for (let i in a) if (i != '__source' && a[i] !== b[i]) return true;
+	for (let i in b) if (i != '__source' && !(i in a)) return true;
 	return false;
 }
 

hooks/src/index.js

@@ -56,7 +56,7 @@ options._render = vnode => {
 
 	const hooks = currentComponent.__hooks;
 	if (hooks) {
-		if (previousComponent === currentComponent) {
+		if (previousComponent == currentComponent) {
 			currentComponent._renderCallbacks = [];
 		} else {
 			hooks._pendingEffects.some(invokeCleanup);
@@ -223,7 +223,7 @@ export function useReducer(reducer, initialState, init) {
 				// have values that aren't equal to one another this pushes
 				// us to update further down the tree
 				let updatedHook = false;
-				let shouldUpdate = this.props !== p;
+				let shouldUpdate = this.props != p;
 				hooks._list.some(hookItem => {
 					if (hookItem._nextValue) {
 						updatedHook = true;
@@ -468,7 +468,7 @@ function afterNextFrame(callback) {
  * @returns {void}
  */
 function afterPaint(newQueueLength) {
-	if (newQueueLength === 1 || prevRaf !== options.requestAnimationFrame) {
+	if (newQueueLength == 1 || prevRaf != options.requestAnimationFrame) {
 		prevRaf = options.requestAnimationFrame;
 		(prevRaf || afterNextFrame)(flushAfterPaintEffects);
 	}
@@ -512,7 +512,7 @@ function invokeEffect(hook) {
 function argsChanged(oldArgs, newArgs) {
 	return (
 		!oldArgs ||
-		oldArgs.length !== newArgs.length ||
+		oldArgs.length != newArgs.length ||
 		newArgs.some((arg, index) => !ObjectIs(arg, oldArgs[index]))
 	);
 }

jsx-runtime/src/index.js

@@ -92,9 +92,9 @@ const CSS_REGEX = /[A-Z]/g;
  * @returns {*}
  */
 function normalizeAttrValue(value) {
-	return value !== null &&
-		typeof value === 'object' &&
-		typeof value.valueOf === 'function'
+	return value != null &&
+		typeof value == 'object' &&
+		typeof value.valueOf == 'function'
 		? value.valueOf()
 		: value;
 }
@@ -110,13 +110,13 @@ function normalizeAttrValue(value) {
 function jsxAttr(name, value) {
 	if (options.attr) {
 		const result = options.attr(name, value);
-		if (typeof result === 'string') return result;
+		if (typeof result == 'string') return result;
 	}
 
 	value = normalizeAttrValue(value);
 
-	if (name === 'ref' || name === 'key') return '';
-	if (name === 'style' && typeof value === 'object') {
+	if (name == 'ref' || name == 'key') return '';
+	if (name == 'style' && typeof value == 'object') {
 		let str = '';
 		for (let prop in value) {
 			let val = value[prop];
@@ -136,8 +136,8 @@ function jsxAttr(name, value) {
 	if (
 		value == null ||
 		value === false ||
-		typeof value === 'function' ||
-		typeof value === 'object'
+		typeof value == 'function' ||
+		typeof value == 'object'
 	) {
 		return '';
 	} else if (value === true) return name;
@@ -155,13 +155,13 @@ function jsxAttr(name, value) {
 function jsxEscape(value) {
 	if (
 		value == null ||
-		typeof value === 'boolean' ||
-		typeof value === 'function'
+		typeof value == 'boolean' ||
+		typeof value == 'function'
 	) {
 		return null;
 	}
 
-	if (typeof value === 'object') {
+	if (typeof value == 'object') {
 		// Check for VNode
 		if (value.constructor === undefined) return value;
 

jsx-runtime/src/utils.js

@@ -3,7 +3,7 @@ const ENCODED_ENTITIES = /["&<]/;
 /** @param {string} str */
 export function encodeEntities(str) {
 	// Skip all work for strings with no entities needing encoding:
-	if (str.length === 0 || ENCODED_ENTITIES.test(str) === false) return str;
+	if (!str.length || !ENCODED_ENTITIES.test(str)) return str;
 
 	let last = 0,
 		i = 0,
@@ -26,11 +26,11 @@ export function encodeEntities(str) {
 				continue;
 		}
 		// Append skipped/buffered characters and the encoded entity:
-		if (i !== last) out += str.slice(last, i);
+		if (i != last) out += str.slice(last, i);
 		out += ch;
 		// Start the next seek/buffer after the entity's offset:
 		last = i + 1;
 	}
-	if (i !== last) out += str.slice(last, i);
+	if (i != last) out += str.slice(last, i);
 	return out;
 }