Uh oh!
There was an error while loading. Please reload this page.
Akhil/unify tool sidebar - #320
Merged
Merged
Conversation
…redicate Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01H3GuKxASDXLwFCLA7o3GGH
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01H3GuKxASDXLwFCLA7o3GGH
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01H3GuKxASDXLwFCLA7o3GGH
…abindex Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01H3GuKxASDXLwFCLA7o3GGH
… the icon rail Collapsing used to unmount the panel, which discarded the body's search text, expanded tree groups and scroll position every time. It also made a rail impossible: entries are published by hooks inside the body, and an unmounted body cannot publish. Hide with display:none instead, mirroring how TabContent keeps inactive tool tabs alive. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01H3GuKxASDXLwFCLA7o3GGH
…d rail password-manager, api-keys, to-do and environment-manager gain a populated rail with no edits of their own — the facets are already declared as filter items. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01H3GuKxASDXLwFCLA7o3GGH
…the rail notes and snippet-manager publish pinned items, api-client its collections, data-explorer and s3-drive their connections, bookmarks and secure-files their root folders. Also fixes secure-files' folder tree calling panel.close() unconditionally on select, which collapsed the whole sidebar on every folder pick on desktop — every other sidebar guards that with isOverlay. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01H3GuKxASDXLwFCLA7o3GGH
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01H3GuKxASDXLwFCLA7o3GGH
…ollapse Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01H3GuKxASDXLwFCLA7o3GGH
… NUL byte The "overview showing, nothing selected" sentinel was a literal NUL inside a string literal in the source. Same value, but the byte made the whole file binary to grep, ripgrep and diff, which then skip it without saying so. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01H3GuKxASDXLwFCLA7o3GGH
…tives Shipped unused. They give the next per-tool sidebar migration somewhere to land without that migration also having to design the pieces. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01H3GuKxASDXLwFCLA7o3GGH
The hero greeted off `useAuth().user.displayName`, which is a hardcoded null — that identity object is deliberately name-free, and the editable profile name lives in local preferences behind `useAppUser` (as top-bar and mobile-nav already read it). So the greeting never named anyone, however the profile was filled in. Reads the profile name instead, via a `greetingFirstName` helper that is unit tested. The now-unused `user` prop is dropped from DashboardHero. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01H3GuKxASDXLwFCLA7o3GGH
The vault gate sat in page.tsx, which is this layout's children — so a locked vault swapped the editor for the placeholder while the sidebar column kept rendering a note tree, a search field and a New note button, all backed by a vault with no key loaded. Gate in the layout instead, above NotesProvider so the provider does not mount and fail to load notes it cannot decrypt. password-manager, api-keys, environment-manager, secure-files and snippet-manager already gate before the component that owns their ToolSidebarLayout; notes differed only because its layout is a separate route-level wrapper. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01H3GuKxASDXLwFCLA7o3GGH
Contributor
The latest updates on your projects. Learn more about Vercel for GitHub.
|
ThreatCrush Security Scan1 finding(s) MEDIUM: 1
Snippets are redacted; ThreatCrush never prints matched credential material. |
Uh oh!
There was an error while loading. Please reload this page.
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
1 participant
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.
No description provided.