rolemodel-rails 0.26.0 → 1.1.0

data/lib/generators/rolemodel/mcp/templates/spec/requests/mcp_controller_spec.rb

@@ -0,0 +1,84 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+RSpec.describe 'MCPController', type: :request do
+  let(:user) { create(:user, name: 'Jane Smith', active: true, employee: true) }
+  let(:application) do
+    Doorkeeper::Application.create!(
+      name: 'MCP Test Client',
+      redirect_uri: 'https://example.com/callback',
+      scopes: 'mcp',
+    )
+  end
+
+  describe 'POST /mcp' do
+    let(:token) do
+      Doorkeeper::AccessToken.create!(
+        application: application,
+        resource_owner_id: user.id,
+        scopes: 'mcp',
+        expires_in: 2.hours.to_i,
+      )
+    end
+
+    let(:initialize_request) do
+      {
+        jsonrpc: '2.0',
+        id: 1,
+        method: 'initialize',
+        params: {
+          protocolVersion: '2025-11-25',
+          capabilities: {},
+          clientInfo: {
+            name: 'rspec',
+            version: '1.0',
+          },
+        },
+      }
+    end
+
+    let(:headers) do
+      {
+        'CONTENT_TYPE' => 'application/json',
+        'ACCEPT' => 'application/json, text/event-stream',
+      }
+    end
+
+    let(:authorized_headers) do
+      headers.merge('Authorization' => "Bearer #{token.token}")
+    end
+
+    it 'returns unauthorized when no bearer token is provided' do
+      post '/mcp', params: initialize_request.to_json, headers: headers
+
+      expect(response).to have_http_status(:unauthorized)
+      expect(response.headers['WWW-Authenticate']).to include('resource_metadata=')
+    end
+
+    it 'returns forbidden when token does not include mcp scope' do
+      token = Doorkeeper::AccessToken.create!(
+        application: application,
+        resource_owner_id: user.id,
+        scopes: 'other',
+        expires_in: 2.hours.to_i,
+      )
+
+      post '/mcp',
+        params: initialize_request.to_json,
+        headers: headers.merge('Authorization' => "Bearer #{token.token}")
+
+      expect(response).to have_http_status(:forbidden)
+    end
+
+    it 'returns initialize response for a valid bearer token' do
+      post '/mcp',
+        params: initialize_request.to_json,
+        headers: authorized_headers
+
+      expect(response).to have_http_status(:ok)
+      expect(response.parsed_body['result']).to be_present
+      expect(response.parsed_body['result']['capabilities']).to include('tools')
+    end
+  end
+end

data/lib/generators/rolemodel/mcp/templates/spec/requests/oauth_registrations_controller_spec.rb

@@ -0,0 +1,62 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+RSpec.describe 'OauthRegistrationsController', type: :request do
+  describe 'POST /oauth/register' do
+    let(:params) do
+      {
+        client_name: 'GitHub Copilot',
+        redirect_uris: ['http://localhost:12345/callback'],
+        grant_types: ['authorization_code'],
+        response_types: ['code'],
+        token_endpoint_auth_method: 'none',
+        scope: 'mcp',
+      }
+    end
+
+    it 'creates an OAuth application and returns client_id' do
+      expect { post '/oauth/register', params: params, as: :json }
+        .to change(Doorkeeper::Application, :count).by(1)
+
+      expect(response).to have_http_status(:created)
+      body = response.parsed_body
+      expect(body['client_id']).to be_present
+      expect(body['client_name']).to eq('GitHub Copilot')
+      expect(body['redirect_uris']).to eq(['http://localhost:12345/callback'])
+      expect(body['token_endpoint_auth_method']).to eq('none')
+      expect(body).not_to have_key('client_secret')
+
+      app = Doorkeeper::Application.last
+      expect(app.name).to eq('GitHub Copilot')
+      expect(app.redirect_uri).to eq('http://localhost:12345/callback')
+      expect(app.scopes).to contain_exactly('mcp')
+      expect(app).not_to be_confidential
+    end
+
+    it 'returns bad_request when redirect_uris is missing' do
+      post '/oauth/register', params: { client_name: 'Test' }, as: :json
+
+      expect(response).to have_http_status(:bad_request)
+      expect(response.parsed_body['error']).to eq('invalid_client_metadata')
+    end
+
+    it 'creates a confidential client when token_endpoint_auth_method is not none' do
+      params[:token_endpoint_auth_method] = 'client_secret_basic'
+      post '/oauth/register', params: params, as: :json
+
+      expect(response).to have_http_status(:created)
+      body = response.parsed_body
+      expect(body['client_secret']).to be_present
+      expect(body['token_endpoint_auth_method']).to eq('client_secret_basic')
+    end
+
+    it 'allows loopback redirect URIs for native clients' do
+      params[:redirect_uris] = ['http://127.0.0.1:33418/']
+      post '/oauth/register', params: params, as: :json
+
+      expect(response).to have_http_status(:created)
+      expect(response.parsed_body['redirect_uris']).to eq(['http://127.0.0.1:33418/'])
+    end
+  end
+end

data/lib/generators/rolemodel/mcp/templates/spec/requests/well_known_controller_spec.rb

@@ -0,0 +1,30 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+RSpec.describe 'WellKnownController', type: :request do
+  describe 'GET /.well-known/oauth-protected-resource' do
+    it 'returns the MCP resource and authorization server' do
+      get '/.well-known/oauth-protected-resource'
+
+      expect(response).to have_http_status(:ok)
+      body = response.parsed_body
+      expect(body['resource']).to end_with('/mcp')
+      expect(body['authorization_servers']).to be_an(Array)
+    end
+  end
+
+  describe 'GET /.well-known/oauth-authorization-server' do
+    it 'returns authorization server metadata with registration_endpoint' do
+      get '/.well-known/oauth-authorization-server'
+
+      expect(response).to have_http_status(:ok)
+      body = response.parsed_body
+      expect(body['registration_endpoint']).to end_with('/oauth/register')
+      expect(body['authorization_endpoint']).to end_with('/oauth/authorize')
+      expect(body['token_endpoint']).to end_with('/oauth/token')
+      expect(body['code_challenge_methods_supported']).to include('S256')
+      expect(body['scopes_supported']).to include('mcp')
+    end
+  end
+end

data/lib/generators/rolemodel/optics/all_generator.rb

@@ -1,6 +1,6 @@
 module Rolemodel
   module Optics
-    class AllGenerator < Rolemodel::BaseGenerator
+    class AllGenerator < Rolemodel::GeneratorBase
       source_root File.expand_path('templates', __dir__)
 
       def run_all_the_generators

data/lib/generators/rolemodel/optics/base/base_generator.rb

@@ -1,6 +1,6 @@
 module Rolemodel
   module Optics
-    class BaseGenerator < Rolemodel::BaseGenerator
+    class BaseGenerator < Rolemodel::GeneratorBase
       source_root File.expand_path('templates', __dir__)
 
       def add_optics_package
@@ -12,7 +12,7 @@ module Rolemodel
       def copy_templates
         say 'importing stylesheet', :green
 
-        prepend_to_file 'app/assets/stylesheets/application.scss', <<~SCSS
+        prepend_to_file Dir.glob('app/assets/stylesheets/application.*').first, <<~SCSS
           @import '@rolemodel/optics/dist/css/optics';
         SCSS
       end

data/lib/generators/rolemodel/optics/icons/icons_generator.rb

@@ -3,7 +3,7 @@
 module Rolemodel
   module Optics
     # Generates the icon helper and icon builders for the chosen icon library
-    class IconsGenerator < Rolemodel::BaseGenerator
+    class IconsGenerator < Rolemodel::GeneratorBase
       SUPPORTED_LIBRARIES = HashWithIndifferentAccess.new(
         material: 'filled, size, weight, emphasis, additional_classes, color, hover_text',
         phosphor: 'duotone, filled, size, weight, additional_classes, color, hover_text',

data/lib/generators/rolemodel/react/react_generator.rb

@@ -1,5 +1,5 @@
 module Rolemodel
-  class ReactGenerator < BaseGenerator
+  class ReactGenerator < GeneratorBase
     source_root File.expand_path('templates', __dir__)
 
     def add_npm_packages

data/lib/generators/rolemodel/readme/readme_generator.rb

@@ -1,5 +1,5 @@
 module Rolemodel
-  class ReadmeGenerator < BaseGenerator
+  class ReadmeGenerator < GeneratorBase
     source_root File.expand_path('templates', __dir__)
 
     def install_readme

data/lib/generators/rolemodel/saas/all_generator.rb

@@ -1,6 +1,6 @@
 module Rolemodel
   module Saas
-    class AllGenerator < BaseGenerator
+    class AllGenerator < GeneratorBase
       source_root File.expand_path('templates', __dir__)
 
       def run_all_the_generators

data/lib/generators/rolemodel/saas/devise/devise_generator.rb

@@ -2,7 +2,7 @@
 
 module Rolemodel
   module Saas
-    class DeviseGenerator < BaseGenerator
+    class DeviseGenerator < GeneratorBase
       source_root File.expand_path('templates', __dir__)
 
       def add_organization

data/lib/generators/rolemodel/semaphore/semaphore_generator.rb

@@ -1,5 +1,5 @@
 module Rolemodel
-  class SemaphoreGenerator < BaseGenerator
+  class SemaphoreGenerator < GeneratorBase
     source_root File.expand_path('templates', __dir__)
 
     def create_base_semaphore_config

data/lib/generators/rolemodel/simple_form/simple_form_generator.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Rolemodel
-  class SimpleFormGenerator < BaseGenerator
+  class SimpleFormGenerator < GeneratorBase
     source_root File.expand_path('templates', __dir__)
 
     def add_gem

data/lib/generators/rolemodel/slim/slim_generator.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Rolemodel
-  class SlimGenerator < BaseGenerator
+  class SlimGenerator < GeneratorBase
     source_root File.expand_path('templates', __dir__)
 
     def add_slim

data/lib/generators/rolemodel/soft_destroyable/soft_destroyable_generator.rb

@@ -1,5 +1,5 @@
 module Rolemodel
-  class SoftDestroyableGenerator < BaseGenerator
+  class SoftDestroyableGenerator < GeneratorBase
     source_root File.expand_path('templates', __dir__)
 
     def add_concern

data/lib/generators/rolemodel/source_map/source_map_generator.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Rolemodel
-  class SourceMapGenerator < BaseGenerator
+  class SourceMapGenerator < GeneratorBase
     source_root File.expand_path('templates', __dir__)
 
     def inject_config_for_production

data/lib/generators/rolemodel/tailored_select/tailored_select_generator.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Rolemodel
-  class TailoredSelectGenerator < BaseGenerator
+  class TailoredSelectGenerator < GeneratorBase
     source_root File.expand_path('templates', __dir__)
 
     def add_tailored_select_package

data/lib/generators/rolemodel/testing/all_generator.rb

@@ -1,6 +1,6 @@
 module Rolemodel
   module Testing
-    class AllGenerator < BaseGenerator
+    class AllGenerator < GeneratorBase
       source_root File.expand_path('templates', __dir__)
 
       def run_all_the_generators

data/lib/generators/rolemodel/testing/factory_bot/factory_bot_generator.rb

@@ -2,7 +2,7 @@
 
 module Rolemodel
   module Testing
-    class FactoryBotGenerator < BaseGenerator
+    class FactoryBotGenerator < GeneratorBase
       source_root File.expand_path('templates', __dir__)
 
       def install_factory_bot_rails

data/lib/generators/rolemodel/testing/jasmine_playwright/jasmine_playwright_generator.rb

@@ -2,7 +2,7 @@
 
 module Rolemodel
   module Testing
-    class JasminePlaywrightGenerator < BaseGenerator
+    class JasminePlaywrightGenerator < GeneratorBase
       include ReplaceContentHelper
       source_root File.expand_path('templates', __dir__)
       class_option :github_package_token, type: :string, default: ENV['GITHUB_PACKAGES_TOKEN'], desc: 'GitHub Packages token with access to @rolemodel packages'

data/lib/generators/rolemodel/testing/parallel_tests/parallel_tests_generator.rb

@@ -2,7 +2,7 @@
 
 module Rolemodel
   module Testing
-    class ParallelTestsGenerator < BaseGenerator # rubocop:disable Style/Documentation
+    class ParallelTestsGenerator < GeneratorBase # rubocop:disable Style/Documentation
       source_root File.expand_path('templates', __dir__)
 
       def install_parallel_tests

data/lib/generators/rolemodel/testing/rspec/rspec_generator.rb

@@ -2,7 +2,7 @@
 
 module Rolemodel
   module Testing
-    class RspecGenerator < BaseGenerator
+    class RspecGenerator < GeneratorBase
       source_root File.expand_path('templates', __dir__)
 
       def install_rspec
@@ -15,7 +15,6 @@ module Rolemodel
           gem 'capybara-playwright-driver'
           gem 'marsh_grass'
           gem 'pry'
-          gem 'webdrivers'
         end
         run_bundle
       end

data/lib/generators/rolemodel/testing/rspec/templates/rails_helper.rb

@@ -9,6 +9,10 @@ abort("The Rails environment is running in production mode!") if Rails.env.produ
 require 'rspec/rails'
 # Add additional requires below this line. Rails is not loaded until this point!
 
+ActiveSupport::Inflector.inflections(:en) do |inflect|
+  inflect.acronym "MCP"
+end
+
 # Requires supporting ruby files with custom matchers and macros, etc, in
 # spec/support/ and its subdirectories. Files matching `spec/**/*_spec.rb` are
 # run as spec files by default. This means that files in spec/support that end

data/lib/generators/rolemodel/testing/rspec/templates/support/capybara_drivers.rb

@@ -47,5 +47,3 @@ Capybara.server = :puma, { Silent: true }
 
 # try to remove any potential for parallel tests to conflict
 Capybara.threadsafe = true
-
-Webdrivers.cache_time = 24.hours.to_i

data/lib/generators/rolemodel/testing/rspec/templates/support/helpers/capybara_helper.rb

@@ -1,51 +1,5 @@
 # frozen_string_literal: true
 
-# Update the link_or_button selector so that we can use `click_on` with gc-menu-item
-Capybara.modify_selector(:link_or_button) do
-  label 'link or button'
-
-  xpath do |locator, **options|
-    %i[link button].map do |selector|
-      expression_for(selector, locator, **options)
-    end.reduce(:union)
-  end
-end
-
-module LoadStateWaitingLogic
-  module_function
-
-  def add_wait_for_load_state(method_names)
-    method_names.each do |method_name|
-      define_method(method_name) do |*args, **kwargs, &block|
-        super(*args, **kwargs, &block).tap do
-          return unless CapybaraHelper.supports_javascript?
-
-          Capybara.page.driver.with_playwright_page do |page|
-            page.wait_for_load_state(state: 'networkidle')
-          end
-        end
-      end
-    end
-  end
-end
-
-module LoadStateWaiter
-  include LoadStateWaitingLogic
-
-  INTERACTIVE_METHODS = %i[choose check uncheck select fill_in].freeze
-  LoadStateWaitingLogic.add_wait_for_load_state(INTERACTIVE_METHODS)
-end
-
-module ElementLoadStateWaiter
-  include LoadStateWaitingLogic
-
-  INTERACTIVE_METHODS = %i[send_keys click select].freeze
-  LoadStateWaitingLogic.add_wait_for_load_state(INTERACTIVE_METHODS)
-end
-
-Capybara::Node::Actions.prepend(LoadStateWaiter)
-Capybara::Node::Element.prepend(ElementLoadStateWaiter)
-
 module CapybaraHelper
   TIMEOUT = 10
 

data/lib/generators/rolemodel/testing/rspec/templates/support/helpers/playwright_helper.rb

@@ -22,64 +22,4 @@ module PlaywrightHelper
 
     puts "Screenshot saved to \e[4;96m#{directory}/#{filename}.png\e[0m" if print_message
   end
-
-  def self.scope_stack
-    @scope_stack ||= []
-  end
-
-  def current_scope
-    PlaywrightHelper.scope_stack.last || pw_page
-  end
-
-  def within(selector, **, &block)
-    if supports_javascript?
-      begin
-        pw_locator = if selector.is_a?(Capybara::Node::Element)
-                       pw_page.locator("xpath=#{selector.path}")
-                     else
-                       pw_page.locator(selector)
-                     end
-
-        PlaywrightHelper.scope_stack.push(pw_locator)
-
-        super
-      ensure
-        PlaywrightHelper.scope_stack.pop
-      end
-    else
-      super
-    end
-  end
-
-  def click_on(text = nil, **args)
-    if supports_javascript? && text.present?
-      scope = current_scope
-      locator = nil
-
-      %w[button link gc-menu-item summary].each do |tag|
-        loc = scope.get_by_role(tag).get_by_text(text)
-        loc = scope.get_by_role(tag).get_by_text(text, exact: true) if loc.count > 1
-        loc = scope.locator(tag).get_by_text(text) unless loc.count == 1
-        locator = loc if loc.count == 1
-        break if locator.present?
-      end
-
-      locator = scope.get_by_text(text, exact: true) if locator.blank?
-
-      raise "No element matching text: '#{text}'" if locator.count < 1
-
-      if locator.count > 1
-        raise "Multiple elements matching text: '#{text}'" unless args[:match] == :first
-
-        result = locator.first.click
-      else
-        result = locator.click
-      end
-
-      pw_page.wait_for_load_state(state: 'networkidle')
-      result
-    else
-      Capybara.click_on(text, **args)
-    end
-  end
 end

data/lib/generators/rolemodel/testing/test_prof/test_prof_generator.rb

@@ -2,7 +2,7 @@
 
 module Rolemodel
   module Testing
-    class TestProfGenerator < BaseGenerator
+    class TestProfGenerator < GeneratorBase
       def install_test_prof
         gem_group :test do
           gem 'test-prof'

data/lib/generators/rolemodel/testing/vitest/vitest_generator.rb

@@ -2,7 +2,7 @@
 
 module Rolemodel
   module Testing
-    class VitestGenerator < BaseGenerator
+    class VitestGenerator < GeneratorBase
       include ReplaceContentHelper
       source_root File.expand_path('templates', __dir__)
 

data/lib/generators/rolemodel/ui_components/all_generator.rb

@@ -1,6 +1,6 @@
 module Rolemodel
   module UiComponents
-    class AllGenerator < BaseGenerator
+    class AllGenerator < GeneratorBase
       source_root File.expand_path('templates', __dir__)
 
       def run_all_the_generators

data/lib/generators/rolemodel/ui_components/flash/flash_generator.rb

@@ -2,7 +2,7 @@
 
 module Rolemodel
   module UiComponents
-    class FlashGenerator < BaseGenerator
+    class FlashGenerator < GeneratorBase
       source_root File.expand_path('templates', __dir__)
 
       def generate_files

data/lib/generators/rolemodel/ui_components/modals/modals_generator.rb

@@ -1,6 +1,6 @@
 module Rolemodel
   module UiComponents
-    class ModalsGenerator < BaseGenerator
+    class ModalsGenerator < GeneratorBase
       source_root File.expand_path('templates', __dir__)
       class_option :panels, type: :boolean, default: false, desc: 'Include RoleModel Panel Setup'
 

data/lib/generators/rolemodel/ui_components/navbar/navbar_generator.rb

@@ -2,7 +2,7 @@
 
 module Rolemodel
   module UiComponents
-    class NavbarGenerator < BaseGenerator
+    class NavbarGenerator < GeneratorBase
       source_root File.expand_path('templates', __dir__)
 
       def copy_navbar_files

data/lib/generators/rolemodel/webpack/webpack_generator.rb

@@ -1,5 +1,5 @@
 module Rolemodel
-  class WebpackGenerator < BaseGenerator
+  class WebpackGenerator < GeneratorBase
     source_root File.expand_path('templates', __dir__)
 
     DEV_DEPS = %w[

data/lib/rolemodel/engine.rb

@@ -2,8 +2,10 @@
 
 module Rolemodel
   class Engine < ::Rails::Engine
+    require_relative 'generator_base'
+
     generators do
-      require 'generators/rolemodel/base_generator'
+      require 'generators/rolemodel/all_generator'
     end
   end
 end

data/lib/rolemodel/generator_base.rb

@@ -0,0 +1,17 @@
+# frozen_string_literal: true
+
+require 'rails/generators'
+require 'rails/generators/bundle_helper'
+require_relative 'replace_content_helper'
+
+module Rolemodel
+  class GeneratorBase < ::Rails::Generators::Base
+    include ::Rails::Generators::BundleHelper, ReplaceContentHelper
+
+    private
+    # based on https://github.com/rails/rails/blob/main/railties/lib/rails/generators/app_base.rb#L713
+    def run_bundle
+      bundle_command("install --quiet", "BUNDLE_IGNORE_MESSAGES" => "1")
+    end
+  end
+end

data/lib/rolemodel/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Rolemodel
-  VERSION = '0.26.0'
+  VERSION = '1.1.0'
 end

data/lib/rolemodel-rails.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Rolemodel
   NODE_VERSION = '24.12.0'
   RUBY_VERSION = '4.0.1'