Skip to content

Commit 7049ebc

Browse files
maxholmanclaude
andcommitted
fix: disambiguate "Role resolved" log line and route display
The log "Role resolved: name=X role=entry" was confusing because it displayed the local node's resolved role next to the peer's name, reading as if the peer's role was entry. Changed to "Role resolved: peer=X local_role=entry" to prevent misreading. Also: route display uses "via" instead of arrow, stale subcommand reference removed from help text. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 508a7da commit 7049ebc

2 files changed

Lines changed: 143 additions & 58 deletions

File tree

crates/daemon/src/mode/auto.rs

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -419,8 +419,9 @@ async fn run_auto_connect_session_dispatch(
419419
..
420420
} => {
421421
tracing::info!(
422-
"Role resolved: name={} addr={peer_addr} role=entry",
422+
"Role resolved: peer={} addr={peer_addr} local_role=entry peer_role={}",
423423
peer_hs.name,
424+
super::peer_role_from_capabilities(peer_hs.capabilities.unwrap_or_default()),
424425
);
425426
node_state.update_role(NodeRole::Entry);
426427

@@ -478,8 +479,9 @@ async fn run_auto_connect_session_dispatch(
478479
..
479480
} => {
480481
tracing::info!(
481-
"Role resolved: name={} addr={peer_addr} role=exit",
482+
"Role resolved: peer={} addr={peer_addr} local_role=exit peer_role={}",
482483
peer_hs.name,
484+
super::peer_role_from_capabilities(peer_hs.capabilities.unwrap_or_default()),
483485
);
484486
node_state.update_role(NodeRole::Exit);
485487
let peer_caps = peer_hs.capabilities.unwrap_or_default();
@@ -877,8 +879,9 @@ async fn run_auto_accept_session_inner(
877879
..
878880
} => {
879881
tracing::info!(
880-
"Role resolved: name={} addr={peer_addr} role=entry",
882+
"Role resolved: peer={} addr={peer_addr} local_role=entry peer_role={}",
881883
peer_hs.name,
884+
super::peer_role_from_capabilities(peer_hs.capabilities.unwrap_or_default()),
882885
);
883886
node_state.update_role(NodeRole::Entry);
884887

@@ -1041,8 +1044,9 @@ async fn run_auto_accept_session_inner(
10411044
..
10421045
} => {
10431046
tracing::info!(
1044-
"Role resolved: name={} addr={peer_addr} role=exit",
1047+
"Role resolved: peer={} addr={peer_addr} local_role=exit peer_role={}",
10451048
peer_hs.name,
1049+
super::peer_role_from_capabilities(peer_hs.capabilities.unwrap_or_default()),
10461050
);
10471051
node_state.update_role(NodeRole::Exit);
10481052

0 commit comments

Comments
 (0)