From a9e0b33002506cf5bd8eb39230837c468808e44e Mon Sep 17 00:00:00 2001
From: John O'Keefe <jrokeefe@protonmail.com>
Date: Sun, 8 Feb 2026 14:31:52 -0500
Subject: [PATCH] refactor(router): update router comments and variable names

- Update comment: 'ebook handler' -> 'scanner handler'
- Rename ebookHandler variable to scannerHandler in router.go
- Update registerProgressRoutes parameter name
- Update registerScannerRoutes calls to use new variable name
---
 internal/router/progress.go |  8 ++++----
 internal/router/router.go   | 10 +++++-----
 2 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/internal/router/progress.go b/internal/router/progress.go
index 48372a0..c3d4f4d 100644
--- a/internal/router/progress.go
+++ b/internal/router/progress.go
@@ -4,7 +4,7 @@ import (
 	"bookhoard/internal/handlers"
 )
 
-func registerProgressRoutes(cfg *Config, ebookHandler *handlers.Handler) {
+func registerProgressRoutes(cfg *Config, scannerHandler *handlers.Handler) {
 	e := cfg.Echo
 
 	// JWT middleware for protected routes
@@ -12,7 +12,7 @@ func registerProgressRoutes(cfg *Config, ebookHandler *handlers.Handler) {
 	protected := e.Group("/api", jwtMiddleware)
 
 	// Universal Progress routes (Phase 1)
-	protected.GET("/progress/:id", ebookHandler.GetUniversalProgress)
-	protected.POST("/progress/:id", ebookHandler.UpdateUniversalProgress)
-	protected.GET("/progress/:id/history", ebookHandler.GetProgressHistory)
+	protected.GET("/progress/:id", scannerHandler.GetUniversalProgress)
+	protected.POST("/progress/:id", scannerHandler.UpdateUniversalProgress)
+	protected.GET("/progress/:id/history", scannerHandler.GetProgressHistory)
 }
diff --git a/internal/router/router.go b/internal/router/router.go
index 6c7ccfc..cdc4bac 100644
--- a/internal/router/router.go
+++ b/internal/router/router.go
@@ -132,15 +132,15 @@ func RegisterRoutes(cfg *Config) *handlers.Handler {
 	registerFrontendRoutes(cfg)
 	registerDocumentationRoutes(cfg)
 
-	// Create ebook handler for scanner routes and progress routes
-	ebookHandler := handlers.SetupRoutes(protected, cfg.Queries, cfg.ConnManager)
+	// Create scanner handler for scanner routes and progress routes
+	scannerHandler := handlers.SetupRoutes(protected, cfg.Queries, cfg.ConnManager)
 
 	// Register progress routes with actual handler
-	registerProgressRoutes(cfg, ebookHandler)
+	registerProgressRoutes(cfg, scannerHandler)
 
 	// Register scanner routes (admin only)
 	admin := protected.Group("", handlers.AdminMiddleware)
-	registerScannerRoutes(admin, ebookHandler)
+	registerScannerRoutes(admin, scannerHandler)
 
-	return ebookHandler
+	return scannerHandler
 }