Option for loading files at package root (#24)

Rails conventions for app/controllers, app/domain etc are not very
useful for projects defining a large domain and utilizing packs to
organize code.

This change allows adding application code to the eager load path
located at the root of where the package.yml is defined giving the user
full control of the file structure.

previously you could have a subpacks that looked like this:

```
- packs/
-   accounting/
-     package.yml
-     app/
-       public/
-         api.rb
-       domain/
-         subprocess/
-           package.yml
-           run_subprocess.rb
-         subprocess2/
-           package.yml
-           run_subprocess.rb

```

The subpack works fine, but all of the dependencies have to reference
into the rails code structure, i.e:

```
dependencies:
  - packs/accounting/app/domain/subprocess
```

Specifying autoload_glob allows you to have complete control for
structuring your pack.

```
-   accounting/
-     package.yml
-     app/
-       public/
-         api.rb
-     subprocess/
-       package.yml
-       run_subprocess.rb
-     subprocess2/
-       package.yml
-       run_subprocess.rb

```

Dependencies can referece the pack by:

```
dependencies:
  - packs/accounting/subprocess
```

.github/workflows/cd.yml

@@ -13,7 +13,7 @@ jobs:
     # https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#running-a-workflow-based-on-the-conclusion-of-another-workflow
     if: ${{ github.event.workflow_run.conclusion == 'success' }}
     steps:
-      - uses: actions/checkout@ac593985615ec2ede58e132d2e21d2b1cbd6127c
+      - uses: actions/checkout@cbb722410c2e876e24abbe8de2cc27693e501dcb
       - name: Tag and Push Gem
         id: tag-and-push-gem
         # This action basically runs `bundle exec rake release` if there is not an existing tag in GitHub

.github/workflows/ci.yml

@@ -10,7 +10,6 @@ jobs:
     strategy:
       matrix:
         ruby:
-          - 2.7
           # Due to https://github.com/actions/runner/issues/849, we have to use quotes for '3.0'
           - '3.0'
           - 3.1

.ruby-version

@@ -0,0 +1 @@
+3.3.4

Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    automatic_namespaces (0.4.0)
+    automatic_namespaces (0.5.0)
       activesupport
       packs-rails
 
@@ -98,6 +98,7 @@ GEM
     marcel (1.0.2)
     method_source (1.0.0)
     mini_mime (1.1.2)
+    mini_portile2 (2.8.8)
     minitest (5.16.3)
     net-imap (0.3.1)
       net-protocol
@@ -107,10 +108,9 @@ GEM
       timeout
     net-smtp (0.3.3)
       net-protocol
-    nio4r (2.5.8)
-    nokogiri (1.13.9-arm64-darwin)
-      racc (~> 1.4)
-    nokogiri (1.13.9-x86_64-linux)
+    nio4r (2.7.4)
+    nokogiri (1.13.9)
+      mini_portile2 (~> 2.8.0)
       racc (~> 1.4)
     packs (0.0.6)
       sorbet-runtime
@@ -196,6 +196,7 @@ GEM
 
 PLATFORMS
   arm64-darwin-21
+  arm64-darwin-24
   x86_64-linux
 
 DEPENDENCIES

README.md

@@ -57,7 +57,7 @@ If bundler is not being used to manage dependencies, install the gem by executin
 
 Given the `package.yml` of a strongly namespaced pack:
 
-```
+```yml
 enforce_dependencies: true
 enforce_privacy: true
 public_path: app/public/
@@ -72,7 +72,7 @@ metadata:
 
 modify the metadata to opt into automatic namespacing:
 
-```
+```yml
 metadata:
  automatic_pack_namespace: true
 ```
@@ -99,7 +99,7 @@ metadata:
 If your package / namespace name requires ActiveSupport inflections, you will probably need to tell `automatic_namespaces`
 what the correct namespace name should be in that package:
 
-```
+```yml
 # packs/shoes_ui/package.yml
 metadata:
   automatic_pack_namespace: true
@@ -109,6 +109,16 @@ metadata:
 This is necessary because `automatic_namespaces` works by modifying the autoloader paths, which has to 
 happen during Rails application initialization; but the inflector is not available for use then.
 
+If you would like to use your own file layout conventions for packs (i.e. not `app/*`) you can specify 
+your own glob by using `autoload_glob` to append the glob to the folder containing package.yml. This defaults
+to `'/**/app/*'` 
+
+```yml
+metadata:
+  # Put the folder containing package.yml as the root for the autoloader.
+  autoload_glob: ''
+```
+
 ## Development
 
 After checking out the repo, run `bundle install` to install dependencies. Then, run `rspec` to run the tests. 

lib/automatic_namespaces/autoloader.rb

@@ -27,7 +27,8 @@ def set_namespace_for_dir(pack_dir, package_namespace)
   end
 
   def pack_directories(pack_root_dir, metadata)
-    Dir.glob("#{pack_root_dir}/**/app/*").select { |dir| namespaced_directory?(dir, metadata) }
+    glob = metadata['autoload_glob'] || "/**/app/*"
+    Dir.glob("#{pack_root_dir}#{glob}").select { |dir| namespaced_directory?(dir, metadata) }
   end
 
   def namespaced_directory?(dir, metadata)

lib/automatic_namespaces/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module AutomaticNamespaces
-  VERSION = "0.4.0"
+  VERSION = "0.5.1"
 end

spec/automatic_namespaces_spec.rb

@@ -49,4 +49,10 @@
       expect(defined?(Sneaker)).to eq("constant")
     end
   end
+
+  it "can find classes that are located at the root of the package.yml" do
+    expect(defined?(Summer::Fun)).to eq("constant")
+    expect(defined?(Summer::Swim)).to eq("constant")
+    expect(defined?(Summer::Swim::Trunks)).to eq("constant")
+  end
 end

spec/fixtures/rails-7.0/packs/summer/app/domain/fun.rb

@@ -0,0 +1 @@
+module Summer::Fun; end

spec/fixtures/rails-7.0/packs/summer/package.yml

@@ -0,0 +1,2 @@
+metadata:
+  automatic_pack_namespace: true

spec/fixtures/rails-7.0/packs/summer/swim/package.yml

@@ -0,0 +1,4 @@
+metadata:
+  automatic_pack_namespace: true
+  namespace_override: "Summer::Swim"
+  autoload_glob: ''

spec/fixtures/rails-7.0/packs/summer/swim/trunks.rb

@@ -0,0 +1 @@
+class Summer::Swim::Trunks; end