do not ship DocsBuilderComponent or YARD::MattrAccessorHandler (#2274)

* do not ship DocsBuilderComponent or YARD::MattrAccessorHandler

I noticed that we were including these two objects in our
published gems, despite them only being used for building docs.

* add changelog

* use more explicit files glob

Author: joelhawksley

Rakefile

@@ -101,8 +101,8 @@ namespace :docs do
     require "rails"
     require "action_controller"
     require "view_component"
-    ViewComponent::Base.config.view_component_path = "view_component"
-    require "view_component/docs_builder_component"
+    ViewComponent::Base.config.view_component_path = "docs"
+    require "docs/docs_builder_component"
 
     error_keys = registry.keys.select { |key| key.to_s.include?("Error::MESSAGE") }.map(&:to_s)
 

docs/CHANGELOG.md

@@ -10,13 +10,17 @@ nav_order: 6
 
 ## main
 
+* Do not include internal `DocsBuilderComponent` or `YARD::MattrAccessorHandler` in published gem.
+
+    *Joel Hawksley*
+
 * Only lock to `concurrent-ruby` `1.3.4` for Rails 6.1.
 
     *Joel Hawksley*
 
 * Fix generation of ViewComponent documentation that was broken due to HTML safety issues.
 
-  *Simon Fish*
+    *Simon Fish*
 
 * Add documentation on how ViewComponent works.
 

lib/view_component/docs_builder_component.html.erb renamed to lib/docs/docs_builder_component.html.erb

@@ -24,7 +24,15 @@ Gem::Specification.new do |spec|
       "public gem pushes."
   end
 
-  spec.files = Dir["LICENSE.txt", "README.md", "app/**/*", "docs/CHANGELOG.md", "lib/**/*"]
+  spec.files = Dir[
+    "LICENSE.txt",
+    "README.md",
+    "app/**/*",
+    "docs/CHANGELOG.md",
+    "lib/rails/**/*.rb",
+    "lib/view_component.rb",
+    "lib/view_component/**/*.rb",
+  ]
   spec.require_paths = ["lib"]
 
   spec.required_ruby_version = ">= 2.7.0"