From 71f371b10aeedde4d27bee31b62a0098f1d8e02a Mon Sep 17 00:00:00 2001
From: Yury Semikhatsky <yurys@chromium.org>
Date: Mon, 14 Sep 2026 11:00:57 -0700
Subject: [PATCH 1/2] fix(build): restore javadoc jar contents

Since the Automatic-Module-Name manifest entries were added (#1947),
maven-javadoc-plugin runs javadoc in module mode with a broken
--patch-module path and the playwright javadoc jar ends up empty.
Use legacy (classpath) mode.
---
 pom.xml | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/pom.xml b/pom.xml
index 25aeef8a7..cb1235ac3 100644
--- a/pom.xml
+++ b/pom.xml
@@ -162,6 +162,9 @@
             <failOnError>false</failOnError>
             <!-- passing 8 causes an assertion in javadoc tool, see https://github.com/microsoft/playwright-java/issues/1533 -->
             <source>11</source>
+            <!-- Automatic-Module-Name in the jar manifest makes the plugin run javadoc in module mode
+                 with a broken patch-module path, which produces an empty javadoc jar. -->
+            <legacyMode>true</legacyMode>
           </configuration>
         </plugin>
         <plugin>

From 60dc95d7eab6662f68ae4d10bcaca1cbeff6bc33 Mon Sep 17 00:00:00 2001
From: Yury Semikhatsky <yurys@chromium.org>
Date: Mon, 14 Sep 2026 11:12:12 -0700
Subject: [PATCH 2/2] fix(build): set javadoc sourcepath instead of legacy mode

The empty javadoc jar comes from the plugin computing the
--patch-module path as null when subpackages is set without an explicit
sourcepath. Setting sourcepath keeps the classic javadoc layout without
disabling module path support globally.
---
 playwright/pom.xml | 3 +++
 pom.xml            | 3 ---
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/playwright/pom.xml b/playwright/pom.xml
index 05c3a6a5b..e97c92a24 100644
--- a/playwright/pom.xml
+++ b/playwright/pom.xml
@@ -29,6 +29,9 @@
         <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-javadoc-plugin</artifactId>
         <configuration combine.self="append">
+          <!-- Without an explicit sourcepath, subpackages combined with the Automatic-Module-Name manifest entry
+               makes the plugin pass "patch-module com.microsoft.playwright=null" and the javadoc jar ends up empty. -->
+          <sourcepath>${project.build.sourceDirectory}</sourcepath>
           <subpackages>com.microsoft.playwright</subpackages>
           <excludePackageNames>com.microsoft.playwright.impl</excludePackageNames>
         </configuration>
diff --git a/pom.xml b/pom.xml
index cb1235ac3..25aeef8a7 100644
--- a/pom.xml
+++ b/pom.xml
@@ -162,9 +162,6 @@
             <failOnError>false</failOnError>
             <!-- passing 8 causes an assertion in javadoc tool, see https://github.com/microsoft/playwright-java/issues/1533 -->
             <source>11</source>
-            <!-- Automatic-Module-Name in the jar manifest makes the plugin run javadoc in module mode
-                 with a broken patch-module path, which produces an empty javadoc jar. -->
-            <legacyMode>true</legacyMode>
           </configuration>
         </plugin>
         <plugin>