Address various linter complains

.golangci.yml

@@ -1,4 +1,6 @@
 linters-settings:
+  exhaustive:
+    default-signifies-exhaustive: true
   errcheck:
     check-type-assertions: true
   goconst:
@@ -24,7 +26,6 @@ linters:
   enable:
     - bodyclose
     - deadcode
-    - depguard
     - dogsled
     - dupl
     - errcheck

cmd/wsget/main.go

@@ -11,7 +11,7 @@ import (
 	"github.com/ksysoev/wsget/pkg/ws"
 )
 
-var wsUrl string
+var wsURL string
 var OutputFH *os.File
 var InputFH *os.File
 
@@ -27,7 +27,7 @@ func init() {
 		os.Exit(0)
 	}
 
-	wsUrl = *url
+	wsURL = *url
 
 	if outputFile != nil && *outputFile != "" {
 		var err error
@@ -39,15 +39,16 @@ func init() {
 }
 
 func main() {
-	fmt.Println("Connecting to", wsUrl, "...")
-	wsInsp, err := ws.NewWS(wsUrl)
+	fmt.Println("Connecting to", wsURL, "...")
+	wsInsp, err := ws.NewWS(wsURL)
 	if err != nil {
 		log.Fatal(err)
 	}
-	fmt.Println("Connected")
 	defer wsInsp.Close()
 
-	cli := cli.NewCLI(wsInsp)
+	fmt.Println("Connected")
+
+	client := cli.NewCLI(wsInsp)
 
 	if InputFH != nil {
 		go func() {
@@ -61,7 +62,7 @@ func main() {
 		}()
 	}
 
-	err = cli.Run(OutputFH)
+	err = client.Run(OutputFH)
 	if err != nil {
 		log.Fatal(err)
 	}

pkg/cli/cli.go

@@ -18,6 +18,10 @@ const (
 
 	HISTORY_FILE  = ".wsget_history"
 	HISTORY_LIMIT = 100
+
+	MACOS_DELETE_KEY = 127
+
+	KEYBOARD_BUFFER_SIZE = 10
 )
 
 type CLI struct {
@@ -47,7 +51,7 @@ func (c *CLI) Run(outputFile *os.File) error {
 	defer keyboard.Close()
 	defer c.history.SaveToFile()
 
-	keysEvents, err := keyboard.GetKeys(10)
+	keysEvents, err := keyboard.GetKeys(KEYBOARD_BUFFER_SIZE)
 	if err != nil {
 		return err
 	}
@@ -57,6 +61,7 @@ func (c *CLI) Run(outputFile *os.File) error {
 	for {
 		select {
 		case event := <-keysEvents:
+			//nolint:gomnd
 			switch event.Key {
 			case keyboard.KeyCtrlC, keyboard.KeyCtrlD:
 				return nil
@@ -81,6 +86,8 @@ func (c *CLI) Run(outputFile *os.File) error {
 				}
 
 				fmt.Println("Connection Mode: Press ESC to enter Request mode")
+			default:
+				continue
 			}
 
 		case msg := <-c.wsConn.Messages:
@@ -113,6 +120,7 @@ func (c *CLI) requestMode(keyStream <-chan keyboard.KeyEvent) (string, error) {
 			return buffer, e.Err
 		}
 
+		//nolint:gomnd
 		switch e.Key {
 		case keyboard.KeyCtrlC, keyboard.KeyCtrlD:
 			return buffer, fmt.Errorf("interrupted")
@@ -136,7 +144,7 @@ func (c *CLI) requestMode(keyStream <-chan keyboard.KeyEvent) (string, error) {
 			buffer += "\n"
 			continue
 
-		case keyboard.KeyBackspace, keyboard.KeyDelete, 127:
+		case keyboard.KeyBackspace, keyboard.KeyDelete, MACOS_DELETE_KEY:
 			if len(buffer) == 0 {
 				continue
 			}

pkg/cli/history.go

@@ -8,14 +8,14 @@ import (
 )
 
 type History struct {
-	file     string
-	limit    uint
 	requests []string
+	fileName string
+	limit    uint
 }
 
-func NewHistory(file string, limit uint) *History {
+func NewHistory(fileName string, limit uint) *History {
 	h := &History{
-		file:     file,
+		fileName: fileName,
 		limit:    limit,
 		requests: make([]string, 0),
 	}
@@ -26,7 +26,7 @@ func NewHistory(file string, limit uint) *History {
 }
 
 func (h *History) loadFromFile() error {
-	fileHandler, err := os.OpenFile(h.file, os.O_RDONLY|os.O_CREATE, 0644)
+	fileHandler, err := os.OpenFile(h.fileName, os.O_RDONLY|os.O_CREATE, 0644)
 	if err != nil {
 		log.Println("Error opening history file:", err)
 		return err
@@ -57,7 +57,7 @@ func (h *History) loadFromFile() error {
 }
 
 func (h *History) SaveToFile() error {
-	fileHandler, err := os.OpenFile(h.file, os.O_WRONLY|os.O_CREATE, 0644)
+	fileHandler, err := os.OpenFile(h.fileName, os.O_WRONLY|os.O_CREATE, 0644)
 	if err != nil {
 		log.Println("Error opening history file:", err)
 		return err

pkg/formater/formater.go

@@ -58,8 +58,10 @@ func (f *Formater) formatTestMessage(msgType ws.MessageType, data string) (strin
 		return f.text.FormatRequest(data)
 	case ws.Response:
 		return f.text.FormatResponse(data)
-	default:
+	case ws.NotDefined:
 		return "", fmt.Errorf("unknown message type")
+	default:
+		panic("Unexpected message type: " + string(msgType))
 	}
 }
 
@@ -70,8 +72,10 @@ func (f *Formater) formatJsonMessage(msgType ws.MessageType, data any) (string,
 		return f.json.FormatRequest(data)
 	case ws.Response:
 		return f.json.FormatResponse(data)
-	default:
+	case ws.NotDefined:
 		return "", fmt.Errorf("unknown message type")
+	default:
+		panic("Unexpected message type: " + string(msgType))
 	}
 }
 

pkg/ws/ws.go

@@ -14,9 +14,13 @@ const (
 	Response
 )
 
+const (
+	WS_MESSAGE_BUFFER_SIZE = 100
+)
+
 type Message struct {
-	Type MessageType `json:"type"`
 	Data string      `json:"data"`
+	Type MessageType `json:"type"`
 }
 
 type WSConnection struct {
@@ -31,7 +35,7 @@ func NewWS(url string) (*WSConnection, error) {
 		return nil, err
 	}
 
-	messages := make(chan Message, 100)
+	messages := make(chan Message, WS_MESSAGE_BUFFER_SIZE)
 
 	go func(messages chan Message) {
 		for {