🦋 Changeset detected

Latest commit: 464b1be

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

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

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

Project	Deployment	Actions	Updated (UTC)
agents-api	Ready	Preview, Comment	Mar 25, 2026 0:08am
agents-manage-ui	Ready	Preview, Comment	Mar 25, 2026 0:08am

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
agents-docs	Skipped		Mar 25, 2026 0:08am

PR Review Complete

• Analyze PR context and diff
• Dispatch domain-specific reviewers
• Consolidate findings and submit review

Result: ✅ Approved — Clean performance optimization that removes dead code (tooltip was never visible due to hover-expand behavior). One minor suggestion posted as inline comment.

View job run

TL;DR — Removes tooltip rendering from sidebar menu items and simplifies the NavItem component by dropping unused props (isExternal, optional icon) and inlining the isActive check.

Key changes

• Remove tooltip prop from SidebarMenuButton — sidebar menu items no longer render tooltips on hover, which were redundant since the item titles are always visible.
• Simplify NavItem props and markup — removes isExternal prop (and its target/rel attributes), makes icon required, and inlines the isActive computation. The <span> around the title is retained to prevent layout issues during sidebar collapse.
• Convert .map() to expression body in NavGroup — replaces the block-body arrow with implicit return for cleaner iteration.
• Add changeset for @inkeep/agents-manage-ui — patch-level changeset documenting the tooltip removal.

_{Summary ｜ 3 files ｜ 5 commits ｜ base: main ← tooltip-sidebar}

Before: NavItem accepted optional icon and isExternal props, stored isActive in a variable, and passed tooltip={title} to SidebarMenuButton.
After: icon is required, isExternal is removed entirely, isActive is inlined, and tooltips are no longer emitted. The title <span> is preserved with an explanatory comment to avoid layout shifts when the sidebar collapses.

The sidebar nav items always display their label inline — the tooltip was duplicating already-visible text. Removing isExternal reflects that no sidebar links currently point outside the app.

nav-item.tsx · nav-group.tsx · remove-sidebar-tooltips.md

  ^{｜ View workflow run ｜ Triggered by Pullfrog ｜ pullfrog.com ｜ 𝕏}

@claude add a changeset based on changes in this PR

Claude finished @dimaMachina's task in 1m 1s —— View job

Adding Changeset

• Analyze PR changes
• Create changeset

Added changeset remove-sidebar-tooltips.md for @inkeep/agents-manage-ui (patch): "Remove unused tooltip rendering from sidebar nav items to improve performance"
|

@dimaMachina This is amazing!

I’ve been trying a lot of ways since morning to reduce the tooltip renders in the sidebar, but I couldn’t find a proper solution.

I think the best approach is to remove them, since the sidebar expands on hover, so tooltips aren’t needed.

Sometimes the best optimization is simply removing things.

Ito Test Report ❌

19 test cases ran. 2 failed, 17 passed.

Overall, 19 sidebar/navigation test cases were executed with 17 passing and 2 failing, showing broad stability across org and project route transitions, nested deep-link active-state consistency, tooltip-free behavior in collapsed/expanded/mobile contexts, icon rendering without runtime errors, auth-proxy protection of unauthorized deep links, cross-tab project isolation, and clean Work Apps feature-flag toggling. The two confirmed pre-existing defects are a high-severity URL-safety issue where unsanitized tenant/project params can produce protocol-relative cross-origin sidebar links, and a medium-severity hover-collapse race where a throttled enter/leave handler can drop the final mouseleave and leave the project sidebar incorrectly expanded.

❌ Failed (2)

Category	Summary	Screenshot
Adversarial	⚠️ Malformed tenant/project segments can produce protocol-relative sidebar hrefs that allow cross-origin navigation.
Edge	🟠 Rapid hover enter/leave can drop the terminal mouseleave update, leaving sidebar open in content area.

const topNavItems: NavItemProps[] = projectId
  ? []
  : [
      {
        title: STATIC_LABELS.projects,
        url: `/${tenantId}/projects`,
        icon: Layers,
      },
      {
        title: STATIC_LABELS.stats,
        url: `/${tenantId}/stats`,
        icon: BarChart3,
      },

const configureNavItems: NavItemProps[] = projectId
  ? [
      {
        title: STATIC_LABELS.agents,
        url: `/${tenantId}/projects/${projectId}/agents`,
        icon: Workflow,
      },
      {
        title: STATIC_LABELS.skills,
        url: `/${tenantId}/projects/${projectId}/skills`,

const handleHover: NonNullable<ComponentProps<'div'>['onMouseEnter']> = useCallback(
  throttle(200, (event) => {
    const isBlur = event.type === 'mouseleave';

    if (isBlur) {
      const blurToElement = event.relatedTarget;
      const insideMainContent =
        blurToElement &&
        blurToElement instanceof HTMLElement &&
        !!blurToElement.closest('#main-content');

      if (!insideMainContent) {
        return;
      }
    }
    setOpen(!isBlur);
  }),
  []
);

export function throttle<F extends (...args: any[]) => any>(delay: number, fn: F) {
  let last = 0;
  return ((...args: any[]) => {
    const now = Date.now();
    if (now - last >= delay) {
      fn(...args);
      last = now;
    }
  }) as F;
}

<main
  id="main-content"
  className={cn(

✅ Passed (17)

Category	Summary	Screenshot
Adversarial	With explicit logged-out marker set, protected deep links redirected to /login with returnUrl and did not render sidebar-protected content, matching intended development auth-proxy behavior.
Adversarial	Rapid alternating nav clicks preserved responsiveness, final route correctness, and active-state sync.
Adversarial	Concurrent tabs preserved isolated project-scoped links and active navigation state during rapid switching.
Adversarial	Work Apps appeared when enabled and disappeared cleanly when disabled, with adjacent links and active states still correct.
Edge	Long labels in collapsed and expanded sidebar states stayed aligned without overlap into main content.
Edge	Double-click and repeated mixed navigation clicks ended on the intended route with one active nav item.
Edge	Mobile sidebar and follow-up desktop navigation remained usable with correct route transitions.
Edge	Verified deep-link entry into nested routes preserves correct single active sidebar state across reload and back/forward navigation.
Logic	Verified no sidebar tooltips appear in collapsed/expanded org and project contexts during hover and keyboard focus interactions.
Logic	Verified nested project routes keep exactly one correct active sidebar item across traces, skills edit, credentials new, refresh, and history navigation.
Logic	All project sidebar items rendered with icons and no runtime console errors were observed.
Happy-path	Org sidebar links (Projects, Statistics, Members, Settings) navigated correctly with one active item and no tooltips in expanded state.
Happy-path	Organization sidebar links remained functional: sequence Projects -> Statistics -> Projects and Organization group Members -> Settings navigated to correct tenant-scoped URLs with proper active-state transitions.
Happy-path	Configure links routed correctly within project scope and maintained a single active nav item.
Happy-path	Register links (MCP Servers, Credentials, External Agents) navigated correctly with no tooltip appearing on hover/focus.
Happy-path	UI/Monitor links (Components, Artifacts, Traces, Evaluations) navigated to expected project routes with correct active state.
Happy-path	Back to org correctly returned from project context to /default/projects and exited project-only sidebar mode.

Commit: 464b1be

View Full Run

Tell us how we did: Give Ito Feedback