stepmod-utils 0.4.13 → 0.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 58a97c3c7381351ce5d0ff391e65f6b5d62ce52f541d51975b53c6e4793b5ae0
-  data.tar.gz: dd7d17312e634aaab93688e8f3fdee84e1895c025d74dd0a505fd357ea336756
+  metadata.gz: 18bfb358046e62d1504ff514004e1c01213a5cd3a86c53985dabcf5ede14e56c
+  data.tar.gz: 37bf06f7198d4066bbabba3038ab4107dd4ac16dc65add33d0cd267b26fd2bdd
 SHA512:
-  metadata.gz: 896b518262254a61c6e0e6a5df7fe5fd992eef5f9a1fb1f9136ee0753a6dc649c9c1ba07fe192c939cfa51b545a4749ffa2610aa9dc4c609b2f7e9385f917437
-  data.tar.gz: 54f31dabc624b11cfe48dde83bd0108c07692c4d3d55ee49a7f70ab518cae6914a9013e99f232d1b0c95e2798627904b3f895dad0ff8b47a48d0cd28ed117f21
+  metadata.gz: 767a9ba90526a414738cbc5bafe61b0e80da0f5b7ed9a714aa10442f87ca200303bd565ad718b0e5c2f5255311d174400fa00be6932f2b291c0b1ee55ed597b8
+  data.tar.gz: a73c95f1f931e1023f44bf308888a38cb1f1f7bc570e7fe910131e50e843b6c2d0384bb863f33fe35456ffdd39512ca85bb0e85f13cb62c7b663dc7178d78e39

data/.github/workflows/release.yml

@@ -7,11 +7,12 @@ on:
     inputs:
       next_version:
         description: |
-          Next release version. Possible values: x.y.z, major, minor, patch or pre|rc|etc
+          Next release version. Possible values: x.y.z, major, minor, patch (or pre|rc|etc).
+          Also, you can pass 'skip' to skip 'git tag' and do 'gem push' for the current version
         required: true
         default: 'skip'
-  push:
-    tags: [ v* ]
+  repository_dispatch:
+    types: [ do-release ]
 
 jobs:
   release:

data/exe/stepmod-annotate-all

@@ -58,17 +58,20 @@ stepmod_dir = options[:stepmod_dir]
 raise StandardError.new("--stepmod-dir not set") unless stepmod_dir
 
 stepmod_path = Pathname.new(stepmod_dir)
-unless (stepmod_path.join("data").exist? && stepmod_path.join("data").directory?)
+unless stepmod_path.join("data").exist? && stepmod_path.join("data").directory?
   raise StandardError.new("--stepmod-dir should be the root STEPmod directory and contains data/")
 end
+
 log "STEPmod data path: `#{stepmod_dir}`"
 
 srl_output_schemas_dir = options[:srl_output_schemas_dir]
 raise StandardError.new("--schemas directory for SRL schemas not set") unless srl_output_schemas_dir
+
 log "SRL schemas output path: `#{srl_output_schemas_dir}`"
 
 srl_output_docs_dir = options[:srl_output_docs_dir]
 raise StandardError.new("--documents directory for SRL documents (generated by stepmod2mn) not set") unless srl_output_docs_dir
+
 log "SRL documents path: `#{srl_output_docs_dir}`"
 
 unless File.exist?(srl_output_docs_dir) && File.directory?(srl_output_docs_dir)
@@ -126,7 +129,7 @@ def file_system_path(filepath, filename_cache)
   filename_cache[dir][filepath.downcase]
 end
 
-MAX_THREADS = 1 #[2, Concurrent.processor_count].max * 2
+MAX_THREADS = 1 # [2, Concurrent.processor_count].max * 2
 MAX_QUEUE_SIZE = MAX_THREADS * 4
 # https://github.com/ruby-concurrency/concurrent-ruby/blob/master/docs-source/thread_pools.md
 # pool = Concurrent::ThreadPoolExecutor.new(
@@ -143,7 +146,6 @@ FileUtils.mkdir_p(srl_output_schemas_dir)
 srl_output_schemas_path = Pathname.new(srl_output_schemas_dir)
 srl_output_docs_path = Pathname.new(srl_output_docs_dir)
 
-
 files.each do |file|
   puts "#{Thread.current.object_id}: `#{file}` processing..."
 
@@ -153,7 +155,6 @@ files.each do |file|
   ).call
 
   destination_rel_path = Pathname.new(file).dirname.relative_path_from(stepmod_path.join("data"))
-  directory_type = destination_rel_path.each_filename.to_a.first
 
   destination_path = srl_output_schemas_path.join(destination_rel_path)
   FileUtils.mkdir_p(destination_path)

data/exe/stepmod-convert-express-description

@@ -17,10 +17,8 @@ input_content = ARGF.read
 
 result = Stepmod::Utils::SmrlDescriptionConverter.convert(input_content, {})
 result.split("\n").each do |line|
-  begin
-    $stdout.puts(line)
-    $stdout.flush
-  rescue Errno::EPIPE
-    exit(74)
-  end
+  $stdout.puts(line)
+  $stdout.flush
+rescue Errno::EPIPE
+  exit(74)
 end

data/exe/stepmod-convert-express-resource

@@ -17,10 +17,8 @@ input_content = ARGF.read
 
 result = Stepmod::Utils::SmrlResourceConverter.convert(input_content, {})
 result.split("\n").each do |line|
-  begin
-    $stdout.puts(line)
-    $stdout.flush
-  rescue Errno::EPIPE
-    exit(74)
-  end
+  $stdout.puts(line)
+  $stdout.flush
+rescue Errno::EPIPE
+  exit(74)
 end

data/exe/stepmod-extract-concepts

@@ -77,7 +77,8 @@ else
   )
 end
 
-default_output_dir = File.join(File.expand_path("..", stepmod_dir), "output_yaml")
+default_output_dir = File.join(File.expand_path("..", stepmod_dir),
+                               "output_yaml")
 output_dir = options[:output_dir] || default_output_dir
 unless File.directory?(output_dir)
   FileUtils.mkdir_p(output_dir)
@@ -164,7 +165,7 @@ end
 yaml_outputs = {
   clause_4_sections: {},
   clause_5_sections: {},
-  bibliographies: {}
+  bibliographies: {},
 }
 
 part_resources.each do |(_bibdata, current_part_resources)|
@@ -204,9 +205,9 @@ extract_bibliographies(resource_concepts, yaml_outputs[:bibliographies])
 {
   clause_4_sections: "resource_sections.yaml",
   clause_5_sections: "module_sections.yaml",
-  bibliographies: "bib.yaml"
+  bibliographies: "bib.yaml",
 }.each_pair do |var, filename|
   path = File.join(output_dir, filename)
   File.write(path, yaml_outputs[var].values.to_yaml)
-  log "INFO: #{var.to_s} written to #{path}."
+  log "INFO: #{var} written to #{path}."
 end

data/exe/stepmod-srl-migrate

@@ -4,6 +4,7 @@
 
 require "pathname"
 require "fileutils"
+require "nokogiri"
 bin_file = Pathname.new(__FILE__).realpath
 
 # add self to libpath
@@ -14,14 +15,14 @@ require "optparse"
 
 def log(message, indent = 0)
   indent_spaces = " " * (indent * 2)
-  puts "[stepmod-utils] #{message}"
+  puts "[stepmod-utils] #{indent_spaces}#{message}"
 end
 
-def copy_files_to_schemas(path, stepmod_path, srl_output_docs_dir)
+def copy_files_to_schemas(path, stepmod_path, srl_output_schemas_dir)
   base_path = path.dirname.relative_path_from(File.join(stepmod_path, "data"))
   file_name = path.basename
 
-  new_dir = File.join(srl_output_docs_dir, base_path.to_s)
+  new_dir = File.join(srl_output_schemas_dir, base_path.to_s)
   new_file_path = File.join(new_dir, file_name)
 
   unless Dir.exist?(new_dir)
@@ -32,6 +33,28 @@ def copy_files_to_schemas(path, stepmod_path, srl_output_docs_dir)
   log("Copied #{path.basename} to #{new_file_path}", 1)
 end
 
+def copy_files_to_document(path, stepmod_path, srl_output_docs_dir)
+  file_name = path.basename
+  module_xml_path = path.dirname.join("module.xml")
+  # puts "module_xml_path #{module_xml_path}"
+  module_xml = Nokogiri::XML(IO.read(module_xml_path)).root
+
+  part_number = module_xml.xpath("//module").first.attr("part")
+  # module_name = module_xml.xpath('//module').first.attr('name')
+
+  new_dir = File.join(srl_output_docs_dir, "iso-10303-#{part_number}")
+  new_file_path = File.join(new_dir, file_name)
+  # new_file_path = File.join(new_dir, "#{module_name}.#{file_name}")
+
+  ## The document directory should already exist.
+  # unless Dir.exist?(new_dir)
+  #   FileUtils.mkdir_p(new_dir)
+  # end
+
+  FileUtils.copy_file(path, new_file_path)
+  log("Copied #{path.basename} to #{new_file_path}", 1)
+end
+
 options = {}
 OptionParser.new do |opts|
   opts.banner = "Usage: #{$0} [options]"
@@ -75,19 +98,19 @@ OptionParser.new do |opts|
 end.parse!
 
 def download_stepmod2mn(path)
-  if path && path.exist?
+  if path&.exist?
     puts "Specified `stepmod2mn.jar` found, skipping download."
     return path
   end
 
   puts "Specified `stepmod2mn.jar` not found, downloading from GitHub."
 
-  require 'octokit'
+  require "octokit"
   require "down"
   require "fileutils"
 
   client = Octokit::Client.new
-  download_url = client.latest_release('metanorma/stepmod2mn').assets.first.browser_download_url
+  download_url = client.latest_release("metanorma/stepmod2mn").assets.first.browser_download_url
 
   filename = Pathname.new(download_url).basename.to_s
 
@@ -123,7 +146,7 @@ end
 
 stepmod_path = Pathname.new(stepmod_dir)
 unless stepmod_path.exist? && stepmod_path.directory? &&
-  stepmod_path.join("data").exist? && stepmod_path.join("data").directory?
+    stepmod_path.join("data").exist? && stepmod_path.join("data").directory?
   puts "Specified STEPmod directory not found, not a directory, or does not contain `data/`. Aborting."
   exit 1
 end
@@ -143,42 +166,51 @@ unless File.directory?(srl_output_docs_dir)
 end
 log "SRL output documents directory path: `#{srl_output_docs_dir}`"
 
-log("*"*30)
+log("*" * 30)
 log("[SRL MIGRATION] Migration starts!")
 
-log("*"*30)
+log("*" * 30)
 log("[SRL MIGRATION: stepmod2mn.jar] START SVG generation.")
-log("*"*30)
+log("*" * 30)
 system "java -Xss5m -jar #{stepmod2mn_path} #{stepmod_dir} --output-documents #{srl_output_docs_dir} --output-schemas #{srl_output_schemas_dir} --svg"
 
-log("*"*30)
+log("*" * 30)
 log("[SRL MIGRATION: stepmod2mn.jar] COMPLETE SVG generation.")
 
-log("*"*30)
+log("*" * 30)
 log("[SRL MIGRATION: stepmod2mn.jar] START document migration and SVG generation.")
-log("*"*30)
+log("*" * 30)
 system "java -Xss5m -jar #{stepmod2mn_path} #{stepmod_dir} --output-documents #{srl_output_docs_dir} --output-schemas #{srl_output_schemas_dir}"
-log("*"*30)
+log("*" * 30)
 log("[SRL MIGRATION: stepmod2mn.jar] COMPLETE document migration and SVG generation.")
 
-log("*"*30)
+log("*" * 30)
 log("[SRL MIGRATION: stepmod-utils] START Annotated EXPRESS generation.")
-log("*"*30)
+log("*" * 30)
 system "stepmod-annotate-all --stepmod-dir #{stepmod_dir} --documents #{srl_output_docs_dir} --schemas #{srl_output_schemas_dir}"
-log("*"*30)
+log("*" * 30)
 log("[SRL MIGRATION: stepmod-utils] COMPLETE Annotated EXPRESS generation.")
 
-log("*"*30)
+log("*" * 30)
 log("[SRL MIGRATION: stepmod-utils] START EXPRESS change YAML extraction.")
 system "stepmod-extract-changes -p #{stepmod_dir}"
+
+# Move arm/mim/arm_lf/mim_lf/mapping changes into schema directories
 Dir.glob("#{stepmod_dir}/data/**/*.changes.yaml").each do |filepath|
   path = Pathname.new(filepath)
   copy_files_to_schemas(path, stepmod_dir, srl_output_schemas_dir)
 end
-log("*"*30)
+
+# Move document changes into document directories
+Dir.glob("#{stepmod_dir}/data/**/changes.yaml").each do |filepath|
+  path = Pathname.new(filepath)
+  copy_files_to_document(path, stepmod_dir, srl_output_docs_dir)
+end
+
+log("*" * 30)
 log("[SRL MIGRATION: stepmod-utils] COMPLETE EXPRESS change YAML extraction.")
 
-log("*"*30)
+log("*" * 30)
 log("[SRL MIGRATION] Migration complete!")
 
-log("*"*30)
+log("*" * 30)

data/exe/stepmod-validate-express

@@ -1,9 +1,9 @@
 #!/usr/bin/env ruby
 
-require 'nokogiri'
-require 'expressir'
-require 'expressir/express/parser'
-require 'expressir/express/formatter'
+require "nokogiri"
+require "expressir"
+require "expressir/express/parser"
+require "expressir/express/formatter"
 
 stepmod_dir = ARGV.pop || "iso-10303-stepmod-wg12"
 stepmod_path = Pathname.new(stepmod_dir)
@@ -38,7 +38,7 @@ end
 
 def parse_express_schema(path)
   Expressir::Express::Parser.from_file(path)
-rescue
+rescue StandardError
   puts "ERROR: Broken schema detected in #{path}!"
   nil
 end
@@ -54,7 +54,7 @@ puts REPO_INDEX_XML
 puts
 puts "NOTE: SHOULD_REWRITE mode is turned #{SHOULD_REWRITE ? 'ON' : 'OFF'}"
 
-all_express_files(stepmod_dir).each_with_index do |exp_file, i|
+all_express_files(stepmod_dir).each_with_index do |exp_file, _i|
   # puts "i #{i}"
   # next if i > 10
 
@@ -62,7 +62,6 @@ all_express_files(stepmod_dir).each_with_index do |exp_file, i|
   exp_path = Pathname.new(exp_file)
   repo = parse_express_schema(exp_file)
 
-
   schemas_status[exp_path.to_s] = repo ? true : false
 
   if repo.nil?
@@ -77,28 +76,27 @@ all_express_files(stepmod_dir).each_with_index do |exp_file, i|
     formatted_schema = format_express_schema(repo)
     # experimental_path = exp_path.dirname.join(exp_path.basename('.exp').to_s+".x.exp")
 
-    puts "rewriting : #{exp_path.to_s}"
+    puts "rewriting : #{exp_path}"
     exp_path.truncate(0)
     exp_path.write(formatted_schema)
   end
-
 end
 
-puts "*"*30
+puts "*" * 30
 
-if schemas_status.values.include?(false)
+if schemas_status.value?(false)
   puts "The following schemas failed validation:"
-  schemas_status.select do |path, status|
+  schemas_status.select do |_path, status|
     status == false
-  end.each_pair do |path, status|
+  end.each_pair do |path, _status|
     puts "SCHEMA: #{path}"
   end
-  puts "*"*30
+  puts "*" * 30
 
   exit 1
 end
 
 puts "All schemas passed validation."
-puts "*"*30
+puts "*" * 30
 
 exit 0

data/lib/stepmod/utils/change.rb

@@ -5,7 +5,7 @@ require "stepmod/utils/change_edition_collection"
 module Stepmod
   module Utils
     class Change
-      attr_accessor :schema_name
+      attr_accessor :schema_name, :mapping_table
       attr_reader :change_editions
 
       MODULE_TYPES = {
@@ -13,11 +13,20 @@ module Stepmod
         mim: "mim",
         arm_longform: "arm_lf",
         mim_longform: "mim_lf",
+        mapping: "mapping",
+        changes: "changes",
+        mapping_table: "mapping",
       }.freeze
 
+      TYPES_WITHOUT_EXTENSION = %w[
+        changes
+        mapping_table
+      ].freeze
+
       def initialize(stepmod_dir:, schema_name:, type:)
         @stepmod_dir = stepmod_dir
         @change_editions = Stepmod::Utils::ChangeEditionCollection.new
+        @mapping_table = {}
         @schema_name = schema_name
         @type = type
       end
@@ -40,13 +49,24 @@ module Stepmod
       alias_method :[], :fetch_change_edition
 
       def save_to_file
-        File.write(filepath(@type), Psych.dump(to_h))
+        change_hash = if @type == "mapping_table"
+                        @mapping_table
+                      else
+                        to_h
+                      end
+        return if change_hash.empty?
+
+        File.write(filepath(@type), Psych.dump(stringify_keys(change_hash)))
       end
 
       def to_h
+        change_editions_list = change_editions.to_h
+
+        return {} if change_editions_list.empty?
+
         {
           "schema" => schema_name,
-          "change_edition" => change_editions.to_h,
+          "change_edition" => change_editions_list,
         }
       end
 
@@ -58,10 +78,16 @@ module Stepmod
           "data",
           base_folder,
           schema_name,
-          "#{MODULE_TYPES[type.to_sym] || schema_name}.changes.yaml",
+          filename(type),
         )
       end
 
+      def filename(type)
+        return "#{MODULE_TYPES[type.to_sym]}.yaml" if TYPES_WITHOUT_EXTENSION.include?(type.to_s)
+
+        "#{MODULE_TYPES[type.to_sym] || schema_name}.changes.yaml"
+      end
+
       def base_folder
         if resource?
           "resources"
@@ -69,6 +95,23 @@ module Stepmod
           "modules"
         end
       end
+
+      # Hash#transform_keys is not available in Ruby 2.4
+      # so we have to do this ourselves :(
+      # symbolize hash keys
+      def stringify_keys(hash)
+        result = {}
+        hash.each_pair do |key, value|
+          result[key.to_s] = if value.is_a?(Hash)
+                               stringify_keys(value)
+                             elsif value.is_a?(Array)
+                               value.map { |v| stringify_keys(v) }
+                             else
+                               value
+                             end
+        end
+        result
+      end
     end
   end
 end

data/lib/stepmod/utils/change_collection.rb

@@ -24,7 +24,7 @@ module Stepmod
       end
 
       def save_to_files
-        @changes.values.each(&:save_to_file)
+        @changes.each_value(&:save_to_file)
       end
 
       def count

data/lib/stepmod/utils/change_edition.rb

@@ -2,7 +2,7 @@ module Stepmod
   module Utils
     class ChangeEdition
       attr_accessor :version, :description
-      attr_reader :additions, :modifications, :deletions, :mapping
+      attr_reader :additions, :modifications, :deletions, :changes
 
       def initialize(options)
         @version = options[:version]
@@ -10,7 +10,7 @@ module Stepmod
         self.additions = options[:additions] || []
         self.modifications = options[:modifications] || []
         self.deletions = options[:deletions] || []
-        self.mapping = options[:mapping] || []
+        self.changes = options[:changes] || []
       end
 
       def additions=(additions)
@@ -31,10 +31,10 @@ module Stepmod
         @deletions = deletions
       end
 
-      def mapping=(mapping)
-        validate_type("mapping", mapping, Array)
+      def changes=(changes)
+        validate_type("changes", changes, Array)
 
-        @mapping = mapping
+        @changes = changes
       end
 
       def to_h
@@ -44,7 +44,7 @@ module Stepmod
           "additions" => additions,
           "modifications" => modifications,
           "deletions" => deletions,
-          "mapping" => mapping,
+          "changes" => changes,
         }.reject { |_k, v| v.nil? || v.empty? }
       end
 

data/lib/stepmod/utils/changes_extractor.rb

@@ -1,6 +1,7 @@
 require "nokogiri"
 require "stepmod/utils/change_collection"
 require "stepmod/utils/smrl_description_converter"
+require_relative "parsers"
 
 module Stepmod
   module Utils
@@ -30,7 +31,11 @@ module Stepmod
         all_modules_changes_files.each do |module_change_file|
           xml_changes = Nokogiri::XML(File.read(module_change_file)).root
           schema_name = Pathname.new(module_change_file).parent.basename.to_s
-          add_module_changes_to_collection(xml_changes, @collection, schema_name)
+          add_module_changes_to_collection(xml_changes, @collection,
+                                           schema_name)
+
+          add_mapping_table_changes_to_collection(xml_changes, schema_name,
+                                                  @collection)
         end
 
         @collection
@@ -38,6 +43,17 @@ module Stepmod
 
       private
 
+      def add_mapping_table_changes_to_collection(xml_data, schema_name,
+collection)
+        mapping_table = xml_data.at_xpath("//mapping_table")
+        return unless mapping_table
+
+        shale_object = Stepmod::Utils::Parsers::MappingTableParser.parse(mapping_table)
+
+        change = collection.fetch_or_initialize(schema_name, "mapping_table")
+        change.mapping_table = shale_object.to_hash
+      end
+
       # rubocop:disable Metrics/MethodLength
       def add_resource_changes_to_collection(xml_data, collection)
         xml_data.xpath("//changes").each do |changes_node|
@@ -64,9 +80,12 @@ module Stepmod
             options = {
               schema_name: schema_name,
               version: change_node.attr("version"),
-              description: converted_description(change_node.xpath("description").first),
+              # description: converted_description(change_node.xpath("description").first),
             }
 
+            add_mapping_changes(collection, change_node, options)
+            add_changes(collection, change_node, options)
+
             MODULE_TYPES.each do |type|
               add_module_changes(
                 collection,
@@ -97,21 +116,34 @@ module Stepmod
         schema_name = options[:schema_name]
         change = collection.fetch_or_initialize(schema_name, type)
 
+        return if changes.nil?
+
         change_edition = extract_change_edition(changes, options)
         change.add_change_edition(change_edition)
-
-        if type == "arm"
-          add_mapping_changes(collection, change_node, options)
-        end
       end
 
       def add_mapping_changes(collection, change_node, options)
+        mapping_changes = extract_mapping_changes(change_node)
+        return if mapping_changes.empty?
+
         change_edition = collection
-                         .fetch_or_initialize(options[:schema_name], "arm")
+                         .fetch_or_initialize(options[:schema_name], "mapping")
                          .change_editions
                          .fetch_or_initialize(options[:version])
 
-        change_edition.mapping = extract_mapping_changes(change_node)
+        change_edition.changes = mapping_changes
+      end
+
+      def add_changes(collection, change_node, options)
+        description = converted_description(change_node.xpath("description").first)
+        return if description.nil? || description.empty?
+
+        change_edition = collection
+                         .fetch_or_initialize(options[:schema_name], "changes")
+                         .change_editions
+                         .fetch_or_initialize(options[:version])
+
+        change_edition.description = description
       end
 
       def extract_mapping_changes(change_node)
@@ -119,17 +151,21 @@ module Stepmod
 
         change_node.xpath("mapping.changes").each do |changes|
           changes.xpath("mapping.change").each do |change|
-            mappings << { "change" => change.text }
+            mappings << { "change" => clean_string(change.text.strip) }
           end
 
           changes.xpath("description").each do |change|
-            mappings << { "description" => change.text }
+            mappings << { "description" => clean_string(change.text.strip) }
           end
         end
 
         mappings
       end
 
+      def clean_string(string)
+        string.gsub(/[\n\t]/, " ").squeeze(" ")
+      end
+
       def extract_change_edition(schema_changes, options)
         type = options[:type] || "schema"
         node_type = type.gsub("_longform", "")

data/lib/stepmod/utils/concept.rb

@@ -17,7 +17,8 @@ module Stepmod
       # TODO: converted_definition is not supposed to be an attribute, it is
       # supposed to be a method!
       class << self
-        def parse(definition_xml, reference_anchor:, reference_clause:, file_path:, language_code: "eng")
+        def parse(definition_xml, reference_anchor:, reference_clause:,
+file_path:, language_code: "eng")
           converted_definition = Stepmod::Utils::StepmodDefinitionConverter.convert(
             definition_xml,
             {
@@ -64,14 +65,15 @@ module Stepmod
             end
 
           term = Stepmod::Utils::Converters::Term
-            .new
-            .convert(
-              definition_xml.xpath(".//term").first
-            )
+                 .new
+                 .convert(
+                   definition_xml.xpath(".//term").first,
+                 )
 
           # [4..-1] because we want to skip the initial `=== {title}`
           designations = [
-            { "designation" => term[4..-1], "type" => "expression", "normative_status" => "preferred" },
+            { "designation" => term[4..-1], "type" => "expression",
+              "normative_status" => "preferred" },
           ]
 
           alts.each do |alt|
@@ -84,10 +86,10 @@ module Stepmod
         def definition_xml_definition(definition_xml, reference_anchor)
           # We reject the <p> that was considered an alternative term (length<=20)
           text_nodes = definition_xml
-            .xpath(".//def")
-            .first
-            .children
-            .reject { |n| n.name == "p" && n.text.length <= 20 }
+                       .xpath(".//def")
+                       .first
+                       .children
+                       .reject { |n| n.name == "p" && n.text.length <= 20 }
 
           wrapper = "<def>#{text_nodes.map(&:to_s).join}</def>"