graphiti_gql 0.2.29 → 0.2.30

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ade099acc586ef5902b1b30659ca47efed5cfb3e05e3fe1a778fa81b31b27528
-  data.tar.gz: 3a2226feab0dd284c3aeb832b30d90dc8165a9abc03fc40921aab47eece3d315
+  metadata.gz: a02dc0813b461497979a165c77f9116922559c46a54ba26a5130e31832c4bcef
+  data.tar.gz: 60a27d5d31c31a39d062d1a65a2c0278ce3ea6f3cba3c4900ed1332525a54460
 SHA512:
-  metadata.gz: 318d2eb2831f5d19bdcbdd9fbe6dd0a21fed8435875d9d4de0ca45b65b97515b5ca4f80afca282a5c1e11b85986133f5942ef4e96b21b4cb31ce492d35a07728
-  data.tar.gz: bc7b778005b056904a8fc4d3be767ee62a711f07fff533ddac833b96723abc5fc8368e9f63ecf2ddc40aee59544990939ff7d26a570a95ccc15ce2d8d5e93542
+  metadata.gz: e2a9a66b5dd624962bfffdc733115f8cf0363846db434bf9dfbe76374cd94dd0b9411ccf2aa93ce543a144ad0505e9f8d1e39bcb34b5593a7f870e17d7f13440
+  data.tar.gz: eb5004a261e6b7b815eb16ab548591c27beda1231e0875103c75d059c885ed2311b7d8c2b4d1a651d08905facdb8478b328eb5f78ec0fb812dab8cfc5f175583

data/lib/graphiti_gql/active_resource.rb

@@ -37,18 +37,20 @@ module GraphitiGql
       def edge(name, node_id)
         found = @edges[name].empty? ? nil : @edges[name]
         if found && node_id
-          found.find { |f| f.instance_variable_get(:@node_id) == node_id.to_s }
+          found.find do |f|
+            gql_node_id = f.instance_variable_get(:@node_id)
+            gql_node_id == node_id ||
+              gql_node_id == node_id.to_s ||
+              gql_node_id == @resource.gid(node_id)
+          end
         else
           found
         end
       end
 
-      def decoded_id
-        Base64.decode64(self.id)
-      end
-
-      def int_id
-        decoded_id.to_i
+      def _id
+        deserialized = @resource.deserialize_gid(id)
+        deserialized.to_i.to_s == deserialized ? deserialized.to_i : deserialized
       end
     end
 
@@ -77,7 +79,7 @@ module GraphitiGql
           data # fire query
           Node.new(underscore(data[data.keys.first]), @resource)
         else
-          nodes.find { |n| n.id == id.to_s }
+          nodes.find { |n| n.id == id.to_s || n.id == @resource.gid(id) }
         end
       end
 

data/lib/graphiti_gql/graphiti_hax.rb

@@ -71,13 +71,46 @@ module GraphitiGql
     end
 
     class_methods do
+      def gid(*ids)
+        ids = ids[0] if ids[0].is_a?(Array) && ids.length == 1
+        gids = ids.map { |id| serialize_gid(id) }
+        ids.length == 1 ? gids.first : gids
+      end
+
+      def serialize_gid(id)
+        Base64.encode64("#{graphql_name}/#{id}")
+      end
+
+      def deserialize_gid(gid, integer = true)
+        id = Base64.decode64(gid).split("/").last
+        id.to_i.to_s == id ? id.to_i : id
+      end
+
       def config
         return @config if @config
         super
         @config = @config.merge(value_objects: {}, is_value_object: false)
       end
 
+      def graphql_name
+        @graphql_name ||= begin
+          if name.nil?
+            'Unknown'
+          else
+            name.gsub('Resource', '').gsub('::', '')
+          end
+        end
+      end
+
       def attribute(*args)
+        # new default is :gid, not integer_id
+        if args[0] == :id && args[1] == :integer_id
+          args[1] = :gid
+          opts = args.extract_options!
+          opts[:null] = false
+          args << opts
+        end
+
         super(*args).tap do
           opts = args.extract_options!
           att = config[:attributes][args[0]]
@@ -320,6 +353,7 @@ module GraphitiGql
     description: 'Datetime with milliseconds'
   }
 
+  # Support ranges
   [:string, :integer, :float, :datetime, :precise_datetime].each do |kind|
     duped_hash = Graphiti::Util::Hash.deep_dup(Graphiti::Types[:hash])
     type = Graphiti::Types[:"#{kind}_range"] = duped_hash
@@ -334,6 +368,29 @@ module GraphitiGql
     }
   end
 
+  # Support gid
+  gid_going_out = ->(id) {
+    resource.gid(id)
+  }
+
+  gid_coming_in = definition.constructor do |input|
+    if defined?(ApplicationResource)
+      ApplicationResource.deserialize_gid(input)
+    else
+      Graphiti::Resource.deserialize_gid(input)
+    end
+  end
+
+  # Register it with Graphiti
+  Graphiti::Types[:gid] = {
+    params: gid_coming_in,
+    read: gid_going_out,
+    write: gid_coming_in,
+    kind: 'scalar',
+    canonical_name: :gid,
+    description: 'Globally-unique identifier',
+  }
+
   module ActiveRecordAdapterExtras
     extend ActiveSupport::Concern
 
@@ -344,6 +401,8 @@ module GraphitiGql
       alias_method :filter_precise_datetime_gte, :filter_gte
       alias_method :filter_precise_datetime_eq, :filter_eq
       alias_method :filter_precise_datetime_not_eq, :filter_not_eq
+      alias_method :filter_gid_eq, :filter_eq
+      alias_method :filter_gid_not_eq, :filter_not_eq
     end
 
     # TODO: integration specs mysql vs postgres for case sensitivity
@@ -393,11 +452,14 @@ module GraphitiGql
     class << self
       alias :old_default_operators :default_operators
       def default_operators
-        old_default_operators.merge({
-          precise_datetime: numerical_operators,
+        ops = old_default_operators.merge({
+          precise_datetime: numerical_operators - [:not_eq],
           string_enum: [:eq, :not_eq],
           integer_enum: [:eq, :not_eq],
+          gid: [:eq, :not_eq]
         })
+        ops[:datetime] = ops[:datetime] - [:not_eq]
+        ops
       end
     end
   end

data/lib/graphiti_gql/loaders/belongs_to.rb

@@ -1,6 +1,10 @@
 module GraphitiGql
   module Loaders
-    class FakeRecord < Struct.new(:id, :type)
+    class FakeRecord < Struct.new(:id, :type, :graphiti_resource)
+      def initialize(*args)
+        super
+        @__graphiti_resource = graphiti_resource
+      end
     end
 
     class BelongsTo < GraphQL::Batch::Loader
@@ -21,11 +25,11 @@ module GraphitiGql
             ids.each do |id|
               child = @sideload.children.values.find { |c| c.group_name == id[:type].to_sym }
               type = Schema::Registry.instance.get(child.resource.class, interface: false)[:type]
-              fulfill(id, FakeRecord.new(id[:id], type))
+              fulfill(id, FakeRecord.new(id[:id], type, child.resource))
             end
           else
             type = Schema::Registry.instance.get(@sideload.resource.class)[:type]
-            ids.each { |id| fulfill(id, FakeRecord.new(id, type)) }
+            ids.each { |id| fulfill(id, FakeRecord.new(id, type, @sideload.resource)) }
           end
           return
         end
@@ -40,9 +44,10 @@ module GraphitiGql
           payload.each_pair do |key, value|
             params = { filter: {} }
             klass = @sideload.children.values.find { |c| c.group_name == key.to_sym }
-            params = {
-              filter: { id: { eq: value.join(",") } }
-            }
+            filter_ids = value.map { |id| @sideload.resource.class.gid(id.to_s) }
+            params = @params.merge({
+              filter: { id: { eq: filter_ids } }
+            })
 
             futures << Concurrent::Future.execute do
                { type: key, data: klass.resource.class.all(params).data }
@@ -55,9 +60,10 @@ module GraphitiGql
           end
         else
           resource = Schema.registry.get(@sideload.resource.class)[:resource]
-          params = {}
+          params = @params.deep_dup
           unless resource.singular
-            params[:filter] = {id: { eq: ids.join(",") } }
+            filter_ids = ids.map { |id| @sideload.resource.class.gid(id.to_s) }
+            params[:filter] = {id: { eq: filter_ids } }
           end
           records = resource.all(params).data
           if resource.singular

data/lib/graphiti_gql/loaders/many.rb

@@ -15,7 +15,7 @@ module GraphitiGql
 
       def initialize(sideload, params)
         @sideload = sideload
-        @params = params
+        @params = params.merge(typecast_filters: false)
       end
 
       def perform(parent_records)
@@ -34,6 +34,7 @@ module GraphitiGql
             pk
           end
         end
+        ids.compact!
 
         build_params(ids, parent_records)
         resource = Schema.registry.get(@sideload.resource.class)[:resource]
@@ -53,13 +54,16 @@ module GraphitiGql
         if @sideload.polymorphic_as
           type = ids[0][:"#{@sideload.polymorphic_as}_type"]
           foreign_keys = ids.map { |id| id[@sideload.foreign_key] }
+          foreign_keys.map! { |id| @sideload.parent_resource.class.gid(id) }
           @params[:filter][:"#{@sideload.polymorphic_as}_type"] = type
-          @params[:filter][@sideload.foreign_key] = foreign_keys.join(",")
+          @params[:filter][@sideload.foreign_key] = foreign_keys
         elsif @sideload.type == :many_to_many
+          filter_ids = ids.map { |id| @sideload.parent_resource.class.gid(id) }
           fk = @sideload.foreign_key.values.first
-          @params[:filter].merge!(fk => { eq: ids.join(",") })
+          @params[:filter].merge!(fk => { eq: filter_ids })
         elsif !@sideload.parent_resource.class.singular
-          @params[:filter].merge!(@sideload.foreign_key => { eq: ids.join(",") })
+          filter_ids = ids.map { |id| @sideload.parent_resource.class.gid(id) }
+          @params[:filter].merge!(@sideload.foreign_key => { eq: filter_ids })
         end
 
         if @params[:stats]

data/lib/graphiti_gql/schema/fields/attribute.rb

@@ -37,7 +37,16 @@ module GraphitiGql
               end
             end
             return if value.nil?
-            Graphiti::Types[_config[:type]][:read].call(value)
+
+            caster = Graphiti::Types[_config[:type]][:read]
+            # Dry::Types can't be instance_exec'd
+            # This dependency has probably served it's purpose and can be
+            # refactored away
+            if caster.is_a?(Proc)
+              instance_exec(value, &caster)
+            else
+              caster.call(value)
+            end
           end
         end
 
@@ -51,7 +60,6 @@ module GraphitiGql
             if !field_type
               canonical_graphiti_type = Graphiti::Types.name_for(@config[:type])
               field_type = GQL_TYPE_MAP[canonical_graphiti_type.to_sym]
-              field_type = String if @name == :id
             end
             field_type = [field_type] if @config[:type].to_s.starts_with?("array_of")
             field_type

data/lib/graphiti_gql/schema/fields/show.rb

@@ -12,7 +12,7 @@ module GraphitiGql
             null: true,
             extras: [:lookahead]
           unless @registered[:resource].singular
-            field.argument(:id, String, required: true)
+            field.argument(:id, GraphQL::Types::ID, required: true)
           end
           _registered = @registered
           query.define_method name do |**arguments|

data/lib/graphiti_gql/schema/list_arguments.rb

@@ -77,7 +77,6 @@ module GraphitiGql
             canonical_graphiti_type = Graphiti::Types
               .name_for(filter_config[:type])
             type = GQL_TYPE_MAP[canonical_graphiti_type]
-            type = String if filter_name == :id
           end
 
           if (allowlist = filter_config[:allow])

data/lib/graphiti_gql/schema/registry.rb

@@ -60,8 +60,7 @@ module GraphitiGql
       private
 
       def key_for_resource(resource)
-        resource.graphql_name ||
-          resource.name.gsub('Resource', '').gsub('::', '')
+        resource.graphql_name
       end
     end
   end

data/lib/graphiti_gql/schema/resource_type.rb

@@ -141,6 +141,17 @@ module GraphitiGql
           klass = Class.new(Schema.base_object)
         end
 
+        klass.define_method :resource do
+          return @resource if @resource
+          resource = object.instance_variable_get(:@__graphiti_resource)
+          resource_class = resource.class
+          if resource_class.polymorphic? && !resource_class.polymorphic_child?
+            resource_class = resource_class.resource_for_model(object)
+          end
+          @resource = resource_class
+          resource_class
+        end
+
         klass
       end
 

data/lib/graphiti_gql/schema.rb

@@ -33,6 +33,7 @@ module GraphitiGql
       integer_id: String,
       string: String,
       uuid: String,
+      gid: GraphQL::Types::ID,
       integer: Integer,
       big_integer: GraphQL::Types::BigInt,
       float: Float,
@@ -68,7 +69,9 @@ module GraphitiGql
     end
 
     def self.base_object
+      return @base_object if @base_object
       klass = Class.new(GraphQL::Schema::Object)
+
       # TODO make this config maybe
       if defined?(ActionView)
         klass.send(:include, ActionView::Helpers::TranslationHelper)
@@ -79,6 +82,7 @@ module GraphitiGql
           end
         end
       end
+      @base_object = klass
       klass
     end
 

data/lib/graphiti_gql/spec_helper.rb

@@ -74,6 +74,9 @@ module GraphitiGql
       resource.gql(params.merge(fields: fields), ctx, q, opts)
     end
 
+    def gid(*ids)
+      resource.gid(*ids)
+    end
 
     def run
       lambda do

data/lib/graphiti_gql/version.rb

@@ -1,3 +1,3 @@
 module GraphitiGql
-  VERSION = "0.2.29"
+  VERSION = "0.2.30"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: graphiti_gql
 version: !ruby/object:Gem::Version
-  version: 0.2.29
+  version: 0.2.30
 platform: ruby
 authors:
 - Lee Richmond
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2022-08-25 00:00:00.000000000 Z
+date: 2022-08-31 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: graphql