Repository navigation
feat(drivers): iOS simulators can live in Xcode's device set (drivers.ios.deviceSet) - #976
Merged
Merged
Conversation
…er deviceSet default (#968)
…nvalid-device-set (#968)
…flicts with deviceRoot (#968)
…rom the other deviceSet (#968)
…ode's set deletes the new simulator (#968)
… by this instance (#968)
…instance's simulators (#968)
…e only devices this instance does not mark (#968)
…e out this instance's simulators (#968)
…es a device set (#968)
…and keeps its refusals (#968)
…ease environment has none (#968)
V3RON
added this pull request to stack #977
October 24, 2026 18:28
…s drop ADR cites, flow row names default (#968)
Contributor
Author
Review ledgerRound 1: 832caec {"findings":[{"id":"R1","role":"code","class":"architecture 17","outcome":"fixed","path":"src/drivers/ios/index.ts","round":"1","fix":"4e02370d1e0fc3c3ea581e9ab0d4b776557c2277"},{"id":"R2","role":"code","class":"architecture 9","outcome":"fixed","path":"src/drivers/ios/index.ts","round":"1","fix":"4e02370d1e0fc3c3ea581e9ab0d4b776557c2277"},{"id":"R3","role":"claims","class":"false claim in flow doc","outcome":"fixed","path":"docs/internal/flows/device-exec.md","round":"1","fix":"4e02370d1e0fc3c3ea581e9ab0d4b776557c2277"}]}Written by an agent. |
Contributor
Author
Review notesNot blocking, not verified. Each is one reviewer's claim.
Written by an agent. |
Contributor
Author
Decision takenThe build-and-run page's switch steps carry a decision taken in delivery, pending the maintainer: an operator with a custom Written by an agent. |
V3RON
marked this pull request as ready for review
October 24, 2026 19:18
This was referenced Oct 24, 2026
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.
Closes #968
Status
Implement: done Review: fix-check passed at e7a2901 (CI bound fix), ADR flip 34876b8 verified by coordinator Hardware: pass at d1cec86 Gate: running
Next: the stack gate on #976 (merges #964 and #976)
pnpm test:changedpasses, with the Tests lines of the issuebuild-and-run.mdxexists and is in the sidebar under "Using devices"docs/internal/EVENTS.mdlistsinvalid-device-setanddevice-set-changed"default"case ofe2e/slow-agent-device.test.tspasses — slow lane at d1cec86 (a first run timed out inside agent-device and did not reproduce)scripts/slow-e2e.sh stopmid-run leaves no simulator whose durable mark names a lane instance — stopped a run mid-way at d1cec86; stop deleted the leased simulator, no mark left in either set#962 Completion conditions:
drivers.ios.deviceSet: "default", a leased simulator is listed by a plainxcrun simctl list devicesand agent-device drives it (snapshot), on a real Mac. — slow lane"default"case at d1cec86"default", a simulator this Simlock did not create is never in its device list, so cleanup, the warm pool,nukeanddoctor --purge-orphansnever touch it. —src/drivers/ios/index.test.tslistManaged and foreignDevices tests under deviceSet defaultsimlock doctorshows; Android keeps running. (Today a refused driver makessimlock doctorfail with an internal error; iOS simulators can live in Xcode's device set (drivers.ios.deviceSet) #968 fixes that.) —src/drivers/ios/index.test.tsdevice-set-changed test,src/daemon/dispatcher.test.tsandsrc/cli/index.test.tsdriver-unavailable doctor testssimlock instructionsgives the build route for each. — website build in CI,src/instructions/index.test.tsiOS builds section testAssumptions
slow-e2e.shat a scratch Devices folder — anSIMLOCK_SLOW_DEVICESoverride, like the script's other scratch overrides.coreSimulatorRootis absent — it falls back to".", assrc/drivers/ios/index.ts:313already does; the daemon always passes it (src/daemon/main.ts:1372).detail— 1000 characters; doctor cuts the driver's summary so the retry remedy stays whole.Review
Spec review: 0 blocking, 0 fixed, 0 notes. Code review: 2 blocking, 2 fixed, 0 notes. Claims review: 1 blocking, 1 fixed, 2 notes.
Rounds: 2. Tokens: 7.9M.
Runs: 20. Cost USD: 8.1105. Agent seconds: 4687.
Assessment: complete
Written by an agent.
Written by an agent.