Skip to content

fix: avoid syncing .codex state#67

Merged
shunkakinoki merged 1 commit intomainfrom
codex/avoid-codex-dotdir-sync
Feb 7, 2026
Merged

fix: avoid syncing .codex state#67
shunkakinoki merged 1 commit intomainfrom
codex/avoid-codex-dotdir-sync

Conversation

@shunkakinoki
Copy link
Owner

@shunkakinoki shunkakinoki commented Feb 7, 2026

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.

Copilot AI review requested due to automatic review settings February 7, 2026 21:22
@shunkakinoki shunkakinoki merged commit 6e8f15d into main Feb 7, 2026
3 of 4 checks passed
@shunkakinoki shunkakinoki deleted the codex/avoid-codex-dotdir-sync branch February 7, 2026 21:22
@coderabbitai
Copy link

coderabbitai bot commented Feb 7, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Summary by CodeRabbit

  • Chores
    • Excluded .codex directory from automatic directory synchronization in build operations.

Walkthrough

The 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

Cohort / File(s) Summary
Makefile Configuration
Makefile
Removed .codex from DOTDIRS variable and added explanatory comment regarding runtime state and potential conflicts during make switch operations.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Poem

🐰 A codex removed, the Makefile now sings,
.codex excluded from the dot-directory rings,
A comment clarifies what might clobber our way,
Cleaner syncs await when we make-switch each day! 🎯

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch codex/avoid-codex-dotdir-sync

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 1 file

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

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 .codex from DOTDIRS so ruler-dotdirs-sync no longer mirrors it into $HOME.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

1 participant