Skip to content

feat: add personal assistant home - #36

Open
sddxl150-star wants to merge 2 commits into
uskyu:masterfrom
sddxl150-star:feat/personal-assistant-home
Open

feat: add personal assistant home#36
sddxl150-star wants to merge 2 commits into
uskyu:masterfrom
sddxl150-star:feat/personal-assistant-home

Conversation

@sddxl150-star

Copy link
Copy Markdown
Contributor

Depends on

This is a stacked PR. After #35 merges, the remaining diff is limited to frontend/src/App.vue and frontend/src/AssistantHome.vue.

Summary

  • add a responsive personal assistant landing page for desktop and mobile
  • surface recent-work recommendations and translation, image, and PPT shortcuts
  • support drag-and-drop attachments plus local answer mode and temperature preferences
  • create a planning room and send the home task before opening chat

Validation

  • npm ci && npm run build (Node 22)
  • sensitive-value scan
  • patch whitespace check

@monkeycode-ai monkeycode-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

合并评估(monkeycode-ai 已本地验证:前端 Vite 构建通过)

功能方向认可:AssistantHome 增量仅 2 个文件约 228 行,响应式与可访问性处理到位。但它作为 #35 的 stacked PR,同样受 #35 两个阻断问题影响:

1. 核心流程在默认配置下不可用(严重)

  • 依赖 #35POST /admin/collaboration/planning-room,该接口默认部署下返回 400(缺 orchestrator 配置)。
  • 更关键:startAssistantChat 在创建失败时静默 return,用户点击发送后无任何反馈。要求补充错误提示,不能静默失败。

2. i18n 缺失(次要)

  • tr('个人助手')tr('助手') 未加入 i18n.js 翻译表,非中文语言下会显示中文。建议补充各语言条目。

依赖 #35 修复后再合并本 PR。

(说明:上方 PR #35 下有一条 "permission probe" 评论为权限测试,可忽略。)

@monkeycode-ai

monkeycode-ai Bot commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

本 PR 的修复版已随 #37 合并到 master(含前端错误提示与 i18n 修复)。建议关闭本 PR。

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.

1 participant