From adc7acbe0f62e8f970da76730899d39bfee2aee1 Mon Sep 17 00:00:00 2001
From: John O'Keefe <jrokeefe@protonmail.com>
Date: Sun, 8 Feb 2026 13:33:04 -0500
Subject: [PATCH] refactor(api): rename ScanEbooks to ScanLibrary for clarity

- Rename ScanEbooks method to ScanLibrary to reflect generic media scanning

- Rename ScanEbooksRequest to ScanLibraryRequest

- Update route handlers in scanner.go and library.go

- Rename scan_ebooks.md to scan_library.md

- No breaking changes: API endpoint remains POST /api/scanner/scan
---
 .../api/scanner/{scan_ebooks.md => scan_library.md}  |  0
 internal/handlers/ebook.go                           | 12 ++++++------
 internal/router/library.go                           |  2 +-
 internal/router/scanner.go                           |  2 +-
 4 files changed, 8 insertions(+), 8 deletions(-)
 rename docs/developer/api/scanner/{scan_ebooks.md => scan_library.md} (100%)

diff --git a/docs/developer/api/scanner/scan_ebooks.md b/docs/developer/api/scanner/scan_library.md
similarity index 100%
rename from docs/developer/api/scanner/scan_ebooks.md
rename to docs/developer/api/scanner/scan_library.md
diff --git a/internal/handlers/ebook.go b/internal/handlers/ebook.go
index defb3e9..2bd4050 100644
--- a/internal/handlers/ebook.go
+++ b/internal/handlers/ebook.go
@@ -69,15 +69,15 @@ func SetupRoutes(g *echo.Group, db *database.Queries, connManager *wsync.Connect
 	return NewHandler(db, connManager)
 }
 
-// ScanEbooksRequest represents the request for scanning ebooks
-type ScanEbooksRequest struct {
+// ScanLibraryRequest represents the request for scanning a library
+type ScanLibraryRequest struct {
 	FolderPaths []string `json:"folder_paths,omitempty"`
 	LibraryID   string   `json:"library_id,omitempty"`
 }
 
-// ScanEbooks handles POST /api/scanner/scan (now runs in background)
-func (h *Handler) ScanEbooks(c echo.Context) error {
-	var req ScanEbooksRequest
+// ScanLibrary handles POST /api/scanner/scan (now runs in background)
+func (h *Handler) ScanLibrary(c echo.Context) error {
+	var req ScanLibraryRequest
 
 	// Check if scan_request is set in context (from library scan route)
 	if scanReq, ok := c.Get("scan_request").(map[string]interface{}); ok {
@@ -160,7 +160,7 @@ func (h *Handler) StartScanner(c echo.Context) error {
 	h.mu.Lock()
 	defer h.mu.Unlock()
 
-	var req ScanEbooksRequest
+	var req ScanLibraryRequest
 	if err := c.Bind(&req); err != nil {
 		return c.JSON(http.StatusBadRequest, map[string]string{"error": "invalid request"})
 	}
diff --git a/internal/router/library.go b/internal/router/library.go
index 0b2e7ca..b090ecd 100644
--- a/internal/router/library.go
+++ b/internal/router/library.go
@@ -41,7 +41,7 @@ func registerLibraryRoutes(cfg *Config) {
 			"library_id": libraryID,
 		}
 		c.Set("scan_request", scanReq)
-		return h.ScanEbooks(c)
+		return h.ScanLibrary(c)
 	})
 	adminLibrary.GET("/:id/media-items", func(c echo.Context) error {
 		libraryID := c.Param("id")
diff --git a/internal/router/scanner.go b/internal/router/scanner.go
index 5344498..147d597 100644
--- a/internal/router/scanner.go
+++ b/internal/router/scanner.go
@@ -8,7 +8,7 @@ import (
 // registerScannerRoutes registers all scanner-related endpoints
 func registerScannerRoutes(admin *echo.Group, h *handlers.Handler) {
 	// Scanner routes (admin only)
-	admin.POST("/scanner/scan", h.ScanEbooks)
+	admin.POST("/scanner/scan", h.ScanLibrary)
 	admin.POST("/scanner/start", h.StartScanner)
 	admin.POST("/scanner/stop", h.StopScanner)
 	admin.GET("/scanner/status/:jobId", h.GetScanStatus)