web-doc 0.7.0 → 0.9.0

package/dist/workers/pdf-edit-worker.js

@@ -5850,13 +5850,16 @@ var pdfOperationSchemas = Object.freeze({
       },
       ["from", "to"]
     ),
+    // Exactly one of `rotation` (absolute) and `by` (a turn from the current
+    // angle); the handler reports a batch that gives both or neither.
     rotatePage: operation(
       "rotatePage",
       {
         pageIndex: { type: "integer", minimum: 0 },
-        rotation: { enum: [0, 90, 180, 270] }
+        rotation: { enum: [0, 90, 180, 270] },
+        by: { enum: [90, 180, 270] }
       },
-      ["pageIndex", "rotation"]
+      ["pageIndex"]
     ),
     insertShape: operation(
       "insertShape",
@@ -6897,7 +6900,7 @@ var FontLibrary = class {
     if (this.#coveringRegistered(request)) return void 0;
     if (this.#fallback && this.#covers(this.#fallback, request.text))
       return void 0;
-    const bad = firstNonWinAnsi(request.text) ?? firstUncovered(request.text, this);
+    const bad = this.#fallback ? this.#firstMissing(this.#fallback, request.text) : firstNonWinAnsi(request.text);
     return {
       path: "/text",
       code: "font-unavailable",
@@ -6955,6 +6958,10 @@ var FontLibrary = class {
     return candidates.sort((a, b) => score(b.font) - score(a.font))[0];
   }
   #covers(bytes, text) {
+    return this.#firstMissing(bytes, text) === void 0;
+  }
+  /** The first character of `text` the font has no glyph for; layout handles line breaks and spaces. */
+  #firstMissing(bytes, text) {
     let coverage = this.#coverage.get(bytes);
     if (!coverage) {
       coverage = parseCmap(bytes);
@@ -6963,9 +6970,9 @@ var FontLibrary = class {
     for (const character of text) {
       const code = character.codePointAt(0);
       if (code === 10 || code === 32) continue;
-      if (!coverage.has(code)) return false;
+      if (!coverage.has(code)) return character;
     }
-    return true;
+    return void 0;
   }
   #load(pdfium, document2, bytes) {
     let handles = this.#handles.get(document2);
@@ -7017,12 +7024,6 @@ function isBold(font) {
 function isItalic(font) {
   return font.style !== "normal";
 }
-function firstUncovered(text, library) {
-  void library;
-  for (const character of text)
-    if (character !== "\n" && character !== " ") return character;
-  return void 0;
-}
 function glyphOutlines(view, tables) {
   const head = tables.get("head");
   const maxp = tables.get("maxp");
@@ -8663,11 +8664,14 @@ var rotatePage = {
   validate(operation2, context, issue) {
     if (operation2.pageIndex >= context.pageCount)
       issue("/pageIndex", "unknown-target", `No page ${operation2.pageIndex}`);
+    if (operation2.rotation === void 0 === (operation2.by === void 0))
+      issue("", "one-of", "Give exactly one of `rotation` and `by`");
   },
   apply(operation2, context) {
     const { lib } = context.pdfium;
     context.withPage(operation2.pageIndex, (page) => {
-      lib.FPDFPage_SetRotation(page, operation2.rotation / 90);
+      const quarters = operation2.by !== void 0 ? (lib.FPDFPage_GetRotation(page) + operation2.by / 90) % 4 : operation2.rotation / 90;
+      lib.FPDFPage_SetRotation(page, quarters);
     });
     context.invalidatePage(operation2.pageIndex);
     return {
@@ -9513,7 +9517,10 @@ var defaultResourceLimits = Object.freeze({
   maxOperationMs: 3e4,
   maxEditOperations: 500,
   maxEditHistory: 200,
-  maxEditCheckpointBytes: 64 * 1024 * 1024
+  maxEditCheckpointBytes: 64 * 1024 * 1024,
+  maxOutlineNodes: 5e3,
+  maxDescribeChars: 2e5,
+  maxEditCheckpoints: 20
 });
 
 // src/edit/pdf/engine/document.ts
@@ -10040,8 +10047,11 @@ var PdfEditDocument = class {
       },
       // Ids name the state the batch leads to, the operation and the item,
       // so replaying the history reproduces them and an undone id is never
-      // handed out again.
-      newId: (pageIndex, suffix = "") => `${this.#pages[pageIndex].key}:n${stateId}.${operationIndex}.${created++}${suffix}`,
+      // handed out again. A file saved by an earlier session carries the ids
+      // that session numbered from 1 as well, so one already on the page
+      // gets the first free `~n` instead (ACTION-886); the saved objects are
+      // part of the base, so a replay meets them and picks the same id.
+      newId: (pageIndex, suffix = "") => `${this.#unusedId(pageIndex, `${this.#pages[pageIndex].key}:n${stateId}.${operationIndex}.${created++}`)}${suffix}`,
       withPage: (pageIndex, use) => this.#writePage(pageIndex, use),
       appendObjects: (pageIndex, records) => {
         const page = this.#pages[pageIndex];
@@ -10197,10 +10207,48 @@ var PdfEditDocument = class {
       }
     }
     record.objects = objects.map(
-      (object, index) => object.mark && stale.has(object.id) ? { id: `${record.key}:o${index}`, type: object.type } : object
+      (object, index) => object.mark && stale.has(object.id) ? {
+        id: `${record.key}:o${index}`,
+        type: object.type,
+        staleMarkId: object.id
+      } : object
     );
     return record.objects;
   }
+  /**
+   * `id`, or its first free `~n` when an object saved by an earlier session
+   * has it already. Only ids a session numbered (`<key>:n…`) can collide, so
+   * only those are gathered; the objects are read without the page's text.
+   */
+  #unusedId(pageIndex, id) {
+    const record = this.#pages[pageIndex];
+    const objects = record.objects ?? this.#loadObjects(pageIndex);
+    const numbered = `${record.key}:n`;
+    const taken = /* @__PURE__ */ new Set();
+    for (const object of objects) {
+      if (object.id.startsWith(numbered)) taken.add(object.id);
+      if (object.staleMarkId?.startsWith(numbered))
+        taken.add(object.staleMarkId);
+    }
+    if (!taken.has(id)) return id;
+    let suffix = 1;
+    while (taken.has(`${id}~${suffix}`)) suffix += 1;
+    return `${id}~${suffix}`;
+  }
+  /** A page's objects, loading the page alone when they are not known yet. */
+  #loadObjects(pageIndex) {
+    const { lib } = this.#pdfium;
+    const page = lib.FPDF_LoadPage(this.#document.handle, pageIndex);
+    if (!page)
+      throw new ViewerError("render-failed", "PDFium could not load the page", {
+        details: { pageIndex }
+      });
+    try {
+      return this.#objectsOf(pageIndex, page);
+    } finally {
+      lib.FPDF_ClosePage(page);
+    }
+  }
   /** Marks from other sessions keep their id only if it cannot collide with ours. */
   #ownMark(mark, pageKey) {
     return mark.id.startsWith(`${pageKey}:`) ? mark : { ...mark, id: `${pageKey}:${mark.id}` };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "web-doc",
-  "version": "0.7.0",
+  "version": "0.9.0",
   "description": "web-doc — embeddable browser-only document viewer with Rust/WASM adapters (a fork of Zrimo)",
   "keywords": [
     "document-viewer",