Skip to content

Commit 8ceb59a

Browse files
authored
Fix message order in agentex UI (#86)
Follow up change to [this PR](#85) — as a result of making sorting the messages by DESC by default, the UI will need to handle reversing the order of the messages upon arrival.
1 parent bfd7554 commit 8ceb59a

2 files changed

Lines changed: 12 additions & 5 deletions

File tree

agentex-ui/hooks/custom-subscribe-task-state.tsx

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -142,9 +142,10 @@ export async function subscribeTaskState(
142142
return res;
143143
}),
144144
client.messages
145-
.list({ task_id: taskID, limit: 1000 }, { signal })
145+
.list({ task_id: taskID }, { signal })
146146
.then(res => {
147-
eventListener.onMessagesChange(res);
147+
const chronologicalMessages = res.slice().reverse();
148+
eventListener.onMessagesChange(chronologicalMessages);
148149
return res;
149150
}),
150151
]);

agentex-ui/hooks/use-task-messages.ts

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -53,8 +53,10 @@ export function useTaskMessages({
5353
task_id: taskId,
5454
});
5555

56+
// API returns messages in descending order (newest first),
57+
// reverse to chronological order (oldest first) for display
5658
return {
57-
messages,
59+
messages: messages.slice().reverse(),
5860
deltaAccumulator: null,
5961
rpcStatus: 'idle',
6062
};
@@ -205,14 +207,18 @@ export function useSendMessage({
205207
task_id: taskId,
206208
});
207209

210+
// API returns messages in descending order (newest first),
211+
// reverse to chronological order (oldest first) for display
212+
const chronologicalMessages = finalMessages.slice().reverse();
213+
208214
queryClient.setQueryData<TaskMessagesData>(queryKey, {
209-
messages: finalMessages,
215+
messages: chronologicalMessages,
210216
deltaAccumulator: null,
211217
rpcStatus: 'success',
212218
});
213219

214220
return {
215-
messages: finalMessages,
221+
messages: chronologicalMessages,
216222
deltaAccumulator: null,
217223
};
218224
}

0 commit comments

Comments
 (0)