Skip to content

fix(config): serialize user settings writes atomically - #170

Open
huyua9 wants to merge 3 commits into
snehmatic:mainfrom
huyua9:codex/mindloop-config-atomic-20260813
Open

fix(config): serialize user settings writes atomically#170
huyua9 wants to merge 3 commits into
snehmatic:mainfrom
huyua9:codex/mindloop-config-atomic-20260813

Conversation

@huyua9

@huyua9 huyua9 commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Summary

  • write user_config.yaml through a same-directory temporary file and atomic rename
  • serialize read-modify-write updates used by settings, editor width, clean-slate, and CLI configuration
  • preserve unrelated settings such as editor width during the settings form update

Validation

  • go test -race ./internal/config
  • go test ./... -run ^
  • go vet ./...
  • Full go test ./... is blocked by the existing TestHabitFlow expectation for success=done; HEAD 0fe2cd7 reproduces the same failure.

Closes #25

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.

New Notes feature

1 participant