Skip to content

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

Merged
snehmatic merged 4 commits into
snehmatic:mainfrom
huyua9:codex/mindloop-config-atomic-20260813
Sep 3, 2026
Merged

fix(config): serialize user settings writes atomically#170
snehmatic merged 4 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

@snehmatic
snehmatic merged commit 3704bad into snehmatic:main Sep 3, 2026
3 checks passed
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

2 participants