@workbench-kit/field-remap 0.0.2-prototype.0.2.30 → 0.0.2-prototype.0.2.31

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@workbench-kit/field-remap",
-  "version": "0.0.2-prototype.0.2.30",
+  "version": "0.0.2-prototype.0.2.31",
   "private": false,
   "type": "module",
   "exports": {

package/src/domain/mapping/mappingOperators.ts

@@ -16,8 +16,13 @@ import type {
   ValueTransformRegistry,
 } from '../types.js';
 import { applyTransformChain } from '../../registry/createValueTransformRegistry.js';
-import { findSourceField, findTargetSlot } from './resolveMappedValue.js';
-import { isPlainObject, readObjectPath, writeObjectPath } from './pathUtils.js';
+import {
+  findSourceField,
+  findTargetSlot,
+  readSourceFieldValue,
+  resolveTargetSlotOutputPath,
+} from './resolveMappedValue.js';
+import { isPlainObject, writeObjectPath } from './pathUtils.js';
 
 /** Max inputs on a combine operator. */
 export const MAX_MAPPING_FAN_IN = 8;
@@ -68,45 +73,6 @@ function leafKey(field: {
   return idParts[idParts.length - 1] ?? field.label;
 }
 
-function outputPathForTarget(slot: TargetSlot): string {
-  const path = slot.path?.trim();
-  if (path) {
-    return path;
-  }
-  const id = slot.id.trim();
-  if (id.includes('.')) {
-    const parts = id.split('.').filter(Boolean);
-    if (parts.length >= 2) {
-      return parts.slice(1).join('.');
-    }
-  }
-  return slot.label.trim() || id;
-}
-
-function readFieldValue(field: SourceField, inputs: Readonly<Record<string, unknown>>): unknown {
-  const shapeId = field.shapeId?.trim();
-  const bag =
-    shapeId && Object.prototype.hasOwnProperty.call(inputs, shapeId)
-      ? inputs[shapeId]
-      : Object.keys(inputs).length === 1
-        ? inputs[Object.keys(inputs)[0]!]
-        : inputs;
-
-  const path = field.path?.trim();
-  if (path) {
-    if (shapeId && Object.prototype.hasOwnProperty.call(inputs, shapeId)) {
-      return readObjectPath(bag, path);
-    }
-    const fromBag = readObjectPath(bag, path);
-    if (fromBag !== undefined) {
-      return fromBag;
-    }
-    return readObjectPath(inputs, path);
-  }
-
-  return field.sampleValue;
-}
-
 function sanitizeOperatorTransformIds(ids: readonly string[] | undefined): string[] | undefined {
   const cleaned = ids
     ?.map((id) => id.trim())
@@ -234,7 +200,7 @@ export async function applyMappingOperators(
         if (!field) {
           throw new MappingOperatorError(operator.id, `Unknown source field "${fieldId}".`);
         }
-        bag[leafKey(field)] = readFieldValue(field, input.inputs);
+        bag[leafKey(field)] = readSourceFieldValue(field, input.inputs);
       }
 
       const target = findTargetSlot(input.targets, operator.outputSlotId);
@@ -251,7 +217,7 @@ export async function applyMappingOperators(
         bag,
         context,
       );
-      output = writeObjectPath(output, outputPathForTarget(target), value);
+      output = writeObjectPath(output, resolveTargetSlotOutputPath(target), value);
       continue;
     }
 
@@ -264,7 +230,7 @@ export async function applyMappingOperators(
       );
     }
 
-    let current = readFieldValue(source, input.inputs);
+    let current = readSourceFieldValue(source, input.inputs);
     current = await applyTransformChain(
       input.transforms,
       operator.transformIds ?? [],
@@ -285,7 +251,7 @@ export async function applyMappingOperators(
         throw new MappingOperatorError(operator.id, `Unknown target slot "${slotId}".`);
       }
       const key = leafKey(target);
-      output = writeObjectPath(output, outputPathForTarget(target), current[key]);
+      output = writeObjectPath(output, resolveTargetSlotOutputPath(target), current[key]);
     }
   }
 

package/src/domain/mapping/resolveMappedValue.ts

@@ -1,11 +1,12 @@
 import type {
   MappingEdge,
   SourceField,
+  TargetSlot,
   TransformContext,
   ValueTransformRegistry,
 } from '../types.js';
 import { edgeItemTransformIds, edgeTransformIds } from '../document/mappingEdge.js';
-import { projectCollectionItems } from './pathUtils.js';
+import { projectCollectionItems, readObjectPath } from './pathUtils.js';
 import { applyTransformChain } from '../../registry/createValueTransformRegistry.js';
 import { BUILTIN_TRANSFORM_IDS } from '../../registry/builtinTransforms.js';
 import { resolveOptionSteps } from './transformOptions.js';
@@ -22,6 +23,44 @@ export function findSourceField(
   return flattenSourceFields(fields).find((field) => field.id === fieldId);
 }
 
+export function readSourceFieldValue(
+  field: SourceField,
+  inputs: Readonly<Record<string, unknown>>,
+): unknown {
+  const shapeId = field.shapeId?.trim();
+  const bag =
+    shapeId && Object.prototype.hasOwnProperty.call(inputs, shapeId)
+      ? inputs[shapeId]
+      : Object.keys(inputs).length === 1
+        ? inputs[Object.keys(inputs)[0]!]
+        : inputs;
+
+  const path = field.path?.trim();
+  if (!path) {
+    return field.sampleValue;
+  }
+  if (shapeId && Object.prototype.hasOwnProperty.call(inputs, shapeId)) {
+    return readObjectPath(bag, path);
+  }
+  const fromBag = readObjectPath(bag, path);
+  return fromBag === undefined ? readObjectPath(inputs, path) : fromBag;
+}
+
+export function resolveTargetSlotOutputPath(slot: TargetSlot): string {
+  const path = slot.path?.trim();
+  if (path) {
+    return path;
+  }
+  const id = slot.id.trim();
+  if (id.includes('.')) {
+    const parts = id.split('.').filter(Boolean);
+    if (parts.length >= 2) {
+      return parts.slice(1).join('.');
+    }
+  }
+  return slot.label.trim() || id;
+}
+
 /**
  * Resolve an edge against a live / sample source value.
  *

package/src/domain/shapes/convertToShape.ts

@@ -7,15 +7,15 @@ import {
   type DataShapeRegistry,
 } from './dataShape.js';
 import { convertArrayWithItemEdges } from '../mapping/convertItemEdges.js';
-import { isPlainObject, readObjectPath, writeObjectPath } from '../mapping/pathUtils.js';
-import { findSourceField, resolveMappedValue } from '../mapping/resolveMappedValue.js';
+import { isPlainObject, writeObjectPath } from '../mapping/pathUtils.js';
+import {
+  findSourceField,
+  readSourceFieldValue,
+  resolveMappedValue,
+  resolveTargetSlotOutputPath,
+} from '../mapping/resolveMappedValue.js';
 import { findTargetSlot, flattenTargetSlots } from '../mapping/treeUtils.js';
-import type {
-  SourceField,
-  TargetSlot,
-  TransformContext,
-  ValueTransformRegistry,
-} from '../types.js';
+import type { TransformContext, ValueTransformRegistry } from '../types.js';
 
 export interface ConvertToShapeInput {
   readonly conversion: ConversionDefinition;
@@ -66,54 +66,6 @@ function resolveShape(
   return shapes.find((shape) => shape.id === id);
 }
 
-function outputPathForTarget(slot: TargetSlot): string {
-  const path = slot.path?.trim();
-  if (path) {
-    return path;
-  }
-  // Prefer last id segment when hosts use dotted ids (`slot.display.timeText`).
-  const id = slot.id.trim();
-  if (id.includes('.')) {
-    const parts = id.split('.').filter(Boolean);
-    // Drop a leading registry prefix like `slot` when present with 3+ segments.
-    if (parts.length >= 3 && (parts[0] === 'slot' || parts[0] === 'tgt')) {
-      return parts.slice(1).join('.');
-    }
-    if (parts.length >= 2) {
-      return parts.slice(1).join('.');
-    }
-  }
-  return slot.label.trim() || id;
-}
-
-function readFieldValue(field: SourceField, inputs: Readonly<Record<string, unknown>>): unknown {
-  const shapeId = field.shapeId?.trim();
-  const bag =
-    shapeId && Object.prototype.hasOwnProperty.call(inputs, shapeId)
-      ? inputs[shapeId]
-      : // Single-bag fallback: use the only input, or the whole inputs record.
-        Object.keys(inputs).length === 1
-        ? inputs[Object.keys(inputs)[0]!]
-        : inputs;
-
-  const path = field.path?.trim();
-  if (path) {
-    // When bags are per-shape, paths are relative to that bag.
-    // When falling back to the whole inputs record, absolute paths still work.
-    if (shapeId && Object.prototype.hasOwnProperty.call(inputs, shapeId)) {
-      return readObjectPath(bag, path);
-    }
-    const fromBag = readObjectPath(bag, path);
-    if (fromBag !== undefined) {
-      return fromBag;
-    }
-    // Absolute path into combined inputs (e.g. `order.totalCents`).
-    return readObjectPath(inputs, path);
-  }
-
-  return field.sampleValue;
-}
-
 /**
  * Apply a conversion to named input bags and build target-shaped JSON.
  *
@@ -170,7 +122,7 @@ export async function convertToShape(input: ConvertToShapeInput): Promise<Conver
       continue;
     }
 
-    const sourceValue = readFieldValue(sourceField, input.inputs);
+    const sourceValue = readSourceFieldValue(sourceField, input.inputs);
     const value =
       edge.itemEdges && edge.itemEdges.length > 0
         ? await convertArrayWithItemEdges({
@@ -191,7 +143,7 @@ export async function convertToShape(input: ConvertToShapeInput): Promise<Conver
                 : undefined,
           });
 
-    const path = outputPathForTarget(targetSlot);
+    const path = resolveTargetSlotOutputPath(targetSlot);
     output = writeObjectPath(output, path, value);
     slots.push({
       edgeId: edge.id,