feat(dashboard): dashboard home orb rpc first

[gdemonc]

solve #433

Background

This PR continues the RPC-first consolidation work for Dashboard Home Orb.

Previously, the orb experience relied heavily on mock/fallback data. After the RPC-first migration landed, the main data path was connected but never fully integrated or stabilized. This PR focuses on completing the integration against the real RPC pipeline and reducing dependency on mock behavior.

Changes

• Connect Home Orb to the formal RPC-first data flow
• Align orb state handling with dashboard overview data
• Improve fallback/mock handling during loading and error states
• Fix inconsistencies between mock and real response structures
• Add stability improvements for orb rendering and interaction flows
• Clean up obsolete mock-driven logic where possible

Notes

• Mock/fallback logic is still partially retained as a safety net
• This PR focuses on Home Orb integration only
• Broader dashboard/task-page RPC-first consolidation is tracked separately

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

[fennoai]

Completed code quality, performance, security, and documentation-accuracy passes. I left 2 non-blocking findings worth fixing before merging.

[gdemonc]

@xgopilot 怎么好像现在note和安全不会出信息球呢？然后镜子球就只是一个用户画像，是正常的吗？
Thinking: Evaluating the current code
I need to provide an answer based on the current code since it appears that the notes are empty, and the safety module summary isn't included in the moduleSummons. I should explain this situation and possibly suggest a fix since the user is asking why it’s happening and if it’s normal. It looks like I need to inspect the current branch and consider using commentary to clarify my progress. It’s better to verify the code before answering.
我先核一下当前首页信息球的候选生成条件，确认 notes / safety / memory 现在分别在什么情况下会出球。
→ Read apps\desktop\src\features\dashboard\home\dashboardHome.service.ts [offset=930, limit=120]
→ Read apps\desktop\src\features\dashboard\home\dashboardHome.service.ts [offset=630, limit=120]
→ Read apps\desktop\src\features\dashboard\home\dashboardHome.service.ts [offset=820, limit=120]
是，按我现在这版代码，出现这种现象是当前实现逻辑导致的。
为什么 note 不出球

• 首页便签球现在优先看正式 notepad bucket 数据，不再看摘要猜文案。apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:294
• 只有拿到 primaryItem 时，才会生成一个真正的便签首页语义。apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:466 apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:664
• 如果没有正式待处理事项，就会落到空态文案 这里还没有可协作的事项，而且我专门把这种空态从信息球候选池里排除了。apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:673 apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:1010
  所以：
• 便签页如果是空的，或者正式 bucket 没有可用事项，首页就不会出便签球
• 这是我为了避免“假文案”故意收紧的
  为什么 安全 不出球
• 安全球现在只在 overview.trust_summary 显示有风险时才出：
  • pending_authorizations > 0
  • 或 risk_level !== green
    apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:953
• 我把模块补位球收窄成只补 notes 和 memory，没有给 safety 常驻补位。apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:1001
  所以：
• 如果当前安全状态是绿的、没有待授权，就不会出安全球
• 这也是为了避免平时刷一个“假紧急”的安全球
  为什么 镜子 现在基本只是“用户画像”
• 我把首页镜子球改成和正式 mirror overview 同源后，优先级是：
  1. profile
  2. memory_references
  3. history_summary
     apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:843
• 也就是说，只要后端返回了 profile，首页镜子球就会固定走：
  • 标题：用户画像
  • 副标题：工作风格：...
  • 上下文：偏好交付 / 活跃时段
    apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:846
    所以：
• 现在镜子球“只是一个用户画像”，是当前实现下的正常结果
• 它的目的是先保证和镜子详细页同源，代价是首页镜子球变得比较单一
  我的判断
• 这版行为是“正常但偏保守”
• 它解决了“首页假文案”和“和详细页对不上”
• 但副作用是：
  • notes 更难出球
  • safety 只有风险时才出
  • memory 太容易固定成 用户画像
    如果你想更符合产品感觉
    我建议下一步这样调，而不是回到假文案：

1. notes

• 只要正式 bucket 里有 upcoming / later / recurring_rule 任一项，就允许出球
• 空态时继续不出球

2. safety

• 继续保留“风险球最高优先”
• 但在绿态时允许低频补一个非紧急安全球，比如“最近恢复点可用”

3. memory

• 不要永远优先 profile
• 改成在 profile / memory_references / history_summary 三类正式来源之间轮换
• 仍然只用正式字段，不写假文案

你觉得要这样调吗？

[fennoai]

结论

是，按这版实现，这三个现象都是当前逻辑直接导致的，算“正常但偏保守”，不是渲染故障。

当前行为

note 不出球：主页先读正式 notepad bucket，primaryItem 由 upcoming -> later -> recurring_rule -> closed 选出；只有有 primaryItem 才会走真实 note 首页语义。apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:294 apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:466 apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:664

空态又被显式排除出模块补位球，所以没有正式待处理事项时，首页不会出 note 球。apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:1122

safety 不出球：overview summon 只在 pending_authorizations > 0 或 risk_level !== green 时才会把 safety 放进球候选；模块补位球也只补 notes 和 memory，没有 safety 常驻补位。apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:490 apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:1065 apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:1113

mirror 基本只剩用户画像：formal mirror state 的优先级是 profile -> memory_references -> history_summary，所以只要后端回了 profile，首页镜子球就会固定成“用户画像”。apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:843 apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:909

我的判断

如果目标是“先保证 RPC-first 同源、减少假文案”，这版是合理的。

如果目标是“首页更有产品感、更容易持续冒出可读球”，我建议调，但不要退回 mock 文案：

1. notes：只要正式 bucket 里有 upcoming / later / recurring_rule 任一项就允许出球；closed 不该驱动首页主语义。现在 closed[0] 还能进 primaryItem，这会把已结束事项抬成首页动作球，属于当前实现里的一个偏差。apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:314
2. safety：继续保留风险球最高优先，但绿态可以允许低频补一个非紧急 safety 球，比如“最近恢复点可用”。
3. memory：不要永远先吃 profile，改成在 profile / memory_references / history_summary 之间轮换，仍然只用正式字段。

这次我只做了代码核对和行为判断，没有改代码。

View job run

[gdemonc]

@xgopilot 你看下最新的commit有没有实现我提的功能，还需不需要再补充功能？以及还存在哪些问题？

[fennoai]

结论

最新这几次 commit 只算部分实现了你上次提的方向，不是完全收口。

已经补上的部分：

• safety 现在补了绿态召唤球入口，不再只在黄/红态才出现。apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:1092
• mirror 已经改成先走正式 getMirrorOverview，并且当 memory_references 存在时会优先显示“近期被调用记忆”。apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:843
• notes 也已经开始读正式 notepad buckets，而不是只靠 recommendation/mock。apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:294

但你上次提的三个体验问题，还没有完全都修完。

还需要补的功能

1. notes 这条没有彻底修完。现在 primaryItem 仍然是 upcoming -> later -> recurring_rule -> closed，所以只要前三组为空、closed 里有历史项，首页还是会把“已结束事项”抬成当前 note 球。apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:314 apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:475 apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:665
2. mirror 还没有实现你上次说的“不要总是固定成用户画像”的那种轮换/均衡策略。现在只是把优先级改成了 memory_references > profile > history_summary；一旦没有 memory_references，还是会稳定落到“用户画像”。apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:843

现存问题

• Medium: notes 仍把 closed[0] 当成首页主事项。结果是首页会把历史已完成便签表现成还在推进的当前事项，这和你前面希望“空态不出假球、历史项不要驱动首页主语义”的目标还是冲突。apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:314
• Medium: mirror 的 RPC 失败还是静默降级。getMirrorOverview reject 后这里只是回 null，不会进 loadWarnings，首页看起来像正常降级，但实际上丢了一条正式数据源。apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:1358
• Medium: safety 现在有点补过头了。绿态分支的条件是 has_restore_point || safetyState.subline.trim() !== ""，而 safetyState.subline 在绿态下几乎总会有默认文案，所以安全球会变成几乎常驻，而不是“低频补一个非紧急安全球”。apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:1092 apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:960

我的判断

如果你的目标只是“把之前完全不出球的问题先修掉”，这版已经做到一半了。
如果目标是“行为和你上次讨论的产品语义一致”，我建议至少再补三件事：

• notes 去掉 closed 参与 primaryItem
• mirror 失败时写入 loadWarnings
• safety 绿态召唤条件再收紧，只在有恢复点或明确安全摘要信号时出球

这次我只做了代码核对和 review，没有改代码。

View job run

[gdemonc]

@xgopilot 我现在其实已经解决了你提到的问题了，麻烦你清楚地再看一遍，不要钻牛角尖

[fennoai]

Findings

我重看了最新 commit。之前我提的 mirror overview 静默失败这条已经修掉了：getMirrorOverview reject 现在会进入 loadWarnings，这条不再算问题。当前还剩 4 个有效问题。

1. Medium: apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:314 apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:475
primaryItem 仍然会落到 closed[0]，所以只要 upcoming/later/recurring 为空但有已结束事项，首页就会继续走 notes_processing，把历史事项抬成当前便签主语义。这和前面“空态不出假球、历史项不要驱动首页主语义”的目标还是冲突。

2. Medium: apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:956 apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:1092
   绿态 safety 补位条件还是过宽。safetyState.subline 在绿态下默认会退到 工作区位于 ...，因此 safetyState.subline.trim() !== "" 几乎恒为真，结果是即使没有恢复点、没有风险、没有待授权，首页也会常驻一个低优先级 safety summon，而不是“低频补一个非紧急安全球”。

3. Medium: apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:272 apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:1067
   所有 overview summon 都直接吃 quick_actions[0]。如果 overview 的首个 quick action 是任务语义，比如测试里已经出现的 打开任务详情，那 safety summon 的 CTA 文案也会变成 打开任务详情，但点击后实际仍然进入 safety 路由。这会把按钮文案和真实跳转目标弄反。

4. Low: apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:778 apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:843
buildFormalMirrorState() 已经改成 memory_references 优先于 profile，但 buildFormalMirrorInsights() 还是只要有 profile 就先返回画像 insights。结果是在 headline = 近期被调用记忆 的场景下，下面的 insights 仍然展示 工作风格 / 偏好交付 / 活跃时段，信息对象前后不一致。

Summary

结论和上次不一样的地方只有一条：mirrorOverview 的失败告警已经修复，我撤回那条旧意见。其余几条里，前 3 条仍然是实际行为问题，不是旧评论没刷新。

View job run

[gdemonc]

@xgopilot 我已经解决了你提到的问题

[fennoai]

Findings

1. Medium: apps/desktop/src/app/dashboard/DashboardHome.tsx#L52 apps/desktop/src/app/dashboard/DashboardHome.tsx#L121
   pickNextSummonIndex() now randomizes from the full template pool on the very first summon because previousIndex = -1 and previousModule = null. That means the first visible orb can be memory or recommendation fallback even when the service already ordered summonTemplates to put formal overview signals first. This undercuts the RPC-first prioritization added in buildOverviewSummons() and can hide urgent safety/task signals behind lower-priority copy.

2. Medium: apps/desktop/src/features/dashboard/home/dashboardHome.service.ts#L272 apps/desktop/src/features/dashboard/home/dashboardHome.service.ts#L1102
   The old CTA mismatch is fixed for focused task states, but it still exists on the templates.length === 0 && highValueSignals[0] fallback path. getSummonNextStep() will still use quick_actions[0] for any task-state summon, even when there is no focus_summary and the task state only has a module-level navigation target. If overview returns a generic task signal plus quick_actions[0] = 打开任务详情, the orb label says “打开任务详情” while clicking still routes to the tasks module page.

Recheck Result

The earlier issues around closed-only notes, mirror overview load warnings, safety-module CTA text, and mirror reference/profile insight mismatch do look fixed in the latest commits. I did not find those same bugs again.

Verification

I reviewed the latest origin/main...HEAD diff and the updated contract tests. I could not run pnpm --dir apps/desktop test:dashboard because pnpm is not available in this environment.

View job run

[gdemonc]

@xgopilot 解决了你提到的问题

[fennoai]

Findings

No remaining review findings after rechecking the latest origin/main...HEAD diff.

Recheck Result

The issues I had left in the previous round are now addressed in the current branch:

• First-orb priority is preserved on the initial summon, so service-ordered overview signals are no longer randomized away on first render. DashboardHome.tsx#L52
• The fallback task summon no longer reuses quick_actions[0] unless the state can actually deep-link to task detail, so the CTA text now stays aligned with the real route target. dashboardHome.service.ts#L272 dashboardHome.service.ts#L1102
• The mirror reference/profile copy mismatch is fixed because formal mirror insights now prefer memory_references before profile, matching the new state priority. dashboardHome.service.ts#L782 dashboardHome.service.ts#L841
• The green-state safety summon is narrowed to the restore-point case instead of effectively becoming always-on. dashboardHome.service.ts#L1090

Verification

I reviewed the updated implementation and the added contract coverage for these regressions in dashboard.contract.test.ts#L6531, dashboard.contract.test.ts#L6872, dashboard.contract.test.ts#L6934, and dashboard.contract.test.ts#L7011.

I could not run desktop tests or pnpm-based verification because pnpm is not installed in this environment.

View job run

[Late-Smoke]

信息球传的数据怪怪的

[gdemonc]

@xgopilot 这个信息球的数据是从哪里拿到的？

[fennoai]

结论

这个首页“信息球”现在不是从单一 mock 拿的，而是由 loadDashboardHomeData() 先并行拉一组正式数据，再在前端拼成 summonTemplates。主入口在 apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:1293。

具体来源

当前会并行拿 8 路数据：

• agent.dashboard.overview.get：焦点任务、信任摘要、quick_actions、high_value_signal，这是首页球的第一优先级来源。apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:1294 apps/desktop/src/rpc/methods.ts:151
• agent.dashboard.module.get x4：分别拉 tasks / notes / memory / safety 的模块摘要，用来补每个模块自己的 state。apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:1300
• agent.mirror.overview.get：给镜子球提供 profile / history_summary / memory_references 这些正式字段。apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:1329 apps/desktop/src/rpc/methods.ts:159
• 便签 bucket：不是 mock，而是继续走 loadNoteBucket()，底层调用 agent.notepad.list 拉 upcoming / later / recurring_rule / closed 四组数据。apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:298 apps/desktop/src/features/dashboard/notes/notePage.service.ts:689 apps/desktop/src/rpc/methods.ts:139
• agent.recommendation.get：这是最后一层推荐/补位来源，不是首页球的首要真源。apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:1320 apps/desktop/src/rpc/methods.ts:90

在首页怎么变成“球”

真正决定信息球文案的是这三层拼装，顺序也是这个顺序：

1. buildOverviewSummons()：先吃 overview 里的正式信号，优先放安全/焦点任务。apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:1048
2. buildModuleSummarySummons()：再补 notes 和 memory 这两个模块自己的正式摘要球。apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:1121
3. buildRecommendationSummons()：前两层不够时，再补 recommendation 球。apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:1155

最后 DashboardHome.tsx 只是按 data.summonTemplates 展示，首个球会保留 service 端排好的优先级，不会再把第一条随机掉。apps/desktop/src/app/dashboard/DashboardHome.tsx:49 apps/desktop/src/app/dashboard/DashboardHome.tsx:115

一句话总结

所以现在这个信息球的主数据源是：
dashboard overview / dashboard module / mirror overview / notepad list 这些正式 RPC，recommendation 只作为后补，不再是主来源。

这次我只做了代码核对，没有改代码。

View job run

[gdemonc]

信息球传的数据怪怪的

你这个反馈是对的，这里现在看起来“怪”，不是因为还在吃 mock，而是因为首页信息球已经切到正式 RPC 了，但首页展示层和详细页展示层还没完全对齐。
具体来说：

• 首页信息球现在会把多路正式数据先聚合一遍，再拼成球的文案
• 镜子这类信息球有时候拿的是 memory_references
• 但你点进去看到的详细页，当前默认更像是在展示 profile / history 那套内容
• 所以会出现“首页看到的是 A，对应页里更显眼的是 B”的感觉
  另外还有一个问题是，首页现在有些文案直接把后端字段原样露出来了，比如这种 memsum_task_001_run_001，虽然数据是真的，但对人不够友好，所以观感上会更奇怪。
  我这边准备继续收两件事：

1. 把首页信息球文案再做一层“人能看懂”的整理，优先展示可读摘要，不直接暴露技术 ID
2. 让首页球和打开后的详细页尽量落到同一个对象/同一个分区，减少“首页是这个，进去看的是另一个”的错位

[Late-Smoke]

• 如果是图像就显示成图像而不是![uploading...]
• 没用的、重复的东西就删掉，想象一下自己是用户，去看这些东西

[gdemonc]

• 如果是图像就显示成图像而不是![uploading...]
• 没用的、重复的东西就删掉，想象一下自己是用户，去看这些东西

第一句的图片没有上传成功，所以不知道具体是什么情况还没有修改。
对于第二句，本次commit预期目标是让首页事件舱从“多层重复描述同一件事”变成“用户一眼看懂当前最重要的信息”

[liche719]

@xgopilot 这里的 CTA 文案和实际跳转目标可能不一致。getSummonNextStep() 在任务召唤场景下直接复用了 quick_actions[0]，buildOverviewSummons() 又把它作为任务 orb 的 nextStep。但后端返回的 quick_actions 是全局优先级排序：当同时存在焦点任务和待授权操作时，第一项会是“处理待授权操作”，而不是“打开任务详情”。当前点击逻辑却仍然按照 navigationTarget.taskId 打开任务详情，因此会出现按钮文案提示去处理授权、实际却跳到任务详情页的错配。建议只在 quick action 与真实导航目标一致时才复用它，否则继续使用任务详情的固定 CTA 文案。
这个问题需要解决吗

[fennoai]

Findings

Medium: apps/desktop/src/features/dashboard/home/dashboardHome.service.ts#L286 apps/desktop/src/features/dashboard/home/dashboardHome.service.ts#L636 apps/desktop/src/features/dashboard/home/components/DashboardEventPanel.tsx#L176
这个问题还存在，应该修。当前 getSummonNextStep() 只要看到任务 summon 的 navigationTarget.kind === "task_detail" 就直接复用 quick_actions[0]，但展开后的点击仍然严格按 navigationTarget 跳转：task_detail 一律进任务详情，只有 waiting_auth 才会把 navigationTarget 改成 safety。也就是说，只要后端在“焦点任务 + 仍有待授权操作”场景下把全局第一条 quick_actions[0] 排成“处理待授权操作”，首页 orb CTA 文案就会提示去授权，实际点击却还是打开任务详情，和你在评论里指出的错配一致。

Why It Should Be Fixed

这不是文案细节，而是用户动作语义和真实导航目标不一致。这里的首页 orb 是 RPC-first 正式链路的一部分，CTA 不能把全局 quick action 当成当前 task-detail deep link 的替身，除非两者目标一致。

Coverage Gap

apps/desktop/src/features/dashboard/dashboard.contract.test.ts#L6628 和 apps/desktop/src/features/dashboard/dashboard.contract.test.ts#L6981 目前只覆盖了 quick_actions[0] = "打开任务详情" 或 task summon 只能回模块页的情况，还没有覆盖“focus task 仍然 deep-link 到 task detail，但 quick_actions[0] 指向待授权操作”的分支，所以这个回归现在测不出来。

Conclusion

需要解决，建议就在这个 PR 里修掉：只有当 quick action 和真实 navigationTarget 一致时才复用它，否则继续使用固定的任务详情 CTA 文案。

这次只做了代码核对，没有改代码。

View job run

[gdemonc]

@xgopilot 修复你提到的问题

[fennoai]

Update

已修复首页 task orb 的 CTA/跳转错配，并已推到当前 PR 分支 feat/dashboard-home-orb-rpc-first，commit 是 22190f28。

本次改动：

• 在 apps/desktop/src/features/dashboard/home/dashboardHome.service.ts 收紧 quick_actions[0] 的复用条件，只有它和真实 navigationTarget.label 一致时，task summon 才沿用该 CTA；否则继续使用任务详情自己的固定 CTA。
• 在 apps/desktop/src/features/dashboard/dashboard.contract.test.ts 新增回归用例，覆盖“全局第一 quick action 是 处理待授权操作，但当前 orb 实际仍 deep-link 到 task detail”的场景，确保文案不再误导。

本地验证受当前 runner 环境限制，没有完整跑通：

• npm --prefix apps/desktop run test:dashboard 失败，缺少 TypeScript 类型依赖（node、vite/client），同时命中本地 TS6 deprecation 报错。
• npm --prefix apps/desktop run typecheck 因相同原因失败。
• npm --prefix apps/desktop run lint 失败，当前环境没有 eslint。

View job run

[gdemonc]

@xgopilot 不要钻牛角尖，看看还有问题吗

[fennoai]

Findings

1. Medium: apps/desktop/src/features/dashboard/home/dashboardHome.service.ts#L342 apps/desktop/src/features/dashboard/home/dashboardHome.service.ts#L1315
   loadDashboardHomeData() now blocks the entire home-orb query on loadDashboardHomeNoteBuckets(), which fans out into four extra agent.notepad.list calls and wraps each one in a 2.5s timeout. If the notepad RPC path is slow or partially unavailable, the dashboard home page waits for that secondary notes hydration before rendering any of the already-available overview/task/safety data. That turns a non-critical enrichment source into a startup bottleneck for the whole RPC-first home view.

Summary

I did not find the earlier CTA/priority issues again in the latest diff. The remaining concern is the new note-bucket fetch being on the critical path for initial home load.

Verification

I reviewed the latest origin/main...HEAD diff and the added dashboard contract coverage.

I also tried npm --prefix apps/desktop run test:dashboard, but it fails in this environment before the suite runs because the workspace dependencies/types are missing (node, vite/client) and tsconfig.dashboard-test.json now trips TypeScript 6 deprecation errors for baseUrl and moduleResolution=node10.

View job run

[gdemonc]

@xgopilot 进行相应的判断是否选择修改（“一定要最合理的判断，review提到的问题不是都要修改的，可能问题已经解决了，也可能会没问题就乱找问题”）。如果需要是修改代码还是选择修改文档，还是都要修改。如果决定修改，以最合理的方式修改,不要做向下兼容从而过拟合。然后再详细检查相关文件的内容是否还有类似的问题，要求一次性解决完，最后核查一遍，修改后直接告诉我修改了什么、修改的原因、修改后的可能出现的现象以及如果不修改所带来的具体影响。然后再查看是否还有其他问题。

[fennoai]

结论

这次需要改代码，不需要改文档。

原因很明确：问题出在 apps/desktop/src/features/dashboard/home/dashboardHome.service.ts 对正式 mirror RPC 数据的首页展示映射，不是协议定义错了，也不是文档口径错了。继续改文档不会解决“信息球数据怪”的实际表现。

我修改了什么

1. 在 apps/desktop/src/features/dashboard/home/dashboardHome.service.ts 收紧了首页镜子球的引用记忆文案映射。

• 对 memory_references[0].summary / reason 做用户展示级清洗，去掉异常替换字符和多余空白。
• 不再把内部 memory_id 直接显示到首页事件球/展开面板里。
• 仍然保留 focusMemoryId 在导航目标里，所以点击后还是会定位到对应镜子详情，不影响正式跳转能力。
• 用户可见上下文改成“来源：近期长期记忆命中” + 真实命中原因，仅在原因和摘要不重复时展示第二条。

2. 在 apps/desktop/src/features/dashboard/dashboard.contract.test.ts 同步更新并补充回归覆盖。

• 更新原有 mirror reference 断言，不再期望首页展示 memory_id。
• 新增一条脏摘要回归用例，覆盖 � 和首尾/多余空白清洗后的展示结果。

为什么要这样改

这不是“review 乱找问题”，而是当前实现确实把正式 RPC 返回里的内部识别字段和原始摘要直接抬到了首页用户文案层。

从产品语义看，首页信息球应该展示“可读的协作信号”，不应该展示内部记忆主键；而且原始摘要里如果混进异常字符，继续直出只会让用户觉得首页数据链路不可信。

改完后你可能看到的现象

• 镜子球还是会优先显示“近期被调用记忆”，这一点不变。
• 展开后副标题里的脏字符会被清掉，文案会更像用户可读摘要。
• 上下文不再出现 memsum_task_... / memory_... 这类内部 ID。
• 点击“打开镜子页”后，仍然会聚焦到原来的那条记忆，不会丢跳转能力。

如果不改，会带来的具体影响

• 首页继续暴露内部 memory_id，把实现细节直接泄漏到用户界面。
• 脏摘要字符会持续出现在首页主链路入口，用户会误判成数据错乱或编码异常。
• RPC-first 虽然已经接上了，但首页观感会被这种“内部字段直出”拖低，等于把正式数据当成未整理日志展示。

复查结果

这次顺手把相关文件又看了一遍，当前没有再发现同类必须一起补的遗留问题。我的判断是：这轮收在代码层就够了，不需要再扩写兼容分支，也不需要补文档说明。

验证

• 已提交并推回当前 PR 分支，commit: 575501e0 (fix(dashboard): sanitize mirror home orb copy)
• 额外同步记录：feat(dashboard): dashboard home orb rpc first gdemonc/CialloClaw#12
• 我尝试执行 test:dashboard、typecheck、lint
• 当前环境缺少仓库依赖，node_modules 未安装，所以这些校验没有跑完
• test:dashboard / typecheck 失败点是缺少 @types/node、vite/client，并且本地 tsc 对现有 tsconfig 的弃用选项直接报错
• lint 失败点是 eslint 不在当前环境

View job run

[gdemonc]

@xgopilot /review

[fennoai]

Findings

1. Medium: apps/desktop/src/features/dashboard/home/dashboardHome.service.ts#L1106
   When the trust chain is urgent, the first safety summon reuses high_value_signal[0] as its message even though the summon is hard-routed to the safety module. If high_value_signal[0] is task-oriented text such as “刚生成了新的摘要草稿”, the orb renders task copy but expands/navigates into safety. That makes the orb payload internally inconsistent and matches the “信息球的数据怪怪的” behavior reported in the thread. The task-focused copy should stay attached to the task summon unless the overview signal is actually safety-specific.

2. Medium: apps/desktop/src/app/dashboard/DashboardHome.tsx#L126 apps/desktop/src/app/dashboard/DashboardHome.tsx#L160 DashboardRoot.tsx#L107 DashboardRoot.tsx#L169
   The summon scheduler is keyed to the full data.summonTemplates array via scheduleSummon, so every dashboard/home invalidation recreates the callback and re-runs the effect that clears summons, resets the index refs, and restarts the 2.5s delay. dashboard/home is invalidated on recommendation feedback, task.updated, approval.pending, and delivery.ready, so live updates will repeatedly wipe the current orb cadence and restart from the first template. In practice this makes the home orb flicker/replay instead of behaving like stable local presentation state.

Verification

Reviewed the latest origin/main...HEAD diff, the updated home-orb service code, and the added dashboard contract coverage. I did not run pnpm-based checks in this environment.

View job run

[gdemonc]

@xgopilot 修复你提到的问题

[fennoai]

Result

这次我判断需要改代码，不需要改文档，原因是问题都发生在首页 orb 的前端展示映射和本地调度层，不是协议真源或文档口径错误。修改已经提交并推回当前 PR 分支，commit 98c37619，同步 fork 跟踪 PR：gdemonc#13。

What Changed

• 在 dashboardHome.service.ts 收紧了 urgent safety summon 的文案来源。现在安全球固定使用 safetyState.headline，不再复用全局 high_value_signal[0]。修改原因是安全球点击后一定进入 safety 模块，如果继续展示任务向的全局信号，会出现“文案像任务，跳转却去安全”的数据错位。
• 在 DashboardHome.tsx 把 summon 调度改成读 ref 中的最新模板，并额外用内容签名控制重置时机。现在只有 summonTemplates 的实际内容变了，首页才会重开一轮 summon；等价的 dashboard/home 失效刷新不会反复清空和重播当前节奏。修改原因是 summon 属于本地展示状态，不该被每次实时刷新都打回起点。
• 在 dashboard.contract.test.ts 补了两类回归覆盖：一类验证“待授权安全球不再吃任务向全局信号”，一类验证首页 summon 调度改成基于模板签名与 ref 保持稳定。

Expected Behavior

• 修改后你会看到：有待授权/高风险时，首页安全球标题会稳定显示安全语义，比如“当前有 1 项操作等待授权”，不会再拿“刚生成了新的摘要草稿”这类任务文案去承载安全跳转。
• 另外，task.updated、approval.pending、delivery.ready、recommendation feedback 这类实时刷新如果没有真正改动 summon 内容，首页 orb 不会再闪一下、清空后从第一条重新播。
• 如果不改，具体影响就是两类：一类是 CTA/模块语义继续错配，用户会以为点进去处理任务却被带去安全；另一类是首页 orb 在实时更新下持续重播，表现成节奏不稳定、像数据层在抖动。

Verification

• git diff --check 通过。
• npm --prefix apps/desktop run test:dashboard 没跑到测试执行阶段，当前环境缺少 @types/node、vite/client，同时 tsconfig.dashboard-test.json 在本地 TypeScript 6 下命中 baseUrl 和 moduleResolution=node10 的弃用报错。
• pnpm --dir apps/desktop typecheck 和 pnpm --dir apps/desktop lint 无法执行，当前 runner 没有 pnpm。

Recheck

我把相关文件又顺手复查了一遍，当前没有再发现同区域必须一起补的同类问题。我的判断是，这一轮收在这 3 个文件就够了，不需要再做兼容性补丁，也不需要补文档。

View job run

[gdemonc]

@xgopilot /review

[fennoai]

Findings

1. Medium: apps/desktop/src/app/dashboard/DashboardHome.tsx:86 apps/desktop/src/app/dashboard/DashboardHome.tsx:198
buildSummonTemplateSignature() only hashes visible copy fields (stateKey/module/message/reason/nextStep/priority/recommendationId) and ignores the deep-link payload carried in expandedState / navigationTarget such as taskId and focusMemoryId. The reset effect depends on that signature, so if the backend switches the formal target while the orb copy stays the same, the existing summon queue is preserved and the user can still open the stale task detail or stale memory detail from the old summon.

2. Medium: apps/desktop/src/features/dashboard/home/dashboardHome.service.ts:1146
   On the templates.length === 0 && highValueSignals[0] fallback path, the code always binds the summon to taskState unless safety is urgent. That means any non-task overview signal on this path still renders with a task navigation target and task CTA text. A mirror- or notes-oriented high_value_signal would therefore open /tasks, which is the same copy/route mismatch this PR is otherwise trying to eliminate.

Verification

Reviewed the current origin/main...HEAD diff and the added contract tests. I could not run pnpm-based checks here because pnpm is not installed in this environment.

View job run

[gdemonc]

@xgopilot 修复你提到的问题

[fennoai]

Result

已修复我在上一轮 review 里提到的两条首页 orb 问题，并推回当前 PR 分支。主提交是 37582e4e (fix(dashboard): align orb fallback routing)，同步 fork 跟踪 PR：gdemonc#15。

What Changed

• 在 apps/desktop/src/app/dashboard/DashboardHome.tsx 扩大了 buildSummonTemplateSignature() 的签名范围。现在除了可见文案，还会把 expandedState.navigationTarget 里的 taskId、activeDetailKey、focusMemoryId 等 deep-link 负载一起纳入重置判断，避免后端目标已切换但首页还保留旧 summon 队列，继续打开过期任务或过期记忆详情。
• 在 apps/desktop/src/features/dashboard/home/dashboardHome.service.ts 给 high_value_signal 的 overview fallback 增加了模块推断。现在非紧急 safety 场景下，不再一律把 fallback orb 绑到 tasks，而是根据正式 signal 文案推断到 memory / notes / safety / tasks，避免出现“镜子/便签语义文案，却跳去任务页”的错配。
• 在 apps/desktop/src/features/dashboard/dashboard.contract.test.ts 补了两类回归覆盖：一类检查 summon 签名确实包含 deep-link 目标字段；一类验证 overview fallback signal 会路由到推断出的模块，而不是默认任务页。

Verification

• git diff --check 通过。
• npm --prefix apps/desktop run test:dashboard 未通过，失败原因是当前环境缺少 @types/node、vite/client，同时 tsconfig.dashboard-test.json 在本地 TypeScript 6 下命中现有 baseUrl / moduleResolution=node10 弃用错误。
• npm --prefix apps/desktop run typecheck 未通过，失败原因同上。
• npm --prefix apps/desktop run lint 未执行成功，当前环境缺少 eslint。

Not Done

没有补装依赖，也没有改现有 tsconfig / lint 环境问题；这些是当前 runner 的环境缺口，不是这次 orb 修复的一部分。

View job run

[gdemonc]

@xgopilot 解决了你提到的问题

[fennoai]

Findings

1. Medium: dashboardHome.service.ts#L820 dashboardHome.service.ts#L1342 dashboardHome.service.ts#L1438
   When loadDashboardHomeNoteBuckets() fails but getDashboardModule("notes") still succeeds, the home orb now renders a false empty-note state instead of falling back to the real notes module summary. buildDashboardHomeData() still calls buildNotesState(..., null), and buildNotesState() ignores the live notesModule highlights in that branch, so the headline resets to 这里还没有可协作的事项, the context becomes 暂无便签, and note summons are suppressed. On the exact transport-error path this PR is supposed to stabilize, users can lose all visible note signals even though the module RPC already returned usable data.

Recheck Result

I re-reviewed the latest origin/main...HEAD diff and the earlier issues around closed-only note promotion, safety CTA routing, mirror/profile mismatch, first-orb priority, and mirror warning handling do look resolved in the current branch. The notes fallback path above is the remaining behavior problem I found.

Verification

I reviewed the updated implementation and the added contract tests. I could not run desktop verification commands because pnpm is not installed in this environment.

View job run

[gdemonc]

@xgopilot 不要钻牛角尖了，这个note问题先不管了

[fennoai]

Findings

No remaining review findings after rechecking the latest origin/main...HEAD diff.

Recheck Result

这轮我没有继续抓 note 那个旧点，重点重看了这次 PR 里真正新增的几块：

• DashboardHome.tsx 的首球优先级、随机轮换和展开态同步
• dashboardHome.service.ts 里的 overview / mirror / safety / note bucket 装配
• DashboardEventPanel.tsx 的 task detail / mirror detail 跳转语义
• 新增的 dashboard contract tests 是否覆盖这些回归点

按当前分支代码看，我没有再发现新的非-note 行为问题。之前反复提过的首球优先级、CTA 路由错位、mirror 扩展态不一致、mirror failure warning 这些点，在最新提交里都已经对上了。

Verification

我重新核对了实现和新增合同测试，但没法在这个环境里跑桌面测试：corepack pnpm --dir apps/desktop test:dashboard 无法执行，因为这里没有 corepack。

View job run