grumlin 0.15.0 → 0.15.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e6bc5993ec07bf70b93c7f7b73e2e3716c14b3457ddedbbda0e6f38d5f2c19ff
-  data.tar.gz: b23c85808cacf2585fbc7c47f60d093049c1976ca13acd0317d0c71bf9871a5d
+  metadata.gz: e796fc1b5bff2cb474d068ad2a05a2b8a195df1981842745d5c90798ab5e02b7
+  data.tar.gz: 39cd9ac6bbedfb6a8707d306c6a6e0223856028162778bb665865e387dcf2bc3
 SHA512:
-  metadata.gz: b752f3c20878758524da9d31de2114bca8d6585c9a9038989acef2ec21a888bbd667fe9ece033f257c67751eb87c33025c2876f80f80e3a983f25fb09c1e4b37
-  data.tar.gz: 9ac44ccf984d626cbb8254185534c75061d139934a613fae13c45cfd0cdbdf1d4d35ca088e0ff2a1073196294d8b38a5c4eff8a5b2a8a3258327537ad66daab6
+  metadata.gz: 072df70e88f7192272cad5c6127740ef10d52b5057f42a557e9f4ea722e55f53d222025e7b4d7607666d4c5038d3e630abb55faec7f05c640efed53d442a2b57
+  data.tar.gz: 32ac57244365458e56c9b2c2705500c87254ba523454817b67f88637e07444d5e0b2abfa361540693f0dedf578b7e5f346c8a2515e536c69ad86e5ee09fa7bf3

data/.github/workflows/main.yml

@@ -22,7 +22,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        ruby: ["2.7", "3.0"]
+        ruby: ["2.7", "3.0", "3.1"]
     steps:
       - uses: actions/checkout@v2
 

data/CHANGELOG.md

@@ -1,3 +1,20 @@
+## [0.15.4] - 2022-01-20
+
+- Move step and expression definitions to a yaml file for better diffs
+- Add `definitions:format` rake task
+
+## [0.15.3] - 2022-01-18
+
+- Fix passing nils as step arguments. Even if they are not supported by the server, they should not be omitted.
+
+## [0.15.2] - 2022-01-17
+
+- New steps: `map` and `identity`
+
+## [0.15.1] - 2022-01-17
+
+- Fix passing arrays as step arguments
+
 ## [0.15.0] - 2022-01-11
 
 - Add `properties` step

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    grumlin (0.15.0)
+    grumlin (0.15.4)
       async-pool (~> 0.3)
       async-websocket (~> 0.19)
       oj (~> 3.12)
@@ -60,13 +60,13 @@ GEM
       rexml
     kramdown-parser-gfm (1.1.0)
       kramdown (~> 2.0)
-    mini_portile2 (2.5.3)
+    mini_portile2 (2.7.1)
     minitest (5.14.4)
     nio4r (2.5.8)
-    nokogiri (1.11.7)
-      mini_portile2 (~> 2.5.0)
+    nokogiri (1.13.1)
+      mini_portile2 (~> 2.7.0)
       racc (~> 1.4)
-    nokogiri (1.11.7-x86_64-linux)
+    nokogiri (1.13.1-x86_64-linux)
       racc (~> 1.4)
     oj (3.13.11)
     overcommit (0.57.0)
@@ -85,7 +85,7 @@ GEM
     protocol-websocket (0.7.5)
       protocol-http (~> 0.2)
       protocol-http1 (~> 0.2)
-    racc (1.5.2)
+    racc (1.6.0)
     rainbow (3.0.0)
     rake (13.0.3)
     regexp_parser (2.2.0)

data/README.md

@@ -258,6 +258,10 @@ To install this gem onto your local machine, run `bundle exec rake install`. To
 the version number in `version.rb`, and then run `bundle exec rake release`, which will create a git tag for the version, 
 push git commits and the created tag, and push the `.gem` file to [rubygems.org](https://rubygems.org).
 
+### Adding new steps and expressions
+To add a new step or an expression simple put it to the corresponding list in [definitions.yml](lib/definitions.yml)
+and run `rake definitions:format`. You don't need to properly sort the lists manually, the rake task will do it for you.
+
 ## Contributing
 
 Bug reports and pull requests are welcome on GitHub at https://github.com/zhulik/grumlin. This project is intended to 

data/Rakefile

@@ -1,12 +1,30 @@
 # frozen_string_literal: true
 
+require "yaml"
+
 require "bundler/gem_tasks"
 require "rspec/core/rake_task"
+require "rubocop/rake_task"
 
 RSpec::Core::RakeTask.new(:spec)
+RuboCop::RakeTask.new
 
-require "rubocop/rake_task"
+task default: %i[rubocop spec]
 
-RuboCop::RakeTask.new
+namespace :definitions do
+  desc "Format definitions.yml"
+  task :format do
+    path = File.join(__dir__, "lib", "definitions.yml")
+    definitions = YAML.safe_load(File.read(path))
+
+    definitions.each_value do |kind|
+      kind.each do |name, list|
+        next if name == "with_options"
+
+        list.sort!
+      end
+    end
 
-task default: %i[spec rubocop]
+    File.write(path, YAML.dump(definitions))
+  end
+end

data/lib/definitions.yml

@@ -0,0 +1,110 @@
+---
+steps:
+  regular:
+  - E
+  - V
+  - addE
+  - addV
+  - aggregate
+  - and
+  - as
+  - both
+  - bothE
+  - by
+  - choose
+  - coalesce
+  - constant
+  - count
+  - dedup
+  - drop
+  - elementMap
+  - emit
+  - fold
+  - from
+  - group
+  - groupCount
+  - has
+  - hasId
+  - hasLabel
+  - hasNot
+  - id
+  - identity
+  - in
+  - inE
+  - inV
+  - is
+  - label
+  - limit
+  - map
+  - not
+  - or
+  - order
+  - out
+  - outE
+  - outV
+  - path
+  - project
+  - properties
+  - property
+  - range
+  - repeat
+  - sack
+  - select
+  - sideEffect
+  - skip
+  - sum
+  - tail
+  - timeLimit
+  - to
+  - unfold
+  - union
+  - until
+  - valueMap
+  - values
+  - where
+  - with
+  start:
+  - E
+  - V
+  - addE
+  - addV
+  configuration:
+  - withSack
+  - withSideEffect
+expressions:
+  operator:
+  - addAll
+  - and
+  - assign
+  - div
+  - max
+  - min
+  - minus
+  - mult
+  - or
+  - sum
+  order:
+  - asc
+  - desc
+  - shuffle
+  pop:
+  - all
+  - first
+  - last
+  - mixed
+  scope:
+  - local
+  t:
+  - id
+  - label
+  with_options:
+    tokens: "~tinkerpop.valueMap.tokens"
+    none: 0
+    ids: 1
+    labels: 2
+    keys: 4
+    values: 8
+    all: 15
+    indexer: "~tinkerpop.index.indexer"
+    list: 0
+    map: 1

data/lib/grumlin/anonymous_step.rb

@@ -4,11 +4,7 @@ module Grumlin
   class AnonymousStep
     attr_reader :name, :previous_step, :configuration_steps
 
-    # TODO: add other steps
-    SUPPORTED_STEPS = %i[E V addE addV aggregate and as both bothE by choose coalesce count dedup drop elementMap emit
-                         fold from group groupCount has hasId hasLabel hasNot id in inE inV is label limit not or order
-                         out outE path project properties property range repeat sack select sideEffect skip sum tail to
-                         unfold union until valueMap values where with].freeze
+    SUPPORTED_STEPS = Grumlin.definitions.dig(:steps, :regular).map(&:to_sym).freeze
 
     def initialize(name, *args, configuration_steps: [], previous_step: nil, **params)
       @name = name
@@ -41,7 +37,9 @@ module Grumlin
     end
 
     def args
-      [*@args, @params.any? ? @params : nil].compact
+      [*@args].tap do |args|
+        args << @params if @params.any?
+      end
     end
   end
 end

data/lib/grumlin/bytecode.rb

@@ -14,7 +14,6 @@ module Grumlin
 
     def initialize(step, no_return: false)
       super(type: "Bytecode")
-
       @step = step
       @no_return = no_return
     end
@@ -49,7 +48,7 @@ module Grumlin
       return arg.public_send(serialization_method) if arg.respond_to?(serialization_method)
       return arg unless arg.is_a?(AnonymousStep)
 
-      arg.args.flatten.each.with_object([arg.name.to_s]) do |a, res|
+      arg.args.each.with_object([arg.name.to_s]) do |a, res|
         res << if a.respond_to?(:bytecode)
                  a.bytecode.public_send(serialization_method)
                else

data/lib/grumlin/expressions/operator.rb

@@ -3,7 +3,7 @@
 module Grumlin
   module Expressions
     module Operator
-      SUPPORTED_STEPS = %i[addAll and assign div max min minus mult or sum].freeze
+      SUPPORTED_STEPS = Grumlin.definitions.dig(:expressions, :operator).map(&:to_sym).freeze
 
       class << self
         extend Expression

data/lib/grumlin/expressions/order.rb

@@ -3,7 +3,7 @@
 module Grumlin
   module Expressions
     module Order
-      SUPPORTED_STEPS = %i[asc desc shuffle].freeze
+      SUPPORTED_STEPS = Grumlin.definitions.dig(:expressions, :order).map(&:to_sym).freeze
 
       class << self
         extend Expression

data/lib/grumlin/expressions/pop.rb

@@ -3,7 +3,7 @@
 module Grumlin
   module Expressions
     module Pop
-      SUPPORTED_STEPS = %i[all first last mixed].freeze
+      SUPPORTED_STEPS = Grumlin.definitions.dig(:expressions, :pop).map(&:to_sym).freeze
 
       class << self
         extend Expression

data/lib/grumlin/expressions/scope.rb

@@ -3,7 +3,7 @@
 module Grumlin
   module Expressions
     module Scope
-      SUPPORTED_STEPS = %i[local].freeze
+      SUPPORTED_STEPS = Grumlin.definitions.dig(:expressions, :scope).map(&:to_sym).freeze
 
       class << self
         extend Expression

data/lib/grumlin/expressions/t.rb

@@ -3,7 +3,7 @@
 module Grumlin
   module Expressions
     module T
-      SUPPORTED_STEPS = %i[id label].freeze
+      SUPPORTED_STEPS = Grumlin.definitions.dig(:expressions, :t).map(&:to_sym).freeze
 
       class << self
         extend Expression

data/lib/grumlin/expressions/u.rb

@@ -2,13 +2,10 @@
 
 module Grumlin
   module Expressions
+    # The module is called U because Underscore and implements __
     module U
-      # TODO: add other start steps
-      SUPPORTED_STEPS = %i[V addV coalesce constant count drop fold has hasLabel hasNot id in inE inV is label out outE
-                           outV project repeat select timeLimit unfold valueMap values].freeze
-
       class << self
-        SUPPORTED_STEPS.each do |step|
+        Grumlin::AnonymousStep::SUPPORTED_STEPS.each do |step|
           define_method step do |*args, **params|
             AnonymousStep.new(step, *args, **params)
           end

data/lib/grumlin/expressions/with_options.rb

@@ -3,18 +3,7 @@
 module Grumlin
   module Expressions
     module WithOptions
-      WITH_OPTIONS = {
-        tokens: "~tinkerpop.valueMap.tokens",
-        none: 0,
-        ids: 1,
-        labels: 2,
-        keys: 4,
-        values: 8,
-        all: 15,
-        indexer: "~tinkerpop.index.indexer",
-        list: 0,
-        map: 1
-      }.freeze
+      WITH_OPTIONS = Grumlin.definitions.dig(:expressions, :with_options).freeze
 
       class << self
         WITH_OPTIONS.each do |k, v|

data/lib/grumlin/shortcuts.rb

@@ -21,7 +21,10 @@ module Grumlin
     def shortcut(name, &block)
       name = name.to_sym
       # TODO: blocklist of names to avoid conflicts with standard methods?
-      raise ArgumentError, "cannot use names of standard gremlin steps" if Grumlin.supported_steps.include?(name)
+      if Grumlin::AnonymousStep::SUPPORTED_STEPS.include?(name)
+        raise ArgumentError,
+              "cannot use names of standard gremlin steps"
+      end
 
       raise ArgumentError, "shortcut '#{name}' already exists" if shortcuts.key?(name) && shortcuts[name] != block
 

data/lib/grumlin/traversal.rb

@@ -2,10 +2,9 @@
 
 module Grumlin
   class Traversal
-    # TODO: add other start steps
-    SUPPORTED_STEPS = %i[E V addE addV].freeze
+    SUPPORTED_STEPS = Grumlin.definitions.dig(:steps, :start).map(&:to_sym).freeze
 
-    CONFIGURATION_STEPS = %i[withSack withSideEffect].freeze
+    CONFIGURATION_STEPS = Grumlin.definitions.dig(:steps, :configuration).map(&:to_sym).freeze
 
     attr_reader :configuration_steps
 

data/lib/grumlin/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Grumlin
-  VERSION = "0.15.0"
+  VERSION = "0.15.4"
 end

data/lib/grumlin.rb

@@ -2,7 +2,9 @@
 
 require "securerandom"
 require "oj"
+require "yaml"
 
+# TODO: use Oj directly
 Oj.mimic_JSON
 Oj.add_to_json
 
@@ -108,10 +110,6 @@ module Grumlin
     end
   end
 
-  def self.supported_steps
-    @supported_steps ||= (Grumlin::AnonymousStep::SUPPORTED_STEPS + Grumlin::Expressions::U::SUPPORTED_STEPS).sort.uniq
-  end
-
   @pool_mutex = Mutex.new
 
   class << self
@@ -145,6 +143,10 @@ module Grumlin
         Thread.current.thread_variable_set(:grumlin_default_pool, nil)
       end
     end
+
+    def definitions
+      @definitions ||= YAML.safe_load(File.read(File.join(__dir__, "definitions.yml")), symbolize_names: true)
+    end
   end
 end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: grumlin
 version: !ruby/object:Gem::Version
-  version: 0.15.0
+  version: 0.15.4
 platform: ruby
 authors:
 - Gleb Sinyavskiy
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-01-12 00:00:00.000000000 Z
+date: 2022-01-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: async-pool
@@ -96,6 +96,7 @@ files:
 - gremlin_server/tinkergraph-empty.properties
 - grumlin.gemspec
 - lib/async/channel.rb
+- lib/definitions.yml
 - lib/grumlin.rb
 - lib/grumlin/anonymous_step.rb
 - lib/grumlin/bytecode.rb