archspec 0.1.0.pre1 → 0.1.0.pre2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 14c27bcdf86ac400321503fdb1f2a5b904068aafaac65921e00a33a80acea45f
-  data.tar.gz: fca658eec0f2f6becfc7159c989ba8ad132c3de0d39c5dfb59561f249afb2407
+  metadata.gz: e3056236344cd4f0ba0b95312353e866c6186313878d0be016f96191e04bd011
+  data.tar.gz: f8fbeea8100183e9258206e227700fcf1a8a6027e53bd57970f495be29addbc7
 SHA512:
-  metadata.gz: 4b51c1666d0ec2b27913e796be1969f81b030582b2faf18f9d18f29a1badd7ae1f4b68f2d196a94e976d16fc40e217699a899c6460e037e6a413a68a6aac2299
-  data.tar.gz: 8853a35c37af66fd6f18d2aaace5c6e44a22fccd941cff3b00fae5ec739184451329cd639d9cba730a3f925b2a66c232c0e85dbd6b5122c85c91437a17515e1e
+  metadata.gz: 20640ab2eb6f87cbd911f5270c3b2b6af99ea7841c3ead3c6c5aee51601bafcf4c5e835867a749edf1e59a89100a345693eda407a69f850b7b7ceed2d556adcb
+  data.tar.gz: 2369c0d26ba6c88fb13d0e7e837bbb8fadffc4391a5ccc7239b59f895683d22c79079a9f6d4f1034aff6882b5c1ab9bf21d68c2f42d0d95c140950f657dd8cb3

data/README.md

@@ -39,6 +39,15 @@ ArchSpec.define "Application architecture" do
 end
 ```
 
+Go vanilla, 37signals style — rich models, no service objects:
+
+```ruby
+ArchSpec.define "Application architecture" do
+  root "."
+  preset :vanilla_rails
+end
+```
+
 Add layers when the app has a clear direction of dependencies:
 
 ```ruby
@@ -124,6 +133,7 @@ end
 - **Protocols:** required methods such as `resolve`, `perform`, or project-specific interfaces
 - **Objects:** rules against one-shot `Something.new(...).whatever` command objects
 - **Zeitwerk names:** conventional file names defining the expected constants
+- **Empty components:** directories that must stay empty, like `app/services` in vanilla Rails
 - **Suppressions:** narrow local exceptions with a reason
 
 ## Installation

data/exe/archspec

@@ -1,7 +1,8 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 
-$LOAD_PATH.unshift File.expand_path("../lib", __dir__)
+$LOAD_PATH.unshift File.expand_path('../lib', __dir__)
 
-require "archspec"
+require 'archspec'
 
 exit ArchSpec::CLI.run(ARGV)

data/lib/archspec/analyzer.rb

@@ -1,6 +1,7 @@
-require "prism"
-require "pathname"
-require "set"
+# frozen_string_literal: true
+
+require 'prism'
+require 'pathname'
 
 module ArchSpec
   module Analyzer
@@ -34,7 +35,7 @@ module ArchSpec
       definition.analysis_patterns.flat_map do |pattern|
         Dir.glob(File.absolute_path(pattern, root))
       end.select do |path|
-        File.file?(path) && path.end_with?(".rb")
+        File.file?(path) && path.end_with?('.rb')
       end.map do |path|
         File.expand_path(path)
       end.uniq.reject do |path|
@@ -58,6 +59,8 @@ module ArchSpec
           Regexp.last_match(1)
         when %r{\Alib/(.+)\.rb\z}
           Regexp.last_match(1)
+        when %r{\A(?:packs|engines)/[^/]+/app/[^/]+/concerns/(.+)\.rb\z}
+          Regexp.last_match(1)
         when %r{\A(?:packs|engines)/[^/]+/app/[^/]+/(.+)\.rb\z}
           Regexp.last_match(1)
         else
@@ -68,9 +71,9 @@ module ArchSpec
     end
 
     def camelize_path(path)
-      path.split("/").map do |part|
-        part.split("_").map { |word| word[0] ? word[0].upcase + word[1..] : word }.join
-      end.join("::")
+      path.split('/').map do |part|
+        part.split('_').map { |word| word[0] ? word[0].upcase + word[1..] : word }.join
+      end.join('::')
     end
 
     def suppressions_for(comments)
@@ -94,7 +97,7 @@ module ArchSpec
         active = Hash.new { |hash, key| hash[key] = [] }
 
         sorted_comments(comments).each do |comment|
-          text = comment.slice.sub(/\A#\s?/, "").strip
+          text = comment.slice.sub(/\A#\s?/, '').strip
           line = comment.location.start_line
 
           if (match = text.match(DISABLE_PATTERN))
@@ -102,9 +105,9 @@ module ArchSpec
             rule = normalize_rule(rule)
 
             case mode
-            when "line"
+            when 'line'
               suppressions << Suppression.new(rule, line, line, reason)
-            when "next-line"
+            when 'next-line'
               suppressions << Suppression.new(rule, line + 1, line + 1, reason)
             else
               active[rule] << [line + 1, reason]
@@ -134,7 +137,7 @@ module ArchSpec
       end
 
       def normalize_rule(rule)
-        return nil if rule.nil? || rule == "*"
+        return nil if rule.nil? || rule == '*'
 
         rule.downcase
       end
@@ -205,7 +208,7 @@ module ArchSpec
           )
         end
 
-        visit(graph, path, node.body, current_constant: constant.name, namespace: constant.name.split("::"))
+        visit(graph, path, node.body, current_constant: constant.name, namespace: constant.name.split('::'))
       end
 
       def visit_module(graph, path, node, current_constant:, namespace:)
@@ -217,11 +220,13 @@ module ArchSpec
           location: SourceLocation.from_prism(path, node.location)
         )
 
-        visit(graph, path, node.body, current_constant: constant.name, namespace: constant.name.split("::"))
+        visit(graph, path, node.body, current_constant: constant.name, namespace: constant.name.split('::'))
       end
 
       def visit_def(graph, path, node, current_constant:, namespace:)
-        if current_constant && (constant = graph.constants_named(current_constant).find { |candidate| candidate.path == path })
+        if current_constant && (constant = graph.constants_named(current_constant).find do |candidate|
+          candidate.path == path
+        end)
           if node.receiver
             constant.add_class_method(node.name, location: SourceLocation.from_prism(path, node.location))
           else
@@ -234,7 +239,7 @@ module ArchSpec
             type: :dynamic_feature,
             from_path: path,
             from_constant: current_constant,
-            to: "method_missing",
+            to: 'method_missing',
             location: SourceLocation.from_prism(path, node.location),
             confidence: :unknown_due_to_dynamic_feature
           )
@@ -244,7 +249,10 @@ module ArchSpec
       end
 
       def visit_call(graph, path, node, current_constant:, namespace:)
-        return visit_children(graph, path, node, current_constant: current_constant, namespace: namespace) unless node.message
+        unless node.message
+          return visit_children(graph, path, node, current_constant: current_constant,
+                                                   namespace: namespace)
+        end
 
         message = node.message.to_sym
         location = SourceLocation.from_prism(path, node.location)
@@ -271,7 +279,9 @@ module ArchSpec
 
         if (edge_type = MIXIN_MESSAGES[message])
           constant_arguments(node).each do |constant_name|
-            if current_constant && (constant = graph.constants_named(current_constant).find { |candidate| candidate.path == path })
+            if current_constant && (constant = graph.constants_named(current_constant).find do |candidate|
+              candidate.path == path
+            end)
               constant.add_mixin(message, constant_name)
             end
 
@@ -342,7 +352,7 @@ module ArchSpec
       def instantiates_and_invokes(node)
         receiver = node.receiver
         return unless receiver.is_a?(Prism::CallNode)
-        return unless receiver.message == "new"
+        return unless receiver.message == 'new'
 
         "#{new_receiver_name(receiver)}##{node.message}"
       end
@@ -350,22 +360,22 @@ module ArchSpec
       def new_receiver_name(node)
         return constant_reference_name(node.receiver) if constant_node?(node.receiver)
 
-        node.receiver&.slice || "(unknown)"
+        node.receiver&.slice || '(unknown)'
       end
 
       def qualified_constant_name(node, namespace)
         raw = constant_reference_name(node)
         absolute = node.respond_to?(:full_name_parts) && node.full_name_parts.first == :""
 
-        if absolute || raw.include?("::") || namespace.empty?
+        if absolute || raw.include?('::') || namespace.empty?
           raw
         else
-          "#{namespace.join("::")}::#{raw}"
+          "#{namespace.join('::')}::#{raw}"
         end
       end
 
       def constant_reference_name(node)
-        node.full_name.to_s.sub(/\A::/, "")
+        node.full_name.to_s.sub(/\A::/, '')
       end
 
       def constant_node?(node)

data/lib/archspec/architectures.rb

@@ -1,26 +1,28 @@
+# frozen_string_literal: true
+
 module ArchSpec
   module Architectures
     extend self
 
     DEFAULT_LAYERED = {
-      interface: "app/controllers/**/*.rb",
+      interface: 'app/controllers/**/*.rb',
       application: %w[app/services/**/*.rb app/jobs/**/*.rb app/mailers/**/*.rb],
-      domain: "app/models/**/*.rb"
+      domain: 'app/models/**/*.rb'
     }.freeze
 
     DEFAULT_RAILS_MVC = {
-      controllers: "app/controllers/**/*.rb",
-      models: "app/models/**/*.rb",
-      helpers: "app/helpers/**/*.rb",
-      mailers: "app/mailers/**/*.rb",
-      jobs: "app/jobs/**/*.rb",
-      services: "app/services/**/*.rb"
+      controllers: 'app/controllers/**/*.rb',
+      models: 'app/models/**/*.rb',
+      helpers: 'app/helpers/**/*.rb',
+      mailers: 'app/mailers/**/*.rb',
+      jobs: 'app/jobs/**/*.rb',
+      services: 'app/services/**/*.rb'
     }.freeze
 
     DEFAULT_HEXAGONAL = {
       application: %w[app/services/**/*.rb app/use_cases/**/*.rb],
-      domain: "app/domain/**/*.rb",
-      ports: "app/ports/**/*.rb",
+      domain: 'app/domain/**/*.rb',
+      ports: 'app/ports/**/*.rb',
       adapters: %w[app/adapters/**/*.rb app/integrations/**/*.rb app/infrastructure/**/*.rb]
     }.freeze
 
@@ -32,15 +34,15 @@ module ArchSpec
     }.freeze
 
     DEFAULT_CQRS = {
-      commands: "app/commands/**/*.rb",
-      queries: "app/queries/**/*.rb",
-      read_models: "app/read_models/**/*.rb"
+      commands: 'app/commands/**/*.rb',
+      queries: 'app/queries/**/*.rb',
+      read_models: 'app/read_models/**/*.rb'
     }.freeze
 
     DEFAULT_EVENT_DRIVEN = {
-      events: "app/events/**/*.rb",
-      publishers: "app/publishers/**/*.rb",
-      subscribers: "app/subscribers/**/*.rb"
+      events: 'app/events/**/*.rb',
+      publishers: 'app/publishers/**/*.rb',
+      subscribers: 'app/subscribers/**/*.rb'
     }.freeze
 
     CONTROLLER_METHODS = %i[render redirect_to params session cookies flash].freeze

data/lib/archspec/baseline.rb

@@ -1,5 +1,6 @@
-require "set"
-require "yaml"
+# frozen_string_literal: true
+
+require 'yaml'
 
 module ArchSpec
   class Baseline
@@ -11,8 +12,8 @@ module ArchSpec
       return empty(root: root) unless path && File.exist?(path)
 
       document = YAML.safe_load_file(path, permitted_classes: [], aliases: false) || {}
-      ids = Array(document["violations"]).filter_map do |entry|
-        entry.is_a?(Hash) ? entry["id"] : entry
+      ids = Array(document['violations']).filter_map do |entry|
+        entry.is_a?(Hash) ? entry['id'] : entry
       end
 
       new(ids.to_set, root: root)
@@ -20,13 +21,13 @@ module ArchSpec
 
     def self.write(path, diagnostics, root:)
       payload = {
-        "violations" => diagnostics.map do |diagnostic|
+        'violations' => diagnostics.map do |diagnostic|
           {
-            "id" => diagnostic.fingerprint(root: root),
-            "rule" => diagnostic.rule,
-            "path" => diagnostic.location.relative_path(root),
-            "line" => diagnostic.location.line,
-            "message" => diagnostic.message
+            'id' => diagnostic.fingerprint(root: root),
+            'rule' => diagnostic.rule,
+            'path' => diagnostic.location.relative_path(root),
+            'line' => diagnostic.location.line,
+            'message' => diagnostic.message
           }
         end
       }

data/lib/archspec/cli.rb

@@ -1,10 +1,12 @@
-require "optparse"
+# frozen_string_literal: true
+
+require 'optparse'
 
 module ArchSpec
   module CLI
     extend self
 
-    CONFIG_FILE = "Archspec.rb"
+    CONFIG_FILE = 'Archspec.rb'
     TEMPLATE = <<~RUBY
       ArchSpec.define "Application architecture" do
         root "."
@@ -14,16 +16,16 @@ module ArchSpec
 
     def run(argv, output: $stdout, error: $stderr)
       argv = argv.dup
-      command = argv.shift || "check"
+      command = argv.shift || 'check'
 
       case command
-      when "init"
+      when 'init'
         init(argv, output)
-      when "check"
+      when 'check'
         check(argv, output)
-      when "explain"
+      when 'explain'
         explain(argv, output)
-      when "version", "--version", "-v"
+      when 'version', '--version', '-v'
         output.puts ArchSpec::VERSION
         0
       else
@@ -31,20 +33,18 @@ module ArchSpec
         error.puts usage
         64
       end
-    rescue Error => exception
-      error.puts exception.message
+    rescue Error => e
+      error.puts e.message
       1
     end
 
     private
 
     def init(argv, output)
-      force = argv.delete("--force")
+      force = argv.delete('--force')
       path = argv.shift || CONFIG_FILE
 
-      if File.exist?(path) && !force
-        raise Error, "#{path} already exists. Use --force to overwrite it."
-      end
+      raise Error, "#{path} already exists. Use --force to overwrite it." if File.exist?(path) && !force
 
       File.write(path, TEMPLATE)
       output.puts "Created #{path}"
@@ -54,14 +54,14 @@ module ArchSpec
     def check(argv, output)
       options = {
         config: CONFIG_FILE,
-        format: "text",
+        format: 'text',
         update_baseline: false
       }
 
-      parser = OptionParser.new do |parser|
-        parser.on("--config PATH") { |value| options[:config] = value }
-        parser.on("--format FORMAT") { |value| options[:format] = value }
-        parser.on("--update-baseline") { options[:update_baseline] = true }
+      parser = OptionParser.new do |opts|
+        opts.on('--config PATH') { |value| options[:config] = value }
+        opts.on('--format FORMAT') { |value| options[:format] = value }
+        opts.on('--update-baseline') { options[:update_baseline] = true }
       end
       parser.parse!(argv)
 
@@ -72,7 +72,10 @@ module ArchSpec
       diagnostics = Evaluator.evaluate(definition, graph, baseline: baseline)
 
       if options[:update_baseline]
-        raise Error, "No baseline configured. Add `baseline \".archspec_todo.yml\"` to #{options[:config]}." unless baseline_path
+        unless baseline_path
+          raise Error,
+                "No baseline configured. Add `baseline \".archspec_todo.yml\"` to #{options[:config]}."
+        end
 
         Baseline.write(baseline_path, diagnostics, root: root)
         output.puts "Updated #{Pathname(baseline_path).relative_path_from(Pathname(root))} with #{diagnostics.size} violations."
@@ -85,13 +88,13 @@ module ArchSpec
 
     def explain(argv, output)
       options = { config: CONFIG_FILE }
-      parser = OptionParser.new do |parser|
-        parser.on("--config PATH") { |value| options[:config] = value }
+      parser = OptionParser.new do |opts|
+        opts.on('--config PATH') { |value| options[:config] = value }
       end
       parser.parse!(argv)
 
       subject = argv.shift
-      raise Error, "Usage: archspec explain PATH_OR_CONSTANT" unless subject
+      raise Error, 'Usage: archspec explain PATH_OR_CONSTANT' unless subject
 
       definition, root = load_definition(options[:config])
       graph = Analyzer.analyze(definition, root: root)
@@ -119,9 +122,9 @@ module ArchSpec
 
     def formatter_for(name)
       case name
-      when "text"
+      when 'text'
         Formatters::Text
-      when "json"
+      when 'json'
         Formatters::JSON
       else
         raise Error, "Unknown format: #{name.inspect}"
@@ -134,12 +137,12 @@ module ArchSpec
       if graph.files.key?(path)
         file = graph.files.fetch(path)
         output.puts file.relative_path
-        output.puts "  expected constant: #{file.expected_constant || "(none)"}"
-        output.puts "  defined constants: #{graph.constants_for_path(path).map(&:name).join(", ")}"
+        output.puts "  expected constant: #{file.expected_constant || '(none)'}"
+        output.puts "  defined constants: #{graph.constants_for_path(path).map(&:name).join(', ')}"
         output_parse_errors(output, file)
         output_component_reasons(output, graph.component_assignment_reasons_for_path(path))
         output_suppressions(output, file)
-        output.puts "  outgoing facts:"
+        output.puts '  outgoing facts:'
 
         graph.edges.select { |edge| edge.from_path == path }.each do |edge|
           output.puts "    #{edge.type} #{edge.to} at #{edge.location.line}:#{edge.location.column}"
@@ -153,29 +156,29 @@ module ArchSpec
           output.puts "  kind: #{constant.kind}"
           output.puts "  file: #{constant.location.relative_path(graph.root)}:#{constant.location.line}"
           output_component_reasons(output, graph.component_assignment_reasons_for_constant(constant.name))
-          output.puts "  superclass: #{constant.superclass || "(none)"}"
-          output.puts "  instance methods: #{constant.instance_methods.to_a.sort.join(", ")}"
-          output.puts "  class methods: #{constant.class_methods.to_a.sort.join(", ")}"
+          output.puts "  superclass: #{constant.superclass || '(none)'}"
+          output.puts "  instance methods: #{constant.instance_methods.to_a.sort.join(', ')}"
+          output.puts "  class methods: #{constant.class_methods.to_a.sort.join(', ')}"
         end
       end
     end
 
     def output_component_reasons(output, assignments)
       if assignments.empty?
-        output.puts "  components: (none)"
+        output.puts '  components: (none)'
         return
       end
 
-      output.puts "  components:"
+      output.puts '  components:'
       assignments.sort_by { |name, _reasons| name.to_s }.each do |name, reasons|
-        output.puts "    #{name}: #{reasons.empty? ? "(no recorded reason)" : reasons.join("; ")}"
+        output.puts "    #{name}: #{reasons.empty? ? '(no recorded reason)' : reasons.join('; ')}"
       end
     end
 
     def output_suppressions(output, file)
       return if file.suppressions.empty?
 
-      output.puts "  suppressions:"
+      output.puts '  suppressions:'
       file.suppressions.each do |suppression|
         line_range =
           if suppression.end_line == Float::INFINITY
@@ -185,8 +188,8 @@ module ArchSpec
           else
             "#{suppression.start_line}-#{suppression.end_line}"
           end
-        rule = suppression.rule || "*"
-        reason = suppression.reason ? " -- #{suppression.reason}" : ""
+        rule = suppression.rule || '*'
+        reason = suppression.reason ? " -- #{suppression.reason}" : ''
         output.puts "    #{rule} on line #{line_range}#{reason}"
       end
     end
@@ -194,7 +197,7 @@ module ArchSpec
     def output_parse_errors(output, file)
       return if file.parse_errors.empty?
 
-      output.puts "  parse errors:"
+      output.puts '  parse errors:'
       file.parse_errors.each do |parse_error|
         output.puts "    #{parse_error.location.line}:#{parse_error.location.column} #{parse_error.message}"
       end

data/lib/archspec/component_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ArchSpec
   class ComponentSpec
     attr_reader :name, :file_patterns, :namespaces, :constants
@@ -28,7 +30,7 @@ module ArchSpec
     private
 
     def normalize_constant(value)
-      value.to_s.sub(/\A::/, "")
+      value.to_s.sub(/\A::/, '')
     end
   end
 end

data/lib/archspec/definition.rb

@@ -1,18 +1,20 @@
+# frozen_string_literal: true
+
 module ArchSpec
   class Definition
     DEFAULT_SOURCE_PATTERNS = [
-      "app/**/*.rb",
-      "lib/**/*.rb",
-      "packs/*/app/**/*.rb",
-      "engines/*/app/**/*.rb"
+      'app/**/*.rb',
+      'lib/**/*.rb',
+      'packs/*/app/**/*.rb',
+      'engines/*/app/**/*.rb'
     ].freeze
 
     DEFAULT_IGNORE_PATTERNS = [
-      ".git/**/*",
-      ".bundle/**/*",
-      "node_modules/**/*",
-      "tmp/**/*",
-      "vendor/**/*"
+      '.git/**/*',
+      '.bundle/**/*',
+      'node_modules/**/*',
+      'tmp/**/*',
+      'vendor/**/*'
     ].freeze
 
     attr_accessor :name, :root_path, :baseline_path
@@ -20,7 +22,7 @@ module ArchSpec
 
     def initialize(name)
       @name = name
-      @root_path = "."
+      @root_path = '.'
       @baseline_path = nil
       @source_patterns = []
       @ignore_patterns = DEFAULT_IGNORE_PATTERNS.dup

data/lib/archspec/diagnostic.rb

@@ -1,4 +1,6 @@
-require "digest"
+# frozen_string_literal: true
+
+require 'digest'
 
 module ArchSpec
   class Diagnostic

data/lib/archspec/dsl.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ArchSpec
   module DSL
     module Context
@@ -15,7 +17,7 @@ module ArchSpec
         add_ignore_patterns(patterns)
       end
 
-      def baseline(path = ".archspec_todo.yml")
+      def baseline(path = '.archspec_todo.yml')
         self.baseline_path = path.to_s
       end
 
@@ -101,6 +103,11 @@ module ArchSpec
         self
       end
 
+      def must_be_empty(because: nil)
+        add_rule(Rules::MustBeEmptyRule.new(name, because: because))
+        self
+      end
+
       def must_implement(*methods)
         methods.each do |method_name|
           add_rule(Rules::MustImplementRule.new(name, method_name))
@@ -121,7 +128,7 @@ module ArchSpec
             candidate.respond_to?(:merge_key) && candidate.merge_key == rule.merge_key
           end
 
-          return existing.merge!(rule) if existing&.respond_to?(:merge!)
+          return existing.merge!(rule) if existing.respond_to?(:merge!)
           return existing if existing
         end
 

data/lib/archspec/evaluator.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ArchSpec
   module Evaluator
     extend self
@@ -6,7 +8,10 @@ module ArchSpec
       (parser_diagnostics(graph) + definition.rules.flat_map { |rule| rule.evaluate(graph) })
         .reject { |diagnostic| graph.suppressed?(diagnostic) }
         .reject { |diagnostic| baseline.include?(diagnostic) }
-        .sort_by { |diagnostic| [diagnostic.location.path, diagnostic.location.line, diagnostic.rule, diagnostic.message] }
+        .sort_by do |diagnostic|
+        [diagnostic.location.path, diagnostic.location.line, diagnostic.rule,
+         diagnostic.message]
+      end
     end
 
     private
@@ -15,7 +20,7 @@ module ArchSpec
       graph.files.values.flat_map do |file|
         file.parse_errors.map do |parse_error|
           Diagnostic.new(
-            rule: "parser.syntax",
+            rule: 'parser.syntax',
             message: parse_error.message,
             location: parse_error.location,
             evidence: file.relative_path,

data/lib/archspec/formatters/json.rb

@@ -1,9 +1,11 @@
-require "json"
+# frozen_string_literal: true
+
+require 'json'
 
 module ArchSpec
   module Formatters
     module JSON
-      extend self
+      module_function
 
       def print(output = $stdout, graph:, diagnostics:)
         output.puts ::JSON.pretty_generate(

data/lib/archspec/formatters/text.rb

@@ -1,7 +1,9 @@
+# frozen_string_literal: true
+
 module ArchSpec
   module Formatters
     module Text
-      extend self
+      module_function
 
       def print(output = $stdout, graph:, diagnostics:)
         if diagnostics.empty?
@@ -9,7 +11,7 @@ module ArchSpec
           return
         end
 
-        output.puts "#{diagnostics.size} architecture #{diagnostics.size == 1 ? "violation" : "violations"}"
+        output.puts "#{diagnostics.size} architecture #{diagnostics.size == 1 ? 'violation' : 'violations'}"
         output.puts
 
         diagnostics.each do |diagnostic|

data/lib/archspec/model.rb

@@ -1,5 +1,6 @@
-require "pathname"
-require "set"
+# frozen_string_literal: true
+
+require 'pathname'
 
 module ArchSpec
   ParseError = Data.define(:message, :location)
@@ -167,7 +168,8 @@ module ArchSpec
           next unless matched_file || matched_constant
 
           component.add_file(constant.path, reason: "defines #{constant.name}") if matched_constant
-          component.add_constant(constant.name, reason: matched_file ? "defined in matched file" : "matched namespace/constant selector")
+          component.add_constant(constant.name,
+                                 reason: matched_file ? 'defined in matched file' : 'matched namespace/constant selector')
         end
 
         @components[component.name] = component
@@ -207,11 +209,11 @@ module ArchSpec
       candidates = []
 
       if from_constant
-        namespace = normalize_constant(from_constant).split("::")
+        namespace = normalize_constant(from_constant).split('::')
         namespace.pop
 
         until namespace.empty?
-          candidates << "#{namespace.join("::")}::#{normalized}"
+          candidates << "#{namespace.join('::')}::#{normalized}"
           namespace.pop
         end
       end
@@ -272,7 +274,7 @@ module ArchSpec
     end
 
     def normalize_constant(value)
-      value.to_s.sub(/\A::/, "")
+      value.to_s.sub(/\A::/, '')
     end
   end
 end

data/lib/archspec/presets.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ArchSpec
   module Presets
     module_function
@@ -8,6 +10,8 @@ module ArchSpec
         rails_way(dsl, **options)
       when :rails_strict
         rails_strict(dsl, **options)
+      when :vanilla_rails
+        vanilla_rails(dsl, **options)
       when :rails_layered
         rails_layered(dsl, **options)
       when :rails_hexagonal
@@ -33,6 +37,23 @@ module ArchSpec
       dsl.no_cycles!(among: %i[controllers models helpers mailers jobs services])
     end
 
+    VANILLA_RAILS_EMPTY = {
+      services: ['app/services/**/*.rb', 'behavior belongs on models, not service objects'],
+      forms: ['app/forms/**/*.rb', 'use strong parameters and model validations'],
+      policies: ['app/policies/**/*.rb', 'authorization is predicate methods on models'],
+      decorators: ['app/decorators/**/*.rb', 'use helpers and partials'],
+      presenters: ['app/presenters/**/*.rb', 'presentation objects are POROs in app/models'],
+      view_components: ['app/components/**/*.rb', 'use helpers and ERB partials']
+    }.freeze
+
+    def vanilla_rails(dsl, **options)
+      rails_way(dsl, **options)
+
+      VANILLA_RAILS_EMPTY.each do |name, (pattern, reason)|
+        dsl.component(name, in: pattern).must_be_empty(because: reason)
+      end
+    end
+
     def rails_layered(dsl, **options)
       Architectures.apply(:layered, dsl, **options)
     end

data/lib/archspec/rules/component_rules.rb

@@ -0,0 +1,38 @@
+# frozen_string_literal: true
+
+module ArchSpec
+  module Rules
+    class MustBeEmptyRule
+      attr_reader :source, :because
+
+      def initialize(source, because: nil)
+        @source = source.to_sym
+        @because = because
+      end
+
+      def merge_key
+        [self.class, source]
+      end
+
+      def id
+        'components.empty'
+      end
+
+      def evaluate(graph)
+        component = graph.components[source]
+        return [] unless component
+
+        component.files.sort.map do |path|
+          relative = graph.files[path]&.relative_path || path
+
+          Diagnostic.new(
+            rule: id,
+            message: because ? "#{source} must stay empty: #{because}" : "#{source} must stay empty",
+            location: SourceLocation.new(path, 1, 1),
+            evidence: "#{relative} belongs to #{source}"
+          )
+        end
+      end
+    end
+  end
+end

data/lib/archspec/rules/cycle_rule.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ArchSpec
   module Rules
     class NoCyclesRule
@@ -8,7 +10,7 @@ module ArchSpec
       end
 
       def id
-        "dependencies.no_cycles"
+        'dependencies.no_cycles'
       end
 
       def evaluate(graph)
@@ -16,9 +18,9 @@ module ArchSpec
           location = first_location_for_cycle(graph, cycle) || SourceLocation.new(graph.root, 1, 1)
           Diagnostic.new(
             rule: id,
-            message: "component dependency cycle: #{cycle.join(" -> ")}",
+            message: "component dependency cycle: #{cycle.join(' -> ')}",
             location: location,
-            evidence: cycle.join(" -> ")
+            evidence: cycle.join(' -> ')
           )
         end
       end
@@ -26,17 +28,17 @@ module ArchSpec
       private
 
       def cycles(graph)
-        adjacency = Hash.new { |hash, key| hash[key] = Set.new }
+        adjacency = {}
         graph.component_dependency_pairs(only: components).each do |source, target|
           next if source == target
 
-          adjacency[source].add(target)
+          (adjacency[source] ||= Set.new).add(target)
         end
 
         found = Set.new
         result = []
 
-        adjacency.keys.each do |start|
+        adjacency.each_key do |start|
           walk(adjacency, start, start, [], found, result)
         end
 
@@ -46,7 +48,7 @@ module ArchSpec
       def walk(adjacency, start, node, path, found, result)
         next_path = path + [node]
 
-        adjacency[node].each do |target|
+        adjacency.fetch(node, []).each do |target|
           if target == start
             cycle = canonical_cycle(next_path + [start])
             key = cycle.join("\0")

data/lib/archspec/rules/dependency_rules.rb

@@ -1,4 +1,4 @@
-require "set"
+# frozen_string_literal: true
 
 module ArchSpec
   module Rules
@@ -36,7 +36,7 @@ module ArchSpec
 
     class AllowDependenciesRule < DependencyRule
       def id
-        "dependencies.allow"
+        'dependencies.allow'
       end
 
       def evaluate(graph)
@@ -57,7 +57,7 @@ module ArchSpec
 
     class ForbidDependenciesRule < DependencyRule
       def id
-        "dependencies.forbid"
+        'dependencies.forbid'
       end
 
       def evaluate(graph)
@@ -81,7 +81,7 @@ module ArchSpec
 
       def initialize(source, constants)
         @source = source.to_sym
-        @constants = Array(constants).flatten.map { |constant| constant.to_s.sub(/\A::/, "") }
+        @constants = Array(constants).flatten.map { |constant| constant.to_s.sub(/\A::/, '') }
       end
 
       def merge_key
@@ -94,7 +94,7 @@ module ArchSpec
       end
 
       def id
-        "constants.forbid"
+        'constants.forbid'
       end
 
       def evaluate(graph)

data/lib/archspec/rules/protocol_rules.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ArchSpec
   module Rules
     class CannotCallRule
@@ -18,7 +20,7 @@ module ArchSpec
       end
 
       def id
-        "methods.forbid"
+        'methods.forbid'
       end
 
       def evaluate(graph)
@@ -50,7 +52,7 @@ module ArchSpec
       end
 
       def id
-        "protocol.must_implement"
+        'protocol.must_implement'
       end
 
       def evaluate(graph)
@@ -61,7 +63,7 @@ module ArchSpec
             rule: id,
             message: "#{constant.name} must implement ##{method_name}",
             location: constant.location,
-            evidence: "#{constant.name} methods: #{constant.instance_methods.to_a.sort.join(", ")}"
+            evidence: "#{constant.name} methods: #{constant.instance_methods.to_a.sort.join(', ')}"
           )
         end
       end
@@ -93,7 +95,7 @@ module ArchSpec
       end
 
       def id
-        "protocol.must_implement_one_of"
+        'protocol.must_implement_one_of'
       end
 
       def evaluate(graph)
@@ -102,9 +104,9 @@ module ArchSpec
 
           Diagnostic.new(
             rule: id,
-            message: "#{constant.name} must implement one of #{method_names.map { |name| "##{name}" }.join(", ")}",
+            message: "#{constant.name} must implement one of #{method_names.map { |name| "##{name}" }.join(', ')}",
             location: constant.location,
-            evidence: "#{constant.name} methods: #{constant.instance_methods.to_a.sort.join(", ")}"
+            evidence: "#{constant.name} methods: #{constant.instance_methods.to_a.sort.join(', ')}"
           )
         end
       end
@@ -136,7 +138,7 @@ module ArchSpec
       end
 
       def id
-        "methods.define_forbid"
+        'methods.define_forbid'
       end
 
       def evaluate(graph)
@@ -165,7 +167,7 @@ module ArchSpec
       end
 
       def id
-        "objects.instantiate_and_invoke_forbid"
+        'objects.instantiate_and_invoke_forbid'
       end
 
       def evaluate(graph)

data/lib/archspec/rules/zeitwerk_rule.rb

@@ -1,8 +1,10 @@
+# frozen_string_literal: true
+
 module ArchSpec
   module Rules
     class ZeitwerkNamingRule
       def id
-        "zeitwerk.naming"
+        'zeitwerk.naming'
       end
 
       def evaluate(graph)
@@ -16,7 +18,7 @@ module ArchSpec
             rule: id,
             message: "#{file.relative_path} should define #{file.expected_constant}",
             location: SourceLocation.new(file.path, 1, 1),
-            evidence: "defined constants: #{defined.empty? ? "(none)" : defined.join(", ")}"
+            evidence: "defined constants: #{defined.empty? ? '(none)' : defined.join(', ')}"
           )
         end
       end

data/lib/archspec/source_location.rb

@@ -1,4 +1,6 @@
-require "pathname"
+# frozen_string_literal: true
+
+require 'pathname'
 
 module ArchSpec
   SourceLocation = Data.define(:path, :line, :column) do

data/lib/archspec/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ArchSpec
-  VERSION = "0.1.0.pre1"
+  VERSION = '0.1.0.pre2'
 end

data/lib/archspec.rb

@@ -1,22 +1,25 @@
-require_relative "archspec/version"
-require_relative "archspec/source_location"
-require_relative "archspec/diagnostic"
-require_relative "archspec/component_spec"
-require_relative "archspec/model"
-require_relative "archspec/definition"
-require_relative "archspec/baseline"
-require_relative "archspec/dsl"
-require_relative "archspec/analyzer"
-require_relative "archspec/evaluator"
-require_relative "archspec/architectures"
-require_relative "archspec/presets"
-require_relative "archspec/rules/dependency_rules"
-require_relative "archspec/rules/protocol_rules"
-require_relative "archspec/rules/cycle_rule"
-require_relative "archspec/rules/zeitwerk_rule"
-require_relative "archspec/formatters/text"
-require_relative "archspec/formatters/json"
-require_relative "archspec/cli"
+# frozen_string_literal: true
+
+require_relative 'archspec/version'
+require_relative 'archspec/source_location'
+require_relative 'archspec/diagnostic'
+require_relative 'archspec/component_spec'
+require_relative 'archspec/model'
+require_relative 'archspec/definition'
+require_relative 'archspec/baseline'
+require_relative 'archspec/dsl'
+require_relative 'archspec/analyzer'
+require_relative 'archspec/evaluator'
+require_relative 'archspec/architectures'
+require_relative 'archspec/presets'
+require_relative 'archspec/rules/component_rules'
+require_relative 'archspec/rules/dependency_rules'
+require_relative 'archspec/rules/protocol_rules'
+require_relative 'archspec/rules/cycle_rule'
+require_relative 'archspec/rules/zeitwerk_rule'
+require_relative 'archspec/formatters/text'
+require_relative 'archspec/formatters/json'
+require_relative 'archspec/cli'
 
 module ArchSpec
   class Error < StandardError; end
@@ -24,7 +27,7 @@ module ArchSpec
   class << self
     attr_accessor :last_definition
 
-    def define(name = "Architecture", &block)
+    def define(name = 'Architecture', &block)
       definition = Definition.new(name)
       definition.extend(DSL::Context)
       definition.instance_eval(&block) if block

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: archspec
 version: !ruby/object:Gem::Version
-  version: 0.1.0.pre1
+  version: 0.1.0.pre2
 platform: ruby
 authors:
 - Carmine Paolino
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-06-02 00:00:00.000000000 Z
+date: 2026-06-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: prism
@@ -77,6 +77,7 @@ files:
 - lib/archspec/formatters/text.rb
 - lib/archspec/model.rb
 - lib/archspec/presets.rb
+- lib/archspec/rules/component_rules.rb
 - lib/archspec/rules/cycle_rule.rb
 - lib/archspec/rules/dependency_rules.rb
 - lib/archspec/rules/protocol_rules.rb