ecoportal-api-graphql 1.3.16 → 2.2.0

data/lib/ecoportal/api/graphql/connection/person_member.rb

@@ -3,7 +3,7 @@ module Ecoportal
     class GraphQL
       module Connection
         class PersonMemberConnection < Logic::Connection
-          class_resolver :node_class, Model::PersonMember
+          nodes_of Model::PersonMember
         end
       end
     end

data/lib/ecoportal/api/graphql/connection/person_member_changelog.rb

@@ -0,0 +1,11 @@
+module Ecoportal
+  module API
+    class GraphQL
+      module Connection
+        class PersonMemberChangelog < Logic::Connection
+          nodes_of Model::PersonMemberChangelog
+        end
+      end
+    end
+  end
+end

data/lib/ecoportal/api/graphql/connection/preset_view.rb

@@ -1,11 +1,11 @@
-module Ecoportal
-  module API
-    class GraphQL
-      module Connection
-        class PresetView < Logic::Connection
-          class_resolver :item_class, 'Ecoportal::API::GraphQL::Model::PresetView'
-        end
-      end
-    end
-  end
-end
+module Ecoportal
+  module API
+    class GraphQL
+      module Connection
+        class PresetView < Logic::Connection
+          class_resolver :item_class, 'Ecoportal::API::GraphQL::Model::PresetView'
+        end
+      end
+    end
+  end
+end

data/lib/ecoportal/api/graphql/connection/preview_page.rb

@@ -3,7 +3,7 @@ module Ecoportal
     class GraphQL
       module Connection
         class PreviewPage < Logic::Connection
-          class_resolver :node_class, 'Ecoportal::API::GraphQL::Model::PreviewPage'
+          nodes_of 'Ecoportal::API::GraphQL::Model::PreviewPage'
         end
       end
     end

data/lib/ecoportal/api/graphql/connection/template_change_log.rb

@@ -0,0 +1,19 @@
+module Ecoportal
+  module API
+    class GraphQL
+      module Connection
+        # `Template.commands` — a `PagesWorkflowCommandInterfaceConnection` carrying
+        # `totalCount` (added by `Types::CustomConnections::BaseConnection`), `pageInfo`
+        # and `nodes`.
+        #
+        # ★ A nil connection here means NOT PERMITTED, not "no changes" — see
+        # `Base::Template::ChangeLogEntry` for the two access caveats.
+        class TemplateChangeLog < Logic::Connection
+          # `nodes_of` (not `class_resolver :node_class`) — see Logic::Connection for why the
+          # shared dynamic collection poisons sibling connections otherwise.
+          nodes_of Model::Template::ChangeLogEntry
+        end
+      end
+    end
+  end
+end

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

@@ -15,3 +15,9 @@ require_relative 'connection/page'
 require_relative 'connection/preview_page'
 require_relative 'connection/preset_view'
 require_relative 'connection/pages_workflow_command'
+require_relative 'connection/activity'
+require_relative 'connection/activity_log'
+require_relative 'connection/auth_log'
+require_relative 'connection/person_member_changelog'
+require_relative 'connection/minimal_dashboard'
+require_relative 'connection/template_change_log'

data/lib/ecoportal/api/graphql/diff/change.rb

@@ -23,13 +23,18 @@ module Ecoportal
         # that parent_id alone cannot carry. Set for an ADDED :field (parent_id = its section id,
         # owner_id = its stage id) because addField requires BOTH sectionId and stageId (both
         # NON_NULL in the schema). nil for every other kind/op.
+        #
+        # value: for an ADDED :option, the select option's `value` (the schema's addSelectFieldOption
+        # requires it — see AddSelectFieldOptionInput, backend authority). Read from the source doc's
+        # option.value (VersionDiff#index_options); nil when the source doc did not carry one. NEVER
+        # defaulted to label/weight by the synthesiser — see CommandSynthesizer#option_command.
         Change = Struct.new(:op, :kind, :id, :label, :path, :attribute, :before, :after, :parent_id,
-                            :by_type, :owner_id, keyword_init: true) do
+                            :by_type, :owner_id, :value, keyword_init: true) do
           def to_h
             {
               op: op, kind: kind, id: id, label: label, path: path,
               attribute: attribute, before: before, after: after, parent_id: parent_id,
-              by_type: by_type, owner_id: owner_id
+              by_type: by_type, owner_id: owner_id, value: value
             }.compact
           end
 

data/lib/ecoportal/api/graphql/diff/command_synthesizer.rb

@@ -277,13 +277,31 @@ module Ecoportal
 
             field_ref = ref(field_id)
             case change.op
-            when :added   then build(:addSelectFieldOption, data_field_id: field_ref, label: change.label)
+            when :added   then add_option(change, field_ref)
             when :removed then build(:removeSelectFieldOption, data_field_id: field_ref, option_id: change.id)
             when :changed then edit_option(change, field_ref)
             else unsupported!(change)
             end
           end
 
+          # addSelectFieldOption requires `value` (backend: AddSelectFieldOptionInput, required: true
+          # — a real, distinct field from label/weight; see OptionType#value). NEVER default it to
+          # the label/weight (a numeric select field stores `value` as a number, so a label stand-in
+          # would silently persist wrong data). When the source doc's option carried no `value`
+          # (Change#value nil — see VersionDiff#index_options), raise loudly instead of guessing.
+          def add_option(change, field_ref)
+            if change.value.nil?
+              raise ArgumentError,
+                    "CommandSynthesizer: cannot build addSelectFieldOption for option " \
+                    "'#{change.label}' (field #{field_ref}) — Change#value is nil. The source doc's " \
+                    "option is missing `value` (VersionDiff#index_options reads it from " \
+                    "option['value']); supply it on the source doc, or on the Change directly when " \
+                    "hand-assembling one."
+            end
+
+            build(:addSelectFieldOption, data_field_id: field_ref, label: change.label, value: change.value)
+          end
+
           def edit_option(change, field_id)
             case change.attribute
             when 'label'  then build(:editSelectFieldOption, data_field_id: field_id, option_id: change.id, label: change.after)

data/lib/ecoportal/api/graphql/diff/pairing/signals.rb

@@ -25,13 +25,33 @@ module Ecoportal
             # MISMATCH scores 0 rather than vetoing (the field may have been re-purposed keeping its
             # genome, or genome may be absent on newer field types). Returns nil when neither side
             # carries a genome — the signal simply does not apply and is excluded from the average.
+            #
+            # LINEAGE (verified live 2026-08-05, same-org backup restore of a full template): a
+            # restore does NOT equal-copy the signature — it APPENDS one segment to it, for every
+            # membrane AND force (e.g. `2zwxkb6gns` → `2zwxkb6gns53jp`, `ktnrya` → `ktnrya877l`;
+            # 28/28 fields + 4/4 forces followed the rule). So an ancestor's signature is a strict
+            # PREFIX of its descendant's, and equality-only scoring would give the exact
+            # cross-object case this engine exists for (UAT backup → PROD restore) a 0.0 — worse
+            # than no signal. Prefix ancestry therefore scores just below an exact match.
             def genome(source, target)
               a = source['genomeSignature']
               b = target['genomeSignature']
               return nil if blank?(a) && blank?(b)
               return 0.0 if blank?(a) || blank?(b)
+              return 1.0 if a == b
 
-              a == b ? 1.0 : 0.0
+              lineage?(a, b) ? 0.9 : 0.0
+            end
+
+            # True when one signature is a strict prefix of the other (restore/copy appends a
+            # lineage segment — see #genome). Direction is unknowable at a pairing call (either
+            # side may be the older document), so both directions count. Very short signatures
+            # are rejected as ancestry evidence — a trivial prefix can collide by chance.
+            def lineage?(sig_a, sig_b)
+              short, long = [sig_a.to_s, sig_b.to_s].minmax_by(&:length)
+              return false if short.length < 4
+
+              short != long && long.start_with?(short)
             end
 
             # Field type (__typename) equality. A type change is a strong DISQUALIFIER for a data

data/lib/ecoportal/api/graphql/diff/version_diff.rb

@@ -91,8 +91,11 @@ module Ecoportal
           # addStageSection.stageId), threading through a placeholder when the parent is created in
           # the same batch. nil (default) means no parent id is known/recorded — the historical
           # behaviour, unchanged.
-          def diff_index(kind, before_idx, after_idx, label:, path:, parent: nil, owner: nil, &compare)
-            procs   = { label: label, path: path, parent: parent, owner: owner }
+          # `value:` (optional proc) is recorded on ADDED changes only — currently used for :option,
+          # whose addSelectFieldOption command requires the source doc's option.value (see
+          # Change#value). nil (default) means the kind has no `value` concept.
+          def diff_index(kind, before_idx, after_idx, label:, path:, parent: nil, owner: nil, value: nil, &compare)
+            procs   = { label: label, path: path, parent: parent, owner: owner, value: value }
             removed = (before_idx.keys - after_idx.keys).map { |id| removed_change(kind, id, before_idx[id], procs) }
             added   = (after_idx.keys - before_idx.keys).map { |id| added_change(kind, id, after_idx[id], procs) }
             matched = (before_idx.keys & after_idx.keys).flat_map { |id| Array(compare.call(id, before_idx[id], after_idx[id])) }
@@ -105,7 +108,8 @@ module Ecoportal
 
           def added_change(kind, id, info, procs)
             Change.new(op: :added, kind: kind, id: id, label: procs[:label].call(info), path: procs[:path].call(info),
-                       parent_id: procs[:parent]&.call(info), owner_id: procs[:owner]&.call(info))
+                       parent_id: procs[:parent]&.call(info), owner_id: procs[:owner]&.call(info),
+                       value: procs[:value]&.call(info))
           end
 
           def stage_changes
@@ -153,7 +157,7 @@ module Ecoportal
 
           def options_diff(field_id, path, before_field, after_field)
             diff_index(:option, index_options(before_field), index_options(after_field),
-                       label: ->(o) { o[:label] }, path: ->(_o) { path }) do |k, b, a|
+                       label: ->(o) { o[:label] }, path: ->(_o) { path }, value: ->(o) { o[:value] }) do |k, b, a|
               cmp = []
               cmp << changed(:option, k, a[:label], path, 'label', b[:label], a[:label], field_id) if b[:label] != a[:label]
               cmp << changed(:option, k, a[:label], path, 'weight', b[:weight], a[:weight], field_id) if b[:weight] != a[:weight]
@@ -294,13 +298,16 @@ module Ecoportal
           end
 
           # Options keyed by id when present, else by value (primary) / name (auxiliary) — the
-          # correct select-option identity (NOT genome).
+          # correct select-option identity (NOT genome). `value` is carried through separately from
+          # `label` (they are distinct schema fields — OptionType#value vs #label/#name) so an ADDED
+          # option's Change can supply the real value addSelectFieldOption requires, never a
+          # label/weight stand-in.
           def index_options(field)
             Array(field['options']).each_with_object({}) do |o, h|
               next unless o.is_a?(Hash)
 
               key = o['id'] || o['value'] || o['label'] || o['name']
-              h[key] = { label: o['label'] || o['name'] || o['value'], weight: o['weight'] }
+              h[key] = { label: o['label'] || o['name'] || o['value'], weight: o['weight'], value: o['value'] }
             end
           end
 

data/lib/ecoportal/api/graphql/error/locations_error/fetch_nested.rb

@@ -1,51 +1,51 @@
-module Ecoportal
-  module API
-    class GraphQL
-      module Error
-        class LocationsError < Logic::BaseModel
-          module FetchNested
-            def locations_error_doc
-              return unless locations_error?
-
-              if loc_err_conflicting_ids?
-                {
-                  conflictingIds: loc_err_conflicting_ids.doc
-                }
-              elsif validationErrors?
-                {
-                  validationErrors: validationErrors.doc
-                }
-              end
-            end
-
-            def locations_error?
-              return false if error.nil?
-
-              !error.empty?
-            end
-
-            def loc_err_conflicting_ids?
-              loc_err_conflicting_ids.any?
-            end
-
-            def validationErrors?
-              validationErrors.any?
-            end
-
-            def loc_err_conflicting_ids
-              return [] unless (err = error)
-
-              err.conflictingIds || []
-            end
-
-            def validationErrors
-              return [] unless (err = error)
-
-              err.validationErrors || []
-            end
-          end
-        end
-      end
-    end
-  end
-end
+module Ecoportal
+  module API
+    class GraphQL
+      module Error
+        class LocationsError < Logic::BaseModel
+          module FetchNested
+            def locations_error_doc
+              return unless locations_error?
+
+              if loc_err_conflicting_ids?
+                {
+                  conflictingIds: loc_err_conflicting_ids.doc
+                }
+              elsif validationErrors?
+                {
+                  validationErrors: validationErrors.doc
+                }
+              end
+            end
+
+            def locations_error?
+              return false if error.nil?
+
+              !error.empty?
+            end
+
+            def loc_err_conflicting_ids?
+              loc_err_conflicting_ids.any?
+            end
+
+            def validationErrors?
+              validationErrors.any?
+            end
+
+            def loc_err_conflicting_ids
+              return [] unless (err = error)
+
+              err.conflictingIds || []
+            end
+
+            def validationErrors
+              return [] unless (err = error)
+
+              err.validationErrors || []
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/lib/ecoportal/api/graphql/error/locations_error.rb

@@ -1,28 +1,28 @@
-module Ecoportal
-  module API
-    class GraphQL
-      module Error
-        class LocationsError < Logic::BaseModel
-          read_only!
-
-          passthrough :type, :message
-          passarray :conflictingIds
-          embeds_many :validationErrors, klass: LocationsValidationError
-
-          def empty?
-            return false unless conflictingIds.empty?
-
-            validationErrors.empty?
-          end
-
-          def any?
-            !empty?
-          end
-        end
-      end
-    end
-  end
-end
-
-
-require_relative 'locations_error/fetch_nested'
+module Ecoportal
+  module API
+    class GraphQL
+      module Error
+        class LocationsError < Logic::BaseModel
+          read_only!
+
+          passthrough :type, :message
+          passarray :conflictingIds
+          embeds_many :validationErrors, klass: LocationsValidationError
+
+          def empty?
+            return false unless conflictingIds.empty?
+
+            validationErrors.empty?
+          end
+
+          def any?
+            !empty?
+          end
+        end
+      end
+    end
+  end
+end
+
+
+require_relative 'locations_error/fetch_nested'

data/lib/ecoportal/api/graphql/error/locations_validation_error.rb

@@ -1,20 +1,20 @@
-module Ecoportal
-  module API
-    class GraphQL
-      module Error
-        class LocationsValidationError < Logic::BaseModel
-          read_only!
-
-          passthrough :error, :message, :property
-
-          def empty?
-            return false unless error.nil?
-            return false unless message.nil?
-
-            property.nil?
-          end
-        end
-      end
-    end
-  end
-end
+module Ecoportal
+  module API
+    class GraphQL
+      module Error
+        class LocationsValidationError < Logic::BaseModel
+          read_only!
+
+          passthrough :error, :message, :property
+
+          def empty?
+            return false unless error.nil?
+            return false unless message.nil?
+
+            property.nil?
+          end
+        end
+      end
+    end
+  end
+end

data/lib/ecoportal/api/graphql/error/validation_errors.rb

@@ -1,28 +1,28 @@
-module Ecoportal
-  module API
-    class GraphQL
-      module Error
-        class ValidationErrors < Logic::BaseModel
-          include Enumerable
-
-          passthrough :details
-          passarray :fullMessages
-
-          def empty?
-            count < 1
-          end
-
-          def any?
-            !empty?
-          end
-
-          def each(&block)
-            return to_enum(:each) unless block
-
-            fullMessages.each(&block)
-          end
-        end
-      end
-    end
-  end
-end
+module Ecoportal
+  module API
+    class GraphQL
+      module Error
+        class ValidationErrors < Logic::BaseModel
+          include Enumerable
+
+          passthrough :details
+          passarray :fullMessages
+
+          def empty?
+            count < 1
+          end
+
+          def any?
+            !empty?
+          end
+
+          def each(&block)
+            return to_enum(:each) unless block
+
+            fullMessages.each(&block)
+          end
+        end
+      end
+    end
+  end
+end

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

@@ -1,12 +1,12 @@
-module Ecoportal
-  module API
-    class GraphQL
-      module Error
-      end
-    end
-  end
-end
-
-require_relative 'error/locations_validation_error'
-require_relative 'error/validation_errors'
-require_relative 'error/locations_error'
+module Ecoportal
+  module API
+    class GraphQL
+      module Error
+      end
+    end
+  end
+end
+
+require_relative 'error/locations_validation_error'
+require_relative 'error/validation_errors'
+require_relative 'error/locations_error'