grumlin 0.15.3 → 0.15.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b44c62fb657fac1ce08d4a4dcb05456a2750284dddcd5adbbd20f56bb2b9ef9c
-  data.tar.gz: eccc2648799e68be2225ce606a06411b49953ce586e4b8249156dc680fc3522e
+  metadata.gz: e796fc1b5bff2cb474d068ad2a05a2b8a195df1981842745d5c90798ab5e02b7
+  data.tar.gz: 39cd9ac6bbedfb6a8707d306c6a6e0223856028162778bb665865e387dcf2bc3
 SHA512:
-  metadata.gz: dfc9f36a53d49749425df0014005e18ddd47b91e5c35bdc4dd4c7dfc0ec3309ba15df4e8a3367a8dba4c92494a2f70cab9fe4a846ded833100dc09937bf4ff9b
-  data.tar.gz: d315dfb2945c27091f76a21af2e30f77eb8753042519cf2bd7813ef544cb805f4970808158a550e5abdc1891c1526d4c3441bda46334bae67e2bf49a8ff9bd87
+  metadata.gz: 072df70e88f7192272cad5c6127740ef10d52b5057f42a557e9f4ea722e55f53d222025e7b4d7607666d4c5038d3e630abb55faec7f05c640efed53d442a2b57
+  data.tar.gz: 32ac57244365458e56c9b2c2705500c87254ba523454817b67f88637e07444d5e0b2abfa361540693f0dedf578b7e5f346c8a2515e536c69ad86e5ee09fa7bf3

data/CHANGELOG.md

@@ -1,3 +1,8 @@
+## [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.

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    grumlin (0.15.3)
+    grumlin (0.15.4)
       async-pool (~> 0.3)
       async-websocket (~> 0.19)
       oj (~> 3.12)

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 identity in inE inV is label limit
-                         map 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

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 identity 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.3"
+  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.3
+  version: 0.15.4
 platform: ruby
 authors:
 - Gleb Sinyavskiy
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-01-18 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