refactor(log): rework logger initialization #27

- Remove verbose flag and terminal logging
- Logger is now file-based only
- Use env vars for debug level
- Remove config for air

Fixes #25

Author: maniac-en

.air.toml

@@ -19,7 +19,7 @@ func (c *CollectionsManager) Create(ctx context.Context, name string) (Collectio
 		return CollectionEntity{}, crud.ErrInvalidInput
 	}
 
-	log.DebugIf("creating collection", "name", name)
+	log.Debug("creating collection", "name", name)
 	collection, err := c.DB.CreateCollection(ctx, name)
 	if err != nil {
 		log.Error("failed to create collection", "name", name, "error", err)
@@ -36,7 +36,7 @@ func (c *CollectionsManager) Read(ctx context.Context, id int64) (CollectionEnti
 		return CollectionEntity{}, crud.ErrInvalidInput
 	}
 
-	log.DebugIf("reading collection", "id", id)
+	log.Debug("reading collection", "id", id)
 	collection, err := c.DB.GetCollection(ctx, id)
 	if err != nil {
 		if err == sql.ErrNoRows {
@@ -60,7 +60,7 @@ func (c *CollectionsManager) Update(ctx context.Context, id int64, name string)
 		return CollectionEntity{}, crud.ErrInvalidInput
 	}
 
-	log.DebugIf("updating collection", "id", id, "name", name)
+	log.Debug("updating collection", "id", id, "name", name)
 	collection, err := c.DB.UpdateCollectionName(ctx, database.UpdateCollectionNameParams{
 		Name: name,
 		ID:   id,
@@ -84,7 +84,7 @@ func (c *CollectionsManager) Delete(ctx context.Context, id int64) error {
 		return crud.ErrInvalidInput
 	}
 
-	log.DebugIf("deleting collection", "id", id)
+	log.Debug("deleting collection", "id", id)
 	err := c.DB.DeleteCollection(ctx, id)
 	if err != nil {
 		log.Error("failed to delete collection", "id", id, "error", err)
@@ -96,7 +96,7 @@ func (c *CollectionsManager) Delete(ctx context.Context, id int64) error {
 }
 
 func (c *CollectionsManager) List(ctx context.Context) ([]CollectionEntity, error) {
-	log.DebugIf("listing all collections with default pagination")
+	log.Debug("listing all collections with default pagination")
 	paginated, err := c.ListPaginated(ctx, 50, 0)
 	if err != nil {
 		return nil, err
@@ -113,7 +113,7 @@ func (c *CollectionsManager) ListPaginated(ctx context.Context, limit, offset in
 		log.Warn("negative pagination offset", "offset", offset)
 	}
 
-	log.DebugIf("listing paginated collections", "limit", limit, "offset", offset)
+	log.Debug("listing paginated collections", "limit", limit, "offset", offset)
 
 	total, err := c.DB.CountCollections(ctx)
 	if err != nil {

internal/collections/manager.go

@@ -25,7 +25,7 @@ func (e *EndpointsManager) Read(ctx context.Context, id int64) (EndpointEntity,
 		return EndpointEntity{}, crud.ErrInvalidInput
 	}
 
-	log.DebugIf("reading endpoint", "id", id)
+	log.Debug("reading endpoint", "id", id)
 	endpoint, err := e.DB.GetEndpoint(ctx, id)
 	if err != nil {
 		if err == sql.ErrNoRows {
@@ -49,7 +49,7 @@ func (e *EndpointsManager) Delete(ctx context.Context, id int64) error {
 		return crud.ErrInvalidInput
 	}
 
-	log.DebugIf("deleting endpoint", "id", id)
+	log.Debug("deleting endpoint", "id", id)
 	err := e.DB.DeleteEndpoint(ctx, id)
 	if err != nil {
 		log.Error("failed to delete endpoint", "id", id, "error", err)
@@ -70,7 +70,7 @@ func (e *EndpointsManager) ListByCollection(ctx context.Context, collectionID in
 		return nil, crud.ErrInvalidInput
 	}
 
-	log.DebugIf("listing paginated endpoints", "collection_id", collectionID, "limit", limit, "offset", offset)
+	log.Debug("listing paginated endpoints", "collection_id", collectionID, "limit", limit, "offset", offset)
 
 	total, err := e.DB.CountEndpointsByCollection(ctx, collectionID)
 	if err != nil {
@@ -132,7 +132,7 @@ func (e *EndpointsManager) CreateEndpoint(ctx context.Context, data EndpointData
 		queryParamsJSON = string(qpBytes)
 	}
 
-	log.DebugIf("creating endpoint", "collection_id", data.CollectionID, "name", data.Name, "method", data.Method, "url", data.URL)
+	log.Debug("creating endpoint", "collection_id", data.CollectionID, "name", data.Name, "method", data.Method, "url", data.URL)
 	endpoint, err := e.DB.CreateEndpoint(ctx, database.CreateEndpointParams{
 		CollectionID: data.CollectionID,
 		Name:         data.Name,
@@ -180,7 +180,7 @@ func (e *EndpointsManager) UpdateEndpoint(ctx context.Context, id int64, data En
 		queryParamsJSON = string(qpBytes)
 	}
 
-	log.DebugIf("updating endpoint", "id", id, "name", data.Name, "method", data.Method, "url", data.URL)
+	log.Debug("updating endpoint", "id", id, "name", data.Name, "method", data.Method, "url", data.URL)
 	endpoint, err := e.DB.UpdateEndpoint(ctx, database.UpdateEndpointParams{
 		Name:        data.Name,
 		Method:      data.Method,

internal/endpoints/manager.go

@@ -31,7 +31,7 @@ func (h *HistoryManager) Read(ctx context.Context, id int64) (HistoryEntity, err
 		return HistoryEntity{}, err
 	}
 
-	log.DebugIf("read history entry", "id", id)
+	log.Debug("read history entry", "id", id)
 	return HistoryEntity{History: history}, nil
 }
 
@@ -111,7 +111,7 @@ func (h *HistoryManager) RecordExecution(ctx context.Context, data ExecutionData
 		return HistoryEntity{}, err
 	}
 
-	log.DebugIf("recording execution", "method", data.Method, "url", data.URL, "status", data.StatusCode)
+	log.Debug("recording execution", "method", data.Method, "url", data.URL, "status", data.StatusCode)
 
 	// Marshal to JSON for database storage
 	requestHeaders, err := json.Marshal(data.Headers)

internal/history/manager.go

@@ -59,7 +59,7 @@ func (h *HTTPManager) ExecuteRequest(req *Request) (*Response, error) {
 		return nil, err
 	}
 
-	log.DebugIf("executing HTTP request", "method", req.Method, "url", req.URL)
+	log.Debug("executing HTTP request", "method", req.Method, "url", req.URL)
 
 	requestURL, err := h.buildURL(req.URL, req.QueryParams)
 	if err != nil {