Merge pull request #4 from SteveDala/development

Development

Author: SteveDala

app/api/sidekiq_api.rb

@@ -17,7 +17,7 @@ class SidekiqApi < Grape::API
     job_data = Sidekiq::Status.get_all(job_id)
     initiator = Sidekiq::Status.get(job_id, :initiator)
 
-    if initiator.nil?
+    if job_data.blank? || initiator.nil?
       error!({ error: 'Job not found or has no owner' }, 404)
     end
 

app/sidekiq/predict_effort_job.rb

@@ -15,12 +15,16 @@ def perform(task_def_id, user_id)
       raise StandardError, "ML_SERVICE_URL is not configured"
     end
     Rails.logger.info("ML payload: #{payload.to_json}")
+
     response = Net::HTTP.post(
       URI("#{ml_url}predict"),
       payload.to_json,
       "Content-Type" => "application/json"
     )
 
+    unless response.is_a?(Net::HTTPSuccess)
+      raise StandardError, "ML service returned #{response.code}: #{response.body}"
+    end
     result = JSON.parse(response.body)
     Rails.logger.info("FastAPI response: #{response.body}")
     td.update(predicted_effort: result["predicted_effort"])