motion 0.4.0 → 0.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 54eb6afb0f21c0333d46c1a21602739146fa1e2a021d3ffda048c9763dc85e66
-  data.tar.gz: c3d9476c0e34869f5d4e46121316dee4f2c78d91b0a9b51d05c14a0b4d4e63ac
+  metadata.gz: 579385926d481a6978aa354e54e7b63cdd92f8bb37dc60b49be0f1530371c0d5
+  data.tar.gz: 9d37a416852746fb7b75fd74a3ff972419baddf822b1283c42b86b18be3a0b64
 SHA512:
-  metadata.gz: b17d8159ba7eae3b144a6380385e68eb9a2b49b0733926d9e285bf60c8f0767646d30841ee90d62d4b3a359f5c5f9beb62d4793c6d48dff6f1887c9fd7367a70
-  data.tar.gz: 0c47ed993c63378f0d562e341a44539bda1578c34d42ae8449c1dfd9dedabf7481e9e5f272c0869416893533fe31b5901368ebcc2e53c0752db120afb0e84c50
+  metadata.gz: 0bcc311420b91f6479e1a8e9e45680e9854f5f719dc74ef7f25eeda5595e960f3d1082bf84d5b42c6e1bd01b61c24f788ef34df4322bee95d8162e4b732c08d1
+  data.tar.gz: d3d8ead03222c53a801116cc20197860cad5bd8aaa5534b728cc37a32ca00b231fbafdee1fa6e649f004f9b897eef1826a5b8c5d8317d1b3fd11da5586776b43

data/lib/generators/motion/templates/motion.js

@@ -1,5 +1,5 @@
-import { createClient } from '@unabridged/motion';
-import consumer from './channels/consumer';
+import { createClient } from '@unabridged/motion'
+import consumer from './channels/consumer'
 
 export default createClient({
 
@@ -11,7 +11,7 @@ export default createClient({
   // Motion can log information about the lifecycle of components to the
   // browser's console. It is recommended to turn this feature off outside of
   // development.
-  logging: process.env["RAILS_ENV"] === "development",
+  logging: process.env.RAILS_ENV === 'development'
 
   // This function will be called for every motion, and the return value will be
   // made available at `Motion::Event#extra_data`:
@@ -30,8 +30,8 @@ export default createClient({
   // The data attributes used by Motion can be customized, but these values must
   // also be updated in the Ruby initializer:
   //
-  //    keyAttribute: "data-motion-key",
-  //    stateAttribute: "data-motion-state",
-  //    motionAttribute: "data-motion",
+  //    keyAttribute: 'data-motion-key',
+  //    stateAttribute: 'data-motion-state',
+  //    motionAttribute: 'data-motion',
 
-});
+})

data/lib/motion.rb

@@ -19,41 +19,43 @@ module Motion
   autoload :Serializer, "motion/serializer"
   autoload :TestHelpers, "motion/test_helpers"
 
-  def self.configure(&block)
-    raise AlreadyConfiguredError if @config
-
-    @config = Configuration.new(&block)
-  end
-
-  def self.config
-    @config ||= Configuration.default
-  end
-
-  singleton_class.alias_method :configuration, :config
-
-  def self.serializer
-    @serializer ||= Serializer.new
-  end
-
-  def self.markup_transformer
-    @markup_transformer ||= MarkupTransformer.new
-  end
-
-  def self.build_renderer_for(websocket_connection)
-    config.renderer_for_connection_proc.call(websocket_connection)
-  end
-
-  def self.notify_error(error, message)
-    config.error_notification_proc&.call(error, message)
-  end
-
-  # This method only exists for testing. Changing configuration while Motion is
-  # in use is not supported. It is only safe to call this method when no
-  # components are currently mounted.
-  def self.reset_internal_state_for_testing!(new_configuration = nil)
-    @config = new_configuration
-    @serializer = nil
-    @markup_transformer = nil
+  class << self
+    def configure(&block)
+      raise AlreadyConfiguredError if @config
+
+      @config = Configuration.new(&block)
+    end
+
+    def config
+      @config ||= Configuration.default
+    end
+
+    alias_method :configuration, :config
+
+    def serializer
+      @serializer ||= Serializer.new
+    end
+
+    def markup_transformer
+      @markup_transformer ||= MarkupTransformer.new
+    end
+
+    def build_renderer_for(websocket_connection)
+      config.renderer_for_connection_proc.call(websocket_connection)
+    end
+
+    def notify_error(error, message)
+      config.error_notification_proc&.call(error, message)
+    end
+
+    # This method only exists for testing. Changing configuration while Motion
+    # is in use is not supported. It is only safe to call this method when no
+    # components are currently mounted.
+    def reset_internal_state_for_testing!(new_configuration = nil)
+      @config = new_configuration
+      @serializer = nil
+      @markup_transformer = nil
+    end
   end
 end
 

data/lib/motion/channel.rb

@@ -66,7 +66,7 @@ module Motion
 
     def synchronize
       component_connection.if_render_required do |component|
-        transmit(renderer.render(component))
+        transmit(renderer.render(component, layout: nil))
       end
 
       streaming_from component_connection.broadcasts,

data/lib/motion/component/broadcasts.rb

@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
 require "active_support/concern"
-require "active_support/core_ext/class/attribute"
 require "active_support/core_ext/object/to_param"
 require "active_support/core_ext/hash/except"
 
@@ -12,6 +11,9 @@ module Motion
     module Broadcasts
       extend ActiveSupport::Concern
 
+      DEFAULT = {}.freeze
+      private_constant :DEFAULT
+
       # Analogous to `module_function` (available on both class and instance)
       module ModuleFunctions
         def stream_from(broadcast, handler)
@@ -37,14 +39,6 @@ module Motion
         end
       end
 
-      included do
-        class_attribute :_broadcast_handlers,
-          instance_reader: false,
-          instance_writer: false,
-          instance_predicate: false,
-          default: {}.freeze
-      end
-
       class_methods do
         include ModuleFunctions
 
@@ -56,6 +50,15 @@ module Motion
           serialize_broadcasting([name, model])
         end
 
+        attr_writer :_broadcast_handlers
+
+        def _broadcast_handlers
+          return @_broadcast_handlers if defined?(@_broadcast_handlers)
+          return superclass._broadcast_handlers if superclass.respond_to?(:_broadcast_handlers)
+
+          DEFAULT
+        end
+
         private
 
         # This definition is copied from ActionCable::Channel::Broadcasting

data/lib/motion/component/lifecycle.rb

@@ -105,8 +105,7 @@ module Motion
 
         run_callbacks(:action, &block)
       ensure
-        # `@_action_callback_context = nil` would still appear in the state
-        remove_instance_variable(:@_action_callback_context)
+        @_action_callback_context = nil
       end
     end
   end

data/lib/motion/component/motions.rb

@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
 require "active_support/concern"
-require "active_support/core_ext/class/attribute"
 require "active_support/core_ext/hash/except"
 
 require "motion"
@@ -11,6 +10,9 @@ module Motion
     module Motions
       extend ActiveSupport::Concern
 
+      DEFAULT = {}.freeze
+      private_constant :DEFAULT
+
       # Analogous to `module_function` (available on both class and instance)
       module ModuleFunctions
         def map_motion(motion, handler = motion)
@@ -28,16 +30,17 @@ module Motion
         end
       end
 
-      included do
-        class_attribute :_motion_handlers,
-          instance_reader: false,
-          instance_writer: false,
-          instance_predicate: false,
-          default: {}.freeze
-      end
-
       class_methods do
         include ModuleFunctions
+
+        attr_writer :_motion_handlers
+
+        def _motion_handlers
+          return @_motion_handlers if defined?(@_motion_handlers)
+          return superclass._motion_handlers if superclass.respond_to?(:_motion_handlers)
+
+          DEFAULT
+        end
       end
 
       include ModuleFunctions

data/lib/motion/component/periodic_timers.rb

@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
 require "active_support/concern"
-require "active_support/core_ext/class/attribute"
 require "active_support/core_ext/hash/except"
 
 require "motion"
@@ -11,6 +10,9 @@ module Motion
     module PeriodicTimers
       extend ActiveSupport::Concern
 
+      DEFAULT = {}.freeze
+      private_constant :DEFAULT
+
       # Analogous to `module_function` (available on both class and instance)
       module ModuleFunctions
         def every(interval, handler, name: handler)
@@ -32,16 +34,17 @@ module Motion
         end
       end
 
-      included do
-        class_attribute :_periodic_timers,
-          instance_reader: false,
-          instance_writer: false,
-          instance_predicate: false,
-          default: {}.freeze
-      end
-
       class_methods do
         include ModuleFunctions
+
+        attr_writer :_periodic_timers
+
+        def _periodic_timers
+          return @_periodic_timers if defined?(@_periodic_timers)
+          return superclass._periodic_timers if superclass.respond_to?(:_periodic_timers)
+
+          DEFAULT
+        end
       end
 
       include ModuleFunctions

data/lib/motion/component/rendering.rb

@@ -5,27 +5,37 @@ require "motion"
 module Motion
   module Component
     module Rendering
-      # Use the presence/absence of the ivar instead of true/false to avoid
-      # extra serialized state (Note that in this scheme, the presence of
-      # the ivar will never be serialized).
-      RERENDER_MARKER_IVAR = :@__awaiting_forced_rerender__
-      private_constant :RERENDER_MARKER_IVAR
-
-      # Some changes to Motion's state are specifically supported during render.
-      ALLOWED_NEW_IVARS_DURING_RENDER = %i[
-        @_broadcast_handlers
-        @_stable_instance_identifier_for_callbacks
-        @_motion_handlers
-        @_periodic_timers
+      STATE_EXCLUDED_IVARS = %i[
+        @_action_callback_context
+        @_awaiting_forced_rerender
+        @_routes
+
+        @view_context
+        @lookup_context
+        @view_renderer
+        @view_flow
+        @virtual_path
+        @variant
+        @current_template
+        @output_buffer
+
+        @helpers
+        @controller
+        @request
+        @tag_builder
+
+        @asset_resolver_strategies
+        @assets_environment
       ].freeze
-      private_constant :ALLOWED_NEW_IVARS_DURING_RENDER
+
+      private_constant :STATE_EXCLUDED_IVARS
 
       def rerender!
-        instance_variable_set(RERENDER_MARKER_IVAR, true)
+        @_awaiting_forced_rerender = true
       end
 
       def awaiting_forced_rerender?
-        instance_variable_defined?(RERENDER_MARKER_IVAR)
+        @_awaiting_forced_rerender
       end
 
       # * This can be overwritten.
@@ -43,10 +53,10 @@ module Motion
           _run_action_callbacks(context: :render) {
             _clear_awaiting_forced_rerender!
 
-            view_context.capture { _without_new_instance_variables { super } }
+            view_context.capture { super }
           }
 
-        raise RenderAborted, self if html == false
+        raise RenderAborted, self unless html
 
         Motion.markup_transformer.add_state_to_html(self, html)
       end
@@ -54,21 +64,19 @@ module Motion
       private
 
       def _clear_awaiting_forced_rerender!
-        return unless awaiting_forced_rerender?
-
-        remove_instance_variable(RERENDER_MARKER_IVAR)
+        @_awaiting_forced_rerender = false
       end
 
-      def _without_new_instance_variables
-        existing_instance_variables = instance_variables
+      def marshal_dump
+        (instance_variables - STATE_EXCLUDED_IVARS)
+          .map { |ivar| [ivar, instance_variable_get(ivar)] }
+          .to_h
+      end
 
-        yield
-      ensure
-        (
-          instance_variables -
-          existing_instance_variables -
-          ALLOWED_NEW_IVARS_DURING_RENDER
-        ).each(&method(:remove_instance_variable))
+      def marshal_load(instance_variables)
+        instance_variables.each do |ivar, value|
+          instance_variable_set(ivar, value)
+        end
       end
     end
   end

data/lib/motion/component_connection.rb

@@ -41,7 +41,7 @@ module Motion
     end
 
     def process_motion(motion, event = nil)
-      timing("Proccessed #{motion}") do
+      timing("Processed #{motion}") do
         component.process_motion(motion, event)
       end
 
@@ -53,7 +53,7 @@ module Motion
     end
 
     def process_broadcast(broadcast, message)
-      timing("Proccessed broadcast to #{broadcast}") do
+      timing("Processed broadcast to #{broadcast}") do
         component.process_broadcast broadcast, message
       end
 
@@ -65,7 +65,7 @@ module Motion
     end
 
     def process_periodic_timer(timer)
-      timing("Proccessed periodic timer #{timer}") do
+      timing("Processed periodic timer #{timer}") do
         component.process_periodic_timer timer
       end
 

data/lib/motion/event.rb

@@ -18,7 +18,7 @@ module Motion
       raw["type"]
     end
 
-    alias name type
+    alias_method :name, :type
 
     def details
       raw.fetch("details", {})
@@ -34,14 +34,12 @@ module Motion
       @target = Motion::Element.from_raw(raw["target"])
     end
 
-    def current_target
-      return @current_target if defined?(@current_target)
+    def element
+      return @element if defined?(@element)
 
-      @current_target = Motion::Element.from_raw(raw["currentTarget"])
+      @element = Motion::Element.from_raw(raw["element"])
     end
 
-    alias element current_target
-
     def form_data
       element&.form_data
     end

data/lib/motion/serializer.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 require "digest"
+require "lz4-ruby"
 require "active_support/message_encryptor"
 
 require "motion"
@@ -37,7 +38,7 @@ module Motion
     end
 
     def serialize(component)
-      state = dump(component)
+      state = deflate(dump(component))
       state_with_revision = "#{revision}#{NULL_BYTE}#{state}"
 
       [
@@ -49,7 +50,7 @@ module Motion
     def deserialize(serialized_component)
       state_with_revision = decrypt_and_verify(serialized_component)
       serialized_revision, state = state_with_revision.split(NULL_BYTE, 2)
-      component = load(state)
+      component = load(inflate(state))
 
       if revision == serialized_revision
         component
@@ -70,6 +71,14 @@ module Motion
       Marshal.load(state)
     end
 
+    def deflate(dumped_component)
+      LZ4.compress(dumped_component)
+    end
+
+    def inflate(deflated_state)
+      LZ4.uncompress(deflated_state)
+    end
+
     def encrypt_and_sign(cleartext)
       encryptor.encrypt_and_sign(cleartext)
     end

data/lib/motion/test_helpers.rb

@@ -16,14 +16,18 @@ module Motion
       component.motions.include?(motion_name.to_s)
     end
 
-    def run_motion(component, motion_name)
+    def run_motion(component, motion_name, event = motion_event)
       if block_given?
         c = component.dup
-        c.process_motion(motion_name.to_s)
+        c.process_motion(motion_name.to_s, event)
         yield c
       else
-        component.process_motion(motion_name.to_s)
+        component.process_motion(motion_name.to_s, event)
       end
     end
+
+    def motion_event(attributes = {})
+      Motion::Event.new(ActiveSupport::JSON.decode(attributes.to_json))
+    end
   end
 end

data/lib/motion/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Motion
-  VERSION = "0.4.0"
+  VERSION = "0.5.0"
 end

metadata

@@ -1,15 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: motion
 version: !ruby/object:Gem::Version
-  version: 0.4.0
+  version: 0.5.0
 platform: ruby
 authors:
 - Alec Larsen
 - Drew Ulmer
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-07-16 00:00:00.000000000 Z
+date: 2021-03-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: nokogiri
@@ -31,14 +31,28 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '5.2'
+        version: '5.1'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '5.2'
+        version: '5.1'
+- !ruby/object:Gem::Dependency
+  name: lz4-ruby
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.3.3
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.3.3
 description: |
   Motion extends Github's `view_component` to allow you to build reactive,
   real-time frontend UI components in your Rails application using pure Ruby.
@@ -88,7 +102,7 @@ metadata:
   source_code_uri: https://github.com/unabridged/motion
 post_install_message: |
   Friendly reminder: When updating the motion gem, don't forget to update the
-  NPM package as well (`bin/yarn add '@unabridged/motion@0.4.0'`).
+  NPM package as well (`bin/yarn add '@unabridged/motion@0.5.0'`).
 rdoc_options: []
 require_paths:
 - lib
@@ -96,15 +110,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.3.0
+      version: 2.5.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
-signing_key: 
+rubygems_version: 3.1.2
+signing_key:
 specification_version: 4
 summary: Reactive frontend UI components for Rails in pure Ruby.
 test_files: []