Don't duplicate stories_path in autoload_paths (#143)

Closes #130
jonspalmer · Dec 10, 2022 · a6ea6c0 · a6ea6c0
1 parent 107f476
commit a6ea6c0
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/lib/view_component/storybook/engine.rb b/lib/view_component/storybook/engine.rb
@@ -14,7 +14,7 @@ class Engine < Rails::Engine
         options.stories_route ||= "/rails/stories"
 
         if options.show_stories
-          options.stories_path ||= defined?(Rails.root) ? Rails.root.join("test/components/stories") : nil
+          options.stories_path ||= defined?(Rails.root) ? Rails.root.join("test/components/stories").to_s : nil
         end
 
         options.stories_title_generator ||= ViewComponent::Storybook.stories_title_generator
@@ -27,7 +27,11 @@ class Engine < Rails::Engine
       initializer "view_component.set_autoload_paths" do |app|
         options = app.config.view_component_storybook
 
-        ActiveSupport::Dependencies.autoload_paths << options.stories_path if options.show_stories && options.stories_path
+        if options.show_stories &&
+           options.stories_path &&
+           ActiveSupport::Dependencies.autoload_paths.exclude?(options.stories_path)
+          ActiveSupport::Dependencies.autoload_paths << options.stories_path
+        end
       end
 
       config.after_initialize do |app|