Feature implementation from commits 29e37e8..2d3126b

[codeOwlAI]

PR Summary

Core Version Update and Network Configuration Enhancements

Overview

This PR updates the core version value, enhances configuration error handling, and improves network interface binding functionality across different platforms.

Change Types

Type	Description
Enhancement	Improved error reporting with more context
Enhancement	Better handling of socket binding for network interfaces
Refactor	Modified JSON marshaling for ports and improved Close() method implementation

---

Affected Modules

Module / File	Change Description
core.go	Changed Version_z byte value from 6 to 31
conf/common.go	Enhanced MarshalJSON implementation for ports with better type handling
conf/xray.go	Improved error messages and added protocol information for debugging
wireguard/gvisortun/tun.go	Modified Close() method to use sync.Once for safe execution
internet/sockopt_darwin.go	Removed functionality binding sockets to specific network interfaces
internet/sockopt_windows.go	Added socket configuration for multicast interfaces

---

Notes for Reviewers

• Please review the socket binding changes carefully as there are conflicting changes in the Darwin implementation
• Verify that the Version_z change from 6 to 31 doesn't cause compatibility issues

[codeowlai]

🐛 Correctness Issue

Breaking API Change: Receiver Type Changed.

Changed from value receiver to pointer receiver which breaks backward compatibility for any code calling MarshalJSON on non-pointer values.

Current Code (Diff):

- func (v *PortList) MarshalJSON() ([]byte, error) {
+ func (v PortList) MarshalJSON() ([]byte, error) {

📝 Committable suggestion

‼️ IMPORTANT
Trust, but verify! 🕵️ Please review this suggestion with the care of a code archaeologist - check that it perfectly replaces the highlighted code, preserves all lines, maintains proper indentation, and won't break anything in production. Your future self will thank you! 🚀

Suggested change

	func (v *PortList) MarshalJSON() ([]byte, error) {
	func (v PortList) MarshalJSON() ([]byte, error) {

[codeowlai]

🐛 Correctness Issue

Inconsistent JSON Type Serialization.

The new implementation creates inconsistent JSON types where single ports become numbers while ranges become strings, which can break clients expecting consistent types.

Current Code (Diff):

- 	port, err := strconv.Atoi(portStr)
- 	if err == nil {
- 		return json.Marshal(port)
- 	} else {
- 		return json.Marshal(portStr)
- 	}
+ 	return json.Marshal(portStr)

📝 Committable suggestion

‼️ IMPORTANT
Trust, but verify! 🕵️ Please review this suggestion with the care of a code archaeologist - check that it perfectly replaces the highlighted code, preserves all lines, maintains proper indentation, and won't break anything in production. Your future self will thank you! 🚀

Suggested change

	port, err := strconv.Atoi(portStr)
	if err == nil {
	return json.Marshal(port)
	} else {
	return json.Marshal(portStr)
	}
	return json.Marshal(portStr)

[codeowlai]

🐛 Correctness Issue

Potential panic on nil channel close.

Removing the nil check before closing channels can cause a panic if Close() is called before the channels are initialized.

Current Code (Diff):

- 		close(tun.events)
+ 		if tun.events != nil {
+ 			close(tun.events)
+ 		}

📝 Committable suggestion

‼️ IMPORTANT
Trust, but verify! 🕵️ Please review this suggestion with the care of a code archaeologist - check that it perfectly replaces the highlighted code, preserves all lines, maintains proper indentation, and won't break anything in production. Your future self will thank you! 🚀

Suggested change

	close(tun.events)
	if tun.events != nil {
	close(tun.events)
	}

[codeowlai]

🐛 Correctness Issue

Potential panic on nil channel close.

Removing the nil check before closing incomingPacket channel can cause a panic if Close() is called before the channel is initialized.

Current Code (Diff):

- 		close(tun.incomingPacket)
+ 		if tun.incomingPacket != nil {
+ 			close(tun.incomingPacket)
+ 		}

📝 Committable suggestion

‼️ IMPORTANT
Trust, but verify! 🕵️ Please review this suggestion with the care of a code archaeologist - check that it perfectly replaces the highlighted code, preserves all lines, maintains proper indentation, and won't break anything in production. Your future self will thank you! 🚀

Suggested change

	close(tun.incomingPacket)
	if tun.incomingPacket != nil {
	close(tun.incomingPacket)
	}