fix(infojson): allow fetching revisions

Improved error message if keyboard isnt found and allow the use of revisions. (e.g. lily58/rev1).
Closes #11

cmd/zmk-viewer/generate/generate.go

@@ -26,7 +26,7 @@ type Cmd struct {
 }
 
 func (g *Cmd) Run() error {
-	return generate(strings.ReplaceAll(g.KeyboardName, "/", "_"), g.LayoutFile, g.Output, g.File, g.Transparent, g.Raw, g.Single, g.Unified)
+	return generate(g.KeyboardName, g.LayoutFile, g.Output, g.File, g.Transparent, g.Raw, g.Single, g.Unified)
 }
 
 func generate(keyboardName, layoutFile, output, keymapFile string, isTransparent, isRaw, single, unified bool) error {
@@ -89,15 +89,16 @@ func generate(keyboardName, layoutFile, output, keymapFile string, isTransparent
 	}
 
 	for path, image := range images {
-		f, err := os.Create(filepath.Join(output, path))
+		sanitized := strings.ReplaceAll(path, "/", "_")
+		f, err := os.Create(filepath.Join(output, sanitized))
 		if err != nil {
 			return err
 		}
 		defer f.Close()
 		if err = png.Encode(f, image); err != nil {
 			return nil
 		}
-		log.Info().Str("Path", path).Msg("Image saved")
+		log.Info().Str("Path", sanitized).Msg("Image saved")
 	}
 
 	return nil

pkg/infojson/infojson.go

@@ -71,6 +71,9 @@ func fetch(url string) (*file, error) {
 	log.Debug().Str("url", url).Send()
 
 	resp, err := http.Get(url) //nolint:gosec
+	if resp.StatusCode != http.StatusOK {
+		return nil, fmt.Errorf("could not fetch keyboard layout: %v", resp.Status)
+	}
 	if err != nil {
 		return nil, err
 	}