Skip to content

feat(agent): 添加 OMP 作为内置 Agent - #445

Merged
J3n5en merged 1 commit into
J3n5en:mainfrom
Mieluoxxx:feat/builtin-omp-agent
Jul 21, 2026
Merged

feat(agent): 添加 OMP 作为内置 Agent#445
J3n5en merged 1 commit into
J3n5en:mainfrom
Mieluoxxx:feat/builtin-omp-agent

Conversation

@Mieluoxxx

Copy link
Copy Markdown
Contributor

Summary

  • 将 Oh My Pi (omp) 接入内置 Agent 列表(类型、CLI 检测、设置默认值、UI 显示名/命令、AgentRegistry)
  • 默认关闭且非 default,启用后可在 Session 菜单中选择并启动 omp
  • 补充 builtin agent 注册契约测试,防止多处列表不同步

Test plan

  • vitest run src/main/services/cli/__tests__/BuiltinAgents.test.ts 通过
  • 设置 → Agent → 刷新检测,确认 OMP 能被检测并显示版本
  • 启用 OMP 后,长按 + 可选择 OMP
  • 新建 OMP Session,终端成功启动 omp

将 Oh My Pi (omp) CLI 接入默认 Agent 列表:类型、检测、设置默认值、UI 映射与注册表,并补充注册契约测试。
@github-actions

Copy link
Copy Markdown
Contributor

Claude Code is working…

I'll analyze this and get back to you.

View job run

@J3n5en J3n5en 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.

审查通过。已核对 OMP 官方包,CLI 二进制名确为 , 接入方式与现有检测逻辑兼容;类型、检测、默认设置和三个 Agent UI 映射点均已同步,持久化设置迁移也会保留新增默认项。建议合并。

@J3n5en J3n5en 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.

补充完整标识符:已核对 OMP 官方包,CLI 二进制名确为 omp--version 接入方式与现有检测逻辑兼容。其余审查结论不变,建议合并。

@J3n5en
J3n5en merged commit 6594b7b into J3n5en:main Jul 21, 2026
2 of 3 checks passed
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