graph_weaver 0.7.1 → 0.7.3

data/lib/graph_weaver.rb

@@ -65,8 +65,8 @@ module GraphWeaver
     #
     # Anything satisfying the execute contract — a Client, a schema class,
     # a transport, a fake (testing's graphql: tag swaps one in per
-    # example). Generated modules resolve per call -> per module
-    # (MyQuery.client=) -> baked constant -> here.
+    # example). Generated modules resolve per call -> a test mode's stand-in
+    # -> the client their graph names -> here.
     attr_accessor :client
 
     # the default client, when one is required
@@ -195,6 +195,7 @@ module GraphWeaver
 
     def generated_paths=(paths)
       @generated_paths = paths && Array(paths)
+      hide_from_autoloading(generated_paths)
     end
 
     def fragments_paths=(paths)
@@ -287,9 +288,24 @@ module GraphWeaver
       # generate! reports in however many times the initializer has re-run
       existing = @graphs.index { |candidate| candidate.name == name }
       existing ? @graphs[existing] = graph : @graphs << graph
+      hide_from_autoloading([graph.output])
       graph
     end
 
+    # Rails only: where a graph writes is hidden from Zeitwerk the moment the
+    # graph says it. Knowing it any later needed an initializer edge waiting on
+    # config/initializers, and such an edge reorders the whole app's boot (see
+    # the Railtie). Registered first, so a refusal can name the graph.
+    def hide_from_autoloading(paths)
+      # by name, not spelled: the railtie is Rails-only and `typed: ignore`, so
+      # sorbet has no constant to resolve
+      return unless const_defined?(:Railtie, false)
+
+      railtie = const_get(:Railtie, false)
+      paths.each { |path| railtie.ignore_output!(path) }
+    end
+    private :hide_from_autoloading
+
     # Every graph an entry point walks: the declared ones, or the single graph
     # the top-level settings describe. Never empty.
     def graphs = @graphs&.dup || [default_graph]
@@ -389,7 +405,7 @@ module GraphWeaver
     # (see #changed_files). Generated files the plan no longer produces are deleted
     # (see #orphaned), so renaming or dropping a .graphql leaves nothing
     # behind. Pair with a freshness spec (docs/generated_modules.md).
-    def generate!(schema: nil, queries: nil, output: nil, client: nil, types_module: nil)
+    def generate!(schema: nil, queries: nil, output: nil, types_module: nil)
       @changed_files = []
       @unmatched_registrations = []
       @untyped_scalars_by_graph = {}
@@ -399,7 +415,7 @@ module GraphWeaver
       # tree exactly as it was — the railtie's watch mode regenerates on a
       # request and promises a failed save changes nothing, and that promise
       # was true within a graph and false across them.
-      planned = graphs_for(schema:, queries:, output:, client:, types_module:).map do |graph|
+      planned = graphs_for(schema:, queries:, output:, types_module:).map do |graph|
         if Internal::Util.query_files(graph.queries).empty?
           # a brand-new app legitimately has none; a mistyped queries_paths looks
           # exactly the same, and prints nothing either way
@@ -488,11 +504,11 @@ module GraphWeaver
     #      it "generated queries are current" do
     #        GraphWeaver.verify_generated!
     #      end
-    def verify_generated!(schema: nil, queries: nil, output: nil, client: nil, types_module: nil)
+    def verify_generated!(schema: nil, queries: nil, output: nil, types_module: nil)
       @unmatched_registrations = []
       @untyped_scalars_by_graph = {}
       seen = new_seen
-      graphs = graphs_for(schema:, queries:, output:, client:, types_module:)
+      graphs = graphs_for(schema:, queries:, output:, types_module:)
 
       # The dump is checked in too, and everything below reads it — so a
       # stale one is answered before staleness downstream of it, because
@@ -860,7 +876,6 @@ module GraphWeaver
           schema:,
           query: Codegen.inline_fragments(source, shared, path),
           name:,
-          client: graph.client,
           graph_name: graph.name,
           types_namespace: graph.types_module,
           hoistable_unions: Codegen.shared_fragment_spreads(source, shared, path),
@@ -1082,8 +1097,10 @@ module GraphWeaver
     # name derived from the file name and the operation — see #module_name) or
     # a raw query string (name derived from the operation name, falling back to
     # "Query" for anonymous operations — collisions are impossible since each
-    # parse gets its own container). Pass name: to override, client: to bake
-    # the module's default client/transport.
+    # parse gets its own container). Pass name: to override, client: to say
+    # what the module runs against — a parsed module generates no file, so it
+    # has no graph to read one off, and this is the only time one is bound
+    # (a per-call `client:` still wins).
     #
     # graph: names the graph this module belongs to, which is what a test mode
     # runs it against in an app with more than one — the same thing generation

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: graph_weaver
 version: !ruby/object:Gem::Version
-  version: 0.7.1
+  version: 0.7.3
 platform: ruby
 authors:
 - Daniel Pepper
@@ -363,7 +363,7 @@ licenses:
 - MIT
 metadata:
   bug_tracker_uri: https://github.com/dpep/graph_weaver/issues
-  changelog_uri: https://github.com/dpep/graph_weaver/blob/v0.7.1/CHANGELOG.md
+  changelog_uri: https://github.com/dpep/graph_weaver/blob/v0.7.3/CHANGELOG.md
   documentation_uri: https://github.com/dpep/graph_weaver/tree/main/docs
   rubygems_mfa_required: 'true'
   source_code_uri: https://github.com/dpep/graph_weaver