rubyn-code 0.1.0

data/skills/solid/single_responsibility.md

@@ -0,0 +1,183 @@
+# SOLID: Single Responsibility Principle (SRP)
+
+## Pattern
+
+A class should have one — and only one — reason to change. Every class encapsulates one concept, one actor's requirements, or one axis of change. When requirements from different stakeholders would cause the same class to change, split it.
+
+```ruby
+# GOOD: Each class has one reason to change
+
+# Reason to change: how orders are persisted
+class Order < ApplicationRecord
+  belongs_to :user
+  has_many :line_items, dependent: :destroy
+
+  validates :shipping_address, presence: true
+
+  def total
+    line_items.sum { |li| li.quantity * li.unit_price }
+  end
+end
+
+# Reason to change: how order totals are calculated (tax rules, discounts)
+class Orders::TotalCalculator
+  def initialize(order)
+    @order = order
+  end
+
+  def call
+    subtotal = @order.line_items.sum { |li| li.quantity * li.unit_price }
+    tax = TaxService.calculate(subtotal, @order.shipping_address)
+    discount = DiscountService.calculate(subtotal, @order.user)
+    subtotal + tax - discount
+  end
+end
+
+# Reason to change: how order confirmations are formatted and delivered
+class Orders::ConfirmationNotifier
+  def initialize(order)
+    @order = order
+  end
+
+  def call
+    OrderMailer.confirmation(@order).deliver_later
+    SmsNotifier.send(@order.user.phone, "Order #{@order.reference} confirmed!") if @order.user.sms_enabled?
+  end
+end
+
+# Reason to change: how order data is exported
+class Orders::CsvExporter
+  HEADERS = %w[reference customer total status created_at].freeze
+
+  def call(orders)
+    CSV.generate(headers: true) do |csv|
+      csv << HEADERS
+      orders.each { |order| csv << row(order) }
+    end
+  end
+
+  private
+
+  def row(order)
+    [order.reference, order.user.email, order.total, order.status, order.created_at.iso8601]
+  end
+end
+```
+
+The controller orchestrates, each specialist does its job:
+
+```ruby
+class OrdersController < ApplicationController
+  def create
+    @order = current_user.orders.build(order_params)
+    @order.total = Orders::TotalCalculator.new(@order).call
+
+    if @order.save
+      Orders::ConfirmationNotifier.new(@order).call
+      redirect_to @order, notice: "Order placed."
+    else
+      render :new, status: :unprocessable_entity
+    end
+  end
+
+  def export
+    orders = current_user.orders.recent
+    csv = Orders::CsvExporter.new.call(orders)
+    send_data csv, filename: "orders-#{Date.current}.csv"
+  end
+end
+```
+
+## Why This Is Good
+
+- **Changes are isolated.** Tax law changes? Modify `TotalCalculator`. Email template changes? Modify `ConfirmationNotifier`. Neither change touches the Order model, the other service, or the controller.
+- **Smaller classes are easier to understand.** A developer opening `CsvExporter` knows it does one thing. They don't need to scroll past tax calculations and email logic to find the export code.
+- **Easier to test.** Test `TotalCalculator` with unit tests against various tax/discount scenarios. Test `ConfirmationNotifier` by asserting emails are enqueued. Each test file is focused and fast.
+- **Team-friendly.** Developer A works on tax calculation while Developer B works on notifications. No merge conflicts because the changes are in separate files.
+- **Reusable.** `TotalCalculator` can be used in the controller, in a background job, in an API endpoint, and in an admin panel. If calculation lived inside the controller, it couldn't be reused.
+
+## Anti-Pattern
+
+A god class that handles persistence, calculation, notification, and export:
+
+```ruby
+class Order < ApplicationRecord
+  belongs_to :user
+  has_many :line_items
+
+  after_create :send_confirmation_email
+  after_create :send_sms_notification
+  after_update :recalculate_total
+
+  def calculate_total
+    subtotal = line_items.sum { |li| li.quantity * li.unit_price }
+    tax = subtotal * tax_rate_for(shipping_address)
+    discount = loyalty_discount_for(user)
+    self.total = subtotal + tax - discount
+  end
+
+  def tax_rate_for(address)
+    case address.state
+    when "CA" then 0.0725
+    when "TX" then 0.0625
+    when "NY" then 0.08
+    else 0.05
+    end
+  end
+
+  def loyalty_discount_for(user)
+    return 0 unless user.loyalty_tier == :gold
+    total * 0.1
+  end
+
+  def send_confirmation_email
+    OrderMailer.confirmation(self).deliver_later
+  end
+
+  def send_sms_notification
+    return unless user.sms_enabled?
+    SmsClient.send(user.phone, "Order #{reference} confirmed!")
+  end
+
+  def to_csv
+    [reference, user.email, total, status, created_at.iso8601].join(",")
+  end
+
+  def self.export_csv(orders)
+    headers = "reference,customer,total,status,created_at\n"
+    headers + orders.map(&:to_csv).join("\n")
+  end
+end
+```
+
+## Why This Is Bad
+
+- **Four reasons to change in one class.** Tax rules, notification channels, export format, and persistence logic are all tangled together. A change to tax rates risks breaking CSV export because they share the same file.
+- **500+ lines.** This Order model is headed toward 500 lines as each responsibility grows. Adding international tax, promo codes, push notifications, and PDF export will push it past 1,000.
+- **Callbacks hide side effects.** `after_create :send_sms_notification` silently sends texts. Creating an order in tests, seeds, or the console triggers SMS delivery.
+- **Untestable in isolation.** Testing `tax_rate_for` requires an Order instance. Testing `send_confirmation_email` requires a saved order. Everything is coupled to the model lifecycle.
+- **Violates Open/Closed.** Adding a new notification channel (push, Slack) means modifying the Order class. With SRP, you modify the Notifier class or add a new one — the Order never changes.
+
+## When To Apply
+
+- **A class is changing for multiple reasons.** If your last 5 commits to `order.rb` were: fix tax calculation, update email template, add CSV header, fix discount logic — that's 4 different responsibilities.
+- **A class is longer than ~100 lines.** Length isn't a hard rule, but it's a signal. A 200-line model usually has at least 2 responsibilities that could be extracted.
+- **You describe a class with "and."** "The Order class persists data AND calculates totals AND sends notifications AND exports CSV." Each "and" is a candidate for extraction.
+- **Different team members frequently edit the same file.** Merge conflicts on a model file are a sign that multiple concerns are competing for the same space.
+
+## When NOT To Apply
+
+- **Don't extract prematurely.** A 30-line model with validations and 2 scopes has one responsibility (data access). Don't split it into 4 classes for theoretical purity.
+- **Simple derived values belong on the model.** `def full_name; "#{first_name} #{last_name}"; end` is fine on User. It's a data representation concern, not a separate responsibility.
+- **Don't create single-method classes for trivial operations.** A `StringCapitalizer` class is over-engineering. SRP means one *responsibility*, not one *method*.
+
+## Edge Cases
+
+**How do you identify responsibilities?**
+Ask: "Who would request this change?" If the answer is different people or departments (accounting wants tax changes, marketing wants email changes, ops wants export changes), those are different responsibilities.
+
+**Rails models seem to violate SRP by default:**
+ActiveRecord models combine data access, validation, association management, and query interface. This is an intentional framework trade-off. The SRP boundary in Rails is: models own persistence and validation, everything else is extracted. Don't fight the framework — work within it.
+
+**Service objects can also violate SRP:**
+A `ProcessOrderService` that calculates totals, sends emails, AND updates inventory violates SRP just as badly as a fat model. Each step should be its own service, orchestrated by a coordinator.

metadata

@@ -0,0 +1,397 @@
+--- !ruby/object:Gem::Specification
+name: rubyn-code
+version: !ruby/object:Gem::Version
+  version: 0.1.0
+platform: ruby
+authors:
+- fadedmaturity
+bindir: exe
+cert_chain: []
+date: 1980-01-02 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
+  name: sqlite3
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.0'
+- !ruby/object:Gem::Dependency
+  name: faraday
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '2.0'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '3.0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '2.0'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '3.0'
+- !ruby/object:Gem::Dependency
+  name: pastel
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.8'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.8'
+- !ruby/object:Gem::Dependency
+  name: rouge
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '4.0'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '5.0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '4.0'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '5.0'
+- !ruby/object:Gem::Dependency
+  name: tty-prompt
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.23'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.23'
+- !ruby/object:Gem::Dependency
+  name: tty-spinner
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.9'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.9'
+- !ruby/object:Gem::Dependency
+  name: tty-markdown
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.7'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.7'
+- !ruby/object:Gem::Dependency
+  name: webrick
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.8'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.8'
+description: An AI-powered CLI coding assistant specialized for Ruby and Rails, featuring
+  a 16-layer agentic architecture, local SQLite persistence, and Claude OAuth.
+executables:
+- rubyn-code
+extensions: []
+extra_rdoc_files: []
+files:
+- LICENSE
+- README.md
+- db/migrations/000_create_schema_migrations.sql
+- db/migrations/001_create_sessions.sql
+- db/migrations/002_create_messages.sql
+- db/migrations/003_create_tasks.sql
+- db/migrations/004_create_task_dependencies.sql
+- db/migrations/005_create_memories.sql
+- db/migrations/006_create_cost_records.sql
+- db/migrations/007_create_hooks.sql
+- db/migrations/008_create_skills_cache.sql
+- db/migrations/009_create_teams.sql
+- db/migrations/010_create_instincts.sql
+- exe/rubyn-code
+- lib/rubyn_code.rb
+- lib/rubyn_code/agent/conversation.rb
+- lib/rubyn_code/agent/loop.rb
+- lib/rubyn_code/agent/loop_detector.rb
+- lib/rubyn_code/auth/oauth.rb
+- lib/rubyn_code/auth/server.rb
+- lib/rubyn_code/auth/token_store.rb
+- lib/rubyn_code/autonomous/daemon.rb
+- lib/rubyn_code/autonomous/idle_poller.rb
+- lib/rubyn_code/autonomous/task_claimer.rb
+- lib/rubyn_code/background/job.rb
+- lib/rubyn_code/background/notifier.rb
+- lib/rubyn_code/background/worker.rb
+- lib/rubyn_code/cli/app.rb
+- lib/rubyn_code/cli/input_handler.rb
+- lib/rubyn_code/cli/renderer.rb
+- lib/rubyn_code/cli/repl.rb
+- lib/rubyn_code/cli/spinner.rb
+- lib/rubyn_code/cli/stream_formatter.rb
+- lib/rubyn_code/config/defaults.rb
+- lib/rubyn_code/config/project_config.rb
+- lib/rubyn_code/config/settings.rb
+- lib/rubyn_code/context/auto_compact.rb
+- lib/rubyn_code/context/compactor.rb
+- lib/rubyn_code/context/manager.rb
+- lib/rubyn_code/context/manual_compact.rb
+- lib/rubyn_code/context/micro_compact.rb
+- lib/rubyn_code/db/connection.rb
+- lib/rubyn_code/db/migrator.rb
+- lib/rubyn_code/db/schema.rb
+- lib/rubyn_code/hooks/built_in.rb
+- lib/rubyn_code/hooks/registry.rb
+- lib/rubyn_code/hooks/runner.rb
+- lib/rubyn_code/hooks/user_hooks.rb
+- lib/rubyn_code/learning/extractor.rb
+- lib/rubyn_code/learning/injector.rb
+- lib/rubyn_code/learning/instinct.rb
+- lib/rubyn_code/llm/client.rb
+- lib/rubyn_code/llm/message_builder.rb
+- lib/rubyn_code/llm/streaming.rb
+- lib/rubyn_code/mcp/client.rb
+- lib/rubyn_code/mcp/config.rb
+- lib/rubyn_code/mcp/sse_transport.rb
+- lib/rubyn_code/mcp/stdio_transport.rb
+- lib/rubyn_code/mcp/tool_bridge.rb
+- lib/rubyn_code/memory/models.rb
+- lib/rubyn_code/memory/search.rb
+- lib/rubyn_code/memory/session_persistence.rb
+- lib/rubyn_code/memory/store.rb
+- lib/rubyn_code/observability/budget_enforcer.rb
+- lib/rubyn_code/observability/cost_calculator.rb
+- lib/rubyn_code/observability/models.rb
+- lib/rubyn_code/observability/token_counter.rb
+- lib/rubyn_code/observability/usage_reporter.rb
+- lib/rubyn_code/output/diff_renderer.rb
+- lib/rubyn_code/output/formatter.rb
+- lib/rubyn_code/permissions/deny_list.rb
+- lib/rubyn_code/permissions/policy.rb
+- lib/rubyn_code/permissions/prompter.rb
+- lib/rubyn_code/permissions/tier.rb
+- lib/rubyn_code/protocols/interrupt_handler.rb
+- lib/rubyn_code/protocols/plan_approval.rb
+- lib/rubyn_code/protocols/shutdown_handshake.rb
+- lib/rubyn_code/skills/catalog.rb
+- lib/rubyn_code/skills/document.rb
+- lib/rubyn_code/skills/loader.rb
+- lib/rubyn_code/sub_agents/runner.rb
+- lib/rubyn_code/sub_agents/summarizer.rb
+- lib/rubyn_code/tasks/dag.rb
+- lib/rubyn_code/tasks/manager.rb
+- lib/rubyn_code/tasks/models.rb
+- lib/rubyn_code/teams/mailbox.rb
+- lib/rubyn_code/teams/manager.rb
+- lib/rubyn_code/teams/teammate.rb
+- lib/rubyn_code/tools/background_run.rb
+- lib/rubyn_code/tools/base.rb
+- lib/rubyn_code/tools/bash.rb
+- lib/rubyn_code/tools/bundle_add.rb
+- lib/rubyn_code/tools/bundle_install.rb
+- lib/rubyn_code/tools/compact.rb
+- lib/rubyn_code/tools/db_migrate.rb
+- lib/rubyn_code/tools/edit_file.rb
+- lib/rubyn_code/tools/executor.rb
+- lib/rubyn_code/tools/git_commit.rb
+- lib/rubyn_code/tools/git_diff.rb
+- lib/rubyn_code/tools/git_log.rb
+- lib/rubyn_code/tools/git_status.rb
+- lib/rubyn_code/tools/glob.rb
+- lib/rubyn_code/tools/grep.rb
+- lib/rubyn_code/tools/load_skill.rb
+- lib/rubyn_code/tools/memory_search.rb
+- lib/rubyn_code/tools/memory_write.rb
+- lib/rubyn_code/tools/rails_generate.rb
+- lib/rubyn_code/tools/read_file.rb
+- lib/rubyn_code/tools/read_inbox.rb
+- lib/rubyn_code/tools/registry.rb
+- lib/rubyn_code/tools/review_pr.rb
+- lib/rubyn_code/tools/run_specs.rb
+- lib/rubyn_code/tools/schema.rb
+- lib/rubyn_code/tools/send_message.rb
+- lib/rubyn_code/tools/spawn_agent.rb
+- lib/rubyn_code/tools/spawn_teammate.rb
+- lib/rubyn_code/tools/task.rb
+- lib/rubyn_code/tools/web_fetch.rb
+- lib/rubyn_code/tools/web_search.rb
+- lib/rubyn_code/tools/write_file.rb
+- lib/rubyn_code/version.rb
+- skills/code_quality/fits_in_your_head.md
+- skills/code_quality/naming_conventions.md
+- skills/code_quality/null_object.md
+- skills/code_quality/technical_debt.md
+- skills/code_quality/value_objects.md
+- skills/code_quality/yagni.md
+- skills/design_patterns/adapter.md
+- skills/design_patterns/bridge_memento_visitor.md
+- skills/design_patterns/builder.md
+- skills/design_patterns/command.md
+- skills/design_patterns/composite.md
+- skills/design_patterns/decorator.md
+- skills/design_patterns/facade.md
+- skills/design_patterns/factory_method.md
+- skills/design_patterns/iterator.md
+- skills/design_patterns/mediator.md
+- skills/design_patterns/observer.md
+- skills/design_patterns/proxy.md
+- skills/design_patterns/singleton.md
+- skills/design_patterns/state.md
+- skills/design_patterns/strategy.md
+- skills/design_patterns/template_method.md
+- skills/gems/devise.md
+- skills/gems/dry_rb.md
+- skills/gems/factory_bot.md
+- skills/gems/faraday.md
+- skills/gems/graphql_ruby.md
+- skills/gems/pundit.md
+- skills/gems/redis.md
+- skills/gems/rubocop.md
+- skills/gems/sidekiq.md
+- skills/gems/stripe.md
+- skills/minitest/assertions.md
+- skills/minitest/fixtures.md
+- skills/minitest/integration_tests.md
+- skills/minitest/mailers_and_jobs.md
+- skills/minitest/mocking_stubbing.md
+- skills/minitest/service_tests_and_performance.md
+- skills/minitest/structure_and_conventions.md
+- skills/minitest/system_tests.md
+- skills/rails/action_cable.md
+- skills/rails/active_record_basics.md
+- skills/rails/active_storage.md
+- skills/rails/api_design.md
+- skills/rails/associations.md
+- skills/rails/background_jobs.md
+- skills/rails/caching.md
+- skills/rails/callbacks.md
+- skills/rails/concerns_controllers.md
+- skills/rails/concerns_models.md
+- skills/rails/controllers.md
+- skills/rails/engines.md
+- skills/rails/form_objects.md
+- skills/rails/hotwire.md
+- skills/rails/internationalization.md
+- skills/rails/logging.md
+- skills/rails/mailers.md
+- skills/rails/migrations.md
+- skills/rails/multitenancy.md
+- skills/rails/n_plus_one.md
+- skills/rails/presenters.md
+- skills/rails/query_objects.md
+- skills/rails/routing.md
+- skills/rails/scopes.md
+- skills/rails/security.md
+- skills/rails/serializers.md
+- skills/rails/service_objects.md
+- skills/rails/testing_strategy.md
+- skills/rails/validations.md
+- skills/refactoring/code_smells.md
+- skills/refactoring/command_query_separation.md
+- skills/refactoring/encapsulate_collection.md
+- skills/refactoring/extract_class.md
+- skills/refactoring/extract_method.md
+- skills/refactoring/replace_conditional.md
+- skills/refactoring/value_objects.md
+- skills/rspec/build_stubbed.md
+- skills/rspec/factory_design.md
+- skills/rspec/let_vs_let_bang.md
+- skills/rspec/mocking_stubbing.md
+- skills/rspec/request_specs.md
+- skills/rspec/service_specs.md
+- skills/rspec/shared_examples.md
+- skills/rspec/system_specs.md
+- skills/rspec/test_performance.md
+- skills/ruby/blocks_procs_lambdas.md
+- skills/ruby/classes.md
+- skills/ruby/concurrency.md
+- skills/ruby/data_struct_openstruct.md
+- skills/ruby/debugging_profiling.md
+- skills/ruby/enumerable_patterns.md
+- skills/ruby/exception_handling.md
+- skills/ruby/file_io.md
+- skills/ruby/hashes.md
+- skills/ruby/metaprogramming.md
+- skills/ruby/modules.md
+- skills/ruby/pattern_matching.md
+- skills/ruby/regular_expressions.md
+- skills/ruby/result_objects.md
+- skills/ruby/strings.md
+- skills/ruby_project/bundler_dependencies.md
+- skills/ruby_project/cli_tools.md
+- skills/ruby_project/rake_tasks.md
+- skills/ruby_project/structure.md
+- skills/sinatra/application_structure.md
+- skills/sinatra/middleware_and_deployment.md
+- skills/sinatra/testing.md
+- skills/solid/dependency_inversion.md
+- skills/solid/interface_segregation.md
+- skills/solid/liskov_substitution.md
+- skills/solid/open_closed.md
+- skills/solid/single_responsibility.md
+homepage: https://rubyn.dev
+licenses:
+- MIT
+metadata: {}
+rdoc_options: []
+require_paths:
+- lib
+required_ruby_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - ">="
+    - !ruby/object:Gem::Version
+      version: '3.3'
+required_rubygems_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - ">="
+    - !ruby/object:Gem::Version
+      version: '0'
+requirements: []
+rubygems_version: 3.6.9
+specification_version: 4
+summary: Ruby & Rails agentic coding assistant
+test_files: []