expressir 1.2.4-x86_64-darwin → 1.2.5-x86_64-darwin

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 596253e028087e7bd843297fd101f9a8787688753d8b4c8482e00e64c823e4c8
-  data.tar.gz: 0e752d5ab11ba75e8286511346d4c40649e6fff4f4b1aa0b7ed8cfc95502873e
+  metadata.gz: 055670ef49f484e32ce8acf74915c62c5b739a1b388615fe75e2c56c3a4a08e2
+  data.tar.gz: ae71aa658d3f40f4ccb6ef8c5a45f31c7bf8b7518353fda42c3599788fffe86b
 SHA512:
-  metadata.gz: 81c916894ca43b0445bd17e6ad9ba3a73aa68b16c3d0875f3fd193eb7e1eca3e49388f3743d0ed57672d0d67b5cf8b5f87c9a5c9c1ace54f52a46f2136e22b3c
-  data.tar.gz: 542550cac5af1a44c39c4a638888559e55a194832e025a22e6b6614584eb349950172d2ebef80be4b460dcdcf4024f77fcbb226de65614706c9d0b933e60d4d2
+  metadata.gz: ab99a00be5e1bf1b1874e7a2b020c833e116545a10365630f05bee6ca2ba5d62cd3a770249d6282dd76bdba6f3a4519aaa7b77347ea0fb0ee8ad8c9889b8438f
+  data.tar.gz: 1405e67409d7dfc3b97c456876dc8f5c436666ac0f38cc2186fa33ac893e237ac2aa0044b99ce15807723faaf5c5aa95eb56cc00c322a46f050976a275f30610

data/.cross_rubies

@@ -1,20 +1,14 @@
-3.1.0:i686-w64-mingw32
 3.1.0:x86_64-w64-mingw32
-3.1.0:i686-linux-gnu
 3.1.0:x86_64-linux-gnu
 3.1.0:aarch64-linux-gnu
 3.1.0:x86_64-darwin
 3.1.0:arm64-darwin
-3.0.0:i686-w64-mingw32
 3.0.0:x86_64-w64-mingw32
-3.0.0:i686-linux-gnu
 3.0.0:x86_64-linux-gnu
 3.0.0:aarch64-linux-gnu
 3.0.0:x86_64-darwin
 3.0.0:arm64-darwin
-2.7.0:i686-w64-mingw32
 2.7.0:x86_64-w64-mingw32
-2.7.0:i686-linux-gnu
 2.7.0:x86_64-linux-gnu
 2.7.0:aarch64-linux-gnu
 2.7.0:x86_64-darwin

data/.github/workflows/rake.yml

@@ -6,9 +6,29 @@ on:
   pull_request:
 
 jobs:
+  rubocop:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v3
+        with:
+          submodules: recursive
+
+      - name: Install Ruby
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: 3.0
+          bundler-cache: true
+
+      - name: Bundle
+        run: bundle install --jobs 4 --retry 3
+
+      - name: Rubocop
+        run: bundle exec rake rubocop
+
   rake:
     name: test on ruby-${{ matrix.ruby }} ${{ matrix.os }}
-    runs-on: ${{ matrix.os }}   
+    runs-on: ${{ matrix.os }}
     continue-on-error: ${{ matrix.experimental }}
     strategy:
       fail-fast: false
@@ -29,7 +49,7 @@ jobs:
 
     steps:
       - name: Checkout
-        uses: actions/checkout@v2
+        uses: actions/checkout@v3
         with:
           submodules: recursive
 
@@ -55,18 +75,18 @@ jobs:
         run: bundle exec rake compile
 
       - name: Run tests
-        run: bundle exec rake
+        run: |
+          bundle exec rake
+          cat .rspec_status
 
   # test release workflow
-  pack:
+  pack-ruby:
     runs-on: ubuntu-latest
     strategy:
       fail-fast: false
-      matrix:
-        host: [ linux, windows, darwin ]
     steps:
       - name: Checkout
-        uses: actions/checkout@v2
+        uses: actions/checkout@v3
         with:
           submodules: recursive
 
@@ -81,16 +101,36 @@ jobs:
         run: bundle install --jobs 4 --retry 3
 
       - name: Build gem without native extension
-        if: matrix.host == 'linux'
         run: gem build expressir.gemspec
 
       - name: Package gem without native extension
-        if: matrix.host == 'linux'
         uses: actions/upload-artifact@v2
         with:
           name: pkg-ruby
           path: expressir-*.gem
 
+  pack:
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        host: [ linux, windows, darwin ]
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v3
+        with:
+          submodules: recursive
+
+      - name: Setup Ruby
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: '3.0'
+          # bundler-cache: true important to not use cache because it leads to "cannot find -lrice"
+          # more info https://github.com/lutaml/expressir/runs/2097658383?check_suite_focus=true#step:7:2126
+
+      - name: Bundle
+        run: bundle install --jobs 4 --retry 3
+
       - name: Enable swap
         run: |
           sudo fallocate -l 15g /compile.swap
@@ -107,3 +147,154 @@ jobs:
         with:
           name: pkg-${{ matrix.host }}
           path: pkg/*.gem
+
+  verify-ruby:
+    name: verify ruby gem on ruby-${{ matrix.ruby }} ${{ matrix.os }}
+    needs: pack-ruby
+    runs-on: ${{ matrix.os }}
+    strategy:
+      fail-fast: false
+      matrix:
+        ruby: [ '3.1', '3.0', '2.7' ]
+        os: [ ubuntu-latest, windows-latest, macos-latest ]
+    steps:
+      - name: Install Ruby
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: ${{ matrix.ruby }}
+          bundler-cache: true
+
+      - name: Checkout
+        uses: actions/checkout@v3
+        with:
+          submodules: recursive
+
+      - name: Bundle
+        run: bundle install --jobs 4 --retry 3
+        
+      - name: Download packaged gem
+        uses: actions/download-artifact@v2
+        with:
+         name: pkg-ruby
+         path: pkg
+
+      - name: Install gem
+        run: gem install -l pkg/expressir-*.gem
+
+      - name: Verify
+        run: |
+          cd $(ruby -e "puts RbConfig::TOPDIR + '\/lib\/ruby\/gems\/' + RbConfig::CONFIG['ruby_version'] + '\/gems\/expressir*'")
+          ruby bin/rspec
+          cat .rspec_status || echo ".rspec_status was not found"
+
+  verify-darwin:
+    name: verify MacOS binary gem on ruby-${{ matrix.ruby }}
+    needs: pack
+    runs-on: macos-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        ruby: [ '3.1', '3.0', '2.7' ]
+    steps:
+      - name: Install Ruby
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: ${{ matrix.ruby }}
+          bundler-cache: true
+
+      - name: Checkout
+        uses: actions/checkout@v3
+
+      - name: Bundle
+        run: bundle install --jobs 4 --retry 3
+        
+      - name: Download packaged gem
+        uses: actions/download-artifact@v2
+        with:
+         name: pkg-darwin
+         path: pkg
+
+      - name: Install binary gem
+        run: gem install -l pkg/expressir-*-$(ruby -e "puts RUBY_PLATFORM.sub(/darwin\d{2}$/, 'darwin')").gem
+# MacOS with have something like x86_64-darwin19, others just x86_64-linux
+
+      - name: Verify
+        run: |
+          cd $(ruby -e "puts RbConfig::TOPDIR + '\/lib\/ruby\/gems\/' + RbConfig::CONFIG['ruby_version'] + '\/gems\/expressir*'")
+          ruby bin/rspec
+          cat .rspec_status || echo ".rspec_status was not found"
+
+  verify-linux:
+    name: verify Linux binary gem on ruby-${{ matrix.ruby }}
+    needs: pack
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        ruby: [ '3.1', '3.0', '2.7' ]
+    steps:
+      - name: Install Ruby
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: ${{ matrix.ruby }}
+          bundler-cache: true
+
+      - name: Checkout
+        uses: actions/checkout@v3
+
+      - name: Bundle
+        run: bundle install --jobs 4 --retry 3
+        
+      - name: Download packaged gem
+        uses: actions/download-artifact@v2
+        with:
+         name: pkg-linux
+         path: pkg
+
+      - name: Install binary gem
+        run: gem install -l pkg/expressir-*-$(ruby -e "puts RUBY_PLATFORM").gem
+
+      - name: Verify
+        run: |
+          cd $(ruby -e "puts RbConfig::TOPDIR + '\/lib\/ruby\/gems\/' + RbConfig::CONFIG['ruby_version'] + '\/gems\/expressir*'")
+          ruby bin/rspec
+          cat .rspec_status || echo ".rspec_status was not found"
+
+  verify-windows:
+    name: verify Windows binary gem on ruby-${{ matrix.ruby }}
+    needs: pack
+    continue-on-error: true
+    runs-on: windows-latest
+    strategy:
+      fail-fast: false
+# Ruby 3.1 fails 
+# https://github.com/lutaml/expressir/issues/103      
+      matrix:
+        ruby: [ '3.1', '3.0', '2.7' ]
+    steps:
+      - name: Install Ruby
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: ${{ matrix.ruby }}
+          bundler-cache: true
+
+      - name: Checkout
+        uses: actions/checkout@v3
+
+      - name: Bundle
+        run: bundle install --jobs 4 --retry 3
+        
+      - name: Download packaged gem
+        uses: actions/download-artifact@v2
+        with:
+         name: pkg-windows
+         path: pkg
+
+      - name: Install binary gem
+        run: gem install -l pkg/expressir-*-$(ruby -e "puts RUBY_PLATFORM").gem
+
+      - name: Verify
+        run: |
+          cd $(ruby -e "puts RbConfig::TOPDIR + '\/lib\/ruby\/gems\/' + RbConfig::CONFIG['ruby_version'] + '\/gems\/expressir*'")
+          ruby bin/rspec
+          cat .rspec_status || echo ".rspec_status was not found"

data/.gitignore

@@ -8,11 +8,16 @@
 /tmp/
 /Gemfile.lock
 /.idea
+/.vscode
 /lib/expressir/express/express_parser.bundle
+/lib/expressir/express/express_parser.so
+/lib/expressir/express/*/express_parser.bundle
+/lib/expressir/express/*/express_parser.so
 /spec/syntax/*-pretty.exp
+/ext/express-parser/rice-embed
 
 # rspec failure tracking
 .rspec_status
 
-/ext/express-parser/rice-embed
-/lib/expressir/express/express_parser.so
+# rubocop guide
+.rubocop-https---*

data/.gitmodules

@@ -1,3 +1,6 @@
 [submodule "ext/express-parser/antlr4-upstream"]
 	path = ext/express-parser/antlr4-upstream
 	url = https://github.com/antlr/antlr4
+[submodule "ext/express-grammar"]
+	path = ext/express-grammar
+	url = https://github.com/lutaml/express-grammar

data/.rubocop.yml

@@ -1,8 +1,17 @@
 inherit_from:
   - https://raw.githubusercontent.com/riboseinc/oss-guides/master/ci/rubocop.yml
 
-# local repo-specific modifications
-# ...
-
 AllCops:
-  TargetRubyVersion: 2.4
+  TargetRubyVersion: 2.6
+  SuggestExtensions: false
+  Exclude:
+    - 'vendor/**/*'
+    - 'expressir.gemspec'
+    - 'tmp/**/*'
+    - 'pkg/**/*'
+    - 'exe/format-test'
+    - 'lib/expressir/express/**/*'
+    - 'lib/expressir/model/**/*'
+
+Layout/LineLength:
+  Max: 160

data/Rakefile

@@ -1,14 +1,17 @@
 require "bundler/gem_tasks"
 require "rspec/core/rake_task"
 require "rake/extensiontask"
+require "rubocop/rake_task"
 require "yard"
 
 RSpec::Core::RakeTask.new(:spec)
 
-task :default => :spec
+task default: :spec
 
 GEMSPEC = Gem::Specification.load("expressir.gemspec")
 
+RuboCop::RakeTask.new
+
 Gem::PackageTask.new(GEMSPEC).define
 
 YARD::Rake::YardocTask.new

data/bin/console

@@ -9,4 +9,3 @@ require "reeper"
 # (If you use this, don't forget to add pry to your Gemfile!)
 require "pry"
 Pry.start
-

data/bin/rspec

@@ -10,12 +10,12 @@
 
 require "pathname"
 ENV["BUNDLE_GEMFILE"] ||= File.expand_path("../../Gemfile",
-  Pathname.new(__FILE__).realpath)
+                                           Pathname.new(__FILE__).realpath)
 
-bundle_binstub = File.expand_path("../bundle", __FILE__)
+bundle_binstub = File.expand_path("bundle", __dir__)
 
 if File.file?(bundle_binstub)
-  if File.read(bundle_binstub, 300) =~ /This file was generated by Bundler/
+  if /This file was generated by Bundler/.match?(File.read(bundle_binstub, 300))
     load(bundle_binstub)
   else
     abort("Your `bin/bundle` was not generated by Bundler, so this binstub cannot run.

data/exe/expressir

@@ -11,8 +11,10 @@ $:.unshift File.expand_path("../../lib", bin_file)
 # Fixes https://github.com/rubygems/rubygems/issues/1420
 require "rubygems/specification"
 
-class Gem::Specification
-  def this; self; end
+module Gem
+  class Specification
+    def this; self; end
+  end
 end
 
 # start up the CLI

data/exe/format

@@ -15,4 +15,4 @@ formatter = Class.new(Expressir::Express::Formatter) do
 end
 result = repository.to_hash(formatter: formatter, skip_empty: true)
 
-puts YAML.dump(result)
+puts YAML.dump(result)

data/exe/format-test

@@ -24,30 +24,30 @@ exp_files = [
   # '../iso-10303-stepmod/data/resources/iso13584_expressions_schema/iso13584_expressions_schema.exp',
   # annotated-express test (12s)
   # see https://github.com/metanorma/annotated-express/blob/master/data/documents/resources/fundamentals_of_product_description_and_support/sections/04-schemas.adoc
-  '../iso-10303-stepmod/data/resources/action_schema/action_schema_annotated.exp',
-  '../iso-10303-stepmod/data/resources/application_context_schema/application_context_schema_annotated.exp',
-  '../iso-10303-stepmod/data/resources/approval_schema/approval_schema_annotated.exp',
-  '../iso-10303-stepmod/data/resources/basic_attribute_schema/basic_attribute_schema_annotated.exp',
-  '../iso-10303-stepmod/data/resources/certification_schema/certification_schema_annotated.exp',
-  '../iso-10303-stepmod/data/resources/contract_schema/contract_schema_annotated.exp',
-  '../iso-10303-stepmod/data/resources/date_time_schema/date_time_schema_annotated.exp',
-  '../iso-10303-stepmod/data/resources/document_schema/document_schema.exp',
-  '../iso-10303-stepmod/data/resources/effectivity_schema/effectivity_schema_annotated.exp',
-  '../iso-10303-stepmod/data/resources/experience_schema/experience_schema_annotated.exp',
-  '../iso-10303-stepmod/data/resources/external_reference_schema/external_reference_schema_annotated.exp',
-  '../iso-10303-stepmod/data/resources/group_schema/group_schema_annotated.exp',
-  '../iso-10303-stepmod/data/resources/language_schema/language_schema_annotated.exp',
-  '../iso-10303-stepmod/data/resources/location_schema/location_schema_annotated.exp',
-  '../iso-10303-stepmod/data/resources/management_resources_schema/management_resources_schema_annotated.exp',
-  '../iso-10303-stepmod/data/resources/measure_schema/measure_schema_annotated.exp',
-  '../iso-10303-stepmod/data/resources/person_organization_schema/person_organization_schema_annotated.exp',
-  '../iso-10303-stepmod/data/resources/process_property_schema/process_property_schema_annotated.exp',
-  '../iso-10303-stepmod/data/resources/product_definition_schema/product_definition_schema_annotated.exp',
-  '../iso-10303-stepmod/data/resources/product_property_definition_schema/product_property_definition_schema_annotated.exp',
-  '../iso-10303-stepmod/data/resources/product_property_representation_schema/product_property_representation_schema_annotated.exp',
-  '../iso-10303-stepmod/data/resources/qualifications_schema/qualifications_schema_annotated.exp',
-  '../iso-10303-stepmod/data/resources/security_classification_schema/security_classification_schema_annotated.exp',
-  '../iso-10303-stepmod/data/resources/support_resource_schema/support_resource_schema_annotated.exp',
+  "../iso-10303-stepmod/data/resources/action_schema/action_schema_annotated.exp",
+  "../iso-10303-stepmod/data/resources/application_context_schema/application_context_schema_annotated.exp",
+  "../iso-10303-stepmod/data/resources/approval_schema/approval_schema_annotated.exp",
+  "../iso-10303-stepmod/data/resources/basic_attribute_schema/basic_attribute_schema_annotated.exp",
+  "../iso-10303-stepmod/data/resources/certification_schema/certification_schema_annotated.exp",
+  "../iso-10303-stepmod/data/resources/contract_schema/contract_schema_annotated.exp",
+  "../iso-10303-stepmod/data/resources/date_time_schema/date_time_schema_annotated.exp",
+  "../iso-10303-stepmod/data/resources/document_schema/document_schema.exp",
+  "../iso-10303-stepmod/data/resources/effectivity_schema/effectivity_schema_annotated.exp",
+  "../iso-10303-stepmod/data/resources/experience_schema/experience_schema_annotated.exp",
+  "../iso-10303-stepmod/data/resources/external_reference_schema/external_reference_schema_annotated.exp",
+  "../iso-10303-stepmod/data/resources/group_schema/group_schema_annotated.exp",
+  "../iso-10303-stepmod/data/resources/language_schema/language_schema_annotated.exp",
+  "../iso-10303-stepmod/data/resources/location_schema/location_schema_annotated.exp",
+  "../iso-10303-stepmod/data/resources/management_resources_schema/management_resources_schema_annotated.exp",
+  "../iso-10303-stepmod/data/resources/measure_schema/measure_schema_annotated.exp",
+  "../iso-10303-stepmod/data/resources/person_organization_schema/person_organization_schema_annotated.exp",
+  "../iso-10303-stepmod/data/resources/process_property_schema/process_property_schema_annotated.exp",
+  "../iso-10303-stepmod/data/resources/product_definition_schema/product_definition_schema_annotated.exp",
+  "../iso-10303-stepmod/data/resources/product_property_definition_schema/product_property_definition_schema_annotated.exp",
+  "../iso-10303-stepmod/data/resources/product_property_representation_schema/product_property_representation_schema_annotated.exp",
+  "../iso-10303-stepmod/data/resources/qualifications_schema/qualifications_schema_annotated.exp",
+  "../iso-10303-stepmod/data/resources/security_classification_schema/security_classification_schema_annotated.exp",
+  "../iso-10303-stepmod/data/resources/support_resource_schema/support_resource_schema_annotated.exp",
   # full test (6m18s + 8s = 6m26s)
   # *`bundle exec ../stepmod-utils/exe/stepmod-find-express-files ../iso-10303-stepmod`.strip.split("\n").map{|file| File.exists?(file.sub(/\.exp$/, '_annotated.exp')) ? file.sub(/\.exp$/, '_annotated.exp') : file}
 ]
@@ -78,4 +78,4 @@ end
 result = repository.to_hash(formatter: formatter, skip_empty: true)
 puts "Repository.to_hash time: #{(Time.now - start).round(2)}s"
 
-# puts YAML.dump(result)
+# puts YAML.dump(result)

data/exe/generate-parser

@@ -1,29 +1,30 @@
 #!/usr/bin/env ruby
 
-require 'fileutils'
-require 'antlr4-native'
+require "fileutils"
+require "antlr4-native"
 
 grammar_file = ARGV.shift
 
 # ANTLR does weird things if the grammar file isn't in the current working directory
-temp_grammar_file = File.join(FileUtils.pwd(), File.basename(grammar_file))
+temp_grammar_file = File.join(FileUtils.pwd, File.basename(grammar_file))
 FileUtils.cp(grammar_file, temp_grammar_file)
 
 # generate parser
 generator = Antlr4Native::Generator.new(
   grammar_files: [File.basename(temp_grammar_file)],
-  output_dir: 'ext',
-  parser_root_method: 'syntax'
+  output_dir: "ext",
+  parser_root_method: "syntax",
 )
 generator.generate
 
 # fix issues with generated parser
-parser_source_file = File.join(*%w(ext express-parser express_parser.cpp))
-parser_source_lines = File.read(parser_source_file).split(/\n/)
+parser_source_file = File.join("ext", "express-parser", "express_parser.cpp")
+parser_source_lines = File.read(parser_source_file).split("\n")
 
 # - add ParserProxy tokens method, simple compensation for missing exposed BufferedTokenStream
-i = parser_source_lines.index{|x| x == '  Object syntax() {'}
-parser_source_lines[i + 6] += "\n" + <<~CPP.split(/\n/).map{|x| x != "" ? "  " + x : x}.join("\n") + "\n"
+i = parser_source_lines.index { |x| x == "  Object syntax() {" }
+parser_source_lines[i + 6] += <<~CPP.split("\n").map { |x| x == "" ? x : "  #{x}" }.join("\n")
+
   Array getTokens() {
     Array a;
 
@@ -35,14 +36,16 @@ parser_source_lines[i + 6] += "\n" + <<~CPP.split(/\n/).map{|x| x != "" ? "  " +
 
     return a;
   }
+
 CPP
-i = parser_source_lines.index{|x| x == '    .define_method("syntax", &ParserProxy::syntax)'}
-parser_source_lines[i] += "\n" + <<~CPP.split(/\n/).map{|x| x != "" ? "    " + x : x}.join("\n")
+i = parser_source_lines.index { |x| x == '    .define_method("syntax", &ParserProxy::syntax, Return().keepAlive())' }
+parser_source_lines[i] += <<~CPP.split("\n").map { |x| x == "" ? x : "    #{x}" }.join("\n")
+
   .define_method("tokens", &ParserProxy::getTokens)
 CPP
 
 # write fixed parser file
-File.write(parser_source_file, parser_source_lines.join("\n") + "\n")
+File.write(parser_source_file, "#{parser_source_lines.join("\n")}\n")
 
 # cleanup
-FileUtils.rm(temp_grammar_file)
+FileUtils.rm(temp_grammar_file)

data/expressir.gemspec

@@ -17,10 +17,10 @@ Gem::Specification.new do |spec|
   spec.metadata["source_code_uri"] = spec.homepage
   spec.metadata["changelog_uri"] = "https://github.com/lutaml/expressir/releases"
 
-  spec.required_ruby_version = Gem::Requirement.new(">= 2.4.0")
+  spec.required_ruby_version = Gem::Requirement.new(">= 2.6.0")
 
   spec.files         = `git ls-files`.split("\n")\
-    + Dir.glob('ext/express-parser/antlr4-upstream/runtime/Cpp/runtime/**/*')
+    + Dir.glob("ext/express-parser/antlr4-upstream/runtime/Cpp/runtime/**/*")
 
   spec.test_files    = `git ls-files -- {spec}/*`.split("\n")
 
@@ -30,17 +30,19 @@ Gem::Specification.new do |spec|
 
   spec.extensions = File.join(*%w(ext express-parser extconf.rb))
 
-  spec.add_runtime_dependency "thor", "~> 1.0"
   spec.add_runtime_dependency "rice", "~> 4.0.3"
+  spec.add_runtime_dependency "thor", "~> 1.0"
+  spec.add_development_dependency "antlr4-native", "~> 2.1.0"
   spec.add_development_dependency "asciidoctor", "~> 2.0.13"
-  spec.add_development_dependency "antlr4-native", "~> 1.1.0"
   spec.add_development_dependency "bundler", "~> 2.0"
-  spec.add_development_dependency "pry", "~> 0.12.2"
   spec.add_development_dependency "byebug", "~> 11.1"
+  spec.add_development_dependency "pry", "~> 0.12.2"
   spec.add_development_dependency "rake", "~> 13.0"
-  spec.add_development_dependency "rake-compiler", "~> 1.1"
-  spec.add_development_dependency "rake-compiler-dock", "~> 1.1"
-  spec.add_development_dependency "rspec", "~> 3.0"
-  spec.add_development_dependency "yard", "~> 0.9.26"
+  spec.add_development_dependency "rake-compiler", "~> 1.2"
+  spec.add_development_dependency "rake-compiler-dock", "~> 1.2.1"
+  spec.add_development_dependency "rspec", "~> 3.11"
+  spec.add_development_dependency "rubocop", "1.5.2"
+  spec.add_development_dependency "rubocop-performance", "~> 1.0"
   spec.add_development_dependency "webrick", "~> 1.7.0"
+  spec.add_development_dependency "yard", "~> 0.9.26"
 end

data/lib/expressir/config.rb

@@ -15,7 +15,7 @@ module Expressir
     attr_accessor :logs
 
     def initialize
-      @logs ||= %i(error)
+      @logs = %i(error)
     end
   end
 

data/lib/expressir/express/parser.rb

@@ -31,13 +31,13 @@ module Expressir
       # @param [Boolean] include_source attach original source code to model elements
       # @return [Model::Repository]
       def self.from_file(file, skip_references: nil, include_source: nil)
-        input = File.read(file)
+        @input = File.read(file)
 
       # An important note re memory management
       # parse, syntax, visitor methods return complex tree structures created in netive (C++) extension
       # visit method references nodes and leaves of this structures but it is totally untransparent for Ruby GarbageCllector
       # so in this class we keep those C++ structure marked for GC so theu are not freed 
-        @parser = ::ExpressParser::Parser.parse(input)
+        @parser = ::ExpressParser::Parser.parse(@input)
         @parse_tree = @parser.syntax()
 
         @visitor = Visitor.new(@parser.tokens, include_source: include_source)

data/lib/expressir/express/visitor.rb

@@ -31,6 +31,8 @@ require "set"
 # - prevents segfault in ANTLR4 C++ runtime, not sure why they are caused
 # - e.g. see visit_schema_decl
 
+require 'objspace'
+
 module Expressir
   module Express
     class Visitor < ::ExpressParser::Visitor
@@ -138,15 +140,15 @@ module Expressir
       end
 
       def attach_remarks(ctx, node)
-        remark_tokens = get_tokens(ctx)
-        remark_tokens = remark_tokens.select{ |x| x.channel == REMARK_CHANNEL
-      }
+        @remark_tokens = get_tokens(ctx)
+        @remark_tokens = @remark_tokens.select{ |x| x.channel == REMARK_CHANNEL
+        }
 
         # skip already attached remarks
-        remark_tokens = remark_tokens.select{|x| !@attached_remark_tokens.include?(x)}
+        @remark_tokens = @remark_tokens.select{|x| !@attached_remark_tokens.include?(x)}
 
         # parse remarks, find remark targets
-        tagged_remark_tokens = remark_tokens.map do |remark_token|
+        tagged_remark_tokens = @remark_tokens.map do |remark_token|
           _, remark_tag, remark_text = if remark_token.text.start_with?('--')
             remark_token.text.match(/^--"([^"]*)"(.*)$/).to_a
           else