smplkit 3.0.23 → 3.0.24

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3e85fd7fac73ac4413a382782c353db02d602232de188ed1c8a9b7a1c53fcc78
-  data.tar.gz: '04096b74f6d24e798c3eff83a53bad846786f4a077503db5c4c826676a0bfc69'
+  metadata.gz: e01aa3cb437ae82f20a01f1e0c8db011df81743411217757e0f7bebebd0210c1
+  data.tar.gz: 885c582b378994dae192d1453020e211e97a3552c5938a695c1fae9e3e316d56
 SHA512:
-  metadata.gz: 50cb93abf4662d156a38e6609bee9932a8079514d30853485e13b517b7f524b4e4fa9cade5ab36fefa90becfdba5964ae46d6c38afb92df19f9f5fd11c6611b1
-  data.tar.gz: cafaee08710826d6238eaec8e552f25dc0a987ab062c84629a84d340ba73001045b8097b35963f19c31adc55d7895b7d2ca881d9cc29570065f694e8e6962ff3
+  metadata.gz: 4eb6729884abcc3a25a82355c426a316b10beef8ecd0fe7f2cbecf96d1a94644730086348bf35c826600cd8a36c3c604aeb495ceada93c0e3471e62a7c8d9cd5
+  data.tar.gz: 8235e725b934305e260595371f2bc330bd409a33a0bb965f2002349f564467891f1afe3777609b5caa1a7b2d423c459d3bcec0abdacb250d1446e205da0675de

data/lib/smplkit/audit/models.rb

@@ -386,10 +386,14 @@ module Smplkit
       # response (including newly-assigned +id+, +created_at+, +updated_at+,
       # +version+).
       #
+      # @raise [ArgumentError] when {#transform} and {#transform_type} are not
+      #   both nil or both set, or when {#transform_type} is +JSONATA+ and
+      #   {#transform} is not a +String+.
       # @return [self]
       def save
         raise "Forwarder was constructed without a client; cannot save" if @client.nil?
 
+        self.class.send(:validate_transform_pair!, @transform, @transform_type)
         updated = @created_at.nil? ? @client._create_forwarder(self) : @client._update_forwarder(self)
         _apply(updated)
         self
@@ -423,6 +427,26 @@ module Smplkit
         @version = other.version
       end
 
+      # Validate the +(transform, transform_type)+ pair.
+      #
+      # Both must be nil or both must be set. When +transform_type+ is
+      # +TransformType::JSONATA+, +transform+ must be a +String+ (the
+      # JSONata expression). Other engines accept any value.
+      #
+      # @api private
+      def self.validate_transform_pair!(transform, transform_type)
+        if transform.nil? != transform_type.nil?
+          raise ArgumentError,
+                "transform and transform_type must be specified together (both nil or both set)"
+        end
+        return if transform.nil?
+        return unless transform_type == TransformType::JSONATA && !transform.is_a?(String)
+
+        raise ArgumentError,
+              "transform must be a String when transform_type is JSONATA " \
+              "(got #{transform.class})"
+      end
+
       def self.from_resource(resource, client: nil)
         a = resource.attributes
         new(

data/lib/smplkit/management/audit.rb

@@ -44,23 +44,22 @@ module Smplkit
       # @param filter [Hash, nil] Optional JSON Logic filter; events that don't
       #   match are recorded as +filtered_out+ deliveries.
       # @param transform [Object, nil] Optional template applied to each event
-      #   before delivery. Free-form — the audit service passes the value
-      #   verbatim to the engine named by +transform_type+. Must be paired with
-      #   a non-nil +transform_type+.
+      #   before delivery. Free-form by default — the audit service passes the
+      #   value verbatim to the engine named by +transform_type+. Must be paired
+      #   with a non-nil +transform_type+; when +transform_type+ is
+      #   +TransformType::JSONATA+, +transform+ must be a +String+ (the JSONata
+      #   expression).
       # @param transform_type [String, nil] Engine that evaluates +transform+ —
-      #   one of {Smplkit::Audit::TransformType::VALUES}. Required when
-      #   +transform+ is provided.
-      # @raise [ArgumentError] when +transform+ is set but +transform_type+ is nil.
+      #   one of {Smplkit::Audit::TransformType::VALUES}. Must be paired with a
+      #   non-nil +transform+.
+      # @raise [ArgumentError] when +transform+ and +transform_type+ are not both
+      #   nil or both set, or when +transform_type+ is +JSONATA+ and +transform+
+      #   is not a +String+.
       # @return [Smplkit::Audit::Forwarder]
       def new_forwarder(name:, forwarder_type:, configuration:,
                         enabled: true, description: nil,
                         filter: nil, transform: nil, transform_type: nil)
-        if !transform.nil? && transform_type.nil?
-          raise ArgumentError,
-                "transform_type is required when transform is provided " \
-                "(one of #{Smplkit::Audit::TransformType::VALUES.inspect})"
-        end
-
+        Smplkit::Audit::Forwarder.send(:validate_transform_pair!, transform, transform_type)
         Smplkit::Audit::Forwarder.new(
           self,
           name: name,

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: smplkit
 version: !ruby/object:Gem::Version
-  version: 3.0.23
+  version: 3.0.24
 platform: ruby
 authors:
 - Smpl Solutions LLC