Remove user_id and points_count parameters from Metrics::Archives::Operation and related calls.

app/services/metrics/archives/operation.rb

@@ -3,11 +3,9 @@
 class Metrics::Archives::Operation
   OPERATIONS = %w[archive verify clear restore].freeze
 
-  def initialize(operation:, status:, user_id: nil, points_count: 0)
+  def initialize(operation:, status:)
     @operation = operation
     @status = status # 'success' or 'failure'
-    @user_id = user_id
-    @points_count = points_count
   end
 
   def call

app/services/points/raw_data/archiver.rb

@@ -26,13 +26,10 @@ module Points
       end
 
       def archive_specific_month(user_id, year, month)
-        month_data = {
+        # Direct call without error handling - allows errors to propagate
-          'user_id' => user_id,
+        # This is intended for use in tests and manual operations where
-          'year' => year,
+        # we want to know immediately if something went wrong
-          'month' => month
+        archive_month(user_id, year, month)
-        }
-
-        process_month(month_data)
       end
 
       private
@@ -83,8 +80,7 @@ module Points
           # Report successful archive operation
           Metrics::Archives::Operation.new(
             operation: 'archive',
-            status: 'success',
+            status: 'success'
-            user_id: user_id
           ).call
 
           true
@@ -99,8 +95,7 @@ module Points
         # Report failed archive operation
         Metrics::Archives::Operation.new(
           operation: 'archive',
-          status: 'failure',
+          status: 'failure'
-          user_id: user_id
         ).call
       end
 

app/services/points/raw_data/clearer.rb

@@ -78,9 +78,7 @@ module Points
         # Report successful clear operation
         Metrics::Archives::Operation.new(
           operation: 'clear',
-          status: 'success',
+          status: 'success'
-          user_id: archive.user_id,
-          points_count: cleared_count
         ).call
 
         # Report points removed (cleared from database)
@@ -95,8 +93,7 @@ module Points
         # Report failed clear operation
         Metrics::Archives::Operation.new(
           operation: 'clear',
-          status: 'failure',
+          status: 'failure'
-          user_id: archive.user_id
         ).call
       end
 

app/services/points/raw_data/restorer.rb

@@ -21,9 +21,7 @@ module Points
           # Report successful restore operation
           Metrics::Archives::Operation.new(
             operation: 'restore',
-            status: 'success',
+            status: 'success'
-            user_id: user_id,
-            points_count: total_points
           ).call
 
           # Report points restored (removed from archived state)
@@ -35,8 +33,7 @@ module Points
           # Report failed restore operation
           Metrics::Archives::Operation.new(
             operation: 'restore',
-            status: 'failure',
+            status: 'failure'
-            user_id: user_id
           ).call
 
           raise

app/services/points/raw_data/verifier.rb

@@ -52,8 +52,7 @@ module Points
           # Report successful verification operation
           Metrics::Archives::Operation.new(
             operation: 'verify',
-            status: 'success',
+            status: 'success'
-            user_id: archive.user_id
           ).call
 
           # Report verification duration
@@ -69,8 +68,7 @@ module Points
           # Report failed verification operation
           Metrics::Archives::Operation.new(
             operation: 'verify',
-            status: 'failure',
+            status: 'failure'
-            user_id: archive.user_id
           ).call
 
           # Report verification duration with check name
@@ -85,8 +83,7 @@ module Points
         # Report failed verification operation
         Metrics::Archives::Operation.new(
           operation: 'verify',
-          status: 'failure',
+          status: 'failure'
-          user_id: archive.user_id
         ).call
 
         # Report verification duration

spec/factories/points_raw_data_archives.rb

@@ -9,7 +9,14 @@ FactoryBot.define do
     point_count { 100 }
     point_ids_checksum { Digest::SHA256.hexdigest('1,2,3') }
     archived_at { Time.current }
-    metadata { { format_version: 1, compression: 'gzip' } }
+    metadata do
+      {
+        format_version: 1,
+        compression: 'gzip',
+        expected_count: point_count,
+        actual_count: point_count
+      }
+    end
 
     after(:build) do |archive|
       # Attach a test file

spec/services/metrics/archives/operation_spec.rb

@@ -5,9 +5,8 @@ require 'prometheus_exporter/client'
 
 RSpec.describe Metrics::Archives::Operation do
   describe '#call' do
-    subject(:operation) { described_class.new(operation: operation_type, status: status, user_id: user_id).call }
+    subject(:operation) { described_class.new(operation: operation_type, status: status).call }
 
-    let(:user_id) { 123 }
     let(:operation_type) { 'archive' }
     let(:status) { 'success' }
     let(:prometheus_client) { instance_double(PrometheusExporter::Client) }