Skip to content

feat: embed AgentTeams activity panel in Better Sidebar tab - #38

Open
n-pll wants to merge 2 commits into
NanmiCoder:mainfrom
n-pll:feat/better-sidebar-tab
Open

feat: embed AgentTeams activity panel in Better Sidebar tab#38
n-pll wants to merge 2 commits into
NanmiCoder:mainfrom
n-pll:feat/better-sidebar-tab

Conversation

@n-pll

@n-pll n-pll commented Aug 17, 2026

Copy link
Copy Markdown

Summary

Embed the AgentTeams activity panel as a DSH Better Sidebar tab when dsh-better-sidebar is present, instead of mounting the fixed top-right floater.

Changes

  • Add an optional �mbedded prop to ActivityPanel:
    • forces the expanded panel view
    • skips the page-level data-agent-teams-panel-open layout shift
    • hides the floating close button
  • Register an �gent-teams tab through ctx.get('betterSidebar').registerTab(...) when Better Sidebar is available, and route the conversation card's OPEN_PANEL_EVENT to �etterSidebar.openTab(...).
  • Add .panel.embedded CSS so the panel fills the sidebar tab instead of using position: fixed.
  • Declare dsh-better-sidebar as an optional peer dependency and dev dependency.

Behavior

  • With Better Sidebar installed: the activity panel becomes a single-instance "Agent Teams" sidebar tab, and the card button opens that tab.
  • Without Better Sidebar: the existing body-portal floater remains the fallback, so behavior is unchanged.

Verification

  • pnpm install, pnpm build, and pnpm typecheck pass.
  • Tested against a DSH web profile: the served client bundle contains the new tab registration and embedded panel markup.

@NanmiCoder NanmiCoder left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

感谢 Better Sidebar 集成。主线已切换 shell.overlay、官方 locale 和新的会话导航,当前 patch 与这些变化冲突。请重基并使用可选服务注册:验证未安装 Sidebar 的浮窗、安装后的单实例 tab、会话切换、插件重载和卡片打开面板。我们会保留这个集成方向,避免为了集成退回旧客户端 API。

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