build(selenium-devtools-py): make the adapter fit to publish, and keep it that way

[vishnuv688]

What & why

Type of change

• Bugfix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Polish (an improvement to an existing feature)
• Breaking change (existing behavior changes for users)
• Documentation
• Internal (build, CI, dependencies, tooling)

Packages touched

• shared (types and contracts)
• core (framework-agnostic capture/reporting)
• elements (published element/snapshot API — @wdio/elements)
• service (WebdriverIO adapter)
• nightwatch-devtools (Nightwatch adapter)
• selenium-devtools (Selenium adapter)
• selenium-devtools-py (Selenium Python adapter)
• backend (server)
• app (UI)
• script (page-injected runtime)
• trace (Trace mode)

Notes for reviewers

Screenshots / recordings

[greptile-apps]

Confidence Score: 4/5

[High risk] Restructures Python package build and release infrastructure.

The PR is not yet safe to merge because a concurrent main update can make the Python release tag point to source absent from the published artifact.

Findings

1.  Tag can misidentify published source ▶
2.  First release skips its tag. ▶
3.  Release workflow changes skip CI. ▶

Summary

The PR adds packaging checks and a fragment-driven release process for the Python adapter. Since the previous review, it also tightens the index preflight, gates changelog-only changes after the first release, and retries release-commit pushes when main advances.

• The first release now receives a tag even when there is nothing to commit.
• CI now includes changes to the release workflow.
• A rebased release commit can still cause the version tag to identify source that was not published.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart LR
  A[Checkout main and apply fragments] --> B[Build and publish artifact]
  B --> C[Commit release state]
  C --> D{Push accepted?}
  D -- Yes --> E[Tag HEAD]
  D -- No --> F[Fetch and rebase onto newer main]
  F --> D
  F -. Can add source absent from artifact .-> E

Loading

_{Reviews (2) · Last reviewed commit: "fix(ci): close the gaps review found in ..."}