@beyondwork/docx-react-component 1.0.57 → 1.0.59

package/README.md

@@ -239,7 +239,7 @@ Engineering work is organized into 9 lanes (5 active now + 2 later polish + 2 fi
 | 6c | [**Context & Review Surfaces**](docs/plans/lane-6c-context-review-surfaces.md) | **~15%** | Gated on 6a.S1+S2 — selection toolbar + suggestion card + rail + scope + context toolbars + health panel restyle; TwCommentPreview / TwEmptyState / TwShortcutHint |
 | 6d | [**Visual Fidelity**](docs/plans/lane-6d-visual-fidelity.md) | **~20%** | Gated on 6a.S1+S2 + external-lane deps — L8 A/B/C shipped; L8 Phase D + P11 overlays + P7 + P12 + P14 next |
 | 7a | [**Visual Fidelity Register**](docs/plans/lane-7a-visual-fidelity-register.md) | **0%** | LATER (gated on 6a–6d) — V5 covers, V6 REF/PAGEREF, V7 cascade audit |
-| 7b | [**Revision & Redline Completion**](docs/plans/lane-7b-revision-redline-completion.md) | **0%** | LATER (gated on 6a–6d) — X4.a/b structural table revisions, X5 ffData, move-pairing |
+| 7b | [**Revision & Redline Completion**](docs/plans/lane-prompts/lane-7b-revision-redline-completion.md) | **✅ CLOSED** | Phases A–N shipped on `lane-7b-revision-redline` (17 commits) — full X4.a parse (container + range markers), X4.b complete (cellIns accept + reject + row-ins/row-del accept + reject), move-promotion (linked-pair atomic accept/reject). Trigger-gated: X4.c + X5 + nested-table row-insertion scope limit. Validator pass handed off to coordinator |
 | 7c | [**Preservation & Format Coverage**](docs/plans/lane-7c-preservation-format-coverage.md) | **0%** | LATER (gated on 6a–6d) — O6 Strict, OLE preserve-only, picture-SDT, w14/kern/VML defer |
 | 7d | [**Platform Hardening & Hygiene**](docs/plans/lane-7d-platform-hardening-hygiene.md) | **0%** | LATER (gated on 6a–6d) — harness-crash-hardening, fastload activation, worktree consolidation |
 | 8 | [**API Ergonomics / Errors / BC**](docs/plans/lane-8-api-ergonomics.md) | **40%** | LATER — Tracks A+C shipped (error catalog + ergonomics fixes); Tracks B+D+E + public-api.md end-to-end refactor remain |

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@beyondwork/docx-react-component",
   "publisher": "beyondwork",
-  "version": "1.0.57",
+  "version": "1.0.59",
   "description": "Embeddable React Word (docx) editor with review, comments, tracked changes, and round-trip OOXML fidelity.",
   "packageManager": "pnpm@10.30.3",
   "type": "module",
@@ -104,6 +104,7 @@
     "test:harness": "pnpm --filter @docx-react-component/react-word-editor-harness test",
     "test:visual": "VISUAL_SMOKE_PROFILE=bare pnpm exec playwright test --project=chromium",
     "test:visual:chrome": "VISUAL_SMOKE_PROFILE=chrome-cycle pnpm exec playwright test --project=chromium",
+    "test:visual:redline": "VISUAL_SMOKE_PROFILE=redline-cycle pnpm exec playwright test --project=chromium",
     "visual:list-runs": "node scripts/visual-smoke-list-runs.mjs",
     "mcp:visual-smoke": "node scripts/visual-smoke-mcp.mjs",
     "lint": "pnpm run lint:no-authored-js && pnpm run lint:docs-contracts && pnpm run lint:tsgo && pnpm run lint:tsgo:harness",

package/src/api/awareness-identity-types.ts

@@ -1,3 +1,5 @@
+import type { CollabParticipantRole } from "./participants-types.ts";
+
 /**
  * Typed shape carried in the Awareness `identity` field. One per
  * connected client. Written by the local client on attach; read by
@@ -7,7 +9,7 @@ export interface AwarenessIdentity {
   userId: string;
   displayName: string;
   collabIdentity?: string;
-  role: "author" | "reviewer" | "observer";
+  role: CollabParticipantRole;
   authorKind: "human" | "agent" | "system";
   /** Identifier of the story the peer is currently looking at, if any. */
   activeStoryId?: string;
@@ -28,7 +30,7 @@ export interface PresenceSnapshot {
 }
 
 export interface CollabPosture {
-  role: "author" | "reviewer" | "observer";
+  role: CollabParticipantRole;
   transport: "none" | "attached";
   /** Count of other connected peers (self excluded). */
   peers: number;

package/src/api/comment-negotiation-types.ts

@@ -1,3 +1,5 @@
+import type { CollabParticipantRole } from "./participants-types.ts";
+
 export type CommentNegotiationState =
   | "proposed"
   | "negotiating"
@@ -105,7 +107,8 @@ export type CommentNegotiationAction =
       actorId: string;
     };
 
-export type NegotiationRole = "author" | "reviewer" | "observer";
+/** @deprecated Use {@link CollabParticipantRole} from participants-types */
+export type NegotiationRole = CollabParticipantRole;
 
 export type NegotiationBlockReason =
   | "negotiation_invalid_transition"

package/src/api/external-custody-types.ts

@@ -65,10 +65,26 @@ export interface ExternalCustodyRestoredContent {
  * this so the library never directly persists internal content.
  */
 export interface ExternalCustodyResolver {
+  /**
+   * Persist the stripped internal content keyed by `custodyId`.
+   * Must be idempotent. Rejections propagate as fatal errors in
+   * the send-to-external flow and are surfaced via `onError`.
+   */
   archive(payload: ExternalCustodyArchivePayload): Promise<void>;
+  /**
+   * Retrieve archived content by custody ID. Returns `undefined`
+   * when the custody record is not found (non-fatal; editor treats
+   * the re-import as a fresh document). Rejections are fatal and
+   * surfaced via `onError`.
+   */
   restore(args: {
     custodyId: string;
     originContentHash: string;
   }): Promise<ExternalCustodyRestoredContent | undefined>;
+  /**
+   * Delete a custody record after a successful internal conversion.
+   * Optional. Rejections are logged non-fatally; conversion completes
+   * regardless.
+   */
   delete?(custodyId: string): Promise<void>;
 }

package/src/api/internal/build-ref-projections.ts

@@ -0,0 +1,108 @@
+// v2.0.0 Track D — purpose-grouped ref projections. Pure dispatch over
+// the already-built WordReviewEditorRef. Called by WordReviewEditor.tsx
+// at both ref-construction sites (the real-runtime bridge and the
+// loading bridge) so projections always point at the live ref.
+
+import type {
+  EditorDiagnosticCode,
+  EditorDiagnosticLlmMetadata,
+  WordReviewEditorChangesFacet,
+  WordReviewEditorCommentsFacet,
+  WordReviewEditorDiagnosticsFacet,
+  WordReviewEditorDocumentFacet,
+  WordReviewEditorFieldsFacet,
+  WordReviewEditorRef,
+  WordReviewEditorScopesFacet,
+} from "../public-types.ts";
+import {
+  CODE_METADATA_TABLE,
+  KNOWN_DIAGNOSTIC_CODES,
+} from "../../runtime/diagnostics/code-metadata-table.ts";
+
+type RefProjections = Pick<
+  WordReviewEditorRef,
+  "comments" | "changes" | "fields" | "document" | "scopes" | "diagnostics"
+>;
+
+/**
+ * Build all six purpose-grouped ref projections. The returned object is
+ * stateless dispatch — each projection method calls back into the live
+ * ref via `getRef()` at call time, so the projections stay current even
+ * if the ref itself is rebuilt (e.g., when the runtime re-mounts).
+ */
+export function buildRefProjections(
+  getRef: () => WordReviewEditorRef,
+): RefProjections {
+  const comments: WordReviewEditorCommentsFacet = {
+    add: (params) => getRef().addComment(params),
+    addReply: (commentId, body) => getRef().addCommentReply(commentId, body),
+    resolve: (commentId) => getRef().resolveComment(commentId),
+    reopen: (commentId) => getRef().reopenComment(commentId),
+    edit: (commentId, body) => getRef().editCommentBody(commentId, body),
+    delete: (commentId) => getRef().deleteComment(commentId),
+    open: (commentId) => getRef().openComment(commentId),
+    scrollTo: (commentId) => getRef().scrollToComment(commentId),
+    get: () => getRef().getComments(),
+  };
+
+  const changes: WordReviewEditorChangesFacet = {
+    accept: (changeId) => getRef().acceptChange(changeId),
+    reject: (changeId) => getRef().rejectChange(changeId),
+    acceptAll: () => getRef().acceptAllChanges(),
+    rejectAll: () => getRef().rejectAllChanges(),
+    acceptGroup: (groupId) => getRef().acceptSuggestionGroup(groupId),
+    rejectGroup: (groupId) => getRef().rejectSuggestionGroup(groupId),
+    scrollTo: (revisionId) => getRef().scrollToRevision(revisionId),
+    get: () => getRef().getTrackedChanges(),
+    getSuggestions: () => getRef().getSuggestionsSnapshot(),
+  };
+
+  const fields: WordReviewEditorFieldsFacet = {
+    getSnapshot: () => getRef().getFieldSnapshot(),
+    update: (options) => getRef().updateFields(options),
+    updateToc: (options) => getRef().updateTableOfContents(options),
+  };
+
+  const document: WordReviewEditorDocumentFacet = {
+    export: (options) => getRef().exportDocx(options),
+    getSessionState: () => getRef().getSessionState(),
+    getSnapshot: () => getRef().getSnapshot(),
+    getRenderSnapshot: () => getRef().getRenderSnapshot(),
+    isDirty: () => getRef().isDirty(),
+    getCompatibilityReport: () => getRef().getCompatibilityReport(),
+    getStats: () => getRef().getRenderSnapshot().documentStats,
+  };
+
+  const scopes: WordReviewEditorScopesFacet = {
+    add: (params) => getRef().addScope(params),
+    get: (scopeId) => getRef().getScope(scopeId),
+    remove: (scopeId) => getRef().removeScope(scopeId),
+    query: (filter) => getRef().queryScopes(filter),
+    findAt: (position, options) => getRef().findScopesAt(position, options),
+    findIntersecting: (range, options) =>
+      getRef().findScopesIntersecting(range, options),
+  };
+
+  // Catalog built once — shape is derived from the static code-metadata table.
+  const catalog: ReadonlyArray<{
+    code: EditorDiagnosticCode;
+    severity: "fatal" | "error" | "warning" | "info";
+    llmMetadata: EditorDiagnosticLlmMetadata;
+  }> = Object.freeze(
+    KNOWN_DIAGNOSTIC_CODES.map((code) => {
+      const row = CODE_METADATA_TABLE[code];
+      return Object.freeze({
+        code,
+        severity: row.severity,
+        llmMetadata: row.llmMetadata,
+      });
+    }),
+  );
+
+  const diagnostics: WordReviewEditorDiagnosticsFacet = {
+    getWarnings: () => getRef().getWarnings(),
+    getCatalog: () => catalog,
+  };
+
+  return { comments, changes, fields, document, scopes, diagnostics };
+}

package/src/api/package-version.ts

@@ -10,4 +10,4 @@
  * the string is committed directly so the runtime works without a build
  * step for tests.
  */
-export const PACKAGE_VERSION = "1.0.36";
+export const PACKAGE_VERSION = "1.0.55";

package/src/api/participants-types.ts

@@ -1,4 +1,14 @@
-export type ParticipantRole = "author" | "reviewer" | "observer";
+/**
+ * Collab-layer role shared across awareness, negotiation, and participant
+ * roster contexts. Distinct from `EditorRole` which governs chrome posture.
+ *
+ * - `"author"` — originating author; full write capability.
+ * - `"reviewer"` — reviewing party; comment + tracked-change capability.
+ * - `"observer"` — read-only presence; no mutations.
+ */
+export type CollabParticipantRole = "author" | "reviewer" | "observer";
+/** @deprecated Use {@link CollabParticipantRole} */
+export type ParticipantRole = CollabParticipantRole;
 export type AuthorKind = "human" | "agent" | "system";
 
 export interface Participant {