flux-pr-1351
graphql-go-tools (Go) · W2 · GPT-5.4
Tests passed. 1/1 commands passed. Strength: strong.
go test -C v2 ./... -count=1 -timeout=300sPartial score: 1/1
Trajectory
codex · partial order onlyprovider-native trajectory captured; validation and decision steps are appended with coarse ordering only
Quality
Equivalence Reasoning
stylistic
Code Review
The patch moves toward per-subgraph attribution and deterministic aggregation, but likely does not fully satisfy the intended name-based grouping semantics and introduces collision-related loss in the by-subgraph view.
Errors are stored under a composite key built from datasource ID and name, so same-name subgraphs with different IDs are tracked separately. This diverges from the requested grouping by subgraph name.
`BySubgraph()` writes one value per `mapKey()` and does not merge on duplicate names, so if multiple internal keys resolve to the same output name, earlier errors are overwritten.