@beyondwork/docx-react-component 1.0.95 → 1.0.97

package/src/io/ooxml/parse-shapes.ts

@@ -10,7 +10,11 @@
  * preserved in the canonical node's rawXml field for lossless round-trip export.
  */
 
-import type { BlockNode, ShapeContent } from "../../model/canonical-document.ts";
+import type {
+  BlockNode,
+  ShapeContent,
+  TextBoxBodyProperties,
+} from "../../model/canonical-document.ts";
 import { parseFill } from "./parse-fill.ts";
 import {
   type XmlElementNode,
@@ -32,6 +36,8 @@ export interface ParsedWpsShape {
   text?: string;
   /** Raw txbxContent XML for structured re-rendering. */
   txbxContentXml?: string;
+  /** Text-box body layout from wps:bodyPr / a:bodyPr. */
+  textBoxBody?: TextBoxBodyProperties;
   /**
    * Parsed block-level structure from `w:txbxContent`, populated when a
    * `blockParser` callback is supplied (coord-02 §14 / coord-11 §22 —
@@ -97,6 +103,7 @@ export function parseShapeXml(
   const txbx = findFirstChild(wsp, "txbx");
   const txbxContent = txbx ? findFirstDescendant(txbx, "txbxContent") : null;
   const text = txbxContent ? extractAllText(txbxContent).trim() || undefined : undefined;
+  const textBoxBody = readTextBoxBody(wsp);
 
   // WordArt detection: geometry preset names that start with "text"
   if (prst && /^text/i.test(prst)) {
@@ -145,6 +152,7 @@ export function parseShapeXml(
     type: "shape",
     ...(isTextBox ? { isTextBox: true } : {}),
     ...(text ? { text } : {}),
+    ...(textBoxBody ? { textBoxBody } : {}),
     ...(txbxContentXml ? { txbxContentXml } : {}),
     ...(txbxBlocks && txbxBlocks.length > 0 ? { txbxBlocks } : {}),
     ...(prst ? { geometry: prst } : {}),
@@ -254,12 +262,14 @@ export function parseShapeContent(
 
   const fill = spPr ? readFill(spPr) : undefined;
   const line = spPr ? readLine(spPr) : undefined;
+  const textBoxBody = readTextBoxBody(wsp);
 
   // Text-box content — preserve raw XML for serialization + recurse via the
   // optional blockParser callback (CO4 F3.3) to populate txbxBlocks.
   const txbx = findFirstChild(wsp, "txbx");
   const txbxContent = txbx ? findFirstDescendant(txbx, "txbxContent") : undefined;
   const txbxContentXml = txbxContent ? extractRawXml(txbxContent) : undefined;
+  const text = txbxContent ? extractAllText(txbxContent).trim() || undefined : undefined;
 
   let txbxBlocks: ReadonlyArray<BlockNode> | undefined;
   if (txbxContentXml && blockParser) {
@@ -286,14 +296,37 @@ export function parseShapeContent(
 
   const result: ShapeContent = { type: "shape", rawXml: drawingRawXml };
   if (geometry) result.geometry = geometry;
+  if (text) result.text = text;
   if (fill) result.fill = fill;
   if (line) result.line = line;
   if (isTextBox) result.isTextBox = true;
+  if (textBoxBody) result.textBoxBody = textBoxBody;
   if (txbxContentXml) result.txbxContentXml = txbxContentXml;
   if (txbxBlocks && txbxBlocks.length > 0) result.txbxBlocks = txbxBlocks;
   return result;
 }
 
+function readTextBoxBody(wsp: XmlElementNode): TextBoxBodyProperties | undefined {
+  const bodyPr = findFirstChild(wsp, "bodyPr");
+  if (!bodyPr) return undefined;
+
+  const result: TextBoxBodyProperties = {};
+  const anchor = bodyPr.attributes.anchor;
+  if (anchor === "t" || anchor === "ctr" || anchor === "b") {
+    result.anchor = anchor;
+  }
+  const left = readIntAttr(bodyPr, "lIns");
+  const top = readIntAttr(bodyPr, "tIns");
+  const right = readIntAttr(bodyPr, "rIns");
+  const bottom = readIntAttr(bodyPr, "bIns");
+  if (left !== undefined) result.insetLeftEmu = left;
+  if (top !== undefined) result.insetTopEmu = top;
+  if (right !== undefined) result.insetRightEmu = right;
+  if (bottom !== undefined) result.insetBottomEmu = bottom;
+
+  return Object.keys(result).length > 0 ? result : undefined;
+}
+
 // F3.4 + P5 — readFill delegates to the shared `parseFill` primitive covering
 // solid / none / gradient / pattern. Lane 5 chart-style cascade can consume
 // the same parser via `src/io/ooxml/parse-fill.ts`.
@@ -320,3 +353,10 @@ function readLine(
   }
   return Object.keys(result).length > 0 ? result : undefined;
 }
+
+function readIntAttr(node: XmlElementNode, attr: string): number | undefined {
+  const raw = node.attributes[attr];
+  if (raw === undefined) return undefined;
+  const parsed = parseInt(raw, 10);
+  return Number.isFinite(parsed) ? parsed : undefined;
+}

package/src/model/canonical-document.ts

@@ -1876,6 +1876,8 @@ export interface ShapeNode {
   text?: string;
   geometry?: string;
   isTextBox?: boolean;
+  /** Text box body layout from `wps:bodyPr` / `a:bodyPr`. */
+  textBoxBody?: TextBoxBodyProperties;
   /** Raw `<w:txbxContent>` XML, preserved for serialization + round-trip. */
   txbxContentXml?: string;
   /**
@@ -2001,6 +2003,8 @@ export interface PictureContent {
   packagePartName?: string;
   /** MIME resolved from the OPC media part, when known. */
   contentType?: string;
+  /** DrawingML a:lum brightness/contrast adjustments on the blip. Values are OOXML fixed percentages. */
+  lum?: { bright?: number; contrast?: number };
   srcRect?: { top: number; bottom: number; left: number; right: number };
   stretch?: boolean;
   /**
@@ -2032,6 +2036,8 @@ export interface PictureContent {
 
 export interface ShapeContent {
   type: "shape";
+  /** Plain-text fallback extracted from `w:txbxContent` when present. */
+  text?: string;
   geometry?: string;
   /**
    * Shape fill — solid, gradient, pattern, or none. srgbClr values on solid +
@@ -2061,6 +2067,8 @@ export interface ShapeContent {
   line?: { color?: string; widthEmu?: number; noLine?: boolean };
   /** True when the shape's geometry + txbxContent presence make it a text box. */
   isTextBox?: boolean;
+  /** Text box body layout from `wps:bodyPr` / `a:bodyPr`. */
+  textBoxBody?: TextBoxBodyProperties;
   /** Raw w:txbxContent XML, preserved for serialization + lossless round-trip. */
   txbxContentXml?: string;
   /**
@@ -2080,6 +2088,15 @@ export interface ShapeContent {
   rawXml: string;
 }
 
+export interface TextBoxBodyProperties {
+  /** OOXML bodyPr anchor: top, center, or bottom. */
+  anchor?: "t" | "ctr" | "b";
+  insetLeftEmu?: number;
+  insetTopEmu?: number;
+  insetRightEmu?: number;
+  insetBottomEmu?: number;
+}
+
 export interface DrawingFrameNode {
   type: "drawing_frame";
   anchor: AnchorGeometry;

package/src/runtime/document-runtime.ts

@@ -13,6 +13,7 @@ import {
   type EditorWarning as InternalEditorWarning,
 } from "../core/state/editor-state.ts";
 import {
+  createPlainText,
   logicalPositionToUnitIndex,
   parseTextStory,
   serializeTextStory,
@@ -3195,6 +3196,13 @@ export function createDocumentRuntime(
     replaceText(text, target, formatting) {
       try {
         const timestamp = clock();
+        const selection = target ? createSelectionFromPublicAnchor(target) : state.selection;
+        if (
+          shouldPreserveEquivalentReplacement(formatting) &&
+          replacementTextMatchesCurrentRange(state.document, activeStory, selection, text)
+        ) {
+          return;
+        }
         applyTextCommandInActiveStory(
           {
             type: "text.insert",
@@ -3203,7 +3211,7 @@ export function createDocumentRuntime(
             origin: createOrigin("api", timestamp),
           },
           {
-            selection: target ? createSelectionFromPublicAnchor(target) : state.selection,
+            selection,
             blockedCommandName: "replaceText",
           },
         );
@@ -6448,6 +6456,43 @@ function createSelectionFromPublicAnchor(
   }
 }
 
+function shouldPreserveEquivalentReplacement(formatting: TextFormattingDirective | undefined): boolean {
+  return !formatting || formatting.mode === "match-replaced-range";
+}
+
+function replacementTextMatchesCurrentRange(
+  document: CanonicalDocumentEnvelope,
+  activeStory: EditorStoryTarget,
+  selection: import("../core/state/editor-state.ts").SelectionSnapshot,
+  replacement: string,
+): boolean {
+  const from = Math.max(0, Math.min(selection.anchor, selection.head));
+  const to = Math.max(0, Math.max(selection.anchor, selection.head));
+  if (from === to) {
+    return replacement.length === 0;
+  }
+
+  const content = activeStory.kind === "main"
+    ? document.content
+    : {
+        type: "doc" as const,
+        children: [...getStoryBlocks(document, activeStory)],
+      };
+  const story = parseTextStory(content);
+  if (from > story.size || to > story.size) {
+    return false;
+  }
+
+  const unitFrom = logicalPositionToUnitIndex(story.units, from, "after");
+  const unitTo = logicalPositionToUnitIndex(story.units, to, "before");
+  const selectedText = createPlainText({
+    firstParagraph: story.firstParagraph,
+    units: story.units.slice(unitFrom, unitTo),
+    size: to - from,
+  });
+  return selectedText === replacement;
+}
+
 /**
  * I2 Tier B Slice 4b — extract the selection range from a document as a
  * `CanonicalDocumentFragment`. The fragment preserves text + marks +

package/src/runtime/layout/layout-engine-version.ts

@@ -1018,8 +1018,15 @@
  *   of reconstructing chapter-prefixed numbering in caller-local formulas.
  *   Cache envelopes from v59 invalidate because page graph line-box,
  *   fragment geometry, and page-numbering payloads changed.
+ *
+ *  61 — Page-story resolver + paginator refresh landed on react-refactor
+ *   alongside the floating-drawings-in-page-view fix (`70acfd9ae`). Both
+ *   `page-story-resolver.ts` and `paginated-layout-engine.ts` changed
+ *   without an upstream bump. Bump here so persisted cache envelopes
+ *   re-derive; no algorithm change beyond the floating-drawing clip
+ *   widening already covered by the resurrected tests.
  */
-export const LAYOUT_ENGINE_VERSION = 60 as const;
+export const LAYOUT_ENGINE_VERSION = 61 as const;
 
 /**
  * Serialization schema version for the LayCache payload (the cache envelope

package/src/runtime/layout/page-story-resolver.ts

@@ -149,6 +149,7 @@ function resolveStoryVariant(
         sectionIndex: layout.sectionIndex,
       };
     }
+    return undefined;
   }
 
   // Even page variant when evenAndOddHeaders is active

package/src/runtime/layout/paginated-layout-engine.ts

@@ -2432,17 +2432,33 @@ function isOutOfFlowFrame(
 }
 
 function hasColumnBreak(block: SurfaceBlockSnapshot): boolean {
-  return block.kind === "paragraph" && block.segments.some(
-    (segment) =>
-      segment.kind === "opaque_inline" &&
-      segment.label === "Column break",
-  );
+  if (block.kind === "paragraph") {
+    return block.segments.some(
+      (segment) =>
+        segment.kind === "opaque_inline" &&
+        segment.label === "Column break",
+    );
+  }
+  return nestedBlocks(block).some(hasColumnBreak);
 }
 
 function hasPageBreak(block: SurfaceBlockSnapshot): boolean {
-  return block.kind === "paragraph" && block.segments.some(
-    (segment) =>
-      segment.kind === "opaque_inline" &&
-      segment.label === "Page break",
-  );
+  if (block.kind === "paragraph") {
+    return block.segments.some(
+      (segment) =>
+        segment.kind === "opaque_inline" &&
+        segment.label === "Page break",
+    );
+  }
+  return nestedBlocks(block).some(hasPageBreak);
+}
+
+function nestedBlocks(block: SurfaceBlockSnapshot): readonly SurfaceBlockSnapshot[] {
+  if (block.kind === "sdt_block") {
+    return block.children;
+  }
+  if (block.kind === "table") {
+    return block.rows.flatMap((row) => row.cells.flatMap((cell) => cell.content));
+  }
+  return [];
 }

package/src/runtime/surface-projection.ts

@@ -49,6 +49,34 @@ import {
   isBlockedImportFeatureKey,
 } from "../preservation/store.ts";
 import { getStoryBlocks } from "./story-targeting.ts";
+
+/**
+ * Refactor/11b Slice A — internal seam for the L1 PM Node identity cache.
+ *
+ * Surface-projection attaches a parallel `(BlockNode | null)[]` array
+ * alongside `EditorSurfaceSnapshot.blocks` via a globally-registered
+ * Symbol.  `pm-state-from-snapshot` reads it through an inline mirror of
+ * the same Symbol — `Symbol.for` guarantees both sides resolve to the
+ * same symbol even though they live in different layers and never
+ * import each other.  This avoids a cross-layer import that would blow
+ * the Layer 11 boundary register.
+ *
+ * The property is non-enumerable, so JSON.stringify and public
+ * consumers of `EditorSurfaceSnapshot` see no change in shape.
+ */
+const CANONICAL_BLOCK_REFS_SYMBOL = Symbol.for("wre.canonical-block-refs");
+
+function attachCanonicalBlockRefs(
+  snapshot: EditorSurfaceSnapshot,
+  refs: readonly (BlockNode | null)[],
+): void {
+  Object.defineProperty(snapshot, CANONICAL_BLOCK_REFS_SYMBOL, {
+    value: refs,
+    enumerable: false,
+    configurable: true,
+    writable: false,
+  });
+}
 import {
   collectSectionContexts,
   findHeaderFooterDocumentEntry,
@@ -182,6 +210,12 @@ export function createEditorSurfaceSnapshot(
     children: [...getStoryBlocks(document, activeStory)],
   });
   const blocks: SurfaceBlockSnapshot[] = [];
+  // Refactor/11b Slice A — canonical BlockNode ref parallel to `blocks`.
+  // Feeds the L1 identity cache in `pm-state-from-snapshot` so unchanged
+  // canonical blocks reuse their PM Node instance across commits, which
+  // lets PM's `node.eq()` short-circuit `ViewDesc.update()`. `null` for
+  // placeholder-culled blocks — those are synthetic, not canonical.
+  const blockRefs: (BlockNode | null)[] = [];
   const lockedFragmentIds: string[] = [];
   // L03 boundary: construct one formatting context per projection pass.
   // The context owns the theme resolver + numbering prefix counter +
@@ -251,6 +285,7 @@ export function createEditorSurfaceSnapshot(
 
     if (isInViewport) {
       blocks.push(surfaceBlock.block);
+      blockRefs.push(root.children[index]);
       lockedFragmentIds.push(...surfaceBlock.lockedFragmentIds);
     } else {
       // Replace with size-preserving placeholder. from/to track the SAME
@@ -270,6 +305,8 @@ export function createEditorSurfaceSnapshot(
         placeholderSize,
         state: "placeholder-culled",
       } as SurfaceBlockSnapshot);
+      // Placeholder has no canonical ref — L1 cache will skip it.
+      blockRefs.push(null);
       // Do NOT push lockedFragmentIds — placeholder has no real fragment.
     }
 
@@ -317,7 +354,7 @@ export function createEditorSurfaceSnapshot(
     }
   }
 
-  return {
+  const snapshot: EditorSurfaceSnapshot = {
     storySize: cursor,
     plainText: createPlainText(blocks),
     blocks,
@@ -325,6 +362,8 @@ export function createEditorSurfaceSnapshot(
     secondaryStories,
     viewportBlockRanges,
   };
+  attachCanonicalBlockRefs(snapshot, blockRefs);
+  return snapshot;
 }
 
 /**
@@ -1427,7 +1466,7 @@ function appendInlineSegments(
       return { nextCursor: start + 1, lockedFragmentIds: [node.fragmentId] };
     }
     case "chart_preview": {
-      const parsedChartId = registerParsedChartPreview(node);
+      const parsedChartId = registerParsedChartPreview(node, document);
       return appendComplexPreviewSegment(paragraph, node, start, "Embedded chart", createChartDetail(node), {
         previewMediaId: node.previewMediaId,
         parsedChartId,
@@ -1438,6 +1477,9 @@ function appendInlineSegments(
         previewMediaId: node.previewMediaId,
       });
     case "shape":
+      if (isMicrosoftSensitivityLabelShape(node)) {
+        return { nextCursor: start + 1, lockedFragmentIds: [] };
+      }
       if (promoteSecondaryStoryTextBoxes && node.isTextBox && node.text) {
         return appendTextBoxSegment(
           paragraph,
@@ -1452,6 +1494,9 @@ function appendInlineSegments(
     case "wordart":
       return appendComplexPreviewSegment(paragraph, node, start, "WordArt", createWordArtDetail(node));
     case "vml_shape":
+      if (isMicrosoftSensitivityLabelShape(node)) {
+        return { nextCursor: start + 1, lockedFragmentIds: [] };
+      }
       if (promoteSecondaryStoryTextBoxes && shouldRenderSecondaryStoryVmlTextBox(node)) {
         return appendTextBoxSegment(
           paragraph,
@@ -1487,7 +1532,8 @@ function appendInlineSegments(
         const label = c.isTextBox ? "Text box" : "Drawing shape";
         const detail = `DrawingFrame shape (${node.anchor.wrapMode}).`;
         const anchor = surfaceAnchorFromGeometry(node.anchor);
-        const txbxText = c.isTextBox ? extractTxbxFirstText(c.txbxBlocks) : undefined;
+        const txbxTextSegment = c.isTextBox ? extractTxbxFirstTextSegment(c.txbxBlocks) : undefined;
+        const txbxText = txbxTextSegment?.text ?? (c.isTextBox ? c.text : undefined);
         const surfaceFill = c.fill;
         paragraph.segments.push({
           segmentId: `${paragraph.blockId}-segment-${paragraph.segments.length}`,
@@ -1501,12 +1547,17 @@ function appendInlineSegments(
           ...(surfaceFill ? { fill: surfaceFill } : {}),
           ...(c.line ? { line: c.line } : {}),
           ...(c.isTextBox ? { isTextBox: true } : {}),
+          ...(c.textBoxBody ? { textBoxBody: c.textBoxBody } : {}),
           ...(txbxText ? { txbxText } : {}),
+          ...(txbxTextSegment?.marks && txbxTextSegment.marks.length > 0
+            ? { txbxMarks: txbxTextSegment.marks }
+            : {}),
+          ...(txbxTextSegment?.markAttrs ? { txbxMarkAttrs: txbxTextSegment.markAttrs } : {}),
         });
         return { nextCursor: start + 1, lockedFragmentIds: [] };
       }
       if (c.type === "chart_preview") {
-        const parsedChartId = registerParsedChartPreview(c);
+        const parsedChartId = registerParsedChartPreview(c, document);
         return appendComplexPreviewSegment(
           paragraph,
           c,
@@ -1752,7 +1803,10 @@ function appendInlineSegments(
   }
 }
 
-function registerParsedChartPreview(node: ChartPreviewNode): string | undefined {
+function registerParsedChartPreview(
+  node: ChartPreviewNode,
+  document: CanonicalDocumentEnvelope,
+): string | undefined {
   if (!node.parsedData) return undefined;
   const parsedChartId = stableChartId(node.rawXml);
   // Always call `set` (even when the entry exists) so the active
@@ -1763,7 +1817,7 @@ function registerParsedChartPreview(node: ChartPreviewNode): string | undefined
     model: node.parsedData,
     widthPx,
     heightPx,
-    theme: undefined,
+    theme: document.subParts?.resolvedTheme,
   });
   return parsedChartId;
 }
@@ -1898,6 +1952,7 @@ function surfacePictureEffectsFromContent(
   const outerShadow = resolveSurfacePictureShadow(content.outerShadow, themeResolver);
   const glow = resolveSurfacePictureGlow(content.glow, themeResolver);
   const has =
+    content.lum !== undefined ||
     content.srcRect !== undefined ||
     content.rotation !== undefined ||
     content.flipH !== undefined ||
@@ -1909,6 +1964,7 @@ function surfacePictureEffectsFromContent(
     glow !== undefined;
   if (!has) return undefined;
   return {
+    ...(content.lum ? { lum: { ...content.lum } } : {}),
     ...(content.srcRect ? { srcRect: { ...content.srcRect } } : {}),
     ...(content.rotation !== undefined ? { rotation: content.rotation } : {}),
     ...(content.flipH !== undefined ? { flipH: content.flipH } : {}),
@@ -2007,16 +2063,51 @@ function flattenSurfaceFieldDisplayText(
  * and read the immediate run content.  Returns `undefined` when no text
  * is present.
  */
-function extractTxbxFirstText(
+function extractTxbxFirstTextSegment(
   blocks: ShapeContent["txbxBlocks"],
-): string | undefined {
+): {
+  text: string;
+  marks?: SurfaceTextMark[];
+  markAttrs?: {
+    backgroundColor?: string;
+    charSpacing?: number;
+    kerning?: number;
+    textFill?: string;
+    fontFamily?: string;
+    fontSize?: number;
+    textColor?: string;
+  };
+} | undefined {
   if (!blocks || blocks.length === 0) return undefined;
   for (const block of blocks) {
     if (block.type !== "paragraph") continue;
-    const runs = (block as { runs?: ReadonlyArray<{ text?: string }> }).runs;
-    if (!runs) continue;
-    const text = runs.map((r) => r.text ?? "").join("").trim();
-    if (text) return text;
+    const children = (block as { children?: ReadonlyArray<{ type?: string; text?: string; marks?: TextMark[] }> }).children;
+    const legacyRuns = (block as { runs?: ReadonlyArray<{ text?: string; marks?: TextMark[] }> }).runs;
+    if (!children && !legacyRuns) continue;
+    let firstMarks: TextMark[] | undefined;
+    const text = (children
+      ? children
+          .map((child) => {
+            if (child.type !== "text") return "";
+            if (!firstMarks && child.marks) firstMarks = child.marks;
+            return child.text ?? "";
+          })
+          .join("")
+      : legacyRuns
+          ?.map((run) => {
+            if (!firstMarks && run.marks) firstMarks = run.marks;
+            return run.text ?? "";
+          })
+          .join("") ?? ""
+    ).trim();
+    if (text) {
+      const cloned = firstMarks ? cloneMarks(firstMarks) : undefined;
+      return {
+        text,
+        ...(cloned?.marks && cloned.marks.length > 0 ? { marks: cloned.marks } : {}),
+        ...(cloned?.markAttrs ? { markAttrs: cloned.markAttrs } : {}),
+      };
+    }
   }
   return undefined;
 }
@@ -2072,6 +2163,17 @@ function shouldRenderSecondaryStoryVmlTextBox(node: VmlShapeNode): boolean {
   return Boolean(node.text) && (!node.shapeType || /_x0000_t202$/iu.test(node.shapeType));
 }
 
+function isMicrosoftSensitivityLabelShape(
+  node: ShapeNode | VmlShapeNode,
+): boolean {
+  if (!/classification/i.test(node.text ?? "")) {
+    return false;
+  }
+  return /\b(?:id|name)="MSIPCM/iu.test(node.rawXml) ||
+    /\b(?:alt|descr)="[^"]*&quot;Placement&quot;:&quot;Footer&quot;/iu.test(node.rawXml) ||
+    /"Placement"\s*:\s*"Footer"/iu.test(node.rawXml);
+}
+
 function createChartDetail(node: ChartPreviewNode): string {
   const parts = ["Embedded chart."];
   if (node.previewMediaId) {

package/src/runtime/workflow/rail/compose.ts

@@ -94,6 +94,11 @@ export function collectScopeRailSegments(
   const activeIds = new Set(input.activeWorkItemScopeIds ?? []);
 
   for (const scope of input.scopes ?? []) {
+    // Invisible scopes are runtime/agent context only. They may still
+    // participate in guard decisions, but they must not surface as rail,
+    // card, or body-tint chrome.
+    if (scope.visibility === "invisible") continue;
+
     const range = anchorToRuntimeRange(scope.anchor);
     if (!range) continue;
     const storyTarget = scope.storyTarget ?? MAIN_STORY_TARGET;

package/src/ui/WordReviewEditor.tsx

@@ -187,7 +187,6 @@ import {
   withExportDelivery,
 } from "./browser-export";
 import { EditorShellView } from "./editor-shell-view.tsx";
-import { TwDebugPresentation } from "../ui-tailwind/debug/index.ts";
 import { shellPasteFragmentParser as SHELL_PASTE_FRAGMENT_PARSER } from "../shell/paste-adapter.ts";
 import { EditorSurfaceController } from "./editor-surface-controller.tsx";
 import type { EditorActionHostCallbacks } from "../ui-tailwind/chrome/editor-action-registry";
@@ -1061,7 +1060,6 @@ export const WordReviewEditor = forwardRef<WordReviewEditorRef, WordReviewEditor
       chromeControllerRef,
       commandPaletteDisabled,
       customSelectionTools,
-      debugMode = "off",
     } = props;
 
     const [activeRailTab, setActiveRailTab] = useState<ReviewRailTab>("comments");
@@ -1140,7 +1138,6 @@ export const WordReviewEditor = forwardRef<WordReviewEditorRef, WordReviewEditor
       hostPosture: {
         reviewMode: reviewMode === "review" ? "reviewer" : "author",
         markupDisplay: normalizeHostMarkupDisplay(markupDisplay),
-        debugMode,
         chromePreset,
       },
       chromePresetInput: {
@@ -1261,7 +1258,6 @@ export const WordReviewEditor = forwardRef<WordReviewEditorRef, WordReviewEditor
       api,
       reviewMode,
       markupDisplay,
-      debugMode,
       chromePreset,
       readOnly,
     ]);
@@ -3395,6 +3391,10 @@ export const WordReviewEditor = forwardRef<WordReviewEditorRef, WordReviewEditor
         activeRuntime.rejectAllChanges();
         setActiveRailTab("changes");
       },
+      onAcceptSuggestionGroup: (groupId: string) =>
+        applySuggestionGroupAction(activeRuntime, groupId, "accept"),
+      onRejectSuggestionGroup: (groupId: string) =>
+        applySuggestionGroupAction(activeRuntime, groupId, "reject"),
       onCloseStory: () => {
         activeRuntime.closeStory();
       },
@@ -3907,10 +3907,10 @@ export const WordReviewEditor = forwardRef<WordReviewEditorRef, WordReviewEditor
           });
         }}
         onScopeAcceptSuggestionGroup={(payload) => {
-          applySuggestionGroupAction(activeRuntime, payload.groupId, "accept");
+          commands.onAcceptSuggestionGroup?.(payload.groupId);
         }}
         onScopeRejectSuggestionGroup={(payload) => {
-          applySuggestionGroupAction(activeRuntime, payload.groupId, "reject");
+          commands.onRejectSuggestionGroup?.(payload.groupId);
         }}
         mediaPreviews={mediaPreviews}
         onActivateFloatingImage={(payload) => {
@@ -3959,10 +3959,6 @@ export const WordReviewEditor = forwardRef<WordReviewEditorRef, WordReviewEditor
         }}
       />
       <TwRuntimeReplDialog runtime={activeRuntime} editorRef={editorRefForRepl} />
-      <TwDebugPresentation
-        mode={debugMode}
-        sessionId={documentId}
-      />
       </>
       </OverlayAnchorBridgeProvider>
       </UiShellChannelsProvider>

package/src/ui/editor-command-bag.ts

@@ -104,6 +104,8 @@ export interface EditorCommandBag {
   onRejectRevision(revisionId: string): void;
   onAcceptAllChanges(): void;
   onRejectAllChanges(): void;
+  onAcceptSuggestionGroup?(groupId: string): void;
+  onRejectSuggestionGroup?(groupId: string): void;
   onCloseStory?(): void;
   /**
    * @deprecated P8.11 — see the matching prop on `TwReviewWorkspaceProps`.

package/src/ui/ui-controller-factory.ts

@@ -79,7 +79,7 @@ let nextControllerId = 0;
 export interface ShellUiControllerDeps {
   /**
    * Current host-provided posture slice — reviewMode / markupDisplay /
-   * debugMode / chromePreset. MUST read through a live ref so the factory
+   * chromePreset. MUST read through a live ref so the factory
    * closure returns current render state, not stale construction-time state.
    */
   readonly getHostPosture?: () => ChromeHostPosture | undefined;
@@ -95,7 +95,7 @@ export interface ShellUiControllerDeps {
   readonly getOverlayAnchor?: (query: OverlayAnchorQuery) => GeometryRect | null;
   /**
    * Posture-change stream. Fires when any of the inputs to `ChromePosture`
-   * change — reviewMode / markupDisplay / debugMode / chromePreset (host),
+   * change — reviewMode / markupDisplay / chromePreset (host),
    * effectiveMode / blockedReasons / documentMode / readOnly (runtime).
    * Returns an unsubscribe function.
    *