feat(runner): perform a sequence of actions in one request with runner actions - #1624
Draft
Goran Gajic (gorangajic) wants to merge 2 commits into
Draft
Goran Gajic (gorangajic) wants to merge 2 commits into
Goran Gajic (gorangajic) wants to merge 2 commits into
Conversation
…r actions An agent that already knows its next few steps paid one round trip per step through runner act. runner actions takes a JSON array of the same actions and sends it as one runner.performActions call; the frames it asks for are written as files, never echoed as base64. Each result carries the effect the contract fixes per reason, and a stop names the action, why, and how many were left unperformed. The contract is listed as hand-written so the generator does not try to give a list of actions a flag shape.
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: trueComment |
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 WIZ-12132. Depends on
@qawolf/api-contracts0.61.0, which ships with qawolf/platform#33491.Overview of Changes
An agent that already knows its next few steps had to send each one as its own
qawolf runner actcall, a full round trip per step.qawolf runner actionstakes a JSON array of up to ten of the same actions and performs them in one request, so the agent looks at the screen once, at the end. The answer reports each action as performed, not performed, or unknown when the runner stopped answering and the action may have landed, and says where the sequence stopped and why.--screenshotsaves a frame after the last action,--screenshot-mode eachsaves one per action, and--continue-on-failurecarries on past an action that did not take effect.Testing
Ran end to end against the platform PR's preview with a real playwright runner: a four-action batch with a final frame, a two-action batch with a frame per step, a batch spanning a navigation, and a batch whose first step was a navigation the runner could not confirm. Frames and per-step results matched the screen.
bun run typecheck bun run lint bun run format:check bun run knip bun run test bun run buildThree
run.createtests fail onmainas well (aworkspaceIddrift unrelated to this change).To Do
@qawolf/api-contractsto 0.61.0 once it is published; typecheck fails against the current pin until then.Checklist
🤖 Generated with Claude Code