feat: skip git module for non ascii check

go.mod

@@ -37,10 +37,10 @@ require (
 	github.com/sergi/go-diff v1.4.0 // indirect
 	github.com/skeema/knownhosts v1.3.1 // indirect
 	github.com/xanzy/ssh-agent v0.3.3 // indirect
-	golang.org/x/crypto v0.45.0 // indirect
+	golang.org/x/crypto v0.43.0 // indirect
-	golang.org/x/net v0.47.0 // indirect
+	golang.org/x/net v0.45.0 // indirect
-	golang.org/x/sys v0.38.0 // indirect
+	golang.org/x/sys v0.37.0 // indirect
-	golang.org/x/text v0.31.0 // indirect
+	golang.org/x/text v0.30.0 // indirect
 	google.golang.org/genproto/googleapis/rpc v0.0.0-20250929231259-57b25ae835d4 // indirect
 	gopkg.in/warnings.v0 v0.1.2 // indirect
 	gopkg.in/yaml.v2 v2.4.0 // indirect

go.sum

@@ -111,27 +111,27 @@ go.opentelemetry.io/otel/sdk/metric v1.37.0/go.mod h1:cNen4ZWfiD37l5NhS+Keb5RXVW
 go.opentelemetry.io/otel/trace v1.37.0 h1:HLdcFNbRQBE2imdSEgm/kwqmQj1Or1l/7bW6mxVK7z4=
 go.opentelemetry.io/otel/trace v1.37.0/go.mod h1:TlgrlQ+PtQO5XFerSPUYG0JSgGyryXewPGyayAWSBS0=
 golang.org/x/crypto v0.0.0-20220622213112-05595931fe9d/go.mod h1:IxCIyHEi3zRg3s0A5j5BB6A9Jmi73HwBIUl50j+osU4=
-golang.org/x/crypto v0.45.0 h1:jMBrvKuj23MTlT0bQEOBcAE0mjg8mK9RXFhRH6nyF3Q=
+golang.org/x/crypto v0.43.0 h1:dduJYIi3A3KOfdGOHX8AVZ/jGiyPa3IbBozJ5kNuE04=
-golang.org/x/crypto v0.45.0/go.mod h1:XTGrrkGJve7CYK7J8PEww4aY7gM3qMCElcJQ8n8JdX4=
+golang.org/x/crypto v0.43.0/go.mod h1:BFbav4mRNlXJL4wNeejLpWxB7wMbc79PdRGhWKncxR0=
 golang.org/x/exp v0.0.0-20240719175910-8a7402abbf56 h1:2dVuKD2vS7b0QIHQbpyTISPd0LeHDbnYEryqj5Q1ug8=
 golang.org/x/exp v0.0.0-20240719175910-8a7402abbf56/go.mod h1:M4RDyNAINzryxdtnbRXRL/OHtkFuWGRjvuhBJpk2IlY=
 golang.org/x/net v0.0.0-20211112202133-69e39bad7dc2/go.mod h1:9nx3DQGgdP8bBQD5qxJ1jj9UTztislL4KSBs9R2vV5Y=
-golang.org/x/net v0.47.0 h1:Mx+4dIFzqraBXUugkia1OOvlD6LemFo1ALMHjrXDOhY=
+golang.org/x/net v0.45.0 h1:RLBg5JKixCy82FtLJpeNlVM0nrSqpCRYzVU1n8kj0tM=
-golang.org/x/net v0.47.0/go.mod h1:/jNxtkgq5yWUGYkaZGqo27cfGZ1c5Nen03aYrrKpVRU=
+golang.org/x/net v0.45.0/go.mod h1:ECOoLqd5U3Lhyeyo/QDCEVQ4sNgYsqvCZ722XogGieY=
 golang.org/x/sys v0.0.0-20191026070338-33540a1f6037/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs=
 golang.org/x/sys v0.0.0-20201119102817-f84b799fce68/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs=
 golang.org/x/sys v0.0.0-20210124154548-22da62e12c0c/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs=
 golang.org/x/sys v0.0.0-20210423082822-04245dca01da/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs=
 golang.org/x/sys v0.0.0-20210615035016-665e8c7367d1/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
 golang.org/x/sys v0.0.0-20220715151400-c0bba94af5f8/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
-golang.org/x/sys v0.38.0 h1:3yZWxaJjBmCWXqhN1qh02AkOnCQ1poK6oF+a7xWL6Gc=
+golang.org/x/sys v0.37.0 h1:fdNQudmxPjkdUTPnLn5mdQv7Zwvbvpaxqs831goi9kQ=
-golang.org/x/sys v0.38.0/go.mod h1:OgkHotnGiDImocRcuBABYBEXf8A9a87e/uXjp9XT3ks=
+golang.org/x/sys v0.37.0/go.mod h1:OgkHotnGiDImocRcuBABYBEXf8A9a87e/uXjp9XT3ks=
 golang.org/x/term v0.0.0-20201126162022-7de9c90e9dd1/go.mod h1:bj7SfCRtBDWHUb9snDiAeCFNEtKQo2Wmx5Cou7ajbmo=
-golang.org/x/term v0.37.0 h1:8EGAD0qCmHYZg6J17DvsMy9/wJ7/D/4pV/wfnld5lTU=
+golang.org/x/term v0.36.0 h1:zMPR+aF8gfksFprF/Nc/rd1wRS1EI6nDBGyWAvDzx2Q=
-golang.org/x/term v0.37.0/go.mod h1:5pB4lxRNYYVZuTLmy8oR2BH8dflOR+IbTYFD8fi3254=
+golang.org/x/term v0.36.0/go.mod h1:Qu394IJq6V6dCBRgwqshf3mPF85AqzYEzofzRdZkWss=
 golang.org/x/text v0.3.6/go.mod h1:5Zoc/QRtKVWzQhOtBMvqHzDpF6irO9z98xDceosuGiQ=
-golang.org/x/text v0.31.0 h1:aC8ghyu4JhP8VojJ2lEHBnochRno1sgL6nEi9WGFGMM=
+golang.org/x/text v0.30.0 h1:yznKA/E9zq54KzlzBEAWn1NXSQ8DIp/NYMy88xJjl4k=
-golang.org/x/text v0.31.0/go.mod h1:tKRAlv61yKIjGGHX/4tP1LTbc13YSec1pxVEWXzfoeM=
+golang.org/x/text v0.30.0/go.mod h1:yDdHFIX9t+tORqspjENWgzaCVXgk0yYnYuSZ8UzzBVM=
 golang.org/x/tools v0.0.0-20180917221912-90fa682c2a6e/go.mod h1:n7NCudcB/nEzxVGmLbDWY5pfWTLqBcC2KZ6jyYvM4mQ=
 gonum.org/v1/gonum v0.16.0 h1:5+ul4Swaf3ESvrOnidPp4GZbzf0mxVQpDCYUQE7OJfk=
 gonum.org/v1/gonum v0.16.0/go.mod h1:fef3am4MQ93R2HHpKnLk4/Tbh/s0+wqD5nfa6Pnwy4E=

internal/parser/log/parser.go

@@ -5,48 +5,31 @@ import (
 	"encoding/json"
 	"fmt"
 	"log/slog"
-	"strings"
 
 	"github.com/joint-online-judge/JOJ3/internal/stage"
 )
 
 func (*Log) parse(executorResult stage.ExecutorResult, conf Conf) stage.ParserResult {
-	content, ok := executorResult.Files[conf.Filename]
+	content := executorResult.Files[conf.Filename]
-	if !ok {
+	var data map[string]any
-		slog.Error("file not found for log parser", "filename", conf.Filename)
+	err := json.Unmarshal([]byte(content), &data)
+	if err != nil {
+		slog.Error(conf.Msg, "error", err)
 		return stage.ParserResult{
 			Score:   0,
-			Comment: fmt.Sprintf("log parser: file %s not found", conf.Filename),
+			Comment: fmt.Sprintf("Failed to parse content: %s", err),
 		}
 	}
-	contentBytes := []byte(content)
+	args := make([]any, 0, len(data)*2)
-	var data map[string]any
+	for key, value := range data {
-	if err := json.Unmarshal(contentBytes, &data); err != nil {
+		args = append(args, key, value)
-		// Not a valid json or failed to unmarshal, log as raw string line by line.
-		for line := range strings.SplitSeq(content, "\n") {
-			if strings.TrimSpace(line) != "" {
-				slog.Default().Log(
-					context.Background(),
-					slog.Level(conf.Level),
-					conf.Msg,
-					"line",
-					line,
-				)
-			}
-		}
-	} else {
-		// Valid json, log as key-value pairs.
-		args := make([]any, 0, len(data)*2)
-		for key, value := range data {
-			args = append(args, key, value)
-		}
-		slog.Default().Log(
-			context.Background(),
-			slog.Level(conf.Level),
-			conf.Msg,
-			args...,
-		)
 	}
+	slog.Default().Log(
+		context.Background(),
+		slog.Level(conf.Level),
+		conf.Msg,
+		args...,
+	)
 	return stage.ParserResult{
 		Score:   0,
 		Comment: "",

pkg/healthcheck/nonascii.go

@@ -9,9 +9,43 @@ import (
 	"strings"
 	"unicode"
 
+	"github.com/go-git/go-git/v5"
 	"github.com/go-git/go-git/v5/plumbing/format/gitattributes"
 )
 
+// getSubmodulePathsFromGoGit uses the go-git library to open the repository
+// at the given root path and retrieve a list of all submodule paths.
+// It returns a set of submodule paths for efficient lookup.
+func getSubmodulePathsFromGoGit(root string) (map[string]struct{}, error) {
+	submodulePaths := make(map[string]struct{})
+
+	// Open the git repository at the given path.
+	repo, err := git.PlainOpen(root)
+	if err != nil {
+		if err == git.ErrRepositoryNotExists {
+			return submodulePaths, nil
+		}
+		return nil, fmt.Errorf("error opening git repository: %w", err)
+	}
+
+	worktree, err := repo.Worktree()
+	if err != nil {
+		return nil, fmt.Errorf("error getting worktree: %w", err)
+	}
+
+	// Get the list of submodules.
+	submodules, err := worktree.Submodules()
+	if err != nil {
+		return nil, fmt.Errorf("error getting submodules: %w", err)
+	}
+
+	for _, sm := range submodules {
+		submodulePaths[filepath.ToSlash(sm.Config().Path)] = struct{}{}
+	}
+
+	return submodulePaths, nil
+}
+
 // getNonASCII retrieves a list of files in the specified root directory that contain non-ASCII characters.
 // It searches for non-ASCII characters in each file's content and returns a list of paths to files containing non-ASCII characters.
 func getNonASCII(root string) ([]string, error) {
@@ -23,6 +57,11 @@ func getNonASCII(root string) ([]string, error) {
 		gitattrExist = false
 	}
 
+	submodules, err := getSubmodulePathsFromGoGit(root)
+	if err != nil {
+		return nil, err
+	}
+
 	if gitattrExist {
 		fs := os.DirFS(".")
 		f, err := fs.Open(".gitattributes")
@@ -42,18 +81,22 @@ func getNonASCII(root string) ([]string, error) {
 			return err
 		}
 
+		relPath, err := filepath.Rel(root, path)
+		if err != nil {
+			return err
+		}
+
 		if info.IsDir() {
 			if info.Name() == ".git" {
 				return filepath.SkipDir
 			}
+			if _, isSubmodule := submodules[relPath]; isSubmodule {
+				return filepath.SkipDir
+			}
 			return nil
 		}
 
 		if gitattrExist {
-			relPath, err := filepath.Rel(root, path)
-			if err != nil {
-				return err
-			}
 			ret, matched := matcher.Match(strings.Split(relPath, "/"), nil)
 			if matched && ret["text"].IsUnset() && !ret["text"].IsSet() {
 				return nil