apunkt/SurfSense
1
0
Fork 0
mirror of https://github.com/MODSetter/SurfSense.git synced 2026-05-12 09:12:40 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

chat: suppress stale step separator emitted during resume rehydration.

Browse source
This commit is contained in:
CREDO23 2026-05-09 18:36:00 +02:00
parent ba0e1e70a0
commit 89e4953800
1 changed files with 15 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

16
surfsense_web/lib/chat/streaming-state.ts
View file

@ -73,6 +73,18 @@ export interface ContentPartsState {
currentTextPartIndex: number; currentTextPartIndex: number;
currentReasoningPartIndex: number; currentReasoningPartIndex: number;
toolCallIndices: Map<string, number>; toolCallIndices: Map<string, number>;
/**
* Set by the resume flow's rehydration to suppress
* ``data-step-separator`` for the rest of this turn. Without it,
* the resume stream's first ``start-step`` fires
* ``addStepSeparator`` while rehydrated OLD content already makes
* ``hasContent`` true → a divider lands between OLD and NEW
* content with no semantic value (OLD content is filtered by
* ``buildTimeline`` + ``filterSupersededAbortedMessages``,
* persisted state carries no separator, so the line vanishes on
* reload).
*/
suppressStepSeparators?: boolean;
} }
function areThinkingStepsEqual(current: ThinkingStepData[], next: ThinkingStepData[]): boolean { function areThinkingStepsEqual(current: ThinkingStepData[], next: ThinkingStepData[]): boolean {
@ -234,7 +246,9 @@ export function addStepSeparator(state: ContentPartsState): void {
// non-step content (so the FIRST step of a turn doesn't // non-step content (so the FIRST step of a turn doesn't
// generate a leading separator) and when the previous part isn't // generate a leading separator) and when the previous part isn't
// itself a separator (defensive against duplicate `start-step` // itself a separator (defensive against duplicate `start-step`
// events). // events). Also skipped during a resume turn (see
// ``suppressStepSeparators`` on ``ContentPartsState``).
if (state.suppressStepSeparators) return;
const hasContent = state.contentParts.some( const hasContent = state.contentParts.some(
(p) => p.type === "text" || p.type === "reasoning" || p.type === "tool-call" (p) => p.type === "text" || p.type === "reasoning" || p.type === "tool-call"
); );