style: refactor inline error handling to standard Go pattern

[blink-so]

Overview

Refactored all inline error handling patterns throughout the codebase to use the standard Go idiom of separate assignment and error checking.

Changes

Before:

if err := someFunction(); err != nil {
    return fmt.Errorf("failed: %v", err)
}

After:

err := someFunction()
if err != nil {
    return fmt.Errorf("failed: %v", err)
}

Files Updated

• network/linux.go: 22 inline error handlers refactored
• network/macos.go: 8 inline error handlers refactored
• proxy/proxy.go: 2 inline error handlers refactored
• tls/tls.go: 3 inline error handlers refactored
• cli/cli.go: 7 inline error handlers refactored

Total: 42 inline error handlers converted to standard Go pattern

Benefits

• Improved readability: Clearer separation of function calls and error handling
• Go best practices: Follows standard Go idioms and conventions
• Consistency: Uniform error handling pattern across the entire codebase
• Debugging: Easier to set breakpoints on function calls vs error checks

Testing

• ✅ All existing tests pass
• ✅ Build succeeds without warnings
• ✅ No functional changes - purely stylistic refactoring