MAJOR: add aspell as additional check for spelling errors

Author: oktalz

.aspell.yml

@@ -0,0 +1,3 @@
+mode: subject
+allowed:
+  - aspell

.github/workflows/actions.yaml

@@ -6,8 +6,13 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
+      - name: Set up Go
+        uses: actions/setup-go@v4
+        with:
+          go-version-file: 'go.mod'
+          check-latest: true
       - name: golangci-lint
-        uses: golangci/golangci-lint-action@v3
+        uses: golangci/golangci-lint-action@v6
   go_build:
     name: Go build
     runs-on: ubuntu-latest
@@ -32,6 +37,8 @@ jobs:
     runs-on: ubuntu-latest
     needs: ["go_build"]
     steps:
+    - name: Install Aspell
+      run: sudo apt-get update && sudo apt-get install -y aspell aspell-en
     - uses: actions/checkout@v4
     - name: Set up Go
       uses: actions/setup-go@v4
@@ -51,6 +58,8 @@ jobs:
     runs-on: ubuntu-latest
     needs: ["go_build"]
     steps:
+    - name: Install Aspell
+      run: sudo apt-get update && sudo apt-get install -y aspell aspell-en
     - 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 aspell-en
 COPY --from=builder /build/check /check
 WORKDIR /
 ENTRYPOINT ["/check"]

README.md

@@ -95,3 +95,24 @@ TagOrder:
 ### Optional parameters
 
 The program accepts an optional parameter to specify the location (path) of the base of the git repository. This can be useful in certain cases where the checked-out repo is in a non-standard location within the CI environment, compared to the running path from which the check-commit binary is being invoked.
+
+### aspell
+
+to check also spell checking errors aspell was added. it can be configured with `.aspell.yml`
+
+example
+```yaml
+mode: subject
+allowed:
+  - aspell
+```
+
+mode can be set as
+
+- `subject`
+  - `default` option
+  - only subject of commit message can will be checked
+- `commit`
+  - whole commit message will be checked
+- `disabled`
+  - check is disabled

aspell/aspell.go

@@ -0,0 +1,81 @@
+package aspell
+
+import (
+	"bytes"
+	"fmt"
+	"log"
+	"os/exec"
+	"slices"
+	"strings"
+)
+
+type Aspell struct {
+	Mode         mode     `yaml:"mode"`
+	AllowedWords []string `yaml:"allowed"`
+}
+
+func (a Aspell) checkSingle(data string) error {
+	var words []string
+	var badWords []string
+
+	checkRes, err := checkWithAspellExec(data)
+	if checkRes != "" {
+		words = strings.Split(checkRes, "\n")
+	}
+	if err != nil {
+		return err
+	}
+
+	for _, word := range words {
+		if len(word) < 1 {
+			continue
+		}
+		if !slices.Contains(a.AllowedWords, word) {
+			badWords = append(badWords, word)
+		}
+	}
+
+	if len(badWords) > 0 {
+		return fmt.Errorf("aspell: %s", badWords)
+	}
+	return nil
+}
+
+func (a Aspell) Check(subjects []string, content []string) error {
+	var checks []string
+	switch a.Mode {
+	case modeDisabled:
+		return nil
+	case modeSubject:
+		checks = subjects
+	default:
+		checks = content
+	}
+
+	var response string
+	for _, subject := range checks {
+		if err := a.checkSingle(subject); err != nil {
+			response += fmt.Sprintf("%s\n", err)
+		}
+	}
+	if len(response) > 0 {
+		return fmt.Errorf("%s", response)
+	}
+	return 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
+	}
+
+	return stdout.String() + stderr.String(), nil
+}

aspell/aspell_test.go

@@ -0,0 +1,31 @@
+package aspell
+
+import "testing"
+
+func Test_checkWithAspell(t *testing.T) {
+	aspell := Aspell{
+		Mode:         modeSubject,
+		AllowedWords: []string{"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.checkSingle(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)
+			}
+		})
+	}
+}

aspell/mode.go

@@ -0,0 +1,10 @@
+package aspell
+
+type mode string
+
+const (
+	modeDisabled mode = "disabled"
+	modeSubject  mode = "subject"
+	modeCommit   mode = "commit"
+	// modeAll     mode = "all"
+)

aspell/new.go

@@ -0,0 +1,37 @@
+package aspell
+
+import (
+	"fmt"
+	"log"
+	"os"
+
+	"gopkg.in/yaml.v3"
+)
+
+func New(filename string) (Aspell, error) {
+	var data []byte
+	var err error
+	if data, err = os.ReadFile(filename); err != nil {
+		log.Printf("warning: aspell exceptions file not found (%s)", err)
+	}
+
+	var aspell Aspell
+	err = yaml.Unmarshal(data, &aspell)
+	if err != nil {
+		return Aspell{}, err
+	}
+
+	switch aspell.Mode {
+	case modeDisabled:
+	case modeSubject:
+	case modeCommit:
+	case "":
+		aspell.Mode = modeSubject
+	default:
+		return Aspell{}, fmt.Errorf("invalid mode: %s", aspell.Mode)
+	}
+
+	log.Printf("aspell mode set to %s", aspell.Mode)
+
+	return aspell, nil
+}