Change body background to sidebar color

[dimaMachina]

just try to overscroll

before

dark	light

after

dark	light

[changeset-bot]

🦋 Changeset detected

Latest commit: 2f7a10e

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 9 packages

Name	Type
@inkeep/agents-manage-ui	Patch
@inkeep/agents-manage-api	Patch
@inkeep/agents-run-api	Patch
@inkeep/agents-cli	Patch
@inkeep/agents-core	Patch
@inkeep/agents-manage-mcp	Patch
@inkeep/agents-sdk	Patch
@inkeep/ai-sdk-provider	Patch
@inkeep/create-agents	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Review	Updated (UTC)
agents-docs	Ready	Preview, Comment	Jan 6, 2026 0:02am
agents-manage-api	Ready	Preview, Comment	Jan 6, 2026 0:02am
agents-manage-ui	Ready	Preview, Comment	Jan 6, 2026 0:02am
agents-run-api	Ready	Preview, Comment	Jan 6, 2026 0:02am

[claude]

Claude finished @dimaMachina's task —— View job

---

Code Review: Change body background to sidebar color

Change Summary: The PR changes the body background from bg-background to bg-sidebar in globals.css:210, creating visual consistency by making the body background match the sidebar color.

Visual Impact: The before/after screenshots clearly demonstrate the improvement - the overscroll area now matches the sidebar color instead of showing a jarring contrast, providing a more polished user experience.

✅ Strengths

1. Visual Consistency: The change elegantly solves the overscroll UX issue by ensuring the body background matches the sidebar color
2. Simple & Focused: Single line change with clear intent and minimal risk
3. Theme Support: Works correctly with both light and dark themes since --sidebar is properly defined for both
4. Design System Adherence: Uses existing design tokens (--sidebar) rather than introducing new values

⚠️ Architecture & Design Considerations

Medium Priority - Layout Architecture Concern:

The change reveals a potential architectural inconsistency. Looking at the layout structure:

// Root layout.tsx applies bg-sidebar to body
body { @apply bg-sidebar text-foreground; }

// SidebarInset component applies bg-background
function SidebarInset() {
  return <main className="bg-background relative flex..." />
}

This creates a layered background pattern where:

• body = bg-sidebar (now matches sidebar color)
• SidebarInset = bg-background (main content area)

Potential Issues:

1. Semantic Confusion: The body background now represents "sidebar color" semantically, which may be confusing for future developers
2. Future Layout Changes: If the layout structure changes (e.g., different sidebar implementations, full-screen modes), this assumption may break
3. Maintenance: Future developers might not understand why body has sidebar colors

Suggested Improvements (not blocking, but worth considering):

1. Add Documentation: Consider adding a CSS comment explaining this design decision:

   body {
     /* Use sidebar color for body to ensure smooth overscroll experience */
     @apply bg-sidebar text-foreground;
   }

2. Consider Alternative Approaches:

   • Using a wrapper div with bg-sidebar around the sidebar+content area
   • CSS body::before pseudo-element for background consistency
   • Layout-specific background handling

🔍 Technical Review

• CSS Variable Usage: ✅ Correctly uses existing --sidebar design token
• Theme Support: ✅ Works with light/dark themes (verified --sidebar definitions exist)
• Specificity: ✅ No specificity conflicts introduced
• Performance: ✅ No performance impact

📋 Process & Standards

Missing Changeset: The PR needs a changeset for the @inkeep/agents-manage-ui package as indicated by changeset-bot.

✅ Recommendation

APPROVE - This is a good UX improvement with minimal risk. The change effectively solves the overscroll visual issue and follows existing design system patterns. While there are architectural considerations worth noting for future development, they don't block this change.

Required Action: Add the missing changeset before merging.

[sarah-inkeep]

hm I think this will change the background color for all pages though, for example the login / onboarding flow pages should have the main background color I think.