Skip to content

fix: improve uninstall wizard cleanup#6

Merged
Lisztos merged 3 commits intomainfrom
fix/uninstall-cleanup
Mar 25, 2026
Merged

fix: improve uninstall wizard cleanup#6
Lisztos merged 3 commits intomainfrom
fix/uninstall-cleanup

Conversation

@Lisztos
Copy link
Copy Markdown
Owner

@Lisztos Lisztos commented Mar 24, 2026

Ported from parcadei/Continuous-Claude-v3#147 by @udhaya10


Summary

The uninstall wizard archives ~/.claude and restores backup, but leaves several artifacts behind.

Changes

  • settings.local.json added to PRESERVE_FILES — users store custom permissions here
  • Log preserve warnings instead of silent except Exception: pass
  • Remove CLAUDE_OPC_DIR from shell config on uninstall
  • Offer to remove TLDR CLIuv tool uninstall llm-tldr (default: No)

Test plan

  • Run --uninstall with CLAUDE_OPC_DIR in ~/.zshrc → verify it's removed
  • Run --uninstall with settings.local.json present → verify it's preserved
  • Simulate preserve failure → verify warning appears
  • Run --uninstall with TLDR installed → verify removal prompt appears

udhaya10 and others added 3 commits March 24, 2026 19:56
- Add settings.local.json to PRESERVE_FILES (user permissions)
- Log warnings instead of silently ignoring preserve failures
- Remove CLAUDE_OPC_DIR from shell config (~/.zshrc, ~/.bashrc)
- Offer to remove TLDR CLI tool (uv tool uninstall llm-tldr)

Closes parcadei#142
Wrap subprocess.run() in try/except to catch TimeoutExpired and OSError,
preventing the uninstall wizard from crashing if uv is missing or times out.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@Lisztos Lisztos force-pushed the fix/uninstall-cleanup branch from 97ce4d7 to 533afd1 Compare March 25, 2026 01:58
@Lisztos Lisztos merged commit 17e2f4f into main Mar 25, 2026
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.

2 participants