fix(frontend/copilot): hide New Chat button on Autopilot homepage by Otto-AGPT · Pull Request #12321 · Significant-Gravitas/AutoGPT

Otto-AGPT · 2026-03-06T15:34:01Z

Requested by @0ubbe

The New Chat button was visible on the Autopilot homepage where clicking it has no effect (since sessionId is already null). This hides the button when no chat session is active, so it only appears when the user is viewing a conversation and wants to start a new one.

Changes:

ChatSidebar.tsx — hide button in both collapsed and expanded sidebar states when sessionId is null
MobileDrawer.tsx — same fix for mobile drawer

Co-authored-by: Ubbe ubbe@users.noreply.github.com

The button has no effect when already on the homepage (sessionId is null). Hide it in both desktop sidebar (collapsed + expanded) and mobile drawer, so it only appears when viewing a chat session.

github-actions · 2026-03-06T15:34:30Z

🔍 PR Overlap Detection

This check compares your PR against all other open PRs targeting the same branch to detect potential merge conflicts early.

🟢 Low Risk — File Overlap Only

These PRs touch the same files but different sections (click to expand)

chore(frontend): Fix react-doctor warnings + add CI job #12163 (0ubbe · updated 3d ago)
- Shared files: autogpt_platform/frontend/src/app/(platform)/copilot/components/ChatSidebar/ChatSidebar.tsx

Summary: 0 conflict(s), 0 medium risk, 1 low risk (out of 1 PRs with file overlap)

Auto-generated on push. Ignores: openapi.json, lock files.

coderabbitai · 2026-03-06T15:34:32Z

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: f632c61a-573c-4ad2-afbf-68df98d2d58f

📥 Commits

Reviewing files that changed from the base of the PR and between 6fcba1e and 8f402ee.

📒 Files selected for processing (2)

autogpt_platform/frontend/src/app/(platform)/copilot/components/ChatSidebar/ChatSidebar.tsx
autogpt_platform/frontend/src/app/(platform)/copilot/components/MobileDrawer/MobileDrawer.tsx

🚧 Files skipped from review as they are similar to previous changes (2)

autogpt_platform/frontend/src/app/(platform)/copilot/components/ChatSidebar/ChatSidebar.tsx
autogpt_platform/frontend/src/app/(platform)/copilot/components/MobileDrawer/MobileDrawer.tsx

📜 Recent review details

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (5)

GitHub Check: types
GitHub Check: end-to-end tests
GitHub Check: Check PR Status
GitHub Check: Analyze (typescript)
GitHub Check: Analyze (python)

Walkthrough

"New Chat" buttons in ChatSidebar and MobileDrawer are now rendered only when a session ID is present; previously they were always rendered.

Changes

Cohort / File(s)	Summary
Conditional New Chat Button Rendering `autogpt_platform/frontend/src/app/(platform)/copilot/components/ChatSidebar/ChatSidebar.tsx`, `autogpt_platform/frontend/src/app/(platform)/copilot/components/MobileDrawer/MobileDrawer.tsx`	Replaced unconditional "New Chat" button renders with guarded conditional rendering that shows the button only when a session ID (`sessionId` or `currentSessionId`) is truthy.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

feat(chat): Add delete chat session endpoint and UI #12112: Touches ChatSidebar New Chat rendering logic and related UI around the button.
fix(frontend): more copilot refinements #11858: Modifies MobileDrawer and copilot sidebar flows related to New Chat and session handling.

Suggested labels

size/s

Suggested reviewers

Bentlybro
kcze

Poem

🐇 I peek and press when sessions glow,
But hide my paw when none will show.
A silent tap, a cautious cheer—
New chats begin when context's near. ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'fix(frontend/copilot): hide New Chat button on Autopilot homepage' directly and clearly summarizes the main change: hiding the New Chat button on the Autopilot homepage.
Description check	✅ Passed	The description is directly related to the changeset, explaining why the New Chat button should be hidden (no effect when sessionId is null) and detailing the files modified and the conditional rendering fix applied.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings (stacked PR)
📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

Create PR with unit tests
Post copyable unit tests in a comment
Commit unit tests in branch otto/hide-new-chat-homepage

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

autogpt-reviewer

PR #12321 — fix(frontend/copilot): hide New Chat button on Autopilot homepage
Author: Otto-AGPT (requested by @0ubbe) | Files: ChatSidebar.tsx (+21/-17), MobileDrawer.tsx (+13/-11)

🎯 Verdict: APPROVE

What This PR Does

Hides the "New Chat" button in the copilot sidebar (both collapsed and expanded states) and mobile drawer when no chat session is active (sessionId is null). Previously, the button was always visible but clicking it on the homepage was a no-op since handleNewChat calls setSessionId(null), which is already the homepage state.

Specialist Findings

🛡️ Security ✅ — No security concerns. Pure UI visibility toggle, no auth/authz changes, no user input rendered, no new data flows.

🏗️ Architecture ✅ — Clean state coupling. ChatSidebar reads from useQueryState, MobileDrawer receives via prop. Idiomatic {sessionId && (...)} pattern placed at the correct level. No new coupling or tech debt.

⚡ Performance ✅ — Net-positive micro-optimization. Fewer DOM nodes on homepage. No new state, effects, or subscriptions introduced.

🧪 Testing ✅ — No blockers. Zero test baseline for both components is pre-existing debt, not introduced here. All CI checks pass (lint, types, e2e, integration). Suggest adding a basic render test for MobileDrawer in a follow-up.

📖 Quality ✅ — Clean, idiomatic React. Consistent naming (sessionId vs currentSessionId appropriate for their contexts). Good detail: MobileDrawer wraps the mt-2 div inside the conditional to avoid empty margin.

📦 Product ✅ — Correctly removes a dead button. One UX suggestion: consider disabled state with tooltip ("Start a conversation first") instead of hiding, especially for the expanded desktop sidebar, to preserve discoverability for new users. Not blocking.

📬 Discussion ✅ — No unresolved comments. CodeRabbit approved with no actionable items. Low-risk file overlap with PR #12163 (different sections). No human reviews yet — REVIEW_REQUIRED status is process, not a code concern.

🔎 QA ⚠️ — Static review only (live environment unavailable due to Docker Hub rate limits). All three button instances verified correct: collapsed sidebar, expanded sidebar, mobile drawer. No layout shift risks — flexbox gap collapses naturally, mt-2 wrapper also hidden. Conditional variables are correct types (string | null, no empty-string edge case). Recommend manual verification on a running instance.

Blockers

None.

Should Fix (Follow-up OK)

MobileDrawer.tsx — Add basic render test covering currentSessionId null/non-null states (easy win, ~15 lines)
ChatSidebar.tsx — Consider disabled+tooltip instead of hidden for the expanded sidebar "New Chat" button to preserve discoverability

Risk Assessment

Merge risk: LOW | Rollback: EASY

The change is 2 conditional wraps on existing UI elements with no logic changes, no API changes, and no new dependencies. All CI green.

@ntindle Clean, minimal fix that removes a no-op button from the copilot homepage. All 8 specialists approve with zero blockers — safe to merge.

fix(frontend/copilot): hide New Chat button on Autopilot homepage

6fcba1e

The button has no effect when already on the homepage (sessionId is null). Hide it in both desktop sidebar (collapsed + expanded) and mobile drawer, so it only appears when viewing a chat session.

Otto-AGPT requested a review from a team as a code owner March 6, 2026 15:34

Otto-AGPT requested review from 0ubbe, Bentlybro and kcze and removed request for a team March 6, 2026 15:34

github-project-automation bot added this to AutoGPT development kanban Mar 6, 2026

github-project-automation bot moved this to 🆕 Needs initial review in AutoGPT development kanban Mar 6, 2026

github-actions bot added platform/frontend AutoGPT Platform - Front end size/m labels Mar 6, 2026

github-project-automation bot added this to Frontend Mar 6, 2026

autogpt-reviewer reviewed Mar 7, 2026

View reviewed changes

refactor: use ternary operator instead of && for conditional rendering

8f402ee

kcze approved these changes Mar 7, 2026 •

edited

Loading

View reviewed changes

github-project-automation bot moved this from 🆕 Needs initial review to 👍🏼 Mergeable in AutoGPT development kanban Mar 7, 2026

0ubbe merged commit 0bbb12d into dev Mar 9, 2026
23 checks passed

0ubbe deleted the otto/hide-new-chat-homepage branch March 9, 2026 14:41

github-project-automation bot moved this from 👍🏼 Mergeable to ✅ Done in AutoGPT development kanban Mar 9, 2026

github-project-automation bot moved this to Done in Frontend Mar 9, 2026

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix(frontend/copilot): hide New Chat button on Autopilot homepage#12321

fix(frontend/copilot): hide New Chat button on Autopilot homepage#12321
0ubbe merged 2 commits intodevfrom
otto/hide-new-chat-homepage

Otto-AGPT commented Mar 6, 2026

Uh oh!

github-actions bot commented Mar 6, 2026 •

edited

Loading

Uh oh!

coderabbitai bot commented Mar 6, 2026 •

edited

Loading

❌ Failed checks (1 warning)

Uh oh!

autogpt-reviewer left a comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Conversation

Otto-AGPT commented Mar 6, 2026

Uh oh!

github-actions bot commented Mar 6, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🔍 PR Overlap Detection

🟢 Low Risk — File Overlap Only

Uh oh!

coderabbitai bot commented Mar 6, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Estimated code review effort

Possibly related PRs

Suggested labels

Suggested reviewers

Poem

❌ Failed checks (1 warning)

Uh oh!

autogpt-reviewer left a comment

Choose a reason for hiding this comment

What This PR Does

Specialist Findings

Blockers

Should Fix (Follow-up OK)

Risk Assessment

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

github-actions bot commented Mar 6, 2026 •

edited

Loading

coderabbitai bot commented Mar 6, 2026 •

edited

Loading