MAJOR: add aspell as aditional check for spelling errors

Author: oktalz

.github/workflows/actions.yaml

@@ -32,6 +32,8 @@ jobs:
     runs-on: ubuntu-latest
     needs: ["go_build"]
     steps:
+    - name: Install Aspell
+      run: sudo apt-get update && sudo apt-get install -y aspell
     - uses: actions/checkout@v4
     - name: Set up Go
       uses: actions/setup-go@v4
@@ -51,6 +53,8 @@ jobs:
     runs-on: ubuntu-latest
     needs: ["go_build"]
     steps:
+    - name: Install Aspell
+      run: sudo apt-get update && sudo apt-get install -y aspell
     - uses: actions/checkout@v4
       with:
         fetch-depth: 0

Dockerfile

@@ -1,10 +1,11 @@
-FROM golang:alpine as builder
+FROM golang:alpine AS builder
 RUN mkdir /build
 ADD . /build/
 WORKDIR /build
 RUN go build -o check
 
 FROM alpine:latest
+RUN apk --no-cache add aspell
 COPY --from=builder /build/check /check
 WORKDIR /
 ENTRYPOINT ["/check"]

aspell.go

@@ -0,0 +1,83 @@
+package main
+
+import (
+	"bytes"
+	"fmt"
+	"log"
+	"os"
+	"os/exec"
+	"slices"
+	"strings"
+)
+
+type Aspell []string
+
+func (a Aspell) Check(subject string) error {
+	var words []string
+	var badWords []string
+
+	data, err := checkWithAspellExec(subject)
+	if data != "" {
+		words = strings.Split(data, "\n")
+	}
+	if err != nil {
+		return err
+	}
+
+	for _, word := range words {
+		if len(word) < 1 {
+			continue
+		}
+		if !slices.Contains(a, word) {
+			badWords = append(badWords, word)
+		}
+	}
+
+	if len(badWords) > 0 {
+		return fmt.Errorf("aspell: %s", badWords)
+	}
+	return nil
+}
+
+func (a Aspell) CheckSubjects(subjects []string) error {
+	var response string
+	for _, subject := range subjects {
+		if err := a.Check(subject); err != nil {
+			response += fmt.Sprintf("%s\n", err)
+		}
+	}
+	if len(response) > 0 {
+		return fmt.Errorf("%s", response)
+	}
+	return nil
+}
+
+func LoadAspellExceptions(filename string) (Aspell, error) {
+	var words []string
+
+	var data []byte
+	var err error
+	if data, err = os.ReadFile(filename); err != nil {
+		log.Printf("warning: aspell exceptions file not found (%s)", err)
+	}
+	words = strings.Split(string(data), "\n")
+
+	return words, nil
+}
+
+func checkWithAspellExec(subject string) (string, error) {
+	cmd := exec.Command("aspell", "--list")
+	cmd.Stdin = strings.NewReader(subject)
+
+	var stdout, stderr bytes.Buffer
+	cmd.Stdout = &stdout
+	cmd.Stderr = &stderr
+	err := cmd.Run()
+	if err != nil {
+		log.Printf("aspell error: %s, stderr: %s", err, stderr.String())
+		return "", err
+	}
+
+	log.Printf("aspell output: stdout: %s, stderr: %s", stdout.String(), stderr.String())
+	return stdout.String() + stderr.String(), nil
+}

aspell_test.go

@@ -0,0 +1,28 @@
+package main
+
+import "testing"
+
+func Test_checkWithAspell(t *testing.T) {
+	aspell := Aspell{"config"}
+	tests := []struct {
+		name    string
+		subject string
+		wantErr bool
+	}{
+		{"OK", "BUG/MEDIUM: config: add default location of path to the configuration file", false},
+		{"error - flie", "BUG/MEDIUM: config: add default location of path to the configuration flie", true},
+		{"error - locatoin", "CLEANUP/MEDIUM: config: add default locatoin of path to the configuration file", true},
+		{"error - locatoin+flie", "CLEANUP/MEDIUM: config: add default locatoin of path to the configuration flie", true},
+	}
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			err := aspell.Check(tt.subject)
+			if tt.wantErr && err == nil {
+				t.Errorf("checkWithAspell() error = %v, wantErr %v", err, tt.wantErr)
+			}
+			if !tt.wantErr && err != nil {
+				t.Errorf("checkWithAspell() error = %v, wantErr %v", err, tt.wantErr)
+			}
+		})
+	}
+}

check.go

@@ -401,6 +401,11 @@ func main() {
 		repoPath = os.Args[1]
 	}
 
+	aspellExceptions, err := LoadAspellExceptions(path.Join(repoPath, ".aspell-exceptions"))
+	if err != nil {
+		log.Fatalf("error reading aspell exceptions: %s", err)
+	}
+
 	commitPolicy, err := LoadCommitPolicy(path.Join(repoPath, ".check-commit.yml"))
 	if err != nil {
 		log.Fatalf("error reading configuration: %s", err)
@@ -425,5 +430,11 @@ func main() {
 		log.Fatalf("%s\n", commitPolicy.HelpText)
 	}
 
+	err = aspellExceptions.CheckSubjects(subjects)
+	if err != nil {
+		log.Printf("encountered one or more commit message spelling errors\n")
+		log.Fatalf("%s\n", err)
+	}
+
 	log.Printf("check completed without errors\n")
 }