[pull] main from Zoo-Code-Org:main - #32
Merged
Merged
Conversation
* fix(webview): throttle large state updates Coalesce repeated full-state webview updates with a leading and trailing debounce and a one-second maximum wait. Keep task-start, API-boundary, and stream-completion updates immediate, and flush pending state during partial-message initialization and task abort. Reset aggregated task costs when switching tasks and ignore delayed responses for inactive tasks so stale per-task data is not retained or displayed. Add regression coverage for debounce timing, flush and disposal behavior, partial-message ordering, queue failures, and task-switch cost cleanup. Signed-off-by: JunyongParkDev <jun94.park@samsung.com> * fix(task): preserve lifecycle on state push failures Log throttled state scheduling and flush failures without dropping message persistence or abort cleanup. Cover Task rejection paths and failures from a pending debounced flush. Signed-off-by: JunyongParkDev <jun94.park@samsung.com> * refactor(webview): clarify throttled state semantics Document the distinction between scheduling and flushing throttled state updates. Use the typed abort event in ordering coverage. Signed-off-by: JunyongParkDev <jun94.park@samsung.com> * fix(webview): skip unused task history computation Avoid sorting task history for state payloads that omit it while preserving a single late history read for full-state payloads. Add focused regression coverage for both paths. Signed-off-by: JunyongParkDev <jun94.park@samsung.com> * test(webview): verify throttled flush completion Strengthen the existing flush test to confirm that the provider waits for a pending asynchronous trailing state post and still invokes it exactly once. Signed-off-by: JunyongParkDev <jun94.park@samsung.com> * test(webview): use shared state fixture in chat tests Align the rebased aggregated-cost task-state helper with the shared extension-state factory and provide the required history item fields. Signed-off-by: JunyongParkDev <jun94.park@samsung.com> --------- Signed-off-by: JunyongParkDev <jun94.park@samsung.com>
#1143) * refactor(webview): canonicalize provider settings identifiers * refactor(webview): centralize model message identifiers
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.4)
Can you help keep this open source service alive? 💖 Please sponsor : )