fix(fs)!: default macOS config dir to Application Support - #100
Conversation
|
Looks good, but I think we're going to unfortunately have to figure out what to do about people that already have important files in the incorrect location, because people may have created files like So for one, this is a breaking change, so we need to change the PR title/commits so they will release this as a breaking version number rather than the patch version bump of a fix. We also need to figure out a migration path for the cli to fix macOS installations. Something like:
I think we'll need that figured out before we integrate this engine change into gddy; it's probably best if we have the plan in place in advance so we don't have the engine change merged into main for too long. |
|
Good call, implemented a migration approach proposal, plus flagged this as breaking. Proposal for the migration, on macOS at startup (before config loads):
Let me know If this closes the gap or If this requires further discussion to flag |
🤖 I have created a release *beep* *boop* --- <details><summary>cli-engine: 0.9.0</summary> ## [0.9.0](cli-engine-v0.8.7...cli-engine-v0.9.0) (2026-08-24) ### ⚠ BREAKING CHANGES * **fs:** default macOS config dir to Application Support ([#100](#100)) * upgrade reqwest to 0.13 ([#102](#102)) ### Features * **error:** let DetailedError contribute structured next_actions to error envelopes ([#99](#99)) ([9c8988a](9c8988a)) ### Bug Fixes * **fs:** default macOS config dir to Application Support ([#100](#100)) ([a8a2291](a8a2291)) ### Build System * upgrade reqwest to 0.13 ([#102](#102)) ([3794ab7](3794ab7)) </details> --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Summary
Test plan
cargo fmt --all --checkcargo clippy --all-targets -- -D warningscargo test --all-targetsManual verification
Setup:
Test WITHOUT the fix (baseline):
Test WITH the fix:
Cleanup: