yaml-sort 1.0.0 → 1.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b17a9fdbfc9612b6b2bb1a75583e9a31c269fbba87c088afc0fc6ffe16e44aa1
-  data.tar.gz: edde89f395f92d321c95e4899c6744bba2b005b93aa4399c00d090c4511a27ac
+  metadata.gz: 232d936095f9aac0e3f4098fa2cc4f1d608ddc79be9cb4e757fb476fa75999e3
+  data.tar.gz: be5949810ea40472c5d80e7f23d96886e964847b26d9526f67b337a77acfe490
 SHA512:
-  metadata.gz: a3a763b77216fad635aec8707d388d1bc1c8afb1ccd2217922b17c6179bc59d6c1998459c3e131375d1fed6e743d94d2d39dc2f080bd4adf45b42a73aff56b94
-  data.tar.gz: 11573f243771dcd0ec7c52748a9927c358472860e0a52e287f8764c960e539aa76b6df6ee6f2fff724eb9f5e780c2b62d7921b65b1d0f9aaf696d388e618b6f4
+  metadata.gz: e90a5d3ff86076d86b0629ec1a387aa179588f678d17c5f60fcae2d01ffda38eb28f308d5d3d7df1a29d6b1a194701caab02409d26d364f8c2c8fb7403ca2a0a
+  data.tar.gz: e2fb3054b83242a9fad8c596c6f79ecfce1f66169234c7e1bc8a179214bd41288e08eb608981b08d2312dff721e3bfa18aadaa429dded7bae1eeb702dc06bdd3

data/CHANGELOG.md

@@ -1,5 +1,18 @@
-## [Unreleased]
+# Changelog
+All notable changes to this project will be documented in this file.
+The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
+and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
-## [0.1.0] - 2022-04-09
+## [1.0.1](https://github.com/smortex/yaml-sort/tree/1.0.1) (2022-04-24)
 
-- Initial release
+[Full Changelog](https://github.com/smortex/yaml-sort/compare/v1.0.0...1.0.1)
+
+**Fixed bugs:**
+
+- Fix processing of some YAML files without a `START_OF_DOCUMENT` [\#6](https://github.com/smortex/yaml-sort/pull/6) ([smortex](https://github.com/smortex))
+- Fix error reporting [\#5](https://github.com/smortex/yaml-sort/pull/5) ([smortex](https://github.com/smortex))
+- Fix display of filename on parse errors [\#1](https://github.com/smortex/yaml-sort/pull/1) ([smortex](https://github.com/smortex))
+
+
+
+\* *This Changelog was automatically generated by [github_changelog_generator](https://github.com/github-changelog-generator/github-changelog-generator)*

data/Rakefile

@@ -21,3 +21,19 @@ end
 
 task spec: ["lib/yaml/sort/parser.rb"]
 task cucumber: ["lib/yaml/sort/parser.rb"]
+
+require "github_changelog_generator/task"
+GitHubChangelogGenerator::RakeTask.new :changelog do |config|
+  config.future_release = Yaml::Sort::VERSION
+  config.header = <<~HEADER.chomp
+    # Changelog
+    All notable changes to this project will be documented in this file.
+    The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
+    and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
+  HEADER
+  config.exclude_labels = %w[duplicate question invalid wontfix wont-fix skip-changelog ignore]
+  config.user = "smortex"
+  config.project = "yaml-sort"
+  config.since_tag = "v1.0.0"
+  config.issues = false
+end

data/lib/yaml/sort/cli.rb

@@ -52,8 +52,11 @@ module Yaml
 
       def process_document(filename, options)
         yaml = read_document(filename)
-        sorted_yaml = sort_yaml(yaml)
+        sorted_yaml = sort_yaml(yaml, filename)
         write_output(yaml, sorted_yaml, filename, options)
+      rescue Racc::ParseError => e
+        warn(e.message)
+        @exit_code = 1
       end
 
       def read_document(filename)
@@ -64,8 +67,8 @@ module Yaml
         end
       end
 
-      def sort_yaml(yaml)
-        document = @parser.parse(yaml)
+      def sort_yaml(yaml, filename)
+        document = @parser.parse(yaml, filename: filename)
         document = document.sort
         "---\n#{document}\n"
       end

data/lib/yaml/sort/parser.rb

@@ -13,9 +13,11 @@ module Yaml
   module Sort
     class Parser < Racc::Parser
 
-module_eval(<<'...end parser.ra/module_eval...', 'parser.ra', 34)
+module_eval(<<'...end parser.ra/module_eval...', 'parser.ra', 33)
 
 def scan(text)
+  text = "---\n#{text}" unless text.start_with?("---\n")
+
   scan_value = false
 
   @lines = text.lines
@@ -29,7 +31,7 @@ def scan(text)
   until s.eos?
     if scan_value
       unless s.match?(/[[:space:]]*\n/)
-        @position += s.matched_size if s.scan(/\s*/)
+        @position += s.matched_size if s.scan(/[[:blank:]]*/)
         case
         when s.scan(/[|>][-+]?(?=\n)/)
           match = s.matched
@@ -68,20 +70,23 @@ def scan(text)
       scan_value = false
     else
       case
-      when s.scan(/---/)              then emit(:START_OF_DOCUMENT, s.matched)
-      when s.scan(/(\n\s*)+(?=\n)/)   then # Ignore empty lines
-      when s.scan(/\n?\s*#.*/)        then emit(:COMMENT, s.matched)
-      when s.scan(/\n?(\s*-) */)      then emit("-", s.matched, indent: s.captures[0])
-      when s.scan(/\n?\s*\.\.\./)     then emit(:END_OF_DOCUMENT, s.matched)
-      when s.scan(/\n?(\s*)([^[[:space:]]\n]+):(?=[ \n])/)
+      when s.scan(/---/)                    then emit(:START_OF_DOCUMENT, s.matched)
+      when s.scan(/\n[[:blank:]]*(?=\n)/)
+        # Ignore empty lines
+        @lineno += 1
+        @position = 0
+      when s.scan(/\n[[:blank:]]*#.*/)      then emit(:COMMENT, s.matched)
+      when s.scan(/\n([[:blank:]]*-) */)    then emit(:ITEM, s.matched, indent: s.captures[0])
+      when s.scan(/\n[[:blank:]]*\.\.\./)   then emit(:END_OF_DOCUMENT, s.matched)
+      when s.scan(/\n?([[:blank:]]*)([^[[:space:]]\n]+):(?=[ \n])/)
         indent = s.captures[0]
         indent = last_indent_value + indent + " " unless s.matched.start_with?("\n")
         emit(:KEY, s.matched, indent: indent)
 
       when s.scan(/\n\z/)
         # Done
-     when s.match?(/./)
-       scan_value = true
+      when s.match?(/./)
+        scan_value = true
       else
         raise "LoST: #{s.rest.inspect} #{s.eos?}"
       end
@@ -128,7 +133,6 @@ def emit(token, value, length: nil, indent: nil)
     value: value,
     lineno: @lineno,
     position: @position,
-    filename: @filename,
     length: length,
     indent: indent,
   }
@@ -146,7 +150,7 @@ end
 def unindent(new_indent = nil)
   while @indent_stack.count > 0 && (new_indent.nil? || @indent_stack.last.length > new_indent.length)
     value = @indent_stack.pop
-    @tokens << [:UNINDENT, { value: value, lineno: @lineno, position: 0, filename: @filename, length: value.length, indent: nil }]
+    @tokens << [:UNINDENT, { value: value, lineno: @lineno, position: 0, length: value.length, indent: nil }]
   end
 end
 
@@ -154,73 +158,73 @@ def next_token
   @current_token = @tokens.shift
 end
 
-def parse(text)
+def parse(text, filename: nil)
+  @filename = filename || "<stdin>"
   scan(text)
   do_parse
 end
 
 def on_error(error_token_id, error_value, value_stack)
-  puts "unexpected #{@current_token[0]} on #{@current_token[1][:filename] || "<stdin>"}:#{@current_token[1][:lineno]}"
-  puts @lines[@current_token[1][:lineno]]
-  puts " " * @current_token[1][:position] + "^" + "~" * (@current_token[1][:length] - 1)
+  message = if @current_token
+              <<~MESSAGE
+                #{@filename}:#{@current_token[1][:lineno]} unexpected #{@current_token[0]}
+                #{@lines[@current_token[1][:lineno] - 1].chomp}
+                #{" " * @current_token[1][:position]}^#{"~" * ([@current_token[1][:length] - 1, 0].max)}
+                MESSAGE
+            else
+              "#{@filename}:#{@lineno} unexpected end-of-file"
+            end
 
-  raise "Error"
+  raise(Racc::ParseError, message)
 end
 ...end parser.ra/module_eval...
 ##### State transition tables begin ###
 
 racc_action_table = [
-     2,    11,     4,     8,    19,    10,     4,     8,    20,    10,
-     4,     8,   nil,    10,     4,     8,   nil,    10,     8,    13,
-    15,    10 ]
+     5,     9,     2,    11,     5,     9,     3,    11,     5,     9,
+    12,    11,     9,    14,    16,    11,    13 ]
 
 racc_action_check = [
-     0,     1,     0,     0,    11,     0,     2,     2,    12,     2,
-     8,     8,   nil,     8,    10,    10,   nil,    10,     5,     5,
-     6,     6 ]
+     2,     2,     0,     2,     9,     9,     1,     9,    11,    11,
+     3,    11,     6,     6,     7,     7,     4 ]
 
 racc_action_pointer = [
-    -2,     1,     2,   nil,   nil,    13,    14,   nil,     6,   nil,
-    10,     4,     5,   nil,   nil,   nil,   nil,   nil,   nil,   nil,
-   nil ]
+     0,     6,    -4,    10,    13,   nil,     7,     8,   nil,     0,
+   nil,     4,   nil,   nil,   nil,   nil,   nil,   nil,   nil,   nil ]
 
 racc_action_default = [
-   -13,   -13,   -13,    -3,    -4,   -13,   -13,    -8,   -13,   -11,
-   -13,   -13,    -2,    -5,    -7,    -6,   -10,    -9,   -12,    21,
-    -1 ]
+   -12,   -12,   -12,   -12,    -2,    -3,   -12,   -12,    -7,   -12,
+   -10,   -12,    20,    -1,    -4,    -6,    -5,    -9,    -8,   -11 ]
 
 racc_goto_table = [
-     3,     1,    12,    14,    16,   nil,   nil,   nil,    17,   nil,
-    18 ]
+     4,     1,    15,    17,   nil,   nil,   nil,    18,   nil,    19 ]
 
 racc_goto_check = [
-     2,     1,     2,     5,     6,   nil,   nil,   nil,     2,   nil,
-     2 ]
+     2,     1,     5,     6,   nil,   nil,   nil,     2,   nil,     2 ]
 
 racc_goto_pointer = [
-   nil,     1,     0,   nil,   nil,    -2,    -2 ]
+   nil,     1,    -2,   nil,   nil,    -4,    -4 ]
 
 racc_goto_default = [
-   nil,   nil,   nil,     5,     6,     7,     9 ]
+   nil,   nil,   nil,     6,     7,     8,    10 ]
 
 racc_reduce_table = [
   0, 0, :racc_error,
   3, 9, :_reduce_1,
   2, 9, :_reduce_2,
-  1, 9, :_reduce_none,
-  1, 10, :_reduce_4,
+  1, 10, :_reduce_3,
+  2, 10, :_reduce_4,
   2, 10, :_reduce_5,
-  2, 10, :_reduce_6,
-  2, 11, :_reduce_7,
-  1, 11, :_reduce_8,
-  2, 13, :_reduce_9,
-  2, 12, :_reduce_10,
-  1, 12, :_reduce_11,
-  2, 14, :_reduce_12 ]
+  2, 11, :_reduce_6,
+  1, 11, :_reduce_7,
+  2, 13, :_reduce_8,
+  2, 12, :_reduce_9,
+  1, 12, :_reduce_10,
+  2, 14, :_reduce_11 ]
 
-racc_reduce_n = 13
+racc_reduce_n = 12
 
-racc_shift_n = 21
+racc_shift_n = 20
 
 racc_token_table = {
   false => 0,
@@ -230,7 +234,7 @@ racc_token_table = {
   :VALUE => 4,
   :KEY => 5,
   :UNINDENT => 6,
-  "-" => 7 }
+  :ITEM => 7 }
 
 racc_nt_base = 8
 
@@ -260,7 +264,7 @@ Racc_token_to_s_table = [
   "VALUE",
   "KEY",
   "UNINDENT",
-  "\"-\"",
+  "ITEM",
   "$start",
   "document",
   "value",
@@ -289,11 +293,16 @@ module_eval(<<'.,.,', 'parser.ra', 10)
   end
 .,.,
 
-# reduce 3 omitted
+module_eval(<<'.,.,', 'parser.ra', 12)
+  def _reduce_3(val, _values, result)
+     result = Scalar.new(val[0])
+    result
+  end
+.,.,
 
 module_eval(<<'.,.,', 'parser.ra', 13)
   def _reduce_4(val, _values, result)
-     result = Scalar.new(val[0])
+     result = val[0]
     result
   end
 .,.,
@@ -305,50 +314,43 @@ module_eval(<<'.,.,', 'parser.ra', 14)
   end
 .,.,
 
-module_eval(<<'.,.,', 'parser.ra', 15)
+module_eval(<<'.,.,', 'parser.ra', 16)
   def _reduce_6(val, _values, result)
-     result = val[0]
+     val[0].add_item(*val[1]); result = val[0]
     result
   end
 .,.,
 
 module_eval(<<'.,.,', 'parser.ra', 17)
   def _reduce_7(val, _values, result)
-     val[0].add_item(*val[1]); result = val[0]
+     result = Dictionary.create(*val[0])
     result
   end
 .,.,
 
-module_eval(<<'.,.,', 'parser.ra', 18)
+module_eval(<<'.,.,', 'parser.ra', 19)
   def _reduce_8(val, _values, result)
-     result = Dictionary.create(*val[0])
+     result = [Scalar.new(val[0]), val[1]]
     result
   end
 .,.,
 
-module_eval(<<'.,.,', 'parser.ra', 20)
+module_eval(<<'.,.,', 'parser.ra', 21)
   def _reduce_9(val, _values, result)
-     result = [Scalar.new(val[0]), val[1]]
+     val[0].add_item(*val[1]); result = val[0]
     result
   end
 .,.,
 
 module_eval(<<'.,.,', 'parser.ra', 22)
   def _reduce_10(val, _values, result)
-     val[0].add_item(*val[1]); result = val[0]
-    result
-  end
-.,.,
-
-module_eval(<<'.,.,', 'parser.ra', 23)
-  def _reduce_11(val, _values, result)
      result = List.create(*val[0])
     result
   end
 .,.,
 
-module_eval(<<'.,.,', 'parser.ra', 25)
-  def _reduce_12(val, _values, result)
+module_eval(<<'.,.,', 'parser.ra', 24)
+  def _reduce_11(val, _values, result)
      result = [Scalar.new(val[0]), val[1]]
     result
   end

data/lib/yaml/sort/version.rb

@@ -2,6 +2,6 @@
 
 module Yaml
   module Sort
-    VERSION = "1.0.0"
+    VERSION = "1.0.1"
   end
 end

data/yaml-sort.gemspec

@@ -32,5 +32,6 @@ Gem::Specification.new do |spec|
   spec.add_dependency "cri"
 
   spec.add_development_dependency "aruba"
+  spec.add_development_dependency "github_changelog_generator"
   spec.add_development_dependency "racc"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: yaml-sort
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.0.1
 platform: ruby
 authors:
 - Romain Tartière
-autorequire: 
+autorequire:
 bindir: exe
 cert_chain: []
-date: 2022-04-20 00:00:00.000000000 Z
+date: 2022-04-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: cri
@@ -38,6 +38,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: github_changelog_generator
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: racc
   requirement: !ruby/object:Gem::Requirement
@@ -52,7 +66,7 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-description: 
+description:
 email:
 - romain@blogreen.org
 executables:
@@ -89,7 +103,7 @@ metadata:
   source_code_uri: https://github.com/smortex/yaml-sort
   changelog_uri: https://github.com/smortex/yaml-sort/blob/master/CHANGELOG.md
   rubygems_mfa_required: 'true'
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -104,8 +118,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.5
-signing_key: 
+rubygems_version: 3.3.10
+signing_key:
 specification_version: 4
 summary: Sort lines in YAML files in a predictable order
 test_files: []