Add project rename support in the sidebar#1798
Conversation
- Inline-edit project names from the sidebar context menu - Handle project.meta-updated events in store state
ApprovabilityVerdict: Needs human review This PR adds a new user-facing feature (project rename in sidebar) with new API integration to dispatch 'project.meta.update' commands. While the implementation follows existing patterns for thread renaming and is well-contained, new features introducing user-facing behavior warrant human review to verify the intended UX and API contract. You can customize Macroscope's approvability policy. Learn more. |
(cherry picked from commit a221542)
|
Thank you for this, having an option to open file location would be nice as well. I will create a PR for this |
…threadId (#2) * Raise slow RPC ack warning threshold to 15s (pingdotgg#1760) * Use active worktree path for workspace saves (pingdotgg#1762) * Stream git status updates over WebSocket (pingdotgg#1763) Co-authored-by: codex <codex@users.noreply.github.com> * fix(web): unwrap windows shell command wrappers (pingdotgg#1719) * Rename "Chat" to "Build" in interaction mode toggle (pingdotgg#1769) Co-authored-by: Julius Marminge <julius0216@outlook.com> * Assign default capabilities to Codex custom models (pingdotgg#1793) * Add project rename support in the sidebar (pingdotgg#1798) * Support multi-select pending user inputs (pingdotgg#1797) * Add Zed support to Open actions via editor command aliases (pingdotgg#1303) Co-authored-by: codex <codex@users.noreply.github.com> Co-authored-by: Julius Marminge <julius0216@outlook.com> * Closes pingdotgg#1795 - Support building and developing in a devcontainer (pingdotgg#1791) * Add explicit timeouts to CI and release workflows (pingdotgg#1825) * fix(web): distinguish singular/plural in pending action submit label (pingdotgg#1826) * Refactor web stores into atomic slices ready to split ChatView (pingdotgg#1708) * Add VSCode Insiders and VSCodium icons (pingdotgg#1847) * Prepare datamodel for multi-environment (pingdotgg#1765) Co-authored-by: justsomelegs <145564979+justsomelegs@users.noreply.github.com> Co-authored-by: codex <codex@users.noreply.github.com> Co-authored-by: Cursor Agent <cursoragent@cursor.com> Co-authored-by: cursor[bot] <206951365+cursor[bot]@users.noreply.github.com> * Implement server auth bootstrap and pairing flow (pingdotgg#1768) Co-authored-by: codex <codex@users.noreply.github.com> Co-authored-by: Julius Marminge <julius@macmini.local> Co-authored-by: Cursor Agent <cursoragent@cursor.com> Co-authored-by: cursor[bot] <206951365+cursor[bot]@users.noreply.github.com> * Use dev proxy for loopback auth and environment requests (pingdotgg#1853) * Refresh local git status on turn completion (pingdotgg#1821) Co-authored-by: codex <codex@users.noreply.github.com> * fix(desktop): add Copy Link action for chat links (pingdotgg#1835) * fix: map runtime modes to correct permission levels (pingdotgg#1587) Co-authored-by: Julius Marminge <julius0216@outlook.com> Co-authored-by: codex <codex@users.noreply.github.com> * Fix persisted composer image hydration typo (pingdotgg#1831) * Clarify environment and workspace picker labels (pingdotgg#1854) * Scope git toast state by thread ref (pingdotgg#1855) * fix build (pingdotgg#1859) * Stabilize keybindings toast stream setup (pingdotgg#1860) Co-authored-by: Julius Marminge <julius@macmini.local> * feat(web): add embeddable thread route for canvas tile hosts Adds /embed/thread/:environmentId/:threadId — a standalone route that renders the existing ChatView without the app sidebar chrome. This is the iframe target for t3-canvas agent shapes (see rororowyourboat/t3-canvas#3). - New file-based route embed.thread.\$environmentId.\$threadId.tsx - __root.tsx bypasses AppSidebarLayout for any /embed/* pathname so the environment connection + websocket surface + toasts still initialize but the sidebar/diff/plan chrome does not render - minimal=1 search param is parsed and wired to a data attribute on the container for future targeted CSS; chrome hiding (BranchToolbar, PlanSidebar, ThreadTerminalDrawer) stays as a follow-up pass - routeTree.gen.ts regenerated by the @tanstack/router-plugin --------- Co-authored-by: Julius Marminge <julius0216@outlook.com> Co-authored-by: codex <codex@users.noreply.github.com> Co-authored-by: legs <145564979+justsomelegs@users.noreply.github.com> Co-authored-by: sonder <168988030+heysonder@users.noreply.github.com> Co-authored-by: Adem Ben Abdallah <96244394+AdemBenAbdallah@users.noreply.github.com> Co-authored-by: Kyle Gottfried <6462596+Spitfire1900@users.noreply.github.com> Co-authored-by: Jacob <589761+jvzijp@users.noreply.github.com> Co-authored-by: Cursor Agent <cursoragent@cursor.com> Co-authored-by: cursor[bot] <206951365+cursor[bot]@users.noreply.github.com> Co-authored-by: Julius Marminge <julius@macmini.local> Co-authored-by: Klemencina <56873773+Klemencina@users.noreply.github.com> Co-authored-by: Oskar Sekutowicz <me.oski646@gmail.com> Co-authored-by: Noxire <59626436+noxire-dev@users.noreply.github.com>
|
This is merged but I can't find the feature in either web or desktop interface (using 0.0.17). Didn't see any explicit revert in any commit messages on the Sidebar.tsx file afterwards. Is this an unintentional revert or did I misunderstand something @juliusmarminge ? |
Closes #1370
Summary
project.meta.updateand clears rename state on blur, Enter, or Escape.project.meta-updatedrefreshes the existing project name and timestamp.Testing
bun fmtbun lintbun typecheckbun run testNote
Low Risk
Low risk UI/state changes that add a new
project.meta.updatecommand dispatch path; main risk is minor rename-state edge cases (blur/escape/enter) affecting sidebar interactions.Overview
Adds inline project rename in the sidebar: a new "Rename project" context-menu action swaps the project title for an input, then commits on Enter/blur (or cancels on Escape) by dispatching
project.meta.update.Refactors thread renaming to pass explicit callbacks (
onRenamingInputMount,hasRenameCommitted,markRenameCommitted) instead of exposing mutable refs, and ensures starting a rename clears any active rename state for the other entity (project vs thread).Extends store tests to cover
project.meta-updatedupdating an existing project’snameandupdatedAt.Reviewed by Cursor Bugbot for commit fe56ef0. Bugbot is set up for automated code reviews on this repo. Configure here.
Note
Add inline project rename support in the sidebar
project.meta.updatecommand via the native API; on Escape, cancels without saving. Shows a toast on error.SidebarThreadRowfrom ref-based to function-based (onRenamingInputMount,hasRenameCommitted,markRenameCommitted).project.meta-updatedevents correctly update project title andupdatedAtin state.Macroscope summarized fe56ef0.