Update RuboCop configuration and enhance gemspec description. Added exclusions for metrics in RuboCop, improved formatting in the gemspec, and refined generator options for better clarity and usability.

Author: puppe1990

.rubocop.yml

@@ -1,8 +1,39 @@
 AllCops:
   TargetRubyVersion: 3.1
+  NewCops: enable
+  SuggestExtensions: false
 
 Style/StringLiterals:
   EnforcedStyle: double_quotes
 
 Style/StringLiteralsInInterpolation:
   EnforcedStyle: double_quotes
+
+Metrics/BlockLength:
+  Exclude:
+    - "lib/tasks/**/*.rake"
+    - "Rakefile"
+    - "memory_bank_rails.gemspec"
+
+Layout/LineLength:
+  Max: 120
+  Exclude:
+    - "lib/generators/**"
+
+Style/Documentation:
+  Enabled: false
+
+Metrics/MethodLength:
+  Max: 25
+  Exclude:
+    - "lib/generators/**"
+
+Metrics/AbcSize:
+  Max: 26
+  Exclude:
+    - "lib/generators/**"
+
+Metrics/ClassLength:
+  Max: 120
+  Exclude:
+    - "lib/generators/**"

lib/generators/memory_bank/install/install_generator.rb

@@ -9,8 +9,14 @@ module Generators
     class InstallGenerator < Rails::Generators::Base
       source_root File.expand_path("templates", __dir__)
 
-      class_option :guide, type: :string, default: "select", desc: "Which guide to install (or 'select' to choose interactively)"
-      class_option :with_rules, type: :boolean, default: false, desc: "Copy editor rules (.cursorrules, RuboCop, Solargraph) if available"
+      class_option :guide,
+                   type: :string,
+                   default: "select",
+                   desc: "Which guide to install (or 'select' to choose interactively)"
+      class_option :with_rules,
+                   type: :boolean,
+                   default: false,
+                   desc: "Copy editor rules (.cursorrules, RuboCop, Solargraph) if available"
 
       def create_config
         template "config/memory_bank.yml.erb", "config/memory_bank.yml"
@@ -66,9 +72,8 @@ def resolve_guide_source(guide, guides_path)
       def select_guide_interactively(guides_path)
         builtin = available_builtin_guides
         external = available_external_guides(guides_path)
-        options = []
-        builtin.each { |g| options << ["📦 #{g} (builtin)", :builtin, g, nil] }
-        external.each { |g, path| options << ["🔧 #{g} (external)", :external, g, path] }
+        options = builtin.map { |g| ["📦 #{g} (builtin)", :builtin, g, nil] } +
+                  external.map { |g, path| ["🔧 #{g} (external)", :external, g, path] }
 
         if options.empty?
           say "No guides available. Falling back to builtin 'rails_web'", :yellow
@@ -78,11 +83,11 @@ def select_guide_interactively(guides_path)
         say "\n🚀 Memory Bank Initializer (Rails)", :green
         say "=================================\n"
         options.each_with_index do |(label, _kind, _slug, _path), idx|
-          say format("%2d) %s", idx + 1, label)
+          say format("%<num>2d) %<label>s", num: idx + 1, label: label)
         end
         choice = ask("\n? What type of memory bank would you like to install? (1-#{options.size})").to_i
         choice = 1 if choice < 1 || choice > options.size
-        label, kind, slug, full_path = options[choice - 1]
+        _label, kind, slug, full_path = options[choice - 1]
         [slug, kind, full_path]
       end
 
@@ -94,8 +99,10 @@ def available_builtin_guides
 
       def available_external_guides(guides_path)
         return [] unless guides_path
+
         expanded = File.expand_path(guides_path)
         return [] unless Dir.exist?(expanded)
+
         Dir.children(expanded).filter_map do |entry|
           full = File.join(expanded, entry)
           guide_md = File.join(full, "developmentGuide.md")
@@ -112,9 +119,9 @@ def copy_external_guide_files(external_dir)
         FileUtils.cp(source_md, File.join(destination_root, ".memory_bank", "developmentGuide.md"))
 
         rules = File.join(external_dir, ".cursorrules")
-        if options["with_rules"] && File.exist?(rules)
-          FileUtils.cp(rules, File.join(destination_root, ".cursorrules"))
-        end
+        return unless options["with_rules"] && File.exist?(rules)
+
+        FileUtils.cp(rules, File.join(destination_root, ".cursorrules"))
       end
 
       def install_editor_rules
@@ -131,5 +138,3 @@ def install_editor_rules
     end
   end
 end
-
-

lib/memory_bank_rails/cli.rb

@@ -36,12 +36,11 @@ def self.load_rake_tasks
 end
 
 # Backwards compatible alias used in README one-off example
+
 module MemoryBank
   class CLI
     def self.run(command)
       MemoryBankRails::CLI.run(command)
     end
   end
 end
-
-

lib/memory_bank_rails/config.rb

@@ -34,5 +34,3 @@ def self.deep_merge(base, override)
     end
   end
 end
-
-

lib/memory_bank_rails/railtie.rb

@@ -5,7 +5,7 @@
 module MemoryBankRails
   class Railtie < ::Rails::Railtie
     rake_tasks do
-      load File.expand_path('../tasks/memory_bank_rails.rake', __dir__)
+      load File.expand_path("../tasks/memory_bank_rails.rake", __dir__)
     end
 
     # Autoload generators
@@ -16,5 +16,3 @@ class Railtie < ::Rails::Railtie
     end
   end
 end
-
-

lib/memory_bank_rails/services/checker.rb

@@ -16,19 +16,17 @@ def initialize(root_path)
         @root_path = root_path
       end
 
-      def verify!
+      def verify?
         missing = REQUIRED_FILES.reject { |p| File.exist?(File.join(@root_path, p)) }
         if missing.empty?
           puts "Memory Bank: OK"
           true
         else
-          puts "Missing files:" 
+          puts "Missing files:"
           missing.each { |m| puts " - #{m}" }
           false
         end
       end
     end
   end
 end
-
-

lib/memory_bank_rails/services/filesystem.rb

@@ -35,5 +35,3 @@ def file_exists?(relative_path)
     end
   end
 end
-
-

lib/memory_bank_rails/services/initializer.rb

@@ -25,5 +25,3 @@ def initialize_docs
     end
   end
 end
-
-

lib/memory_bank_rails/services/report.rb

@@ -16,8 +16,8 @@ def generate
 
         Dir.glob(File.join(specs_root, "**", "*.md")).each do |file|
           content = File.read(file)
-          total += content.scan(/\- \[.\]/).size
-          done += content.scan(/\- \[x\]/i).size
+          total += content.scan(/- \[.\]/).size
+          done += content.scan(/- \[x\]/i).size
         end
 
         percent = total.zero? ? 0 : ((done.to_f / total) * 100).round(1)
@@ -26,5 +26,3 @@ def generate
     end
   end
 end
-
-

lib/memory_bank_rails/services/spec_generator.rb

@@ -4,6 +4,7 @@
 
 module MemoryBankRails
   module Services
+    # Generates SPEC scaffolding files under `.specs/<feature-slug>`.
     class SpecGenerator
       def initialize(root_path)
         @fs = Filesystem.new(root_path)
@@ -58,5 +59,3 @@ def tasks_template(feature_slug)
     end
   end
 end
-
-