From c307f1a1bc62d508a034f3445cdabd0eaeee4451 Mon Sep 17 00:00:00 2001 From: Daryl Date: Thu, 26 Feb 2026 17:32:16 -0400 Subject: [PATCH] Revert Copilot mistake in inheriting prior swipe data; testing in practice reveals inheritance does not work after applying its suggestion. --- src/systems/integration/sillytavern.js | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/systems/integration/sillytavern.js b/src/systems/integration/sillytavern.js index be13b47..d55d8f8 100644 --- a/src/systems/integration/sillytavern.js +++ b/src/systems/integration/sillytavern.js @@ -303,8 +303,9 @@ export async function onMessageReceived(data) { // When auto-update is disabled, no tracker API call will run for this message. // Inherit the prior assistant message's tracker data into this swipe slot so that - // commitTrackerDataFromPriorMessage can find a valid state next turn instead of nulling everything - if (!extensionSettings.autoUpdate && isAwaitingNewMessage) { + // commitTrackerDataFromPriorMessage can find a valid state next turn instead of nulling everything. + // Inheritance does not overwrite existing data, so it's safe to call even if the condition misses an edge case. + if (!extensionSettings.autoUpdate || !isAwaitingNewMessage) { inheritSwipeDataFromPriorMessage(lastMessage, chat.length - 1); } }