diff --git a/ingestion/internal/pipeline/pipeline_test.go b/ingestion/internal/pipeline/pipeline_test.go index 38976b8..ac39b3d 100644 --- a/ingestion/internal/pipeline/pipeline_test.go +++ b/ingestion/internal/pipeline/pipeline_test.go @@ -125,6 +125,9 @@ func TestRun_MergesDuplicatePaths(t *testing.T) { } func mustJSON(v any) string { - b, _ := json.Marshal(v) + b, err := json.Marshal(v) + if err != nil { + panic(err) + } return string(b) } diff --git a/ingestion/internal/wiki/inventory.go b/ingestion/internal/wiki/inventory.go index 42398d6..43a4ad1 100644 --- a/ingestion/internal/wiki/inventory.go +++ b/ingestion/internal/wiki/inventory.go @@ -3,6 +3,7 @@ package wiki import ( "bufio" + "fmt" "os" "path/filepath" "strings" @@ -23,7 +24,7 @@ func LoadInventory(brainDir string) (map[PageType][]Entry, error) { continue } if err != nil { - return nil, err + return nil, fmt.Errorf("read dir %s: %w", dir, err) } for _, e := range entries { if e.IsDir() || !strings.HasSuffix(e.Name(), ".md") { diff --git a/ingestion/internal/wiki/log.go b/ingestion/internal/wiki/log.go index 283d3b8..ade924c 100644 --- a/ingestion/internal/wiki/log.go +++ b/ingestion/internal/wiki/log.go @@ -33,6 +33,8 @@ func AppendLog(brainDir, source string, pages, warnings []string, date string) e return fmt.Errorf("open log: %w", err) } defer f.Close() - _, err = f.WriteString(sb.String()) - return err + if _, err = f.WriteString(sb.String()); err != nil { + return fmt.Errorf("write log: %w", err) + } + return nil }