rsmp 0.52.0 → 0.53.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cfba3255efe256eae701725584e74d80a7ba66fe94708ef9bdc76cbd678c7700
-  data.tar.gz: b0aebcd26383d1ed77fbcdd18351fecc0a7afcc962d582f17a3f488a53aaeb22
+  metadata.gz: 753bf9789b6bf177b83fb54c6fb95856b5f6ceed515a9ff17ba4993109a88d29
+  data.tar.gz: 0ab04b23c2e57490c605f26bb2962d79a88dc6a1e4b165085dedbc4dab52a834
 SHA512:
-  metadata.gz: f7796d54dbed77a9d230a4e49f3ddadd22b3833b29fa05c882f9d7a6c540ed64ae55745f5b8dbe38a9fe095fa49b45684f9ebaf0a954d9fa021d892f2bdc5cec
-  data.tar.gz: 5bf194b21059fd35d82e41093c68658585fe083cc061f5759e98d95df1f45e5fbd0958051cb3dbac8b771f915fc0511c056f72c887339f1b5df6c60f73d6a57f
+  metadata.gz: e66c0016019dff11724b0989dd3da9d319aa8a24ce8ad3b326068bd03abe73580d212c15023470378cc21690b92cb298666fde785195b1267e28a3b4fe5cbdbd
+  data.tar.gz: ccd8d17f4eef01bf389f7b17851ffc321b3d64bf2b562e06700809e693065f9b71bac39bc50fb6630a896d1e7fb1ea60a59e7fc6f0182e1cef36e9bb41582310

data/CHANGELOG.md

@@ -735,3 +735,9 @@ Initial release.
 - add `start` and `stop` aliases to queues for use with proxy collection factories
 - fix status collectors to honor `component:` and `responses:`; the misspelled `reponses:` option is no longer used
 
+## 0.53.0
+- update core 3.3.0 support to sync with latest core 3.3.0 draft
+- allow fractional status subscription intervals for all older core versions to align with their specs.
+
+## 0.53.1
+- skip repeated status value checks for buffered values (`q=old`) and unknown subscription settings

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    rsmp (0.52.0)
+    rsmp (0.53.1)
       async (~> 2.45)
       colorize (~> 1.1)
       io-endpoint (~> 0.17)

data/documentation/configuration.md

@@ -21,8 +21,11 @@ and `required_supervisor_settings.json` schemas then check required settings aft
 supervisor site inheritance are resolved:
 
 - Every configured signal plan must have a `cycle_time`.
-- Configured components must include exactly one `main` component. Sites supply a default main
-  component when omitted; supervisor site settings can inherit it from `default`.
+- Before Core 3.3.0, exactly one `main` component is required. Sites supply a default main component
+  when omitted; supervisor site settings can inherit it from `default`. From Core 3.3.0, a main
+  component is not used. With `core_version: "3.3.0"`, no default components are added, even if
+  `components` is omitted. Configurations that allow older Core versions (including an omitted
+  `core_version` or `"all"`) retain the default main component.
 - A site in `server` role must have a `port`, either explicitly or from its first supervisor endpoint.
   With `supervisors: []`, the port must be explicit.
 
@@ -36,6 +39,54 @@ Earlier Core versions retain their historical wire formats for compatibility. In
 
 Legacy SXL strings such as `1.1` and `1.2` remain supported for connections using a Core version before 3.3.0. Their three-part equivalents are used only for internal schema lookup.
 
+## Core 3.3.0
+
+Preliminary Core 3.3.0 support follows `rsmp_core` branch `3.3.0` at commit
+`1aa02faf82da4f47fee57c43c546c092420aa576`, including PR #318. These rules apply only
+when Core 3.3.0 is selected. Earlier Core versions retain their existing behavior.
+
+SXL responses use `ok`, `unsupported`, `mismatch` and `expected`. Only `ok` enables
+an SXL, at the exact offered version. `supported` is diagnostic and does not trigger
+fallback. A connection can become ready even if no SXL is accepted.
+
+Set `use_alarms: false` in the supervisor's `default` or per-site configuration to
+disable every Alarm message in both directions on a 3.3.0 connection. The default
+is true.
+
+The 3.3.0 handshake ends with ComponentList followed by a site-wide AggregatedStatus.
+Readiness follows acknowledgement of that status; either MessageAck or MessageNotAck
+completes this step. The initial status is mandatory even with `send_after_connect: false`.
+
+AggregatedStatus has no `cId`, `fP` or `fS`. All false bits mean normal operation;
+bits 2, 6 and 8 are always false in site messages. Site-wide conditions include all
+components and active alarm priorities, including SXLs rejected on a connection.
+Read received bits through `SiteProxy#site_aggregated_status` and override
+`Supervisor#site_aggregated_status_changed(site_proxy)` for notifications.
+For 3.3.0, call `request_aggregated_status` or `request_aggregated_status_and_collect`
+without a component. A supplied component argument is ignored on the wire. Legacy
+requests remain component-addressed.
+
+Use `add_component`, `remove_component` and `ComponentBase#update_metadata` to update
+the inventory. Ready 3.3.0 connections receive a complete ComponentList after a change;
+disconnected connections receive the current list in their next handshake. The
+supervisor preserves received ordering and retains unchanged component objects.
+Component names are optional on the wire but must be nonempty strings when supplied.
+The supervisor uses the component ID when a list entry omits its name, including
+when a later list removes a previously supplied name.
+Removing the last component is prohibited because the Core 3.3.0 schema requires a
+nonempty ComponentList.
+
+Subscription intervals use `uRt`, an integer or decimal
+number represented as a string, e.g. `"2"` or `"0.5"`. Negative intervals are invalid.
+Zero disables periodic updates and is valid only with `sOc: true`.
+The Core source and vendored 3.3.0 schemas include this correction to the pinned baseline.
+The isolated main-component requirement in `site_configuration.rst` is treated as
+obsolete for 3.3.0 in favor of the component model and site-wide status definitions.
+
+Validator follow-up should cover the new Version statuses and failures, both alarm
+directions, site-wide status bits and fields, final-handshake ACK/NotAck, component
+changes, and accepted-SXL filtering during initial synchronization and buffer replay.
+
 ## Loading a configuration file
 
 Use the option classes to load a YAML configuration file:

data/lib/rsmp/component/component_base.rb

@@ -41,8 +41,10 @@ module RSMP
     end
 
     def update_metadata(type: nil, name: nil)
+      changed = (type && type != @component_type) || (name && name != @name)
       @component_type = type if type
       @name = name if name
+      @node.component_list_changed if changed
     end
 
     def now
@@ -73,7 +75,14 @@ module RSMP
       raise ArgumentError, 'aggregated status must be an Array' unless status.is_a? Array
 
       input = status & AGGREGATED_STATUS_KEYS
-      return unless input != @aggregated_status
+      if @node.respond_to?(:site_wide_aggregated_status?) && @node.site_wide_aggregated_status?
+        bits = AGGREGATED_STATUS_KEYS.map { |key| input.include?(key) }
+        return if bits == @aggregated_status_bools
+
+        @aggregated_status = input
+      else
+        return unless input != @aggregated_status
+      end
 
       AGGREGATED_STATUS_KEYS.each_with_index do |key, index|
         @aggregated_status_bools[index] = status.include?(key)

data/lib/rsmp/component/component_proxy.rb

@@ -18,8 +18,8 @@ module RSMP
       end
     end
 
-    # Check that were not receiving repeated update values.
-    # The check is not performed for item with an update interval.
+    # Check live values only for known change-only subscriptions. Buffered
+    # values may have been generated under different subscription settings.
     def check_repeat_values(message, subscription_list)
       message.attribute('sS').each do |item|
         check_status_item_for_repeats(item, subscription_list)
@@ -27,6 +27,8 @@ module RSMP
     end
 
     def check_status_item_for_repeats(item, subscription_list)
+      return if item['q'] == 'old'
+
       status_code = item['sCI']
       status_name = item['n']
       return if update_rate_set?(subscription_list, status_code, status_name)
@@ -39,12 +41,13 @@ module RSMP
 
     def update_rate_set?(subscription_list, status_code, status_name)
       urt = subscription_list.dig(c_id, status_code, status_name, 'uRt')
-      urt.to_i.positive?
+      urt.to_f.positive?
     end
 
     def should_check_repeats?(subscription_list, status_code, status_name)
-      soc = subscription_list.dig(c_id, status_code, status_name, 'sOc')
-      return false if soc == false
+      subscription = subscription_list.dig(c_id, status_code, status_name)
+      return false unless subscription
+      return false if subscription['sOc'] == false
       return false if @allow_repeat_updates[status_code]&.include?(status_name)
 
       true

data/lib/rsmp/component/components.rb

@@ -13,7 +13,7 @@ module RSMP
     def setup_components(settings)
       return unless settings
 
-      check_main_component settings
+      check_main_component settings unless core_3_3_components?
       settings.each_pair do |type, components_by_type|
         next unless components_by_type
 
@@ -30,6 +30,10 @@ module RSMP
       end
     end
 
+    def core_3_3_components?
+      @site_settings && @site_settings['core_version'] == '3.3.0'
+    end
+
     def check_main_component(settings)
       raise ConfigurationError, 'main component must be defined' unless settings['main'] && settings['main'].size >= 1
       return unless settings['main'].size > 1
@@ -39,11 +43,29 @@ module RSMP
 
     def add_component(component)
       @components[component.c_id] = component
+      component_list_changed
+      component
+    end
+
+    def component_list_changed; end
+
+    def remove_component(id)
+      raise ConfigurationError, 'Cannot remove the legacy main component' if @main&.c_id == id &&
+                                                                             @site_settings['core_version'] != '3.3.0'
+
+      if core_3_3_components? && @components.size == 1 && @components.key?(id)
+        raise ConfigurationError, 'Core 3.3 ComponentList must not be empty'
+      end
+
+      removed = @components.delete(id)
+      @main = nil if removed == @main
+      component_list_changed if removed
+      removed
     end
 
     def natural_sort_key(value)
       value.to_s.split(/(\d+)/).map do |part|
-        part.match?(/\A\d+\z/) ? part.to_i : part
+        part.match?(/\A\d+\z/) ? [0, part.to_i, part.length] : [1, part]
       end
     end
 

data/lib/rsmp/node/site/site.rb

@@ -1,7 +1,10 @@
+require_relative 'status'
+
 module RSMP
   # RSMP site implementation that manages proxies and components.
   class Site < Node
     include Components
+    include SiteStatus
     include SiteConnections
 
     attr_reader :core_version, :site_settings, :logger, :proxies, :ready_condition
@@ -127,12 +130,6 @@ module RSMP
       start_status_timer(parent: parent)
     end
 
-    def aggregated_status_changed(component, _options = {})
-      @proxies.each do |proxy|
-        proxy.send_aggregated_status component
-      end
-    end
-
     def alarm_acknowledged(alarm_state)
       send_alarm AlarmAcknowledged.new(alarm_state.to_hash)
     end
@@ -143,11 +140,28 @@ module RSMP
 
     def alarm_activated_or_deactivated(alarm_state)
       send_alarm AlarmIssue.new(alarm_state.to_hash)
+      notify_site_status_change
     end
 
     def send_alarm(alarm)
+      # Snapshot before any connection encodes the message for its SXL version.
+      independent = @proxies.any? { |proxy| proxy.respond_to?(:core_3_3_0_or_newer?) && proxy.core_3_3_0_or_newer? }
+      attributes = JSON.parse(JSON.generate(alarm.attributes)) if independent
       @proxies.each do |proxy|
-        proxy.send_generated_message alarm if proxy.receive_alarms?
+        next unless proxy.use_alarms?
+
+        send_connection_alarm(proxy, alarm, attributes)
+      end
+    end
+
+    def send_connection_alarm(proxy, alarm, attributes)
+      return if proxy.core_3_3_0_or_newer? && !proxy.sxl_message_allowed?(alarm)
+
+      alarm = alarm.class.new(JSON.parse(JSON.generate(attributes)).merge('mId' => Message.make_m_id)) if attributes
+      if proxy.core_3_3_0_or_newer? && !proxy.ready?
+        proxy.buffer_message alarm
+      else
+        proxy.send_generated_message alarm
       end
     end
 

data/lib/rsmp/node/site/status.rb

@@ -0,0 +1,39 @@
+module RSMP
+  # Site-wide status and component change notifications for Core 3.3.
+  module SiteStatus
+    def aggregated_status_changed(component, _options = {})
+      @proxies.each do |proxy|
+        if proxy.core_3_3_0_or_newer?
+          proxy.send_changed_site_status
+        else
+          proxy.send_aggregated_status component
+        end
+      end
+    end
+
+    def site_wide_aggregated_status?
+      @site_settings['core_version'] == '3.3.0' || @proxies&.any?(&:core_3_3_0_or_newer?)
+    end
+
+    # Core 3.3 status includes every SXL, independently of connection negotiation.
+    def site_aggregated_status
+      bits = Array.new(8, false)
+      @components.each_value do |component|
+        [0, 2, 3, 4, 6].each { |index| bits[index] ||= component.aggregated_status_bools[index] }
+        component.alarms.each_value do |alarm|
+          bits[alarm.priority + 1] = true if alarm.active && (1..3).cover?(alarm.priority)
+        end
+      end
+      bits
+    end
+
+    def notify_site_status_change
+      @proxies.each { |proxy| proxy.send_changed_site_status if proxy.core_3_3_0_or_newer? }
+    end
+
+    def component_list_changed
+      @proxies&.each { |proxy| proxy.send_component_list if proxy.core_3_3_0_or_newer? && proxy.ready? }
+      notify_site_status_change if @proxies
+    end
+  end
+end

data/lib/rsmp/node/supervisor/modules/connection.rb

@@ -92,11 +92,28 @@ module RSMP
             site_id: site_id,
             site_settings: site_settings
           }
+        rescue HandshakeError => e
+          reject_initial_version(protocol, e)
+          raise
+        end
+
+        def reject_initial_version(protocol, error)
+          attributes = Message.parse_attributes(protocol.peek_line)
+          id = attributes['mId']
+          if id.is_a?(String) && /\A[0-9a-f]{8}-[0-9a-f]{4}-4[0-9a-f]{3}-[89ab][0-9a-f]{3}-[0-9a-f]{12}\z/i.match?(id)
+            protocol.write_lines(JSON.generate('mType' => 'rSMsg', 'type' => 'MessageNotAck',
+                                               'oMId' => id, 'rea' => error.message))
+          end
         end
 
         def retrieve_site_id(protocol)
           version_message = peek_version_message protocol
-          version_message.attribute('siteId').first['sId']
+          ids = version_message.attributes['siteId']
+          unless ids.is_a?(Array) && ids.first.is_a?(Hash) && ids.first['sId'].is_a?(String)
+            raise HandshakeError, 'Invalid siteId in Version request'
+          end
+
+          ids.first['sId']
         end
 
         def setup_proxy(proxy, settings, id)
@@ -112,10 +129,9 @@ module RSMP
           proxy
         end
 
-        def validate_and_start_proxy(proxy, protocol)
+        def validate_and_start_proxy(proxy, _protocol)
           proxy.setup_site_settings
-          proxy.check_core_version peek_version_message(protocol)
-          log "Validating using core version #{proxy.core_version}", level: :debug
+
           proxy.start
           proxy.wait
         ensure

data/lib/rsmp/node/supervisor/modules/sites.rb

@@ -68,6 +68,8 @@ module RSMP
           @site_id_condition.signal
         end
 
+        def site_aggregated_status_changed(site_proxy); end
+
         def aggregated_status_changed(site_proxy, component); end
       end
     end

data/lib/rsmp/options/schemas/required_settings.json

@@ -3,17 +3,7 @@
   "$comment": "Validate after applying defaults and inherited site settings; structural validation uses the other config schemas.",
   "type": "object",
   "properties": {
-    "components": {
-      "type": "object",
-      "required": ["main"],
-      "properties": {
-        "main": {
-          "type": "object",
-          "minProperties": 1,
-          "maxProperties": 1
-        }
-      }
-    },
+    "components": { "type": "object" },
     "signal_plans": {
       "type": "object",
       "additionalProperties": {
@@ -26,5 +16,23 @@
     "required": ["connection_role"],
     "properties": { "connection_role": { "const": "server" } }
   },
-  "then": { "required": ["port"] }
+  "then": { "required": ["port"] },
+  "allOf": [
+    {
+      "if": {
+        "required": ["core_version"],
+        "properties": { "core_version": { "const": "3.3.0" } }
+      },
+      "else": {
+        "properties": {
+          "components": {
+            "required": ["main"],
+            "properties": {
+              "main": { "type": "object", "minProperties": 1, "maxProperties": 1 }
+            }
+          }
+        }
+      }
+    }
+  ]
 }

data/lib/rsmp/options/schemas/supervisor_site.json

@@ -24,7 +24,7 @@
     "intervals": { "$ref": "config_common.json#/$defs/intervals" },
     "timeouts": { "$ref": "config_common.json#/$defs/timeouts" },
     "send_after_connect": { "type": "boolean" },
-    "receive_alarms": { "type": "boolean" },
+    "use_alarms": { "type": "boolean" },
     "message_buffer": { "$ref": "config_common.json#/$defs/message_buffer" },
     "skip_validation": { "type": "array", "items": { "type": "string" } },
     "security_codes": { "$ref": "config_common.json#/$defs/security_codes" },

data/lib/rsmp/options/site_options.rb

@@ -56,6 +56,7 @@ module RSMP
 
       def apply_defaults(options)
         defaults = defaults()
+        defaults['components'] = {} if options['core_version'] == '3.3.0'
         defaults['components']['main'] = options['components']['main'] if options.dig('components', 'main')
         data = defaults.deep_merge(options)
         port = data.dig('supervisors', 0, 'port')

data/lib/rsmp/proxy/modules/acknowledgements.rb

@@ -111,6 +111,7 @@ module RSMP
 
             @acknowledgements[original.m_id] = message
             @acknowledgement_condition.signal message
+            initial_status_acknowledged(original) if respond_to?(:initial_status_acknowledged)
           else
             log_acknowledgement_for_unknown message
           end
@@ -124,6 +125,7 @@ module RSMP
             log_acknowledgement_for_original message, original
             @acknowledgements[original.m_id] = message
             @acknowledgement_condition.signal message
+            initial_status_acknowledged(original) if respond_to?(:initial_status_acknowledged)
           else
             log_acknowledgement_for_unknown message
           end

data/lib/rsmp/proxy/modules/receive.rb

@@ -26,11 +26,12 @@ module RSMP
         def process_packet(json)
           attributes = Message.parse_attributes(json)
           message = Message.build(attributes, json)
+          message.attributes['mId'] = attributes['mId'] if message.is_a?(Version)
           process_incoming_message(message)
         rescue InvalidPacket => e
           reject_invalid_packet(json, e)
         rescue MalformedMessage => e
-          reject_malformed_packet(json, attributes, e)
+          reject_malformed_version_or_packet(json, attributes, e)
         rescue PeerMessageError => e
           reject_processed_message(message, e)
         rescue HandshakeError, FatalError => e
@@ -39,7 +40,28 @@ module RSMP
           @node&.clear_deferred
         end
 
+        def reject_malformed_version_or_packet(json, attributes, error)
+          if attributes.is_a?(Hash) && attributes['type'] == 'Version' && core_3_3_0_or_newer?
+            message = Version.new(attributes)
+            message.attributes['mId'] = attributes['mId']
+            reject_fatal_message(message, error)
+          else
+            reject_malformed_packet(json, attributes, error)
+          end
+        end
+
+        def check_core_3_3_permissions!(message)
+          if core_3_3_0_or_newer? && message.is_a?(Alarm) && !use_alarms?
+            raise MessageRejected,
+                  'Alarm exchange is disabled'
+          end
+          return unless core_3_3_0_or_newer? && @version_determined && !ready? && application_message?(message)
+
+          raise MessageRejected, 'Communication establishment is incomplete'
+        end
+
         def process_incoming_message(message)
+          check_core_3_3_permissions!(message)
           validate = should_validate_ingoing_message?(message)
           if validate
             validation = message.validate(schemas)
@@ -86,6 +108,7 @@ module RSMP
           publish_peer_failure(failure, message: message)
           log(text, message: message, level: :warning)
           dont_acknowledge(message, text, reason)
+          complete_initial_status(message) if respond_to?(:complete_initial_status)
           Result.failure(failure: failure)
         end
 
@@ -109,11 +132,18 @@ module RSMP
           text = "Rejected #{message&.type || 'message'}: #{error.message}"
           failure = peer_failure(:protocol_failure, text, message: message)
           publish_peer_failure(failure, message: message)
-          dont_acknowledge(message, "Rejected #{message.type}", error.message.to_s) if message
+          if message && (!message.is_a?(Version) || valid_received_message_id?(message))
+            dont_acknowledge(message, "Rejected #{message.type}", error.message.to_s)
+          end
           close(reason: :protocol_failure, failure: failure)
           Result.failure(failure: failure)
         end
 
+        def valid_received_message_id?(message)
+          id = message.m_id
+          id.is_a?(String) && /\A[0-9a-f]{8}-[0-9a-f]{4}-4[0-9a-f]{3}-[89ab][0-9a-f]{3}-[0-9a-f]{12}\z/i.match?(id)
+        end
+
         def peer_failure(code, text, message: nil, **context)
           Failure.new(
             code: code,

data/lib/rsmp/proxy/modules/send.rb

@@ -38,8 +38,24 @@ module RSMP
           write_protocol(message)
         end
 
+        def core_3_3_send_failure(message)
+          return unless core_3_3_0_or_newer?
+          return 'Alarm exchange is disabled' if message.is_a?(Alarm) && !use_alarms?
+
+          'Communication establishment is incomplete' if !ready? && application_message?(message)
+        end
+
+        def nts_attributes?(message)
+          message.attributes.key?('cId') || message.is_a?(Alarm) ||
+            message.is_a?(AggregatedStatus) || message.is_a?(AggregatedStatusRequest)
+        end
+
         def prepare_message(message, validate:)
+          permission = core_3_3_send_failure(message)
+          return invalid_outbound_result(message, permission) if permission
+
           message.direction = :out
+          apply_nts_message_attributes(message) if core_3_3_0_or_newer? && nts_attributes?(message)
           message.encode_for(schemas) unless validate == false
           message.generate_json
           unless validate == false
@@ -135,10 +151,14 @@ module RSMP
         end
 
         def apply_nts_message_attributes(message)
-          return if core_3_3?
+          if core_3_3_0_or_newer?
+            message.attributes['ntsOId'] = ''
+            message.attributes['xNId'] = ''
+            return
+          end
 
-          message.attributes['ntsOId'] = main && main.ntsoid ? main.ntsoid : ''
-          message.attributes['xNId'] = main && main.xnid ? main.xnid : ''
+          message.attributes['ntsOId'] = main&.ntsoid || ''
+          message.attributes['xNId'] = main&.xnid || ''
         end
       end
     end

data/lib/rsmp/proxy/modules/version_validation.rb

@@ -0,0 +1,76 @@
+module RSMP
+  class Proxy
+    module Modules
+      # Shape and cross-message constraints for version negotiation.
+      module VersionValidation
+        def validate_version_array!(message)
+          items = message.attributes['RSMP']
+          valid = items.is_a?(Array) && !items.empty? && items.all? do |item|
+            item.is_a?(Hash) && item['vers'].is_a?(String) &&
+              (RSMP::Schema.strict_version?(item['vers']) || RSMP::Schema.normalize_core_version(item['vers']))
+          end
+          raise HandshakeError, 'Invalid RSMP version array' unless valid
+        end
+
+        def validate_version!(message, step:)
+          validation = message.validate(core: core_version)
+          raise HandshakeError, validation.message if validation.invalid?
+          return unless core_3_3_0_or_newer?
+
+          raise HandshakeError, "Expected Version #{step}" unless message.step == step
+
+          validate_sxl_names!(message.sxls)
+          validate_primary_sxl!(message) if step == 'Request'
+        end
+
+        def validate_sxl_names!(items)
+          names = items.map { |item| item['name'] }
+          raise HandshakeError, 'Duplicate SXL names' unless names.uniq == names
+
+          items.each do |item|
+            version = item['version']
+            if version && !RSMP::Schema.strict_version?(version)
+              raise HandshakeError, 'SXL version must use MAJOR.MINOR.PATCH'
+            end
+          end
+        end
+
+        def validate_primary_sxl!(message)
+          primary = message.attributes['SXL']
+          return unless primary && !primary.empty? && message.sxls.none? { |item| item['version'] == primary }
+
+          raise HandshakeError, 'Primary SXL version does not match SXLS'
+        end
+
+        def validate_sxl_response!(sxls)
+          offered = sxl_request_items
+          names = offered.map { |item| item['name'] }
+          returned = sxls.reject { |item| item['status'] == 'expected' }.map { |item| item['name'] }
+          raise HandshakeError, 'SXLS must report every offered SXL exactly once' unless returned.sort == names.sort
+
+          sxls.each { |item| validate_sxl_status!(item, names) }
+        end
+
+        def validate_sxl_status!(item, names)
+          case item['status']
+          when 'ok'
+            raise HandshakeError, 'SXL version must match the request exactly' unless valid_sxl_response?(item)
+          when 'expected'
+            raise HandshakeError, 'Expected SXL was offered' if names.include?(item['name'])
+          when 'mismatch'
+            supported = item['supported']
+            unless supported == supported.sort_by { |version| Gem::Version.new(version) }
+              raise HandshakeError, 'Supported SXL versions must be sorted'
+            end
+          end
+        end
+
+        def valid_sxl_response?(accepted)
+          requested = sxl_request_items.find { |item| item['name'] == accepted['name'] }
+          RSMP::Schema.strict_version?(accepted['version']) && requested &&
+            requested['version'] == accepted['version']
+        end
+      end
+    end
+  end
+end