@workbench-kit/field-remap 0.0.2-prototype.0.2.25 → 0.0.2-prototype.0.2.27

package/package.json

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

package/src/domain/document/mappingEdge.ts

@@ -22,23 +22,9 @@ function sanitizeTransformIds(ids: readonly string[] | undefined): string[] {
   return cleaned.slice(0, MAX_TRANSFORM_CHAIN);
 }
 
-/**
- * Resolve the effective transform chain for an edge.
- * Prefers `transformIds`; falls back to legacy `transformId`.
- * Identity / empty / null → `[]` (pass-through).
- */
+/** Resolve the effective transform chain for an edge. Identity / empty → pass-through. */
 export function edgeTransformIds(edge: MappingEdge): string[] {
-  const fromList = sanitizeTransformIds(edge.transformIds);
-  if (fromList.length > 0) {
-    return fromList;
-  }
-
-  const legacy = edge.transformId ? canonicalizeTransformId(edge.transformId) : '';
-  if (legacy && legacy !== IDENTITY_TRANSFORM_ID) {
-    return [legacy];
-  }
-
-  return [];
+  return sanitizeTransformIds(edge.transformIds);
 }
 
 /** Per-item transform chain applied after optional `itemSourcePath` projection. */
@@ -47,9 +33,6 @@ export function edgeItemTransformIds(edge: MappingEdge): string[] {
 }
 
 /**
- * Normalize an edge to the `transformIds` model while keeping `transformId`
- * as the first chain step (or `null`) for older hosts.
- *
  * Option bags:
  * - Prefer `transformOptionSteps` when present (aligned to chain length).
  * - Keep `transformOptions` as the first non-empty step (or the legacy shared bag).
@@ -85,7 +68,6 @@ export function normalizeMappingEdge(edge: MappingEdge): MappingEdge {
     sourceFieldId: edge.sourceFieldId,
     targetSlotId: edge.targetSlotId,
     transformIds: ids.length > 0 ? ids : undefined,
-    transformId: ids[0] ?? null,
     ...(transformOptionSteps ? { transformOptionSteps } : {}),
     ...(transformOptions ? { transformOptions } : {}),
     ...(itemSourcePath && !itemEdges ? { itemSourcePath } : {}),
@@ -106,8 +88,6 @@ export function createMappingEdge(input: {
   readonly sourceFieldId: string;
   readonly targetSlotId: string;
   readonly transformIds?: readonly string[];
-  /** Legacy single id; ignored when `transformIds` is provided. */
-  readonly transformId?: string | null;
   readonly transformOptionSteps?: readonly (Readonly<Record<string, unknown>> | undefined)[];
   readonly transformOptions?: Readonly<Record<string, unknown>>;
   /** Optional per-item projection path for array mappings. */
@@ -124,7 +104,6 @@ export function createMappingEdge(input: {
     sourceFieldId: input.sourceFieldId,
     targetSlotId: input.targetSlotId,
     transformIds: input.transformIds,
-    transformId: input.transformId,
     transformOptionSteps: input.transformOptionSteps,
     transformOptions: input.transformOptions,
     itemSourcePath: input.itemSourcePath,

package/src/domain/mapping/objectPathSafety.ts

@@ -103,21 +103,6 @@ export function parseObjectPath(path: string): ObjectPathSegment[] {
   });
 }
 
-/**
- * Parse dotted path segments as plain property names (no `[index]` / `[*]`).
- * Rejects unsafe segments when any parts exist.
- */
-export function requireObjectPathParts(path: string): string[] {
-  const segments = parseObjectPath(path);
-  if (segments.some((segment) => segment.kind !== 'property')) {
-    throw new InvalidObjectPathError(
-      path,
-      'index/wildcard segments are not allowed in this context',
-    );
-  }
-  return segments.map((segment) => segment.name);
-}
-
 export function isSafeObjectPath(path: string): boolean {
   const trimmed = path.trim();
   if (!SAFE_PATH_RE.test(trimmed)) {

package/src/domain/mapping/resolveMappedValue.ts

@@ -28,7 +28,7 @@ export function findSourceField(
  * Apply order:
  * 1. Optional `itemSourcePath` projection (array of objects → projected array)
  * 2. Optional `itemTransformIds` per element (when the value is still an array)
- * 3. `transformIds` / legacy `transformId` on the whole value (including array reduces)
+ * 3. `transformIds` on the whole value (including array reduces)
  *
  * Per-step options (`transformOptionSteps` / `itemTransformOptionSteps`) win when
  * present; otherwise shared `transformOptions` / `itemTransformOptions` apply to all steps.

package/src/domain/types.ts

@@ -81,17 +81,10 @@ export interface MappingEdge {
   readonly targetSlotId: string;
   /**
    * Ordered transform chain (max 3). Empty / omitted means identity.
-   * Prefer this over `transformId` for new writers.
    * For arrays: use reduce builtins (`array:join`, `array:first`, …) here after
    * optional item projection / item transforms.
    */
   readonly transformIds?: readonly string[];
-  /**
-   * Legacy single transform. Prefer `transformIds`.
-   * Readers should use `edgeTransformIds` / `normalizeMappingEdge`.
-   * `null` / omitted (with no `transformIds`) means identity (pass-through).
-   */
-  readonly transformId?: string | null;
   /**
    * Per-step options aligned with `transformIds` (index N applies to step N).
    * Prefer this when steps need different bags (e.g. `showSeconds` then `maxLength`).