@portone/docx-editor 0.3.0 → 0.5.0

package/dist/docx/comments/grammar.js

@@ -1,14 +1,9 @@
 // src/docx/comments/grammar.ts
-import {
-  attrsText,
-  elementXml,
-  xmlnsAttr
-} from "../../ooxml/element.js";
-import { NAMESPACES, qualify, wName } from "../../ooxml/names.js";
+import { attrsText, elementXml } from "../../ooxml/element.js";
+import { NAMESPACES, qualify } from "../../ooxml/names.js";
 import {
   attributeByLocalName,
   elementChildren,
-  escapeXml,
   parseXml,
   W_NS
 } from "../../ooxml/xml.js";
@@ -16,7 +11,6 @@ import { COMMENT_AUTHOR_PROVIDER, W14_NS, W15_NS } from "./constants.js";
 var ELEMENT_NODE = 1;
 var TEXT_NODE = 3;
 var XMLNS_NS = "http://www.w3.org/2000/xmlns/";
-var XML_NS = "http://www.w3.org/XML/1998/namespace";
 var nameKey = (namespace, localName) => `${namespace ?? ""} ${localName}`;
 var COMMENT_ATTRIBUTES = /* @__PURE__ */ new Set([
   nameKey(W_NS, "id"),
@@ -29,12 +23,6 @@ var COMMENT_EX_ATTRIBUTES = /* @__PURE__ */ new Set([
   nameKey(W15_NS, "paraIdParent"),
   nameKey(W15_NS, "done")
 ]);
-var PARAGRAPH_ATTRIBUTES = /* @__PURE__ */ new Set([
-  nameKey(W14_NS, "paraId")
-]);
-var TEXT_ATTRIBUTES = /* @__PURE__ */ new Set([
-  nameKey(XML_NS, "space")
-]);
 var PERSON_ATTRIBUTES = /* @__PURE__ */ new Set([
   nameKey(W15_NS, "author")
 ]);
@@ -42,7 +30,6 @@ var PRESENCE_ATTRIBUTES = /* @__PURE__ */ new Set([
   nameKey(W15_NS, "providerId"),
   nameKey(W15_NS, "userId")
 ]);
-var NO_ATTRIBUTES = /* @__PURE__ */ new Set();
 var THREAD_KEY = /^[0-9A-Fa-f]{8}$/;
 function threadKeyOrNone(el, localName) {
   const value = attributeByLocalName(el, localName);
@@ -57,6 +44,11 @@ function declarationWritten(attr) {
   if (!attr.name.startsWith("xmlns:")) return false;
   return WRITTEN_NAMESPACES.get(attr.name.slice("xmlns:".length)) === attr.value;
 }
+function declarationsWritten(el) {
+  return Array.from(el.attributes).every(
+    (attr) => !attr.name.startsWith("xmlns") || declarationWritten(attr)
+  ) && Array.from(el.children).every(declarationsWritten);
+}
 function attributesWithin(el, allowed) {
   return Array.from(el.attributes).every(
     (attr) => attr.namespaceURI === XMLNS_NS ? declarationWritten(attr) : allowed.has(nameKey(attr.namespaceURI, attr.localName))
@@ -70,22 +62,13 @@ function holdsElementsOnly(el) {
     (node) => node.nodeType === ELEMENT_NODE
   );
 }
-function holdsTextOnly(el) {
-  return Array.from(el.childNodes).every((node) => node.nodeType === TEXT_NODE);
-}
-function renderCommentBody(text, paraId) {
-  const lines = text.split("\n");
-  const pieces = [];
-  lines.forEach((line, index) => {
-    if (index > 0) pieces.push(elementXml(wName("br"), []));
-    if (line.length > 0 || lines.length === 1) {
-      pieces.push(
-        elementXml(wName("t"), [["xml:space", "preserve"]], [escapeXml(line)])
-      );
-    }
-  });
-  const key = paraId === null ? [] : [xmlnsAttr("w14"), [qualify("w14", "paraId"), paraId]];
-  return elementXml(wName("p"), key, [elementXml(wName("r"), [], pieces)]);
+function isLayoutText(node) {
+  return node.nodeType === TEXT_NODE && /^[ \t\r\n]*$/.test(node.nodeValue ?? "");
+}
+function holdsElementsAndLayout(el) {
+  return Array.from(el.childNodes).every(
+    (node) => node.nodeType === ELEMENT_NODE || isLayoutText(node)
+  );
 }
 var PARAGRAPH_OR_SKIPPED = /<!--[\s\S]*?-->|<!\[CDATA\[[\s\S]*?\]\]>|<([^\s<>/:="']+:)?p(?=[\s/>])[^>]*>/g;
 function lastBodyParagraph(comment) {
@@ -132,25 +115,6 @@ function withThreadKey(commentXml, paraId, arrived) {
   const keyed = opening[0].slice(0, selfClosing ? -2 : -1) + ` ${attrsText([[qualify("w14", "paraId"), paraId]])}` + (selfClosing ? "/>" : ">");
   return commentXml.slice(0, opening.index) + keyed + commentXml.slice(opening.index + opening[0].length);
 }
-function readRunText(run) {
-  if (!holdsElementsOnly(run)) return null;
-  const pieces = [];
-  for (const child of Array.from(run.children)) {
-    if (isNamed(child, W_NS, "br")) {
-      if (!attributesWithin(child, NO_ATTRIBUTES) || child.childNodes.length > 0) {
-        return null;
-      }
-      pieces.push("\n");
-      continue;
-    }
-    if (!isNamed(child, W_NS, "t")) return null;
-    if (!attributesWithin(child, TEXT_ATTRIBUTES) || child.getAttributeNS(XML_NS, "space") !== "preserve" || !holdsTextOnly(child)) {
-      return null;
-    }
-    pieces.push(child.textContent ?? "");
-  }
-  return pieces.join("");
-}
 function wellFormedCommentExtension(entry) {
   return attributesWithin(entry, COMMENT_EX_ATTRIBUTES) && entry.childNodes.length === 0 && threadKeyOrNone(entry, "paraId") && threadKeyOrNone(entry, "paraIdParent") && // The writer says a thread is settled or open, and nothing else `ST_OnOff` would take
   ["0", "1", null].includes(attributeByLocalName(entry, "done"));
@@ -167,21 +131,6 @@ function recordedIdentity(person) {
   const [presence] = Array.from(person.children);
   return presence === void 0 ? null : attributeByLocalName(presence, "userId");
 }
-function readStrictCommentBody(comment) {
-  if (!holdsElementsOnly(comment)) return null;
-  const paragraphs = Array.from(comment.children);
-  if (paragraphs.length !== 1) return null;
-  const [paragraph] = paragraphs;
-  if (!isNamed(paragraph, W_NS, "p")) return null;
-  if (!attributesWithin(paragraph, PARAGRAPH_ATTRIBUTES)) return null;
-  if (!holdsElementsOnly(paragraph)) return null;
-  const runs = Array.from(paragraph.children);
-  if (runs.length !== 1) return null;
-  const [run] = runs;
-  if (!isNamed(run, W_NS, "r")) return null;
-  if (!attributesWithin(run, NO_ATTRIBUTES)) return null;
-  return readRunText(run);
-}
 function renderCommentExtension(comment) {
   if (comment.extensionXml !== null) return comment.extensionXml;
   const parent = "parentParaId" in comment ? [[qualify("w15", "paraIdParent"), comment.parentParaId]] : [];
@@ -211,10 +160,12 @@ export {
   arrivedEntries,
   attributesWithin,
   declarationWritten,
+  declarationsWritten,
+  holdsElementsAndLayout,
+  holdsElementsOnly,
+  isLayoutText,
   lastBodyParagraph,
-  readStrictCommentBody,
   recordedIdentity,
-  renderCommentBody,
   renderCommentExtension,
   renderPerson,
   wellFormedCommentExtension,

package/dist/docx/comments/model.d.ts

@@ -3,15 +3,17 @@
  */
 import type { Node as PMNode } from "prosemirror-model";
 import type { ImportedComments } from "./reading";
+/**
+ * A comment as the reference node in the story carries it: who wrote it, where its thread state
+ * hangs, and the replies under it. What it says is not here - a body is a story of its own, held
+ * on the document node under `comment:<id>` (`docx/story`).
+ */
 export interface CommentReferenceData {
     id: string;
     author: string | null;
     authorId: string | null;
     initials: string | null;
     date: string | null;
-    text: string;
-    commentXml: string | null;
-    imported: boolean;
     paraId: string;
     resolved: boolean;
     extensionXml: string | null;
@@ -24,9 +26,6 @@ export interface CommentReplyData {
     authorId: string | null;
     initials: string | null;
     date: string | null;
-    text: string;
-    commentXml: string | null;
-    imported: boolean;
     paraId: string;
     parentParaId: string;
     extensionXml: string | null;

package/dist/docx/comments/model.js

@@ -26,9 +26,6 @@ function importedCommentReplies(comments, rootId) {
       authorId: reply.authorId,
       initials: reply.initials,
       date: reply.date,
-      text: reply.text,
-      commentXml: reply.xml,
-      imported: true,
       paraId,
       parentParaId,
       extensionXml: reply.extensionXml
@@ -59,9 +56,6 @@ function replyData(value) {
         authorId: nullableString(entry.authorId),
         initials: nullableString(entry.initials),
         date: nullableString(entry.date),
-        text: nullableString(entry.text) ?? "",
-        commentXml: nullableString(entry.commentXml),
-        imported: entry.imported === true,
         paraId,
         parentParaId,
         extensionXml: nullableString(entry.extensionXml)
@@ -79,9 +73,6 @@ function referenceData(node) {
     authorId: nullableString(node.attrs.authorId),
     initials: nullableString(node.attrs.initials),
     date: nullableString(node.attrs.date),
-    text: nullableString(node.attrs.text) ?? "",
-    commentXml: nullableString(node.attrs.commentXml),
-    imported: node.attrs.imported === true,
     paraId: nullableString(node.attrs.paraId) ?? "00000001",
     resolved: node.attrs.resolved === true,
     extensionXml: nullableString(node.attrs.extensionXml),

package/dist/docx/comments/parts.d.ts

@@ -11,16 +11,17 @@
  * have made the change. Asking them as one would refuse a moderator rewriting somebody else's
  * comment under `editableComments: "all"`, which is a thing the editor lets them do.
  */
-import type { EditableComments } from "../../schema/protection";
+import { type EditableComments } from "../../schema/protection";
 import type { StoryPartKind } from "../protectionPolicy";
 import { type ImportedPeople } from "./people";
 /**
  * Whether this editor's writer could have put the entry out, whoever it belongs to.
  *
- * Shape alone: which attributes it carries and what stands inside it. Who may have written it is
- * `entryAllowed`'s question. Each kind is read where it is written (`./grammar`).
+ * Shape alone: which attributes it carries and what stands inside it, judged against the entry the
+ * original file held under the same id. Who may have written it is `entryAllowed`'s question.
+ * Each kind is read where it is written (`./grammar`).
  */
-export declare function wellFormedEntry(entry: Element): boolean;
+export declare function wellFormedEntry(entry: Element, original: Element | null): boolean;
 /**
  * Whether the only thing between the two is that a thread gained its key.
  *
@@ -34,13 +35,18 @@ export declare function threadKeyAlone(entry: Element, original: Element): boole
  * Whether `authorId` could have written or rewritten this entry.
  *
  * Permission alone: whether the shape is one this editor writes is `wellFormedEntry`'s question.
- * An entry that appeared has to be this author's. One that arrived keeps the identity it arrived
- * with; settling its thread or replying to it belong to everyone and leave what it says alone,
- * while rewriting what it says is its author's, a moderator's, or anyone's where no identity was
- * recorded for it. That last is the rule the editor holds to (`schema/protection`), and the two
- * have to answer alike or a file the editor wrote would be turned down here.
+ * An entry that appeared has to be one this author could have written, which
+ * `commentAdditionAllowed` decides here as it does over the story. One that arrived keeps the
+ * identity it arrived with; settling its thread or replying to it belong to everyone and leave
+ * what it says alone, while rewriting what it says is its author's, a moderator's, or anyone's
+ * where no identity was recorded for it. Both are the rules the editor holds to
+ * (`schema/protection`), and the two have to answer alike or a file the editor wrote would be
+ * turned down here.
+ *
+ * `unattributed` is read from the file that arrived, since it says what the submission is judged
+ * against; `people` is the submission's, since it says who the submission names.
  */
-export declare function entryAllowed(entry: Element, original: Element | null, authorId: string, editableComments: EditableComments, people: ImportedPeople): boolean;
+export declare function entryAllowed(entry: Element, original: Element | null, authorId: string, editableComments: EditableComments, people: ImportedPeople, unattributed: ReadonlySet<string>): boolean;
 export declare const commentsPart: StoryPartKind;
 export declare const commentsExtendedPart: StoryPartKind;
 export declare const peoplePart: StoryPartKind;

package/dist/docx/comments/parts.js

@@ -7,7 +7,16 @@ import {
   serializeXml,
   W_NS
 } from "../../ooxml/xml.js";
+import { isPreservedNode } from "../../schema/preservedFragments.js";
+import {
+  commentAdditionAllowed
+} from "../../schema/protection.js";
+import { NO_FORMATTING } from "../formatting.js";
+import { NO_IMPORT_SOURCES } from "../importParagraph.js";
 import { availablePartPath } from "../packageParts.js";
+import { serializeBlock } from "../serializeBlock.js";
+import { buildBlock } from "../story.js";
+import { isModelledBlock } from "../storyProjection.js";
 import {
   COMMENTS_CONTENT_TYPE,
   COMMENTS_EXTENDED_CONTENT_TYPE,
@@ -21,9 +30,11 @@ import {
 import {
   attributesWithin,
   COMMENT_ATTRIBUTES,
+  declarationsWritten,
   declarationWritten,
+  holdsElementsAndLayout,
+  isLayoutText,
   lastBodyParagraph,
-  readStrictCommentBody,
   recordedIdentity,
   wellFormedCommentExtension,
   wellFormedPerson
@@ -42,9 +53,46 @@ function sameAttributes(entry, original, names) {
     (name) => attributeByLocalName(entry, name) === attributeByLocalName(original, name)
   );
 }
-function wellFormedEntry(entry) {
+function holdsPreserved(node) {
+  if (isPreservedNode(node)) return true;
+  let found = false;
+  node.descendants((child) => {
+    if (isPreservedNode(child)) found = true;
+    return !found;
+  });
+  return found;
+}
+var BLOCK_DECLARATIONS = Object.entries(NAMESPACES).map(([prefix, uri]) => `xmlns:${prefix}="${uri}"`).join(" ");
+function normalizedBlock(xml) {
+  try {
+    const root = parseXml(
+      `<b ${BLOCK_DECLARATIONS}>${xml}</b>`
+    ).documentElement;
+    const [block] = Array.from(root.children);
+    return block === void 0 || root.children.length !== 1 ? null : serializeXml(block);
+  } catch {
+    return null;
+  }
+}
+function writtenBlock(el) {
+  if (!declarationsWritten(el)) return false;
+  const node = buildBlock(el, "", NO_IMPORT_SOURCES, NO_FORMATTING);
+  if (!isModelledBlock(node) || holdsPreserved(node)) return false;
+  const written = normalizedBlock(serializeBlock(node));
+  return written !== null && written === normalizedBlock(serializeXml(el));
+}
+function wellFormedStoryBody(entry, original) {
+  if (!holdsElementsAndLayout(entry)) return false;
+  const kept = new Set(
+    original === null ? [] : Array.from(original.children, serializeXml)
+  );
+  return Array.from(entry.children).every(
+    (block) => kept.has(serializeXml(block)) || writtenBlock(block)
+  );
+}
+function wellFormedEntry(entry, original) {
   if (entry.namespaceURI === W_NS && entry.localName === "comment") {
-    return attributesWithin(entry, COMMENT_ATTRIBUTES) && readStrictCommentBody(entry) !== null;
+    return attributesWithin(entry, COMMENT_ATTRIBUTES) && wellFormedStoryBody(entry, original);
   }
   if (entry.namespaceURI === W15_NS && entry.localName === "commentEx") {
     return wellFormedCommentExtension(entry);
@@ -67,11 +115,17 @@ function threadKeyAlone(entry, original) {
   const kept = before === null || after === before;
   return kept && withoutThreadKey(entry) === withoutThreadKey(original);
 }
-function entryAllowed(entry, original, authorId, editableComments, people) {
+function entryAllowed(entry, original, authorId, editableComments, people, unattributed) {
   if (entry.namespaceURI === W_NS && entry.localName === "comment") {
     const author = attributeByLocalName(entry, "author");
     const recorded = author === null ? null : commentAuthorId(people, author);
-    if (original === null) return recorded === authorId;
+    if (original === null) {
+      return commentAdditionAllowed(
+        { author, authorId: recorded },
+        authorId,
+        unattributed
+      );
+    }
     const paraId = lastParagraphId(original);
     if (!sameAttributes(entry, original, COMMENT_IDENTITY) || paraId !== null && lastParagraphId(entry) !== paraId) {
       return false;
@@ -83,23 +137,20 @@ function entryAllowed(entry, original, authorId, editableComments, people) {
     return sameAttributes(entry, original, ["paraId", "paraIdParent"]);
   }
   if (entry.namespaceURI === W15_NS && entry.localName === "person") {
-    return original === null && recordedIdentity(entry) === authorId;
+    const author = attributeByLocalName(entry, "author");
+    return original === null && recordedIdentity(entry) === authorId && author !== null && !unattributed.has(author);
   }
   return false;
 }
-var TEXT_NODE = 3;
 var COMPATIBILITY = /* @__PURE__ */ new Map([
   ["xmlns:w14", NAMESPACES.w14],
   ["xmlns:mc", NAMESPACES.mc]
 ]);
 var IGNORABLE = "mc:Ignorable";
 var IGNORED = "w14";
-function isLayout(node) {
-  return node.nodeType === TEXT_NODE && /^[ \t\r\n]*$/.test(node.nodeValue ?? "");
-}
 function surroundingNodes(parent) {
   return parent === null ? [] : Array.from(parent.childNodes).filter(
-    (node) => !isElement(node) && !isLayout(node)
+    (node) => !isElement(node) && !isLayoutText(node)
   );
 }
 function sameNodes(now, was) {
@@ -192,12 +243,13 @@ function storyPart(shape) {
     referents: shape.referents,
     wellFormed: wellFormedEntry,
     anyonesChange: threadKeyAlone,
-    allowed: (entry, original, authorId, options, session) => entryAllowed(
+    allowed: (entry, original, authorId, options, session, unattributed) => entryAllowed(
       entry,
       original,
       authorId,
       options.editableComments,
-      session.comments.people
+      session.comments.people,
+      unattributed
     )
   };
 }

package/dist/docx/comments/people.d.ts

@@ -8,6 +8,8 @@
  * part by, so a name stands for one identity per file: a name the part already records is read as
  * it stands and never appended to. Appending a second person for it would leave a file naming two
  * identities for one name, where a reader keying by name hands one author's comments to the other.
+ * A name the file's comments already write under carries the same weight, recorded or not: a
+ * person appended for it would claim those comments too.
  */
 import type { PartPlanContext } from "../partPlan";
 import type { StoryPartKind } from "../protectionPolicy";
@@ -33,10 +35,16 @@ export declare function readPeople(parts: Map<string, Uint8Array>, mainPartPath:
  */
 export declare function commentAuthorId(people: ImportedPeople, author: string): string | null;
 /**
- * The authors the part records no person for at all, each under the name they write as. A name it
- * already records is left as it stands, whichever provider recorded it and whatever it resolves to.
+ * The authors to record a person for, each under the name they write as: the ones the comments
+ * name one identity for and the part does not record already. A name the part records is left as
+ * it stands, whichever provider recorded it and whatever it resolves to. So is a name the comments
+ * carry under more than one identity, one identity and none among them: recording it would hand
+ * the comments carrying no identity to whoever the name then resolved to, and a document saved by
+ * a Word that wrote no people part carries every one of its comments that way.
+ * `unattributed` also reserves names from the original Comments part, including preserved entries
+ * the current editable bodies do not carry. The planner and export invariants use this same set.
  */
-export declare function unrecordedAuthors(bodies: Iterable<CommentReferenceData | CommentReplyData>, people: ImportedPeople): Map<string, string>;
+export declare function unrecordedAuthors(bodies: Iterable<CommentReferenceData | CommentReplyData>, people: ImportedPeople, unattributed: ReadonlySet<string>): Map<string, string>;
 /**
  * Plans the people part, its relationship and its content type for every identity the current
  * comments carry that the document has not recorded. Null when it has recorded them all, which

package/dist/docx/comments/people.js

@@ -10,6 +10,7 @@ import {
   encodeUtf8,
   parseXml
 } from "../../ooxml/xml.js";
+import { unattributedCommentAuthors } from "../../schema/protection.js";
 import { relatedPartPath } from "../packageParts.js";
 import { directoryOf } from "../relationships.js";
 import { COMMENT_AUTHOR_PROVIDER, PEOPLE_REL_TYPE, W15_NS } from "./constants.js";
@@ -51,12 +52,21 @@ function readPeople(parts, mainPartPath) {
 function commentAuthorId(people, author) {
   return people.byAuthor.get(author) ?? null;
 }
-function unrecordedAuthors(bodies, people) {
-  const unrecorded = /* @__PURE__ */ new Map();
+function unrecordedAuthors(bodies, people, unattributed) {
+  const identities = /* @__PURE__ */ new Map();
   for (const body of bodies) {
-    if (body.author === null || body.authorId === null) continue;
-    if (people.byAuthor.has(body.author)) continue;
-    unrecorded.set(body.author, body.authorId);
+    if (body.author === null) continue;
+    if (people.byAuthor.has(body.author) || unattributed.has(body.author))
+      continue;
+    const ids = identities.get(body.author) ?? /* @__PURE__ */ new Set();
+    ids.add(body.authorId);
+    identities.set(body.author, ids);
+  }
+  const unrecorded = /* @__PURE__ */ new Map();
+  for (const [author, ids] of identities) {
+    if (ids.size !== 1) continue;
+    const only = Array.from(ids)[0] ?? null;
+    if (only !== null) unrecorded.set(author, only);
   }
   return unrecorded;
 }
@@ -94,7 +104,11 @@ function peopleXml(people, added) {
 }
 function planPeoplePart(part, bodies, session, context) {
   const people = session.comments.people;
-  const added = unrecordedAuthors(bodies, people);
+  const added = unrecordedAuthors(
+    bodies,
+    people,
+    unattributedCommentAuthors(session.comments.ordered)
+  );
   if (added.size === 0) return null;
   const parts = /* @__PURE__ */ new Map();
   const addingPart = part.pathIn(session) === null;

package/dist/docx/comments/policy.js

@@ -3,6 +3,7 @@ import {
   commentAdditionsBy,
   commentEditsOwned,
   commentIdentitiesKept,
+  unattributedCommentAuthors,
   withoutComments
 } from "../../schema/protection.js";
 import {
@@ -18,7 +19,12 @@ function sameBody(before, after) {
 }
 function commentsStoryKept(before, after, authorId, { editableComments }) {
   if (!sameBody(before, after)) return refused("body-changed");
-  if (!commentIdentitiesKept(before.doc, after.doc) || !commentAdditionsBy(before.doc, after.doc, authorId)) {
+  if (!commentIdentitiesKept(before.doc, after.doc) || !commentAdditionsBy(
+    before.doc,
+    after.doc,
+    authorId,
+    unattributedCommentAuthors(before.session.comments.ordered)
+  )) {
     return refused("comment-author-forged");
   }
   if (!commentEditsOwned(before.doc, after.doc, {

package/dist/docx/comments/reading.d.ts

@@ -2,6 +2,10 @@
  * Reads comment and comment-extension package parts.
  */
 import { type ImportedPeople } from "./people";
+/**
+ * One entry of the Comments part as it arrived, everything about it but what it says: that is a
+ * story of its own, sliced and read the way a body block is (`docx/story`).
+ */
 export interface ImportedComment {
     id: string;
     author: string | null;
@@ -13,8 +17,6 @@ export interface ImportedComment {
     authorId: string | null;
     initials: string | null;
     date: string | null;
-    text: string;
-    xml: string;
     paraId: string | null;
     parentParaId: string | null;
     resolved: boolean;

package/dist/docx/comments/reading.js

@@ -4,8 +4,7 @@ import {
   decodeUtf8,
   elementChildren,
   parseXml,
-  serializeXml,
-  W_NS
+  serializeXml
 } from "../../ooxml/xml.js";
 import { relatedPartPath } from "../packageParts.js";
 import { COMMENTS_EXTENDED_REL_TYPE, COMMENTS_REL_TYPE } from "./constants.js";
@@ -15,16 +14,6 @@ import {
   NO_PEOPLE,
   readPeople
 } from "./people.js";
-function inlineCommentText(node) {
-  if (node.localName === "t") return node.textContent ?? "";
-  if (node.localName === "tab") return "	";
-  if (node.localName === "br" || node.localName === "cr") return "\n";
-  return elementChildren(node).map(inlineCommentText).join("");
-}
-function commentText(el) {
-  const paragraphs = Array.from(el.getElementsByTagNameNS(W_NS, "p"));
-  return paragraphs.map(inlineCommentText).join("\n");
-}
 var NO_COMMENTS = {
   partPath: null,
   xml: null,
@@ -117,8 +106,6 @@ function readComments(parts, mainPartPath) {
         authorId: author === null ? null : commentAuthorId(people, author),
         initials: attributeByLocalName(el, "initials"),
         date: attributeByLocalName(el, "date"),
-        text: commentText(el),
-        xml: serializeXml(el),
         paraId,
         parentParaId: extension?.parentParaId ?? null,
         resolved: extension?.resolved ?? false,