Conversation
|
Caution Review failedThe pull request is closed. 📝 WalkthroughSummary by CodeRabbit
WalkthroughThe Makefile's DOTDIRS variable was modified to exclude the .codex directory and includes a clarifying comment about runtime state and potential clobbering during make switch operations. This changes the set of dot directories synced by related targets without altering their underlying logic. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes Poem
✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Pull request overview
This PR updates the dot-directory sync list to avoid syncing Codex’s ~/.codex runtime state into a user’s home directory during the repo’s sync workflow, preventing accidental overwrites of Nix-managed Codex configuration.
Changes:
- Add a Makefile note explaining why
.codex/should not be synced wholesale. - Remove
.codexfromDOTDIRSsoruler-dotdirs-syncno longer mirrors it into$HOME.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Summary by cubic
Stop syncing the ~/.codex directory in dotfile sync. Removing .codex from DOTDIRS prevents overwriting Nix-managed ~/.codex/config.toml and avoids syncing runtime state (auth, history, sessions) during make switch.
Written for commit e389c0a. Summary will update on new commits.