feat: printing all the errors from goparser

gnovm/cmd/gno/lint.go

@@ -5,6 +5,8 @@
 	"errors"
 	"flag"
 	"fmt"
+	"go/scanner"
+	"io"
 	"os"
 	"path/filepath"
 	"regexp"
@@ -68,10 +70,6 @@
 	}
 
 	hasError := false
-	addIssue := func(issue lintIssue) {
-		hasError = true
-		fmt.Fprint(io.Err(), issue.String()+"\n")
-	}
 
 	for _, pkgPath := range pkgPaths {
 		if verbose {
@@ -81,16 +79,18 @@
 		// Check if 'gno.mod' exists
 		gnoModPath := filepath.Join(pkgPath, "gno.mod")
 		if !osm.FileExists(gnoModPath) {
-			addIssue(lintIssue{
+			hasError = true
+			issue := lintIssue{
 				Code:       lintNoGnoMod,
 				Confidence: 1,
 				Location:   pkgPath,
 				Msg:        "missing 'gno.mod' file",
-			})
+			}
+			fmt.Fprint(io.Err(), issue.String()+"\n")
 		}
 
 		// Handle runtime errors
-		catchRuntimeError(pkgPath, addIssue, func() {
+		hasError = catchRuntimeError(pkgPath, io.Err(), func() {
 			stdout, stdin, stderr := io.Out(), io.In(), io.Err()
 			testStore := tests.TestStore(
 				rootDir, "",
@@ -130,7 +130,7 @@
 			}
 
 			tm.RunFiles(testfiles.Files...)
-		})
+		}) || hasError
 
 		// TODO: Add more checkers
 	}
@@ -164,47 +164,33 @@
 // XXX: Ideally, error handling should encapsulate location details within a dedicated error type.
 var reParseRecover = regexp.MustCompile(`^([^:]+):(\d+)(?::\d+)?:? *(.*)$`)
 
-func catchRuntimeError(pkgPath string, addIssue func(issue lintIssue), action func()) {
+func catchRuntimeError(pkgPath string, stderr io.WriteCloser, action func()) (hasError bool) {
 	defer func() {
 		// Errors catched here mostly come from: gnovm/pkg/gnolang/preprocess.go
 		r := recover()
 		if r == nil {
 			return
 		}
-
-		var err error
+		hasError = true
 		switch verr := r.(type) {
 		case *gno.PreprocessError:
-			err = verr.Unwrap()
+			err := verr.Unwrap()
+			fmt.Fprint(stderr, issueFromError(pkgPath, err).String()+"\n")
+		case scanner.ErrorList:
+			for _, err := range verr {
+				fmt.Fprint(stderr, issueFromError(pkgPath, err).String()+"\n")
+			}
 		case error:
-			err = verr
+			fmt.Fprint(stderr, issueFromError(pkgPath, verr).String()+"\n")
 		case string:
-			err = errors.New(verr)
+			fmt.Fprint(stderr, issueFromError(pkgPath, errors.New(verr)).String()+"\n")
 		default:
 			panic(r)
 		}
-
-		var issue lintIssue
-		issue.Confidence = 1
-		issue.Code = lintGnoError
-
-		parsedError := strings.TrimSpace(err.Error())
-		parsedError = strings.TrimPrefix(parsedError, pkgPath+"/")
-
-		matches := reParseRecover.FindStringSubmatch(parsedError)
-		if len(matches) == 4 {
-			sourcepath := guessSourcePath(pkgPath, matches[1])
-			issue.Location = fmt.Sprintf("%s:%s", sourcepath, matches[2])
-			issue.Msg = strings.TrimSpace(matches[3])
-		} else {
-			issue.Location = fmt.Sprintf("%s:0", filepath.Clean(pkgPath))
-			issue.Msg = err.Error()
-		}
-
-		addIssue(issue)
 	}()
 
 	action()
+	return
 }
 
 type lintCode int
@@ -229,3 +215,23 @@
 	// TODO: consider crafting a doc URL based on Code.
 	return fmt.Sprintf("%s: %s (code=%d).", i.Location, i.Msg, i.Code)
 }
+
+func issueFromError(pkgPath string, err error) lintIssue {
+	var issue lintIssue
+	issue.Confidence = 1
+	issue.Code = lintGnoError
+
+	parsedError := strings.TrimSpace(err.Error())
+	parsedError = strings.TrimPrefix(parsedError, pkgPath+"/")
+
+	matches := reParseRecover.FindStringSubmatch(parsedError)
+	if len(matches) == 4 {
+		sourcepath := guessSourcePath(pkgPath, matches[1])
+		issue.Location = fmt.Sprintf("%s:%s", sourcepath, matches[2])
+		issue.Msg = strings.TrimSpace(matches[3])
+	} else {
+		issue.Location = fmt.Sprintf("%s:0", filepath.Clean(pkgPath))
+		issue.Msg = err.Error()
+	}
+	return issue
+}

gnovm/cmd/gno/lint_test.go

@@ -16,6 +16,9 @@ func TestLintApp(t *testing.T) {
 		}, {
 			args:                []string{"lint", "--set-exit-status=0", "../../tests/integ/package_not_declared/main.gno"},
 			stderrShouldContain: "main.gno:4: name fmt not declared (code=2).",
+		}, {
+			args:                []string{"lint", "--set-exit-status=0", "../../tests/integ/several-lint-errors/main.gno"},
+			stderrShouldContain: "../../tests/integ/several-lint-errors/main.gno:5: expected ';', found example (code=2).\n../../tests/integ/several-lint-errors/main.gno:6",
 		}, {
 			args:                []string{"lint", "--set-exit-status=0", "../../tests/integ/run_main/"},
 			stderrShouldContain: "./../../tests/integ/run_main: missing 'gno.mod' file (code=1).",

gnovm/cmd/gno/run.go

@@ -5,6 +5,7 @@
 	"errors"
 	"flag"
 	"fmt"
+	"io"
 	"os"
 	"path/filepath"
 	"strings"
@@ -102,7 +103,7 @@
 	}
 
 	// read files
-	files, err := parseFiles(args)
+	files, err := parseFiles(args, stderr)
 	if err != nil {
 		return err
 	}
@@ -135,15 +136,16 @@
 	return nil
 }
 
-func parseFiles(fnames []string) ([]*gno.FileNode, error) {
+func parseFiles(fnames []string, stderr io.WriteCloser) ([]*gno.FileNode, error) {
 	files := make([]*gno.FileNode, 0, len(fnames))
+	var hasError bool
 	for _, fname := range fnames {
 		if s, err := os.Stat(fname); err == nil && s.IsDir() {
 			subFns, err := listNonTestFiles(fname)
 			if err != nil {
 				return nil, err
 			}
-			subFiles, err := parseFiles(subFns)
+			subFiles, err := parseFiles(subFns, stderr)
 			if err != nil {
 				return nil, err
 			}
@@ -154,7 +156,14 @@
 			// in either case not a file we can parse.
 			return nil, err
 		}
-		files = append(files, gno.MustReadFile(fname))
+
+		hasError = catchRuntimeError(fname, stderr, func() {
+			files = append(files, gno.MustReadFile(fname))
+		})
+	}
+
+	if hasError {
+		os.Exit(1)
 	}
 	return files, nil
 }

gnovm/cmd/gno/test.go

@@ -325,7 +325,15 @@
 		memPkg := gno.ReadMemPackage(pkgPath, gnoPkgPath)
 
 		// tfiles, ifiles := gno.ParseMemPackageTests(memPkg)
-		tfiles, ifiles := parseMemPackageTests(memPkg)
+		var tfiles, ifiles *gno.FileSet
+
+		hasError := catchRuntimeError(gnoPkgPath, stderr, func() {
+			tfiles, ifiles = parseMemPackageTests(memPkg)
+		})
+
+		if hasError {
+			os.Exit(1)
+		}
 		testPkgName := getPkgNameFromFileset(ifiles)
 
 		// run test files in pkg
@@ -639,7 +647,7 @@
 		}
 		n, err := gno.ParseFile(mfile.Name, mfile.Body)
 		if err != nil {
-			panic(errors.Wrap(err, "parsing file "+mfile.Name))
+			panic(err)
 		}
 		if n == nil {
 			panic("should not happen")

gnovm/pkg/gnolang/nodes.go

@@ -1179,7 +1179,7 @@
 		}
 		n, err := ParseFile(mfile.Name, mfile.Body)
 		if err != nil {
-			panic(errors.Wrap(err, "parsing file "+mfile.Name))
+			panic(err)
 		}
 		if memPkg.Name != string(n.PkgName) {
 			panic(fmt.Sprintf(

gnovm/tests/integ/several-lint-errors/gno.mod

@@ -0,0 +1 @@
+module gno.land/tests/severalerrors

gnovm/tests/integ/several-lint-errors/main.gno

@@ -0,0 +1,6 @@
+package main
+
+func main() {
+	for {
+		_ example
+}