i merged a pr which broke the build, this updates the build to work

move all arguments to add page to a setting object so it can be extended
in future in a non-breaking api change

Author: EliotJones

src/UglyToad.PdfPig.Tests/Writer/PdfDocumentBuilderTests.cs

@@ -139,7 +139,10 @@ public void CanFastAddPageAndStripAllAnnots()
             using (var existing = PdfDocument.Open(contents, ParsingOptions.LenientParsingOff))
             using (var output = new PdfDocumentBuilder())
             {
-                output.AddPage(existing, 1, false);
+                output.AddPage(existing, 1, new PdfDocumentBuilder.AddPageOptions
+                {
+                    KeepAnnotations = false
+                });
                 results = output.Build();
                 var pg = existing.GetPage(1);
                 var annots = pg.ExperimentalAccess.GetAnnotations().ToList();

src/UglyToad.PdfPig/Writer/PdfDocumentBuilder.cs

@@ -307,21 +307,21 @@ private sealed class PageInfo(DictionaryToken page, IReadOnlyList<DictionaryToke
         /// </summary>
         /// <param name="document">Source document.</param>
         /// <param name="pageNumber">Page to copy.</param>
-        /// <param name="keepAnnotations">Flag to set whether annotation of page should be kept</param>
+        /// <param name="options">Control how copying for the page occurs.</param>
         /// <returns>A builder for editing the page.</returns>
-        public PdfPageBuilder AddPage(PdfDocument document, int pageNumber, bool keepAnnotations = true)
+        public PdfPageBuilder AddPage(PdfDocument document, int pageNumber)
         {
-            return AddPage(document, pageNumber, null);
+            return AddPage(document, pageNumber, new AddPageOptions());
         }
 
         /// <summary>
         /// Add a new page with the specified size, this page will be included in the output when <see cref="Build"/> is called.
         /// </summary>
         /// <param name="document">Source document.</param>
         /// <param name="pageNumber">Page to copy.</param>
-        /// <param name="copyLink">If set, links are copied based on the result of the delegate.</param>
+        /// <param name="options">Control how copying for the page occurs.</param>
         /// <returns>A builder for editing the page.</returns>
-        public PdfPageBuilder AddPage(PdfDocument document, int pageNumber, Func<PdfAction, PdfAction?>? copyLink)
+        public PdfPageBuilder AddPage(PdfDocument document, int pageNumber, AddPageOptions options)
         {
             if (!existingCopies.TryGetValue(document.Structure.TokenScanner, out var refs))
             {
@@ -454,7 +454,7 @@ public PdfPageBuilder AddPage(PdfDocument document, int pageNumber, Func<PdfActi
 
                 if (kvp.Key == NameToken.Annots)
                 {
-                    if (!keepAnnotations)
+                    if (!options.KeepAnnotations)
                     {
                         continue;
                     }
@@ -490,7 +490,7 @@ public PdfPageBuilder AddPage(PdfDocument document, int pageNumber, Func<PdfActi
 
                         if (tk.TryGet(NameToken.Subtype, out var st) && st is NameToken nm && nm == NameToken.Link)
                         {
-                            if (copyLink is null)
+                            if (options.CopyLinkFunc is null)
                             {
                                 // ignore link if don't know how to copy
                                 continue;
@@ -503,7 +503,7 @@ public PdfPageBuilder AddPage(PdfDocument document, int pageNumber, Func<PdfActi
                                 continue;
                             }
 
-                            var copiedLink = copyLink(link);
+                            var copiedLink = options.CopyLinkFunc(link);
                             if (copiedLink is null)
                             {
                                 // ignore if caller wants to skip the link
@@ -1248,5 +1248,21 @@ public void Dispose()
 
             context.Dispose();
         }
+
+        /// <summary>
+        /// Options for how a page should be copied to the current builder when using AddPage.
+        /// </summary>
+        public class AddPageOptions
+        {
+            /// <summary>
+            /// Whether to preserve annotation objects in the copied page.
+            /// </summary>
+            public bool KeepAnnotations { get; set; } = true;
+
+            /// <summary>
+            /// Intercept how actions are copied from the source to destination page.
+            /// </summary>
+            public Func<PdfAction, PdfAction?>? CopyLinkFunc { get; set; }
+        }
     }
 }