fix: suppress recursive hint when already configured #71

2026-04-10T13:29:39+08:00

razvandimescu commented

2026-04-10 13:29:39 +08:00

(Migrated from github.com)

Summary

sudo numa install no longer prints the "Want full DNS sovereignty?" hint when numa.toml already has upstream.mode = "recursive"
Replaces 3 hardcoded copies (macOS, Linux, Windows) with a shared print_recursive_hint() helper that loads the config first

Test plan

cargo test --lib — 142 passed
cargo clippy -- -D warnings — clean
cargo fmt --check — clean
CI green on all three platforms
sudo numa install with mode = "recursive" in numa.toml — hint should NOT appear
sudo numa install with default config — hint should appear

🤖 Generated with Claude Code

## Summary - `sudo numa install` no longer prints the "Want full DNS sovereignty?" hint when `numa.toml` already has `upstream.mode = "recursive"` - Replaces 3 hardcoded copies (macOS, Linux, Windows) with a shared `print_recursive_hint()` helper that loads the config first ## Test plan - [x] `cargo test --lib` — 142 passed - [x] `cargo clippy -- -D warnings` — clean - [x] `cargo fmt --check` — clean - [x] CI green on all three platforms - [x] `sudo numa install` with `mode = "recursive"` in numa.toml — hint should NOT appear - [x] `sudo numa install` with default config — hint should appear 🤖 Generated with [Claude Code](https://claude.com/claude-code)

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: dearsky/numa#71