gempilot 0.2.4 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ed510959d35011586d23fc0dcc0baff3a453a2acbd1a2d1c8fb4489cee63e6e0
-  data.tar.gz: 455ba876df7b3fb40df26932737cae4062a6e6e2210f1fd02b18a231566bd23d
+  metadata.gz: 72e379c74f157a19076c82e54b15e2d3581610d11089542ebd32b346971cfdb5
+  data.tar.gz: e7b8f296a83f2c4b9e24f4bb3fd7757a034b1cca00fa461f4baa3e5c878b6f51
 SHA512:
-  metadata.gz: 4ce5185ea5c07adc72435dee47b2f749f4f6146f26a09c8d8c557490e4be1b69b2b048e4c9ab32b8548d043c9a0d393cc32a0d8dd23c94c98cabd3a8737334d9
-  data.tar.gz: 6094227a45e9bb173f0455dc155bd3ac7593698a7c41c7ac1482c5e7c44a59a90c547bb20206d3c7887b87a84977fc1f7ca615b2975fe53dc6a8ab85384a83e8
+  metadata.gz: bcb608a185c3808045cf017dcf0629d9b6034be93fd8457129728abed9cfd5191288893c214b3b38a46fdbda267676cdb7976ec2013c053a4b7796cd551a9c73
+  data.tar.gz: 867f02bab2332352a5035db336f5b1073f5d4a4f08f49f75b76b84eb2fdd4d490dba4c9f36fed4b60d0a7844791509dce074d3f13d488b08000e16bc0c70bdc8

data/.rubocop.yml

@@ -279,17 +279,3 @@ RSpec/NamedSubject:
 # have_file_content matcher accepts a filename string in expect()
 RSpec/ExpectActual:
   Enabled: false
-
-Design/PositionalArguments:
-  Enabled: false
-
-Design/KeywordArguments:
-  Enabled: false
-
-Design/TotalArguments:
-  Enabled: false
-
-Design/AgentNounClassName:
-  Exclude:
-    - 'lib/gempilot/generator.rb'
-    - 'test/gempilot/cli/generator_test.rb'

data/data/templates/gem/Rakefile.erb

@@ -18,16 +18,8 @@ RuboCop::RakeTask.new
 require "gempilot/version_task"
 Gempilot::VersionTask.new
 
-namespace :zeitwerk do
-  desc "Verify all files follow Zeitwerk naming conventions"
-  task :validate do
-    ruby "-e", <<~RUBY
-      require '<%= @require_path %>'
-      <%= @module_name %>::LOADER.eager_load(force: true)
-      puts 'Zeitwerk: All files loaded successfully.'
-    RUBY
-  end
-end
+require "gempilot/zeitwerk_task"
+Gempilot::ZeitwerkTask.new
 
 <% if @test_framework == :minitest -%>
 task default: [:test, :rubocop]

data/data/templates/gem/lib/gem_name.rb.erb

@@ -6,8 +6,7 @@ require "zeitwerk"
 ##
 # Top-level namespace for the <%= @gem_name %> gem.
 module <%= @module_name %>
-  LOADER = Zeitwerk::Loader.for_gem
-  LOADER.setup
+  LOADER = Zeitwerk::Loader.for_gem.tap(&:setup)
 
   ##
   # Base error class for <%= @gem_name %>.

data/data/templates/gem/lib/gem_name_extension.rb.erb

@@ -9,8 +9,7 @@ require "zeitwerk"
 <% end -%>
 <%= "  " * i %>module <%= part %>
 <% end -%>
-<%= "  " * @module_parts.size %>LOADER = Zeitwerk::Loader.for_gem_extension(<%= @base_module %>)
-<%= "  " * @module_parts.size %>LOADER.setup
+<%= "  " * @module_parts.size %>LOADER = Zeitwerk::Loader.for_gem_extension(<%= @base_module %>).tap(&:setup)
 
 <%= "  " * @module_parts.size %>##
 <%= "  " * @module_parts.size %># Base error class for <%= @gem_name %>.

data/data/templates/gem/test/test_helper.rb.erb

@@ -1,7 +1,10 @@
 ENV["MT_NO_PLUGINS"] = "1" unless ENV["RM_INFO"]
 
 require "minitest/autorun"
-require "minitest/reporters"
-Minitest::Reporters.use! [Minitest::Reporters::DefaultReporter.new(color: true)]
+
+unless ENV["RM_INFO"]
+  require "minitest/reporters"
+  Minitest::Reporters.use! [Minitest::Reporters::DefaultReporter.new(color: true)]
+end
 
 require "<%= @require_path %>"

data/issues.rec

@@ -95,7 +95,7 @@ Id: 4EB6E51C-4322-11F1-B1B2-FE6CB9572C2F
 Updated: Tue, 28 Apr 2026 12:50:05 -0400
 Title: Existing git repo on gempilot create needs new commit message
 Description: Currently it commits "Initial commit". Inappropriate for a git repo that already has a history
-Status: open
+Status: closed
 
 Id: 0411D4DA-4323-11F1-847C-FE6CB9572C2F
 Updated: Tue, 28 Apr 2026 12:55:09 -0400
@@ -110,21 +110,21 @@ Description: e.g.
 +  Zeitwerk::Loader.for_gem.tap do |l|
 +     l.setup
 +   end
-+ 
-+ 
-Status: open
++
++
+Status: closed
 
 Id: 5618BB20-4325-11F1-BF5A-FE6CB9572C2F
 Updated: Tue, 28 Apr 2026 13:11:46 -0400
 Title: Move zeitwerk check task into gempilot
 Description: Can't iterate on it when its not managed by gempilot. A new feature needs to be added
-Status: open
+Status: closed
 
 Id: 6BE168E4-4325-11F1-BE0F-FE6CB9572C2F
 Updated: Tue, 28 Apr 2026 13:12:22 -0400
 Title: Add zeitwerk:all task
 Description: Add rake task that runs LOADER.all_expected_cpaths and prints in a nice table, similar to rails routes
-Status: open
+Status: closed
 
 Id: 56DB9634-4327-11F1-8D0F-FE6CB9572C2F
 Updated: Fri, 05 Jun 2026 00:00:00 -0400
@@ -155,13 +155,13 @@ Description: When doing `gempilot new Reversal::Server`, in a project called "re
 +         create  test/reversal/store_test.rb
 + 
 + The test file is incorrectly named. Should be server_test.rb
-Status: open
+Status: closed
 
 Id: F5486B48-71DB-11F1-980F-FE6CB9572C2E
 Updated: Fri, 26 Jun 2026 23:54:55 -0400
 Title: Rubymine <=> Minitest usage causes "no reporters allowed error"
 Description: Has to do with how the ENV var RM_INFO is being interpreted
-Status: open
+Status: closed
 
 Id: 8D847572-7273-11F1-B508-FE6CB9572C2E
 Updated: Fri, 03 Jul 2026 22:38:22 +0000

data/lib/gempilot/cli/gem_builder.rb

@@ -97,9 +97,21 @@ module Gempilot
         cd @gem_name do
           sh "git", "init", "-q", "-b", @branch
           sh "git", "add", "."
-          sh "git", "commit", "-q", "-m", "Initial commit."
+          sh "git", "commit", "-q", "-m", commit_message
         end
       end
+
+      # "Initial commit." is wrong when scaffolding into a directory that is
+      # already a repository with history (e.g. a cloned remote), so name the
+      # commit after the gem in that case.
+      def commit_message
+        existing_history? ? "Add #{@gem_name} gem scaffolding." : "Initial commit."
+      end
+
+      def existing_history?
+        system("git", "rev-parse", "--verify", "--quiet", "HEAD",
+               out: File::NULL, err: File::NULL)
+      end
     end
   end
 end

data/lib/gempilot/gem_constant.rb

@@ -30,18 +30,14 @@ module Gempilot
 
     ## Path to the constant's source file, e.g. +lib/my_gem/services/auth.rb+.
     def lib_path
-      "#{File.join("lib", *path_segments)}.rb"
+      path_for("lib", ".rb")
     end
 
     ## Path to the constant's test file for +framework+ (+:rspec+ or
-    ## +:minitest+); correct for multi-segment (hyphenated) gem modules.
+    ## +:minitest+). Mirrors +lib_path+ so the test always tracks the source,
+    ## including for multi-segment (hyphenated) gem modules.
     def test_path(framework)
-      rest = path_segments.drop(require_path.split("/").length)
-      if framework == :rspec
-        "#{File.join("spec", require_path, *rest)}_spec.rb"
-      else
-        "#{File.join("test", require_path, *rest)}_test.rb"
-      end
+      framework == :rspec ? path_for("spec", "_spec.rb") : path_for("test", "_test.rb")
     end
 
     private
@@ -57,5 +53,9 @@ module Gempilot
     def path_segments
       parts.map(&:underscore)
     end
+
+    def path_for(root, suffix)
+      "#{File.join(root, *path_segments)}#{suffix}"
+    end
   end
 end

data/lib/gempilot/project.rb

@@ -33,8 +33,16 @@ module Gempilot
       project_segments.join("-")
     end
 
+    def require_path
+      project_segments.join("/")
+    end
+
+    def module_name
+      project_segments.map(&:camelize).join("::")
+    end
+
     def klass
-      Object.const_get(project_segments.map(&:camelize).join("::"))
+      Object.const_get(module_name)
     end
 
     def version

data/lib/gempilot/version.rb

@@ -1,3 +1,3 @@
 module Gempilot
-  VERSION = "0.2.4".freeze
+  VERSION = "0.3.0".freeze
 end

data/lib/gempilot/zeitwerk_task.rb

@@ -0,0 +1,61 @@
+require "rake/tasklib"
+require_relative "../gempilot"
+
+module Gempilot
+  ## Rake tasks for validating and inspecting a gem's Zeitwerk loader.
+  ##
+  ## Owned by gempilot and consumed by generated gems via
+  ## <tt>require "gempilot/zeitwerk_task"; Gempilot::ZeitwerkTask.new</tt>, so
+  ## the logic rolls forward on a gempilot bump instead of being copied into
+  ## every gem's Rakefile. Each task boots a clean child process so eager
+  ## loading surfaces naming errors without polluting the Rake process.
+  class ZeitwerkTask < Rake::TaskLib
+    attr_reader :project
+
+    def initialize(root: Dir.pwd)
+      super()
+      @project = Project.new(root)
+      define_tasks
+    end
+
+    private
+
+    def define_tasks
+      namespace :zeitwerk do
+        define_validate_task
+        define_all_task
+      end
+    end
+
+    def define_validate_task
+      desc "Verify all files follow Zeitwerk naming conventions"
+      task(:validate) { ruby "-Ilib", "-e", validate_script }
+    end
+
+    def define_all_task
+      desc "List every constant Zeitwerk manages and the file it expects"
+      task(:all) { ruby "-Ilib", "-e", all_script }
+    end
+
+    def loader
+      "#{project.module_name}::LOADER"
+    end
+
+    def validate_script
+      <<~RUBY
+        require '#{project.require_path}'
+        #{loader}.eager_load(force: true)
+        puts 'Zeitwerk: All files loaded successfully.'
+      RUBY
+    end
+
+    def all_script
+      <<~RUBY
+        require '#{project.require_path}'
+        rows = #{loader}.all_expected_cpaths.sort_by(&:last)
+        width = rows.map { |_path, cpath| cpath.length }.max || 0
+        rows.each { |path, cpath| puts format("%-\#{width}s  %s", cpath, path) }
+      RUBY
+    end
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: gempilot
 version: !ruby/object:Gem::Version
-  version: 0.2.4
+  version: 0.3.0
 platform: ruby
 authors:
 - David Gillis
@@ -134,6 +134,7 @@ files:
 - lib/gempilot/version.rb
 - lib/gempilot/version_tag.rb
 - lib/gempilot/version_task.rb
+- lib/gempilot/zeitwerk_task.rb
 - vendor/vendored.gemv
 homepage: https://github.com/gillisd/gempilot
 licenses: