@ai-matrx/content-ir-react 0.17.1 → 0.18.0

package/CHANGELOG.md

@@ -1,5 +1,27 @@
 # Changelog
 
+## 0.18.0 — 2026-10-02
+
+The G6A review of the action cards (nightly clone). Guard:
+`__tests__/directive-card-g6a.test.tsx` (3 red on 0.17.1).
+
+- **An applied update says what it changed, from the record of the change.** After
+  an Update applied, a reload read the record live for the old value — which by
+  then IS the new value — and the row said "Title → X (no change)". The rows now
+  compare against the live record only while the block is open; once applied they
+  draw the old value from `before` — new on `DirectiveApplyResult` and
+  `DirectiveApplyState`, each item's replaced values by item index, which aidream
+  ledgers at apply time (1122747fb5) — and without it say only "Title → new".
+  Until the ledger answers, a row compares against nothing. New exports:
+  `DirectiveApplyPhase`, `readsApplyLedger`; `ApplyDirectiveButton` takes `onPhase`.
+- **An update that sets no field says so before the click.** "Nothing to change"
+  beside a muted Apply (`data-apply-tone="muted"`, `data-apply-nothing`). Apply
+  still works — validation offers, never blocks.
+
+Consumer action: a host's `confirm` passes `before` from the server's per-item
+receipts, and its `applyState` passes `before` from each item of
+`/directives/apply_state`.
+
 ## 0.17.1
 
 Automatic changed-only republish (docs/metadata drift since the last tag — see

package/dist/{directive-host-BOML4BQ7.d.cts → directive-host-Bw2_PrgL.d.cts}

@@ -15,6 +15,14 @@ interface DirectiveApplyResult {
      * (no dead ends). Rendered through `renderRecord`; absent → the sentence alone.
      */
     records?: DirectiveRecordRef[];
+    /**
+     * For an UPDATE: what each item's fields held just BEFORE the write, by item
+     * index (`null` for an item the server did not record). The card draws the
+     * applied change from this record — never from a live re-read, which after the
+     * apply answers the NEW value (G6A review, 2026-10-02: "(no change)" for a
+     * change that happened).
+     */
+    before?: ReadonlyArray<Record<string, unknown> | null>;
 }
 /**
  * What the server's action ledger says about ONE block, read on mount so a
@@ -43,6 +51,12 @@ interface DirectiveApplyState {
      * two records, and the card says so (reviewer, 2026-10-02).
      */
     copies?: number;
+    /**
+     * For an applied UPDATE: each item's replaced values, by item index, from the
+     * ledger row (`null` where the ledger has none — a row from before 2026-10-02).
+     * Absent → the card says only what each field was set to.
+     */
+    before?: ReadonlyArray<Record<string, unknown> | null>;
 }
 /** One record a directive names: the noun it is, and its id. */
 interface DirectiveRecordRef {

package/dist/{directive-host-BOML4BQ7.d.ts → directive-host-Bw2_PrgL.d.ts}

@@ -15,6 +15,14 @@ interface DirectiveApplyResult {
      * (no dead ends). Rendered through `renderRecord`; absent → the sentence alone.
      */
     records?: DirectiveRecordRef[];
+    /**
+     * For an UPDATE: what each item's fields held just BEFORE the write, by item
+     * index (`null` for an item the server did not record). The card draws the
+     * applied change from this record — never from a live re-read, which after the
+     * apply answers the NEW value (G6A review, 2026-10-02: "(no change)" for a
+     * change that happened).
+     */
+    before?: ReadonlyArray<Record<string, unknown> | null>;
 }
 /**
  * What the server's action ledger says about ONE block, read on mount so a
@@ -43,6 +51,12 @@ interface DirectiveApplyState {
      * two records, and the card says so (reviewer, 2026-10-02).
      */
     copies?: number;
+    /**
+     * For an applied UPDATE: each item's replaced values, by item index, from the
+     * ledger row (`null` where the ledger has none — a row from before 2026-10-02).
+     * Absent → the card says only what each field was set to.
+     */
+    before?: ReadonlyArray<Record<string, unknown> | null>;
 }
 /** One record a directive names: the noun it is, and its id. */
 interface DirectiveRecordRef {

package/dist/directives.cjs

@@ -39,6 +39,7 @@ __export(directives_exports, {
   itemChanges: () => itemChanges,
   itemRecordId: () => itemRecordId,
   provisionalDirectiveSlug: () => provisionalDirectiveSlug,
+  readsApplyLedger: () => readsApplyLedger,
   recordFallbackName: () => recordFallbackName,
   registerDirectiveRenderer: () => registerDirectiveRenderer,
   resetDirectiveRenderers: () => resetDirectiveRenderers,
@@ -290,17 +291,22 @@ function classIcon(directiveClass, noun) {
 var import_jsx_runtime3 = require("react/jsx-runtime");
 var FINISH_POLL_MS = 2e3;
 var FINISH_POLLS_MAX = 30;
+function readsApplyLedger(host, directive) {
+  return !!host.applyState && !!host.confirm && directive.parsed.executes && directive.items.length > 0;
+}
 function ApplyDirectiveButton({
   directive,
   label = "Apply",
   itemCount,
-  onApplied
+  onApplied,
+  onPhase,
+  nothingToChange = false
 }) {
   const host = useDirectiveHost();
   const scope = useDirectiveScope();
   const conversationId = scope.conversationId || void 0;
   const items = directive.items;
-  const readsLedger = !!host.applyState && !!host.confirm && directive.parsed.executes && items.length > 0;
+  const readsLedger = readsApplyLedger(host, directive);
   const initial = readsLedger ? { status: "checking" } : { status: "idle" };
   const [state, setState] = (0, import_react2.useState)(initial);
   const [elapsed, setElapsed] = (0, import_react2.useState)(0);
@@ -341,7 +347,8 @@ function ApplyDirectiveButton({
               failed: 0,
               message: answer.message?.trim() ? answer.message.trim() : null,
               records: answer.records ?? [],
-              copies: Math.max(1, answer.copies ?? 1)
+              copies: Math.max(1, answer.copies ?? 1),
+              before: answer.before ?? null
             } : prev
           );
         } else if (answer.state === "in_flight" && polls < FINISH_POLLS_MAX) {
@@ -366,6 +373,15 @@ function ApplyDirectiveButton({
       if (timer) clearTimeout(timer);
     };
   }, [shellKey, read, report]);
+  const appliedClean = state.status === "applied" && state.failed === 0 ? state : null;
+  const appliedBefore = appliedClean ? appliedClean.before : null;
+  const phaseKey = state.status === "checking" || state.status === "finishing" ? "pending" : appliedClean ? `applied:${JSON.stringify(appliedBefore)}` : "open";
+  (0, import_react2.useEffect)(() => {
+    if (!onPhase) return;
+    if (phaseKey === "pending") onPhase({ phase: "pending" });
+    else if (phaseKey === "open") onPhase({ phase: "open" });
+    else onPhase({ phase: "applied", before: appliedBefore });
+  }, [phaseKey, onPhase, appliedBefore]);
   (0, import_react2.useEffect)(() => {
     if (state.status !== "applying") return;
     startedAt.current = Date.now();
@@ -380,7 +396,7 @@ function ApplyDirectiveButton({
   if (!confirm) return null;
   async function handleApply(again = false) {
     if (!confirm) return;
-    const before = state;
+    const prior = state;
     const askedFor = identity;
     const stillHere = () => currentIdentity.current === askedFor;
     const priorCopies = state.status === "applied" ? state.copies : 0;
@@ -407,7 +423,7 @@ function ApplyDirectiveButton({
       if (!stillHere()) return;
       setAskingAgain(false);
       if (!ok) {
-        setState(again ? before : { status: "idle" });
+        setState(again ? prior : { status: "idle" });
         return;
       }
     }
@@ -426,7 +442,10 @@ function ApplyDirectiveButton({
         message: result.message?.trim() ? result.message.trim() : null,
         records: result.records ?? [],
         // "Run again" made one more; a first apply (or a replay of one) is one.
-        copies: again ? Math.max(1, priorCopies) + 1 : Math.max(1, priorCopies)
+        copies: again ? Math.max(1, priorCopies) + 1 : Math.max(1, priorCopies),
+        // "Run again" wrote the same values over themselves; what the FIRST
+        // apply replaced is still the change this block made.
+        before: again && prior.status === "applied" && prior.before ? prior.before : result.before ?? null
       });
       onApplied?.();
     } catch (error) {
@@ -528,15 +547,18 @@ function ApplyDirectiveButton({
     );
   }
   const destructive = directive.directiveClass === "delete";
+  const muted = nothingToChange && state.status === "idle";
   return /* @__PURE__ */ (0, import_jsx_runtime3.jsxs)("span", { className: "inline-flex items-center gap-2", children: [
+    muted ? /* @__PURE__ */ (0, import_jsx_runtime3.jsx)("span", { className: "text-xs text-muted-foreground", "data-apply-nothing": "", children: "Nothing to change" }) : null,
     /* @__PURE__ */ (0, import_jsx_runtime3.jsxs)(
       "button",
       {
         type: "button",
         onClick: () => void handleApply(),
         disabled: state.status === "applying" || state.status === "asking" || items.length === 0,
-        "data-apply-tone": destructive ? "destructive" : "default",
-        className: `inline-flex items-center gap-1 rounded-md border px-2 py-1 text-xs font-medium disabled:opacity-50 ${destructive ? "border-destructive/50 bg-background text-destructive hover:bg-destructive/10" : "border-border bg-background hover:bg-accent"}`,
+        "data-apply-tone": muted ? "muted" : destructive ? "destructive" : "default",
+        title: muted ? "It sets no fields, so applying it changes nothing." : void 0,
+        className: `inline-flex items-center gap-1 rounded-md border px-2 py-1 text-xs font-medium disabled:opacity-50 ${muted ? "border-border/60 bg-background text-muted-foreground hover:bg-accent" : destructive ? "border-destructive/50 bg-background text-destructive hover:bg-destructive/10" : "border-border bg-background hover:bg-accent"}`,
         children: [
           state.status === "applying" ? /* @__PURE__ */ (0, import_jsx_runtime3.jsx)(LoaderIcon, { className: "h-3.5 w-3.5 animate-spin" }) : /* @__PURE__ */ (0, import_jsx_runtime3.jsx)(PlayIcon, { className: "h-3.5 w-3.5" }),
           state.status === "applying" ? `Applying${itemCount ? ` ${itemCount} ${itemCount === 1 ? "item" : "items"}` : ""}\u2026${elapsed ? ` ${elapsed}s` : ""}` : label
@@ -646,12 +668,13 @@ function ItemRow({
   directive,
   item,
   index,
-  total
+  total,
+  phase
 }) {
   const host = useDirectiveHost();
   const recordId = TARGETING_CLASSES.has(directive.directiveClass) ? itemRecordId(item) : null;
   if (recordId) {
-    return /* @__PURE__ */ (0, import_jsx_runtime4.jsx)(TargetRow, { directive, item, recordId });
+    return /* @__PURE__ */ (0, import_jsx_runtime4.jsx)(TargetRow, { directive, item, recordId, index, phase });
   }
   const titleColumn = (0, import_content_ir3.nounTitleColumn)(directive.noun, host.nouns);
   const title = (0, import_content_ir3.itemTitle)(item, titleColumn, index, total);
@@ -738,16 +761,20 @@ function DirectiveChangeList({
 function TargetRow({
   directive,
   item,
-  recordId
+  recordId,
+  index,
+  phase
 }) {
   const host = useDirectiveHost();
   const fallback = recordFallbackName((0, import_content_ir3.nounLabel)(directive.noun, host.nouns), recordId);
   const isUpdate = directive.directiveClass === "update";
-  const current = useRecordValues(
-    isUpdate ? host.readRecord : void 0,
-    isUpdate ? { noun: directive.noun, id: recordId } : null
+  const readsLive = isUpdate && phase.phase === "open";
+  const live = useRecordValues(
+    readsLive ? host.readRecord : void 0,
+    readsLive ? { noun: directive.noun, id: recordId } : null
   );
-  const changes = isUpdate ? itemChanges(item, current.values, { titleColumn: (0, import_content_ir3.nounTitleColumn)(directive.noun, host.nouns) }) : [];
+  const compareTo = phase.phase === "applied" ? phase.before?.[index] ?? null : readsLive ? live.values : null;
+  const changes = isUpdate ? itemChanges(item, compareTo, { titleColumn: (0, import_content_ir3.nounTitleColumn)(directive.noun, host.nouns) }) : [];
   const name = host.renderRecord ? host.renderRecord({ noun: directive.noun, id: recordId, fallback, context: "row" }) : /* @__PURE__ */ (0, import_jsx_runtime4.jsx)("span", { className: "truncate font-medium text-foreground", children: fallback });
   return /* @__PURE__ */ (0, import_jsx_runtime4.jsxs)("div", { className: "py-1 text-xs", "data-directive-target": recordId, children: [
     /* @__PURE__ */ (0, import_jsx_runtime4.jsxs)("div", { className: "flex min-w-0 items-center gap-2", children: [
@@ -764,6 +791,10 @@ function SideEffectDirectiveCard({ directive }) {
   const total = items.length;
   const display = (0, import_content_ir3.directiveDisplay)(directive.directiveClass, directive.noun, host.nouns);
   const Icon = classIcon(directive.directiveClass, directive.noun);
+  const [phase, setPhase] = (0, import_react4.useState)(
+    () => readsApplyLedger(host, directive) ? { phase: "pending" } : { phase: "open" }
+  );
+  const nothingToChange = directive.directiveClass === "update" && total > 0 && items.every((item) => itemChanges(item).length === 0);
   const shown = expanded ? items : items.slice(0, ROWS_BEFORE_FOLD);
   const hidden = total - shown.length;
   return /* @__PURE__ */ (0, import_jsx_runtime4.jsxs)("div", { className: "my-3 rounded-md border border-border bg-card px-2.5 py-1.5", "data-directive": directive.slug, children: [
@@ -778,13 +809,21 @@ function SideEffectDirectiveCard({ directive }) {
       ] }),
       /* @__PURE__ */ (0, import_jsx_runtime4.jsxs)("div", { className: "ml-auto flex items-center gap-1", children: [
         host.renderCopy?.({ label: display.title, value: directive.shell, size: "icon" }),
-        /* @__PURE__ */ (0, import_jsx_runtime4.jsx)(ApplyDirectiveButton, { directive, itemCount: total })
+        /* @__PURE__ */ (0, import_jsx_runtime4.jsx)(
+          ApplyDirectiveButton,
+          {
+            directive,
+            itemCount: total,
+            onPhase: setPhase,
+            nothingToChange
+          }
+        )
       ] })
     ] }),
     total === 0 ? (
       // Stated, never silent: an empty batch is a real (and suspicious) state.
       /* @__PURE__ */ (0, import_jsx_runtime4.jsx)("p", { className: "mt-1 text-xs text-muted-foreground", children: "No items \u2014 nothing would be written." })
-    ) : /* @__PURE__ */ (0, import_jsx_runtime4.jsx)("div", { className: "mt-0.5 divide-y divide-border/60", children: shown.map((item, i) => /* @__PURE__ */ (0, import_jsx_runtime4.jsx)(ItemRow, { directive, item, index: i, total }, i)) }),
+    ) : /* @__PURE__ */ (0, import_jsx_runtime4.jsx)("div", { className: "mt-0.5 divide-y divide-border/60", children: shown.map((item, i) => /* @__PURE__ */ (0, import_jsx_runtime4.jsx)(ItemRow, { directive, item, index: i, total, phase }, i)) }),
     hidden > 0 || expanded ? /* @__PURE__ */ (0, import_jsx_runtime4.jsxs)(
       "button",
       {