Add critical user flow test gate - #125
Open
iatopilskii wants to merge 2 commits into
Open
Conversation
iatopilskii
marked this pull request as ready for review
August 1, 2026 06:28
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds a dedicated Playwright release gate for DevHub’s most critical developer and coding-agent user flows.
The suite runs against a local production build. It covers desktop Chromium and WebKit plus explicitly named Playwright-emulated Pixel and iPhone profiles. No production monitoring, scheduled checks, or alerts are added.
Why
DevHub already had broad Vitest and Chromium regression coverage, but there was no isolated cross-browser gate for journeys whose failure prevents a developer or coding agent from getting useful content.
Critical flows covered
Cmd/Ctrl+Kor mobile-menu search → open Databricks CLI documentation → copy useful Markdown./api/bootstrap-prompt/llms.txttools/listlist_docs_resourcesandget_doc_resourceImplementation
playwright.critical.config.tsand a documented P0/P1 test plan.pnpm test:criticaland an independent critical-flow CI step.emulated-pixel-chromiumandemulated-iphone-webkitso CI output cannot be mistaken for physical-device coverage.Validation
pnpm build: passed.pnpm typecheck: passed.fallow dead-code: no issues.Existing baseline failures
The broader regression suite still reports unrelated failures in files not changed by this PR:
fallow dupesreports existing duplication groups in untouched production files.Test plan
See
tests/critical/README.mdfor priorities, expected outcomes, browser coverage, and local commands.