From a465e3fec184464042c8ae8d5e16e9045ef56e52 Mon Sep 17 00:00:00 2001 From: Tam Nhu Tran Date: Thu, 30 Apr 2026 15:26:06 -0400 Subject: [PATCH] fix(ui): include message in coalesce key so distinct entries stay visible MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit PR-Agent flagged that the leaf and trace-child coalesce keys collapsed on `(event, module, level, requestId, source)` which merges adjacent entries that share that tuple but report different content (e.g. `User logged in: alice` vs `User logged in: bob` would appear as a single `× 2` row, hiding bob). Add `message` to both coalesce keys so distinct content stays distinct. Skip `latencyMs` and `metadata` — those drift per request even on truly redundant polls and would defeat the dedup the feature exists for. Refs #1138, #1151 --- ui/src/components/logs/derive-trace-groups.ts | Bin 4739 -> 4991 bytes ui/src/components/logs/logs-trace-row.tsx | Bin 4940 -> 5175 bytes 2 files changed, 0 insertions(+), 0 deletions(-) diff --git a/ui/src/components/logs/derive-trace-groups.ts b/ui/src/components/logs/derive-trace-groups.ts index d1e9059f038835b54d6364208fbc73473c602701..bbe747450b8bf3d8365244b150ee7358e3182c9f 100644 GIT binary patch delta 362 zcmZXQF-`+95Je%8C@46?G$5eZh?brL2}=5%wP!a*UVHJ33EL1Pjt~wKiKB1<#6@7d zNk?PL{+|Ee_xtJZ*>;?*E4U5DXG;uqPt4d+4IDvM5zs!Op&;O6#~@3TW4HufVj`et z6&8K8+0(-7$wQL`5)Cm4Sl70c%BYD!%FcqLUbDDhCYy7rx(e3y15+vi9a(VU8qhnV z2pBB<)0)`S7Zu#R?1&3Z?NR;Bc+dNpo&+rl*3gi_%%tE^SDRcbFlbn!r%WtUg*%9$ z*htELRK@0CQE`XpJ;Tr2hccBg7drK!9#%1X3qzXvE9IyS(GweevH70aqun2FuI8_Y K2cO&d*U29t{es8< delta 150 zcmWNJJqp4=6b4aKrSJ?N0Yzoe-pW?=63gakf^YZf{urz*JVLHvA$Sx^52IVoz|3#@ zn7nSo`x>zmkc+;h0PP7?(3Y~1bZja_fe~hX4uq~DS7Lz~U2~vdrA|x=oS79oT@r=3 rtrdbKZ)gt946CDQmbqJ%hvZCb#pR`D3Wj$5ceL*}#c6=|v-phvmEtr< diff --git a/ui/src/components/logs/logs-trace-row.tsx b/ui/src/components/logs/logs-trace-row.tsx index 32bd1bf4c8f712253483357511e84e30d479d467..5f6d6233d4df3131cab6f08c75d77fb98bc00a27 100644 GIT binary patch delta 461 zcmZvY!Ab)$5QY_$Lg_>NC$|;ZK0;5PMDQ-crkmMrBu!!_Y28zO2w%aw(1S1G(I@ap zoLvP`8UmSQX1>Y)zn{HLUq4QEW1P#z6?I5CSb}l2&6*h1nwZs!DzHb|dep74I+88njJQpht1y2Hk>SqfA!n1YHb5u)F6iONds>h78=7#@D~I7ra=e0qkF>CrTa z5vb=rWa0$nG`2q5QbSWiq9?T^q0Rwb6vk}39u?)qX(b;H-df(@oW30IZpRqbU`RRW r$6|q^xcHO!@$XP-aO+TFFz_Gd+s)qr6Wo>e-neOTRLu65qtWCGUQCvH delta 226 zcmdn4aYk*!6_$F1;KbZih3wQyE(I-xlG1{lRE5N1h2oOLyp+V8{Jd1%oYcfLh2;Fi zoYdmv%)E4kM-5XH5;RiFQu9i5K9FSqj8JRgLMX7lT z#Tki3sR{{2sfDGf#U-982|yJ^sfj6-dRz*dnb{UG1|+CdgH`K+ja9I>S5Q~4t$_)G aEkPFqTZvGUo1an&bUH+5!e#>wb{+u7?@nm|