feat: stream replay test progress#605
Merged
Merged
Conversation
|
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
1 participant
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
Stream
agent-device testprogress progressively over daemon transports so CI logs show one-linepass,fail, orskipupdates as tests finish or retry.Progress lines include current/total position, retry state, and elapsed seconds with two decimals, for example
pass 3/6 ... duration=12.34s, before the existing final suite summary.Review/fallow follow-up: hardened socket progress handling after settle, added HTTP NDJSON progress coverage, removed synthetic
duration=0.00sfrom skipped entries, and consolidated duplicated line-stream/auth/listener helpers flagged by Fallow.Touched 15 files; scope stayed within replay test progress plumbing, CLI transport output, tests, docs, and cleanup required by Fallow.
Validation
Verified with
pnpm check:fallow --base origin/main, focused progress/client/contract tests,pnpm format,pnpm check:quick,pnpm build, and replay e2e runs. The macOS replay suite streamedpass 1/1 ... attempt=1/1progressively before the final summary, and iOS replay suite output showed per-testpass/failprogress across the suite.Known gap:
pnpm check:unitwas attempted but hit existing environment-sensitive loopback failures (EADDRNOTAVAIL) unrelated to this change.