panda-cms 0.10.0 → 0.10.2

data/config/importmap.rb

@@ -2,12 +2,14 @@
 
 # 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 "application_panda_cms", to: "panda/cms/application_panda_cms.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 "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/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

@@ -10,7 +10,7 @@ Panda::CMS::Engine.routes.draw do
 
     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/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/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,25 +130,8 @@ module Panda
         end
 
         def development_javascript_url
-          # In development, use importmap for live-reloading
-          # Only use compiled assets in test/CI environments
-          if Rails.env.development?
-            "/panda/cms/application_panda_cms.js"
-          else
-            # Try cached assets for test environment
-            version = asset_version
-            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
-              "/panda/cms/application_panda_cms.js"
-            end
-          end
+          # Always use importmap (no compiled bundles)
+          "/panda/cms/application.js"
         end
 
         def development_css_url
@@ -207,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?
@@ -222,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?
@@ -239,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
@@ -260,6 +203,13 @@ module Panda
           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?