panda-cms 0.10.0 → 0.10.3

data/db/migrate/20240923234535_add_depth_to_panda_cms_menus.rb

@@ -3,11 +3,5 @@
 class AddDepthToPandaCMSMenus < ActiveRecord::Migration[7.2]
   def change
     add_column :panda_cms_menus, :depth, :integer, null: true, default: nil
-
-    homepage = Panda::CMS::Page.find_by(path: "/")
-    return unless homepage
-
-    main_menu = Panda::CMS::Menu.find_by(start_page_id: homepage.id, kind: :auto)
-    main_menu&.update(depth: 2)
   end
 end

data/db/migrate/20250106223303_add_author_id_to_panda_cms_posts.rb

@@ -2,8 +2,6 @@
 
 class AddAuthorIdToPandaCMSPosts < ActiveRecord::Migration[8.0]
   def change
-    unless column_exists?(:panda_cms_posts, :author_id)
-      add_reference :panda_cms_posts, :author, type: :uuid, foreign_key: {to_table: :panda_core_users}
-    end
+    add_reference :panda_cms_posts, :author, type: :uuid, foreign_key: {to_table: :panda_core_users}
   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/db/migrate/20251117234530_add_index_to_visited_at_on_panda_cms_visits.rb

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+class AddIndexToVisitedAtOnPandaCMSVisits < ActiveRecord::Migration[7.1]
+  def change
+    add_index :panda_cms_visits, :visited_at
+  end
+end

data/db/migrate/20251118015100_backfill_visited_at_for_existing_visits.rb

@@ -0,0 +1,17 @@
+# frozen_string_literal: true
+
+class BackfillVisitedAtForExistingVisits < ActiveRecord::Migration[7.1]
+  def up
+    # Backfill visited_at from created_at for all existing visits where visited_at is NULL
+    # This ensures historical visit data shows up in dashboard statistics
+    execute <<-SQL
+      UPDATE panda_cms_visits
+      SET visited_at = created_at
+      WHERE visited_at IS NULL
+    SQL
+  end
+
+  def down
+    # No need to reverse - visited_at values are still valid
+  end
+end

data/db/seeds.rb

@@ -5,3 +5,8 @@ generator.create_templates
 generator.create_pages
 generator.create_menus
 Panda::CMS::Template.generate_missing_blocks
+
+if (homepage = Panda::CMS::Page.find_by(path: "/"))
+  main_menu = Panda::CMS::Menu.find_by(start_page_id: homepage.id, kind: :auto)
+  main_menu&.update(depth: 2)
+end

data/lib/panda/cms/asset_loader.rb

@@ -35,11 +35,22 @@ 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?)
+          # In test, never use GitHub assets
+          return false if Rails.env.test? || in_test_environment?
+
+          # In production, prefer local assets over GitHub
+          # CMS uses importmap for JS and gets CSS from panda-core
+          # Only use GitHub as fallback if explicitly enabled
+          if Rails.env.production?
+            # Check if compiled assets exist locally (though CMS doesn't bundle)
+            return false if compiled_assets_available?
+
+            # Only use GitHub as fallback if explicitly enabled
+            return ENV["PANDA_CMS_USE_GITHUB_ASSETS"] == "true"
+          end
+
+          # In development, use GitHub assets only when explicitly enabled or development assets unavailable
+          ENV["PANDA_CMS_USE_GITHUB_ASSETS"] == "true" || !development_assets_available?
         end
 
         # Download assets from GitHub to local cache
@@ -105,39 +116,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 +143,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
@@ -202,19 +167,13 @@ module Panda
           if (Rails.env.test? || in_test_environment?) && compiled_assets_available?
             "/panda-cms-assets/"
           else
-            "https://github.com/tastybamboo/panda-cms/releases/download/#{version}/"
+            # Use raw.githubusercontent.com instead of releases
+            "https://raw.githubusercontent.com/tastybamboo/panda-cms/main/public/panda-cms-assets/"
           end
         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 +181,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 +190,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 +217,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?