Add optional task-scoped Android testing for coding agents - #684
Merged
Merged
Conversation
Member
Author
|
Quick integration review against main a607464: this branch contains the unmerged PR682 base and has worker merge conflicts. Integrate against the accepted final PR682 rather than merging both old worker snapshots independently. The shared host-capacity lock serialises coding and emulator launches; same-host nested coding/emulator work is not supported by this baseline. Preserve the validated DGX-to-Mac separate-host route and verify cleanup/interruption with the integrated worker. No fresh end-to-end test was run for this quick review. |
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.
Adds optional task-scoped Android testing to coding-agent instances, reusing the operator registry and capacity lock. The existing coding controller handles native requests without passing credentials to its coding child; screenshots and reports return through canonical task attachments.
Ready to merge after core PR682, now merged. Current-main integration preserves timing, token accounting, retry and supervision. 131 focused tests pass. Prior native emulator/controller acceptance remains valid: its unchanged host/controller/service files have not been altered during integration, so no emulator campaign was repeated.
Same-host nested coding/emulator reservations remain unsupported by the single-slot baseline; the intended DGX/Mac route has separate host budgets. Chrome152 and physical devices are outside acceptance. Publishing does not activate a production Android registry, host relay or default emulator launch.
Task:
#V#task_add_optional_androidemulator_testing_to_53400265. Evidence:docs/engineering/coding_agent_android_testing.mdand canonical task attachments. Parent VS Code owner handles native close-out and the separately authorised final deployment.