ecoportal-api-graphql 2.0.0 → 2.2.0

data/lib/ecoportal/api/graphql/input/workflow_command.rb

@@ -17,6 +17,8 @@ module Ecoportal
             createPublicConfig:                'Ecoportal::API::GraphQL::Input::WorkflowCommand::CreatePublicConfig',
             editPublicConfig:                  'Ecoportal::API::GraphQL::Input::WorkflowCommand::EditPublicConfig',
             archivePublicConfig:               'Ecoportal::API::GraphQL::Input::WorkflowCommand::ArchivePublicConfig',
+            # Copy-page configuration (20260819)
+            manageCopyPageConfiguration:       'Ecoportal::API::GraphQL::Input::WorkflowCommand::ManageCopyPageConfiguration',
             # Stage structural
             addStage:                          'Ecoportal::API::GraphQL::Input::WorkflowCommand::AddStage',
             removeStage:                       'Ecoportal::API::GraphQL::Input::WorkflowCommand::RemoveStage',
@@ -176,6 +178,7 @@ require_relative 'workflow_command/edit_template_container_uid'
 require_relative 'workflow_command/create_public_config'
 require_relative 'workflow_command/edit_public_config'
 require_relative 'workflow_command/archive_public_config'
+require_relative 'workflow_command/manage_copy_page_configuration'
 require_relative 'workflow_command/add_checklist_field_item'
 require_relative 'workflow_command/edit_checklist_field_item'
 require_relative 'workflow_command/remove_checklist_field_item'

data/lib/ecoportal/api/graphql/interface/base_page.rb

@@ -37,6 +37,20 @@ module Ecoportal
             Array(doc['locations']).map { |l| l.is_a?(Hash) ? l['id'] : l }.compact
           end
 
+          # Compiled tag set (client-side build of the backend-only "compiled_tags").
+          # This base implementation covers a non-phased page (and is the fallback for a
+          # phased page with no stages): there are no per-stage tags, so the set is just the
+          # page-level components, de-duplicated:
+          #
+          #   all_tags = Unique(baseTags + otherTags + location_ids)
+          #
+          # Model::Page::Phased overrides this to take the union across its stages' #all_tags.
+          # Returns a de-duplicated Array of tag strings.
+          def all_tags
+            (Array(baseTags) + Array(otherTags) + Array(location_ids)).
+              compact.uniq
+          end
+
           def location_ids=(ids)
             doc['locations'] = Array(ids).map { |id_val| { 'id' => id_val } }
           end

data/lib/ecoportal/api/graphql/interface/location_structure/node.rb

@@ -1,43 +1,43 @@
-module Ecoportal
-  module API
-    class GraphQL
-      module Interface
-        module LocationStructure
-          class Node < Logic::BaseModel
-            root!
-
-            class_resolver :location_classification_class, 'Ecoportal::API::GraphQL::Base::LocationClassificationType'
-            class_resolver :location_node_class, self
-
-            passkey :id
-            passthrough :name
-            embeds_many :classifications, klass: :location_classification_class
-            passarray :classificationObjectIds
-
-            passthrough :weight
-
-            passboolean :archived
-            passthrough :archivedToken
-
-            passthrough :structureId
-
-            embeds_one :parent, klass: :location_node_class, nullable: true
-            passthrough :parentObjectId
-
-            def parentId
-              parent&.id
-            end
-            alias_method :parent_id, :parentId
-
-            def parent_id_upcase
-              parent_id&.upcase
-            end
-
-            embeds_many :ancestors, klass: :location_node_class, read_only: true
-            embeds_many :children, klass: :location_node_class, read_only: true
-          end
-        end
-      end
-    end
-  end
-end
+module Ecoportal
+  module API
+    class GraphQL
+      module Interface
+        module LocationStructure
+          class Node < Logic::BaseModel
+            root!
+
+            class_resolver :location_classification_class, 'Ecoportal::API::GraphQL::Base::LocationClassificationType'
+            class_resolver :location_node_class, self
+
+            passkey :id
+            passthrough :name
+            embeds_many :classifications, klass: :location_classification_class
+            passarray :classificationObjectIds
+
+            passthrough :weight
+
+            passboolean :archived
+            passthrough :archivedToken
+
+            passthrough :structureId
+
+            embeds_one :parent, klass: :location_node_class, nullable: true
+            passthrough :parentObjectId
+
+            def parentId
+              parent&.id
+            end
+            alias_method :parent_id, :parentId
+
+            def parent_id_upcase
+              parent_id&.upcase
+            end
+
+            embeds_many :ancestors, klass: :location_node_class, read_only: true
+            embeds_many :children, klass: :location_node_class, read_only: true
+          end
+        end
+      end
+    end
+  end
+end

data/lib/ecoportal/api/graphql/interface/location_structure.rb

@@ -1,14 +1,14 @@
-module Ecoportal
-  module API
-    class GraphQL
-      module Interface
-        module LocationStructure
-        end
-      end
-    end
-  end
-end
-
-require_relative 'location_structure/node'
-require_relative 'location_structure/nodes'
-require_relative 'location_structure/command'
+module Ecoportal
+  module API
+    class GraphQL
+      module Interface
+        module LocationStructure
+        end
+      end
+    end
+  end
+end
+
+require_relative 'location_structure/node'
+require_relative 'location_structure/nodes'
+require_relative 'location_structure/command'

data/lib/ecoportal/api/graphql/interface.rb

@@ -1,11 +1,11 @@
-module Ecoportal
-  module API
-    class GraphQL
-      module Interface
-      end
-    end
-  end
-end
-
-require_relative 'interface/location_structure'
-require_relative 'interface/base_page'
+module Ecoportal
+  module API
+    class GraphQL
+      module Interface
+      end
+    end
+  end
+end
+
+require_relative 'interface/location_structure'
+require_relative 'interface/base_page'

data/lib/ecoportal/api/graphql/logic/base_model.rb

@@ -1,15 +1,15 @@
-module Ecoportal
-  module API
-    class GraphQL
-      module Logic
-        class BaseModel < Ecoportal::API::Common::GraphQL::Model
-          include Ecoportal::API::GraphQL::Concerns::SnakeCamelAccess
-
-          class << self
-            attr_accessor :client
-          end
-        end
-      end
-    end
-  end
-end
+module Ecoportal
+  module API
+    class GraphQL
+      module Logic
+        class BaseModel < Ecoportal::API::Common::GraphQL::Model
+          include Ecoportal::API::GraphQL::Concerns::SnakeCamelAccess
+
+          class << self
+            attr_accessor :client
+          end
+        end
+      end
+    end
+  end
+end

data/lib/ecoportal/api/graphql/logic/base_query.rb

@@ -1,193 +1,260 @@
-module Ecoportal
-  module API
-    class GraphQL
-      module Logic
-        class BaseQuery
-          include Ecoportal::API::Common::GraphQL::ClassHelpers
-
-          inheritable_attrs :accepted_params, :param_defaults
-
-          DETAIL_ERRORS = [
-            Ecoportal::API::Common::GraphQL::ResponseError,
-            Ecoportal::API::Common::Client::Error::UnexpectedServerError,
-            Faraday::ParsingError,
-            Graphlient::Errors::FaradayServerError,
-            Graphlient::Errors::GraphQLError,
-            Graphlient::Errors::ClientError
-          ].freeze
-
-          class << self
-            def param_defaults
-              @param_defaults ||= {}
-            end
-
-            def accepted_params(*keys, default: :unused)
-              @accepted_params ||= []
-              return @accepted_params if keys.empty?
-
-              keys.map(&:to_sym).uniq.each do |key|
-                @accepted_params   |= [key]
-                param_defaults[key] = default unless default == :unused
-              end
-
-              @accepted_params.push(*keys).tap(&:uniq!)
-              @accepted_params
-            end
-
-            def clear_accepted_params
-              @param_defaults  = {}
-              @accepted_params = []
-            end
-
-            def slice_params(kargs)
-              kargs.slice(*accepted_params)
-            end
-
-            # Used to obtain the full `path` in the GraphQL query by using `base_path`
-            # @note it is meant for reusability of queries from different end-points
-            def field_name(str = nil)
-              return @field_name unless str
-
-              @field_name = nil
-              @field_name = str.to_s if str
-              @field_name
-            end
-
-            def base_path(path = :unused)
-              return @base_path if path == :unused
-
-              path ||= []
-              path   = path.to_s.split('.') if path.is_a?(String)
-              path   = path.map(&:to_s).compact
-
-              @base_path = path
-            end
-          end
-
-          include Ecoportal::API::Common::Concerns::Benchmarkable
-
-          attr_reader :client, :base_path
-
-          def initialize(client, path: nil, base_path: self.class.base_path)
-            @path      = path
-            @base_path = base_path
-            @client    = client
-          end
-
-          # Resolves the `path` by using `path` or `base_path` + `class.field_name`.
-          def path(field_name = self.class.field_name)
-            result   = @path
-            result ||= default_path if respond_to?(:default_path, true)
-            result ||= (base_path + [field_name]) if base_path && field_name
-            result ||= [field_name]
-            result
-          end
-
-          # Query rely that manages the different blocks.
-          # @return [Class] an object of `response_class` with the results hanging from `path`.
-          def query(path: self.path, raw_response: {}, **kargs, &block)
-            print = !is_a?(Mutation)
-
-            benchmarking("#{self.class}##{__method__}", print: print) do
-              graphql_query(
-                path:         path,
-                raw_response: raw_response,
-                **kargs,
-                &basic_block(&block)
-              )
-            end
-          end
-
-          def response_class
-            raise "You should override this method in #{self.class}"
-          end
-
-          def access_point(path = [])
-            path.last
-          end
-
-          private
-
-          def basic_block
-            raise "This method should be implemented in the child class #{self.class}"
-          end
-
-          # Types that should be passed through as-is as GraphQL variable values.
-          # Anything else that responds to to_h is treated as a serialisable value object.
-          VARIABLE_PRIMITIVES = [
-            ::NilClass, ::TrueClass, ::FalseClass,
-            ::Integer, ::Float, ::String, ::Symbol,
-            ::Hash, ::Array
-          ].freeze
-          private_constant :VARIABLE_PRIMITIVES
-
-          def normalize_variables(kargs)
-            kargs.transform_values do |v|
-              VARIABLE_PRIMITIVES.any? { |t| v.is_a?(t) } || !v.respond_to?(:to_h) ? v : v.to_h
-            end
-          end
-
-          def graphql_query(path: self.path, raw_response: {}, **kargs, &block)
-            default_params = self.class.param_defaults.dup
-            query_params   = self.class.slice_params(normalize_variables(default_params.merge(kargs)))
-
-            query_str    = client.to_query_string(&block)
-            fragment_str = assemble_fragments(query_str)
-            full_query   = fragment_str.empty? ? query_str : "#{query_str}\n\n#{fragment_str}"
-
-            request(*path) do
-              client.http_client.execute(full_query, variables: query_params).tap do |response|
-                raw_response[:data] = response
-              end
-            end
-          rescue *DETAIL_ERRORS => _err
-            pp kargs
-            puts "Internal Error with these params:\n"
-            pp query_params
-            pp block
-            raise
-          end
-
-          def request(*path)
-            response = yield
-            wrap_response(response, path)
-          end
-
-          # Resolves all fragment spreads in query_str, recursively following
-          # spreads inside assembled fragment bodies until no new names are found.
-          # Names that don't resolve to a registered fragment are skipped safely.
-          def assemble_fragments(query_str)
-            tried     = {}
-            assembled = {}
-            pending   = query_str.scan(/\.\.\.(\w+)/).flatten.map(&:to_sym).uniq
-
-            until (unresolved = pending.reject { |n| tried.key?(n) }).empty?
-              unresolved.each do |name|
-                tried[name] = true
-                frag        = Ecoportal::API::GraphQL::Fragment.assemble(name)
-                next if frag.empty?
-
-                assembled[name] = frag
-                pending |= frag.scan(/\.\.\.(\w+)/).flatten.map(&:to_sym)
-              end
-            end
-
-            assembled.values.join("\n\n")
-          end
-
-          def wrap_response(response, path = [])
-            # we use the Hash, because fragments are persmissioned
-            # see Use of Fragments at https://github.com/ashkan18/graphlient/tree/master?tab=readme-ov-file#use-of-fragments
-            unless (res = response.to_h.dig(*path.dup.unshift('data')))
-              msg = "Complete failure on request. Path: #{path}\n#{response.to_h.pretty_inspect}"
-              raise msg
-            end
-
-            data = Ecoportal::API::Common::GraphQL::HashHelpers.deep_dup(res)
-            response_class.new(data)
-          end
-        end
-      end
-    end
-  end
-end
+module Ecoportal
+  module API
+    class GraphQL
+      module Logic
+        class BaseQuery
+          include Ecoportal::API::Common::GraphQL::ClassHelpers
+
+          inheritable_attrs :accepted_params, :param_defaults
+
+          DETAIL_ERRORS = [
+            Ecoportal::API::Common::GraphQL::ResponseError,
+            Ecoportal::API::Common::Client::Error::UnexpectedServerError,
+            Faraday::ParsingError,
+            Graphlient::Errors::FaradayServerError,
+            Graphlient::Errors::GraphQLError,
+            Graphlient::Errors::ClientError
+          ].freeze
+
+          class << self
+            def param_defaults
+              @param_defaults ||= {}
+            end
+
+            def accepted_params(*keys, default: :unused)
+              @accepted_params ||= []
+              return @accepted_params if keys.empty?
+
+              keys.map(&:to_sym).uniq.each do |key|
+                @accepted_params   |= [key]
+                param_defaults[key] = default unless default == :unused
+              end
+
+              @accepted_params.push(*keys).tap(&:uniq!)
+              @accepted_params
+            end
+
+            def clear_accepted_params
+              @param_defaults  = {}
+              @accepted_params = []
+            end
+
+            def slice_params(kargs)
+              kargs.slice(*accepted_params)
+            end
+
+            # Used to obtain the full `path` in the GraphQL query by using `base_path`
+            # @note it is meant for reusability of queries from different end-points
+            def field_name(str = nil)
+              return @field_name unless str
+
+              @field_name = nil
+              @field_name = str.to_s if str
+              @field_name
+            end
+
+            def base_path(path = :unused)
+              return @base_path if path == :unused
+
+              path ||= []
+              path   = path.to_s.split('.') if path.is_a?(String)
+              path   = path.map(&:to_s).compact
+
+              @base_path = path
+            end
+          end
+
+          include Ecoportal::API::Common::Concerns::Benchmarkable
+
+          attr_reader :client, :base_path
+
+          def initialize(client, path: nil, base_path: self.class.base_path)
+            @path      = path
+            @base_path = base_path
+            @client    = client
+          end
+
+          # Resolves the `path` by using `path` or `base_path` + `class.field_name`.
+          def path(field_name = self.class.field_name)
+            result   = @path
+            result ||= default_path if respond_to?(:default_path, true)
+            result ||= (base_path + [field_name]) if base_path && field_name
+            result ||= [field_name]
+            result
+          end
+
+          # Query rely that manages the different blocks.
+          # @return [Class] an object of `response_class` with the results hanging from `path`.
+          def query(path: self.path, raw_response: {}, **kargs, &block)
+            print = !is_a?(Mutation)
+
+            benchmarking("#{self.class}##{__method__}", print: print) do
+              graphql_query(
+                path:         path,
+                raw_response: raw_response,
+                **kargs,
+                &basic_block(&block)
+              )
+            end
+          end
+
+          def response_class
+            raise "You should override this method in #{self.class}"
+          end
+
+          def access_point(path = [])
+            path.last
+          end
+
+          private
+
+          def basic_block
+            raise "This method should be implemented in the child class #{self.class}"
+          end
+
+          # Types that should be passed through as-is as GraphQL variable values.
+          # Anything else that responds to to_h is treated as a serialisable value object.
+          VARIABLE_PRIMITIVES = [
+            ::NilClass, ::TrueClass, ::FalseClass,
+            ::Integer, ::Float, ::String, ::Symbol,
+            ::Hash, ::Array
+          ].freeze
+          private_constant :VARIABLE_PRIMITIVES
+
+          def normalize_variables(kargs)
+            kargs.transform_values do |v|
+              VARIABLE_PRIMITIVES.any? { |t| v.is_a?(t) } || !v.respond_to?(:to_h) ? v : v.to_h
+            end
+          end
+
+          def graphql_query(path: self.path, raw_response: {}, **kargs, &block)
+            default_params = self.class.param_defaults.dup
+            query_params   = self.class.slice_params(normalize_variables(default_params.merge(kargs)))
+
+            query_str    = client.to_query_string(&block)
+            fragment_str = assemble_fragments(query_str)
+            full_query   = fragment_str.empty? ? query_str : "#{query_str}\n\n#{fragment_str}"
+
+            request(*path) do
+              client.http_client.execute(full_query, variables: query_params).tap do |response|
+                raw_response[:data] = response
+              end
+            end
+          rescue *DETAIL_ERRORS => err
+            report_query_failure(err, query_params)
+            raise
+          end
+
+          # Reports WHERE a query failed, on stderr, and re-raises untouched.
+          #
+          # ★ This replaces an unconditional `pp kargs` / `pp query_params` / `pp block`
+          # to STDOUT. That dump was NOT debug litter — it existed because graphlient
+          # masks the origin of a failure: `Client#parse`/`#execute` do
+          # `raise Graphlient::Errors::ClientError, e.message`, which leaves graphlient's
+          # own `inner_exception` nil AND replaces the backtrace, discarding the source
+          # location `graphql-client` deliberately sets via
+          # `set_backtrace(["#{filename}:#{line}"])`. With that gone there was no other
+          # way to see which query blew up. (Tracked upstream; see
+          # `.ai-assistance/code/refactoring/opportunities.md`.)
+          #
+          # But the location was never actually destroyed. Ruby populates `.cause`
+          # automatically when you raise inside a rescue, so the original error — with the
+          # location intact — is one hop away. Verified by execution:
+          #
+          #   err.inner_exception        => nil                    (graphlient's api, broken)
+          #   err.backtrace.first        => ".../client.rb:20"     (the raise site, useless)
+          #   err.cause.backtrace.first  => "app/queries/x.rb:198" (the QUERY site)
+          #
+          # So this reports MORE of what the dump was for (the failing query's own source
+          # location) and LESS of what it should never have emitted: `query_params` holds
+          # GraphQL variables — page ids, emails, field values — and printing them to
+          # stdout put customer data into CI logs. Variables are now shown only when the
+          # client's existing `deep_logging` flag is on (see `dump_variables?` for why NOT
+          # the logger level). `pp block` is dropped outright: inspecting a Proc never
+          # identified anything.
+          #
+          # Emitted with `warn` (stderr, and suppressible via the Warning module) rather than
+          # `puts`, so tools that pipe query results keep a clean stdout. Tracked as Q-43.
+          def report_query_failure(err, query_params = nil)
+            origin = err.cause
+            lines  = ["[#{self.class.name || self.class}] #{err.class}: #{failure_message(err)}"]
+            lines << "  origin: #{origin.class}" if origin
+            lines << "  at:     #{origin.backtrace.first}" if origin&.backtrace&.first
+            lines << "  params: #{query_params.inspect}" if dump_variables?
+
+            warn(lines.join("\n"))
+          end
+
+          # graphlient's `GraphQLError#to_s` / `ExecutionError#to_s` format from
+          # `errors.details` and render an EMPTY string when that is empty — and because
+          # they override `to_s`, the fallback text their constructors pass to `super` is
+          # unreachable. An exception with a blank message is invisible in a log, so say
+          # so rather than printing nothing.
+          def failure_message(err)
+            msg = err.message.to_s
+            return msg unless msg.empty?
+
+            "(blank — #{err.class} renders its message from errors.details, which was empty)"
+          end
+
+          # Opt-in via the client's EXISTING `deep_logging` flag ("whether or not batch
+          # responses should be logged") — the same class of thing as dumping variables,
+          # and it defaults to false. No new env var and no new config knob.
+          #
+          # ⚠ Do NOT gate this on `logger.debug?`, which is the obvious-looking choice and
+          # is wrong: `HttpClient`'s default logger is `::Logger.new(IO::NULL)`, and a
+          # fresh Logger's level is DEBUG (0) — so `logger.debug?` is `true` out of the
+          # box and variables would leak by default, which is the whole thing being fixed.
+          # A spec pins this.
+          def dump_variables?
+            http = client&.http_client
+            return false unless http.respond_to?(:deep_logging?, true)
+
+            !!http.send(:deep_logging?)
+          rescue StandardError
+            false
+          end
+
+          def request(*path)
+            response = yield
+            wrap_response(response, path)
+          end
+
+          # Resolves all fragment spreads in query_str, recursively following
+          # spreads inside assembled fragment bodies until no new names are found.
+          # Names that don't resolve to a registered fragment are skipped safely.
+          def assemble_fragments(query_str)
+            tried     = {}
+            assembled = {}
+            pending   = query_str.scan(/\.\.\.(\w+)/).flatten.map(&:to_sym).uniq
+
+            until (unresolved = pending.reject { |n| tried.key?(n) }).empty?
+              unresolved.each do |name|
+                tried[name] = true
+                frag        = Ecoportal::API::GraphQL::Fragment.assemble(name)
+                next if frag.empty?
+
+                assembled[name] = frag
+                pending |= frag.scan(/\.\.\.(\w+)/).flatten.map(&:to_sym)
+              end
+            end
+
+            assembled.values.join("\n\n")
+          end
+
+          def wrap_response(response, path = [])
+            # we use the Hash, because fragments are persmissioned
+            # see Use of Fragments at https://github.com/ashkan18/graphlient/tree/master?tab=readme-ov-file#use-of-fragments
+            unless (res = response.to_h.dig(*path.dup.unshift('data')))
+              msg = "Complete failure on request. Path: #{path}\n#{response.to_h.pretty_inspect}"
+              raise msg
+            end
+
+            data = Ecoportal::API::Common::GraphQL::HashHelpers.deep_dup(res)
+            response_class.new(data)
+          end
+        end
+      end
+    end
+  end
+end