inertia_rails 3.3.0 → 3.5.0

data/lib/inertia_rails/generators/scaffold_template_base.rb

@@ -0,0 +1,45 @@
+# frozen_string_literal: true
+
+require 'rails/generators/resource_helpers'
+require_relative 'controller_template_base'
+
+module InertiaRails
+  module Generators
+    class ScaffoldTemplateBase < ControllerTemplateBase
+      include Rails::Generators::ResourceHelpers
+
+      remove_argument :actions
+
+      argument :attributes, type: :array, default: [], banner: 'field:type field:type'
+
+      def copy_view_files
+        available_views.each do |view|
+          template "#{options.frontend_framework}/#{view}.#{template_extension}",
+                   File.join(base_path, "#{view}.#{extension}")
+        end
+
+        template "#{options.frontend_framework}/#{partial_name}.#{template_extension}",
+                 File.join(base_path, "#{inertia_component_name}.#{extension}")
+
+        template "#{options.frontend_framework}/types.ts", File.join(base_path, 'types.ts') if typescript?
+      end
+
+      private
+
+      def template_extension
+        return extension unless typescript?
+        return 'tsx' if options.frontend_framework == 'react'
+
+        "ts.#{extension}"
+      end
+
+      def available_views
+        %w[Index Edit Show New Form]
+      end
+
+      def partial_name
+        'One'
+      end
+    end
+  end
+end

data/lib/inertia_rails/inertia_rails.rb

@@ -1,20 +1,26 @@
-# Needed for `thread_mattr_accessor`
-require 'active_support/core_ext/module/attribute_accessors_per_thread'
-require 'inertia_rails/lazy'
+require 'inertia_rails/base_prop'
+require 'inertia_rails/always_prop'
+require 'inertia_rails/lazy_prop'
 require 'inertia_rails/configuration'
 
 module InertiaRails
-  CONFIGURATION = Configuration.default
+  class << self
+    CONFIGURATION = Configuration.default
 
-  def self.configure
-    yield(CONFIGURATION)
-  end
+    def configure
+      yield(CONFIGURATION)
+    end
 
-  def self.configuration
-    CONFIGURATION
-  end
+    def configuration
+      CONFIGURATION
+    end
+
+    def lazy(value = nil, &block)
+      LazyProp.new(value, &block)
+    end
 
-  def self.lazy(value = nil, &block)
-    InertiaRails::Lazy.new(value, &block)
+    def always(&block)
+      AlwaysProp.new(&block)
+    end
   end
 end

data/lib/inertia_rails/lazy_prop.rb

@@ -0,0 +1,20 @@
+# frozen_string_literal: true
+
+module InertiaRails
+  class LazyProp < BaseProp
+    def initialize(value = nil, &block)
+      raise ArgumentError, 'You must provide either a value or a block, not both' if value && block
+
+      @value = value
+      @block = block
+    end
+
+    def call(controller)
+      value.respond_to?(:call) ? controller.instance_exec(&value) : value
+    end
+
+    def value
+      @value.nil? ? @block : @value
+    end
+  end
+end

data/lib/inertia_rails/renderer.rb

@@ -1,9 +1,14 @@
+# frozen_string_literal: true
+
 require 'net/http'
 require 'json'
 require_relative "inertia_rails"
 
 module InertiaRails
   class Renderer
+    KEEP_PROP = :keep
+    DONT_KEEP_PROP = :dont_keep
+
     attr_reader(
       :component,
       :configuration,
@@ -64,24 +69,30 @@ module InertiaRails
     # Functionally, this permits using either string or symbol keys in the controller. Since the results
     # is cast to json, we should treat string/symbol keys as identical.
     def merge_props(shared_data, props)
-      shared_data.deep_symbolize_keys.send(@deep_merge ? :deep_merge : :merge, props.deep_symbolize_keys)
+      if @deep_merge
+        shared_data.deep_symbolize_keys.deep_merge!(props.deep_symbolize_keys)
+      else
+        shared_data.symbolize_keys.merge(props.symbolize_keys)
+      end
     end
 
     def computed_props
-      _props = merge_props(shared_data, props).select do |key, prop|
-        if rendering_partial_component?
-          key.in? partial_keys
+      _props = merge_props(shared_data, props)
+
+      deep_transform_props _props do |prop, path|
+        next [DONT_KEEP_PROP] unless keep_prop?(prop, path)
+
+        transformed_prop = case prop
+        when BaseProp
+          prop.call(controller)
+        when Proc
+          controller.instance_exec(&prop)
         else
-          !prop.is_a?(InertiaRails::Lazy)
+          prop
         end
-      end
 
-      deep_transform_values(
-        _props,
-        lambda do |prop|
-          prop.respond_to?(:call) ? controller.instance_exec(&prop) : prop
-        end
-      )
+        [KEEP_PROP, transformed_prop]
+      end
     end
 
     def page
@@ -93,18 +104,32 @@ module InertiaRails
       }
     end
 
-    def deep_transform_values(hash, proc)
-      return proc.call(hash) unless hash.is_a? Hash
+    def deep_transform_props(props, parent_path = [], &block)
+      props.reduce({}) do |transformed_props, (key, prop)|
+        current_path = parent_path + [key]
 
-      hash.transform_values {|value| deep_transform_values(value, proc)}
+        if prop.is_a?(Hash) && prop.any?
+          nested = deep_transform_props(prop, current_path, &block)
+          transformed_props.merge!(key => nested) unless nested.empty?
+        else
+          action, transformed_prop = block.call(prop, current_path)
+          transformed_props.merge!(key => transformed_prop) if action == KEEP_PROP
+        end
+
+        transformed_props
+      end
     end
 
     def partial_keys
-      (@request.headers['X-Inertia-Partial-Data'] || '').split(',').compact.map(&:to_sym)
+      (@request.headers['X-Inertia-Partial-Data'] || '').split(',').compact
+    end
+
+    def partial_except_keys
+      (@request.headers['X-Inertia-Partial-Except'] || '').split(',').compact
     end
 
     def rendering_partial_component?
-      @request.inertia_partial? && @request.headers['X-Inertia-Partial-Component'] == component
+      @request.headers['X-Inertia-Partial-Component'] == component
     end
 
     def resolve_component(component)
@@ -112,5 +137,34 @@ module InertiaRails
 
       configuration.component_path_resolver(path: controller.controller_path, action: controller.action_name)
     end
+
+    def keep_prop?(prop, path)
+      return true if prop.is_a?(AlwaysProp)
+
+      if rendering_partial_component?
+        path_with_prefixes = path_prefixes(path)
+        return false if excluded_by_only_partial_keys?(path_with_prefixes)
+        return false if excluded_by_except_partial_keys?(path_with_prefixes)
+      end
+
+      # Precedence: Evaluate LazyProp only after partial keys have been checked
+      return false if prop.is_a?(LazyProp) && !rendering_partial_component?
+
+      true
+    end
+
+    def path_prefixes(parts)
+      (0...parts.length).map do |i|
+        parts[0..i].join('.')
+      end
+    end
+
+    def excluded_by_only_partial_keys?(path_with_prefixes)
+      partial_keys.present? && (path_with_prefixes & partial_keys).empty?
+    end
+
+    def excluded_by_except_partial_keys?(path_with_prefixes)
+      partial_except_keys.present? && (path_with_prefixes & partial_except_keys).any?
+    end
   end
 end

data/lib/inertia_rails/version.rb

@@ -1,3 +1,3 @@
 module InertiaRails
-  VERSION = "3.3.0"
+  VERSION = "3.5.0"
 end

data/lib/patches/better_errors.rb

@@ -4,14 +4,16 @@
 # https://github.com/BetterErrors/better_errors/blob/v2.5.1/lib/better_errors/middleware.rb
 #
 
-if defined?(BetterErrors)
-  BetterErrors::Middleware.class_eval do
-    prepend(InertiaBetterErrors = Module.new do
-      def text?(env)
-        return false if env["HTTP_X_INERTIA"]
+module InertiaRails
+  module InertiaBetterErrors
+    def text?(env)
+      return false if env["HTTP_X_INERTIA"]
 
-        super
-      end
-    end)
+      super
+    end
   end
 end
+
+if defined?(BetterErrors)
+  BetterErrors::Middleware.include InertiaRails::InertiaBetterErrors
+end

data/lib/patches/debug_exceptions/patch-5-0.rb

@@ -4,8 +4,8 @@
 # https://github.com/rails/rails/blob/5-0-stable/actionpack/lib/action_dispatch/middleware/debug_exceptions.rb
 #
 
-ActionDispatch::DebugExceptions.class_eval do
-  prepend(InertiaDebugExceptions = Module.new do
+module InertiaRails
+  module InertiaDebugExceptions
     def render_for_default_application(request, wrapper)
       template = create_template(request, wrapper)
       file = "rescues/#{wrapper.rescue_template}"
@@ -19,5 +19,9 @@ ActionDispatch::DebugExceptions.class_eval do
       end
       render(wrapper.status_code, body, format)
     end
-  end)
+  end
+end
+
+if defined?(ActionDispatch::DebugExceptions)
+  ActionDispatch::DebugExceptions.prepend InertiaRails::InertiaDebugExceptions
 end

data/lib/patches/debug_exceptions/patch-5-1.rb

@@ -6,8 +6,8 @@
 # https://github.com/rails/rails/blob/6-0-stable/actionpack/lib/action_dispatch/middleware/debug_exceptions.rb
 #
 
-ActionDispatch::DebugExceptions.class_eval do
-  prepend(InertiaDebugExceptions = Module.new do
+module InertiaRails
+  module InertiaDebugExceptions
     def render_for_browser_request(request, wrapper)
       template = create_template(request, wrapper)
       file = "rescues/#{wrapper.rescue_template}"
@@ -22,5 +22,9 @@ ActionDispatch::DebugExceptions.class_eval do
 
       render(wrapper.status_code, body, format)
     end
-  end)
+  end
+end
+
+if defined?(ActionDispatch::DebugExceptions)
+  ActionDispatch::DebugExceptions.prepend InertiaRails::InertiaDebugExceptions
 end

data/lib/patches/mapper.rb

@@ -1,8 +1,12 @@
-ActionDispatch::Routing::Mapper.class_eval do
-  def inertia(args, &block)
-    route = args.keys.first
-    component = args.values.first
+module InertiaRails
+  module InertiaMapper
+    def inertia(args, &block)
+      route = args.keys.first
+      component = args.values.first
 
-    get(route => 'inertia_rails/static#static', defaults: {component: component})
+      get(route => 'inertia_rails/static#static', defaults: { component: component })
+    end
   end
 end
+
+ActionDispatch::Routing::Mapper.include InertiaRails::InertiaMapper

data/lib/patches/request.rb

@@ -1,9 +1,13 @@
-ActionDispatch::Request.class_eval do
-  def inertia?
-    key? 'HTTP_X_INERTIA'
-  end
+module InertiaRails
+  module InertiaRequest
+    def inertia?
+      key? 'HTTP_X_INERTIA'
+    end
 
-  def inertia_partial?
-    key? 'HTTP_X_INERTIA_PARTIAL_DATA'
+    def inertia_partial?
+      key?('HTTP_X_INERTIA_PARTIAL_COMPONENT')
+    end
   end
 end
+
+ActionDispatch::Request.include InertiaRails::InertiaRequest