annotaterb 4.14.0 → 4.15.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 915a9aee6e9396080e4b115bc34466f422cc866aa31f64e8563a409f4ed53753
-  data.tar.gz: 30e4511ae77313d4d1f7348599286815a85e56f2c100f31ca73cc7a1feaa59d4
+  metadata.gz: a31463c338bd23299c7249e3c444d8c1b630157bf9905cfeb203dfafb1a4d16a
+  data.tar.gz: ad12fccdeeb0186d6627ba237c417c637b1b6136f354313e81f09de90cef4665
 SHA512:
-  metadata.gz: 8e1c00298956404ba517a54b803f0cae4235da530a1f2dfe6db35e58c3747ba4bc7304759375ecf51237870f2e4701b965192b235e1c430b08bd4d0340bf6dc1
-  data.tar.gz: 63f51a4bbc1ebf2455aeeb8b3a7459a87787b309f5deae0a0e91df971bc28a36f77fa71f7b80a4fbe8f32d3660936fa41576f56569e72526fc2505a77c8fc6df
+  metadata.gz: af6ec6a16d36b65b00eb28f4736c46682aa0d47da4869503eaecb8a58a8ff34f783ac3986c09200f125279666a8b30658af1ba91a9ad5829d789846c1c9e82dd
+  data.tar.gz: ad7fbd08360072a24c2fbea2808c180806ae89da5775689d1f7cc2824fead770b771ac41667056e68da0921ae13948a6f3da48883ab2f0328506cbf7158a63c0

data/CHANGELOG.md

@@ -1,5 +1,31 @@
 # Changelog
 
+## [v4.14.0](https://github.com/drwl/annotaterb/tree/v4.14.0) (2025-02-17)
+
+[Full Changelog](https://github.com/drwl/annotaterb/compare/v4.13.0...v4.14.0)
+
+**Closed issues:**
+
+- How do I annotate models but NOT routes? [\#178](https://github.com/drwl/annotaterb/issues/178)
+- Model Annotation Not Updated When Modifying Table Columns Using change\_table. [\#169](https://github.com/drwl/annotaterb/issues/169)
+- annotate\_rb:install failing on Rails 8 [\#168](https://github.com/drwl/annotaterb/issues/168)
+- Annotations with enums changing between db:create db:migrate and then db:migrate [\#167](https://github.com/drwl/annotaterb/issues/167)
+- JSON parse error [\#155](https://github.com/drwl/annotaterb/issues/155)
+- Feature: Add Support for dynamic fixtures with ERB [\#149](https://github.com/drwl/annotaterb/issues/149)
+
+**Merged pull requests:**
+
+- Bump version to v4.14.0 [\#190](https://github.com/drwl/annotaterb/pull/190) ([drwl](https://github.com/drwl))
+- Add expected file to automatically require [\#185](https://github.com/drwl/annotaterb/pull/185) ([drwl](https://github.com/drwl))
+- Fix translation foreign key exclusion bug [\#181](https://github.com/drwl/annotaterb/pull/181) ([galori](https://github.com/galori))
+- Lock `concurrent-ruby` gem to fix CI [\#180](https://github.com/drwl/annotaterb/pull/180) ([tagliala](https://github.com/tagliala))
+- Chore: alert when multiple conmmands were selected [\#179](https://github.com/drwl/annotaterb/pull/179) ([OdenTakashi](https://github.com/OdenTakashi))
+- Updated COLUMN\_PATTERN to handle optional metadata \(e.g., constraints or descriptions\) enclosed in parentheses. [\#170](https://github.com/drwl/annotaterb/pull/170) ([hatsu38](https://github.com/hatsu38))
+- Opt-in for MFA requirement [\#166](https://github.com/drwl/annotaterb/pull/166) ([tagliala](https://github.com/tagliala))
+- Fix typos [\#165](https://github.com/drwl/annotaterb/pull/165) ([tagliala](https://github.com/tagliala))
+- Add support for virtual columns [\#163](https://github.com/drwl/annotaterb/pull/163) ([robbevp](https://github.com/robbevp))
+- Generate changelog for v4.13.0 [\#160](https://github.com/drwl/annotaterb/pull/160) ([drwl](https://github.com/drwl))
+
 ## [v4.13.0](https://github.com/drwl/annotaterb/tree/v4.13.0) (2024-10-21)
 
 [Full Changelog](https://github.com/drwl/annotaterb/compare/v4.12.0...v4.13.0)
@@ -293,7 +319,7 @@
 - Fix the default behavior for model annotator [\#20](https://github.com/drwl/annotaterb/pull/20) ([drwl](https://github.com/drwl))
 - Refactor model annotator [\#19](https://github.com/drwl/annotaterb/pull/19) ([drwl](https://github.com/drwl))
 - Removed unused helper methods and `Env` class [\#18](https://github.com/drwl/annotaterb/pull/18) ([drwl](https://github.com/drwl))
-- Update dummy app depdencies [\#17](https://github.com/drwl/annotaterb/pull/17) ([drwl](https://github.com/drwl))
+- Update dummy app dependencies [\#17](https://github.com/drwl/annotaterb/pull/17) ([drwl](https://github.com/drwl))
 
 ## [v4.0.0](https://github.com/drwl/annotaterb/tree/v4.0.0) (2023-05-03)
 

data/VERSION

@@ -1 +1 @@
-4.14.0
+4.15.0

data/lib/annotate_rb/eager_loader.rb

@@ -15,10 +15,9 @@ module AnnotateRb
             klass.eager_load!
           end
         else
-          options[:model_dir].each do |dir|
-            ::Rake::FileList["#{dir}/**/*.rb"].each do |fname|
-              require File.expand_path(fname)
-            end
+          model_files = ModelAnnotator::ModelFilesGetter.call(options)
+          model_files&.each do |model_file|
+            require model_file
           end
         end
       end

data/lib/annotate_rb/model_annotator/column_annotation/attributes_builder.rb

@@ -70,10 +70,16 @@ module AnnotateRb
             sorted_column_indices&.each do |index|
               indexed_columns = index.columns.reject { |i| i == @column.name }
 
-              attrs << if indexed_columns.empty?
+              index_text = if index.unique
+                "uniquely indexed"
+              else
                 "indexed"
+              end
+
+              attrs << if indexed_columns.empty?
+                index_text
               else
-                "indexed => [#{indexed_columns.join(", ")}]"
+                "#{index_text} => [#{indexed_columns.join(", ")}]"
               end
             end
           end

data/lib/annotate_rb/model_annotator/model_files_getter.rb

@@ -13,7 +13,8 @@ module AnnotateRb
 
           return model_files if model_files.any?
 
-          options[:model_dir].each do |dir|
+          model_dirs = options[:model_dir].flat_map { |model_dir| Dir[model_dir] }
+          model_dirs.each do |dir|
             Dir.chdir(dir) do
               list = if options[:ignore_model_sub_dir]
                 Dir["*.rb"].map { |f| [dir, f] }
@@ -26,12 +27,14 @@ module AnnotateRb
             end
           end
 
-          model_files
-        rescue SystemCallError
-          warn "No models found in directory '#{options[:model_dir].join("', '")}'."
-          warn "Either specify models on the command line, or use the --model-dir option."
-          warn "Call 'annotaterb --help' for more info."
-          # exit 1 # TODO: Return exit code back to caller. Right now it messes up RSpec being able to run
+          if model_files.empty?
+            warn "No models found in directory '#{options[:model_dir].join("', '")}'."
+            warn "Either specify models on the command line, or use the --model-dir option."
+            warn "Call 'annotaterb --help' for more info."
+            # exit 1 # TODO: Return exit code back to caller. Right now it messes up RSpec being able to run
+          else
+            model_files
+          end
         end
 
         private

data/lib/annotate_rb/model_annotator/model_wrapper.rb

@@ -6,6 +6,8 @@ module AnnotateRb
       # Should be the wrapper for an ActiveRecord model that serves as the source of truth of the model
       # of the model that we're annotating
 
+      DEFAULT_TIMESTAMP_COLUMNS = %w[created_at updated_at]
+
       def initialize(klass, options)
         @klass = klass
         @options = options
@@ -107,6 +109,10 @@ module AnnotateRb
       end
 
       def retrieve_indexes_from_table
+        @indexes_from_table ||= _retrieve_indexes_from_table
+      end
+
+      def _retrieve_indexes_from_table
         table_name = @klass.table_name
         return [] unless table_name
 
@@ -114,7 +120,7 @@ module AnnotateRb
         return indexes if indexes.any? || !@klass.table_name_prefix
 
         # Try to search the table without prefix
-        table_name_without_prefix = table_name.to_s.sub(@klass.table_name_prefix, "")
+        table_name_without_prefix = table_name.to_s.sub(@klass.table_name_prefix.to_s, "")
         begin
           @klass.connection.indexes(table_name_without_prefix)
         rescue ActiveRecord::StatementInvalid => _e
@@ -143,10 +149,13 @@ module AnnotateRb
         associations = []
         id = nil
 
+        # specs don't load defaults, so ensure we have defaults here
+        timestamp_columns = @options[:timestamp_columns] || DEFAULT_TIMESTAMP_COLUMNS
+
         cols.each do |c|
           if c.name.eql?("id")
             id = c
-          elsif c.name.eql?("created_at") || c.name.eql?("updated_at")
+          elsif timestamp_columns.include?(c.name)
             timestamps << c
           elsif c.name[-3, 3].eql?("_id")
             associations << c
@@ -154,7 +163,10 @@ module AnnotateRb
             rest_cols << c
           end
         end
-        [rest_cols, timestamps, associations].each { |a| a.sort_by!(&:name) }
+
+        timestamp_order = timestamp_columns.each_with_index.to_h
+        timestamps.sort_by! { |col| timestamp_order[col.name] }
+        [rest_cols, associations].each { |a| a.sort_by!(&:name) }
 
         ([id] << rest_cols << timestamps << associations).flatten.compact
       end

data/lib/annotate_rb/model_annotator/pattern_getter.rb

@@ -64,7 +64,10 @@ module AnnotateRb
       def get
         current_patterns = []
 
-        @options[:root_dir].each do |root_directory|
+        root_dirs = @options[:root_dir].flat_map do |root_dir|
+          root_dir.empty? ? root_dir : Dir[root_dir]
+        end
+        root_dirs.each do |root_directory|
           Array(@pattern_types).each do |pattern_type|
             patterns = generate(root_directory, pattern_type)
 

data/lib/annotate_rb/model_annotator/project_annotator.rb

@@ -34,6 +34,7 @@ module AnnotateRb
       private
 
       def build_instructions_for_file(file)
+        start = Time.now
         klass = ModelClassGetter.call(file, @options)
 
         instructions = []
@@ -52,6 +53,10 @@ module AnnotateRb
         end
         instructions.concat(related_file_instructions)
 
+        if @options[:debug]
+          puts "Built instructions for #{file} in #{Time.now - start}s"
+        end
+
         instructions
       end
 

data/lib/annotate_rb/options.rb

@@ -69,6 +69,9 @@ module AnnotateRb
       # ModelAnnotator
       hide_limit_column_types: "",
 
+      # ModelAnnotator
+      timestamp_columns: ModelAnnotator::ModelWrapper::DEFAULT_TIMESTAMP_COLUMNS,
+
       ignore_columns: nil, # ModelAnnotator
       ignore_routes: nil, # RouteAnnotator
       ignore_unknown_models: false, # ModelAnnotator
@@ -130,6 +133,7 @@ module AnnotateRb
       :debug,
       :hide_default_column_types,
       :hide_limit_column_types,
+      :timestamp_columns,
       :ignore_columns,
       :ignore_routes,
       :ignore_unknown_models,

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: annotaterb
 version: !ruby/object:Gem::Version
-  version: 4.14.0
+  version: 4.15.0
 platform: ruby
 authors:
 - Andrew W. Lee
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2025-02-17 00:00:00.000000000 Z
+date: 2025-05-30 00:00:00.000000000 Z
 dependencies: []
 description: Annotates Rails/ActiveRecord Models, routes, fixtures, and others based
   on the database schema.
@@ -120,7 +120,7 @@ licenses:
 metadata:
   homepage_uri: https://github.com/drwl/annotaterb
   source_code_uri: https://github.com/drwl/annotaterb
-  changelog_uri: https://github.com/drwl/annotaterb/blob/master/CHANGELOG.md
+  changelog_uri: https://github.com/drwl/annotaterb/blob/main/CHANGELOG.md
   bug_tracker_uri: https://github.com/drwl/annotaterb/issues
   rubygems_mfa_required: 'true'
 post_install_message: