fix: Standardize parameter naming and enhance documentation across multiple files

Author: hashkode

internal/app/app.go

@@ -102,12 +102,12 @@ var UpdaterModule = fx.Module("updater",
 	),
 	fx.Invoke(func(lc fx.Lifecycle, us updater.Service, logger log.Logger) {
 		lc.Append(fx.Hook{
-			OnStart: func(ctx context.Context) error {
+			OnStart: func(_ctx context.Context) error {
 				logger.Info().Msg("Registering Updater Service start hook")
-				us.Start(ctx)
+				us.Start(_ctx)
 				return nil
 			},
-			OnStop: func(ctx context.Context) error {
+			OnStop: func(_ctx context.Context) error {
 				logger.Info().Msg("Registering Updater Service stop hook")
 				us.Stop()
 				return nil

internal/backend/database/database.go

@@ -53,6 +53,8 @@ CREATE TABLE IF NOT EXISTS tools (
 CREATE INDEX IF NOT EXISTS idx_tools_server_id ON tools(source_server_id);
 `
 
+// DB is the database implementation of the DBInterface.
+// It handles SQL operations for the agent-browser data persistence.
 type DB struct {
 	*sql.DB
 	// logger log.Logger // Optional: Inject logger if DB methods need logging

internal/backend/models/models.go

@@ -1,3 +1,5 @@
+// Package models defines the data structures used by the agent-browser application.
+// It includes entity definitions for MCP servers, tools, and related data types.
 package models
 
 import "time"

internal/config/exporter_test.go

@@ -34,27 +34,27 @@ func (m *mockBackend) ListMCPServers() ([]models.MCPServer, error) {
 }
 
 //nolint:unused
-func (m *mockBackend) AddMCPServer(name, url string) (*models.MCPServer, error) {
+func (m *mockBackend) AddMCPServer(_name, _url string) (*models.MCPServer, error) {
 	return nil, errors.New("not implemented in mock")
 }
 
 //nolint:unused
-func (m *mockBackend) RemoveMCPServer(id int64) error {
+func (m *mockBackend) RemoveMCPServer(_id int64) error {
 	return errors.New("not implemented in mock")
 }
 
 //nolint:unused
-func (m *mockBackend) GetMCPServer(id int64) (*models.MCPServer, error) {
+func (m *mockBackend) GetMCPServer(_id int64) (*models.MCPServer, error) {
 	return nil, errors.New("not implemented in mock")
 }
 
 //nolint:unused
-func (m *mockBackend) ProcessFetchedTools(serverID int64, fetchedTools []models.FetchedTool) (added, updated int, err error) {
+func (m *mockBackend) ProcessFetchedTools(_serverID int64, _fetchedTools []models.FetchedTool) (added, updated int, err error) {
 	return 0, 0, errors.New("not implemented in mock")
 }
 
 //nolint:unused
-func (m *mockBackend) UpdateMCPServerStatus(id int64, checkErr error) {
+func (m *mockBackend) UpdateMCPServerStatus(_id int64, _checkErr error) {
 	// No-op in mock
 }
 

internal/events/events.go

@@ -1,3 +1,5 @@
+// Package events provides an event bus implementation for internal application events.
+// It defines event types and structures for communication between components.
 package events
 
 import (

internal/updater/service_test.go

@@ -108,24 +108,24 @@ func (m *mockBackendService) UpdateMCPServerStatus(id int64, checkErr error) {
 // Add unused methods to satisfy interface
 //
 //nolint:unused
-func (m *mockBackendService) AddMCPServer(name, url string) (*models.MCPServer, error) {
+func (m *mockBackendService) AddMCPServer(_name, _url string) (*models.MCPServer, error) {
 	panic("unused")
 }
 
 //nolint:unused
-func (m *mockBackendService) RemoveMCPServer(id int64) error { panic("unused") }
+func (m *mockBackendService) RemoveMCPServer(_id int64) error { panic("unused") }
 
 //nolint:unused
-func (m *mockBackendService) GetMCPServer(id int64) (*models.MCPServer, error) { panic("unused") }
+func (m *mockBackendService) GetMCPServer(_id int64) (*models.MCPServer, error) { panic("unused") }
 
 // Helper to get recorded status update
 //
 //nolint:unused
-func (m *mockBackendService) getLastStatusUpdate(id int64) (error, bool) {
+func (m *mockBackendService) getLastStatusUpdate(id int64) (bool, error) {
 	m.mu.Lock()
 	defer m.mu.Unlock()
 	err, ok := m.statusUpdates[id]
-	return err, ok
+	return ok, err
 }
 
 // Helper to get recorded processed tools

internal/web/handlers/api.go

@@ -301,7 +301,6 @@ func (h *APIHandlers) ListTools(w http.ResponseWriter, r *http.Request) {
 	// Get all tools - not implemented yet
 	h.logger.Info().Msg("Listing all tools is not yet implemented")
 	http.Error(w, "Listing all tools is not implemented yet", http.StatusNotImplemented)
-	return
 }
 
 // RediscoverTools handles POST /api/mcp/rediscover-tools

internal/web/handlers/api_test.go

@@ -117,12 +117,12 @@ func (m *mockBackendServiceForAPI) GetMCPServer(id int64) (*models.MCPServer, er
 // --- Unused Service Methods (Panic if called) ---
 
 //nolint:unused
-func (m *mockBackendServiceForAPI) ProcessFetchedTools(serverID int64, fetchedTools []models.FetchedTool) (int, int, error) {
+func (m *mockBackendServiceForAPI) ProcessFetchedTools(_serverID int64, _fetchedTools []models.FetchedTool) (int, int, error) {
 	panic("ProcessFetchedTools not implemented/needed for API tests")
 }
 
 //nolint:unused
-func (m *mockBackendServiceForAPI) UpdateMCPServerStatus(id int64, checkErr error) {
+func (m *mockBackendServiceForAPI) UpdateMCPServerStatus(_id int64, _checkErr error) {
 	panic("UpdateMCPServerStatus not implemented/needed for API tests")
 }