[lint] minor fixes

Author: capcom6

internal/sms-gateway/handlers/mobile.go

@@ -121,7 +121,7 @@ func (h *mobileHandler) postDevice(c *fiber.Ctx) error {
 		}
 	}
 
-	device, err := h.authSvc.RegisterDevice(*user, req.Name, req.PushToken)
+	device, err := h.authSvc.RegisterDevice(user, req.Name, req.PushToken)
 	if err != nil {
 		return fmt.Errorf("failed to register device: %w", err)
 	}

internal/sms-gateway/handlers/root.go

@@ -25,7 +25,7 @@ func (h *rootHandler) Register(app *fiber.App) {
 				c.Set(fiber.HeaderLocation, path.Join(h.config.PublicPath, after))
 			}
 
-			return err //nolint:wrapcheck //wrapped internally
+			return err //nolint:wrapcheck // passed through to fiber's error handler
 		})
 	}
 

internal/sms-gateway/modules/auth/service.go

@@ -112,7 +112,7 @@ func (s *Service) RegisterUser(login, password string) (*models.User, error) {
 	return user, nil
 }
 
-func (s *Service) RegisterDevice(user models.User, name, pushToken *string) (*models.Device, error) {
+func (s *Service) RegisterDevice(user *models.User, name, pushToken *string) (*models.Device, error) {
 	device := models.NewDevice(name, pushToken)
 
 	if err := s.devicesSvc.Insert(user.ID, device); err != nil {

internal/worker/executor/service.go

@@ -69,7 +69,7 @@ func (s *Service) Start() error {
 }
 
 func (s *Service) runTask(ctx context.Context, task PeriodicTask) {
-	//nolint:gosec // it's ok
+	//nolint:gosec // weak random is acceptable for scheduling jitter
 	initialDelay := time.Duration(math.Floor(rand.Float64()*task.Interval().Seconds())) * time.Second
 
 	s.logger.Info("initial delay", zap.String("name", task.Name()), zap.Duration("delay", initialDelay))