Skip to content

fix: setup hygiene — parse-miss preservation, atomic writes, timestamped backups (supersedes #348)#376

Merged
buildingjoshbetter merged 1 commit into
mainfrom
fix/setup-hygiene
May 22, 2026
Merged

fix: setup hygiene — parse-miss preservation, atomic writes, timestamped backups (supersedes #348)#376
buildingjoshbetter merged 1 commit into
mainfrom
fix/setup-hygiene

Conversation

@buildingjoshbetter
Copy link
Copy Markdown
Owner

Summary

  1. mcp_server.py: Parse-miss on claude mcp list preserves existing config instead of clobbering
  2. cli.py: Atomic settings.json write via tmp + rename
  3. cli.py: Timestamped CLAUDE.md.bak prevents overwrite on re-install

2 files, +17/-5. Clean rewrite of Hunter's #348 (stripped #346 duplication and 300-line test file). Supersedes #348.

Co-Authored-By: Huntehhh hunter@users.noreply.github.com

…ped backups (supersedes #348)

1. mcp_server.py: Empty existing_cmd from parse miss now preserves
   config instead of clobbering it (Claude Code + Desktop).
2. cli.py: settings.json written via tmp + rename for atomicity.
3. cli.py: CLAUDE.md.bak includes timestamp to prevent overwrite.

Based on Huntehhh's PR #348. Supersedes #348.

Co-Authored-By: Huntehhh <hunter@users.noreply.github.com>
@buildingjoshbetter buildingjoshbetter merged commit 658b00a into main May 22, 2026
6 of 18 checks passed
@buildingjoshbetter buildingjoshbetter deleted the fix/setup-hygiene branch May 22, 2026 02:49
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