mirror of
https://github.com/tiennm99/ccs.git
synced 2026-10-11 03:13:12 +00:00
fix(ui): include stage in leaf coalesce key
PR-Agent flagged that the leaf coalesceKey omitted `stage`, so two adjacent no-requestId logs differing only by stage (e.g. one logging `route` and the next `dispatch`) would merge into a single `×2` row, hiding a distinct entry. Add `stage` to the coalesce tuple. New unit test asserts that two leaves identical except for stage stay as separate rows. Refs #1138, #1151
This commit is contained in:
1 parent
2ddc5d5afd
commit
e7058b6b6d
2 files changed
+12
No files matched your search
@@ -33,6 +33,7 @@ function coalesceKey(entry: LogsEntry): string {
|
|||||||
return [
|
return [
|
||||||
entry.event ?? '',
|
entry.event ?? '',
|
||||||
entry.message ?? '',
|
entry.message ?? '',
|
||||||
|
entry.stage ?? '',
|
||||||
entry.module ?? entry.source ?? '',
|
entry.module ?? entry.source ?? '',
|
||||||
entry.level,
|
entry.level,
|
||||||
entry.requestId ?? '',
|
entry.requestId ?? '',
|
||||||
|
|||||||
@@ -119,6 +119,17 @@ describe('deriveTraceGroups', () => {
|
|||||||
expect(leaves.every((l) => l.repeatCount === undefined)).toBe(true);
|
expect(leaves.every((l) => l.repeatCount === undefined)).toBe(true);
|
||||||
});
|
});
|
||||||
|
|
||||||
|
it('keeps adjacent leaves with different stages as separate rows (round-6 fix)', () => {
|
||||||
|
const result = deriveTraceGroups(
|
||||||
|
leafEntries(
|
||||||
|
{ id: '1', timestamp: 't1', event: 'e', message: 'm', stage: 'route' },
|
||||||
|
{ id: '2', timestamp: 't2', event: 'e', message: 'm', stage: 'dispatch' }
|
||||||
|
)
|
||||||
|
);
|
||||||
|
expect(result).toHaveLength(2);
|
||||||
|
expect(result.every((r) => r.kind === 'leaf' && r.repeatCount === undefined)).toBe(true);
|
||||||
|
});
|
||||||
|
|
||||||
it('keeps adjacent leaves with different messages as separate rows', () => {
|
it('keeps adjacent leaves with different messages as separate rows', () => {
|
||||||
const result = deriveTraceGroups(
|
const result = deriveTraceGroups(
|
||||||
leafEntries(
|
leafEntries(
|
||||||
|
|||||||
Reference in new issue
Block a user