Skip to content

fix(chat): keep the composer branch footer from jumping - #294

Open
oxwen11 wants to merge 5 commits into
mainfrom
port/pie-55-branch-footer
Open

fix(chat): keep the composer branch footer from jumping#294
oxwen11 wants to merge 5 commits into
mainfrom
port/pie-55-branch-footer

Conversation

@oxwen11

@oxwen11 oxwen11 commented Aug 29, 2026

Copy link
Copy Markdown
Member

Summary

Test plan

  • Open a session in a git workspace: footer slot is reserved immediately, then the branch name appears without a layout jump
  • Open a session whose cwd is not a git repo: reserved slot stays empty after the probe

Made with Cursor

oxwen11 and others added 2 commits August 29, 2026 12:33
@oxwen11 @cursoragent
feat(chat): show the workspace branch under the composer
942c95b 
Sessions in a git work tree had no branch cue next to the prompt;
probe the current HEAD and render it in the composer footer.

Co-authored-by: Cursor <cursoragent@cursor.com>
@oxwen11 @cursoragent
fix(chat): keep the composer branch footer from jumping
31126a9 
The footer appeared only after the git probe landed, so the composer
shifted; reserve the slot and prefetch the branch with the session.

Co-authored-by: Cursor <cursoragent@cursor.com>
@github-actions

github-actions Bot commented Aug 29, 2026 •
edited
Loading

Copy link
Copy Markdown

React Doctor found 1 issue in 1 file · 1 warning · score 92 / 100 (Great) · full project

1 warning

src/features/chat/runtime/client-prompt-queue.ts

  • ⚠️ L0 unused-file

Reviewed by React Doctor for commit 49b898f. See inline comments for fixes.

oxwen11 and others added 2 commits August 29, 2026 12:50
@oxwen11 @cursoragent
Query the session branch from the composer, like pie #52.
f67c77d 
The footer still hides when git.branch has no current name; Chat only
forwards the route-resolved cwd because vibest's RPC keys on path, not sessionRef.

Co-authored-by: Cursor <cursoragent@cursor.com>
@oxwen11 @cursoragent
Keep the branch query in the composer while reserving footer height.
2c14b07 
pie #55 still owns git.branch inside the composer and pulses on isPending;
prefetch stays fire-and-forget so prepare is not blocked on the project list.

Co-authored-by: Cursor <cursoragent@cursor.com>
Base automatically changed from port/pie-52-composer-branch to main August 29, 2026 17:03
@oxwen11 @cursoragent
Merge origin/main into port/pie-55-branch-footer.
49b898f 
Keep the reserved-height composer footer from #55; main already has #293's
hide-when-empty footer, which this PR is meant to replace.

Co-authored-by: Cursor <cursoragent@cursor.com>
@oxwen11 oxwen11 mentioned this pull request Aug 30, 2026
Open
3 of 4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@oxwen11