Address code review feedback - improve readability and restore working directory

Copilot · nomeguy · Copilot · commit a27c139af63c · 2025-10-25T10:11:51.000Z
Co-authored-by: nomeguy &lt;85475922+nomeguy@users.noreply.github.com&gt;
diff --git a/ip/ip_test.go b/ip/ip_test.go
@@ -25,6 +25,7 @@ func TestIsAbroadIp(t *testing.T) {
 	wd, _ := os.Getwd()
 	if filepath.Base(wd) == "ip" {
 		os.Chdir("..")
+		defer os.Chdir(wd)
 	}
 
 	// Initialize the IP database
diff --git a/util/network.go b/util/network.go
@@ -45,5 +45,8 @@ func IsIntranetIp(ip string) bool {
 		return false
 	}
 
-	return parsedIP.IsPrivate() || parsedIP.IsLoopback() || parsedIP.IsLinkLocalUnicast() || parsedIP.IsLinkLocalMulticast()
+	return parsedIP.IsPrivate() ||
+		parsedIP.IsLoopback() ||
+		parsedIP.IsLinkLocalUnicast() ||
+		parsedIP.IsLinkLocalMulticast()
 }

Original file line number	Diff line number	Diff line change
`@@ -25,6 +25,7 @@ func TestIsAbroadIp(t *testing.T) {`
`25`	`25`	`wd, _ := os.Getwd()`
`26`	`26`	`if filepath.Base(wd) == "ip" {`
`27`	`27`	`os.Chdir("..")`
	`28`	`+ defer os.Chdir(wd)`
`28`	`29`	`}`
`29`	`30`
`30`	`31`	`// Initialize the IP database`
Original file line number	Diff line number	Diff line change
`@@ -45,5 +45,8 @@ func IsIntranetIp(ip string) bool {`
`45`	`45`	`return false`
`46`	`46`	`}`
`47`	`47`
`48`		`- return parsedIP.IsPrivate() \|\| parsedIP.IsLoopback() \|\| parsedIP.IsLinkLocalUnicast() \|\| parsedIP.IsLinkLocalMulticast()`
	`48`	`+ return parsedIP.IsPrivate() \|\|`
	`49`	`+ parsedIP.IsLoopback() \|\|`
	`50`	`+ parsedIP.IsLinkLocalUnicast() \|\|`
	`51`	`+ parsedIP.IsLinkLocalMulticast()`
`49`	`52`	`}`