fine tune comments and messages

Author: wxiaoguang

modules/options/base.go

@@ -86,6 +86,7 @@ func walkAssetDir(root string, callback func(path, name string, d fs.DirEntry, e
 func mustLocalPathAbs(s string) string {
 	abs, err := filepath.Abs(s)
 	if err != nil {
+		// This should never happen in a real system. If it happens, the user must have already been in trouble: the system is not able to resolve its own paths.
 		log.Fatal("Unable to get absolute path for %q: %v", s, err)
 	}
 	return abs
@@ -106,7 +107,7 @@ func listLocalDirIfExist(baseDirs []string, subDir string, elems ...string) (fil
 	for _, localPath := range joinLocalPaths(baseDirs, subDir, elems...) {
 		isDir, err := util.IsDir(localPath)
 		if err != nil {
-			return nil, fmt.Errorf("unable to check if static directory %s is a directory. %w", localPath, err)
+			return nil, fmt.Errorf("unable to check if path %q is a directory. %w", localPath, err)
 		} else if !isDir {
 			continue
 		}

modules/util/path.go

@@ -5,6 +5,7 @@ package util
 
 import (
 	"errors"
+	"fmt"
 	"net/url"
 	"os"
 	"path"
@@ -82,7 +83,7 @@ func SafeFilePathAbs(elem ...string) string {
 	}
 	if !filepath.IsAbs(elems[0]) {
 		// This shouldn't happen. If there is really necessary to pass in relative path, return the full path with filepath.Abs() instead
-		panic("SafeFilePathAbs: result is not absolute, do not guess a relative path based on current working directory")
+		panic(fmt.Sprintf("SafeFilePathAbs: %q (for path %v) is not absolute, do not guess a relative path based on current working directory", elems[0], elems))
 	}
 
 	for i := 1; i < len(elem); i++ {