feat(insights): parallel-subagent benefit (concurrency + time saved)#8
Merged
Merged
Conversation
Surfaces the value of fanning out: the subagent phase is aggregated by wall-clock union (correct elapsed time), but sum−union = time saved by running agents concurrently and sum/union = effective concurrency. Adds an Insights line shown only when ≥2 subagents actually overlapped (saved >1s), so sequential sessions get no spurious line. Exports unionMs from analyzer for reuse. +3 formatter tests. 102 pass; tsc clean.
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.
What
Follow-up to the wall-clock-union fix. The union gives correct elapsed time, but the sum of subagent durations still carries signal — so this surfaces the benefit of fanning out:
sum − union= time saved by concurrency;sum / union= effective concurrency factor.unionMsfrom analyzer for reuse.Tests
+3 formatter tests (overlap → line; single agent → none; sequential → none). 102 pass;
tscclean.Note
Builds on the union fix (uses
unionMs). Upstream version waits until the union PR (dioptx#7) merges, then goes up on a dedicatedupstream/*branch.🤖 Generated with Claude Code