Merge pull request #2740 from hardkoded/v19

V19 to Master

Author: kblok

.github/workflows/demo.yml

@@ -6,7 +6,6 @@ on:
       - master
       - release-*
   pull_request:
-    branches: [master]
     paths:
       - "**.cs"
       - "**.csproj"
@@ -35,7 +34,17 @@ jobs:
       working-directory: ./demos/PuppeteerSharpPdfDemo
       run: |
         dotnet restore PuppeteerSharpPdfDemo-Local.csproj
-    - name: Run Project
+    - name: Run on .NET
       working-directory: ./demos/PuppeteerSharpPdfDemo
       run: |
-          dotnet run --project PuppeteerSharpPdfDemo-Local.csproj auto-exit
+          dotnet run --project PuppeteerSharpPdfDemo-Local.csproj auto-exit -f net8.0
+    - name: Run with AOT
+      if: matrix.os == 'macos-latest'
+      working-directory: ./demos/PuppeteerSharpPdfDemo
+      run: |
+          dotnet run --project PuppeteerSharpPdfDemo-Local.csproj  -r osx-arm64  -c Release -f net8.0 auto-exit
+    - name: Run on .NET Framework
+      if: matrix.os == 'windows-2022'
+      working-directory: ./demos/PuppeteerSharpPdfDemo
+      run: |
+          dotnet run --project PuppeteerSharpPdfDemo-Local.csproj auto-exit -f net471

.github/workflows/dotnet.yml

@@ -8,7 +8,6 @@ on:
       - release-*
   merge_group:
   pull_request:
-    branches: [ master ]
     paths:
     - '**.yml'
     - '**.cs'
@@ -84,7 +83,7 @@ jobs:
         Get-ChildItem -Path cert:\CurrentUSer\my | where { $_.friendlyname -eq "Puppeteer" } | Export-Certificate -FilePath $env:GITHUB_WORKSPACE\lib\PuppeteerSharp.TestServer\testCert.cer
     - name: Check formatting
       if: ${{ matrix.os == 'ubuntu-latest' && matrix.browser == 'CHROME' && matrix.mode == 'headless' }}
-      run: dotnet format ./lib/PuppeteerSharp.sln --verify-no-changes
+      run: dotnet format ./lib/PuppeteerSharp.sln --verify-no-changes --exclude-diagnostics CA1865
     - name: Build
       working-directory: lib
       run: dotnet build PuppeteerSharp.sln

README.md

@@ -112,10 +112,10 @@ var result = await page.GetContentAsync();
 ```cs
 await using var page = await browser.NewPageAsync();
 var seven = await page.EvaluateExpressionAsync<int>("4 + 3");
-var someObject = await page.EvaluateFunctionAsync<dynamic>("(value) => ({a: value})", 5);
-Console.WriteLine(someObject.a);
+var someObject = await page.EvaluateFunctionAsync<JsonElement>("(value) => ({a: value})", 5);
+Console.WriteLine(someObject.GetProperty("a").GetString());
 ```
-<sup><a href='https://github.com/hardkoded/puppeteer-sharp/blob/master/lib/PuppeteerSharp.Tests/QuerySelectorTests/ElementHandleQuerySelectorEvalTests.cs#L16-L21' title='Snippet source file'>snippet source</a> | <a href='#snippet-Evaluate' title='Start of snippet'>anchor</a></sup>
+<sup><a href='https://github.com/hardkoded/puppeteer-sharp/blob/master/lib/PuppeteerSharp.Tests/QuerySelectorTests/ElementHandleQuerySelectorEvalTests.cs#L17-L22' title='Snippet source file'>snippet source</a> | <a href='#snippet-Evaluate' title='Start of snippet'>anchor</a></sup>
 <!-- endSnippet -->
 
 ### Wait For Selector

demos/PuppeteerSharpPdfDemo/Program.cs

@@ -1,6 +1,8 @@
 using System;
 using System.Linq;
 using System.IO;
+using System.Text.Json.Serialization;
+using System.Text.Json.Serialization.Metadata;
 using System.Threading.Tasks;
 using PuppeteerSharp;
 
@@ -10,6 +12,10 @@ class MainClass
     {
         public static async Task Main(string[] args)
         {
+#if NET8_0_OR_GREATER
+            Puppeteer.ExtraJsonSerializerContext = DemoJsonSerializationContext.Default;
+#endif
+
             var options = new LaunchOptions { Headless = true };
 
             Console.WriteLine("Downloading chromium");
@@ -28,10 +34,26 @@ public static async Task Main(string[] args)
 
             Console.WriteLine("Export completed");
 
+#if NET8_0_OR_GREATER
+            // AOT Test
+            var result = await page.EvaluateFunctionAsync<TestClass>("test => test", new TestClass { Name = "Dario"});
+            Console.WriteLine($"Name evaluated to {result.Name}");
+#endif
             if (!args.Any(arg => arg == "auto-exit"))
             {
                 Console.ReadLine();
             }
         }
     }
+
+#if NET8_0_OR_GREATER
+    public class TestClass
+    {
+        public string Name { get; set; }
+    }
+
+    [JsonSerializable(typeof(TestClass))]
+    public partial class DemoJsonSerializationContext : JsonSerializerContext
+    {}
+#endif
 }

demos/PuppeteerSharpPdfDemo/PuppeteerSharpPdfDemo-Local.csproj

@@ -1,9 +1,11 @@
 <Project Sdk="Microsoft.NET.Sdk">
-
   <PropertyGroup>
     <OutputType>Exe</OutputType>
-    <TargetFramework>net8.0</TargetFramework>
-    <LangVersion>8.0</LangVersion>
+    <TargetFrameworks>net8.0;net471</TargetFrameworks>
+    <LangVersion>12</LangVersion>
+  </PropertyGroup>
+  <PropertyGroup Condition="'$(TargetFramework)' == 'net8.0'">
+    <PublishAot>true</PublishAot>
   </PropertyGroup>
   <ItemGroup>
     <ProjectReference Include="..\..\lib\PuppeteerSharp\PuppeteerSharp.csproj" />

lib/.editorconfig

@@ -247,6 +247,18 @@ dotnet_diagnostic.CA1054.severity = none
 dotnet_diagnostic.CA1056.severity = none
 # Avoid empty interfaces
 dotnet_diagnostic.CA1040.severity = none
+# Use 'ArgumentNullException.ThrowIfNull' instead of explicitly throwing a new exception instance
+# This is not netstandard compatible
+dotnet_diagnostic.CA1510.severity = none
+# Use 'string.StartsWith(char)' instead of 'string.StartsWith(string)' when you have a string with a single char
+# This is not .NET 8 compatible
+dotnet_diagnostic.CA1865.severity = none
+# Use 'ObjectDisposedException.ThrowIf' instead of explicitly throwing a new exception instance
+# This is not netstandard compatible
+dotnet_diagnostic.CA1513.severity = none
+# Use Change the 'WriteAsync' method call to use the 'Stream.WriteAsync(ReadOnlyMemory<byte>, CancellationToken)' overload
+# This is not netstandard compatible
+dotnet_diagnostic.CA1835.severity = none
 # Use ConfigureAwait
 dotnet_diagnostic.CA2007.severity = error
 # CA1711: Identifiers should not have incorrect suffix
@@ -264,6 +276,8 @@ dotnet_diagnostic.CA1815.severity = suggestion
 # CA1848: Use the LoggerMessage delegates
 # TODO: REMOVE
 dotnet_diagnostic.CA1848.severity = none
+# internal sealed
+dotnet_diagnostic.CA1852.severity = error
 # CA1859: Use concrete types when possible for improved performance
 # It collides with https://www.jetbrains.com/help/rider/ReturnTypeCanBeEnumerable.Local.html
 dotnet_diagnostic.CA1859.severity = none
@@ -303,7 +317,7 @@ dotnet_diagnostic.CA1001.severity = error
 dotnet_diagnostic.CA1304.severity = error
 # CA1305: String.Format with culture
 dotnet_diagnostic.CA1305.severity = error
-# CA1304: CA1308: Normalize strings to uppercase
+# CA1308: Normalize strings to uppercase
 dotnet_diagnostic.CA1308.severity = none
 # CA1721: The property name 'DefaultArgs' is confusing given the existence of method 'GetDefaultArgs'. Rename or remove one of these members.
 dotnet_diagnostic.CA1721.severity = error

lib/Common/CommonProps.props

@@ -11,6 +11,6 @@
     </None>
   </ItemGroup>
   <Target Name="PreBuild" BeforeTargets="PreBuildEvent">
-    <Exec Command="dotnet format $(ProjectPath)" />
+    <Exec Command="dotnet format $(ProjectPath) --exclude-diagnostics CA1865" />
   </Target>
 </Project>

lib/PuppeteerSharp.DevicesFetcher/Device.cs

@@ -1,14 +1,14 @@
-using Newtonsoft.Json;
+using System.Text.Json.Serialization;
 
 namespace PuppeteerSharp.DevicesFetcher
 {
     public class Device
     {
-        [JsonProperty("userAgent")]
+        [JsonPropertyName("userAgent")]
         public string UserAgent { get; set; }
-        [JsonProperty("name")]
+        [JsonPropertyName("name")]
         public string Name { get; set; }
-        [JsonProperty("viewport")]
+        [JsonPropertyName("viewport")]
         public ViewPort Viewport { get; set; }
     }
 }

lib/PuppeteerSharp.DevicesFetcher/Program.cs

@@ -4,8 +4,9 @@
 using System.Linq;
 using System.Net.Http;
 using System.Text;
+using System.Text.Json;
 using System.Threading.Tasks;
-using Newtonsoft.Json;
+
 namespace PuppeteerSharp.DevicesFetcher
 {
     static class Program
@@ -34,7 +35,7 @@ static async Task Main(string[] args)
             Device[] devices;
             try
             {
-                devices = JsonConvert.DeserializeObject<Device[]>(text);
+                devices = JsonSerializer.Deserialize<Device[]>(text);
             }
             catch (Exception ex)
             {
@@ -70,14 +71,6 @@ public static class DeviceDescriptors
             new Lazy<IReadOnlyDictionary<DeviceDescriptorName, DeviceDescriptor>>(() => new ReadOnlyDictionary<DeviceDescriptorName, DeviceDescriptor>(Devices));
 
         internal static IReadOnlyDictionary<DeviceDescriptorName, DeviceDescriptor> ToReadOnly() => _readOnlyDevices.Value;
-
-        /// <summary>
-        /// Get the specified device description.
-        /// </summary>
-        /// <returns>The device descriptor.</returns>
-        /// <param name=""name"">Device Name.</param>
-        [Obsolete(""Use Puppeteer.Devices instead"")]
-        public static DeviceDescriptor Get(DeviceDescriptorName name) => Devices[name];
     }
 }";
 

lib/PuppeteerSharp.DevicesFetcher/PuppeteerSharp.DevicesFetcher.csproj

@@ -5,10 +5,6 @@
     <TargetFramework>net8.0</TargetFramework>
     <LangVersion>12</LangVersion>
   </PropertyGroup>
-  <ItemGroup>
-    <PackageReference Include="Newtonsoft.Json" Version="13.0.1" />
-  </ItemGroup>
-
   <ItemGroup>
     <ProjectReference Include="..\PuppeteerSharp\PuppeteerSharp.csproj" />
   </ItemGroup>