发布

  • [OPIK-6511] [FE] fix: hide Assistant sidebar wrapper when ollieEnabled toggle is off

    frostbyte_neo 发布于 2026-05-15 09:03:52 +00:00

    The PageLayout outer wrapper for the Assistant sidebar gated only on plugin
    registration (!!AssistantSidebar), so Comet-built FE deployments with Ollie
    disabled (e.g. self-hosted-eks, comet.opik.ollie.enabled=false) reserved a
    ~400px blank strip on every page even though the inner component correctly
    returned null.

    Consumes the new ollieEnabled service toggle (BE in #6716, helm chart in
    comet-ml-helm-chart#447) on the FE:

    • Add OLLIE_ENABLED to FeatureToggleKeys and DEFAULT_STATE (default false,
      matching BE default; OSS Opik also doesn't register the AssistantSidebar
      plugin so the OSS path is unchanged).
    • PageLayout includes ollieEnabled in showAssistantSidebar; the wrapper width
      and --assistant-sidebar-width CSS var collapse to 0 when disabled.
    • SideBarMenuItems gates the "Opik Connect" sidebar entry on the same toggle,
      hiding it in deployments where the Ollie BE isn't reachable.

    The runtime state machine in useAssistantBackend (compute → health → ready →
    error) is untouched and continues to drive the inner sidebar — that's the
    right place for runtime concerns.

    Co-Authored-By: Claude Opus 4.7 (1M context) noreply@anthropic.com

    下载附件