@signiphi/page-assembly 0.2.0-beta.12 → 0.2.0-beta.14

package/dist/index.d.mts

@@ -309,24 +309,78 @@ interface ExtractedField {
     signer?: string;
 }
 /**
- * SigniphiMetadata structure stored in PDF Info dictionary
+ * One field's metadata entry, as stored in the block.
+ */
+interface SigniphiFieldMetadata {
+    fieldId?: string;
+    label?: string;
+    signer?: string;
+    placeholder?: string;
+    required?: boolean;
+    options?: string[];
+    acknowledgements?: Array<{
+        id: string;
+        title: string;
+        description: string;
+    }>;
+    /**
+     * TEXT_LABEL only: this entry's text is already in the page stream, so the
+     * sign-time pass must not redraw it. See `mergeSourceMetadata`, which sets it
+     * when a source that had baked its labels is merged with one that deferred.
+     */
+    textLabelBaked?: boolean;
+    /**
+     * Persisted geometry, written by `document-prepare` for RADIO and TEXT_LABEL
+     * fields only. Declared here for ONE reason: its `page` is the SOURCE
+     * document's page number, and the merge has to rewrite it to the combined
+     * one. Nothing in page-assembly reads x/y/width/height.
+     */
+    position?: SigniphiFieldPosition;
+    /** Radio-only, per-option. Re-paged alongside {@link SigniphiFieldMetadata.position}. */
+    radioPositions?: SigniphiFieldPosition[];
+    /** Radio-only bound list (options ↔ circle positions); its pages are re-paged too. */
+    radioOptions?: Array<{
+        label: string;
+        exportValue: string;
+        position: SigniphiFieldPosition;
+    }>;
+}
+/**
+ * A persisted field rect, as `document-prepare` writes it. `page` is 1-indexed
+ * and scoped to the document it was written in — which is exactly why the merge
+ * cannot carry it across unchanged.
+ */
+interface SigniphiFieldPosition {
+    x: number;
+    y: number;
+    width: number;
+    height: number;
+    page: number;
+}
+/**
+ * SigniphiMetadata structure stored in PDF Info dictionary.
+ *
+ * Exported because `getSigniphiMetadata` / `setSigniphiMetadata` are public
+ * (`utils/index.ts`) and a caller cannot name their argument otherwise — which
+ * is what pushed the call sites here into `as never` casts that would have
+ * silenced any future shape mismatch.
  */
 interface SigniphiMetadata {
     version: string;
-    fields: Record<string, {
-        fieldId?: string;
-        label?: string;
-        signer?: string;
-        placeholder?: string;
-        required?: boolean;
-        options?: string[];
-        acknowledgements?: Array<{
-            id: string;
-            title: string;
-            description: string;
-        }>;
-    }>;
+    fields: Record<string, SigniphiFieldMetadata>;
     fieldIdIndex?: Record<string, string>;
+    /**
+     * Document-level: radio group and option text is already drawn into the page
+     * stream, so the sign-time pass must not redraw it. Absent on legacy PDFs,
+     * which pdf-signer reads as "draw".
+     */
+    radioLabelsBaked?: boolean;
+    /**
+     * Document-level: `false` means TEXT_LABEL text was deliberately NOT baked
+     * (multi-signer), so the sign-time pass redraws it per signer. Absent/`true`
+     * means it is already in the page stream.
+     */
+    textLabelsBaked?: boolean;
 }
 /**
  * Extract SigniphiMetadata from a PDF document

package/dist/index.d.ts

@@ -309,24 +309,78 @@ interface ExtractedField {
     signer?: string;
 }
 /**
- * SigniphiMetadata structure stored in PDF Info dictionary
+ * One field's metadata entry, as stored in the block.
+ */
+interface SigniphiFieldMetadata {
+    fieldId?: string;
+    label?: string;
+    signer?: string;
+    placeholder?: string;
+    required?: boolean;
+    options?: string[];
+    acknowledgements?: Array<{
+        id: string;
+        title: string;
+        description: string;
+    }>;
+    /**
+     * TEXT_LABEL only: this entry's text is already in the page stream, so the
+     * sign-time pass must not redraw it. See `mergeSourceMetadata`, which sets it
+     * when a source that had baked its labels is merged with one that deferred.
+     */
+    textLabelBaked?: boolean;
+    /**
+     * Persisted geometry, written by `document-prepare` for RADIO and TEXT_LABEL
+     * fields only. Declared here for ONE reason: its `page` is the SOURCE
+     * document's page number, and the merge has to rewrite it to the combined
+     * one. Nothing in page-assembly reads x/y/width/height.
+     */
+    position?: SigniphiFieldPosition;
+    /** Radio-only, per-option. Re-paged alongside {@link SigniphiFieldMetadata.position}. */
+    radioPositions?: SigniphiFieldPosition[];
+    /** Radio-only bound list (options ↔ circle positions); its pages are re-paged too. */
+    radioOptions?: Array<{
+        label: string;
+        exportValue: string;
+        position: SigniphiFieldPosition;
+    }>;
+}
+/**
+ * A persisted field rect, as `document-prepare` writes it. `page` is 1-indexed
+ * and scoped to the document it was written in — which is exactly why the merge
+ * cannot carry it across unchanged.
+ */
+interface SigniphiFieldPosition {
+    x: number;
+    y: number;
+    width: number;
+    height: number;
+    page: number;
+}
+/**
+ * SigniphiMetadata structure stored in PDF Info dictionary.
+ *
+ * Exported because `getSigniphiMetadata` / `setSigniphiMetadata` are public
+ * (`utils/index.ts`) and a caller cannot name their argument otherwise — which
+ * is what pushed the call sites here into `as never` casts that would have
+ * silenced any future shape mismatch.
  */
 interface SigniphiMetadata {
     version: string;
-    fields: Record<string, {
-        fieldId?: string;
-        label?: string;
-        signer?: string;
-        placeholder?: string;
-        required?: boolean;
-        options?: string[];
-        acknowledgements?: Array<{
-            id: string;
-            title: string;
-            description: string;
-        }>;
-    }>;
+    fields: Record<string, SigniphiFieldMetadata>;
     fieldIdIndex?: Record<string, string>;
+    /**
+     * Document-level: radio group and option text is already drawn into the page
+     * stream, so the sign-time pass must not redraw it. Absent on legacy PDFs,
+     * which pdf-signer reads as "draw".
+     */
+    radioLabelsBaked?: boolean;
+    /**
+     * Document-level: `false` means TEXT_LABEL text was deliberately NOT baked
+     * (multi-signer), so the sign-time pass redraws it per signer. Absent/`true`
+     * means it is already in the page stream.
+     */
+    textLabelsBaked?: boolean;
 }
 /**
  * Extract SigniphiMetadata from a PDF document

package/dist/index.js

@@ -1399,6 +1399,51 @@ function downloadPdf(pdfBytes, filename) {
   a.click();
   URL.revokeObjectURL(url);
 }
+function toCombinedPage(entry, page) {
+  if (!entry.position && !entry.radioPositions && !entry.radioOptions) return entry;
+  const carried = { ...entry };
+  if (entry.position) carried.position = { ...entry.position, page };
+  if (entry.radioPositions) carried.radioPositions = entry.radioPositions.map((p) => ({ ...p, page }));
+  if (entry.radioOptions) {
+    carried.radioOptions = entry.radioOptions.map((o) => ({ ...o, position: { ...o.position, page } }));
+  }
+  return carried;
+}
+function mergeSourceMetadata(sources) {
+  const merged = { version: "1.0", fields: {}, fieldIdIndex: {} };
+  let radioLabelsBaked;
+  let textLabelsBaked;
+  for (const source of sources) {
+    const meta = source.metadata;
+    if (!meta || source.fields.length === 0) continue;
+    if (meta.radioLabelsBaked === false) radioLabelsBaked = false;
+    else if (meta.radioLabelsBaked === true && radioLabelsBaked === void 0) radioLabelsBaked = true;
+    if (meta.textLabelsBaked === false) textLabelsBaked = false;
+    else if (meta.textLabelsBaked === true && textLabelsBaked === void 0) textLabelsBaked = true;
+  }
+  for (const source of sources) {
+    const meta = source.metadata;
+    if (!meta) continue;
+    const sourceTextLabelsBaked = meta.textLabelsBaked !== false;
+    for (const { name, page } of source.fields) {
+      const key = `${name}_p${page}`;
+      const entry = meta.fields?.[name];
+      if (entry) {
+        const carried = toCombinedPage(entry, page);
+        merged.fields[key] = textLabelsBaked === false && sourceTextLabelsBaked ? { ...carried, textLabelBaked: true } : carried;
+      }
+      const fieldId = meta.fieldIdIndex?.[name];
+      if (fieldId) merged.fieldIdIndex[key] = fieldId;
+    }
+  }
+  if (radioLabelsBaked !== void 0) merged.radioLabelsBaked = radioLabelsBaked;
+  if (textLabelsBaked !== void 0) merged.textLabelsBaked = textLabelsBaked;
+  const hasEntries = Object.keys(merged.fields).length > 0;
+  const hasMarkers = radioLabelsBaked !== void 0 || textLabelsBaked !== void 0;
+  if (!hasEntries && !hasMarkers) return null;
+  if (Object.keys(merged.fieldIdIndex).length === 0) delete merged.fieldIdIndex;
+  return merged;
+}
 function getSigniphiMetadata(pdfDoc) {
   try {
     const infoRef = pdfDoc.context.trailerInfo.Info;
@@ -1414,12 +1459,17 @@ function getSigniphiMetadata(pdfDoc) {
     if (!metadataObj) {
       return null;
     }
-    const metadataStr = metadataObj.toString();
-    let jsonStr = metadataStr;
-    if (metadataStr.startsWith("(") && metadataStr.endsWith(")")) {
-      jsonStr = metadataStr.slice(1, -1);
-    } else if (metadataStr.startsWith("<") && metadataStr.endsWith(">")) {
-      jsonStr = metadataStr.slice(1, -1);
+    let jsonStr;
+    if (typeof metadataObj.decodeText === "function") {
+      jsonStr = metadataObj.decodeText();
+    } else {
+      const metadataStr = metadataObj.toString();
+      jsonStr = metadataStr;
+      if (metadataStr.startsWith("(") && metadataStr.endsWith(")")) {
+        jsonStr = metadataStr.slice(1, -1);
+      } else if (metadataStr.startsWith("<") && metadataStr.endsWith(">")) {
+        jsonStr = metadataStr.slice(1, -1);
+      }
     }
     const metadata = JSON.parse(jsonStr);
     return metadata;
@@ -1612,7 +1662,7 @@ function setSigniphiMetadata(pdfDoc, metadata) {
     }
     const infoDict = infoObj;
     const metadataString = JSON.stringify(metadata);
-    infoDict.set(pdfLib.PDFName.of("SigniphiMetadata"), pdfLib.PDFString.of(metadataString));
+    infoDict.set(pdfLib.PDFName.of("SigniphiMetadata"), pdfLib.PDFHexString.fromText(metadataString));
   } catch (error) {
     console.error("Failed to set Signiphi metadata:", error);
     throw error;
@@ -1972,6 +2022,7 @@ function mapFieldPositionsAfterAssembly(fields, pageMapping) {
     return pageMapping.has(mappingKey) || pageMapping.size === 0;
   });
 }
+var VISIBLE_PAGE_ROWS = 2;
 var isMultiSelectModifier = (event) => {
   return event.ctrlKey || event.metaKey;
 };
@@ -2298,21 +2349,48 @@ var ReorderablePageGrid = ({
     newPages.splice(hoverIndex, 0, draggedPage);
     onReorderPages(file.id, newPages);
   }, [file.id, file.pages, onReorderPages]);
-  return /* @__PURE__ */ jsxRuntime.jsx("div", { className: "grid grid-cols-2 sm:grid-cols-3 md:grid-cols-4 lg:grid-cols-6 gap-4", children: file.pages.map((page, index) => /* @__PURE__ */ jsxRuntime.jsx(
-    DraggablePageThumbnail,
+  const gridRef = React7.useRef(null);
+  const [maxHeight, setMaxHeight] = React7.useState(void 0);
+  React7.useEffect(() => {
+    const grid = gridRef.current;
+    if (!grid || typeof ResizeObserver === "undefined") return;
+    const measure = () => {
+      const firstThumbnail = grid.firstElementChild;
+      if (!firstThumbnail) return;
+      const rowHeight = firstThumbnail.getBoundingClientRect().height;
+      if (rowHeight <= 0) return;
+      const rowGap = parseFloat(getComputedStyle(grid).rowGap) || 0;
+      setMaxHeight(Math.ceil(rowHeight * VISIBLE_PAGE_ROWS + rowGap * (VISIBLE_PAGE_ROWS - 1)));
+    };
+    measure();
+    const observer = new ResizeObserver(measure);
+    observer.observe(grid);
+    if (grid.firstElementChild) observer.observe(grid.firstElementChild);
+    return () => observer.disconnect();
+  }, [file.pages]);
+  return /* @__PURE__ */ jsxRuntime.jsx(
+    "div",
     {
-      page,
-      fileId: file.id,
-      index,
-      isSelected: selectedPageIds.includes(page.id),
-      onSingleSelect: onSingleSelectPage,
-      onToggleSelect: onTogglePageSelect,
-      onRangeSelect: onRangeSelectPage,
-      movePageInFile,
-      isDraggingPageRef
-    },
-    page.id
-  )) });
+      ref: gridRef,
+      className: "grid grid-cols-2 sm:grid-cols-3 md:grid-cols-4 lg:grid-cols-6 gap-4 overflow-y-auto overflow-x-hidden",
+      style: { maxHeight },
+      children: file.pages.map((page, index) => /* @__PURE__ */ jsxRuntime.jsx(
+        DraggablePageThumbnail,
+        {
+          page,
+          fileId: file.id,
+          index,
+          isSelected: selectedPageIds.includes(page.id),
+          onSingleSelect: onSingleSelectPage,
+          onToggleSelect: onTogglePageSelect,
+          onRangeSelect: onRangeSelectPage,
+          movePageInFile,
+          isDraggingPageRef
+        },
+        page.id
+      ))
+    }
+  );
 };
 var FileDropZone = ({
   targetIndex,
@@ -2897,7 +2975,7 @@ var PageAssembler = React7.forwardRef(
         setIsLoading(true);
         setProcessingStatus("Assembling PDF...");
         try {
-          const { PDFDocument: PDFDocument5, PDFName: PDFName4, PDFString: PDFString4 } = await import('pdf-lib');
+          const { PDFDocument: PDFDocument5, PDFName: PDFName4 } = await import('pdf-lib');
           const loadedPdfs = /* @__PURE__ */ new Map();
           const fileMap = /* @__PURE__ */ new Map();
           const pdfFiles = state.files.filter((f) => f.type === "pdf" && f.originalBytes?.length > 0);
@@ -2976,11 +3054,7 @@ var PageAssembler = React7.forwardRef(
           }
           console.log("[PageAssembler] Starting multi-PDF assembly with field preservation");
           const allExtractedFields = [];
-          const mergedMetadata = {
-            version: "1.0",
-            fields: {},
-            fieldIdIndex: {}
-          };
+          const metadataSources = [];
           const filePageUsage = /* @__PURE__ */ new Map();
           for (const file of state.files) {
             for (const page of file.pages) {
@@ -3005,28 +3079,12 @@ var PageAssembler = React7.forwardRef(
           }
           console.log("[PageAssembler] Page mapping:", Object.fromEntries(pageIdToNewPageNumber));
           const extractAndMapFields = async (fileId, fileName, pdfBytes2, sourcePdf) => {
-            let sourceMetadata = null;
-            if (sourcePdf) {
-              try {
-                const sourceInfoRef = sourcePdf.context.trailerInfo.Info;
-                if (sourceInfoRef) {
-                  const sourceInfoDict = sourcePdf.context.lookup(sourceInfoRef);
-                  if (sourceInfoDict && typeof sourceInfoDict.get === "function") {
-                    const metadataObj = sourceInfoDict.get(PDFName4.of("SigniphiMetadata"));
-                    if (metadataObj) {
-                      const metadataStr = metadataObj.toString();
-                      let jsonStr = metadataStr;
-                      if (metadataStr.startsWith("(") && metadataStr.endsWith(")")) {
-                        jsonStr = metadataStr.slice(1, -1);
-                      }
-                      sourceMetadata = JSON.parse(jsonStr);
-                    }
-                  }
-                }
-              } catch (metadataError) {
-                console.warn(`Could not read metadata from PDF ${fileName}:`, metadataError);
-              }
+            const sourceMetadata = sourcePdf ? getSigniphiMetadata(sourcePdf) : null;
+            if (sourcePdf && !sourceMetadata) {
+              console.warn(`[PageAssembler] No readable SigniphiMetadata in "${fileName}"`);
             }
+            const placements = [];
+            metadataSources.push({ metadata: sourceMetadata, fields: placements });
             console.log(`[PageAssembler] Extracting fields from "${fileName}"`);
             const extractedFields = await extractFieldsFromPdf(pdfBytes2);
             console.log(`[PageAssembler] Found ${extractedFields.length} fields in "${fileName}"`);
@@ -3060,13 +3118,7 @@ var PageAssembler = React7.forwardRef(
                   };
                   allExtractedFields.push(trackedField);
                   console.log(`[PageAssembler] Mapped field "${field.name}" from page ${originalPage} to page ${newPageNumber}`);
-                  const metadataKey = `${field.name}_p${newPageNumber}`;
-                  if (sourceMetadata?.fields?.[field.name]) {
-                    mergedMetadata.fields[metadataKey] = sourceMetadata.fields[field.name];
-                  }
-                  if (sourceMetadata?.fieldIdIndex?.[field.name]) {
-                    mergedMetadata.fieldIdIndex[metadataKey] = sourceMetadata.fieldIdIndex[field.name];
-                  }
+                  placements.push({ name: field.name, page: newPageNumber });
                 }
               }
             }
@@ -3144,21 +3196,15 @@ var PageAssembler = React7.forwardRef(
               }
             }
           }
-          if (Object.keys(mergedMetadata.fields).length > 0) {
+          const mergedMetadata = mergeSourceMetadata(metadataSources);
+          if (mergedMetadata) {
             try {
-              const newInfoRef = combinedDoc.context.trailerInfo.Info;
-              if (newInfoRef) {
-                const newInfoDict = combinedDoc.context.lookup(newInfoRef);
-                if (newInfoDict && typeof newInfoDict.set === "function") {
-                  newInfoDict.set(
-                    PDFName4.of("SigniphiMetadata"),
-                    PDFString4.of(JSON.stringify(mergedMetadata))
-                  );
-                }
-              }
+              setSigniphiMetadata(combinedDoc, mergedMetadata);
             } catch (metadataError) {
               console.warn("Could not write merged metadata:", metadataError);
             }
+          } else {
+            console.warn("[PageAssembler] No source metadata survived the merge; the assembled document will read as unprepared downstream");
           }
           let pdfBytes = await combinedDoc.save();
           if (allExtractedFields.length > 0) {