Ref: moving cleaning into makeTransformer() helper.

RobinTail · RobinTail · commit 7a3f123f77dc · 2025-10-27T06:13:28.000+01:00
diff --git a/zod-plugin/src/remap.ts b/zod-plugin/src/remap.ts
@@ -27,23 +27,20 @@ type Mapper = <T extends Record<string, unknown>>(
   subject: T,
 ) => { [P in string | keyof T]: T[keyof T] };
 
-/** @todo naming */
-const ensure = (tool: Record<string, string>) => {
-  const targets = Object.values(tool);
+const makeTransformer = (mapping: Partial<Record<string, string>>) => {
+  const clean = R.reject(R.isNil, mapping); // rejecting undefined
+  const targets = Object.values(clean);
   if (new Set(targets).size !== targets.length)
-    throw new Error("remap(): duplicate target keys", { cause: tool });
-  return tool;
+    throw new Error("remap(): duplicate target keys", { cause: mapping });
+  return R.renameKeys(clean);
 };
 
 /** Used by runtime (bound) */
 export const remap = function (
   this: z.ZodObject,
-  tool: Partial<Record<string, string>> | Mapper,
+  tool: Parameters<typeof makeTransformer>[0] | Mapper,
 ) {
-  const transformer =
-    typeof tool === "function"
-      ? tool
-      : R.renameKeys(ensure(R.reject(R.isNil, tool))); // rejecting undefined
+  const transformer = typeof tool === "function" ? tool : makeTransformer(tool);
   const nextShape = transformer(
     R.map(R.invoker(0, "clone"), this._zod.def.shape), // immutable, changed from R.clone due to failure
   );
