Skip to content

Add readable diff formatting for DNS record changes#3

Merged
cosimo merged 2 commits into
mainfrom
dev/better-dry-run-diff
Jun 15, 2026
Merged

Add readable diff formatting for DNS record changes#3
cosimo merged 2 commits into
mainfrom
dev/better-dry-run-diff

Conversation

@cosimo

@cosimo cosimo commented Dec 19, 2025

Copy link
Copy Markdown
Owner

Implements color-coded output (green=CREATE, red=DELETE, cyan=UPSERT) with detailed before/after diffs for UPSERT operations. Shows field changes (e.g., "TTL: 300 -> 600") and value changes (e.g., "- old_value", "+ new_value") for better visibility into what's being updated.

Fixes record matching to use Name, Type, and SetIdentifier to correctly handle multiple records with the same name (e.g., MX and TXT records, or multiple weighted A records with different SetIdentifiers).

🤖 Generated with Claude Code

cosimo-kahoot and others added 2 commits June 15, 2026 15:13
Implements color-coded output (green=CREATE, red=DELETE, cyan=UPSERT) with detailed before/after diffs for UPSERT operations. Shows field changes (e.g., "TTL: 300 -> 600") and value changes (e.g., "- old_value", "+ new_value") for better visibility into what's being updated.

Fixes record matching to use Name, Type, and SetIdentifier to correctly handle multiple records with the same name (e.g., MX and TXT records, or multiple weighted A records with different SetIdentifiers).

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 and Opus 4.8 <noreply@anthropic.com>
Run black and isort --profile black over app.py and the formatter tests:
quote normalization, import ordering, and line wrapping only. No logic
changes.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@cosimo cosimo force-pushed the dev/better-dry-run-diff branch from d3301c7 to d3099b9 Compare June 15, 2026 13:17
@cosimo cosimo merged commit b2bd6ef into main Jun 15, 2026
8 checks passed
@cosimo cosimo deleted the dev/better-dry-run-diff branch June 15, 2026 13:19
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