added :none naming_convention and moved it to Hyperstack module

Author: catmando

ruby/examples/misc/stock-tickers/app/hyperstack/components/hyper_component.rb

@@ -1,11 +1,5 @@
-module Hyperstack
-  module State
-    module Observable
-      def self.naming_convention
-        :prefix_params
-      end
-    end
-  end
+def Hyperstack.naming_convention
+  :prefix_params
 end
 
 class HyperComponent

ruby/hyper-component/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: ../hyper-spec
   specs:
-    hyper-spec (0.1)
+    hyper-spec (1.0.rc1)
       capybara
       chromedriver-helper (= 1.2.0)
       libv8 (~> 6.3.0)
@@ -20,22 +20,22 @@ PATH
 PATH
   remote: ../hyper-state
   specs:
-    hyper-state (0.1)
-      hyperstack-config (= 0.1)
+    hyper-state (1.0.rc1)
+      hyperstack-config (= 1.0.rc1)
       opal (>= 0.11.0, < 0.12.0)
 
 PATH
   remote: ../hyper-store
   specs:
-    hyper-store (0.1)
-      hyper-state (= 0.1)
-      hyperstack-config (= 0.1)
+    hyper-store (1.0.rc1)
+      hyper-state (= 1.0.rc1)
+      hyperstack-config (= 1.0.rc1)
       opal (>= 0.11.0, < 0.12.0)
 
 PATH
   remote: ../hyperstack-config
   specs:
-    hyperstack-config (0.1)
+    hyperstack-config (1.0.rc1)
       libv8 (~> 6.3.0)
       listen (~> 3.0)
       mini_racer (~> 0.1.15)
@@ -47,9 +47,9 @@ PATH
 PATH
   remote: .
   specs:
-    hyper-component (0.1)
-      hyper-state (= 0.1)
-      hyperstack-config (= 0.1)
+    hyper-component (1.0.rc1)
+      hyper-state (= 1.0.rc1)
+      hyperstack-config (= 1.0.rc1)
       libv8 (~> 6.3.0)
       mini_racer (~> 0.1.15)
       opal (>= 0.11.0, < 0.12.0)
@@ -115,13 +115,14 @@ GEM
       babel-source (>= 4.0, < 6)
       execjs (~> 2.0)
     builder (3.2.3)
-    capybara (3.9.0)
+    capybara (3.10.1)
       addressable
       mini_mime (>= 0.1.3)
       nokogiri (~> 1.8)
       rack (>= 1.6.0)
       rack-test (>= 0.6.3)
-      xpath (~> 3.1)
+      regexp_parser (~> 1.2)
+      xpath (~> 3.2)
     childprocess (0.9.0)
       ffi (~> 1.0, >= 1.0.11)
     chromedriver-helper (1.2.0)
@@ -151,7 +152,7 @@ GEM
       rails-dom-testing (>= 1, < 3)
       railties (>= 4.2.0)
       thor (>= 0.14, < 2.0)
-    libv8 (6.3.292.48.1)
+    libv8 (6.3.292.48.1-x86_64-darwin-15)
     listen (3.1.5)
       rb-fsevent (~> 0.9, >= 0.9.4)
       rb-inotify (~> 0.9, >= 0.9.7)
@@ -261,6 +262,7 @@ GEM
       execjs
       railties (>= 3.2)
       tilt
+    regexp_parser (1.2.0)
     rspec-core (3.8.0)
       rspec-support (~> 3.8.0)
     rspec-expectations (3.8.2)
@@ -288,7 +290,7 @@ GEM
     ruby-progressbar (1.10.0)
     ruby_dep (1.5.0)
     rubyzip (1.2.2)
-    selenium-webdriver (3.14.1)
+    selenium-webdriver (3.141.0)
       childprocess (~> 0.5)
       rubyzip (~> 1.2, >= 1.2.2)
     sourcemap (0.1.1)

ruby/hyper-component/lib/hyperstack/internal/component/props_wrapper.rb

@@ -7,7 +7,7 @@ class PropsWrapper
 
         class << self
           def instance_var_name_for(name)
-            case Hyperstack::State::Observable.naming_convention
+            case Hyperstack.naming_convention
             when :camelize_params
               name.camelize
             when :prefix_params

ruby/hyper-state/lib/hyperstack/state/observable.rb

@@ -5,10 +5,6 @@ def self.bulk_update(&block)
         Internal::State::Mapper.bulk_update(&block)
       end
 
-      def self.naming_convention
-        :camelize_params
-      end
-
       def self.included(base)
         base.include Internal::AutoUnmount
         %i[singleton_method method].each do |kind|
@@ -28,26 +24,26 @@ def self.included(base)
           end
           if RUBY_ENGINE == 'opal'
             base.send(:"define_#{kind}", :set) do |var|
-              if Hyperstack::State::Observable.naming_convention == :prefix_state
+              if Hyperstack.naming_convention == :prefix_state
                 var = "_#{var}" if var !~ /^_/
-              elsif var =~ /^[a-z]/
-                add_mutate = true
+              elsif Hyperstack.naming_convention != :none && var !~ /^[a-z]/
+                dont_mutate = true
               end
               lambda do |val|
                 `self[#{var}] = #{val}`
-                mutate if add_mutate
+                mutate unless dont_mutate
               end
             end
           else
             base.send(:"define_#{kind}", :set) do |var|
-              if Hyperstack::State::Observable.naming_convention == :prefix_state
+              if Hyperstack.naming_convention == :prefix_state
                 var = "_#{var}" if var !~ /^_/
-              elsif var =~ /^[a-z]/
-                add_mutate = true
+              elsif Hyperstack.naming_convention != :none && var !~ /^[a-z]/
+                dont_mutate = true
               end
               lambda do |val|
                 instance_variable_set(:"@#{var}", val)
-                mutate if add_mutate
+                mutate unless dont_mutate
               end
             end
           end
@@ -67,7 +63,7 @@ def self.included(base)
           end
           base.singleton_class.send(:"define_#{kind}", :state_reader) do |*names|
             names.each do |name|
-              var_name = if Hyperstack::State::Observable.naming_convention == :prefix_state && name !~ /^_/
+              var_name = if Hyperstack.naming_convention == :prefix_state && name !~ /^_/
                            "_#{name}"
                          else
                            name
@@ -79,7 +75,7 @@ def self.included(base)
           end
           base.singleton_class.send(:"define_#{kind}", :state_writer) do |*names|
             names.each do |name|
-              var_name = if Hyperstack::State::Observable.naming_convention == :prefix_state && name !~ /^_/
+              var_name = if Hyperstack.naming_convention == :prefix_state && name !~ /^_/
                            "_#{name}"
                          else
                            name

ruby/hyperstack-config/lib/hyperstack-config.rb

@@ -1,4 +1,9 @@
 require 'hyperstack/boot'
+module Hyperstack
+  def self.naming_convention
+    :camelize_params
+  end
+end
 if RUBY_ENGINE == 'opal'
   require 'hyperstack/deprecation_warning'
   require 'hyperstack/string'