panda-core 0.4.1 → 0.6.0

data/app/helpers/panda/core/sessions_helper.rb

@@ -10,11 +10,36 @@ module Panda
         github: "github"
       }.freeze
 
+      # Map of providers that don't use fa-brands (use fa-solid instead)
+      PROVIDER_NON_BRAND_ICONS = {
+        developer: "code"
+      }.freeze
+
+      # Map OAuth provider names to their display names
+      PROVIDER_NAME_MAP = {
+        google_oauth2: "Google",
+        microsoft_graph: "Microsoft",
+        github: "GitHub",
+        developer: "Developer"
+      }.freeze
+
       # Returns the FontAwesome icon name for a given provider
       # Checks provider config first, then falls back to the mapping, then uses the provider name as-is
       def oauth_provider_icon(provider)
         provider_config = Panda::Core.config.authentication_providers[provider]
-        provider_config&.dig(:icon) || PROVIDER_ICON_MAP[provider.to_sym] || provider.to_s
+        provider_config&.dig(:icon) || PROVIDER_ICON_MAP[provider.to_sym] || PROVIDER_NON_BRAND_ICONS[provider.to_sym] || provider.to_s
+      end
+
+      # Returns true if the provider uses a non-brand icon (fa-solid, fa-regular, etc.)
+      def oauth_provider_non_brand?(provider)
+        PROVIDER_NON_BRAND_ICONS.key?(provider.to_sym)
+      end
+
+      # Returns the display name for a given provider
+      # Checks provider config first, then falls back to the mapping, then humanizes the provider name
+      def oauth_provider_name(provider, provider_config = nil)
+        provider_config ||= Panda::Core.config.authentication_providers[provider]
+        provider_config&.dig(:name) || PROVIDER_NAME_MAP[provider.to_sym] || provider.to_s.humanize
       end
     end
   end

data/app/javascript/panda/core/application.js

@@ -7,4 +7,11 @@ const application = Application.start()
 application.debug = false
 window.Stimulus = application
 
-export { application }
+export { application }
+
+// Note: controllers/index.js must be loaded separately in the HTML to avoid circular dependency
+// It will import this application and register all controllers
+
+// Tailwind Plus Elements can be loaded by importing "panda/core/tailwindplus-elements"
+// or by adding the script tag directly to your HTML:
+// <script src="https://cdn.jsdelivr.net/npm/@tailwindplus/elements@1" type="module"></script>

data/app/javascript/panda/core/controllers/alert_controller.js

@@ -0,0 +1,38 @@
+import { Controller } from "@hotwired/stimulus"
+
+export default class extends Controller {
+  static values = {
+    dismissAfter: Number
+  }
+
+  connect() {
+    // Auto-dismiss if dismissAfter value is set
+    if (this.hasDismissAfterValue && this.dismissAfterValue > 0) {
+      this.timeout = setTimeout(() => {
+        this.close()
+      }, this.dismissAfterValue)
+    }
+  }
+
+  disconnect() {
+    // Clean up timeout if controller is disconnected
+    if (this.timeout) {
+      clearTimeout(this.timeout)
+    }
+  }
+
+  close() {
+    // Clear any pending timeout
+    if (this.timeout) {
+      clearTimeout(this.timeout)
+    }
+
+    // Remove the element with a fade-out animation
+    this.element.style.transition = "opacity 0.3s ease-out"
+    this.element.style.opacity = "0"
+
+    setTimeout(() => {
+      this.element.remove()
+    }, 300)
+  }
+}

data/app/javascript/panda/core/controllers/index.js

@@ -1,12 +1,12 @@
 // Import and register Core Stimulus controllers
-import { application } from "../application"
+import { application } from "../application.js"
 
-import ThemeFormController from "./theme_form_controller"
+import ThemeFormController from "./theme_form_controller.js"
 application.register("theme-form", ThemeFormController)
 
 // Import and register TailwindCSS Stimulus Components
 // These are needed for UI components like slideover, modals, alerts, etc.
-import { Alert, Autosave, ColorPreview, Dropdown, Modal, Tabs, Popover, Toggle, Slideover } from "../tailwindcss-stimulus-components"
+import { Alert, Autosave, ColorPreview, Dropdown, Modal, Tabs, Popover, Toggle, Slideover } from "../tailwindcss-stimulus-components.js"
 application.register('alert', Alert)
 application.register('autosave', Autosave)
 application.register('color-preview', ColorPreview)

data/app/javascript/panda/core/controllers/toggle_controller.js

@@ -0,0 +1,41 @@
+import { Controller } from "@hotwired/stimulus"
+
+// Toggle controller for showing/hiding elements
+// Usage:
+//   <div data-controller="toggle">
+//     <button data-action="click->toggle#toggle">Toggle</button>
+//     <div data-toggle-target="toggleable" class="hidden">Content</div>
+//   </div>
+export default class extends Controller {
+  static targets = ["toggleable"]
+
+  toggle(event) {
+    if (event) {
+      event.preventDefault()
+    }
+
+    this.toggleableTargets.forEach(target => {
+      target.classList.toggle("hidden")
+    })
+  }
+
+  show(event) {
+    if (event) {
+      event.preventDefault()
+    }
+
+    this.toggleableTargets.forEach(target => {
+      target.classList.remove("hidden")
+    })
+  }
+
+  hide(event) {
+    if (event) {
+      event.preventDefault()
+    }
+
+    this.toggleableTargets.forEach(target => {
+      target.classList.add("hidden")
+    })
+  }
+}

data/app/javascript/panda/core/tailwindplus-elements.js

@@ -0,0 +1,31 @@
+/**
+ * Tailwind Plus Elements Loader
+ *
+ * This module loads the Tailwind Plus Elements library for vanilla JS interactive components.
+ * It provides eight foundational primitives:
+ * - Autocomplete: enables custom combobox implementations
+ * - Command palette: builds searchable command interfaces
+ * - Dialog: creates modals, drawers, and overlays
+ * - Disclosure: handles collapsible sections and mobile menus
+ * - Dropdown menu: constructs option menus
+ * - Popover: manages floating UI elements
+ * - Select: builds custom dropdown selects
+ * - Tabs: creates tabbed interfaces
+ *
+ * Usage:
+ *   import "@tailwindplus/elements"
+ *
+ * Or lazy-load when needed:
+ *   import("@tailwindplus/elements").then(() => {
+ *     // Elements are now available
+ *   })
+ *
+ * Documentation: https://tailwindcss.com/blog/vanilla-js-support-for-tailwind-plus
+ */
+
+// Auto-import Tailwind Plus Elements
+// This makes custom elements like <el-dropdown>, <el-dialog>, etc. available
+import "@tailwindplus/elements"
+
+// Export for module consumers
+export default "@tailwindplus/elements"

data/app/models/panda/core/user.rb

@@ -5,27 +5,59 @@ module Panda
     class User < ApplicationRecord
       self.table_name = "panda_core_users"
 
+      # Active Storage attachment for avatar
+      has_one_attached :avatar
+
       validates :email, presence: true, uniqueness: {case_sensitive: false}
 
       before_save :downcase_email
 
       # Scopes
-      scope :admin, -> { where(is_admin: true) }
+      scope :admins, -> { where(is_admin: true) }
 
       def self.find_or_create_from_auth_hash(auth_hash)
         user = find_by(email: auth_hash.info.email.downcase)
-        return user if user
 
-        create!(
+        # Handle avatar for both new and existing users
+        avatar_url = auth_hash.info.image
+        if user
+          # Update avatar if URL has changed or no avatar is attached
+          if avatar_url.present? && (avatar_url != user.oauth_avatar_url || !user.avatar.attached?)
+            AttachAvatarService.call(user: user, avatar_url: avatar_url)
+          end
+          return user
+        end
+
+        # Support both schema versions: 'name' column or 'firstname'/'lastname' columns
+        attributes = {
           email: auth_hash.info.email.downcase,
-          name: auth_hash.info.name || "Unknown User",
           image_url: auth_hash.info.image,
           is_admin: User.count.zero? # First user is admin
-        )
+        }
+
+        # Add name attributes based on schema
+        if column_names.include?("name")
+          attributes[:name] = auth_hash.info.name || "Unknown User"
+        elsif column_names.include?("firstname") && column_names.include?("lastname")
+          # Split name into firstname/lastname if provided
+          full_name = auth_hash.info.name || "Unknown User"
+          name_parts = full_name.split(" ", 2)
+          attributes[:firstname] = name_parts[0] || "Unknown"
+          attributes[:lastname] = name_parts[1] || "User"
+        end
+
+        user = create!(attributes)
+
+        # Attach avatar for new user
+        if avatar_url.present?
+          AttachAvatarService.call(user: user, avatar_url: avatar_url)
+        end
+
+        user
       end
 
       def admin?
-        is_admin?
+        is_admin
       end
 
       def active_for_authentication?
@@ -45,6 +77,17 @@ module Panda
         end
       end
 
+      # Returns the URL for the user's avatar
+      # Prefers Active Storage attachment over OAuth provider URL
+      def avatar_url
+        if avatar.attached?
+          Rails.application.routes.url_helpers.rails_blob_path(avatar, only_path: true)
+        elsif self[:image_url].present?
+          # Fallback to OAuth provider URL if no avatar is attached yet
+          self[:image_url]
+        end
+      end
+
       private
 
       def downcase_email

data/app/services/panda/core/attach_avatar_service.rb

@@ -0,0 +1,67 @@
+# frozen_string_literal: true
+
+require "open-uri"
+
+module Panda
+  module Core
+    class AttachAvatarService < Services::BaseService
+      def initialize(user:, avatar_url:)
+        @user = user
+        @avatar_url = avatar_url
+      end
+
+      def call
+        return success if @avatar_url.blank?
+        return success if @avatar_url == @user.oauth_avatar_url && @user.avatar.attached?
+
+        begin
+          download_and_attach_avatar
+          @user.update_column(:oauth_avatar_url, @avatar_url)
+          success(avatar_attached: true)
+        rescue => e
+          Rails.logger.error("Failed to attach avatar for user #{@user.id}: #{e.message}")
+          failure(["Failed to attach avatar: #{e.message}"])
+        end
+      end
+
+      private
+
+      def download_and_attach_avatar
+        # Open the URL with a timeout and size limit
+        URI.open(@avatar_url, read_timeout: 10, open_timeout: 10, redirect: true) do |downloaded_file|
+          # Validate file size (max 5MB)
+          if downloaded_file.size > 5.megabytes
+            raise "Avatar file too large (#{downloaded_file.size} bytes)"
+          end
+
+          # Determine content type and filename
+          content_type = downloaded_file.content_type || "image/jpeg"
+          extension = determine_extension(content_type)
+          filename = "avatar_#{@user.id}_#{Time.current.to_i}#{extension}"
+
+          # Attach the avatar
+          @user.avatar.attach(
+            io: downloaded_file,
+            filename: filename,
+            content_type: content_type
+          )
+        end
+      end
+
+      def determine_extension(content_type)
+        case content_type
+        when /jpeg|jpg/
+          ".jpg"
+        when /png/
+          ".png"
+        when /gif/
+          ".gif"
+        when /webp/
+          ".webp"
+        else
+          ".jpg"
+        end
+      end
+    end
+  end
+end

data/app/views/layouts/panda/core/admin_simple.html.erb

@@ -1,4 +1,5 @@
 <%= render "panda/core/shared/header", html_class: "h-full", body_class: "bg-gradient-admin" %>
+<%= render "panda/core/admin/shared/flash" %>
 <div class="flex flex-col items-center justify-center min-h-screen px-4">
   <%= yield %>
 </div>

data/app/views/panda/core/admin/my_profile/edit.html.erb

@@ -8,7 +8,32 @@
               data: { controller: "theme-form" } do |f| %>
     <%= render Panda::Core::Admin::FormErrorComponent.new(model: user) %>
 
-    <div class="space-y-4">
+    <div class="space-y-6">
+      <!-- Avatar Section -->
+      <div class="col-span-full">
+        <%= f.label :avatar, "Profile Picture", class: "block text-sm/6 font-medium text-gray-900 dark:text-white" %>
+        <div class="mt-2 flex items-center gap-x-3">
+          <% if user.avatar.attached? %>
+            <%= image_tag main_app.url_for(user.avatar), alt: user.name, class: "w-48 h-48 max-w-48 rounded-full object-cover" %>
+          <% else %>
+            <svg viewBox="0 0 24 24" fill="currentColor" aria-hidden="true" class="w-48 h-48 text-gray-300 dark:text-gray-500">
+              <path d="M18.685 19.097A9.723 9.723 0 0 0 21.75 12c0-5.385-4.365-9.75-9.75-9.75S2.25 6.615 2.25 12a9.723 9.723 0 0 0 3.065 7.097A9.716 9.716 0 0 0 12 21.75a9.716 9.716 0 0 0 6.685-2.653Zm-12.54-1.285A7.486 7.486 0 0 1 12 15a7.486 7.486 0 0 1 5.855 2.812A8.224 8.224 0 0 1 12 20.25a8.224 8.224 0 0 1-5.855-2.438ZM15.75 9a3.75 3.75 0 1 1-7.5 0 3.75 3.75 0 0 1 7.5 0Z" clip-rule="evenodd" fill-rule="evenodd" />
+            </svg>
+          <% end %>
+          <label for="<%= f.field_id(:avatar) %>" class="rounded-md bg-white px-3 py-2 text-sm font-semibold text-gray-900 shadow-xs inset-ring inset-ring-gray-300 hover:bg-gray-50 cursor-pointer dark:bg-white/10 dark:text-white dark:shadow-none dark:inset-ring-white/5 dark:hover:bg-white/20">
+            Change
+          </label>
+          <%= f.file_field :avatar,
+              accept: "image/png,image/jpeg,image/jpg,image/gif,image/webp",
+              class: "sr-only" %>
+        </div>
+        <% if user.avatar.attached? %>
+          <p class="mt-2 text-xs text-gray-600 dark:text-gray-400">
+            Current: <%= user.avatar.filename %> (<%= number_to_human_size(user.avatar.byte_size) %>)
+          </p>
+        <% end %>
+      </div>
+
       <div class="field">
         <%= f.label :name %>
         <%= f.text_field :name %>

data/app/views/panda/core/admin/sessions/new.html.erb

@@ -11,13 +11,12 @@
     <% providers = @providers || Panda::Core.config.authentication_providers.keys %>
     <% providers.each do |provider| %>
       <% provider_config = Panda::Core.config.authentication_providers[provider] %>
-      <% provider_name = provider_config&.dig(:name) || provider.to_s.humanize %>
       <% provider_path = provider_config&.dig(:path_name) || provider %>
       <div class="mt-4 text-center sm:mx-auto sm:w-full sm:max-w-sm">
         <%= form_tag "#{Panda::Core.config.admin_path}/auth/#{provider_path}", method: "post", data: {turbo: false} do %>
           <button type="submit" id="button-sign-in-<%= provider_path %>" class="inline-flex gap-x-2 items-center py-2.5 px-3.5 mx-auto mb-4 bg-white text-gray-900 rounded-md border min-w-56 border-neutral-400 hover:bg-gray-50">
-            <i class="fa-brands fa-<%= oauth_provider_icon(provider) %> text-xl mr-1"></i>
-            Sign in with <%= provider_name %>
+            <i class="<%= oauth_provider_non_brand?(provider) ? 'fa-solid' : 'fa-brands' %> fa-<%= oauth_provider_icon(provider) %> text-xl mr-1"></i>
+            Sign in with <%= oauth_provider_name(provider, provider_config) %>
           </button>
         <% end %>
       </div>

data/app/views/panda/core/admin/shared/_breadcrumbs.html.erb

@@ -5,13 +5,13 @@
     <ol role="list" class="px-4 w-full sm:px-6">
       <li class="inline-block">
         <a href="<%= Panda::Core.config.admin_path %>" class="<%= breadcrumb_styles %>">
-          <i class="fa fa-solid fa-house"></i>
+          <i class="fa fa-solid fa-house fa-fw inline-block w-4 text-center"></i>
           <span class="sr-only">Home</span>
         </a>
       </li>
       <% breadcrumbs.each do |crumb| %>
         <li class="inline-block">
-          <i class="fa fa-regular fa-chevron-right font-light <%= breadcrumb_styles %> py-5 px-2"></i>
+          <i class="fa-solid fa-chevron-right fa-fw inline-block w-4 text-center font-light <%= breadcrumb_styles %> px-2"></i>
           <a href="<%= crumb.path %>" class="text-sm font-normal <%= breadcrumb_styles %>"><%= crumb.name %></a>
         </li>
       <% end %>
@@ -19,7 +19,7 @@
   </nav>
 
   <% if content_for :sidebar %>
-    <div class="pt-4 pr-8 text-black/80" tabindex="-1" data-controller="toggle">
+    <div class="pt-4 pr-8 text-black/80" tabindex="-1">
       <button type="button" id="slideover-toggle" data-action="click->toggle#toggle touch->toggle#toggle" class="text-sm font-light">
         <i class="mr-1 fa-light fa-gear"></i> <%= yield :sidebar_title %>
       </button>

data/app/views/panda/core/admin/shared/_sidebar.html.erb

@@ -1,18 +1,21 @@
 <nav class="flex flex-col flex-1">
   <ul role="list" class="flex flex-col flex-1">
     <a class="block p-0 mt-4 mb-4 ml-2 text-xl font-medium text-white"><%= Panda::Core.config.admin_title || "Panda Admin" %></a>
-    <% Panda::Core.config.admin_navigation_items&.call(current_user)&.each do |item| %>
+    <%
+      # Get navigation items in original order for display
+      nav_items = Panda::Core.config.admin_navigation_items&.call(current_user) || []
+
+      # Find the most specific matching path by checking longest paths first
+      active_path = nav_items
+        .sort_by { |item| -item[:path].length }
+        .find { |item| request.path == item[:path] || request.path.starts_with?(item[:path] + "/") }
+        &.dig(:path)
+    %>
+    <% nav_items.each do |item| %>
       <li>
         <%
-          # Exact match for dashboard, starts_with for others
-          # Check if current path matches this nav item
-          is_active = if request.path == item[:path]
-            true
-          elsif request.path.starts_with?(item[:path] + "/")
-            true
-          else
-            false
-          end
+          # Check if this item is the active one
+          is_active = item[:path] == active_path
         %>
         <%= link_to item[:path], class: "#{is_active ? "bg-mid text-white relative flex items-center transition-all py-3 px-2 mb-2 rounded-md group gap-x-3 text-base leading-6 font-normal" : "text-white hover:bg-mid/60 transition-all group flex items-center gap-x-3 py-3 px-2 mb-2 rounded-md text-base leading-6 font-normal"}" do %>
           <span class="text-center w-6"><i class="<%= item[:icon] %> text-xl fa-fw"></i></span>
@@ -28,10 +31,12 @@
     </li>
     <li class="mt-auto">
       <%= link_to panda_core.edit_admin_my_profile_path, class: "text-white hover:bg-mid/60 transition-all group flex items-center gap-x-3 py-3 px-2 mb-2 rounded-md text-base leading-6 font-normal w-full", title: "Edit my Profile" do %>
-        <% if !current_user.image_url.to_s.empty? %>
+        <% if current_user.avatar.attached? %>
+          <span class="text-center w-6"><%= image_tag main_app.url_for(current_user.avatar), alt: current_user.name, class: "w-auto h-7 rounded-full object-cover" %></span>
+        <% elsif !current_user.image_url.to_s.empty? %>
           <span class="text-center w-6"><img src="<%= current_user.image_url %>" class="w-auto h-7 rounded-full"></span>
         <% else %>
-          <span class="text-center w-6"><i class="text-xl fa-regular fa-circle-user fa-fw"></i></span>
+          <span class="text-center w-6"><i class="text-xl fa-solid fa-circle-user fa-fw"></i></span>
         <% end %>
         <span><%= current_user.name %></span>
       <% end %>

data/config/importmap.rb

@@ -1,15 +1,23 @@
 # frozen_string_literal: true
 
 # Panda Core application and controllers
-pin "panda/core/application", to: "panda/core/application.js"
-pin "panda/core/controllers/index", to: "panda/core/controllers/index.js"
-pin_all_from Panda::Core::Engine.root.join("app/javascript/panda/core/controllers"), under: "panda/core/controllers"
+# Served via Rack::Static middleware from app/javascript
+pin "panda/core/application", to: "/panda/core/application.js"
+pin "panda/core/controllers/index", to: "/panda/core/controllers/index.js"
+pin "panda/core/controllers/toggle_controller", to: "/panda/core/controllers/toggle_controller.js"
+pin "panda/core/controllers/theme_form_controller", to: "/panda/core/controllers/theme_form_controller.js"
+pin "panda/core/tailwindplus-elements", to: "/panda/core/tailwindplus-elements.js"
 
 # Base JavaScript dependencies for Panda Core (vendored for reliability)
-pin "@hotwired/stimulus", to: "panda/core/vendor/@hotwired--stimulus.js", preload: true # @3.2.2
-pin "@hotwired/turbo", to: "panda/core/vendor/@hotwired--turbo.js", preload: true # @8.0.18
-pin "@rails/actioncable/src", to: "panda/core/vendor/@rails--actioncable--src.js", preload: true # @8.0.201
-pin "tailwindcss-stimulus-components", to: "panda/core/tailwindcss-stimulus-components.js" # @6.1.3
+pin "@hotwired/stimulus", to: "/panda/core/vendor/@hotwired--stimulus.js", preload: true # @3.2.2
+pin "@hotwired/turbo", to: "/panda/core/vendor/@hotwired--turbo.js", preload: true # @8.0.18
+pin "@rails/actioncable/src", to: "/panda/core/vendor/@rails--actioncable--src.js", preload: true # @8.0.201
+pin "tailwindcss-stimulus-components", to: "/panda/core/tailwindcss-stimulus-components.js" # @6.1.3
 
 # Font Awesome icons (from CDN)
 pin "@fortawesome/fontawesome-free", to: "https://ga.jspm.io/npm:@fortawesome/fontawesome-free@7.1.0/js/all.js"
+
+# Tailwind Plus Elements - Vanilla JS interactive components (from CDN)
+# Provides: Autocomplete, Command palette, Dialog, Disclosure, Dropdown menu, Popover, Select, Tabs
+# Note: Using esm.sh instead of jsdelivr for better ES module support
+pin "@tailwindplus/elements", to: "https://esm.sh/@tailwindplus/elements@1", preload: false

data/config/routes.rb

@@ -18,5 +18,14 @@ Panda::Core::Engine.routes.draw do
 
     # Profile management
     resource :my_profile, only: %i[edit update], controller: "admin/my_profile", path: "my_profile"
+
+    # Test-only authentication endpoint (available in development and test environments)
+    # This bypasses OAuth for faster, more reliable test execution
+    # Development: Used by Capybara system tests which run Rails server in development mode
+    # Test: Used by controller/request tests
+    unless Rails.env.production?
+      get "/test_login/:user_id", to: "admin/test_sessions#create", as: :test_login
+      post "/test_sessions", to: "admin/test_sessions#create", as: :test_sessions
+    end
   end
 end

data/db/migrate/20250811120000_add_oauth_avatar_url_to_panda_core_users.rb

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+class AddOauthAvatarUrlToPandaCoreUsers < ActiveRecord::Migration[8.0]
+  def change
+    add_column :panda_core_users, :oauth_avatar_url, :string
+  end
+end

data/lib/panda/core/engine.rb

@@ -19,17 +19,26 @@ module Panda
       config.autoload_paths += Dir[root.join("app", "controllers")]
       config.autoload_paths += Dir[root.join("app", "builders")]
       config.autoload_paths += Dir[root.join("app", "components")]
+      config.autoload_paths += Dir[root.join("app", "services")]
 
       # Make files in public available to the main app (e.g. /panda-core-assets/panda-logo.png)
-      config.app_middleware.use(
-        Rack::Static,
+      config.middleware.use Rack::Static,
         urls: ["/panda-core-assets"],
         root: Panda::Core::Engine.root.join("public"),
         header_rules: [
           # Disable caching in development for instant CSS updates
           [:all, {"Cache-Control" => Rails.env.development? ? "no-cache, no-store, must-revalidate" : "public, max-age=31536000"}]
         ]
-      )
+
+      # Make JavaScript files available for importmap
+      # Serve from app/javascript with proper MIME types
+      config.middleware.use Rack::Static,
+        urls: ["/panda", "/panda/core"],
+        root: Panda::Core::Engine.root.join("app/javascript"),
+        header_rules: [
+          [:all, {"Cache-Control" => Rails.env.development? ? "no-cache, no-store, must-revalidate" : "public, max-age=31536000",
+                  "Content-Type" => "text/javascript; charset=utf-8"}]
+        ]
 
       config.generators do |g|
         g.test_framework :rspec

data/lib/panda/core/services/base_service.rb

@@ -2,18 +2,33 @@ module Panda
   module Core
     module Services
       class BaseService
-        def self.call(*args)
-          new(*args).call
+        # Simple result object for service responses
+        class Result
+          attr_reader :payload, :errors
+
+          def initialize(success:, payload: {}, errors: nil)
+            @success = success
+            @payload = payload
+            @errors = errors
+          end
+
+          def success?
+            @success
+          end
+        end
+
+        def self.call(**kwargs)
+          new(**kwargs).call
         end
 
         private
 
         def success(payload = {})
-          OpenStruct.new(success?: true, payload: payload)
+          Result.new(success: true, payload: payload)
         end
 
         def failure(errors)
-          OpenStruct.new(success?: false, errors: errors)
+          Result.new(success: false, errors: errors)
         end
       end
     end

data/lib/panda/core/version.rb

@@ -2,6 +2,6 @@
 
 module Panda
   module Core
-    VERSION = "0.4.1"
+    VERSION = "0.6.0"
   end
 end

data/lib/panda/core.rb

@@ -14,4 +14,5 @@ require_relative "core/version"
 require_relative "core/configuration"
 require_relative "core/asset_loader"
 require_relative "core/debug"
+require_relative "core/services/base_service"
 require_relative "core/engine" if defined?(Rails)