[ChrisGVE]

Summary

• atomic_write() replaced symlinked settings.json with a regular file, breaking setups that symlink config files (e.g. dotfile managers, multi-machine sync)
• Now resolves symlink targets before the tempfile+rename so the link itself is preserved

Closes #653

Supersedes #675 — rebased on current develop. The portable hook path change from #675 is no longer needed since the codebase moved from shell scripts to rtk hook claude binary command.

Changes Made

• Added resolve_atomic_target() that detects symlinks and resolves to their target (absolute or relative)
• Updated atomic_write() to write through to the resolved target
• Added 2 unix-specific tests for absolute and relative symlink preservation

Testing

• cargo fmt --all --check
• cargo clippy --all-targets — no warnings
• cargo test --all — 1942 passed, 6 ignored
• Specific tests: cargo test init::tests::test_atomic_write
• Verified both fixes merge cleanly together (1947 tests pass combined)

Checklist

• Code follows project style guidelines
• Self-review completed
• No breaking changes introduced
• Tests added for new functionality

[CLAassistant]

All committers have signed the CLA.

[ChrisGVE]

@aeppling may I ask you to review this PR, this has been in the list for long and would help many user if it could be integrated into rtk. Thank you for your great work and for your time reviewing this PR.

[KuSh]

Hi, thanks for your contribution!
As it is a rebase of someone else submission, you'll have to add "Co-authored-by" trailer to the commit's message.

[ChrisGVE]

Thanks @KuSh for the review! Both items addressed:

1. fs::canonicalize — replaced custom resolve_atomic_target() with fs::canonicalize + fallback for non-existent paths (new file creation). Handles chained symlinks, relative paths, all edge cases.

2. Co-authored-by — added Co-authored-by: Roopesh <roopesh1724989@gmail.com> trailer (original author of fix(init): preserve settings.json symlink and portable hook path #675).

All tests green: cargo fmt, cargo clippy, cargo test — 1951 passed.

[KuSh]

LGTM, thanks