@pyreon/attrs 0.24.0 → 0.24.2

package/lib/index.js

@@ -1,11 +1,14 @@
 import { compose, hoistNonReactStatics, isEmpty, omit, pick } from "@pyreon/ui-core";
 
 //#region src/utils/attrs.ts
-const removeUndefinedProps = ((props) => Object.keys(props).reduce((acc, key) => {
-	const currentValue = props[key];
-	if (currentValue !== void 0) acc[key] = currentValue;
-	return acc;
-}, {}));
+const removeUndefinedProps = ((props) => {
+	const result = {};
+	for (const key in props) {
+		const value = props[key];
+		if (value !== void 0) result[key] = value;
+	}
+	return result;
+});
 const calculateChainOptions = (options) => (args) => {
 	const result = {};
 	if (!options || isEmpty(options)) return result;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pyreon/attrs",
-  "version": "0.24.0",
+  "version": "0.24.2",
   "description": "Attrs HOC chaining for Pyreon components",
   "license": "MIT",
   "repository": {
@@ -41,14 +41,14 @@
     "typecheck": "tsc --noEmit"
   },
   "devDependencies": {
-    "@pyreon/typescript": "^0.24.0",
+    "@pyreon/typescript": "^0.24.2",
     "@vitus-labs/tools-rolldown": "^2.4.0"
   },
   "engines": {
     "node": ">= 22"
   },
   "dependencies": {
-    "@pyreon/core": "^0.24.0",
-    "@pyreon/ui-core": "^0.24.0"
+    "@pyreon/core": "^0.24.2",
+    "@pyreon/ui-core": "^0.24.2"
   }
 }

package/src/utils/attrs.ts

@@ -10,12 +10,18 @@ type RemoveUndefinedProps = <T extends Record<string, any>>(
   props: T,
 ) => { [I in keyof T as T[I] extends undefined ? never : I]: T[I] }
 
-export const removeUndefinedProps = (<T extends Record<string, any>>(props: T) =>
-  Object.keys(props).reduce<Record<string, unknown>>((acc, key) => {
-    const currentValue = props[key]
-    if (currentValue !== undefined) acc[key] = currentValue
-    return acc
-  }, {})) as RemoveUndefinedProps
+export const removeUndefinedProps = (<T extends Record<string, any>>(props: T) => {
+  // Direct for-in loop avoids the `Object.keys` array allocation that
+  // the prior `reduce` over `Object.keys(props)` paid on every render.
+  // The hot path fires on every content-equal re-render of any attrs-
+  // wrapped component. Ported from vitus-labs `b003de47`.
+  const result: Record<string, unknown> = {}
+  for (const key in props) {
+    const value = props[key]
+    if (value !== undefined) result[key] = value
+  }
+  return result
+}) as RemoveUndefinedProps
 
 /**
  * Reduces an array of option functions (from chained `.attrs()` calls)