{"entries":[{"name":"client","mode":"0040000","size":0,"lastCommit":{"hash":"521a64cd01979bb5b1a466152c576a9d809b068d","oid":"521a64cd01979bb5b1a466152c576a9d809b068d","parents":["681c29d36a13be54d317ee147b272da9163dbef3"],"author":{"name":"Aleksander Grygier","email":"aleksander.grygier@gmail.com","when":"2026-08-20T19:02:04+02:00"},"committer":{"name":"GitHub","email":"noreply@github.com","when":"2026-08-20T19:02:04+02:00"},"message":"ui: Stores split refactor (#27240)\n\n* ui: Extract server stream lifecycle from chatStore into ChatStreamManager\n\nDiscovery, attach/replay, resume retry and the remote-running snapshot\nformed a cohesive cluster inside chatStore. It now lives in\nchat-streams.svelte.ts as ChatStreamManager, owned by chatStore, which\nkeeps the public entry points as delegates so components are\nunchanged. chatStore: 2877 -> 2418 lines.\n\n* ui: Extract user interaction gates from agenticStore into AgenticGates\n\nTool permission requests, turn-limit continue prompts and queued\nsteering messages are the state the loop waits on between turns. They\nhad no coupling to session state, so they now live in\nagentic-gates.svelte.ts; agenticStore keeps delegates so components\nare unchanged. agenticStore: 1196 -> 1073 lines.\n\n* ui: Compose MCP resources under mcpStore.resources\n\nResource state was a second import scope next to mcpStore. Consumers\nnow go through mcpStore.resources, so the MCP surface is one store;\nmcp-resources.svelte.ts stays a separate file owned by mcpStore.\n\n* ui: Reorganize stores into domain namespaces\n\n* fix: Update stale doc comments\n\n* ui: Consolidate conv running-state into a chat activity ledger\n\nRunning-state was split across chatStore.chatLoadingStates (local\npipes), ChatStreamManager.remoteRunningConvs (backend sessions) and\nattachingConvs (attach lifecycle), unioned by hand in\ngetAllLoadingChats and cross-cleaned by setChatLoading calling\nstreams.clearRemoteRunning - the 'spinner ghosts until tab toggle'\nworkaround.\n\nchatActivityStore now owns both sets with one transition per event:\nmarkLocal / localEnded (local pipe end also drops the stale remote\nhint, no cross-owner call) / applyRemoteSnapshot (diffed). The\nsidebar reads chatStore.activity.loadingConvs through the unchanged\ngetAllLoadingChats entry point.\n\nConsequences:\n- isStreamingActive and its five manual writers are gone; isStreaming()\n now reports whether the active conversation has a live streaming\n pipe, which is what all four consumers (assistant row, stop action,\n context gauge, chat screen) actually check\n- isLoading/isReasoning become derived from the per-conv maps plus\n the active conversation, dropping the manual resync in\n syncLoadingStateForChat and clearUIState\n- attachingConvs and the last-attach coordination disappear from\n ChatStreamManager\n- getAllStreamingChats (no consumers) is removed\n\n* ui: Give store collaborators narrow host interfaces\n\nCollaborators took 'host: typeof ', i.e. the store's entire\npublic surface, which is how chatStore's streamChatCompletion,\ncreateAssistantMessage, getApiOptions and setStreamingActive got\nwidened to public. Replace with per-collaborator interfaces carrying\nonly the members each one drives:\n\n- ChatStreamHost (chat/streams) - activity, processing, streaming\n states, abort controller, loading/streaming setters\n- ChatFlowsHost (chat/flows) - streaming core, message creation,\n per-conv state setters\n- McpHealthHost (mcp/health) - connection registry + reconnection\n- ModelPropsHost / ModelStatusHost (models) - model rows, feed\n updates; the managers write modalities/status back onto the host's\n rows, so those members stay writable\n- ConversationsPreferencesHost (conversations) - the active row and\n the conversation list\n\nThe store classes now declare 'implements ' so the contract is\nvisible at the class level, and the 'import type { }' back\nreferences in the collaborators disappear entirely - the host\ncontract is local to each collaborator file, and collaborators can\nno longer reach around their slice. Members stay public (structural\ntyping), but the collaborator side is now compiler-enforced.\n\n* test: Chat Activity store test\n\n* refactor: Cleanup\n\n* chore: Remove legacy architecture docs\n\n* ui: Memoize findMessageIndex for the streaming hot path\n\nStreaming looks up the same message index on every chunk, a linear\nscan of activeMessages each time. Cache the last lookup and reuse it\nafter validating the id still sits at the same position (O(1)); any\nstructural change to the array fails validation and falls back to a\nfull scan.\n\n* ui: Throttle per-chunk stream state writes to localStorage\n\nsaveStreamState ran JSON.stringify + a synchronous localStorage.setItem\non every decoded chunk of the stream. The read loop now goes through a\nnew saveStreamStateThrottled (one write per conversation per 500ms,\nlatest value held pending); the public saveStreamState keeps its\nimmediate-write contract for stream start and pre-fetch, and also\nresets the throttle window.\n\nA pending offset is force-flushed at resume boundaries (resumeStream\nreads the offset back from localStorage), on visibilitychange->hidden\nand on pagehide, so a reload always finds a usable offset. The resume\noffset only needs to be roughly current since the server retransmits\nfrom a line boundary and the client discards its partial line.\n\nAdds unit tests for the throttled/flush/clear interplay.\n\n* ui: Compute context gauge timing stats in one pass\n\ncurrentRead/Fresh/Cache/Output were separate deriveds, each running a\nfull reverse scan of activeMessages for the last assistant timings,\nand cumulative ran its own forward scan plus an agentic filter - 4-5\nO(n) passes per chunk while streaming. Replace with a single\nsummarizeAssistantTimings() pass (last assistant timings, last\nagentic llm totals and the cumulative sums) feeding a shared derived\nsnapshot. Semantics unchanged, including the live-stats overrides and\nthe agentic llm-totals branch.\n\n* agentic : clear session state when a conversation is deleted\n\nEvery conversation that ran an agentic flow left an AgenticSession in the\nstore forever; clearSession was never called. conversationsStore now\nnotifies deletion listeners and agenticStore drops the matching sessions,\navoiding a circular import back into conversationsStore.\n\n* chat : extract ChatService.normalizeMessagesForApi\n\nThe DB->API message normalization (convert + drop empty system messages)\nwas duplicated in sendMessage, preEncode and the agentic flow. Extract it\ninto one shared method and call it from all three.\n\n* sse : share record splitting and data extraction\n\nsplitSseRecords and extractSseDataPayload centralize the record-boundary\nsplitting and data: line extraction used by parseSseJsonStream and the\nmodels status feed. chat.service keeps its own line-based parser for\nresume support.\n\n* api : delegate apiFetchWithParams to apiFetch\n\napiFetchWithParams duplicated apiFetch's headers/fetch/error handling\nbody-for-body; it only differs in URL construction. Build the URL and\ndelegate.\n\n* chat flows : dedupe title, timings and cleanup handling\n\n- conversationsStore.applyTitleFromContent centralizes the title-from-first-\n message logic duplicated in 5 places\n- ChatProcessingStore.applyStreamTimings centralizes the onTimings handler\n shared by the chat and continue flows\n- host.cleanupStreaming centralizes the loading/streaming/processing reset\n repeated across the continue flow's exit paths\n\n* conversations : centralize conversation update mirroring\n\nrename, pin, mcp override, reasoning effort and cwd all repeated the same\nwrite-DB-then-mirror-into-list-and-active dance. A single\napplyConversationUpdate(id, updates) on the host collapses all five and\nremoves the forgot-to-mirror-one-field bug class. Drops the redundant\narray reassignment in setCwd (deep field assignment is reactive).\n\n* mcp : dedupe tool execution, server parsing and tool indexing\n\n- executeTool delegates to executeToolByName (only diff was argument parsing)\n- drop the private #parseServerSettings copy; use parseMcpServerSettings\n- cache getServers() keyed on the raw config value (hot path)\n- indexServerTools() unifies the three identical toolsIndex rebuild loops\n\nAssisted-by: Claude\n\n* mcp : share cursor pagination and tool indexing\n\n- MCPService.paginate() collapses the identical do-while loops in\n listAllResources and listAllResourceTemplates\n- promoteHealthCheckToConnection now uses indexServerTools like the other\n connect paths\n\nAssisted-by: Claude\n\n* database : share message parent-child bookkeeping\n\n- addChildToParent() dedups the append-to-children update in createMessageBranch\n and createSystemMessage\n- removeChildFromParent() dedups the remove-from-children cleanup in deleteMessage\n and deleteMessageCascading\n- bulkAdd the cloned messages when forking a conversation instead of one add\n per message\n\nAssisted-by: Claude\n\n* chore: Lint/format\n\n* fix: `pagehide` event from `window`\n\n* refactor: Api Fetch util\n\n* docs : rewrite architecture sections in README\n\nUpdate the high-level diagram, routes, hooks, stores, services and data\nflow tables to match the current UI structure (mcp/settings/search\nroutes, agentic/tools/mcp stores, MCPService/ToolsService/SandboxService,\n/tools API). Fix stale architectural patterns for per-conversation state\nand modality validation.\n\n* chore : add ESLint rule for blank lines between accessors\n\nEnforce a blank line between consecutive class accessors. The core\npadding-line-between-statements rule does not cover class members, so a\nlocal rule is needed.\n\n* refactor : reorder store members and unify naming\n\nOrder store class members as public fields, private fields, constructor,\ngetters, public methods, then private methods. Normalize private naming\nto the `private` keyword (drop `#` and the `_` prefix where there is no\nmatching public getter). Rename conversationsStore.init() to\ninitialize() to match the other stores.\n\n* refactor : prefix lookup methods with get in agentic and chat stores\n\nUnify bare-name lookup methods with the get* prefix used across the\nother stores (mcp, models, tools, settings). Renames currentTurn,\ntotalToolCalls, lastError, streamingToolCall, executingToolCallId,\npendingPermissionRequest, pendingContinueRequest,\npendingSteeringMessageContent, pendingSteeringMessageExtras in the\nagentic store and pendingMessageContent, pendingMessageExtras in the\nchat store. Updates the two consuming components and a doc comment.\n\n* refactor: Clean up comments in stores' and services' code\n\n* chore : add ESLint rule for class member ordering\n\nEnforce structural order (public fields -> private fields -> constructor ->\ngetters -> setters -> public methods -> private methods) with alphabetical\nsorting within each group via perfectionist/sort-classes. Dependency\ndetection keeps Svelte $derived fields in a valid dependency order instead\nof alphabetizing them, since Svelte rejects forward references.\n\nAssisted-by: Claude\n\n* refactor : reorder class members to match new ESLint rule\n\nApply the sort-classes rule across stores, services, hooks and utils.\nPure reordering - verified no logic changes by comparing sorted line\nmultisets before/after. All tests and svelte-check pass.","tree":"c5664a77330acafaa7748fed43cb7404815ef63a"}},{"name":"e2e","mode":"0040000","size":0,"lastCommit":{"hash":"92d1bb0c996f49bb09561f755ba784565e1aa51c","oid":"92d1bb0c996f49bb09561f755ba784565e1aa51c","parents":["1e396e72a84b8bc688e428bc1d08623988dfb3a8"],"author":{"name":"Aleksander Grygier","email":"aleksander.grygier@gmail.com","when":"2026-08-10T08:38:37+02:00"},"committer":{"name":"GitHub","email":"noreply@github.com","when":"2026-08-10T08:38:37+02:00"},"message":"ui: Linting & Formatting scripts (#26819)","tree":"8b62e931bce233b6440cae8ff4217835272cd1c2"}},{"name":"stories","mode":"0040000","size":0,"lastCommit":{"hash":"8144f3192e5a3131cd043f284525e6ceebf82d0f","oid":"8144f3192e5a3131cd043f284525e6ceebf82d0f","parents":["6657ded4faa3b8450221119fc6b4d002e35104a2"],"author":{"name":"Aleksander Grygier","email":"aleksander.grygier@gmail.com","when":"2026-08-23T10:46:49+02:00"},"committer":{"name":"GitHub","email":"noreply@github.com","when":"2026-08-23T10:46:49+02:00"},"message":"ui: Chat Conversation Tabbed navigation (#27263)\n\n* ui : add browser-style conversation tabs store\n\nTrack open conversation tabs in order, persisted to localStorage and\npruned against the loaded conversation list on init. The chat layout\nsyncs the route's tab on every navigation, so any way of reaching a\nconversation opens a tab for it.\n\n* ui : add temporary new-chat tabs\n\nNew-chat tabs are unsaved conversations carrying a temporary id used\ndirectly as the route (#/chat/). They live in memory and are only\npersisted to the database - keeping the same id so the route and tab\nstay stable - when the first message is sent. Deleting one drops it\nwithout confirmation, and deleting conversations now closes their tabs.\n\n* ui : render conversation tab bar in chat layout\n\nDesktop-only tab bar above the chat screen, one tab per open\nconversation or new-chat tab. The active tab follows the route id;\nclicking navigates, middle-click or the close button closes (switching\nto the left neighbor), and a trailing + starts a new chat. Tabs appear\nonly on chat-id routes; the bare #/ new-chat view has none. The bare\nroute stays put unless a prompt/model deep-link routes it to a new-chat\ntab.\n\n* ui : route new-chat entry points through tabs\n\nThe sidebar New chat item, Cmd+Shift+O, the search page and the\narrow-key fallback now open a new-chat tab instead of navigating to the\n?new_chat URL, which is removed. New chat is no longer a special route\nbut a tab like any other conversation.\n\n* ui : track sidebar expanded state in a shared ui store\n\nMove the desktop sidebar expanded/collapsed state out of deviceStore into a\ndedicated uiStore so the chat tab bar can react to it.\n\nAssisted-by: pi\n\n* chat : add opt-in conversation tabs setting\n\nAdd a Display setting that turns browser-style conversation tabs on or off,\nenabled by default.\n\nAssisted-by: pi\n\n* chat : add browser-style conversation tabs with a new-chat screen\n\nTrack open conversations as tabs above the chat, one per open chat, plus a\nsingle New chat tab for the bare `#/` route. New chat is just the `#/`\nscreen - no temporary conversations - and its tab is dropped when navigating\naway. Sending the first message creates a real conversation and opens a tab\nfor it.\n\nAssisted-by: pi\n\n* chat : turn tab bar into a horizontally scrollable carousel\n\nMake the tab bar a horizontally scrollable carousel with edge scroll buttons\nand active-tab centering, and align its styling with the sidebar.\n\nAssisted-by: pi\n\n* chat : restyle the scroll-to-bottom button to match tab styling\n\nAssisted-by: pi\n\n* chat : add close-tab keyboard shortcut\n\nAssisted-by: pi\n\n* chat : soften tab bar fade and dim inactive tabs\n\nAssisted-by: pi\n\n* feat: Add stop button to tabs\n\n* refactor: Componentize\n\n* ui : fix carousel scrollability detection\n\nObserve the content wrapper as well as the container, since adding overflowing items does not change the container's own box size. Also expose an onScrollableChange callback.\n\nAssisted-by: pi\n\n* ui : add unified ScrollCarousel component\n\nSingle carousel component with top/center variants, gap and scroll options, and hover-revealed chevrons. Rename the HorizontalScrollCarousel accessibility story accordingly.\n\nAssisted-by: pi\n\n* ui : migrate carousels to ScrollCarousel\n\nSwitch the settings mobile header, attachments list, thumbnail strip, and MCP resources to the unified component, and drop HorizontalScrollCarousel.\n\nAssisted-by: pi\n\n* ui : improve chat tabs carousel UX\n\nScroll newly added tabs into view, fade overflowing tabs at the edges, and hide the New chat button while a new-chat tab is open.\n\nAssisted-by: pi\n\n* refactor: Naming\n\n* chat : add keyboard shortcut to jump between conversation tabs\n\nShift+Cmd/Ctrl+Left/Right cycles the open tabs, mirroring the existing\nShift+Cmd/Ctrl+Up/Down conversation navigation.\n\nAssisted-by: pi\n\n* chat : make the whole tab item act as a link\n\nThe full tab is now a link instead of only the inner label button, while\nthe stop and close buttons stay interactive by swallowing their clicks.\n\nAssisted-by: pi\n\n* chat : adjust tab bar width and use a shared offset variable\n\nWiden the tab bar for the expanded sidebar and rename the tab bar height\nvariable to --chat-tabs-offset with a smaller value so the chat screen\nmin-height accounts for the overlay without overshooting.\n\nAssisted-by: pi\n\n* chat : account for the tab bar offset in the assistant min-height\n\nSubtract the tab bar offset when it is shown so the last assistant message\ndoes not overflow the available viewport space.\n\nAssisted-by: pi\n\n* refactor: Post-review fixes\n\n* ui : restore deep links on the chat start page\n\n- handle ?model selection, with ?load=true eager router loading\n- ?q now creates a conversation, sends the prompt, and clears the params\n- show the not-available-model dialog for unknown models\n- never block mount on the conversation list\n\nAssisted-by: pi\n\n* ui : fix tab item link nesting and centralize tab constants\n\n- the tab anchor covers the whole item while stop/close stay siblings,\n so interactive elements are never nested inside the anchor\n- cmd/ctrl/middle clicks are left to the browser (new window)\n- extract the tab labels, the active-tab data attribute, and the\n sidebar-offset max widths into constants\n\nAssisted-by: pi\n\n* ui : tidy scroll carousel hook and keep mobile header arrows on\n\n- drop the dead scrollLeft/scrollRight helpers and the unused\n onScrollableChange/scrollBy props\n- init the carousel once instead of inside a derived\n- restore items-start on the center variant\n- always show the settings header arrows on touch\n\nAssisted-by: pi\n\n* ui : keep the new-chat tab across reloads and fall back on close\n\n- the new-chat sentinel is no longer pruned on init, so reloading on\n the bare new-chat route keeps the tab the user is on\n- closing the active conversation falls back to the new-chat screen\n when Conversation tabs are off\n\nAssisted-by: pi\n\n* ui : don't block startup on the conversation list\n\n- prune persisted tabs after the list loads in the background instead\n of awaiting it during init\n- openNewChat now returns void; its return value was never read\n\nAssisted-by: pi\n\n* ui: fix routing nits\n\n* chore: Update doc comments\n\n* refactor: Mark fire-and-forget openNewChat calls as `void`\n\n* chat: fix the deep-linked prompt, the tab width and the tab shortcuts\n\nThe chat start page creates the conversation and hands the prompt over\nto the chat route, which still sees it in the query string. Sending it\non both sides queues the second copy as a pending message, which shows\nup as a stray user bubble once the answer lands and vanishes on reload\nsince it never reaches the database.\n\nThe tab bar takes the max width of the collapsed sidebar while it is\nexpanded, and the other way round.\n\nThe tab list is pruned against a snapshot of the loaded conversations,\nso a conversation created while that list is still loading loses its\ntab even though the route just opened it. The active tab then falls out\nof the list and the cycling shortcut jumps to an edge on every keypress\ninstead of moving one tab over. Tabs synced from the route are kept as\nthey are, only the persisted ones are pruned.\n\nThe rich chat input claims ctrl or alt with shift and an arrow for its\nbadge-aware word jump, which now belongs to the tab cycling shortcut.\nHolding shift hands the key combination over, the plain word jump is\nunchanged.\n\nThe close-tab shortcut consumes the event before checking whether the\nsetting is on, and the logo background loses its importance flag.\n\n---------\n\nCo-authored-by: Pascal ","tree":"6f6c5b445090628326ea7924ca133acb3e0f1e49"}},{"name":"unit","mode":"0040000","size":0,"lastCommit":{"hash":"521a64cd01979bb5b1a466152c576a9d809b068d","oid":"521a64cd01979bb5b1a466152c576a9d809b068d","parents":["681c29d36a13be54d317ee147b272da9163dbef3"],"author":{"name":"Aleksander Grygier","email":"aleksander.grygier@gmail.com","when":"2026-08-20T19:02:04+02:00"},"committer":{"name":"GitHub","email":"noreply@github.com","when":"2026-08-20T19:02:04+02:00"},"message":"ui: Stores split refactor (#27240)\n\n* ui: Extract server stream lifecycle from chatStore into ChatStreamManager\n\nDiscovery, attach/replay, resume retry and the remote-running snapshot\nformed a cohesive cluster inside chatStore. It now lives in\nchat-streams.svelte.ts as ChatStreamManager, owned by chatStore, which\nkeeps the public entry points as delegates so components are\nunchanged. chatStore: 2877 -> 2418 lines.\n\n* ui: Extract user interaction gates from agenticStore into AgenticGates\n\nTool permission requests, turn-limit continue prompts and queued\nsteering messages are the state the loop waits on between turns. They\nhad no coupling to session state, so they now live in\nagentic-gates.svelte.ts; agenticStore keeps delegates so components\nare unchanged. agenticStore: 1196 -> 1073 lines.\n\n* ui: Compose MCP resources under mcpStore.resources\n\nResource state was a second import scope next to mcpStore. Consumers\nnow go through mcpStore.resources, so the MCP surface is one store;\nmcp-resources.svelte.ts stays a separate file owned by mcpStore.\n\n* ui: Reorganize stores into domain namespaces\n\n* fix: Update stale doc comments\n\n* ui: Consolidate conv running-state into a chat activity ledger\n\nRunning-state was split across chatStore.chatLoadingStates (local\npipes), ChatStreamManager.remoteRunningConvs (backend sessions) and\nattachingConvs (attach lifecycle), unioned by hand in\ngetAllLoadingChats and cross-cleaned by setChatLoading calling\nstreams.clearRemoteRunning - the 'spinner ghosts until tab toggle'\nworkaround.\n\nchatActivityStore now owns both sets with one transition per event:\nmarkLocal / localEnded (local pipe end also drops the stale remote\nhint, no cross-owner call) / applyRemoteSnapshot (diffed). The\nsidebar reads chatStore.activity.loadingConvs through the unchanged\ngetAllLoadingChats entry point.\n\nConsequences:\n- isStreamingActive and its five manual writers are gone; isStreaming()\n now reports whether the active conversation has a live streaming\n pipe, which is what all four consumers (assistant row, stop action,\n context gauge, chat screen) actually check\n- isLoading/isReasoning become derived from the per-conv maps plus\n the active conversation, dropping the manual resync in\n syncLoadingStateForChat and clearUIState\n- attachingConvs and the last-attach coordination disappear from\n ChatStreamManager\n- getAllStreamingChats (no consumers) is removed\n\n* ui: Give store collaborators narrow host interfaces\n\nCollaborators took 'host: typeof ', i.e. the store's entire\npublic surface, which is how chatStore's streamChatCompletion,\ncreateAssistantMessage, getApiOptions and setStreamingActive got\nwidened to public. Replace with per-collaborator interfaces carrying\nonly the members each one drives:\n\n- ChatStreamHost (chat/streams) - activity, processing, streaming\n states, abort controller, loading/streaming setters\n- ChatFlowsHost (chat/flows) - streaming core, message creation,\n per-conv state setters\n- McpHealthHost (mcp/health) - connection registry + reconnection\n- ModelPropsHost / ModelStatusHost (models) - model rows, feed\n updates; the managers write modalities/status back onto the host's\n rows, so those members stay writable\n- ConversationsPreferencesHost (conversations) - the active row and\n the conversation list\n\nThe store classes now declare 'implements ' so the contract is\nvisible at the class level, and the 'import type { }' back\nreferences in the collaborators disappear entirely - the host\ncontract is local to each collaborator file, and collaborators can\nno longer reach around their slice. Members stay public (structural\ntyping), but the collaborator side is now compiler-enforced.\n\n* test: Chat Activity store test\n\n* refactor: Cleanup\n\n* chore: Remove legacy architecture docs\n\n* ui: Memoize findMessageIndex for the streaming hot path\n\nStreaming looks up the same message index on every chunk, a linear\nscan of activeMessages each time. Cache the last lookup and reuse it\nafter validating the id still sits at the same position (O(1)); any\nstructural change to the array fails validation and falls back to a\nfull scan.\n\n* ui: Throttle per-chunk stream state writes to localStorage\n\nsaveStreamState ran JSON.stringify + a synchronous localStorage.setItem\non every decoded chunk of the stream. The read loop now goes through a\nnew saveStreamStateThrottled (one write per conversation per 500ms,\nlatest value held pending); the public saveStreamState keeps its\nimmediate-write contract for stream start and pre-fetch, and also\nresets the throttle window.\n\nA pending offset is force-flushed at resume boundaries (resumeStream\nreads the offset back from localStorage), on visibilitychange->hidden\nand on pagehide, so a reload always finds a usable offset. The resume\noffset only needs to be roughly current since the server retransmits\nfrom a line boundary and the client discards its partial line.\n\nAdds unit tests for the throttled/flush/clear interplay.\n\n* ui: Compute context gauge timing stats in one pass\n\ncurrentRead/Fresh/Cache/Output were separate deriveds, each running a\nfull reverse scan of activeMessages for the last assistant timings,\nand cumulative ran its own forward scan plus an agentic filter - 4-5\nO(n) passes per chunk while streaming. Replace with a single\nsummarizeAssistantTimings() pass (last assistant timings, last\nagentic llm totals and the cumulative sums) feeding a shared derived\nsnapshot. Semantics unchanged, including the live-stats overrides and\nthe agentic llm-totals branch.\n\n* agentic : clear session state when a conversation is deleted\n\nEvery conversation that ran an agentic flow left an AgenticSession in the\nstore forever; clearSession was never called. conversationsStore now\nnotifies deletion listeners and agenticStore drops the matching sessions,\navoiding a circular import back into conversationsStore.\n\n* chat : extract ChatService.normalizeMessagesForApi\n\nThe DB->API message normalization (convert + drop empty system messages)\nwas duplicated in sendMessage, preEncode and the agentic flow. Extract it\ninto one shared method and call it from all three.\n\n* sse : share record splitting and data extraction\n\nsplitSseRecords and extractSseDataPayload centralize the record-boundary\nsplitting and data: line extraction used by parseSseJsonStream and the\nmodels status feed. chat.service keeps its own line-based parser for\nresume support.\n\n* api : delegate apiFetchWithParams to apiFetch\n\napiFetchWithParams duplicated apiFetch's headers/fetch/error handling\nbody-for-body; it only differs in URL construction. Build the URL and\ndelegate.\n\n* chat flows : dedupe title, timings and cleanup handling\n\n- conversationsStore.applyTitleFromContent centralizes the title-from-first-\n message logic duplicated in 5 places\n- ChatProcessingStore.applyStreamTimings centralizes the onTimings handler\n shared by the chat and continue flows\n- host.cleanupStreaming centralizes the loading/streaming/processing reset\n repeated across the continue flow's exit paths\n\n* conversations : centralize conversation update mirroring\n\nrename, pin, mcp override, reasoning effort and cwd all repeated the same\nwrite-DB-then-mirror-into-list-and-active dance. A single\napplyConversationUpdate(id, updates) on the host collapses all five and\nremoves the forgot-to-mirror-one-field bug class. Drops the redundant\narray reassignment in setCwd (deep field assignment is reactive).\n\n* mcp : dedupe tool execution, server parsing and tool indexing\n\n- executeTool delegates to executeToolByName (only diff was argument parsing)\n- drop the private #parseServerSettings copy; use parseMcpServerSettings\n- cache getServers() keyed on the raw config value (hot path)\n- indexServerTools() unifies the three identical toolsIndex rebuild loops\n\nAssisted-by: Claude\n\n* mcp : share cursor pagination and tool indexing\n\n- MCPService.paginate() collapses the identical do-while loops in\n listAllResources and listAllResourceTemplates\n- promoteHealthCheckToConnection now uses indexServerTools like the other\n connect paths\n\nAssisted-by: Claude\n\n* database : share message parent-child bookkeeping\n\n- addChildToParent() dedups the append-to-children update in createMessageBranch\n and createSystemMessage\n- removeChildFromParent() dedups the remove-from-children cleanup in deleteMessage\n and deleteMessageCascading\n- bulkAdd the cloned messages when forking a conversation instead of one add\n per message\n\nAssisted-by: Claude\n\n* chore: Lint/format\n\n* fix: `pagehide` event from `window`\n\n* refactor: Api Fetch util\n\n* docs : rewrite architecture sections in README\n\nUpdate the high-level diagram, routes, hooks, stores, services and data\nflow tables to match the current UI structure (mcp/settings/search\nroutes, agentic/tools/mcp stores, MCPService/ToolsService/SandboxService,\n/tools API). Fix stale architectural patterns for per-conversation state\nand modality validation.\n\n* chore : add ESLint rule for blank lines between accessors\n\nEnforce a blank line between consecutive class accessors. The core\npadding-line-between-statements rule does not cover class members, so a\nlocal rule is needed.\n\n* refactor : reorder store members and unify naming\n\nOrder store class members as public fields, private fields, constructor,\ngetters, public methods, then private methods. Normalize private naming\nto the `private` keyword (drop `#` and the `_` prefix where there is no\nmatching public getter). Rename conversationsStore.init() to\ninitialize() to match the other stores.\n\n* refactor : prefix lookup methods with get in agentic and chat stores\n\nUnify bare-name lookup methods with the get* prefix used across the\nother stores (mcp, models, tools, settings). Renames currentTurn,\ntotalToolCalls, lastError, streamingToolCall, executingToolCallId,\npendingPermissionRequest, pendingContinueRequest,\npendingSteeringMessageContent, pendingSteeringMessageExtras in the\nagentic store and pendingMessageContent, pendingMessageExtras in the\nchat store. Updates the two consuming components and a doc comment.\n\n* refactor: Clean up comments in stores' and services' code\n\n* chore : add ESLint rule for class member ordering\n\nEnforce structural order (public fields -> private fields -> constructor ->\ngetters -> setters -> public methods -> private methods) with alphabetical\nsorting within each group via perfectionist/sort-classes. Dependency\ndetection keeps Svelte $derived fields in a valid dependency order instead\nof alphabetizing them, since Svelte rejects forward references.\n\nAssisted-by: Claude\n\n* refactor : reorder class members to match new ESLint rule\n\nApply the sort-classes rule across stores, services, hooks and utils.\nPure reordering - verified no logic changes by comparing sorted line\nmultisets before/after. All tests and svelte-check pass.","tree":"c5664a77330acafaa7748fed43cb7404815ef63a"}}],"lastCommit":{"hash":"8144f3192e5a3131cd043f284525e6ceebf82d0f","oid":"8144f3192e5a3131cd043f284525e6ceebf82d0f","parents":["6657ded4faa3b8450221119fc6b4d002e35104a2"],"author":{"name":"Aleksander Grygier","email":"aleksander.grygier@gmail.com","when":"2026-08-23T10:46:49+02:00"},"committer":{"name":"GitHub","email":"noreply@github.com","when":"2026-08-23T10:46:49+02:00"},"message":"ui: Chat Conversation Tabbed navigation (#27263)\n\n* ui : add browser-style conversation tabs store\n\nTrack open conversation tabs in order, persisted to localStorage and\npruned against the loaded conversation list on init. The chat layout\nsyncs the route's tab on every navigation, so any way of reaching a\nconversation opens a tab for it.\n\n* ui : add temporary new-chat tabs\n\nNew-chat tabs are unsaved conversations carrying a temporary id used\ndirectly as the route (#/chat/). They live in memory and are only\npersisted to the database - keeping the same id so the route and tab\nstay stable - when the first message is sent. Deleting one drops it\nwithout confirmation, and deleting conversations now closes their tabs.\n\n* ui : render conversation tab bar in chat layout\n\nDesktop-only tab bar above the chat screen, one tab per open\nconversation or new-chat tab. The active tab follows the route id;\nclicking navigates, middle-click or the close button closes (switching\nto the left neighbor), and a trailing + starts a new chat. Tabs appear\nonly on chat-id routes; the bare #/ new-chat view has none. The bare\nroute stays put unless a prompt/model deep-link routes it to a new-chat\ntab.\n\n* ui : route new-chat entry points through tabs\n\nThe sidebar New chat item, Cmd+Shift+O, the search page and the\narrow-key fallback now open a new-chat tab instead of navigating to the\n?new_chat URL, which is removed. New chat is no longer a special route\nbut a tab like any other conversation.\n\n* ui : track sidebar expanded state in a shared ui store\n\nMove the desktop sidebar expanded/collapsed state out of deviceStore into a\ndedicated uiStore so the chat tab bar can react to it.\n\nAssisted-by: pi\n\n* chat : add opt-in conversation tabs setting\n\nAdd a Display setting that turns browser-style conversation tabs on or off,\nenabled by default.\n\nAssisted-by: pi\n\n* chat : add browser-style conversation tabs with a new-chat screen\n\nTrack open conversations as tabs above the chat, one per open chat, plus a\nsingle New chat tab for the bare `#/` route. New chat is just the `#/`\nscreen - no temporary conversations - and its tab is dropped when navigating\naway. Sending the first message creates a real conversation and opens a tab\nfor it.\n\nAssisted-by: pi\n\n* chat : turn tab bar into a horizontally scrollable carousel\n\nMake the tab bar a horizontally scrollable carousel with edge scroll buttons\nand active-tab centering, and align its styling with the sidebar.\n\nAssisted-by: pi\n\n* chat : restyle the scroll-to-bottom button to match tab styling\n\nAssisted-by: pi\n\n* chat : add close-tab keyboard shortcut\n\nAssisted-by: pi\n\n* chat : soften tab bar fade and dim inactive tabs\n\nAssisted-by: pi\n\n* feat: Add stop button to tabs\n\n* refactor: Componentize\n\n* ui : fix carousel scrollability detection\n\nObserve the content wrapper as well as the container, since adding overflowing items does not change the container's own box size. Also expose an onScrollableChange callback.\n\nAssisted-by: pi\n\n* ui : add unified ScrollCarousel component\n\nSingle carousel component with top/center variants, gap and scroll options, and hover-revealed chevrons. Rename the HorizontalScrollCarousel accessibility story accordingly.\n\nAssisted-by: pi\n\n* ui : migrate carousels to ScrollCarousel\n\nSwitch the settings mobile header, attachments list, thumbnail strip, and MCP resources to the unified component, and drop HorizontalScrollCarousel.\n\nAssisted-by: pi\n\n* ui : improve chat tabs carousel UX\n\nScroll newly added tabs into view, fade overflowing tabs at the edges, and hide the New chat button while a new-chat tab is open.\n\nAssisted-by: pi\n\n* refactor: Naming\n\n* chat : add keyboard shortcut to jump between conversation tabs\n\nShift+Cmd/Ctrl+Left/Right cycles the open tabs, mirroring the existing\nShift+Cmd/Ctrl+Up/Down conversation navigation.\n\nAssisted-by: pi\n\n* chat : make the whole tab item act as a link\n\nThe full tab is now a link instead of only the inner label button, while\nthe stop and close buttons stay interactive by swallowing their clicks.\n\nAssisted-by: pi\n\n* chat : adjust tab bar width and use a shared offset variable\n\nWiden the tab bar for the expanded sidebar and rename the tab bar height\nvariable to --chat-tabs-offset with a smaller value so the chat screen\nmin-height accounts for the overlay without overshooting.\n\nAssisted-by: pi\n\n* chat : account for the tab bar offset in the assistant min-height\n\nSubtract the tab bar offset when it is shown so the last assistant message\ndoes not overflow the available viewport space.\n\nAssisted-by: pi\n\n* refactor: Post-review fixes\n\n* ui : restore deep links on the chat start page\n\n- handle ?model selection, with ?load=true eager router loading\n- ?q now creates a conversation, sends the prompt, and clears the params\n- show the not-available-model dialog for unknown models\n- never block mount on the conversation list\n\nAssisted-by: pi\n\n* ui : fix tab item link nesting and centralize tab constants\n\n- the tab anchor covers the whole item while stop/close stay siblings,\n so interactive elements are never nested inside the anchor\n- cmd/ctrl/middle clicks are left to the browser (new window)\n- extract the tab labels, the active-tab data attribute, and the\n sidebar-offset max widths into constants\n\nAssisted-by: pi\n\n* ui : tidy scroll carousel hook and keep mobile header arrows on\n\n- drop the dead scrollLeft/scrollRight helpers and the unused\n onScrollableChange/scrollBy props\n- init the carousel once instead of inside a derived\n- restore items-start on the center variant\n- always show the settings header arrows on touch\n\nAssisted-by: pi\n\n* ui : keep the new-chat tab across reloads and fall back on close\n\n- the new-chat sentinel is no longer pruned on init, so reloading on\n the bare new-chat route keeps the tab the user is on\n- closing the active conversation falls back to the new-chat screen\n when Conversation tabs are off\n\nAssisted-by: pi\n\n* ui : don't block startup on the conversation list\n\n- prune persisted tabs after the list loads in the background instead\n of awaiting it during init\n- openNewChat now returns void; its return value was never read\n\nAssisted-by: pi\n\n* ui: fix routing nits\n\n* chore: Update doc comments\n\n* refactor: Mark fire-and-forget openNewChat calls as `void`\n\n* chat: fix the deep-linked prompt, the tab width and the tab shortcuts\n\nThe chat start page creates the conversation and hands the prompt over\nto the chat route, which still sees it in the query string. Sending it\non both sides queues the second copy as a pending message, which shows\nup as a stray user bubble once the answer lands and vanishes on reload\nsince it never reaches the database.\n\nThe tab bar takes the max width of the collapsed sidebar while it is\nexpanded, and the other way round.\n\nThe tab list is pruned against a snapshot of the loaded conversations,\nso a conversation created while that list is still loading loses its\ntab even though the route just opened it. The active tab then falls out\nof the list and the cycling shortcut jumps to an edge on every keypress\ninstead of moving one tab over. Tabs synced from the route are kept as\nthey are, only the persisted ones are pruned.\n\nThe rich chat input claims ctrl or alt with shift and an arrow for its\nbadge-aware word jump, which now belongs to the tab cycling shortcut.\nHolding shift hands the key combination over, the plain word jump is\nunchanged.\n\nThe close-tab shortcut consumes the event before checking whether the\nsetting is on, and the logo background loses its importance flag.\n\n---------\n\nCo-authored-by: Pascal ","tree":"6f6c5b445090628326ea7924ca133acb3e0f1e49"}}