graphiti_graphql 0.1.8 → 0.1.12

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 40128f0eaf0ad7859fa35c67bb73b2a2fccf51829bb3914877ce0c19730def91
-  data.tar.gz: 6f3fe80c618aaa093dd2daec4aa726621c32d2228bb92ddfdfd00f6a610490af
+  metadata.gz: e50aa66c6cf3fda89d96665c6f2fc61fa92db821b4d00461101970d72c24eea6
+  data.tar.gz: 1e058e770b73378802fd43b3e852720d9cfa9b5b80cfc77c40ea1e2e2ea602df
 SHA512:
-  metadata.gz: f957bb6f2092f052d683c94008940823a29cb0a995d4741eaadbf2b87f0e3585268e01819a7fde057afd0a99c0f2b4dd14d2fc76903f3e44cc51c05724a0bd69
-  data.tar.gz: 30c3b26190d74c0559553458d05fdd36367fb9742b320322138d0f7a9ac503775d14edddb3a25b28276b5c099588186554f6ac686da0095175c557a921c1029e
+  metadata.gz: 7ecb9fda5f17d3414b2ae8a39f5e9fddb40f34ed683fc9d966c1098bd86f93422e9ee77439c8f27faed1384ab5876ee67bbfc296980f314c7f656930edee3734
+  data.tar.gz: f00540102b8cccd704a2ef2d6b7cb787c750f23858b78e425e1dc59e452065b16c9d8788065c0a0c2e87a653fac70d803f743f72ead06405f207e549e0f8e132

data/Gemfile.lock

@@ -1,17 +1,17 @@
 PATH
   remote: .
   specs:
-    graphiti_graphql (0.1.6)
+    graphiti_graphql (0.1.12)
       activesupport (>= 4.1)
-      graphiti (~> 1.3)
+      graphiti (~> 1.3.2)
       graphql (~> 1.12)
 
 GEM
   remote: https://rubygems.org/
   specs:
-    activemodel (6.1.1)
-      activesupport (= 6.1.1)
-    activesupport (6.1.1)
+    activemodel (6.1.4.1)
+      activesupport (= 6.1.4.1)
+    activesupport (6.1.4.1)
       concurrent-ruby (~> 1.0, >= 1.0.2)
       i18n (>= 1.6, < 2)
       minitest (>= 5.1)
@@ -23,7 +23,7 @@ GEM
     ast (2.4.2)
     byebug (11.1.3)
     coderay (1.1.3)
-    concurrent-ruby (1.1.8)
+    concurrent-ruby (1.1.9)
     diff-lcs (1.4.4)
     dry-configurable (0.12.1)
       concurrent-ruby (~> 1.0)
@@ -43,8 +43,8 @@ GEM
       dry-core (~> 0.5, >= 0.5)
       dry-inflector (~> 0.1, >= 0.1.2)
       dry-logic (~> 1.0, >= 1.0.2)
-    google-protobuf (3.15.2)
-    graphiti (1.3.0)
+    google-protobuf (3.17.3)
+    graphiti (1.3.2)
       activesupport (>= 4.1)
       concurrent-ruby (~> 1.0)
       dry-types (>= 0.15.0, < 2.0)
@@ -56,11 +56,11 @@ GEM
     graphiti_spec_helpers (1.0.5)
       graphiti (>= 1.0.alpha.1)
       rspec (~> 3.0)
-    graphql (1.12.3)
+    graphql (1.12.15)
     graphql-batch (0.4.3)
       graphql (>= 1.3, < 2)
       promise.rb (~> 0.7.2)
-    i18n (1.8.9)
+    i18n (1.8.10)
       concurrent-ruby (~> 1.0)
     jsonapi-renderer (0.2.2)
     jsonapi-serializable (0.3.1)
@@ -68,19 +68,19 @@ GEM
     method_source (1.0.0)
     minitest (5.14.4)
     parallel (1.20.1)
-    parser (3.0.0.0)
+    parser (3.0.2.0)
       ast (~> 2.4.1)
     promise.rb (0.7.4)
-    pry (0.13.1)
+    pry (0.14.1)
       coderay (~> 1.1)
       method_source (~> 1.0)
-    pry-byebug (3.9.0)
+    pry-byebug (3.8.0)
       byebug (~> 11.0)
-      pry (~> 0.13.0)
+      pry (~> 0.10)
     rainbow (3.0.0)
     rake (10.5.0)
-    regexp_parser (2.0.3)
-    rexml (3.2.4)
+    regexp_parser (2.1.1)
+    rexml (3.2.5)
     rspec (3.10.0)
       rspec-core (~> 3.10.0)
       rspec-expectations (~> 3.10.0)
@@ -94,24 +94,24 @@ GEM
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.10.0)
     rspec-support (3.10.2)
-    rubocop (1.8.1)
+    rubocop (1.20.0)
       parallel (~> 1.10)
       parser (>= 3.0.0.0)
       rainbow (>= 2.2.2, < 4.0)
       regexp_parser (>= 1.8, < 3.0)
       rexml
-      rubocop-ast (>= 1.2.0, < 2.0)
+      rubocop-ast (>= 1.9.1, < 2.0)
       ruby-progressbar (~> 1.7)
       unicode-display_width (>= 1.4.0, < 3.0)
-    rubocop-ast (1.4.1)
-      parser (>= 2.7.1.5)
-    rubocop-performance (1.9.2)
-      rubocop (>= 0.90.0, < 2.0)
+    rubocop-ast (1.11.0)
+      parser (>= 3.0.1.1)
+    rubocop-performance (1.11.5)
+      rubocop (>= 1.7.0, < 2.0)
       rubocop-ast (>= 0.4.0)
     ruby-progressbar (1.11.0)
-    standard (0.12.0)
-      rubocop (= 1.8.1)
-      rubocop-performance (= 1.9.2)
+    standard (1.3.0)
+      rubocop (= 1.20.0)
+      rubocop-performance (= 1.11.5)
     standardrb (1.0.0)
       standard
     tzinfo (2.0.4)
@@ -137,4 +137,4 @@ DEPENDENCIES
   standardrb
 
 BUNDLED WITH
-   2.2.16
+   2.2.25

data/graphiti_graphql.gemspec

@@ -27,7 +27,7 @@ Gem::Specification.new do |spec|
   spec.executables = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
   spec.require_paths = ["lib"]
 
-  spec.add_dependency "graphiti", "~> 1.3"
+  spec.add_dependency "graphiti", "~> 1.3.2"
   spec.add_dependency "activesupport", ">= 4.1"
   spec.add_dependency "graphql", "~> 1.12"
 

data/lib/graphiti_graphql/engine.rb

@@ -64,7 +64,6 @@ module GraphitiGraphQL
     end
 
     initializer "graphiti_graphql.define_controller" do
-      GraphitiGraphQL::Engine.reloader_class.to_prepare do
         app_controller = GraphitiGraphQL.config.federation_application_controller || ::ApplicationController
         # rubocop:disable Lint/ConstantDefinitionInBlock(Standard)
         class GraphitiGraphQL::ExecutionController < app_controller
@@ -74,7 +73,6 @@ module GraphitiGraphQL
             render json: Graphiti.gql(params[:query], params[:variables])
           end
         end
-      end
     end
 
     initializer "graphiti_graphql.federation" do

data/lib/graphiti_graphql/federation/resource_dsl.rb

@@ -7,10 +7,11 @@ module GraphitiGraphQL
           @type_name = type_name
         end
 
-        def has_many(relationship_name, foreign_key: nil, &blk)
+        def has_many(relationship_name, foreign_key: nil, magic: true, &blk)
           @caller.federated_has_many relationship_name,
             type: @type_name,
             foreign_key: foreign_key,
+            magic: magic,
             &blk
         end
       end
@@ -29,14 +30,18 @@ module GraphitiGraphQL
           config[:federated_resources] ||= []
         end
 
+        # NB - for external use, use federated_type("Type").has_many instead
+        #
         # * Add to the list of external graphql-ruby types we need in schema
         # * Add a readable and filterable FK, without clobbering pre-existing
-        def federated_has_many(name, type:, foreign_key: nil, &blk)
+        def federated_has_many(name, type:, magic: true, foreign_key: nil, &blk)
           foreign_key ||= :"#{type.underscore}_id"
           resource = FederatedResource.new(type)
           federated_resources << resource
           resource.add_relationship(:has_many, name, self, foreign_key, &blk)
 
+          return unless magic
+
           attribute = attributes.find { |name, config|
             name.to_sym == foreign_key &&
               !!config[:readable] &&

data/lib/graphiti_graphql/schema.rb

@@ -52,14 +52,18 @@ module GraphitiGraphQL
       @base_interface || BaseInterface
     end
 
-    def self.generate(entrypoint_resources = nil)
+    def self.generate(resources: nil, entrypoints: nil)
       instance = new
       schema = Class.new(::GraphitiGraphQL.schema_class || GraphQL::Schema)
       graphiti_schema = GraphitiGraphQL::GraphitiSchema::Wrapper
         .new(Graphiti::Schema.generate)
       instance.graphiti_schema = graphiti_schema
+
+      resources ||= graphiti_schema.resources
+      entrypoints ||= self.entrypoints || resources
+
       instance.schema = schema
-      instance.apply_query(entrypoint_resources || entrypoints)
+      instance.apply_query(resources, entrypoints)
       instance
     end
 
@@ -72,8 +76,8 @@ module GraphitiGraphQL
       @query_fields = {}
     end
 
-    def apply_query(entries)
-      query_type = generate_schema_query(entries)
+    def apply_query(resources, entrypoints)
+      query_type = generate_schema_query(resources, entrypoints)
       Federation::SchemaDecorator.decorate(self) if self.class.federation?
 
       # NB - don't call .query here of federation will break things
@@ -86,7 +90,7 @@ module GraphitiGraphQL
       schema.query # Actually fires the federation code
     end
 
-    def generate_schema_query(entrypoint_resources = nil)
+    def generate_schema_query(resources, entrypoints)
       existing_query = schema.instance_variable_get(:@query)
       existing_query ||= schema.send(:find_inherited_value, :query)
       # NB - don't call graphql_schema.query here of federation will break things
@@ -95,10 +99,13 @@ module GraphitiGraphQL
       query_class.graphql_name "Query"
       query_class.field_class BaseField
 
-      get_entrypoints(entrypoint_resources).each do |resource|
-        next if resource.remote?
+      # Ensure all types are generated, even if they aren't in a top-level Query
+      # We might now want to expose the query, but do want to federate the Type
+      get_schema_resources(resources).each do |resource|
         generate_type(resource)
+      end
 
+      get_schema_resources(entrypoints).each do |resource|
         add_index(query_class, resource)
         add_show(query_class, resource)
       end
@@ -198,14 +205,14 @@ module GraphitiGraphQL
         top_level_single: true
     end
 
-    def get_entrypoints(manually_specified)
+    def get_schema_resources(manually_specified)
       resources = graphiti_schema.resources
       if manually_specified
         resources = resources.select { |r|
           manually_specified.map(&:name).include?(r.name)
         }
       end
-      resources
+      resources.reject(&:remote?)
     end
 
     def generate_sort_att_type_for(resource)
@@ -505,7 +512,7 @@ module GraphitiGraphQL
     end
 
     class PageType < GraphQL::Schema::InputObject
-      graphql_name "Page"
+      graphql_name "PageInput"
       argument :size, Int, required: false
       argument :number, Int, required: false
       argument :after, String, required: false
@@ -516,8 +523,8 @@ module GraphitiGraphQL
       graphql_name "PageInfo"
       field :has_next_page, Boolean, null: false
       field :has_previous_page, Boolean, null: false
-      field :start_cursor, String, null: false
-      field :end_cursor, String, null: false
+      field :start_cursor, String, null: true
+      field :end_cursor, String, null: true
     end
 
     class SortDirType < GraphQL::Schema::Enum

data/lib/graphiti_graphql/version.rb

@@ -1,3 +1,3 @@
 module GraphitiGraphQL
-  VERSION = "0.1.8"
+  VERSION = "0.1.12"
 end

data/lib/graphiti_graphql.rb

@@ -53,8 +53,11 @@ module GraphitiGraphQL
       @generated ||= GraphitiGraphQL::Schema.generate
     end
 
-    def generate!(entrypoint_resources = nil)
-      @generated = GraphitiGraphQL::Schema.generate(entrypoint_resources)
+    # resources: All resources to put in the schema. Mostly just for testing.
+    # entrypoints: All resources to expose as top-level query objects
+    def generate!(resources: nil, entrypoints: nil)
+      @generated = GraphitiGraphQL::Schema
+        .generate(resources: resources, entrypoints: entrypoints)
     end
 
     def generated?

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: graphiti_graphql
 version: !ruby/object:Gem::Version
-  version: 0.1.8
+  version: 0.1.12
 platform: ruby
 authors:
 - Lee Richmond
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2021-08-08 00:00:00.000000000 Z
+date: 2021-08-31 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: graphiti
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.3'
+        version: 1.3.2
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.3'
+        version: 1.3.2
 - !ruby/object:Gem::Dependency
   name: activesupport
   requirement: !ruby/object:Gem::Requirement