🎨 chore: resolve some code style error and update some comments

envutil/envutil_test.go

@@ -81,7 +81,7 @@ func TestSetEnvs(t *testing.T) {
 		assert.Eq(t, val, Getenv(key))
 	}
 	assert.Panics(t, func() {
-		SetEnvs("name")
+		SetEnvs("name", "one", "two")
 	})
 
 	UnsetEnvs(keys...)

stdio/stdio.go

@@ -61,28 +61,28 @@ func NewScanner(in any) *bufio.Scanner {
 	}
 }
 
-// WriteByte to stdout
+// WriteByte to stdout, will ignore error
 func WriteByte(b byte) {
 	_, _ = os.Stdout.Write([]byte{b})
 }
 
-// WriteBytes to stdout
+// WriteBytes to stdout, will ignore error
 func WriteBytes(bs []byte) {
 	_, _ = os.Stdout.Write(bs)
 }
 
-// WritelnBytes to stdout
+// WritelnBytes to stdout, will ignore error
 func WritelnBytes(bs []byte) {
 	_, _ = os.Stdout.Write(bs)
 	_, _ = os.Stdout.Write([]byte("\n"))
 }
 
-// WriteString to stdout
+// WriteString to stdout, will ignore error
 func WriteString(s string) {
 	_, _ = os.Stdout.WriteString(s)
 }
 
-// Writeln string to stdout
+// Writeln string to stdout, will ignore error
 func Writeln(s string) {
 	_, _ = os.Stdout.WriteString(s)
 	_, _ = os.Stdout.Write([]byte("\n"))

structs/smap.go

@@ -1,6 +1,6 @@
 package structs
 
-// SMap simple map[string]string struct.
+// SMap simple map[string]string struct. TODO
 type SMap struct {
-	data map[string]string
+	data map[string]string //lint:ignore U1000 for unused
 }

structs/wrapper.go

@@ -16,8 +16,8 @@ type Wrapper struct {
 	FieldTagName string
 
 	// caches for field rv and name and tag name TODO
-	fieldNames []string
-	fvCacheMap map[string]reflect.Value
+	fieldNames []string                 //lint:ignore U1000 for unused
+	fvCacheMap map[string]reflect.Value //lint:ignore U1000 for unused
 }
 
 // Wrap create a struct wrapper

sysutil/clipboard/clipboard.go

@@ -79,12 +79,12 @@ func (c *Clipboard) Reset() error {
 // ---------------------------------------- write ----------------------------------------
 //
 
-// Write bytes data to clipboard
+// Write bytes data to buffer. should call Flush() to write to clipboard
 func (c *Clipboard) Write(p []byte) (int, error) {
 	return c.WriteString(string(p))
 }
 
-// WriteString data to clipboard
+// WriteString data to buffer. should call Flush() to write to clipboard
 func (c *Clipboard) WriteString(s string) (int, error) {
 	// if c.addSlashes {
 	// 	s = strutil.AddSlashes(s)
@@ -104,8 +104,6 @@ func (c *Clipboard) Flush() error {
 
 // WriteFromFile contents to clipboard
 func (c *Clipboard) WriteFromFile(filepath string) error {
-	// eg:
-	// 	Mac: pbcopy < tempfile.txt
 	file, err := fsutil.OpenReadFile(filepath)
 	if err != nil {
 		return err
@@ -134,7 +132,7 @@ func (c *Clipboard) WriteFrom(r io.Reader) error {
 // ---------------------------------------- read ----------------------------------------
 //
 
-// Read contents from clipboard
+// Read bytes contents from clipboard
 func (c *Clipboard) Read() ([]byte, error) {
 	buf, err := c.ReadToBuffer()
 	if err != nil {
@@ -143,7 +141,7 @@ func (c *Clipboard) Read() ([]byte, error) {
 	return buf.Bytes(), nil
 }
 
-// ReadToBuffer contents from clipboard
+// ReadToBuffer read clipboard contents to new buffer.
 func (c *Clipboard) ReadToBuffer() (*bytes.Buffer, error) {
 	var buf bytes.Buffer
 	if err := c.ReadTo(&buf); err != nil {
@@ -152,6 +150,15 @@ func (c *Clipboard) ReadToBuffer() (*bytes.Buffer, error) {
 	return &buf, nil
 }
 
+// SafeString read contents as string from clipboard, will return empty string on error
+func (c *Clipboard) SafeString() string {
+	s, err := c.ReadString()
+	if err != nil {
+		return ""
+	}
+	return s
+}
+
 // ReadString contents as string from clipboard
 func (c *Clipboard) ReadString() (string, error) {
 	bts, err := c.Read()

sysutil/clipboard/clipboard_test.go

@@ -1,6 +1,7 @@
 package clipboard_test
 
 import (
+	"fmt"
 	"testing"
 
 	"github.com/gookit/goutil/fsutil"
@@ -17,6 +18,10 @@ func TestClipboard_WriteFromFile(t *testing.T) {
 		return
 	}
 
+	cb.WithVerbose(true)
+	assert.True(t, cb.Writeable())
+	assert.True(t, cb.Readable())
+
 	srcFile := "testdata/testcb.txt"
 	srcStr := string(fsutil.MustReadFile(srcFile))
 	assert.NotEmpty(t, srcStr)
@@ -38,4 +43,12 @@ func TestClipboard_WriteFromFile(t *testing.T) {
 
 	dstStr := string(fsutil.MustReadFile(dstFile))
 	assert.Eq(t, srcStr, strutil.Trim(dstStr))
+
+	assert.NoErr(t, cb.Clean())
+
+	_, err = cb.Write([]byte("hello"))
+	assert.NoErr(t, err)
+	assert.NoErr(t, cb.Flush())
+	assert.Eq(t, "hello", cb.SafeString())
+	fmt.Println("...end...")
 }

sysutil/clipboard/util.go

@@ -42,11 +42,12 @@ const (
 )
 
 var (
-	writerOnLin = []string{"xclip", "xsel"}
-)
+	// TODO select an valid driver on New()
+	writerOnLin = []string{"xclip", "xsel"} //lint:ignore U1000 for TODO
 
-// std instance
-var std = New()
+	// std instance
+	std = New()
+)
 
 // Std get
 func Std() *Clipboard {