fix(cli): address PR review issues for TUI transcript handlers

- Replace non-null assertions on allMessages.data with nullish coalescing
- Move kilocode_change end markers to after sessionMessages declaration
This commit is contained in:
Aarav Sharma
2026-05-15 16:50:38 -06:00
parent 2113986133
commit 19dc9d04d2
@@ -981,14 +981,14 @@ export function Session() {
if (!sessionData) return
// kilocode_change start - fetch all messages from server instead of truncated sync store
const allMessages = await sdk.client.session.messages({ sessionID: sessionData.id })
const sessionMessages = allMessages.data!.map((msg) => ({
const sessionMessages = (allMessages.data ?? []).map((msg) => ({
info: msg.info,
parts: msg.parts,
}))
// kilocode_change end
const transcript = formatTranscript(
sessionData,
sessionMessages,
// kilocode_change end
sessionMessages, // kilocode_change
{
thinking: showThinking(),
toolDetails: showDetails(),
@@ -1032,15 +1032,15 @@ export function Session() {
// kilocode_change start - fetch all messages from server instead of truncated sync store
const allMessages = await sdk.client.session.messages({ sessionID: sessionData.id })
const sessionMessages = allMessages.data!.map((msg) => ({
const sessionMessages = (allMessages.data ?? []).map((msg) => ({
info: msg.info,
parts: msg.parts,
}))
// kilocode_change end
const transcript = formatTranscript(
sessionData,
sessionMessages,
// kilocode_change end
sessionMessages, // kilocode_change
{
thinking: options.thinking,
toolDetails: options.toolDetails,