Skip to content

feat: add Automatic-Module-Name manifest entries#1947

Open
raphw wants to merge 1 commit into
microsoft:mainfrom
raphw:automatic-module-name
Open

feat: add Automatic-Module-Name manifest entries#1947
raphw wants to merge 1 commit into
microsoft:mainfrom
raphw:automatic-module-name

Conversation

@raphw

@raphw raphw commented Jul 5, 2026

Copy link
Copy Markdown

Give the published jars stable Java module names so they can be required from the module path instead of relying on filename-derived automatic names:

  • playwright: com.microsoft.playwright
  • driver: com.microsoft.playwright.driver
  • driver-bundle: com.microsoft.playwright.driver.bundle

The playwright test-jar carries its own name
(com.microsoft.playwright.test) so the main jar and the test-jar never claim the same module.

Give the published jars stable Java module names so they can be
required from the module path instead of relying on filename-derived
automatic names:

- playwright: com.microsoft.playwright
- driver: com.microsoft.playwright.driver
- driver-bundle: com.microsoft.playwright.driver.bundle

The playwright test-jar carries its own name
(com.microsoft.playwright.test) so the main jar and the test-jar
never claim the same module.
@raphw

raphw commented Jul 5, 2026

Copy link
Copy Markdown
Author

@microsoft-github-policy-service agree

@yury-s

yury-s commented Jul 9, 2026

Copy link
Copy Markdown
Member

What practical issue does it solve?

@raphw

raphw commented Jul 9, 2026

Copy link
Copy Markdown
Author

It avoids that Playwright depends on file names when you use Java's module system and want to use Playwright from this application.

Right now two modular applications might choose different names and become incompatible if they choose different file names and therefore module names. If Playwright makes this choice it provides a stable name for consumption.

Also, some tooling requires a module name at least on the manifest level.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants