Skip to content

test(desktop): 补齐 localDb 夹具缺失的 sessions.writable_dirs 列 - #3842

Merged
MagicLizi merged 1 commit into
makecindy:mainfrom
sakiko-toyokawa:test/localdb-writable-dirs-fixtures
Sep 3, 2026
Merged

test(desktop): 补齐 localDb 夹具缺失的 sessions.writable_dirs 列#3842
MagicLizi merged 1 commit into
makecindy:mainfrom
sakiko-toyokawa:test/localdb-writable-dirs-fixtures

Conversation

@sakiko-toyokawa

Copy link
Copy Markdown

这次改了什么

摘要

#3587sessions schema 新增 writable_dirs 列后,drizzle 的全列查询使三个手工 CREATE TABLE 的测试夹具(orcaTeamStore / sessionAutoTitlePersist / sessionsRestoreIfArchived)持续报 no such column: sessions.writable_dirspnpm test:unit:related 门禁因此常红。本 PR 按兄弟夹具 sessionsUpdate.test.ts 已有写法为三处补上 writable_dirs TEXT NOT NULL DEFAULT '[]',每处一行,共 3 行。

纯测试夹具维护,无生产代码改动。从 PR #3829 拆出(review 意见:与侧栏 UI 决策正交,独立放行)。

变更类型

  • feat 新功能
  • fix 缺陷修复
  • refactor / perf 重构或性能优化
  • docs / test / chore 文档、测试或工程维护
  • 其他:测试夹具修复(test)

范围

UI 变化

不涉及:纯测试夹具改动,无 UI 代码路径变更。

怎么验证的

自动验证

pnpm exec vitest run src/main/localDb/__tests__/orcaTeamStore.test.ts src/main/localDb/ipc/__tests__/sessionAutoTitlePersist.test.ts src/main/localDb/ipc/__tests__/sessionsRestoreIfArchived.test.ts
结果:35 passed(修复前 21 failed)

pnpm test:unit:related
结果:desktop related 218 文件 / 5454 用例全 PASS(在含本修复的工作区验证)

手工验证

不涉及。

未执行的验证

无。

风险

风险分类

  • 无已知风险
  • SQLite / migration
  • system prompt
  • 协议兼容
  • 权限 / 安全 / 用户数据
  • 存量插件兼容(批准状态 / 指纹 / manifest 校验 / 安装布局 / 包格式)
  • 原生层 / fingerprint / OTA
  • 跨平台差异
  • 其他:

影响与回滚

  • 影响范围:仅本地测试门禁,无运行时影响。
  • 回滚 / 降级方式:revert 即可。

提交前检查

  • 已 review 完整 diff
  • 每个 commit 都带 DCO 签名(git commit -s,见 DCO
  • UI 改动已在「UI 变化」注明引用的设计规范章节(不涉及 UI)
  • 未提交凭证、令牌或授权文件
  • 已补充必要文档(无新增文档需求)
  • 已确认测试结果或说明未执行原因

makecindy#3587 给 sessions schema 新增 writable_dirs 列后,drizzle 的全列查询
会让三个手工 CREATE TABLE 的测试(orcaTeamStore / sessionAutoTitlePersist /
sessionsRestoreIfArchived)报 no such column。按兄弟夹具
sessionsUpdate.test.ts 已有写法补上该列,恢复 related 门禁。

从 PR makecindy#3829 拆出:与侧栏 UI 建议稿正交,独立放行。

Signed-off-by: sakiko-toyokawa <203236246+sakiko-toyokawa@users.noreply.github.com>
@greptile-apps

greptile-apps Bot commented Sep 3, 2026

Copy link
Copy Markdown

Greptile Summary

此 PR 为三个手工创建 sessions 表的 localDb 测试夹具补充 writable_dirs TEXT NOT NULL DEFAULT '[]',使夹具 schema 与当前 Drizzle schema 保持一致。

  • 修复 orcaTeamStore 测试夹具缺失的 writable_dirs
  • 修复 sessionAutoTitlePersist 测试夹具缺失的 writable_dirs
  • 修复 sessionsRestoreIfArchived 测试夹具缺失的 writable_dirs

Confidence Score: 5/5

此 PR 仅同步三个测试夹具的 sessions 表结构,未发现阻碍合并的问题。

新增列在三处均采用一致的 TEXT、NOT NULL 和空数组默认值,修复了全列查询与夹具 schema 不匹配的问题,且不影响生产代码。

Important Files Changed

Filename Overview
apps/desktop/src/main/localDb/tests/orcaTeamStore.test.ts 为手工创建的 sessions 表补充默认值与当前 schema 一致的 writable_dirs 列,未发现问题。
apps/desktop/src/main/localDb/ipc/tests/sessionAutoTitlePersist.test.ts 为自动标题持久化测试夹具补充 writable_dirs 列,声明与其他目录字段及 PR 目标一致。
apps/desktop/src/main/localDb/ipc/tests/sessionsRestoreIfArchived.test.ts 为归档会话恢复测试夹具补充 writable_dirs 列,未引入运行时或测试语义风险。

Reviews (1): Last reviewed commit: "test(desktop): 补齐 localDb 夹具缺失的 sessions..." | Re-trigger Greptile

@MagicLizi MagicLizi added status:ci-running CI 还在跑(review-pr 自动维护,仅展示) and removed status:ci-running CI 还在跑(review-pr 自动维护,仅展示) labels Sep 3, 2026

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

代码重审通过:当前 head 把 sessions 测试夹具补上 writable_dirs TEXT NOT NULL DEFAULT '[]',与 schema / 0099 companion 一致,无生产路径改动,没有 P0/P1。全部 conversation 已 resolve。

@MagicLizi
MagicLizi merged commit 6710e5e into makecindy:main Sep 3, 2026
11 checks passed
@MagicLizi

Copy link
Copy Markdown
Contributor

三处 sessions 夹具补上 writable_dirs 默认空数组,related 单测终于能对着真实 schema 跑了,谢谢。

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