Skip to content

fix: resolve caller data paths in Playwright UI - #122

Merged
longlho merged 1 commit into
mainfrom
codex/ui-location-data
Oct 9, 2026
Merged

longlho merged 1 commit into
mainfrom
codex/ui-location-data

Conversation

@longlho

@longlho longlho commented Oct 9, 2026

Copy link
Copy Markdown
Contributor

Integrating Playwright UI with a real application's preview server requires caller data paths in environment variables. The UI launcher currently hides those direct dependencies behind its manifest target, preventing Bazel from expanding $(rootpath ...) references.

Expose caller data directly to the developer executable and run compiled Playwright from the workspace's runfiles directory, where expanded rootpaths resolve. Source-mode working-directory behavior is unchanged.

The nested-suite discovery fixture now reads declared package data through an expanded environment path, covering both prerequisite expansion and runtime path resolution.

@longlho
longlho marked this pull request as ready for review October 9, 2026 16:55
@longlho
longlho merged commit b0ce71b into main Oct 9, 2026
16 checks passed
@longlho
longlho deleted the codex/ui-location-data branch October 9, 2026 16:56
longlho pushed a commit that referenced this pull request Oct 10, 2026
🤖 I have created a release *beep* *boop*
---


## 3.9.0 (2026-10-09)

## What's Changed
* feat: prune source maps from browser input directories by @longlho in
#114
* fix: let callers control browser test selection by @longlho in
#116
* fix: preserve npm package paths in runtime inputs by @longlho in
#117
* fix: allow remote caching for local browser tests by @longlho in
#119
* fix: leave host browser execution restrictions to callers by @longlho
in #120
* feat: aggregate E2E suites in local Playwright UI by @longlho in
#121
* fix: resolve caller data paths in Playwright UI by @longlho in
#122
* fix: build Playwright UI bundles as declared Bazel outputs by @longlho
in #123


**Full Changelog**:
v3.8.1...v3.9.0

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
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.

1 participant