ggaction 0.0.15 → 0.0.16

package/src/actions/encodings/strokeDash.js

@@ -20,7 +20,7 @@ const STROKE_DASH_ENCODING_OPTIONS = Object.freeze([
   "field", "value", "target", "fieldType", "scale"
 ]);
 
-const clearStrokeDashEncoding = action(
+const clearStrokeDashEncoding = /* @__PURE__ */ action(
   {
     op: "clearStrokeDashEncoding",
     description: "Remove the current semantic stroke-dash assignment."
@@ -58,7 +58,7 @@ function reconcileLegendAfterDashRemoval(program, target) {
     .rematerializeLegend();
 }
 
-const encodeStrokeDash = action(
+const encodeStrokeDash = /* @__PURE__ */ action(
   {
     op: "encodeStrokeDash",
     description: "Encode a nominal field as line stroke dash."

package/src/actions/encodings/text.js

@@ -25,7 +25,7 @@ function validateTextField(rows, field, format) {
   if (!found) throw new Error(`Unknown text field "${field}".`);
 }
 
-export const encodeText = action(
+export const encodeText = /* @__PURE__ */ action(
   {
     op: "encodeText",
     description: "Assign field, constant, or source semantic text content."

package/src/actions/errorBands/create.js

@@ -1,9 +1,7 @@
-import { action } from "../../core/action.js";
+import { editGraphicProperties } from "../primitives/graphicProperties.js";
+import { action, closedAction } from "../../core/action.js";
 import { isPlainObject } from "../../core/immutable.js";
-import {
-  validateKeys,
-  validateNonEmptyString,
-} from "../../core/validation.js";
+import { validateNonEmptyString } from "../../core/validation.js";
 import { DEFAULT_COLORS } from "../../theme/defaults.js";
 import { validateCurveInterpolation } from "../../grammar/curveCommands.js";
 import {
@@ -82,7 +80,7 @@ function positionOptions({ target, field, fieldType, coordinate, scale, temporal
     ...(temporalUnit === undefined ? {} : { temporalUnit }) };
 }
 
-export const createErrorBandBoundary = action(
+export const createErrorBandBoundary = /* @__PURE__ */ action(
   {
     op: "createErrorBandBoundary",
     description: "Create one lower or upper error-band boundary line."
@@ -138,14 +136,11 @@ export const createErrorBandBoundary = action(
     if (groupBy !== undefined) {
       next = next.encodeGroup({ target: id, field: groupBy });
     }
-    return next
-      .editGraphics({ target: id, property: "stroke", value: stroke })
-      .editGraphics({
-        target: id,
-        property: "strokeDash",
-        value: next.graphicSpec.objects[id].items.map(() => strokeDash)
-      })
-      .editGraphics({ target: id, property: "opacity", value: opacity });
+    return editGraphicProperties(next, id, {
+      stroke: stroke,
+      strokeDash: next.graphicSpec.objects[id].items.map(() => strokeDash),
+      opacity: opacity
+    });
   }
 );
 
@@ -171,13 +166,12 @@ function rangeArgs(resolved) {
   };
 }
 
-export const createErrorBand = action(
+export const createErrorBand = /* @__PURE__ */ closedAction(
   {
     op: "createErrorBand",
     description: "Create a statistical or explicit interval band."
-  },
+  }, OPTIONS,
   function (args = {}) {
-    validateKeys(args, OPTIONS, "createErrorBand");
     const strokeDetails = requestedStrokeDetails(args, "createErrorBand");
     const resolved = resolveErrorBand(this, args);
     const curve = validateCurveInterpolation(args.curve ?? "linear");

package/src/actions/errorBands/edit.js

@@ -1,13 +1,8 @@
-import { action } from "../../core/action.js";
+import { currentIntervalRoleArgs, resolveIntervalRoleScales, applyIntervalStatistics } from "../intervals/revision.js";
+import { closedAction } from "../../core/action.js";
 import { isPlainObject } from "../../core/immutable.js";
 import { validateUserId } from "../../core/identifiers.js";
-import {
-  validateOptionObject,
-  validateKeys,
-  validateNonEmptyString,
-  validateNonNegativeFinite,
-  validateUnitInterval
-} from "../../core/validation.js";
+import { validateKeys, validateNonEmptyString, validateNonNegativeFinite, validateUnitInterval } from "../../core/validation.js";
 import { validateCurveInterpolation } from "../../grammar/curveCommands.js";
 import { normalizeStrokeDashPattern } from "../../grammar/scales/index.js";
 import {
@@ -15,12 +10,10 @@ import {
   STROKE_STYLE_PROPERTIES
 } from "../../grammar/strokeStyle.js";
 import { findLayer } from "../../selectors/layers.js";
-import { findSemanticScale } from "../../selectors/scales.js";
 import { removeOwnedMark } from "../marks/remove.js";
 import { DEFAULT_COLORS } from "../../theme/defaults.js";
 import { resolveIntervalComposite } from "../intervals/resolve.js";
 import {
-  resolveDistributionScalePlan,
   setCartesianPosition,
   updateDistributionPositions
 } from "../distributions/revision.js";
@@ -68,7 +61,6 @@ const EDIT_OPTIONS = Object.freeze([
   "target", "data", "x", "y", "groupBy", "fill", "opacity", "curve",
   "statistics", "boundaries", ...STROKE_STYLE_PROPERTIES
 ]);
-const STATISTICS_OPTIONS = Object.freeze(["center", "extent", "method", "level"]);
 const EDIT_POLICY = Object.freeze({
   operation: "editErrorBand",
   resourceLabel: "error-band",
@@ -116,48 +108,13 @@ function resolveOwner(program, requested) {
   });
 }
 
-function currentRoleArgs(program, owner, current) {
-  const transform = findIntervalTransform(program, current.data);
-  const intervalChannel = current.orientation === "vertical" ? "y" : "x";
-  const positionChannel = intervalChannel === "x" ? "y" : "x";
-  const position = {
-    field: current.position.field,
-    fieldType: current.position.fieldType,
-    ...(current.position.temporalUnit === undefined
-      ? {}
-      : { temporalUnit: current.position.temporalUnit }),
-    scale: { id: current.positionScale }
-  };
-  const interval = transform === undefined
-    ? {
-        center: current.centerField ?? owner.encoding[intervalChannel].title,
-        lower: current.lowerField,
-        upper: current.upperField,
-        scale: { id: current.intervalScale }
-      }
-    : {
-        field: transform.field,
-        center: transform.center,
-        extent: transform.extent,
-        ...(transform.method === undefined ? {} : { method: transform.method }),
-        ...(transform.level === undefined ? {} : { level: transform.level }),
-        scale: { id: current.intervalScale }
-      };
-  return {
-    source: current.source ?? transform?.source ?? current.data,
-    x: positionChannel === "x" ? position : interval,
-    y: positionChannel === "y" ? position : interval,
-    groupBy: current.groupBy
-  };
-}
-
 function channelArgs(requested, fallback) {
   if (requested === undefined) return fallback;
   return requested;
 }
 
 function resolveRoleCandidate(program, owner, current, args) {
-  const previous = currentRoleArgs(program, owner, current);
+  const previous = currentIntervalRoleArgs(program, owner, current, false);
   const full = {
     data: Object.hasOwn(args, "data") ? args.data : previous.source,
     x: channelArgs(args.x, previous.x),
@@ -179,75 +136,11 @@ function resolveRoleCandidate(program, owner, current, args) {
       "editErrorBand groupBy must differ from the independent position field."
     );
   }
-  if (Object.hasOwn(args, "statistics")) {
-    if (!isPlainObject(args.statistics)) {
-      throw new TypeError("editErrorBand statistics must be a plain object.");
-    }
-    validateKeys(args.statistics, STATISTICS_OPTIONS, "editErrorBand statistics");
-    if (!STATISTICS_OPTIONS.some(key => Object.hasOwn(args.statistics, key))) {
-      throw new Error(
-        "editErrorBand statistics requires center, extent, method, or level."
-      );
-    }
-    full[resolved.interval.channel] = {
-      ...full[resolved.interval.channel],
-      ...args.statistics
-    };
-    resolved = resolveIntervalComposite(program, full, policy);
-  }
-  const positionRole = {
-    field: resolved.position.field,
-    fieldType: resolved.position.fieldType,
-    scale: resolved.position.scale
-  };
-  const intervalTitle = resolved.interval.mode === "statistical"
-    ? resolved.interval.field
-    : resolved.interval.title;
-  const intervalRole = {
-    field: intervalTitle,
-    fieldType: "quantitative",
-    scale: resolved.interval.scale
-  };
-  const x = resolved.position.channel === "x" ? positionRole : intervalRole;
-  const y = resolved.position.channel === "y" ? positionRole : intervalRole;
-  const plan = (channel, role) => {
-    const fallback = role === intervalRole
-      ? current.intervalScale
-      : current.positionScale;
-    const stored = findSemanticScale(program, fallback);
-    const compatible = role.fieldType === "temporal"
-      ? stored?.type === "time"
-      : ["linear", "log", "pow", "sqrt", "symlog"].includes(stored?.type);
-    const requested = role.scale.type !== undefined || compatible
-      ? role.scale
-      : { ...role.scale, type: role.fieldType === "temporal" ? "time" : "linear" };
-    return resolveDistributionScalePlan(program, {
-      channel,
-      fieldType: role.fieldType,
-      requested,
-      fallback,
-      defaults: role === intervalRole
-        ? EDIT_POLICY.intervalScaleDefaults
-        : EDIT_POLICY.scaleDefaults(role.fieldType)
-    });
-  };
-  const xScale = plan("x", x);
-  const yScale = plan("y", y);
+  resolved = applyIntervalStatistics(program, full, policy, args, resolved);
+  const roles = resolveIntervalRoleScales(program, resolved, current, EDIT_POLICY, owner, previous);
   return {
     ...resolved,
-    x: { ...x, scale: xScale.id },
-    y: { ...y, scale: yScale.id },
-    xScale,
-    yScale,
-    category: resolved.position.field,
-    categoryType: resolved.position.fieldType,
-    measure: intervalTitle,
-    previous: {
-      x: { field: previous.x.center ?? previous.x.field,
-        fieldType: owner.encoding.x.fieldType, scale: owner.encoding.x.scale },
-      y: { field: previous.y.center ?? previous.y.field,
-        fieldType: owner.encoding.y.fieldType, scale: owner.encoding.y.scale }
-    }
+    ...roles
   };
 }
 
@@ -389,17 +282,12 @@ function removeBoundary(program, id) {
   return removeOwnedMark(program, id);
 }
 
-export const rematerializeErrorBandBoundary = action(
+export const rematerializeErrorBandBoundary = /* @__PURE__ */ closedAction(
   {
     op: "rematerializeErrorBandBoundary",
     description: "Rematerialize one owned error-band boundary."
-  },
+  }, REMATERIALIZE_OPTIONS,
   function (args = {}) {
-    validateOptionObject(
-      args,
-      REMATERIALIZE_OPTIONS,
-      "rematerializeErrorBandBoundary"
-    );
     const id = validateUserId(args.id, "Error-band boundary id");
     currentBoundaryAppearance(this, id);
     const graphic = this.graphicSpec.objects[id];
@@ -422,13 +310,12 @@ export const rematerializeErrorBandBoundary = action(
   }
 );
 
-export const editErrorBand = action(
+export const editErrorBand = /* @__PURE__ */ closedAction(
   {
     op: "editErrorBand",
     description: "Revise one error band's roles, statistics, body, and boundaries."
-  },
+  }, EDIT_OPTIONS,
   function (args = {}) {
-    validateOptionObject(args, EDIT_OPTIONS, "editErrorBand");
     if (!["data", "x", "y", "groupBy", "fill", "opacity", "curve",
       "statistics", "boundaries", ...STROKE_STYLE_PROPERTIES]
       .some(key => Object.hasOwn(args, key))) {
@@ -585,17 +472,12 @@ export const editErrorBand = action(
   }
 );
 
-export const editErrorBandBoundary = action(
+export const editErrorBandBoundary = /* @__PURE__ */ closedAction(
   {
     op: "editErrorBandBoundary",
     description: "Edit one or both owned error-band boundaries."
-  },
+  }, BOUNDARY_EDIT_OPTIONS,
   function (args = {}) {
-    validateOptionObject(
-      args,
-      BOUNDARY_EDIT_OPTIONS,
-      "editErrorBandBoundary"
-    );
     if (!ERROR_BAND_BOUNDARY_OPTIONS.some(key => Object.hasOwn(args, key))) {
       throw new Error("editErrorBandBoundary requires an appearance change.");
     }

package/src/actions/errorBars/create.js

@@ -1,8 +1,5 @@
-import { action } from "../../core/action.js";
-import {
-  validateKeys,
-  validateOptionObject
-} from "../../core/validation.js";
+import { closedAction } from "../../core/action.js";
+import { validateOptionObject } from "../../core/validation.js";
 import { findLayer } from "../../selectors/layers.js";
 import { findDataset } from "../../selectors/datasets.js";
 import { findSemanticScale } from "../../selectors/scales.js";
@@ -144,20 +141,19 @@ function resolveAppearance(args) {
   });
 }
 
-export const createErrorBarCap = action(
+export const createErrorBarCap = /* @__PURE__ */ closedAction(
   {
     op: "createErrorBarCap",
     description: "Create one fixed-pixel error-bar cap."
-  },
-  function (args = {}) {
-    validateKeys(args, [
+  }, [
       "id", "data", "orientation", "positionField", "positionFieldType",
       "intervalField", "coordinate", "positionScale", "intervalScale", "positionTemporalUnit",
       "offsetChannel", "offsetField", "offsetFieldType", "offsetScale",
       "offsetPaddingInner", "offsetPaddingOuter",
       "capSize", "stroke", "strokeWidth", "strokeDash", "opacity",
       ...STROKE_STYLE_PROPERTIES
-    ], "createErrorBarCap");
+    ],
+  function (args = {}) {
     if (!["vertical", "horizontal"].includes(args.orientation)) {
       throw new Error(`Unsupported error-bar orientation "${args.orientation}".`);
     }
@@ -256,13 +252,12 @@ function intervalArgs(resolved, field) {
   };
 }
 
-export const createErrorBar = action(
+export const createErrorBar = /* @__PURE__ */ closedAction(
   {
     op: "createErrorBar",
     description: "Create a statistical or explicit vertical or horizontal interval."
-  },
+  }, OPTIONS,
   function (args = {}) {
-    validateKeys(args, OPTIONS, "createErrorBar");
     const resolved = resolveErrorBar(this, args);
     const appearance = resolveAppearance(args);
     let next = createResolvedIntervalData(this, resolved);

package/src/actions/errorBars/edit.js

@@ -1,5 +1,5 @@
-import { action } from "../../core/action.js";
-import { isPlainObject } from "../../core/immutable.js";
+import { currentIntervalRoleArgs, resolveIntervalRoleScales, applyIntervalStatistics } from "../intervals/revision.js";
+import { closedAction } from "../../core/action.js";
 import { validateUserId } from "../../core/identifiers.js";
 import {
   validateKeys,
@@ -14,10 +14,8 @@ import {
   validateRuleStrokeWidth
 } from "../../grammar/ruleAppearance.js";
 import { findLayer } from "../../selectors/layers.js";
-import { findSemanticScale } from "../../selectors/scales.js";
 import { resolveIntervalComposite } from "../intervals/resolve.js";
 import {
-  resolveDistributionScalePlan,
   setCartesianPosition,
   updateDistributionPositions
 } from "../distributions/revision.js";
@@ -46,7 +44,6 @@ export const ERROR_BAR_APPEARANCE_OPTIONS = Object.freeze([
   "caps", "capSize", "stroke", "strokeWidth", "strokeDash", "opacity",
   ...STROKE_STYLE_PROPERTIES
 ]);
-const STATISTICS_OPTIONS = Object.freeze(["center", "extent", "method", "level"]);
 const EDIT_POLICY = Object.freeze({
   operation: "editErrorBar",
   resourceLabel: "error-bar",
@@ -115,45 +112,8 @@ function resolveOwner(program, requested) {
   });
 }
 
-function currentRoleArgs(program, owner, current) {
-  const transform = findIntervalTransform(program, current.data);
-  const intervalChannel = current.orientation === "vertical" ? "y" : "x";
-  const positionChannel = intervalChannel === "x" ? "y" : "x";
-  const position = {
-    field: current.positionField,
-    fieldType: current.positionFieldType,
-    ...(current.positionTemporalUnit === undefined
-      ? {}
-      : { temporalUnit: current.positionTemporalUnit }),
-    scale: { id: current.positionScale }
-  };
-  const interval = transform === undefined
-    ? {
-        center: current.centerField ?? owner.encoding[intervalChannel].title,
-        lower: current.lowerField,
-        upper: current.upperField,
-        scale: { id: current.intervalScale }
-      }
-    : {
-        field: transform.field,
-        center: transform.center,
-        extent: transform.extent,
-        ...(transform.method === undefined ? {} : { method: transform.method }),
-        ...(transform.level === undefined ? {} : { level: transform.level }),
-        scale: { id: current.intervalScale }
-      };
-  return {
-    source: current.source ?? transform?.source ?? current.data,
-    x: positionChannel === "x" ? position : interval,
-    y: positionChannel === "y" ? position : interval,
-    groupBy: current.groupField ?? transform?.groupBy?.find(
-      field => field !== current.positionField
-    )
-  };
-}
-
 function resolveRoleCandidate(program, owner, current, args) {
-  const previous = currentRoleArgs(program, owner, current);
+  const previous = currentIntervalRoleArgs(program, owner, current, true);
   const full = {
     data: Object.hasOwn(args, "data") ? args.data : previous.source,
     x: Object.hasOwn(args, "x") ? args.x : previous.x,
@@ -170,65 +130,8 @@ function resolveRoleCandidate(program, owner, current, args) {
     generatedFields: findIntervalTransform(program, current.data)?.as
   };
   let resolved = resolveIntervalComposite(program, full, policy);
-  if (Object.hasOwn(args, "statistics")) {
-    if (!isPlainObject(args.statistics)) {
-      throw new TypeError("editErrorBar statistics must be a plain object.");
-    }
-    validateKeys(args.statistics, STATISTICS_OPTIONS, "editErrorBar statistics");
-    if (!STATISTICS_OPTIONS.some(key => Object.hasOwn(args.statistics, key))) {
-      throw new Error(
-        "editErrorBar statistics requires center, extent, method, or level."
-      );
-    }
-    full[resolved.interval.channel] = {
-      ...full[resolved.interval.channel],
-      ...args.statistics
-    };
-    resolved = resolveIntervalComposite(program, full, policy);
-  }
-  const positionRole = {
-    field: resolved.position.field,
-    fieldType: resolved.position.fieldType,
-    scale: resolved.position.scale
-  };
-  const intervalTitle = resolved.interval.mode === "statistical"
-    ? resolved.interval.field
-    : resolved.interval.title;
-  const intervalRole = {
-    field: intervalTitle,
-    fieldType: "quantitative",
-    scale: resolved.interval.scale
-  };
-  const x = resolved.position.channel === "x" ? positionRole : intervalRole;
-  const y = resolved.position.channel === "y" ? positionRole : intervalRole;
-  const plan = (channel, role) => {
-    const fallback = role === intervalRole
-      ? current.intervalScale
-      : current.positionScale;
-    const stored = findSemanticScale(program, fallback);
-    const compatible = role.fieldType === "temporal"
-      ? stored?.type === "time"
-      : ["nominal", "ordinal"].includes(role.fieldType)
-        ? ["band", "point"].includes(stored?.type)
-        : ["linear", "log", "pow", "sqrt", "symlog"].includes(stored?.type);
-    const defaultType = role.fieldType === "temporal"
-      ? "time"
-      : ["nominal", "ordinal"].includes(role.fieldType) ? "band" : "linear";
-    const requested = role.scale.type !== undefined || compatible
-      ? role.scale
-      : { ...role.scale, type: defaultType };
-    return resolveDistributionScalePlan(program, {
-      channel,
-      fieldType: role.fieldType,
-      requested,
-      fallback,
-      defaults: role === intervalRole
-        ? EDIT_POLICY.intervalScaleDefaults
-        : EDIT_POLICY.scaleDefaults(role.fieldType)
-    });
-  };
-  const xScale = plan("x", x);
-  const yScale = plan("y", y);
+  resolved = applyIntervalStatistics(program, full, policy, args, resolved);
+  const roles = resolveIntervalRoleScales(program, resolved, current, EDIT_POLICY, owner, previous);
   const expectedOffset = `${resolved.position.channel}Offset`;
   const incompatibleOffset = expectedOffset === "xOffset" ? "yOffset" : "xOffset";
   if (Object.hasOwn(args, incompatibleOffset)) {
@@ -270,19 +173,7 @@ function resolveRoleCandidate(program, owner, current, args) {
   return {
     ...resolved,
     offset: normalizedOffset,
-    x: { ...x, scale: xScale.id },
-    y: { ...y, scale: yScale.id },
-    xScale,
-    yScale,
-    category: resolved.position.field,
-    categoryType: resolved.position.fieldType,
-    measure: intervalTitle,
-    previous: {
-      x: { field: previous.x.center ?? previous.x.field,
-        fieldType: owner.encoding.x.fieldType, scale: owner.encoding.x.scale },
-      y: { field: previous.y.center ?? previous.y.field,
-        fieldType: owner.encoding.y.fieldType, scale: owner.encoding.y.scale }
-    }
+    ...roles
   };
 }
 
@@ -460,13 +351,12 @@ function updateErrorBarRoles(program, owner, current, candidate, dataId, capIds)
   return next.rematerializeErrorBar({ id: owner.id });
 }
 
-export const rematerializeErrorBar = action(
+export const rematerializeErrorBar = /* @__PURE__ */ closedAction(
   {
     op: "rematerializeErrorBar",
     description: "Reconcile one error bar and its owned caps."
-  },
+  }, REMATERIALIZE_OPTIONS,
   function (args = {}) {
-    validateKeys(args, REMATERIALIZE_OPTIONS, "rematerializeErrorBar");
     const id = validateUserId(args.id, "Error-bar id");
     const layer = findLayer(this, id);
     const config = this.markConfigs[id]?.errorBar;
@@ -494,13 +384,12 @@ export const rematerializeErrorBar = action(
   }
 );
 
-export const editErrorBar = action(
+export const editErrorBar = /* @__PURE__ */ closedAction(
   {
     op: "editErrorBar",
     description: "Revise one error bar's roles, statistics, and owned caps."
-  },
+  }, ERROR_BAR_EDIT_OPTIONS,
   function (args = {}) {
-    validateKeys(args, ERROR_BAR_EDIT_OPTIONS, "editErrorBar");
     const editable = ERROR_BAR_EDIT_OPTIONS.filter(option => option !== "target");
     if (!editable.some(option => Object.hasOwn(args, option))) {
       throw new Error("editErrorBar requires at least one change.");

package/src/actions/facets/actions.js

@@ -1,4 +1,5 @@
-import { action } from "../../core/action.js";
+import { action, closedAction } from "../../core/action.js";
+import { registerFacetDataRevision, reviseUnitData } from "../data/revise.js";
 import { freezeOwned, isPlainObject } from "../../core/immutable.js";
 import { validateUserId } from "../../core/identifiers.js";
 import {
@@ -183,13 +184,12 @@ function rederiveFacet(program, { scales, guides }) {
   }, compositionSpec.children);
 }
 
-export const facet = action(
+export const facet = /* @__PURE__ */ closedAction(
   {
     op: "facet",
     description: "Repeat one direct-source chart by field value."
-  },
+  }, FACET_OPTIONS,
   function (args = {}) {
-    validateOptionObject(args, FACET_OPTIONS, "facet");
     const guides = normalizeGuides(args.guides);
     const definition = resolveFacetDefinition(this.semanticSpec, args);
     if (definition.family !== "cartesian" && guides.axes === "outer") {
@@ -428,13 +428,12 @@ function deriveRepeatChildren(base, definition, scales, closeInheritedAction) {
   );
 }
 
-export const repeatCharts = action(
+export const repeatCharts = /* @__PURE__ */ closedAction(
   {
     op: "repeatCharts",
     description: "Repeat one direct chart across an ordered field-role list."
-  },
+  }, REPEAT_OPTIONS,
   function (args = {}) {
-    validateOptionObject(args, REPEAT_OPTIONS, "repeatCharts");
     const guides = normalizeGuides(args.guides);
     if (guides.axes === "outer") {
       throw new Error("repeatCharts does not promote axes across different repeated fields.");
@@ -495,13 +494,12 @@ export const repeatCharts = action(
   }
 );
 
-export const facetGrid = action(
+export const facetGrid = /* @__PURE__ */ closedAction(
   {
     op: "facetGrid",
     description: "Repeat one direct-source Cartesian chart across a row and column field grid."
-  },
+  }, FACET_GRID_OPTIONS,
   function (args = {}) {
-    validateOptionObject(args, FACET_GRID_OPTIONS, "facetGrid");
     const guides = normalizeGuides(args.guides);
     const definition = resolveFacetGridDefinition(this.semanticSpec, args);
     if (definition.family !== "cartesian" && guides.axes === "outer") {
@@ -565,7 +563,7 @@ export const facetGrid = action(
   }
 );
 
-export const editFacetHeaders = action(
+export const editFacetHeaders = /* @__PURE__ */ action(
   {
     op: "editFacetHeaders",
     description: "Edit parent-owned facet header appearance.",
@@ -656,7 +654,7 @@ export const editFacetHeaders = action(
   }
 );
 
-export const editFacetScales = action(
+export const editFacetScales = /* @__PURE__ */ action(
   {
     op: "editFacetScales",
     description: "Edit facet scale-resolution policies and rederive every cell.",
@@ -691,7 +689,7 @@ export const editFacetScales = action(
   }
 );
 
-export const editFacetGuides = action(
+export const editFacetGuides = /* @__PURE__ */ action(
   {
     op: "editFacetGuides",
     description: "Edit facet guide ownership and rederive every cell.",
@@ -764,14 +762,13 @@ function adoptUnitState(program, actionOwner) {
   });
 }
 
-export const editFacetSource = action(
+export const editFacetSource = /* @__PURE__ */ closedAction(
   {
     op: "editFacetSource",
     description: "Reapply one facet, grid, or repeat recipe to a revised complete unit program.",
     scope: "composition"
-  },
+  }, SOURCE_EDIT_OPTIONS,
   function (args = {}) {
-    validateOptionObject(args, SOURCE_EDIT_OPTIONS, "editFacetSource");
     requireFacetProgram(this, "editFacetSource");
     const current = this.compositionSpec;
     const facetConfig = this.materializationConfigs.facets?.[current.id];
@@ -838,6 +835,23 @@ export const editFacetSource = action(
 );
 
 export function registerFacetActions(ProgramClass) {
+  registerFacetDataRevision((program, args) => {
+    const { program: unit, plan } = reviseUnitData(facetUnitTemplate(program), args);
+    const current = program.compositionSpec;
+    // Adopt an already-materialized unit revision into the retained parent.
+    const revised = new program.constructor({
+      ...unit,
+      graphicSpec: program.graphicSpec,
+      children: program.children,
+      materializationConfigs: { ...unit.materializationConfigs,
+        facets: program.materializationConfigs.facets,
+        ...(program.titleConfig === undefined ? {} : { title: program.titleConfig }) },
+      actionSequence: unit._actionSequence,
+      compositionSpec: { ...current, facet: { ...current.facet,
+        data: plan.replacements.get(current.facet.data) ?? current.facet.data } }
+    });
+    return rederiveFacet(revised, current.facet);
+  });
   ProgramClass.prototype.replayDerivedData = replayDerivedData;
   ProgramClass.prototype.composeFacetGuides = composeFacetGuides;
   ProgramClass.prototype.facet = facet;