vigiles 0.1.0.pre.beta6 → 0.1.0.pre.beta8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6ad47542ac3fa029cef8be2b99b104eb7995abf67765c8e0582426249f2d5a5a
-  data.tar.gz: a468738d5ef0ad815be016420198ef939b254997509b25228f5b3779fa51976f
+  metadata.gz: 9ae2f2c7a8b3a0642d0fb228264821c882b9b98cf7a59380dcf139e1a514221c
+  data.tar.gz: 7ca651c0579f55907094ade6d43825cae72627fef6c8e995fd621d7f94e9acc4
 SHA512:
-  metadata.gz: df8d3f3c5361e26b23b6a8cce014c4c55f3b15cc09f754a5ed3c34f15250360e09d6a0964ed23df5b1eb81cb87bec575198f71237fd24e1ce5f3ff201b279dc6
-  data.tar.gz: fe0759a9e54c564ee235f421dbd8c7d8cfe34378777c1c3737fb29fa152a7f277c4a94808e138ec4fb61c0ef7a619bed8b60a9d14d3059299772a3ef4c2dbcda
+  metadata.gz: a942ca1c61aefdb02f843b7d3defd046ebce9935b47d406e8193c2c54b7befd360b19f974618f24f42c23278e8c313be513d25b68fece32985d2272f504b8c2e
+  data.tar.gz: 2a70d2be56a5c86fa4f30f881b6b9002e6967cfe6cd04339f1bb4f70f1a6f8bd28b6ae34281a19b5504ad82a594fa8b0660591ff0ab28e9ad3fdd94214c26d78

data/lib/vigiles/archive/request.rb

@@ -37,7 +37,7 @@ module Vigiles
 
       sig { params(request: ActionDispatch::Request).returns(Request) }
       def self.from(request)
-        preferred_headers = Vigiles.specification.request_headers
+        preferred_headers = Vigiles.spec.request_headers
         available_headers = request.original_headers
         recorded_headers  = (available_headers if preferred_headers.empty?)
         recorded_headers ||= preferred_headers.to_h { |h| [h, available_headers[h]] }

data/lib/vigiles/archive/response.rb

@@ -16,14 +16,17 @@ module Vigiles
         when Array
           return { body: :empty_no_content } if body.empty?
 
-          { body: :not_empty_handle_later }
+          { __false_body: :not_empty_handle_later }
         when Rack::BodyProxy
           body_proxy = body
           body_proxy = body_proxy.instance_variable_get(:@body) until body_proxy.is_a?(Array)
-          JSON.parse(body_proxy[0])
+          begin
+            JSON.parse(body_proxy[0])
+          rescue StandardError
+            { __false_body: body_proxy[0] }
+          end
         else
-          debugger
-          { body: :unknown_response_payload_type }
+          { __false_body: :unknown_response_payload_type }
         end
       end
 

data/lib/vigiles/archive.rb

@@ -18,23 +18,14 @@ module Vigiles
     
     sig { params(req: ActionDispatch::Request, res: Rack::Response).returns(T.nilable(Conversation)) }
     def self.record_conversation(req:, res:)
-        # preferring to call `response.request` instead of preparing a new
-        # request (via `ActionDispatch::Request.new(env)` and passing it
-        # as an argument to this method because we can always recover the
-        # specific request that elicited a given response, according to
-        # https://github.com/rails/rails/blob/cacb8475a9d4373c0db437e7be4905685f03cefa/actionpack/lib/action_dispatch/http/response.rb#L53
-      response   = Response.from(res)
-      metadata   = Metadata.from(req.env)
-      request    = Request.from(req)
-      extras     = Extras.from(req.env)
-
-      case (content_type = request.content_type)
-      when ContentType::ApplicationJson.serialize then record_json_conversation(request:, response:, metadata:, extras:)
-      when ContentType::TextHtml.serialize        then record_html_conversation(request:, response:, metadata:, extras:)
-      else record_conversation_with_unknown_content_type(request:, response:, metadata:, extras:)
+      content_type = req.content_type
+      if (recorder = Vigiles.spec.recorders[content_type]).nil?
+        raise \
+          UnrecordableRequestError,
+          "no recorder configured for content type: #{content_type}"
       end
-    rescue Request::InvalidParameterError => e
-      raise UnrecordableRequestError, "#{e.parameter} considered invalid"
+
+      recorder.record(req:, res:)
     end
 
     sig do

data/lib/vigiles/conversation_recorder.rb

@@ -3,9 +3,24 @@
 
 module Vigiles
   class ConversationRecorder
+    class MisconfiguredRecorderError < StandardError
+      sig { returns(String) }
+      attr_reader :expected
+
+      sig { returns(String) }
+      attr_reader :actual
+
+      sig { params(expected: String, actual: String).void }
+      def initialize(expected:, actual:)
+        @expected = expected
+        @actual = actual
+        super
+      end
+    end
+
     abstract!
 
-    sig { abstract.params(response: ActionDispatch::Response).returns(Archive::Conversation) }
-    def record(response); end
+    sig { abstract.params(req: ActionDispatch::Request, res: Rack::Response).returns(Archive::Conversation) }
+    def record(req:, res:); end
   end
 end

data/lib/vigiles/conversation_recorders/application_json.rb

@@ -6,8 +6,45 @@ module Vigiles
     class ApplicationJson < ConversationRecorder
       include Singleton
 
-      sig { override.params(_response: ActionDispatch::Response).returns(Archive::Conversation) }
-      def record(_response) = Archive::Conversation.new
+      ConversationRecorder = Vigiles::ConversationRecorder
+      ContentType          = Vigiles::Types::ContentType
+      Conversation         = Vigiles::Archive::Conversation
+      Response             = Vigiles::Archive::Response
+      Metadata             = Vigiles::Archive::Metadata
+      Request              = Vigiles::Archive::Request
+      Extras               = Vigiles::Archive::Extras
+
+      sig { override.params(req: ActionDispatch::Request, res: Rack::Response).returns(Archive::Conversation) }
+      def record(req:, res:)
+        unless req.content_type == ContentType::ApplicationJson.serialize
+          raise ConversationRecorder::MisconfiguredRecorderError.new(
+            expected: ContentType::ApplicationJson.serialize,
+            actual: req.content_type
+          )
+        end
+
+        response = Response.from(res)
+        request  = Request.from(req)
+
+        Conversation.create!(
+          request_content_type: request.content_type,
+          request_user_agent: request.user_agent,
+          request_timestamp: request.timestamp,
+          request_remote_ip: request.remote_ip,
+          request_protocol: request.protocol,
+          request_headers: request.headers,
+          request_origin: request.origin,
+          request_payload: request.payload,
+          request_method: request.http_method.serialize,
+          request_path: request.path,
+          request_url: request.url,
+          request_id: request.id,
+          response_content_type: response.content_type,
+          response_headers: response.headers,
+          response_payload: response.payload,
+          response_status: response.status
+        )
+      end
     end
   end
 end

data/lib/vigiles/conversation_recorders/unknown.rb

@@ -4,8 +4,10 @@
 module Vigiles
   module ConversationRecorders
     class Unknown < ConversationRecorder
-      sig { override.params(_response: ActionDispatch::Response).returns(Archive::Conversation) }
-      def record(_response) = Archive::Conversation.new
+      include Singleton
+
+      sig { override.params(req: ActionDispatch::Request, res: Rack::Response).returns(Archive::Conversation) }
+      def record(req:, res:) = Archive::Conversation.new
     end
   end
 end

data/lib/vigiles/spec.rb

@@ -3,16 +3,16 @@
 
 module Vigiles
   class Spec < T::Struct
-    const :content_type_recorders, T::Hash[String, ConversationRecorder]
     const :request_content_types,  T::Set[String]
     const :request_headers,        T::Set[String]
+    const :recorders,              T::Hash[String, ConversationRecorder]
 
     sig { returns(Spec) }
     def self.make_default_spec
       Spec.new(
-        content_type_recorders: Constants::DEFAULT_CONTENT_TYPE_RECORDERS,
         request_content_types: Constants::DEFAULT_CONTENT_TYPES,
-        request_headers: Set[]
+        request_headers: Set[].freeze,
+        recorders: Constants::DEFAULT_CONTENT_TYPE_RECORDERS
       )
     end
   end

data/lib/vigiles/version.rb

@@ -2,5 +2,5 @@
 # frozen_string_literal: true
 
 module Vigiles
-  VERSION = "0.1.0-beta6"
+  VERSION = "0.1.0-beta8"
 end

data/lib/vigiles.rb

@@ -17,16 +17,16 @@ module Vigiles
   extend T::Sig
 
   sig { returns(Vigiles::Spec) }
-  def self.specification
-    @specification ||= T.let(
+  def self.spec
+    @spec ||= T.let(
       Vigiles::Spec.make_default_spec,
       T.nilable(Vigiles::Spec)
     )
   end
 
   sig { params(spec: Vigiles::Spec).returns(Vigiles::Spec) }
-  def self.specification=(spec)
-    @specification = spec
+  def self.spec=(spec)
+    @spec = spec
   end
 
   sig { params(req: ActionDispatch::Request, res: Rack::Response).returns(T.nilable(Archive::Conversation)) }
@@ -40,9 +40,9 @@ module Vigiles
 
   sig { params(blk: T.untyped).void }
   def self.configure(&blk)
-    blk.call(specification)
+    blk.call(spec)
 
-    # TODO(yaw, 2024-06-15): ensure that the specification is valid.
+    # TODO(yaw, 2024-06-15): ensure that the spec is valid.
     # ensure that for every content type a recorder is configured. otherwise
     # assign the general recorder for unknown content types.
   end

data/sorbet/tapioca/require.rb

@@ -3,10 +3,10 @@
 
 require "action_dispatch"
 require "active_record"
+require "logger"
 require "minitest/autorun"
 require "rails/generators"
 require "rails/generators/active_record"
 require "securerandom"
 require "sorbet-runtime"
-require "uri"
 require "zeitwerk"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: vigiles
 version: !ruby/object:Gem::Version
-  version: 0.1.0.pre.beta6
+  version: 0.1.0.pre.beta8
 platform: ruby
 authors:
 - Yaw Boakye