Skip to content

feat(frontend): show workspace name in sidebar#1150

Open
patjlm wants to merge 2 commits intoambient-code:mainfrom
patjlm:feat/sidebar-workspace-name
Open

feat(frontend): show workspace name in sidebar#1150
patjlm wants to merge 2 commits intoambient-code:mainfrom
patjlm:feat/sidebar-workspace-name

Conversation

@patjlm
Copy link
Copy Markdown
Contributor

@patjlm patjlm commented Apr 2, 2026

Summary

  • Displays the current workspace name next to the "< Workspaces" back-link in the sidebar
  • Users can always see which workspace they are in without navigating away

Test plan

  • Run make kind-up LOCAL_IMAGES=true
  • Run make dev-env to generate .env.local with token
  • Run make dev COMPONENT=frontend and navigate to a workspace
  • Verify workspace name appears on the Workspaces nav line
  • Verify long names are truncated with tooltip on hover
  • Verify clicking the link still navigates back to workspace list

🤖 Generated with Claude Code

Display the current workspace name next to the Workspaces back-link
in the sidebar, so users always know which workspace they are in.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 2, 2026

📝 Walkthrough

Walkthrough

The sessions sidebar workspace navigation button is reorganized to display the project name on the right side with text truncation, switching from left-aligned to space-between layout alignment.

Changes

Cohort / File(s) Summary
Sessions Sidebar UI Layout
components/frontend/src/app/projects/[name]/sessions/[sessionName]/components/sessions-sidebar.tsx
Updated workspace navigation button layout: changed from left-aligned (justify-start) to space-between (justify-between), added project name display on right side with truncation (max-w-[60%], truncate) and tooltip support.
🚥 Pre-merge checks | ✅ 5 | ❌ 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 (5 passed)
Check name Status Explanation
Title check ✅ Passed Title follows Conventional Commits format (feat(frontend): ...) and accurately describes the main change: adding workspace name display in the sidebar navigation.
Performance And Algorithmic Complexity ✅ Passed PR introduces minor UI enhancement displaying workspace name with native CSS truncation and HTML title attribute—no performance impact.
Security And Secret Handling ✅ Passed Pull request adds safe frontend-only UI enhancement displaying workspace name in sidebar via React JSX (XSS-safe). Maintains authentication checks, no secrets or injection vulnerabilities introduced.
Kubernetes Resource Safety ✅ Passed Kubernetes Resource Safety check not applicable; PR modifies only frontend React component without infrastructure resources.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
✨ Simplify code
  • Create PR with simplified code

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.

@ambient-code ambient-code bot added this to the Review Queue milestone Apr 2, 2026
@Gkrumbach07
Copy link
Copy Markdown
Contributor

@patjlm can you attach a screenshot

@Gkrumbach07
Copy link
Copy Markdown
Contributor

@amber what does this look like, can you describe it in detail

1 similar comment
@Gkrumbach07
Copy link
Copy Markdown
Contributor

@amber what does this look like, can you describe it in detail

@patjlm
Copy link
Copy Markdown
Contributor Author

patjlm commented Apr 3, 2026

here is a screenshot

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants