@contember/bindx-repeater 0.1.20 → 0.1.22

package/dist/components/BlockRepeater.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"BlockRepeater.d.ts","sourceRoot":"","sources":["../../src/components/BlockRepeater.tsx"],"names":[],"mappings":"AAAA,OAAc,EAAW,KAAK,YAAY,EAAkB,MAAM,OAAO,CAAA;AACzE,OAAO,KAAK,EAGX,QAAQ,EAIR,MAAM,kBAAkB,CAAA;AAEzB,OAAO,EAAyC,eAAe,EAAa,KAAK,iBAAiB,EAAwB,MAAM,wBAAwB,CAAA;AACxJ,OAAO,KAAK,EACX,kBAAkB,EAMlB,MAAM,aAAa,CAAA;AAMpB;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAmCG;AACH,wBAAgB,aAAa,CAC5B,OAAO,SAAS,MAAM,GAAG,MAAM,EAC/B,SAAS,GAAG,OAAO,EACnB,MAAM,SAAS,QAAQ,GAAG,QAAQ,EAClC,WAAW,SAAS,MAAM,GAAG,MAAM,EACnC,OAAO,SAAS,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,GAAG,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,EAC/D,WAAW,SAAS,MAAM,GAAG,MAAM,EAClC,EACD,KAAK,EACL,mBAAmB,EACnB,UAAU,EACV,MAAM,EACN,QAAQ,GACR,EAAE,kBAAkB,CAAC,OAAO,EAAE,SAAS,EAAE,MAAM,EAAE,WAAW,EAAE,OAAO,EAAE,WAAW,CAAC,GAAG,YAAY,CAgIlG;AAGD,QAAA,MAAM,0BAA0B,EAAoB,OAAO,aAAa,GAAG,iBAAiB,GAAG;IAAE,CAAC,eAAe,CAAC,EAAE,IAAI,CAAA;CAAE,CAAA;AA+F1H,OAAO,EAAE,0BAA0B,IAAI,qBAAqB,EAAE,CAAA"}
+{"version":3,"file":"BlockRepeater.d.ts","sourceRoot":"","sources":["../../src/components/BlockRepeater.tsx"],"names":[],"mappings":"AAAA,OAAc,EAAW,KAAK,YAAY,EAAkB,MAAM,OAAO,CAAA;AACzE,OAAO,KAAK,EAGX,QAAQ,EAIR,MAAM,kBAAkB,CAAA;AAEzB,OAAO,EAAyC,eAAe,EAAa,KAAK,iBAAiB,EAAwB,MAAM,wBAAwB,CAAA;AACxJ,OAAO,KAAK,EACX,kBAAkB,EAMlB,MAAM,aAAa,CAAA;AAMpB;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAmCG;AACH,wBAAgB,aAAa,CAC5B,OAAO,SAAS,MAAM,GAAG,MAAM,EAC/B,SAAS,GAAG,OAAO,EACnB,MAAM,SAAS,QAAQ,GAAG,QAAQ,EAClC,WAAW,SAAS,MAAM,GAAG,MAAM,EACnC,OAAO,SAAS,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,GAAG,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,EAC/D,WAAW,SAAS,MAAM,GAAG,MAAM,EAClC,EACD,KAAK,EACL,mBAAmB,EACnB,UAAU,EACV,MAAM,EACN,QAAQ,GACR,EAAE,kBAAkB,CAAC,OAAO,EAAE,SAAS,EAAE,MAAM,EAAE,WAAW,EAAE,OAAO,EAAE,WAAW,CAAC,GAAG,YAAY,CAgIlG;AAsGD,QAAA,MAAM,0BAA0B;uBAhGoE,IAAI;CAgGnC,CAAA;AAErE,OAAO,EAAE,0BAA0B,IAAI,qBAAqB,EAAE,CAAA"}

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@contember/bindx-repeater",
-	"version": "0.1.20",
+	"version": "0.1.22",
 	"description": "Repeater components for has-many list management",
 	"type": "module",
 	"main": "./src/index.ts",
@@ -15,8 +15,8 @@
 		"react": ">=18.0.0"
 	},
 	"dependencies": {
-		"@contember/bindx": "0.1.20",
-		"@contember/bindx-react": "0.1.20"
+		"@contember/bindx": "0.1.22",
+		"@contember/bindx-react": "0.1.22"
 	},
 	"files": [
 		"dist",

package/src/components/BlockRepeater.tsx

@@ -201,13 +201,16 @@ export function BlockRepeater<
 	return <>{children(items, methods)}</>
 }
 
-// Static method for selection extraction
-const blockRepeaterWithSelection = BlockRepeater as typeof BlockRepeater & SelectionProvider & { [BINDX_COMPONENT]: true }
+// Wrap BlockRepeater with selection extraction inside a factory function
+// so the property assignments are not module-level side effects
+// (Vite dep optimizer marks module-level side effects for DCE).
+function createBlockRepeaterWithSelection() {
+	const component = BlockRepeater as typeof BlockRepeater & SelectionProvider & { [BINDX_COMPONENT]: true }
 
-blockRepeaterWithSelection.getSelection = (
-	props: BlockRepeaterProps<unknown>,
-	collectNested: (children: ReactNode) => SelectionMeta,
-): SelectionFieldMeta | null => {
+	component.getSelection = (
+		props: BlockRepeaterProps<unknown>,
+		collectNested: (children: ReactNode) => SelectionMeta,
+	): SelectionFieldMeta | null => {
 	// Check if the field is a collector proxy with a scope reference (collection phase).
 	// When present, we merge the collected selection directly into the scope tree,
 	// which correctly handles deeply nested relations (e.g., page.blocks.items).
@@ -218,15 +221,6 @@ blockRepeaterWithSelection.getSelection = (
 	const scope = new SelectionScope()
 	const collectorEntity = createCollectorProxy<unknown>(scope)
 
-	// Collect block render/form JSX so the collector proxy records field accesses.
-	// Done inside mockItems.map callback to survive Rolldown dead-code elimination
-	// (the callback is invoked by props.children which Rolldown cannot analyze).
-	const blockJsx: ReactNode[] = []
-	for (const blockDef of Object.values(props.blocks) as BlockDefinition[]) {
-		if (blockDef.render) blockJsx.push(blockDef.render(collectorEntity as EntityAccessor<object>))
-		if (blockDef.form) blockJsx.push(blockDef.form(collectorEntity as EntityAccessor<object>))
-	}
-
 	const mockItems: BlockRepeaterItems<unknown> = {
 		map: (fn) => {
 			fn(collectorEntity, {
@@ -251,6 +245,14 @@ blockRepeaterWithSelection.getSelection = (
 	}
 
 	const syntheticChildren = props.children(mockItems, mockMethods)
+
+	// Call block render/form functions so the collector proxy records field accesses
+	const blockJsx: ReactNode[] = []
+	for (const blockDef of Object.values(props.blocks) as BlockDefinition[]) {
+		if (blockDef.render) blockJsx.push(blockDef.render(collectorEntity as EntityAccessor<object>))
+		if (blockDef.form) blockJsx.push(blockDef.form(collectorEntity as EntityAccessor<object>))
+	}
+
 	const jsxSelection = collectNested([syntheticChildren, ...blockJsx])
 
 	const nestedSelection = scope.toSelectionMeta()
@@ -295,6 +297,10 @@ blockRepeaterWithSelection.getSelection = (
 	}
 }
 
-blockRepeaterWithSelection[BINDX_COMPONENT] = true
+	component[BINDX_COMPONENT] = true
+	return component
+}
+
+const blockRepeaterWithSelection = createBlockRepeaterWithSelection()
 
 export { blockRepeaterWithSelection as BlockRepeaterWithMeta }