hyper-component 0.99.6 → 1.0.alpha1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 65b78c8ba3d9ce408f7c5b1675a7e0840bdbaebb0d274dcc4f44c59379fd91e9
-  data.tar.gz: a216f84309f2f80f1532528bd805e4693e5fbea667cd55fef082916313419e11
+  metadata.gz: 8f4d73fa137be90a6669fde5e74f3398c33503882a6af53bbcdcedca1ecfcba4
+  data.tar.gz: b2b61ace1614403cc7f682465571d1ae389f716ea84e2aa0edb76b7a34f6f344
 SHA512:
-  metadata.gz: 9cbd1e8b45a0074a42227790018fe5ea707187ca67a8c81ee77fc8799bcec37c3e742d15b38bd4dd8caf4f812fe17683fb73dc2812079a402b7593a97808d4f2
-  data.tar.gz: ffcae57c7ee360d0f72f790acc3b6eb3be8c26b14a6ab1ea4e9fff3621c0e867ece5d07f3e54d12fab5b419fdf5f29b28b4468c6044e20faedf6b610c92e07f1
+  metadata.gz: f0e0166388bd05c172e2925b5179ebc6717b5a25c036eebb81db2d6afb8c2bac50a8ece0cec4755aa0b972957fab466917b7bbcb51603c2b5139e8b0e71ffed1
+  data.tar.gz: 7ad7cf61f431a260ad32f6ce40bf0687339e36c186e9358987de7d034fb974caa94bd23aa301c1512348aad06ed95b5fd994a406071b676244bdb62441204230

data/CHANGELOG.md

@@ -32,7 +32,7 @@ Whitespace conventions:
 ### Deprecated
 
 - Current ref callback behavior is deprecated. Require `"react/ref_callback"` to get the updated behavior. (#188)
-- `React.render_to_string` & `React.render_to_static_markup` is deprecated, use `React::Server.render_to_string` & `React::Server.render_to_static_markup` instead. (#186)
+- `Hyperstack::Component::ReactAPI.render_to_string` & `Hyperstack::Component::ReactAPI.render_to_static_markup` is deprecated, use `React::Server.render_to_string` & `React::Server.render_to_static_markup` instead. (#186)
 - `react/react-source` is deprecated, use `react/react-source-browser` or `react/react-source-server` instead. For most usecase, `react/react-source-browser` is sufficient. If you are using the built-in server side rendering feature, the actual `ReactDOMServer` is already provided by the `react-rails` gem. Therefore, unless you are building a custom server side rendering mechanism, it's not suggested to use `react/react-source-server` in browser code. (#186)
 
 ### Removed
@@ -57,8 +57,8 @@ Whitespace conventions:
 
 ### Deprecated
 
-- Current `React.render` behavior is deprecated. Require `"react/top_level_render"` to get the updated behavior. (#187)
-- `React.is_valid_element` is deprecated in favor of `React.is_valid_element?`.
+- Current `Hyperstack::Component::ReactAPI.render` behavior is deprecated. Require `"react/top_level_render"` to get the updated behavior. (#187)
+- `React.is_valid_element` is deprecated in favor of `Hyperstack::Component::ReactAPI.is_valid_element?`.
 - `expect(component).to render('<div />')` is now deprecated in favor of `expect(component).to render_static_html('<div />')`, which is much clearer.
 
 ### Fixed

data/Gemfile

@@ -1,7 +1,8 @@
 source 'https://rubygems.org'
 #gem "opal-jquery", git: "https://github.com/opal/opal-jquery.git", branch: "master"
-gem 'hyper-spec',       path: '../hyper-spec'
-gem 'hyperloop-config', path: '../hyperloop-config'
-gem 'hyper-store',      path: '../hyper-store'
+gem 'hyper-spec',        path: '../hyper-spec'
+gem 'hyperstack-config', path: '../hyperstack-config'
+gem 'hyper-store',       path: '../hyper-store'
+gem 'hyper-state',       path: '../hyper-state'
 #gem 'puma', '~> 3.11.0' # As of adding, version 3.12.0 isn't working so we are locking
 gemspec

data/Gemfile.lock

@@ -1,12 +1,12 @@
 PATH
   remote: ../hyper-spec
   specs:
-    hyper-spec (0.99.6)
+    hyper-spec (1.0.rc1)
       capybara
       chromedriver-helper (= 1.2.0)
-      libv8 (~> 6.7.0)
+      libv8 (~> 6.3.0)
       method_source
-      mini_racer (~> 0.2.4)
+      mini_racer (~> 0.1.15)
       opal (>= 0.11.0, < 0.12.0)
       parser (>= 2.3.3.1)
       pry
@@ -14,34 +14,44 @@ PATH
       selenium-webdriver
       timecop (~> 0.8.1)
       uglifier
-      unparser (~> 0.3.0)
+      unparser
       webdrivers
 
+PATH
+  remote: ../hyper-state
+  specs:
+    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.99.6)
-      hyperloop-config (= 0.99.6)
+    hyper-store (1.0.rc1)
+      hyper-state (= 1.0.rc1)
+      hyperstack-config (= 1.0.rc1)
       opal (>= 0.11.0, < 0.12.0)
 
 PATH
-  remote: ../hyperloop-config
+  remote: ../hyperstack-config
   specs:
-    hyperloop-config (0.99.6)
-      libv8 (~> 6.7.0)
-      mini_racer (~> 0.2.4)
+    hyperstack-config (1.0.rc1)
+      libv8 (~> 6.3.0)
+      listen (~> 3.0)
+      mini_racer (~> 0.1.15)
       opal (>= 0.11.0, < 0.12.0)
       opal-browser (~> 0.2.0)
       uglifier
+      websocket
 
 PATH
   remote: .
   specs:
-    hyper-component (0.99.6)
-      hyper-store (= 0.99.6)
-      hyperloop-config (= 0.99.6)
-      libv8 (~> 6.7.0)
-      mini_racer (~> 0.2.4)
+    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)
       opal-activesupport (~> 0.3.1)
       react-rails (>= 2.4.0, < 2.5.0)
@@ -94,7 +104,7 @@ GEM
     adamantium (0.2.0)
       ice_nine (~> 0.11.0)
       memoizable (~> 0.4.0)
-    addressable (2.6.0)
+    addressable (2.5.2)
       public_suffix (>= 2.0.2, < 4.0)
     archive-zip (0.11.0)
       io-like (~> 0.3.0)
@@ -105,7 +115,7 @@ GEM
       babel-source (>= 4.0, < 6)
       execjs (~> 2.0)
     builder (3.2.3)
-    capybara (3.13.2)
+    capybara (3.10.1)
       addressable
       mini_mime (>= 0.1.3)
       nokogiri (~> 1.8)
@@ -133,15 +143,16 @@ GEM
     globalid (0.4.1)
       activesupport (>= 4.2.0)
     hike (1.2.3)
-    i18n (1.1.0)
+    i18n (1.1.1)
       concurrent-ruby (~> 1.0)
     ice_nine (0.11.2)
+    interception (0.5)
     io-like (0.3.0)
     jquery-rails (4.3.3)
       rails-dom-testing (>= 1, < 3)
       railties (>= 4.2.0)
       thor (>= 0.14, < 2.0)
-    libv8 (6.7.288.46.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)
@@ -149,9 +160,9 @@ GEM
     loofah (2.2.2)
       crass (~> 1.0.2)
       nokogiri (>= 1.5.9)
-    mail (2.7.0)
+    mail (2.7.1)
       mini_mime (>= 0.1.1)
-    marcel (0.3.2)
+    marcel (0.3.3)
       mimemagic (~> 0.3.2)
     memoizable (0.4.2)
       thread_safe (~> 0.3, >= 0.3.1)
@@ -162,12 +173,11 @@ GEM
     mimemagic (0.3.2)
     mini_mime (1.0.1)
     mini_portile2 (2.3.0)
-    mini_racer (0.2.4)
-      libv8 (>= 6.3)
+    mini_racer (0.1.15)
+      libv8 (~> 6.3)
     minitest (5.11.3)
-    net_http_ssl_fix (0.0.10)
     nio4r (2.3.1)
-    nokogiri (1.8.4)
+    nokogiri (1.8.5)
       mini_portile2 (~> 2.3.0)
     opal (0.11.3)
       ast (>= 2.3.0)
@@ -204,8 +214,11 @@ GEM
     pry (0.11.3)
       coderay (~> 1.1.0)
       method_source (~> 0.9.0)
+    pry-rescue (1.4.5)
+      interception (>= 0.5)
+      pry
     public_suffix (3.0.3)
-    puma (3.11.4)
+    puma (3.12.0)
     rack (2.0.5)
     rack-test (1.1.0)
       rack (>= 1.0, < 3)
@@ -249,16 +262,16 @@ GEM
       execjs
       railties (>= 3.2)
       tilt
-    regexp_parser (1.3.0)
+    regexp_parser (1.2.0)
     rspec-core (3.8.0)
       rspec-support (~> 3.8.0)
-    rspec-expectations (3.8.1)
+    rspec-expectations (3.8.2)
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.8.0)
     rspec-mocks (3.8.0)
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.8.0)
-    rspec-rails (3.8.0)
+    rspec-rails (3.8.1)
       actionpack (>= 3.0)
       activesupport (>= 3.0)
       railties (>= 3.0)
@@ -295,10 +308,10 @@ GEM
     timecop (0.8.1)
     tzinfo (1.2.5)
       thread_safe (~> 0.1)
-    uglifier (4.1.20)
+    uglifier (4.1.19)
       execjs (>= 0.3.0, < 3)
     unicode-display_width (1.4.0)
-    unparser (0.3.0)
+    unparser (0.2.8)
       abstract_type (~> 0.0.7)
       adamantium (~> 0.2.0)
       concord (~> 0.1.5)
@@ -306,11 +319,11 @@ GEM
       equalizer (~> 0.0.9)
       parser (>= 2.3.1.2, < 2.6)
       procto (~> 0.0.2)
-    webdrivers (3.6.0)
-      net_http_ssl_fix
+    webdrivers (3.4.3)
       nokogiri (~> 1.6)
       rubyzip (~> 1.0)
       selenium-webdriver (~> 3.0)
+    websocket (1.2.8)
     websocket-driver (0.7.0)
       websocket-extensions (>= 0.1.0)
     websocket-extensions (0.1.3)
@@ -321,12 +334,13 @@ PLATFORMS
   ruby
 
 DEPENDENCIES
-  bundler (~> 1.16)
+  bundler (~> 1.16.0)
   chromedriver-helper
   hyper-component!
   hyper-spec!
+  hyper-state!
   hyper-store!
-  hyperloop-config!
+  hyperstack-config!
   jquery-rails
   listen
   mime-types
@@ -335,6 +349,7 @@ DEPENDENCIES
   opal-rails (~> 0.9.4)
   opal-rspec
   pry
+  pry-rescue
   puma
   rails (>= 4.0.0)
   rails-controller-testing
@@ -345,4 +360,4 @@ DEPENDENCIES
   timecop (~> 0.8.1)
 
 BUNDLED WITH
-   1.17.3
+   1.16.1

data/{misc/how-component-name-lookup-works.md → how-component-name-lookup-works.md}

@@ -107,7 +107,7 @@ def register_component_dsl_method(component)
   parent = split_name.inject([Module]) { |nesting, next_const| nesting + [nesting.last.const_get(next_const)] }[-2]
   class << parent
     define_method component_name do |*args, &block|
-      React::RenderingContext.render(name, *args, &block)
+      RenderingContext.render(name, *args, &block)
     end
     define_method "#{component_name}_as_node" do |*args, &block|
       React::Component.deprecation_warning("..._as_node is deprecated.  Render component and then use the .node method instead")

data/hyper-component.gemspec

@@ -1,10 +1,10 @@
 # -*- encoding: utf-8 -*-
 $:.push File.expand_path('../lib/', __FILE__)
-require 'reactive-ruby/version'
+require 'hyperstack/component/version'
 
 Gem::Specification.new do |spec|
   spec.name          = 'hyper-component'
-  spec.version       = Hyperloop::Component::VERSION
+  spec.version       = Hyperstack::Component::VERSION
 
   spec.authors       = ['David Chang', 'Adam Jahn', 'Mitch VanDuyn', 'Jan Biedermann', 'Adam Creekroad']
   spec.email         = ['mitch@catprint.com']
@@ -21,17 +21,17 @@ Gem::Specification.new do |spec|
   spec.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")
   spec.require_paths = ['lib']
 
-  spec.add_dependency 'hyper-store', Hyperloop::Component::VERSION
-  spec.add_dependency 'hyperloop-config', Hyperloop::Component::VERSION
-  spec.add_dependency 'libv8', '~> 6.7.0'
-  spec.add_dependency 'mini_racer', '~> 0.2.4'
+  spec.add_dependency 'hyper-state', Hyperstack::Component::VERSION
+  spec.add_dependency 'hyperstack-config', Hyperstack::Component::VERSION
+  spec.add_dependency 'libv8', '~> 6.3.0' # see https://github.com/discourse/mini_racer/issues/92
+  spec.add_dependency 'mini_racer', '~> 0.1.15'
   spec.add_dependency 'opal', '>= 0.11.0', '< 0.12.0'
   spec.add_dependency 'opal-activesupport', '~> 0.3.1'
   spec.add_dependency 'react-rails', '>= 2.4.0', '< 2.5.0'
 
-  spec.add_development_dependency 'bundler', '~> 1.16'
+  spec.add_development_dependency 'bundler', '~> 1.16.0'
   spec.add_development_dependency 'chromedriver-helper'
-  spec.add_development_dependency 'hyper-spec', Hyperloop::Component::VERSION
+  spec.add_development_dependency 'hyper-spec', Hyperstack::Component::VERSION
   spec.add_development_dependency 'jquery-rails'
   spec.add_development_dependency 'listen'
   spec.add_development_dependency 'mime-types'
@@ -40,6 +40,7 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency 'opal-rails', '~> 0.9.4'
   spec.add_development_dependency 'opal-rspec'
   spec.add_development_dependency 'pry'
+  spec.add_development_dependency 'pry-rescue'
   spec.add_development_dependency 'puma'
   spec.add_development_dependency 'rails', '>= 4.0.0'
   spec.add_development_dependency 'rails-controller-testing'

data/lib/hyper-component.rb

@@ -1,52 +1,40 @@
-require 'hyperloop-config'
-Hyperloop.import 'hyper-store'
-Hyperloop.js_import 'react/react-source-browser', client_only: true, defines: ['ReactDOM', 'React']
-Hyperloop.js_import 'react/react-source-server', server_only: true, defines: 'React'
-Hyperloop.import 'browser/delay', client_only: true
-Hyperloop.js_import 'react_ujs', defines: 'ReactRailsUJS'
+require 'hyperstack/internal/component'
+
+Hyperstack.import    'hyper-state'
+Hyperstack.js_import 'react/react-source-browser', client_only: true, defines: %w[ReactDOM React]
+Hyperstack.js_import 'react/react-source-server', server_only: true, defines: 'React'
+Hyperstack.import    'browser/delay', client_only: true
+Hyperstack.js_import 'react_ujs', defines: 'ReactRailsUJS'
+Hyperstack.import    'hyper-component'  # TODO: confirm this does not break anything.  Added while converting hyperloop->hyperstack
+Hyperstack.import    'hyperstack/component/auto-import'  # TODO: confirm we can cancel the import
 
 if RUBY_ENGINE == 'opal'
-  module Hyperloop
-    class Component
-    end
-  end
+  require 'hyperstack/internal/callbacks'
+  require 'hyperstack/internal/auto_unmount'
   require 'native'
-  require 'react/observable'
-  require 'react/validator'
-  require 'react/element'
-  require 'react/api'
-  require 'react/component'
-  require 'react/component/dsl_instance_methods'
-  require 'react/component/should_component_update'
-  require 'react/component/tags'
-  require 'react/component/base'
-  require 'react/event'
-  require 'react/rendering_context'
-  require 'react/state'
-  require 'react/object'
-  require 'react/to_key'
-  #require 'react/ext/opal-jquery/element' # now have to manually require this
-  require 'reactive-ruby/isomorphic_helpers'
-  require 'react/top_level'
-  require 'react/top_level_render'
-  require 'rails-helpers/top_level_rails_component'
-  require 'reactive-ruby/version'
-  module Hyperloop
-    class Component
-      def self.inherited(child)
-        child.include(Mixin)
-      end
-    end
-  end
+  require 'hyperstack/state/observer'
+  require 'hyperstack/internal/component/validator'
+  require 'hyperstack/component/element'
+  require 'hyperstack/internal/component/react_wrapper'
+  require 'hyperstack/component'
+  require 'hyperstack/internal/component/should_component_update'
+  require 'hyperstack/internal/component/tags'
+  require 'hyperstack/component/event'
+  require 'hyperstack/internal/component/rendering_context'
+  require 'hyperstack/ext/component/object'
+  require 'hyperstack/ext/component/number'
+  require 'hyperstack/ext/component/boolean'
+  require 'hyperstack/component/isomorphic_helpers'
+  require 'hyperstack/component/react_api'
+  require 'hyperstack/internal/component/top_level_rails_component'
+  require 'hyperstack/component/version'
 else
   require 'opal'
-
-  require 'hyper-store'
   require 'opal-activesupport'
-  require 'reactive-ruby/version'
-  require 'reactive-ruby/rails' if defined?(Rails)
-  require 'reactive-ruby/isomorphic_helpers'
-  require 'reactive-ruby/serializers'
+  require 'hyperstack/component/version'
+  require 'hyperstack/internal/component/rails'
+  require 'hyperstack/component/isomorphic_helpers'
+  require 'hyperstack/ext/component/serializers'
 
   Opal.append_path File.expand_path('../', __FILE__).untaint
   require 'react/react-source'

data/lib/hyperstack/component.rb

@@ -0,0 +1,145 @@
+require 'hyperstack/ext/component/string'
+require 'hyperstack/ext/component/hash'
+require 'active_support/core_ext/class/attribute'
+require 'hyperstack/internal/auto_unmount'
+require 'hyperstack/internal/component/rendering_context'
+require 'hyperstack/internal/component'
+require 'hyperstack/internal/component/instance_methods'
+require 'hyperstack/internal/component/class_methods'
+require 'hyperstack/internal/component/props_wrapper'
+module Hyperstack
+
+  module Component
+
+    def self.included(base)
+      base.include(Hyperstack::State::Observer)
+      base.include(Hyperstack::Internal::Component::InstanceMethods)
+      base.include(Hyperstack::Internal::AutoUnmount) # pulls in the CallBacks module as well
+      base.include(Hyperstack::Internal::Component::Tags)
+      base.include(Hyperstack::Internal::Component::ShouldComponentUpdate)
+      base.class_eval do
+        class_attribute :initial_state
+        define_callback :before_mount
+        define_callback :after_mount
+        define_callback :before_receive_props
+        define_callback :before_update
+        define_callback :after_update
+        #define_callback :before_unmount defined already by Async module
+        define_callback(:after_error) { Hyperstack::Internal::Component::ReactWrapper.add_after_error_hook(base) }
+      end
+      base.extend(Hyperstack::Internal::Component::ClassMethods)
+    end
+
+    def self.force_update!
+      components = Hyperstack::Internal::Component.mounted_components.to_a
+      components.each do |comp|
+        next unless Hyperstack::Internal::Component.mounted_components.include? comp
+        comp.force_update!
+      end
+    end
+
+    def self.deprecation_warning(message)
+      Hyperstack.deprecation_warning(name, message)
+    end
+
+    def deprecation_warning(message)
+      Hyperstack.deprecation_warning(self.class.name, message)
+    end
+
+    def initialize(native_element)
+      @__hyperstack_component_native = native_element
+    end
+
+    def emit(event_name, *args)
+      if Event::BUILT_IN_EVENTS.include?(built_in_event_name = "on#{event_name.to_s.event_camelize}")
+        params[built_in_event_name].call(*args)
+      else
+        params["on_#{event_name}"].call(*args)
+      end
+    end
+
+    def component_will_mount
+      @__hyperstack_component_params_wrapper = self.class.props_wrapper.new(self)
+      IsomorphicHelpers.load_context(true) if IsomorphicHelpers.on_opal_client?
+      observing(immediate_update: true) do
+        Hyperstack::Internal::Component.mounted_components << self
+        run_callback(:before_mount, props)
+      end
+    end
+
+    def component_did_mount
+      observing(update_objects: true) do
+        run_callback(:after_mount)
+      end
+    end
+
+    def component_will_receive_props(next_props)
+      # need to rethink how this works in opal-react, or if its actually that useful within the react.rb environment
+      # for now we are just using it to clear processed_params
+      observing(immediate_update: true) { run_callback(:before_receive_props, next_props) }
+      @__hyperstack_component_receiving_props = true
+    end
+
+    def component_will_update(next_props, next_state)
+      observing { run_callback(:before_update, next_props, next_state) }
+      if @__hyperstack_component_receiving_props
+        @__hyperstack_component_params_wrapper.reload(next_props)
+      end
+      @__hyperstack_component_receiving_props = false
+    end
+
+    def component_did_update(prev_props, prev_state)
+      observing(update_objects: true) { run_callback(:after_update, prev_props, prev_state) }
+    end
+
+    def component_will_unmount
+      observing do
+        unmount # runs unmount callbacks as well
+        remove
+        Hyperstack::Internal::Component.mounted_components.delete self
+      end
+    end
+
+    def component_did_catch(error, info)
+      observing { run_callback(:after_error, error, info) }
+    end
+
+    def mutations(_objects)
+      # if we have to we may have to require that all objects respond to a "name" method (see legacy method update_react_js_state below)
+      set_state('***_state_updated_at-***' => `Date.now() + Math.random()`)
+    end
+
+    def update_react_js_state(object, name, value)
+      if object
+        name = "#{object.class}.#{name}" unless object == self
+        # Date.now() has only millisecond precision, if several notifications of
+        # observer happen within a millisecond, updates may get lost.
+        # to mitigate this the Math.random() appends some random number
+        # this way notifactions will happen as expected by the rest of hyperloop
+        set_state(
+          '***_state_updated_at-***' => `Date.now() + Math.random()`,
+          name => value
+        )
+      else
+        set_state name => value
+      end
+    end
+
+    def render
+      raise 'no render defined'
+    end unless method_defined?(:render)
+
+    def waiting_on_resources
+      @__hyperstack_component_waiting_on_resources
+    end
+
+    def _render_wrapper
+      observing(rendering: true) do
+        element = Hyperstack::Internal::Component::RenderingContext.render(nil) { render || '' }
+        @__hyperstack_component_waiting_on_resources =
+          element.waiting_on_resources if element.respond_to? :waiting_on_resources
+        element
+      end
+    end
+  end
+end