panda-cms 0.8.2 → 0.10.2

data/config/importmap.rb

@@ -1,15 +1,15 @@
 # frozen_string_literal: true
 
-pin "application_panda_cms", to: "panda_cms/application_panda_cms.js", preload: true
+# Base dependencies are now in panda-core (Stimulus, Turbo, Font Awesome, etc.)
+# This file only contains CMS-specific pins
+# NOTE: Paths must be absolute (starting with /) because Rack::Static serves
+# from /panda/cms/, not from asset pipeline /assets/
 
-pin "@hotwired/turbo", to: "@hotwired--turbo.js", preload: true # @8.0.12
-pin "@rails/actioncable/src", to: "@rails--actioncable--src.js", preload: true # @7.2.101
-pin "@hotwired/stimulus", to: "@hotwired--stimulus.js" # @3.2.2
-pin "@hotwired/stimulus-loading", to: "panda_cms/stimulus-loading.js", preload: true
-pin "tailwindcss-stimulus-components" # @6.1.2
-pin "@editorjs/editorjs", to: "panda/cms/editor/editorjs.js" # @2.30.6
+pin "panda/cms/application", to: "/panda/cms/application.js", preload: true
+pin "@hotwired/stimulus-loading", to: "/panda/cms/stimulus-loading.js", preload: true
+pin "@editorjs/editorjs", to: "/panda/cms/editor/editorjs.js" # @2.30.6
 
 # Pin the controllers directory
-pin "controllers", to: "panda/cms/controllers/index.js"
-pin_all_from Panda::CMS::Engine.root.join("app/javascript/panda/cms/controllers"), under: "controllers"
-pin_all_from Panda::CMS::Engine.root.join("app/javascript/panda/cms/editor"), under: "editor"
+pin "panda/cms/controllers/index", to: "/panda/cms/controllers/index.js"
+pin_all_from Panda::CMS::Engine.root.join("app/javascript/panda/cms/controllers"), under: "controllers", to: "/panda/cms/controllers"
+pin_all_from Panda::CMS::Engine.root.join("app/javascript/panda/cms/editor"), under: "editor", to: "/panda/cms/editor"

data/config/initializers/panda/cms/healthcheck_log_silencer.rb.disabled

@@ -0,0 +1,31 @@
+# frozen_string_literal: true
+
+require "silencer/rails/logger"
+
+# Don't log requests to the healthcheck endpoint
+Rails.application.configure do
+  # Rails 8 renamed Rails::Rack::Logger to ActionDispatch::Request::Logger
+  # Determine which logger middleware class to use
+  logger_class = if Object.const_defined?("ActionDispatch::Request::Logger")
+    ActionDispatch::Request::Logger
+  elsif Object.const_defined?("Rails::Rack::Logger")
+    Rails::Rack::Logger
+  else
+    # If neither exists, skip the middleware swap
+    nil
+  end
+
+  if logger_class
+    begin
+      config.middleware.swap(
+        logger_class,
+        Silencer::Logger,
+        config.log_tags,
+        silence: ["/up"]
+      )
+    rescue RuntimeError => e
+      # Silently fail if middleware doesn't exist in stack
+      raise unless e.message.include?("No such middleware")
+    end
+  end
+end

data/config/initializers/panda/cms.rb

@@ -1,15 +1,57 @@
 # frozen_string_literal: true
 
-Panda::CMS.configure do |config|
-  # The main title of your website
-  config.title = "Demo Site"
-
-  # Site access control
-  config.require_login_to_view = false
-end
-
-# Admin path is now configured via Panda::Core
+# This file is an example of Panda configuration.
+# In your application, this should be at config/initializers/panda.rb
 Panda::Core.configure do |config|
-  # The path to the administration panel
   config.admin_path = "/admin"
+
+  config.login_page_title = "Panda Admin"
+
+  # Configure authentication providers
+  # Uncomment and configure the providers you want to use
+  # Don't forget to add the corresponding gems (e.g., omniauth-google-oauth2)
+  #
+  # config.authentication_providers = {
+  #   google_oauth2: {
+  #     enabled: true,
+  #     name: "Google",  # Display name for the button
+  #     client_id: Rails.application.credentials.dig(:google, :client_id),
+  #     client_secret: Rails.application.credentials.dig(:google, :client_secret),
+  #     options: {
+  #       scope: "email,profile",
+  #       prompt: "select_account",
+  #       hd: "yourdomain.com" # Specify your domain here if you want to restrict admin logins
+  #     }
+  #   }
+  # }
+
+  # Configure the session token cookie name
+  config.session_token_cookie = :panda_session
+
+  # Configure the user class for the application
+  config.user_class = "Panda::Core::User"
+
+  # Configure the user identity class for the application
+  config.user_identity_class = "Panda::Core::UserIdentity"
+
+  # Configure the storage provider (default: :active_storage)
+  # config.storage_provider = :active_storage
+
+  # Configure the cache store (default: :memory_store)
+  # config.cache_store = :memory_store
 end
+
+# Optional CMS-specific configuration
+# Panda::CMS.configure do |config|
+#   # Site access control
+#   config.require_login_to_view = false
+# end
+
+# Optional EditorJS configuration
+# Panda::Editor.configure do |config|
+#   # Additional EditorJS tools to load
+#   # config.editor_js_tools = []
+#
+#   # EditorJS tool configurations
+#   # config.editor_js_tool_config = {}
+# end

data/config/locales/en.yml

@@ -29,6 +29,17 @@ en:
         title: Title
         path: URL
         panda_cms_template_id: Template
+        page_type: Page Type
+        seo_title: SEO Title
+        seo_description: SEO Description
+        seo_keywords: SEO Keywords
+        seo_index_mode: Search Engine Visibility
+        canonical_url: Canonical URL
+        og_title: OpenGraph Title
+        og_description: OpenGraph Description
+        og_type: OpenGraph Type
+        og_image: OpenGraph Image
+        inherit_seo: Inherit Settings
       panda/cms/post:
         title: Title
         slug: URL
@@ -36,6 +47,15 @@ en:
         user_id: Author
         published_at: Published At
         post_content: Content
+        seo_title: SEO Title
+        seo_description: SEO Description
+        seo_keywords: SEO Keywords
+        seo_index_mode: Search Engine Visibility
+        canonical_url: Canonical URL
+        og_title: OpenGraph Title
+        og_description: OpenGraph Description
+        og_type: OpenGraph Type
+        og_image: OpenGraph Image
         statuses:
           active: Active
           draft: Draft
@@ -52,3 +72,24 @@ en:
         lastname: Last Name
         email: Email Address
         current_theme: Theme
+    enums:
+      panda/cms/page:
+        seo_index_mode:
+          visible: Visible to search engines
+          invisible: Hidden from search engines
+        og_type:
+          website: Website
+          article: Article
+          profile: Profile
+          video: Video
+          book: Book
+      panda/cms/post:
+        seo_index_mode:
+          visible: Visible to search engines
+          invisible: Hidden from search engines
+        og_type:
+          website: Website
+          article: Article
+          profile: Profile
+          video: Video
+          book: Book

data/config/routes.rb

@@ -1,14 +1,16 @@
 # frozen_string_literal: true
 
 Panda::CMS::Engine.routes.draw do
-  constraints Panda::Core::AdminConstraint.new(&:present?) do
+  # Test authentication endpoint moved to panda-core at /admin/test_login/:user_id
+
+  constraints Panda::Core::AdminConstraint.new do
     # CMS-specific dashboard (using Core's admin_path)
-    admin_path = Panda::Core.configuration.admin_path
+    admin_path = Panda::Core.config.admin_path
     get "#{admin_path}/cms", to: "admin/dashboard#show", as: :admin_cms_dashboard
 
     namespace admin_path.delete_prefix("/").to_sym, path: "#{admin_path}/cms", as: :admin_cms, module: :admin do
       resources :files
-      resources :forms, only: %i[index show]
+      resources :forms
       resources :menus
       resources :pages do
         resources :block_contents, only: %i[update]

data/db/migrate/20240305000000_convert_html_content_to_editor_js.rb

@@ -4,12 +4,12 @@ class ConvertHtmlContentToEditorJs < ActiveRecord::Migration[7.1]
   def up
     # First, let's check if the converter service exists
     converter_path = Panda::CMS::Engine.root.join("app/services/panda/cms/html_to_editor_js_converter.rb")
-    
+
     unless File.exist?(converter_path)
       Rails.logger.info "HtmlToEditorJsConverter service not found. Skipping HTML to EditorJS conversion."
       return
     end
-    
+
     require converter_path
 
     # Check if we have any existing valid EditorJS content

data/db/migrate/20240315125421_add_nested_sets_to_panda_cms_pages.rb

@@ -8,7 +8,12 @@ class AddNestedSetsToPandaCMSPages < ActiveRecord::Migration[7.1]
 
     # This is necessary to update :lft and :rgt columns
     Panda::CMS::Page.reset_column_information
-    Panda::CMS::Page.rebuild!
+
+    # Only rebuild if there are existing pages
+    # On fresh installs, there won't be any pages yet
+    if Panda::CMS::Page.any?
+      Panda::CMS::Page.rebuild!
+    end
   end
 
   def self.down

data/db/migrate/20250809231125_migrate_users_to_panda_core.rb

@@ -7,27 +7,29 @@ class MigrateUsersToPandaCore < ActiveRecord::Migration[8.0]
     if table_exists?(:panda_cms_users) && table_exists?(:panda_core_users)
       # Check if there's any data to migrate
       cms_user_count = ActiveRecord::Base.connection.select_value("SELECT COUNT(*) FROM panda_cms_users")
-      return if cms_user_count == 0
-      # Copy all user data
-      execute <<-SQL
-        INSERT INTO panda_core_users (
-          id, name, email, image_url, is_admin, created_at, updated_at
-        )
-        SELECT 
-          id,
-          COALESCE(name, CONCAT(firstname, ' ', lastname), 'Unknown User'),
-          email,
-          image_url,
-          COALESCE(admin, false),
-          created_at,
-          updated_at
-        FROM panda_cms_users
-        WHERE NOT EXISTS (
-          SELECT 1 FROM panda_core_users WHERE panda_core_users.id = panda_cms_users.id
-        )
-      SQL
 
-      # Update foreign key references in other tables
+      if cms_user_count > 0
+        # Copy all user data
+        execute <<-SQL
+          INSERT INTO panda_core_users (
+            id, name, email, image_url, is_admin, created_at, updated_at
+          )
+          SELECT
+            id,
+            COALESCE(name, CONCAT(firstname, ' ', lastname), 'Unknown User'),
+            email,
+            image_url,
+            COALESCE(admin, false),
+            created_at,
+            updated_at
+          FROM panda_cms_users
+          WHERE NOT EXISTS (
+            SELECT 1 FROM panda_core_users WHERE panda_core_users.id = panda_cms_users.id
+          )
+        SQL
+      end
+
+      # Update foreign key references in other tables (always do this, even if no data)
 
       # Posts author_id
       if column_exists?(:panda_cms_posts, :author_id)
@@ -47,7 +49,7 @@ class MigrateUsersToPandaCore < ActiveRecord::Migration[8.0]
         add_foreign_key :panda_cms_visits, :panda_core_users, column: :user_id, primary_key: :id
       end
 
-      # Drop the old table
+      # Always drop the old table if it exists (even if it was empty)
       drop_table :panda_cms_users
     end
   end

data/db/migrate/20251104150640_add_cached_last_updated_at_to_panda_cms_pages.rb

@@ -0,0 +1,22 @@
+class AddCachedLastUpdatedAtToPandaCMSPages < ActiveRecord::Migration[8.0]
+  def change
+    add_column :panda_cms_pages, :cached_last_updated_at, :datetime
+    add_index :panda_cms_pages, :cached_last_updated_at
+
+    # Backfill existing pages
+    reversible do |dir|
+      dir.up do
+        execute <<-SQL
+          UPDATE panda_cms_pages
+          SET cached_last_updated_at = GREATEST(
+            updated_at,
+            COALESCE(
+              (SELECT MAX(updated_at) FROM panda_cms_block_contents WHERE panda_cms_page_id = panda_cms_pages.id),
+              updated_at
+            )
+          )
+        SQL
+      end
+    end
+  end
+end

data/db/migrate/20251104172242_add_page_type_to_panda_cms_pages.rb

@@ -0,0 +1,6 @@
+class AddPageTypeToPandaCMSPages < ActiveRecord::Migration[8.0]
+  def change
+    add_column :panda_cms_pages, :page_type, :string, default: "standard", null: false
+    add_index :panda_cms_pages, :page_type
+  end
+end

data/db/migrate/20251104172638_set_page_types_for_existing_pages.rb

@@ -0,0 +1,27 @@
+class SetPageTypesForExistingPages < ActiveRecord::Migration[8.0]
+  def up
+    # Set system type for error pages
+    execute <<-SQL
+      UPDATE panda_cms_pages
+      SET page_type = 'system'
+      WHERE path IN ('/404', '/500')
+    SQL
+
+    # Set posts type for news/blog pages
+    execute <<-SQL
+      UPDATE panda_cms_pages
+      SET page_type = 'posts'
+      WHERE path LIKE '%news%' OR path LIKE '%blog%' OR path LIKE '%updates%'
+    SQL
+
+    # All other pages remain as 'standard' (the default)
+  end
+
+  def down
+    # Reset all to standard
+    execute <<-SQL
+      UPDATE panda_cms_pages
+      SET page_type = 'standard'
+    SQL
+  end
+end

data/db/migrate/20251105000001_add_pending_review_status_to_pages_and_posts.panda_cms.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+class AddPendingReviewStatusToPagesAndPosts < ActiveRecord::Migration[8.0]
+  def up
+    # Add pending_review status to pages and posts for content approval workflow
+    # This allows content creators to submit drafts for editorial review
+
+    # Note: We're not modifying the enum directly in the database
+    # Rails enums are string-based, so we just document the new allowed value
+    # The application code will handle the new status value
+
+    # Update any existing pages/posts that might benefit from this status
+    # (In a real migration, you'd add logic here if needed)
+  end
+
+  def down
+    # Convert any pending_review items back to draft
+    Panda::CMS::Page.where(status: "pending_review").update_all(status: "draft")
+    Panda::CMS::Post.where(status: "pending_review").update_all(status: "draft")
+  end
+end

data/db/migrate/20251109131150_add_seo_fields_to_pages.rb

@@ -0,0 +1,32 @@
+class AddSEOFieldsToPages < ActiveRecord::Migration[8.0]
+  def change
+    # Create enum types for SEO fields
+    create_enum :panda_cms_seo_index_mode, ["visible", "invisible"]
+    create_enum :panda_cms_og_type, ["website", "article", "profile", "video", "book"]
+
+    # Add SEO basic fields
+    add_column :panda_cms_pages, :seo_title, :string
+    add_column :panda_cms_pages, :seo_description, :text
+    add_column :panda_cms_pages, :seo_keywords, :string
+
+    # Robots control (visible/invisible in base CMS)
+    add_column :panda_cms_pages, :seo_index_mode, :enum,
+      enum_type: "panda_cms_seo_index_mode",
+      default: "visible",
+      null: false
+
+    # Canonical URL
+    add_column :panda_cms_pages, :canonical_url, :string
+
+    # OpenGraph / Social Sharing
+    add_column :panda_cms_pages, :og_title, :string
+    add_column :panda_cms_pages, :og_description, :text
+    add_column :panda_cms_pages, :og_type, :enum,
+      enum_type: "panda_cms_og_type",
+      default: "website",
+      null: false
+
+    # Inheritance (pages only - they have hierarchy via nested sets)
+    add_column :panda_cms_pages, :inherit_seo, :boolean, default: true, null: false
+  end
+end

data/db/migrate/20251109131205_add_seo_fields_to_posts.rb

@@ -0,0 +1,27 @@
+class AddSEOFieldsToPosts < ActiveRecord::Migration[8.0]
+  def change
+    # Add SEO basic fields
+    add_column :panda_cms_posts, :seo_title, :string
+    add_column :panda_cms_posts, :seo_description, :text
+    add_column :panda_cms_posts, :seo_keywords, :string
+
+    # Robots control (visible/invisible in base CMS)
+    add_column :panda_cms_posts, :seo_index_mode, :enum,
+      enum_type: "panda_cms_seo_index_mode",
+      default: "visible",
+      null: false
+
+    # Canonical URL
+    add_column :panda_cms_posts, :canonical_url, :string
+
+    # OpenGraph / Social Sharing
+    add_column :panda_cms_posts, :og_title, :string
+    add_column :panda_cms_posts, :og_description, :text
+    add_column :panda_cms_posts, :og_type, :enum,
+      enum_type: "panda_cms_og_type",
+      default: "article",  # Posts default to 'article' type
+      null: false
+
+    # Note: No inherit_seo for posts - they don't have hierarchy
+  end
+end

data/db/migrate/20251110114258_add_spam_tracking_to_form_submissions.rb

@@ -0,0 +1,7 @@
+class AddSpamTrackingToFormSubmissions < ActiveRecord::Migration[8.0]
+  def change
+    add_column :panda_cms_form_submissions, :ip_address, :string
+    add_column :panda_cms_form_submissions, :user_agent, :text
+    add_index :panda_cms_form_submissions, :ip_address
+  end
+end

data/db/migrate/20251110122812_add_performance_indexes_to_pages_and_redirects.rb

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+
+class AddPerformanceIndexesToPagesAndRedirects < ActiveRecord::Migration[8.0]
+  def change
+    # Add index to pages.path for fast page lookups
+    # This is the most critical query path in the CMS
+    add_index :panda_cms_pages, :path, name: "index_panda_cms_pages_on_path"
+
+    # Add index to redirects.origin_path for fast redirect lookups
+    # Checked on every request in PagesController#handle_redirects
+    add_index :panda_cms_redirects, :origin_path, name: "index_panda_cms_redirects_on_origin_path"
+  end
+end

data/lib/generators/panda/cms/install_generator.rb

@@ -10,11 +10,8 @@ module Generators
         desc "Adds the basic configuration for Panda CMS to your Rails app."
 
         def create_initializer_file
-          # Add the initializer
-          initializer_path = "config/initializers/panda/cms.rb"
-          unless File.exist?("#{::Rails.root}/#{initializer_path}")
-            FileUtils.cp "#{::Panda::CMS::Engine.root}/#{initializer_path}", "#{::Rails.root}/#{initializer_path}"
-          end
+          # Skip creating initializer - Panda::Core already creates config/initializers/panda.rb
+          # See config/initializers/panda/cms.rb in the gem for an example configuration
 
           # Add the seed loader to the seeds.rb file
           unless File.read("#{::Rails.root}/db/seeds.rb")&.include?("Panda::CMS::Engine.load_seed")

data/lib/panda/cms/asset_loader.rb

@@ -35,11 +35,9 @@ module Panda
 
         # Check if GitHub-hosted assets should be used
         def use_github_assets?
-          # Use GitHub assets in production or when explicitly enabled
-          Rails.env.production? ||
-            ENV["PANDA_CMS_USE_GITHUB_ASSETS"] == "true" ||
-            !development_assets_available? ||
-            ((Rails.env.test? || in_test_environment?) && compiled_assets_available?)
+          # Panda CMS uses importmaps for JavaScript (no compilation needed)
+          # Only use GitHub assets in production or when explicitly enabled
+          false # Always use importmaps like panda-core
         end
 
         # Download assets from GitHub to local cache
@@ -105,39 +103,10 @@ module Panda
         end
 
         def development_asset_tags(options = {})
-          # In test environment with CI, always use compiled assets
-          if (Rails.env.test? || ENV["CI"].present?) && compiled_assets_available?
-            # Use the same logic as GitHub assets but with local paths
-            version = asset_version
-            js_url = "/panda-cms-assets/panda-cms-#{version}.js"
-            css_url = "/panda-cms-assets/panda-cms-#{version}.css"
-
-            tags = []
-
-            # JavaScript tag
-            tags << content_tag(:script, "", {
-              src: js_url,
-              defer: true
-            })
-
-            # CSS tag if exists
-            if cached_asset_exists?(css_url)
-              tags << tag(:link, {
-                rel: "stylesheet",
-                href: css_url
-              })
-            end
-
-            tags.join("\n").html_safe
-          else
-            # In development, just use a simple script tag
-            # The view will handle importmap tags separately
-            content_tag(:script, "", {
-              src: development_javascript_url,
-              type: "module",
-              defer: true
-            })
-          end
+          # Panda CMS uses importmaps for JavaScript (no compiled bundles)
+          # The view will handle importmap tags via <%= javascript_importmap_tags %>
+          # We don't need to return anything here - CSS comes from panda-core
+          "".html_safe
         end
 
         def github_javascript_url
@@ -161,20 +130,8 @@ module Panda
         end
 
         def development_javascript_url
-          # Try cached assets first, then importmap
-          version = asset_version
-          # Try root level first (standalone bundle), then versioned directory
-          root_path = "/panda-cms-assets/panda-cms-#{version}.js"
-          versioned_path = "/panda-cms-assets/#{version}/panda-cms-#{version}.js"
-
-          if cached_asset_exists?(root_path)
-            root_path
-          elsif cached_asset_exists?(versioned_path)
-            versioned_path
-          else
-            # Fallback to importmap or engine asset
-            "/assets/panda/cms/controllers/index.js"
-          end
+          # Always use importmap (no compiled bundles)
+          "/panda/cms/application.js"
         end
 
         def development_css_url
@@ -202,14 +159,7 @@ module Panda
         end
 
         def asset_version
-          # In test environment, use VERSION constant for consistency with compiled assets
-          # In other environments, use git SHA for dynamic versioning
-          # Also check for test environment indicators since Rails.env might be development in specs
-          if Rails.env.test? || ENV["CI"].present? || in_test_environment?
-            Panda::CMS::VERSION
-          else
-            `git rev-parse --short HEAD`.strip
-          end
+          Panda::CMS::VERSION
         end
 
         def in_test_environment?
@@ -217,14 +167,6 @@ module Panda
           defined?(RSpec) && RSpec.respond_to?(:configuration)
         end
 
-        def compiled_assets_available?
-          # Check if compiled assets exist in test location
-          version = asset_version
-          js_file = Rails.public_path.join("panda-cms-assets", "panda-cms-#{version}.js")
-          css_file = Rails.public_path.join("panda-cms-assets", "panda-cms-#{version}.css")
-          js_file.exist? && css_file.exist?
-        end
-
         def development_assets_available?
           # Check if local development assets exist (importmap, etc.)
           importmap_available? || engine_assets_available?
@@ -234,12 +176,18 @@ module Panda
           return false unless defined?(Rails.application.importmap)
 
           begin
-            # Rails 8+ uses a different API for accessing importmap entries
-            if Rails.application.importmap.respond_to?(:to_json)
-              importmap_json = JSON.parse(Rails.application.importmap.to_json)
-              importmap_json.any? { |name, _| name.include?("panda") }
+            # Try different API methods depending on importmap-rails version
+            if Rails.application.importmap.respond_to?(:packages)
+              # importmap-rails 2.x - check packages hash
+              Rails.application.importmap.packages.keys.any? { |name| name.to_s.include?("panda") }
             elsif Rails.application.importmap.respond_to?(:entries)
+              # importmap-rails 1.x - check entries array
               Rails.application.importmap.entries.any? { |entry| entry.name.include?("panda") }
+            elsif Rails.application.importmap.respond_to?(:to_json)
+              # Fallback with proper resolver
+              # Note: to_json requires a resolver in newer versions
+              # We'll just return false if we can't access packages/entries
+              false
             else
               false
             end
@@ -251,10 +199,17 @@ module Panda
 
         def engine_assets_available?
           # Check if engine's JavaScript files are available
-          engine_js_path = Rails.root.join("..", "..", "app", "javascript", "panda", "cms", "controllers", "index.js")
+          engine_js_path = Panda::CMS::Engine.root.join("app", "javascript", "panda", "cms", "controllers", "index.js")
           File.exist?(engine_js_path)
         end
 
+        def compiled_assets_available?
+          # Check if compiled JavaScript bundle exists
+          version = asset_version
+          js_file = Rails.public_path.join("panda-cms-assets", "panda-cms-#{version}.js")
+          js_file.exist?
+        end
+
         def cached_assets_exist?(version)
           cache_dir = local_cache_directory.join(version)
           cache_dir.exist? && cache_dir.join("panda-cms-#{version}.js").exist?
@@ -312,6 +267,21 @@ module Panda
           end
         end
 
+        # Simple response object for HTTP requests
+        class Response
+          attr_reader :code, :body
+
+          def initialize(success:, code:, body:)
+            @success = success
+            @code = code
+            @body = body
+          end
+
+          def success?
+            @success
+          end
+        end
+
         def fetch_url(url)
           uri = URI(url)
           http = Net::HTTP.new(uri.host, uri.port)
@@ -324,14 +294,14 @@ module Panda
 
           response = http.request(request)
 
-          OpenStruct.new(
-            success?: response.code.to_i == 200,
+          Response.new(
+            success: response.code.to_i == 200,
             code: response.code,
             body: response.body
           )
         rescue => e
           Rails.logger.error "[Panda CMS] Network error: #{e.message}"
-          OpenStruct.new(success?: false, code: "error", body: nil)
+          Response.new(success: false, code: "error", body: nil)
         end
 
         def asset_integrity(version, filename)