Uh oh!
There was an error while loading. Please reload this page.
feat: add pear logo in sidebar header - #1
Merged
Merged
Conversation
7 tasks
miyaontherelay added a commit
that referenced
this pull request
Jun 8, 2026
…lush Fix#1/#13: initIfReady schedules a requestAnimationFrame retry when the container has no layout (hidden tab, split page mounted at display:none). mount() now always invokes initIfReady regardless of prior parentage so the retry actually fires once the container becomes visible. Fix#6: dispose() cancels the init rAF and calls clearPtyBuffer(key) before flipping the disposed flag, so a queued rAF flush can't write into a torn-down xterm. writeFromBuffer also early-returns when disposed as belt-and-braces.
miyaontherelay added a commit
that referenced
this pull request
Jun 8, 2026
…lush Fix#1/#13: initIfReady schedules a requestAnimationFrame retry when the container has no layout (hidden tab, split page mounted at display:none). mount() now always invokes initIfReady regardless of prior parentage so the retry actually fires once the container becomes visible. Fix#6: dispose() cancels the init rAF and calls clearPtyBuffer(key) before flipping the disposed flag, so a queued rAF flush can't write into a torn-down xterm. writeFromBuffer also early-returns when disposed as belt-and-braces.
miyaontherelay added a commit
that referenced
this pull request
Jun 8, 2026
…stic on reconcile; drop window-focus auto-focus Three fixes from reviewer-indep's NO-GO cold-eyes pass: 1. terminal-runtime-registry: detach(token) now calls cancelPendingInit(). The recursive initIfReady rAF retry loop (Fix#1 for hidden split-page init) was not being cancelled when a runtime was parked. In split mode, any non-visible display:none split page would spin a permanent rAF against a disconnected container. Now matched-token detach drains it. 2. agent-store: reconcileChatMessages detects the optimistic-vs-canonical human-echo case by (channel, body, project, time-window) and replaces the optimistic local-UUID record with the canonical broker-event-id record instead of appending both. This closes the duplicate-chat path left open after the view-layer dedupe removal — the store-level isDuplicateHumanEcho only guarded the optimistic-side append; the reconcile path appended unconditionally on id miss. 3. use-terminal: removed the window-focus handler's auto-focus on alt-tab back to pear. Same DECSET ?1004 emission class as the visibility-effect focus removed in 701e2a7. Alt-tabbing is less frequent than tab-switching but in the same bug class. User-initiated clicks still focus via the pointerdown handler. Gates: tsc clean on all three files; npm run build clean. Note: the diagnostic instrumentation pushed in 5707dc7 is kept in this commit pending the user's capture data for the per-/mcp-invocation double-render (which neither the focus fix nor these three address — that's still unaccounted-for and the instrumentation is how we find it).
4 tasks
This was referenced Jul 17, 2026
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 freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary\n- add pear logo SVG asset\n- render logo at the top-left of the sidebar header next to Workspaces\n\n## Testing\n- npm run build\n- npm run dev\n