Skip to content

feat(settings): enable unused Git checkpoint cleanup by default#649

Merged
XingYu-Zhong merged 1 commit into
KunAgent:developfrom
musnows:codex/enable-checkpoint-cleanup-by-default
Jun 30, 2026
Merged

feat(settings): enable unused Git checkpoint cleanup by default#649
XingYu-Zhong merged 1 commit into
KunAgent:developfrom
musnows:codex/enable-checkpoint-cleanup-by-default

Conversation

@musnows

@musnows musnows commented Jun 29, 2026

Copy link
Copy Markdown
Collaborator

Summary

自动清理未使用的 Git checkpoint 默认是关闭的(opt-in)。本 PR 将其改为默认开启,避免 stale checkpoint 目录持续堆积。希望保留所有 checkpoint 的用户仍可在设置中手动关闭。

Changes

  • src/shared/app-settings-types.ts: DEFAULT_CHECKPOINT_CLEANUP_ENABLEDfalse 改为 true,并更新注释说明默认开启、可在设置中 opt out。
  • src/main/settings-store.test.ts: 引入 DEFAULT_CHECKPOINT_CLEANUP_ENABLED 并将断言改为基于常量断言,更新注释。
  • src/renderer/src/locales/en/settings.json: 描述由 "Off by default." 改为 "On by default."。
  • src/renderer/src/locales/zh/settings.json: 描述由 "默认关闭。" 改为 "默认开启。"。

Tests

  • npm run typecheck 通过
  • vitest run src/main/settings-store.test.ts(23 tests)通过
  • vitest run src/shared/app-settings.test.ts src/shared/app-settings-provider.test.ts src/renderer/src/components/settings-section-general.test.ts(106 tests)通过

@XingYu-Zhong XingYu-Zhong left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Reviewed locally; no blocking issues found. Confirmed the default applies when checkpoint cleanup settings are absent.

@XingYu-Zhong XingYu-Zhong merged commit b355bc1 into KunAgent:develop Jun 30, 2026
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