feat(runner): tap, swipe and fill in runner actions sequences (AGE-5427) - #1642
Draft
Erzhan Torokulov (erzhtor) wants to merge 1 commit into
Draft
Erzhan Torokulov (erzhtor) wants to merge 1 commit into
Erzhan Torokulov (erzhtor) wants to merge 1 commit into
Conversation
4 tasks done
Atchyut Preetham Pulavarthi (theonly1me)
approved these changes
Oct 1, 2026
runner actions validated each step as a browser action, so a sequence refused tap, swipe and fill locally even though its description says it takes the same actions as runner act. api-contracts 0.74.0 widens the sequence contract to every runner action and adds the action-not-supported-on-browser step reason. bun.lock is left for after 0.74.0 publishes.
Erzhan Torokulov (erzhtor)
force-pushed
the
erzhan/age-5427-runner-actions-mobile
branch
from
October 2, 2026 08:35
4009c15 to
02bc353
Compare
This branch has not been deployed
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.
Relates to AGE-5427. Platform side: qawolf/platform#34509 (not merged yet; publishes
@qawolf/api-contracts0.78.0). Follows #1638.Overview of Changes
After #1638,
runner actionssaid it takes "the same actionsrunner acttakes", but it checked each step against the browser-only schema. That meanttap,swipeandfillwere refused locally before any request went out. Contracts 0.78.0 widens the sequence contract to every runner action, so this PR:parseRunnerAction, the same checkrunner actuses (and drops the now-unusedparseBrowserAction)action-not-supported-on-browserstep reason (exit code 2), for when a browser runner refuses a mobile step; with--continue-on-failurethe rest still runsrunner actions --helpThe description is unchanged, because it is now accurate.
Testing
Ran against a local pack of 0.78.0 built from qawolf/platform#34509:
typecheck,lint,format:check,knipandtestpass, except 3 keychain tests that also fail onmainon this machine. New tests are inperformActions.touchscreen.test.ts.To Do
@qawolf/api-contracts0.78.0 to publishrun acceptScreenshotBaseline,listScreenshotComparisonsandrestoreScreenshotBaseline.mainpins 0.77.0, which was published from the unmerged screenshot stack (qawolf/platform#34307); a 0.78.0 built from platformmainwithout it would drop those commandsbun installto updatebun.lock, thenbun run generateand commit anySKILL.mddriftChecklist