Derive strict_helpers_enabled from component-local config

Author: boardfish

lib/view_component/base.rb

@@ -228,7 +228,7 @@ def controller
     def helpers
       raise HelpersCalledBeforeRenderError if view_context.nil?
 
-      raise StrictHelperError if ViewComponent::Base.strict_helpers_enabled
+      raise StrictHelperError if !GlobalConfig.helpers_enabled || component_config.strict_helpers_enabled
       # Attempt to re-use the original view_context passed to the first
       # component rendered in the rendering pipeline. This prevents the
       # instantiation of a new view_context via `controller.view_context` which
@@ -520,6 +520,16 @@ def inherited(child)
         # `compile` defines
         compile
 
+        if child.superclass == ViewComponent::Base
+          child.define_singleton_method(:component_config) do
+            @@component_config ||= Rails.application.config.view_component.component_defaults.inheritable_copy
+          end
+        else
+          child.define_singleton_method(:component_config) do
+            @@component_config ||= superclass.component_config.inheritable_copy
+          end
+        end
+
         # Give the child its own personal #render_template_for to protect against the case when
         # eager loading is disabled and the parent component is rendered before the child. In
         # such a scenario, the parent will override ViewComponent::Base#render_template_for,

lib/view_component/config.rb

@@ -26,7 +26,10 @@ def defaults
           test_controller: "ApplicationController",
           default_preview_layout: nil,
           capture_compatibility_patch_enabled: false,
-          helpers_enabled: true
+          helpers_enabled: true,
+          component_defaults: ActiveSupport::InheritableOptions.new({
+            strict_helpers_enabled: false
+          })
         })
       end
 

test/sandbox/test/rendering_test.rb

@@ -1277,9 +1277,12 @@ def test_with_format
       render_inline(MultipleFormatsComponent.new)
 
       assert_equal(rendered_json["hello"], "world")
+    end
   end
 
-  def test_strict_helpers
+  # Since helpers_enabled is true by default, we assume that the converse case
+  # is covered. If that default changes, tests should generally be updated.
+  def test_helpers_proxy_raises_if_helpers_disabled
     with_helpers_enabled_config(false) do
       assert_raises ViewComponent::StrictHelperError do
         render_inline(HelpersProxyComponent.new)
@@ -1313,4 +1316,14 @@ def test_render_anonymous_component_without_template
       render_inline(mock_component.new)
     end
   end
+
+  def test_helpers_proxy_raises_if_strict_helpers_enabled
+    with_helpers_enabled_config(true) do
+      with_strict_helpers_config(true) do
+        assert_raises ViewComponent::StrictHelperError do
+          render_inline(HelpersProxyComponent.new)
+        end
+      end
+    end
+  end
 end

test/test_helper.rb

@@ -179,7 +179,7 @@ def with_compiler_development_mode(mode)
 end
 
 def with_strict_helpers_config(enabled, &block)
-  with_config_option(:strict_helpers_enabled, enabled, &block)
+  with_config_option(:strict_helpers_enabled, enabled, config_entrypoint: Rails.application.config.view_component.component_defaults, &block)
 end
 
 def with_helpers_enabled_config(enabled, &block)