Skip to content

feat(dns): complete inspection UX - #1163

Merged
ryanfowler merged 1 commit into
mainfrom
feat/dns-inspection-final-ux
Aug 30, 2026
Merged

feat(dns): complete inspection UX#1163
ryanfowler merged 1 commit into
mainfrom
feat/dns-inspection-final-ux

Conversation

@ryanfowler

Copy link
Copy Markdown
Owner

Summary

  • sort every DNS record family semantically and deterministically, including canonical HTTPS/SVCB parameters and numeric unknown RR types
  • split DNS inspection orchestration, lookup/parsing, result rendering, and record rendering into focused files
  • expand unit and CLI integration coverage for ordering, stream separation, complete and partial results, warnings, and exit codes
  • update user documentation and bundled agent diagnostics guidance, and complete the side-by-side DNS/TLS UX review

Validation

  • go test -v -parallel 8 ./...
  • go test -race ./internal/dnsinspect ./integration
  • CGO_ENABLED=0 go build .
  • go mod verify
  • staticcheck ./...
  • live fetch --inspect-dns example.com / fetch --inspect-tls example.com comparison

Review

Two independent subagent review rounds were completed. Round-one findings were addressed; round two reported no actionable findings.

@ryanfowler
ryanfowler enabled auto-merge August 30, 2026 20:09
@ryanfowler
ryanfowler merged commit 1ab50b4 into main Aug 30, 2026
12 checks passed
@ryanfowler
ryanfowler deleted the feat/dns-inspection-final-ux branch August 30, 2026 20:11
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