ci: add more test tools

Author: YoSev

.github/workflows/go.test.yml

@@ -1,19 +1,65 @@
-name: Go
-
+name: CI
 on:
   push:
+    branches:
+      - "**"
+    tags-ignore:
+      - "v*.*.*"
   pull_request:
-
+    branches:
+      - "**"
 jobs:
-  build:
+  tests:
+    name: Unit and integrations tests
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        go: ["1.25.1"]
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-go@v5
+        with:
+          go-version: ${{ matrix.go }}
+      - name: Run tests
+        run: |
+          make test+coverage
+      - uses: shogo82148/actions-goveralls@v1
+        with:
+          path-to-profile: coverage.txt
+
+
+  spellcheck:
+    name: Spellcheck
     runs-on: ubuntu-latest
     steps:
-    - uses: actions/checkout@v4
+      - uses: actions/checkout@v4
+      - uses: actions/setup-go@v5
+        with:
+          go-version: "1.25.1"
+          cache: false
+      - name: Run spellcheck
+        run: |
+          go install github.com/client9/misspell/cmd/misspell@latest
+          misspell -error docs
 
-    - name: Set up Go
-      uses: actions/setup-go@v4
-      with:
-        go-version: '1.23.4'
+  lint:
+    name: Lint
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-go@v5
+        with:
+          go-version: "1.25.1"
+          cache: false
+      - name: Run lint
+        uses: golangci/golangci-lint-action@v8
+        with:
+          version: v2.4
+          args: --timeout 5m
 
-    - name: Test
-      run: make test
+  govulncheck:
+    name: Check vulnerabilities
+    runs-on: ubuntu-latest
+    steps:
+      - id: govulncheck
+        uses: golang/govulncheck-action@v1

.gitignore

@@ -1,2 +1,3 @@
 test.go
-tmp/
+tmp/
+coverage.txt

.golangci.yml

@@ -0,0 +1,71 @@
+version: "2"
+linters:
+  enable:
+    - bodyclose
+    - copyloopvar
+    - gocyclo
+    - inamedparam
+    - misspell
+    - revive
+    - testifylint
+    - whitespace
+    - unused
+    - staticcheck
+    - ineffassign
+    - govet
+    - sqlclosecheck
+    - errcheck
+  settings:
+    gocyclo:
+      min-complexity: 18
+    govet:
+      disable:
+        - shadow
+        - fieldalignment
+      enable-all: true
+    misspell:
+      locale: US
+    revive:
+      rules:
+        - name: var-naming
+          disabled: false
+          arguments:
+            - [] # AllowList
+            - ["DTO"] # DenyList
+        - name: unused-parameter
+          disabled: false
+        - name: use-any
+          disabled: false
+    testifylint:
+      disable:
+        - require-error
+        - error-nil
+        - formatter
+        - encoded-compare
+    staticcheck:
+      checks: ["all", "-QF1001"]
+  exclusions:
+    generated: strict
+    rules:
+      - linters:
+          - gocyclo
+        path: _test\.go
+      - path: (.+)\.go$
+        text: should have a package comment
+    paths:
+      - .github
+      - resources
+issues:
+  max-issues-per-linter: 0
+  max-same-issues: 0
+formatters:
+  enable:
+    - gofmt
+  settings:
+    gofmt:
+      simplify: true
+  exclusions:
+    generated: strict
+    paths:
+      - .github
+      - resources

Makefile

@@ -1,6 +1,11 @@
 test:
 	go test -race -v ./...
-	
+
+test+coverage:
+	go test -race -coverprofile=coverage.txt -covermode=atomic -coverpkg=./... ./...
+
 changelog:
 	auto-changelog --output CHANGELOG.md
 
+lint:
+	golangci-lint run --timeout=5m ./...

README.md

@@ -4,6 +4,8 @@
   <img src="debugo.png" width="500" title="debugo">
 </p>
 
+[![Coverage Status](https://coveralls.io/repos/github/YoSev/debugo/badge.svg?branch=main)](https://coveralls.io/github/YoSev/debugo?branch=main)
+
 A lightweight, colorful, and flexible debugging utility for Go — inspired by [debug](https://github.com/debug-js/debug) in Node.js.
 
 `debugo` provides namespaced, timestamped, and color-coded logging with support for inclusion/exclusion patterns and pluggable output streams.

debugo.go

@@ -2,7 +2,6 @@ package debugo
 
 import (
 	"io"
-	"os"
 	"sync"
 	"time"
 
@@ -44,10 +43,6 @@ func (d *Debugger) SetOutput(output io.Writer) {
 	d.mutex.Lock()
 	defer d.mutex.Unlock()
 
-	if output == nil {
-		output = os.Stderr
-	}
-
 	d.output = output
 }
 

go.mod

@@ -1,6 +1,6 @@
 module github.com/yosev/debugo
 
-go 1.23.4
+go 1.25.1
 
 require (
 	github.com/fatih/color v1.18.0

namespace.go

@@ -49,10 +49,7 @@ func matchPattern(namespace, pattern string) bool {
 		base := strings.TrimSuffix(pattern, ":?")
 		// Match exactly the base or the base followed by anything
 		regexPattern := "^" + regexp.QuoteMeta(base) + "(:.*)?$"
-		re, err := regexp.Compile(regexPattern)
-		if err != nil {
-			return false
-		}
+		re, _ := regexp.Compile(regexPattern) // can not fail due to QuoteMeta
 		return re.MatchString(namespace)
 	}
 

namespace_test.go

@@ -11,77 +11,77 @@ func TestMatchNamespace(t *testing.T) {
 
 	// Empty debug string
 	SetNamespace("")
-	assert.Equal(t, l.matchNamespace(), false, "they should be equal")
+	assert.False(t, l.matchNamespace(), "they should be equal")
 
 	// Single wildcard
 	SetNamespace("*")
-	assert.Equal(t, l.matchNamespace(), true, "they should be equal")
+	assert.True(t, l.matchNamespace(), "they should be equal")
 
 	// Single negative pattern
 	SetNamespace("-test:*")
-	assert.Equal(t, l.matchNamespace(), false, "they should be equal")
+	assert.False(t, l.matchNamespace(), "they should be equal")
 
 	// Multiple negative patterns
 	SetNamespace("-test:a,-test:b")
-	assert.Equal(t, l.matchNamespace(), false, "they should be equal")
+	assert.False(t, l.matchNamespace(), "they should be equal")
 
 	// Leading/trailing spaces in debug string
 	SetNamespace("   test:a   ")
-	assert.Equal(t, l.matchNamespace(), true, "they should be equal")
+	assert.True(t, l.matchNamespace(), "they should be equal")
 
 	// Overlapping inclusion and exclusion
 	SetNamespace("test:*, -test:a")
-	assert.Equal(t, l.matchNamespace(), false, "they should be equal")
+	assert.False(t, l.matchNamespace(), "they should be equal")
 
 	// Overlapping inclusion and exclusion
 	SetNamespace("*, -test:a")
-	assert.Equal(t, l.matchNamespace(), false, "they should be equal")
+	assert.False(t, l.matchNamespace(), "they should be equal")
 
 	// Overlapping inclusion and exclusion
 	SetNamespace("*, -test:?")
-	assert.Equal(t, l.matchNamespace(), false, "they should be equal")
+	assert.False(t, l.matchNamespace(), "they should be equal")
 
 	// Overlapping inclusion and exclusion
 	SetNamespace("*, -test:*")
-	assert.Equal(t, l.matchNamespace(), false, "they should be equal")
+	assert.False(t, l.matchNamespace(), "they should be equal")
 
 	// Exact match
 	SetNamespace("test:a")
-	assert.Equal(t, l.matchNamespace(), true, "they should be equal")
+	assert.True(t, l.matchNamespace(), "they should be equal")
 
 	// Case sensitivity
 	SetNamespace("TEST:A")
-	assert.Equal(t, l.matchNamespace(), true, "they should be equal") // Assuming case-sensitive
+	assert.True(t, l.matchNamespace(), "they should be equal") // Assuming case-sensitive
 
 	// No namespace provided
 	l = New("")
 	SetNamespace("test:*")
-	assert.Equal(t, l.matchNamespace(), false, "they should be equal")
+	assert.False(t, l.matchNamespace(), "they should be equal")
 
 	// Non-matching wildcards
 	SetNamespace("test:x*")
-	assert.Equal(t, l.matchNamespace(), false, "they should be equal")
+	assert.False(t, l.matchNamespace(), "they should be equal")
 
 	// Invalid patterns
 	SetNamespace("--test:*")
-	assert.Equal(t, l.matchNamespace(), false, "they should be equal")
+	assert.False(t, l.matchNamespace(), "they should be equal")
 
 	// --- Optional pattern tests ---
 	l = New("info")
 	SetNamespace("info:?")
-	assert.Equal(t, l.matchNamespace(), true, "should match the base namespace")
+	assert.True(t, l.matchNamespace(), "should match the base namespace")
 
 	l = New("info:test")
 	SetNamespace("info:?")
-	assert.Equal(t, l.matchNamespace(), true, "should match the namespace with suffix")
+	assert.True(t, l.matchNamespace(), "should match the namespace with suffix")
 
 	l = New("info:test:sub")
 	SetNamespace("info:?")
-	assert.Equal(t, l.matchNamespace(), true, "should match namespace with deeper suffix")
+	assert.True(t, l.matchNamespace(), "should match namespace with deeper suffix")
 
 	l = New("other")
 	SetNamespace("info:?")
-	assert.Equal(t, l.matchNamespace(), false, "should not match unrelated namespace")
+	assert.False(t, l.matchNamespace(), "should not match unrelated namespace")
 }
 
 func TestMatchPattern(t *testing.T) {
@@ -133,3 +133,10 @@ func TestMatchPattern(t *testing.T) {
 		})
 	}
 }
+
+func TestMatchpatterns(t *testing.T) {
+	ok := matchPattern("namespace", "[invalid")
+	assert.False(t, ok)
+	ok = matchPattern("namespace", "[invalid:?")
+	assert.False(t, ok)
+}

runtime.go

@@ -32,23 +32,23 @@ type Timestamp struct {
 	Format string
 }
 
-// Set the global color usage for debugging.
+// SetUseColors sets the global color usage for debugging.
 func SetUseColors(use bool) {
 	runtime.mutex.Lock()
 	defer runtime.mutex.Unlock()
 
 	runtime.useColors = use
 }
 
-// GetNamespace retrieves the current global color usage for debugging.
+// GetUseColors retrieves the current global color usage for debugging.
 func GetUseColors() bool {
 	runtime.mutex.RLock()
 	defer runtime.mutex.RUnlock()
 
 	return runtime.useColors
 }
 
-// Set the global namespace for debugging.
+// SetNamespace sets the global namespace for debugging.
 func SetNamespace(namespace string) {
 	runtime.mutex.Lock()
 	defer runtime.mutex.Unlock()
@@ -64,31 +64,31 @@ func GetNamespace() string {
 	return runtime.namespace
 }
 
-// Sets the global timestamp configuration for debugging.
+// SetTimestamp sets the global timestamp configuration for debugging.
 func SetTimestamp(timestamp *Timestamp) {
 	runtime.mutex.Lock()
 	defer runtime.mutex.Unlock()
 
 	runtime.timestamp = timestamp
 }
 
-// Retrieves the current global timestamp configuration for debugging.
+// GetTimestamp retrieves the current global timestamp configuration for debugging.
 func GetTimestamp() *Timestamp {
 	runtime.mutex.RLock()
 	defer runtime.mutex.RUnlock()
 
 	return runtime.timestamp
 }
 
-// Sets the global output configuration for debugging.
+// SetOutput sets the global output configuration for debugging.
 func SetOutput(output io.Writer) {
 	runtime.mutex.Lock()
 	defer runtime.mutex.Unlock()
 
 	runtime.output = output
 }
 
-// Retrieves the current global output configuration for debugging.
+// GetOutput retrieves the current global output configuration for debugging.
 func GetOutput() io.Writer {
 	runtime.mutex.RLock()
 	defer runtime.mutex.RUnlock()