claude-agent-framework 1.0.0

package/framework/skills/core/testing.md

@@ -0,0 +1,393 @@
+# Skill: Testing
+
+## Purpose
+Write comprehensive tests using RSpec, FactoryBot, Faker, and Shoulda Matchers.
+
+## Setup
+
+### Gemfile
+```ruby
+group :development, :test do
+  gem "rspec-rails"
+  gem "factory_bot_rails"
+  gem "faker"
+end
+
+group :test do
+  gem "shoulda-matchers"
+  gem "capybara"
+  gem "selenium-webdriver"
+end
+```
+
+### Install RSpec
+```bash
+rails generate rspec:install
+```
+
+### Configure RSpec
+```ruby
+# spec/rails_helper.rb
+require "spec_helper"
+ENV["RAILS_ENV"] ||= "test"
+require_relative "../config/environment"
+
+abort("Running in production!") if Rails.env.production?
+require "rspec/rails"
+
+Dir[Rails.root.join("spec/support/**/*.rb")].sort.each { |f| require f }
+
+RSpec.configure do |config|
+  config.fixture_paths = [Rails.root.join("spec/fixtures")]
+  config.use_transactional_fixtures = true
+  config.infer_spec_type_from_file_location!
+  config.filter_rails_from_backtrace!
+
+  config.include FactoryBot::Syntax::Methods
+end
+
+Shoulda::Matchers.configure do |config|
+  config.integrate do |with|
+    with.test_framework :rspec
+    with.library :rails
+  end
+end
+```
+
+## Factories
+
+### Basic Factory
+```ruby
+# spec/factories/users.rb
+FactoryBot.define do
+  factory :user do
+    email_address { Faker::Internet.unique.email }
+    password { "password123" }
+    password_confirmation { "password123" }
+
+    trait :admin do
+      role { "admin" }
+    end
+
+    trait :with_articles do
+      transient do
+        articles_count { 3 }
+      end
+
+      after(:create) do |user, evaluator|
+        create_list(:article, evaluator.articles_count, user: user)
+      end
+    end
+  end
+end
+```
+
+### Associated Factory
+```ruby
+# spec/factories/articles.rb
+FactoryBot.define do
+  factory :article do
+    title { Faker::Lorem.sentence }
+    body { Faker::Lorem.paragraphs(number: 3).join("\n\n") }
+    published { false }
+    user
+
+    trait :published do
+      published { true }
+    end
+
+    trait :with_comments do
+      transient do
+        comments_count { 5 }
+      end
+
+      after(:create) do |article, evaluator|
+        create_list(:comment, evaluator.comments_count, article: article)
+      end
+    end
+  end
+end
+```
+
+## Model Specs
+
+```ruby
+# spec/models/article_spec.rb
+require "rails_helper"
+
+RSpec.describe Article, type: :model do
+  describe "validations" do
+    it { should validate_presence_of(:title) }
+    it { should validate_length_of(:title).is_at_most(255) }
+    it { should validate_presence_of(:body) }
+  end
+
+  describe "associations" do
+    it { should belong_to(:user) }
+    it { should have_many(:comments).dependent(:destroy) }
+    it { should have_many(:taggings).dependent(:destroy) }
+    it { should have_many(:tags).through(:taggings) }
+  end
+
+  describe "scopes" do
+    describe ".published" do
+      it "returns only published articles" do
+        published = create(:article, :published)
+        draft = create(:article, published: false)
+
+        expect(Article.published).to include(published)
+        expect(Article.published).not_to include(draft)
+      end
+    end
+
+    describe ".recent" do
+      it "orders by created_at descending" do
+        old = create(:article, created_at: 1.week.ago)
+        new = create(:article, created_at: 1.day.ago)
+
+        expect(Article.recent).to eq([new, old])
+      end
+    end
+  end
+
+  describe "#publish!" do
+    it "marks article as published" do
+      article = create(:article, published: false)
+
+      article.publish!
+
+      expect(article.reload).to be_published
+    end
+  end
+
+  describe "callbacks" do
+    it "generates slug before save" do
+      article = build(:article, title: "Hello World")
+
+      article.save
+
+      expect(article.slug).to eq("hello-world")
+    end
+  end
+end
+```
+
+## Request Specs
+
+```ruby
+# spec/requests/articles_spec.rb
+require "rails_helper"
+
+RSpec.describe "Articles", type: :request do
+  let(:user) { create(:user) }
+  let(:article) { create(:article, user: user) }
+
+  describe "GET /articles" do
+    it "returns success" do
+      get articles_path
+      expect(response).to have_http_status(:success)
+    end
+
+    it "displays articles" do
+      article = create(:article, :published)
+      get articles_path
+      expect(response.body).to include(article.title)
+    end
+  end
+
+  describe "GET /articles/:id" do
+    context "when article is published" do
+      it "returns success" do
+        article = create(:article, :published)
+        get article_path(article)
+        expect(response).to have_http_status(:success)
+      end
+    end
+
+    context "when article is draft" do
+      it "redirects unauthorized users" do
+        article = create(:article, published: false)
+        get article_path(article)
+        expect(response).to redirect_to(new_session_path)
+      end
+    end
+  end
+
+  describe "POST /articles" do
+    context "when authenticated" do
+      before { sign_in(user) }
+
+      it "creates article with valid params" do
+        expect {
+          post articles_path, params: { article: { title: "Test", body: "Content" } }
+        }.to change(Article, :count).by(1)
+      end
+
+      it "does not create with invalid params" do
+        expect {
+          post articles_path, params: { article: { title: "", body: "" } }
+        }.not_to change(Article, :count)
+      end
+    end
+
+    context "when not authenticated" do
+      it "redirects to login" do
+        post articles_path, params: { article: { title: "Test", body: "Content" } }
+        expect(response).to redirect_to(new_session_path)
+      end
+    end
+  end
+
+  describe "PATCH /articles/:id" do
+    before { sign_in(user) }
+
+    it "updates article" do
+      patch article_path(article), params: { article: { title: "Updated" } }
+      expect(article.reload.title).to eq("Updated")
+    end
+  end
+
+  describe "DELETE /articles/:id" do
+    before { sign_in(user) }
+
+    it "destroys article" do
+      article # create it
+      expect {
+        delete article_path(article)
+      }.to change(Article, :count).by(-1)
+    end
+  end
+end
+```
+
+## System Specs
+
+```ruby
+# spec/system/articles_spec.rb
+require "rails_helper"
+
+RSpec.describe "Articles", type: :system do
+  let(:user) { create(:user) }
+
+  before do
+    driven_by(:selenium_chrome_headless)
+  end
+
+  describe "creating an article" do
+    before { sign_in(user) }
+
+    it "allows user to create article" do
+      visit new_article_path
+
+      fill_in "Title", with: "My Article"
+      fill_in "Body", with: "Article content here"
+      click_button "Create Article"
+
+      expect(page).to have_content("Article created successfully")
+      expect(page).to have_content("My Article")
+    end
+
+    it "shows validation errors" do
+      visit new_article_path
+
+      click_button "Create Article"
+
+      expect(page).to have_content("can't be blank")
+    end
+  end
+
+  describe "editing an article" do
+    let!(:article) { create(:article, user: user, title: "Original") }
+
+    before { sign_in(user) }
+
+    it "allows owner to edit" do
+      visit article_path(article)
+      click_link "Edit"
+
+      fill_in "Title", with: "Updated Title"
+      click_button "Update Article"
+
+      expect(page).to have_content("Updated Title")
+    end
+  end
+
+  describe "with Turbo interactions" do
+    before { sign_in(user) }
+
+    it "adds comment without page reload", js: true do
+      article = create(:article, :published)
+      visit article_path(article)
+
+      fill_in "Comment", with: "Great article!"
+      click_button "Post Comment"
+
+      within("#comments") do
+        expect(page).to have_content("Great article!")
+      end
+    end
+  end
+end
+```
+
+## Test Helpers
+
+```ruby
+# spec/support/authentication_helper.rb
+module AuthenticationHelper
+  def sign_in(user)
+    post session_path, params: {
+      email_address: user.email_address,
+      password: "password123"
+    }
+  end
+end
+
+RSpec.configure do |config|
+  config.include AuthenticationHelper, type: :request
+end
+
+# spec/support/system_helper.rb
+module SystemHelper
+  def sign_in(user)
+    visit new_session_path
+    fill_in "Email", with: user.email_address
+    fill_in "Password", with: "password123"
+    click_button "Sign In"
+  end
+end
+
+RSpec.configure do |config|
+  config.include SystemHelper, type: :system
+end
+```
+
+## Running Tests
+
+```bash
+# All tests
+bundle exec rspec
+
+# Specific file
+bundle exec rspec spec/models/article_spec.rb
+
+# Specific test
+bundle exec rspec spec/models/article_spec.rb:25
+
+# By type
+bundle exec rspec spec/models
+bundle exec rspec spec/requests
+bundle exec rspec spec/system
+
+# With format
+bundle exec rspec --format documentation
+```
+
+## Best Practices
+
+1. **One assertion per example** when possible
+2. **Use let/let!** for setup, not before blocks
+3. **Use traits** for variations
+4. **Test behavior, not implementation**
+5. **Use factories, not fixtures**
+6. **Keep tests fast** - mock external services

package/framework/skills/core/views.md

@@ -0,0 +1,260 @@
+# Skill: Views
+
+## Purpose
+Create Rails views with ERB, Tailwind CSS, and proper structure following mobile-first design principles.
+
+## View Structure
+
+```
+app/views/
+├── layouts/
+│   ├── application.html.erb
+│   └── _navigation.html.erb
+├── shared/
+│   ├── _flash.html.erb
+│   ├── _footer.html.erb
+│   └── _pagination.html.erb
+└── articles/
+    ├── index.html.erb
+    ├── show.html.erb
+    ├── new.html.erb
+    ├── edit.html.erb
+    ├── _article.html.erb
+    └── _form.html.erb
+```
+
+## Layout Template
+
+```erb
+<!DOCTYPE html>
+<html lang="<%= I18n.locale %>" class="h-full">
+<head>
+  <meta charset="UTF-8">
+  <meta name="viewport" content="width=device-width, initial-scale=1.0">
+  <title><%= content_for(:title) || "App Name" %></title>
+  <%= csrf_meta_tags %>
+  <%= csp_meta_tag %>
+  <%= stylesheet_link_tag "application", "data-turbo-track": "reload" %>
+  <%= javascript_importmap_tags %>
+</head>
+<body class="h-full bg-gray-50">
+  <%= render "shared/navigation" %>
+
+  <main class="container mx-auto px-4 py-8">
+    <%= render "shared/flash" %>
+    <%= yield %>
+  </main>
+
+  <%= render "shared/footer" %>
+</body>
+</html>
+```
+
+## Flash Messages
+
+```erb
+<%# app/views/shared/_flash.html.erb %>
+<div id="flash" class="fixed top-4 right-4 z-50 space-y-2">
+  <% flash.each do |type, message| %>
+    <div class="<%= flash_class(type) %> px-4 py-3 rounded-lg shadow-lg flex items-center gap-2"
+         data-controller="flash"
+         data-flash-target="message">
+      <span><%= message %></span>
+      <button type="button" data-action="flash#dismiss" class="ml-auto">
+        <svg class="w-4 h-4" fill="currentColor" viewBox="0 0 20 20">
+          <path fill-rule="evenodd" d="M4.293 4.293a1 1 0 011.414 0L10 8.586l4.293-4.293a1 1 0 111.414 1.414L11.414 10l4.293 4.293a1 1 0 01-1.414 1.414L10 11.414l-4.293 4.293a1 1 0 01-1.414-1.414L8.586 10 4.293 5.707a1 1 0 010-1.414z" clip-rule="evenodd"/>
+        </svg>
+      </button>
+    </div>
+  <% end %>
+</div>
+```
+
+## Index View
+
+```erb
+<%# app/views/articles/index.html.erb %>
+<% content_for :title, "Articles" %>
+
+<div class="flex justify-between items-center mb-6">
+  <h1 class="text-2xl font-bold text-gray-900">Articles</h1>
+  <%= link_to "New Article", new_article_path,
+      class: "bg-blue-600 hover:bg-blue-700 text-white px-4 py-2 rounded-lg" %>
+</div>
+
+<%= turbo_frame_tag "articles" do %>
+  <div class="space-y-4">
+    <%= render @articles %>
+  </div>
+
+  <% if @articles.empty? %>
+    <div class="text-center py-12 text-gray-500">
+      <p>No articles yet.</p>
+      <%= link_to "Create your first article", new_article_path, class: "text-blue-600 hover:underline" %>
+    </div>
+  <% end %>
+<% end %>
+
+<%= render "shared/pagination", pagy: @pagy %>
+```
+
+## Show View
+
+```erb
+<%# app/views/articles/show.html.erb %>
+<% content_for :title, @article.title %>
+
+<article class="max-w-3xl mx-auto">
+  <header class="mb-8">
+    <h1 class="text-3xl font-bold text-gray-900 mb-2"><%= @article.title %></h1>
+    <div class="flex items-center gap-4 text-sm text-gray-500">
+      <span>By <%= @article.user.name %></span>
+      <time datetime="<%= @article.created_at.iso8601 %>">
+        <%= @article.created_at.strftime("%B %d, %Y") %>
+      </time>
+    </div>
+  </header>
+
+  <div class="prose prose-lg max-w-none">
+    <%= simple_format(@article.body) %>
+  </div>
+
+  <footer class="mt-8 pt-8 border-t flex gap-4">
+    <%= link_to "Edit", edit_article_path(@article), class: "text-blue-600 hover:underline" %>
+    <%= button_to "Delete", @article, method: :delete,
+        data: { turbo_confirm: "Are you sure?" },
+        class: "text-red-600 hover:underline" %>
+    <%= link_to "Back", articles_path, class: "text-gray-600 hover:underline" %>
+  </footer>
+</article>
+```
+
+## Form Partial
+
+```erb
+<%# app/views/articles/_form.html.erb %>
+<%= form_with model: article, class: "space-y-6" do |f| %>
+  <% if article.errors.any? %>
+    <div class="bg-red-50 border border-red-200 rounded-lg p-4">
+      <h3 class="text-red-800 font-medium mb-2">Please fix the following errors:</h3>
+      <ul class="list-disc list-inside text-red-700 text-sm">
+        <% article.errors.full_messages.each do |error| %>
+          <li><%= error %></li>
+        <% end %>
+      </ul>
+    </div>
+  <% end %>
+
+  <div>
+    <%= f.label :title, class: "block text-sm font-medium text-gray-700 mb-1" %>
+    <%= f.text_field :title,
+        class: "w-full rounded-lg border-gray-300 shadow-sm focus:border-blue-500 focus:ring-blue-500",
+        required: true %>
+  </div>
+
+  <div>
+    <%= f.label :body, class: "block text-sm font-medium text-gray-700 mb-1" %>
+    <%= f.text_area :body,
+        rows: 10,
+        class: "w-full rounded-lg border-gray-300 shadow-sm focus:border-blue-500 focus:ring-blue-500" %>
+  </div>
+
+  <div class="flex items-center gap-2">
+    <%= f.check_box :published, class: "rounded border-gray-300 text-blue-600 focus:ring-blue-500" %>
+    <%= f.label :published, "Publish immediately", class: "text-sm text-gray-700" %>
+  </div>
+
+  <div class="flex gap-4">
+    <%= f.submit class: "bg-blue-600 hover:bg-blue-700 text-white px-6 py-2 rounded-lg cursor-pointer" %>
+    <%= link_to "Cancel", articles_path, class: "text-gray-600 hover:underline py-2" %>
+  </div>
+<% end %>
+```
+
+## Card Partial
+
+```erb
+<%# app/views/articles/_article.html.erb %>
+<%= turbo_frame_tag dom_id(article) do %>
+  <article class="bg-white rounded-lg shadow p-6 hover:shadow-md transition-shadow">
+    <h2 class="text-xl font-semibold mb-2">
+      <%= link_to article.title, article, class: "text-gray-900 hover:text-blue-600" %>
+    </h2>
+
+    <p class="text-gray-600 mb-4 line-clamp-2"><%= truncate(article.body, length: 150) %></p>
+
+    <div class="flex items-center justify-between text-sm text-gray-500">
+      <span><%= article.user.name %></span>
+      <time><%= time_ago_in_words(article.created_at) %> ago</time>
+    </div>
+  </article>
+<% end %>
+```
+
+## Helper Methods
+
+```ruby
+# app/helpers/application_helper.rb
+module ApplicationHelper
+  def flash_class(type)
+    case type.to_sym
+    when :notice, :success
+      "bg-green-100 text-green-800 border border-green-200"
+    when :alert, :error
+      "bg-red-100 text-red-800 border border-red-200"
+    when :warning
+      "bg-yellow-100 text-yellow-800 border border-yellow-200"
+    else
+      "bg-blue-100 text-blue-800 border border-blue-200"
+    end
+  end
+
+  def active_link_class(path)
+    current_page?(path) ? "text-blue-600 font-medium" : "text-gray-600 hover:text-gray-900"
+  end
+end
+```
+
+## Responsive Design
+
+```erb
+<%# Mobile-first grid %>
+<div class="grid grid-cols-1 md:grid-cols-2 lg:grid-cols-3 gap-6">
+  <%= render @articles %>
+</div>
+
+<%# Responsive navigation %>
+<nav class="flex flex-col md:flex-row md:items-center gap-4">
+  <%= link_to "Home", root_path %>
+  <%= link_to "Articles", articles_path %>
+</nav>
+
+<%# Hidden on mobile %>
+<div class="hidden md:block">Desktop only</div>
+
+<%# Visible on mobile %>
+<div class="md:hidden">Mobile only</div>
+```
+
+## Turbo Frames
+
+```erb
+<%# Lazy loading %>
+<%= turbo_frame_tag "comments", src: article_comments_path(@article), loading: :lazy do %>
+  <p class="text-gray-500">Loading comments...</p>
+<% end %>
+
+<%# Modal frame %>
+<%= turbo_frame_tag "modal" %>
+
+<%# Form that updates frame %>
+<%= turbo_frame_tag "search_results" do %>
+  <%= form_with url: search_path, method: :get, data: { turbo_frame: "search_results" } do |f| %>
+    <%= f.search_field :q, placeholder: "Search..." %>
+  <% end %>
+
+  <div id="results">
+    <%= render @results %>
+  </div>
+<% end %>
+```