graphiti 1.12.2 → 1.13.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: de4409a60f63682b5bb513238e1101ca1997943a18b136c3d9c507f1eeda9651
-  data.tar.gz: 62b1019afe5fbb095b1323f17c06f7c97f7c15aa0b9dfff3afded57b428ff326
+  metadata.gz: 5b0c2234b78e4616fea8ab95f8b6f044d18aa4feafb103cb4eb7d0fed35f8dd6
+  data.tar.gz: 2b276253a3b5b802dbf143ccd6fdc7d17287ee542c60f352ab939a33f16da233
 SHA512:
-  metadata.gz: b4da6af791178d1a3229dd0c9515792cecacd2bc60ee96274dfc8723af232531a6190e04697f5a1f6e2f9f2b6562a950cfe916657198969e33b0a5251b0854ab
-  data.tar.gz: 1f6561906d4b70bc148563be8c9d20b2dc028006a238993449d59a51fc125efd7c5110d9564a983ed09d749fcd53f46740667aab737e1e7e13d47977569b4246
+  metadata.gz: 01ac700f8481142c1842488db436df2bedc508acabf3bde7fcd4d9e0e1a9882ab0df8c23ac5473b68a8ab1717b6041a34fde6b18125d269ccad88f23d8a3f3aa
+  data.tar.gz: be85a9c17b7b1df85c5ff755b4aebfeb6bd814e4a88a87b926f05056d776f5b77ec643ab1c8800e155cf541149c30d9c343471ef9427b0dcc69ae6c2e79b3295

data/CHANGELOG.md

@@ -1,5 +1,13 @@
 graphiti changelog
 
+# [1.13.0](https://github.com/graphiti-api/graphiti/compare/v1.12.2...v1.13.0) (2026-07-30)
+
+
+### Features
+
+* add Resource.wrap as an easy way to use graphiti serialization on models fetched via other means (and not via graphiti's finders) ([#513](https://github.com/graphiti-api/graphiti/issues/513)) ([fcd19e2](https://github.com/graphiti-api/graphiti/commit/fcd19e2d023091d85947dbdd89259b587e0f582b))
+* deprecate mutating attributes in around_persistence hooks ([#514](https://github.com/graphiti-api/graphiti/issues/514)) [skip ci] ([8410ab0](https://github.com/graphiti-api/graphiti/commit/8410ab0a916077601cf85d8260a05ef9098149eb))
+
 ## [1.12.2](https://github.com/graphiti-api/graphiti/compare/v1.12.1...v1.12.2) (2026-07-29)
 
 

data/lib/graphiti/errors.rb

@@ -467,6 +467,19 @@ module Graphiti
       end
     end
 
+    class InvalidWrapModel < Base
+      def initialize(resource_class, model)
+        @resource_class = resource_class
+        @model = model
+      end
+
+      def message
+        <<~MSG
+          #{@resource_class.name}.wrap was given a #{@model.class}, but this resource is for #{@resource_class.model} (subclasses are fine).
+        MSG
+      end
+    end
+
     class ResourceEndpointConflict < Base
       def initialize(path, action, resource_a, resource_b)
         @path = path

data/lib/graphiti/resource/interface.rb

@@ -53,8 +53,28 @@ module Graphiti
           runner.proxy(base_scope, single: true, raise_on_missing: true)
         end
 
+        # Wrap models fetched outside graphiti so they render like any other proxy
+        def wrap(models, base_scope = nil)
+          validate_wrap_models!(models)
+          runner = Runner.new(self, {}, nil, :find)
+          runner.proxy(base_scope, bypass_required_filters: true).tap do |proxy|
+            proxy.data = models
+          end
+        end
+
         private
 
+        # Skip polymorphic parents - resource_for_model raises a better error for unknown children
+        def validate_wrap_models!(models)
+          return if abstract_class? || (polymorphic? && !polymorphic_child?)
+
+          [models].flatten.compact.each do |model|
+            unless model.is_a?(self.model)
+              raise Errors::InvalidWrapModel.new(self, model)
+            end
+          end
+        end
+
         def caching_options
           {cache: @cache_resource, cache_expires_in: @cache_expires_in, cache_tag: @cache_tag}
         end

data/lib/graphiti/resource/persistence.rb

@@ -71,8 +71,11 @@ module Graphiti
 
       def create(create_params, meta = nil)
         model_instance = nil
+        snapshot = attributes_snapshot(:create, create_params)
 
         run_callbacks :persistence, :create, create_params, meta do
+          warn_attributes_mutated_in_around_persistence(:create, snapshot, create_params)
+
           run_callbacks :attributes, :create, create_params, meta do |params|
             model_instance = call_with_meta(:build, model, meta)
             call_with_meta(:assign_attributes, model_instance, params, meta)
@@ -92,7 +95,11 @@ module Graphiti
         id = update_params[:id]
         update_params = update_params.except(:id)
 
+        snapshot = attributes_snapshot(:update, update_params)
+
         run_callbacks :persistence, :update, update_params, meta do
+          warn_attributes_mutated_in_around_persistence(:update, snapshot, update_params)
+
           run_callbacks :attributes, :update, update_params, meta do |params|
             model_instance = self.class._find(id: id).data
             call_with_meta(:assign_attributes, model_instance, params, meta)
@@ -133,6 +140,37 @@ module Graphiti
 
       private
 
+      # In Graphiti 2.0, attributes are assigned to the model before the
+      # persistence hooks fire, so hash modifications made by an
+      # around_persistence hook before its yield will no longer be applied.
+      # Snapshot-and-compare detects exactly those hooks: the comparison
+      # happens before any attributes-phase callback has run, so hooks that
+      # only wrap their yield never trigger the warning.
+      def attributes_snapshot(action, params)
+        return unless around_persistence_hooks?(action)
+        Util::Hash.deep_dup(params)
+      end
+
+      def around_persistence_hooks?(action)
+        hooks = self.class.config[:callbacks][:persistence].try(:[], :around) || []
+        hooks.any? { |hook| hook[:only].include?(action) }
+      end
+
+      def warn_attributes_mutated_in_around_persistence(action, snapshot, params)
+        return if snapshot.nil? || snapshot == params
+
+        changed_keys = (snapshot.keys | params.keys).reject { |key| snapshot[key] == params[key] }
+        Graphiti::DEPRECATOR.warn(<<~MSG)
+          #{self.class}'s around_persistence hook modified the attributes hash before yield (changed keys: #{changed_keys.map(&:inspect).join(", ")}). In Graphiti 2.0, attributes are assigned to the model before persistence hooks run, and these modifications will be silently ignored. Move the modification to a hook that runs before assignment:
+
+            before_attributes do |attributes|
+              attributes[#{changed_keys.first.inspect}] = ...
+            end
+
+          or set the value on the model itself in a before_save hook. See UPGRADING.md in the Graphiti 2.0 release.
+        MSG
+      end
+
       def run_callbacks(kind, action, *args)
         fire_around_callbacks(kind, action, *args) do |*yieldargs|
           fire_callbacks(kind, :before, action, *yieldargs)

data/lib/graphiti/resource_proxy.rb

@@ -80,6 +80,12 @@ module Graphiti
       Renderer.new(self, options).as_graphql
     end
 
+    # Records supplied directly, no scope resolution
+    def data=(models)
+      @data = models
+      [@data].flatten.compact.each { |record| @resource.decorate_record(record) }
+    end
+
     def data
       @data ||= begin
         records = @scope.resolve

data/lib/graphiti/version.rb

@@ -1,3 +1,3 @@
 module Graphiti
-  VERSION = "1.12.2"
+  VERSION = "1.13.0"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: graphiti
 version: !ruby/object:Gem::Version
-  version: 1.12.2
+  version: 1.13.0
 platform: ruby
 authors:
 - Lee Richmond
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-07-29 00:00:00.000000000 Z
+date: 2026-07-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: jsonapi-serializable