graphql-fragment_cache 0.1.4 → 0.1.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f5ab2d2bd11b30a0c40476378f1b9ad0c064ccffec7f4595f9d81dbb0ad85eac
-  data.tar.gz: 44c09dce73b8836f656f186291ed4a45b9cdab6c0ed8844dea38bb9fe9b276bc
+  metadata.gz: 69226c4789fd364bcd6bf0991ebd358c39a32fee383a8ce7445418d0af46eb04
+  data.tar.gz: d7a1901dd86a3311f60a02153422044c055ffd46b0297026ab9f434eca1667dd
 SHA512:
-  metadata.gz: c225d6650aa383253eda036366f92d9a9ae2109c873cc9f4f69e9b736367c31c9c7b0c59c42ad830f4554184af75016239a0c3a739d2e3f1b4621fcbed9814e1
-  data.tar.gz: 556d2eced9aef328be63f06de9404e78e958a06f5cba2394c1fda4d449c63d04399730978767a40a7b4544b41418f1bab96384aafddd819cd4c6bbad79b62af0
+  metadata.gz: 5a22448276b2618d0f85c67aa371fa20828ac24f3f943779d748214214a85d0a55c129b31928e96b904e6e1c8734f4cf5ef32bcb2debf38b9208bfea7fefc29f
+  data.tar.gz: 17033312f4e5c89c3a135da2c2e12b2980fc5b5582ce22b3089ae1e3b7c0e95de44ca1e19f42cb0a236b6fcd7b6ff9630bb300ee1592d9ec4cf6a759f7e82839

data/CHANGELOG.md

@@ -2,6 +2,11 @@
 
 ## master
 
+## 0.1.5 (2020-04-28)
+
+- [PR#19](https://github.com/DmitryTsepelev/graphql-ruby-fragment_cache/pull/19) Add connections support ([@DmitryTsepelev][])
+- [PR#18](https://github.com/DmitryTsepelev/graphql-ruby-fragment_cache/pull/18) Support aliases in cache key generation ([@palkan][], [@DmitryTsepelev][])
+
 ## 0.1.4 (2020-04-25)
 
 - Fix railtie to set up null store for tests ([@DmitryTsepelev][])

data/README.md

@@ -61,6 +61,15 @@ class QueryType < BaseObject
 end
 ```
 
+If you use [connections](https://graphql-ruby.org/pagination/connection_concepts.html) and plan to cache them—please turn on [brand new](https://github.com/rmosolgo/graphql-ruby/blob/master/lib/graphql/pagination/connections.rb#L5) connections hierarchy in your schema:
+
+```ruby
+class GraphqSchema < GraphQL::Schema
+  # ...
+  use GraphQL::Pagination::Connections
+end
+```
+
 ## Cache key generation
 
 Cache keys consist of implicit and explicit (provided by user) parts.
@@ -266,10 +275,6 @@ class QueryType < BaseObject
 end
 ```
 
-## Limitations
-
-- [Field aliases](https://spec.graphql.org/June2018/#sec-Field-Alias) are not currently supported (take a look at the failing spec [here](https://github.com/DmitryTsepelev/graphql-ruby-fragment_cache/pull/7))
-
 ## Credits
 
 Based on the original [gist](https://gist.github.com/palkan/faad9f6ff1db16fcdb1c071ec50e4190) by [@palkan](https://github.com/palkan) and [@ssnickolay](https://github.com/ssnickolay).

data/lib/.rbnext/2.7/graphql/fragment_cache/cache_key_builder.rb

@@ -18,6 +18,71 @@ module GraphQL
           }.join(".")
         end
       end
+
+      refine ::GraphQL::Language::Nodes::AbstractNode do
+        def alias?(_)
+          false
+        end
+      end
+
+      refine ::GraphQL::Language::Nodes::Field do
+        def alias?(val)
+          self.alias == val
+        end
+      end
+
+      refine ::GraphQL::Execution::Lookahead do
+        def selection_with_alias(name, **kwargs)
+          return selection(name, **kwargs) if selects?(name, **kwargs)
+          alias_selection(name, **kwargs)
+        end
+
+        def alias_selection(name, selected_type: @selected_type, arguments: nil)
+          return alias_selections[name] if alias_selections.key?(name)
+
+          alias_node = lookup_alias_node(ast_nodes, name)
+          return ::GraphQL::Execution::Lookahead::NULL_LOOKAHEAD unless alias_node
+
+          next_field_name = alias_node.name
+
+          # From https://github.com/rmosolgo/graphql-ruby/blob/1a9a20f3da629e63ea8e5ee8400be82218f9edc3/lib/graphql/execution/lookahead.rb#L91
+          next_field_defn = get_class_based_field(selected_type, next_field_name)
+
+          alias_selections[name] =
+            if next_field_defn
+              next_nodes = []
+              arguments = @query.arguments_for(alias_node, next_field_defn)
+              arguments = arguments.is_a?(::GraphQL::Execution::Interpreter::Arguments) ? arguments.keyword_arguments : arguments
+              @ast_nodes.each do |ast_node|
+                ast_node.selections.each do |selection|
+                  find_selected_nodes(selection, next_field_name, next_field_defn, arguments: arguments, matches: next_nodes)
+                end
+              end
+
+              if next_nodes.any?
+                ::GraphQL::Execution::Lookahead.new(query: @query, ast_nodes: next_nodes, field: next_field_defn, owner_type: selected_type)
+              else
+                ::GraphQL::Execution::Lookahead::NULL_LOOKAHEAD
+              end
+            else
+              ::GraphQL::Execution::Lookahead::NULL_LOOKAHEAD
+            end
+        end
+
+        def alias_selections
+          return @alias_selections if defined?(@alias_selections)
+          @alias_selections ||= {}
+        end
+
+        def lookup_alias_node(nodes, name)
+          return if nodes.empty?
+          nodes.find do |node|
+            return node if node.alias?(name)
+            child = lookup_alias_node(node.children, name)
+            return child if child
+          end
+        end
+      end
     })
 
     # Builds cache key for fragment
@@ -59,7 +124,7 @@ module GraphQL
 
       def selections_cache_key
         current_root =
-          path.reduce(query.lookahead) { |lkhd, name| lkhd.selection(name) }
+          path.reduce(query.lookahead) { |lkhd, name| lkhd.selection_with_alias(name) }
 
         current_root.selections.to_selections_key
       end
@@ -68,12 +133,13 @@ module GraphQL
         lookahead = query.lookahead
 
         path.map { |field_name|
-          lookahead = lookahead.selection(field_name)
+          lookahead = lookahead.selection_with_alias(field_name)
+          raise "Failed to look ahead the field: #{field_name}" if lookahead.is_a?(::GraphQL::Execution::Lookahead::NullLookahead)
 
-          next field_name if lookahead.arguments.empty?
+          next lookahead.field.name if lookahead.arguments.empty?
 
           args = lookahead.arguments.map { |_1, _2| "#{_1}:#{traverse_argument(_2)}" }.sort.join(",")
-          "#{field_name}(#{args})"
+          "#{lookahead.field.name}(#{args})"
         }.join("/")
       end
 

data/lib/graphql/fragment_cache/cache_key_builder.rb

@@ -18,6 +18,71 @@ module GraphQL
           }.join(".")
         end
       end
+
+      refine ::GraphQL::Language::Nodes::AbstractNode do
+        def alias?(_)
+          false
+        end
+      end
+
+      refine ::GraphQL::Language::Nodes::Field do
+        def alias?(val)
+          self.alias == val
+        end
+      end
+
+      refine ::GraphQL::Execution::Lookahead do
+        def selection_with_alias(name, **kwargs)
+          return selection(name, **kwargs) if selects?(name, **kwargs)
+          alias_selection(name, **kwargs)
+        end
+
+        def alias_selection(name, selected_type: @selected_type, arguments: nil)
+          return alias_selections[name] if alias_selections.key?(name)
+
+          alias_node = lookup_alias_node(ast_nodes, name)
+          return ::GraphQL::Execution::Lookahead::NULL_LOOKAHEAD unless alias_node
+
+          next_field_name = alias_node.name
+
+          # From https://github.com/rmosolgo/graphql-ruby/blob/1a9a20f3da629e63ea8e5ee8400be82218f9edc3/lib/graphql/execution/lookahead.rb#L91
+          next_field_defn = get_class_based_field(selected_type, next_field_name)
+
+          alias_selections[name] =
+            if next_field_defn
+              next_nodes = []
+              arguments = @query.arguments_for(alias_node, next_field_defn)
+              arguments = arguments.is_a?(::GraphQL::Execution::Interpreter::Arguments) ? arguments.keyword_arguments : arguments
+              @ast_nodes.each do |ast_node|
+                ast_node.selections.each do |selection|
+                  find_selected_nodes(selection, next_field_name, next_field_defn, arguments: arguments, matches: next_nodes)
+                end
+              end
+
+              if next_nodes.any?
+                ::GraphQL::Execution::Lookahead.new(query: @query, ast_nodes: next_nodes, field: next_field_defn, owner_type: selected_type)
+              else
+                ::GraphQL::Execution::Lookahead::NULL_LOOKAHEAD
+              end
+            else
+              ::GraphQL::Execution::Lookahead::NULL_LOOKAHEAD
+            end
+        end
+
+        def alias_selections
+          return @alias_selections if defined?(@alias_selections)
+          @alias_selections ||= {}
+        end
+
+        def lookup_alias_node(nodes, name)
+          return if nodes.empty?
+          nodes.find do |node|
+            return node if node.alias?(name)
+            child = lookup_alias_node(node.children, name)
+            return child if child
+          end
+        end
+      end
     })
 
     # Builds cache key for fragment
@@ -59,7 +124,7 @@ module GraphQL
 
       def selections_cache_key
         current_root =
-          path.reduce(query.lookahead) { |lkhd, name| lkhd.selection(name) }
+          path.reduce(query.lookahead) { |lkhd, name| lkhd.selection_with_alias(name) }
 
         current_root.selections.to_selections_key
       end
@@ -68,12 +133,13 @@ module GraphQL
         lookahead = query.lookahead
 
         path.map { |field_name|
-          lookahead = lookahead.selection(field_name)
+          lookahead = lookahead.selection_with_alias(field_name)
+          raise "Failed to look ahead the field: #{field_name}" if lookahead.is_a?(::GraphQL::Execution::Lookahead::NullLookahead)
 
-          next field_name if lookahead.arguments.empty?
+          next lookahead.field.name if lookahead.arguments.empty?
 
           args = lookahead.arguments.map { "#{_1}:#{traverse_argument(_2)}" }.sort.join(",")
-          "#{field_name}(#{args})"
+          "#{lookahead.field.name}(#{args})"
         }.join("/")
       end
 

data/lib/graphql/fragment_cache/fragment.rb

@@ -6,7 +6,9 @@ module GraphQL
   module FragmentCache
     # Represents a single fragment to cache
     class Fragment
-      attr_reader :options, :path, :context
+      attr_reader :options, :path, :context, :raw_connection
+
+      attr_writer :raw_connection
 
       def initialize(context, **options)
         @context = context
@@ -19,7 +21,7 @@ module GraphQL
       end
 
       def persist(final_value)
-        value = resolve(final_value)
+        value = raw_connection || resolve(final_value)
         FragmentCache.cache_store.write(cache_key, value, **options)
       end
 

data/lib/graphql/fragment_cache/object_helpers.rb

@@ -6,23 +6,78 @@ module GraphQL
   module FragmentCache
     using Ext
 
+    RawConnection = Struct.new(:items, :nodes, :paged_nodes_offset, :has_previous_page, :has_next_page)
+
     # Adds #cache_fragment method
     module ObjectHelpers
+      extend Forwardable
+
       NO_OBJECT = Object.new
 
+      def_delegator :field, :connection?
+
       def cache_fragment(object_to_cache = NO_OBJECT, **options, &block)
         raise ArgumentError, "Block or argument must be provided" unless block_given? || object_to_cache != NO_OBJECT
 
         options[:object] = object_to_cache if object_to_cache != NO_OBJECT
+
         fragment = Fragment.new(context, options)
 
         if (cached = fragment.read)
-          return raw_value(cached)
+          return restore_cached_value(cached)
+        end
+
+        (block_given? ? block.call : object_to_cache).tap do |resolved_value|
+          cache_value(resolved_value, fragment)
+        end
+      end
+
+      private
+
+      def restore_cached_value(cached)
+        connection? ? restore_cached_connection(cached) : raw_value(cached)
+      end
+
+      def cache_value(resolved_value, fragment)
+        if connection?
+          unless context.schema.new_connections?
+            raise StandardError,
+              "GraphQL::Pagination::Connections should be enabled for connection caching"
+          end
+
+          connection = wrap_connection(resolved_value)
+
+          fragment.raw_connection = RawConnection.new(
+            connection.items,
+            connection.nodes,
+            connection.instance_variable_get(:@paged_nodes_offset),
+            connection.has_previous_page,
+            connection.has_next_page
+          )
         end
 
         context.fragments << fragment
+      end
+
+      def field
+        interpreter_context[:current_field]
+      end
+
+      def interpreter_context
+        @interpreter_context ||= context.namespace(:interpreter)
+      end
+
+      def restore_cached_connection(raw_connection)
+        wrap_connection(raw_connection.items).tap do |connection|
+          connection.instance_variable_set(:@nodes, raw_connection.nodes)
+          connection.instance_variable_set(:@paged_nodes_offset, raw_connection.paged_nodes_offset)
+          connection.instance_variable_set(:@has_previous_page, raw_connection.has_previous_page)
+          connection.instance_variable_set(:@has_next_page, raw_connection.has_next_page)
+        end
+      end
 
-        block_given? ? block.call : object_to_cache
+      def wrap_connection(object)
+        context.schema.connections.wrap(field, object, interpreter_context[:current_arguments], context)
       end
     end
   end

data/lib/graphql/fragment_cache/version.rb

@@ -2,6 +2,6 @@
 
 module GraphQL
   module FragmentCache
-    VERSION = "0.1.4"
+    VERSION = "0.1.5"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: graphql-fragment_cache
 version: !ruby/object:Gem::Version
-  version: 0.1.4
+  version: 0.1.5
 platform: ruby
 authors:
 - DmitryTsepelev
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-04-25 00:00:00.000000000 Z
+date: 2020-04-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: graphql
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.10.3
+        version: 1.10.8
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.10.3
+        version: 1.10.8
 - !ruby/object:Gem::Dependency
   name: ruby-next-core
   requirement: !ruby/object:Gem::Requirement
@@ -52,6 +52,34 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.1'
+- !ruby/object:Gem::Dependency
+  name: activerecord
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: sqlite3
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement