[Changed] bump playwright-core to v1.26.0

assets/package.json

@@ -1,6 +1,6 @@
 {
   "dependencies": {
     "electron": "24.0.0",
-    "playwright": "1.25.0"
+    "playwright": "1.26.0"
   }
 }

lib/playwright/browser_context.ex

@@ -615,10 +615,16 @@ defmodule Playwright.BrowserContext do
   #         break
   #
   # ...hoping for a test to drive that out.
-  defp on_route(context, %{params: %{request: request} = params} = _event) do
+
+  # NOTE(20240525):
+  # Do not love this; See Page.on_route/2 (which is an exact copy of this) for why.
+  defp on_route(context, %{params: %{route: %{request: request} = route} = _params} = _event) do
     Enum.reduce_while(context.routes, [], fn handler, acc ->
+      catalog = Playwright.Channel.Session.catalog(context.session)
+      request = Playwright.Channel.Catalog.get(catalog, request.guid)
+
       if Helpers.RouteHandler.matches(handler, request.url) do
-        Helpers.RouteHandler.handle(handler, params)
+        Helpers.RouteHandler.handle(handler, %{request: request, route: route})
         # break
         {:halt, acc}
       else

lib/playwright/page.ex

@@ -677,26 +677,21 @@ defmodule Playwright.Page do
   # private
   # ---------------------------------------------------------------------------
 
-  defp on_route(page, %{params: %{request: request} = params} = _event) do
+  # Do not love this.
+  # It's good enough for now (to deal with v1.26.0 changes). However, it feels
+  # dirty for API resource implementations to be reaching into Catalog.
+  defp on_route(page, %{params: %{route: %{request: request} = route} = _params} = _event) do
     Enum.reduce_while(page.routes, [], fn handler, acc ->
+      catalog = Playwright.Channel.Session.catalog(page.session)
+      request = Playwright.Channel.Catalog.get(catalog, request.guid)
+
       if Helpers.RouteHandler.matches(handler, request.url) do
-        Helpers.RouteHandler.handle(handler, params)
+        Helpers.RouteHandler.handle(handler, %{request: request, route: route})
         # break
         {:halt, acc}
       else
         {:cont, [handler | acc]}
       end
     end)
-
-    # task =
-    #   Task.async(fn ->
-    #     IO.puts("fetching context for page...")
-
-    #     context(page)
-    #     |> IO.inspect(label: "task context")
-    #     |> BrowserContext.on_route(event)
-    #   end)
-
-    # Task.await(task)
   end
 end