panda-cms 0.7.3 → 0.7.5

data/app/helpers/panda/cms/asset_helper.rb

@@ -0,0 +1,182 @@
+# frozen_string_literal: true
+
+module Panda
+  module CMS
+    module AssetHelper
+      # Include Panda CMS JavaScript and CSS assets
+      # Automatically chooses between GitHub-hosted assets (production)
+      # and local development assets
+      def panda_cms_assets
+        Panda::CMS::AssetLoader.asset_tags.html_safe
+      end
+
+      # Include only Panda CMS JavaScript
+      def panda_cms_javascript
+        js_url = Panda::CMS::AssetLoader.javascript_url
+        return "" unless js_url
+
+        if Panda::CMS::AssetLoader.use_github_assets?
+          # GitHub-hosted assets with integrity check
+          version = Panda::CMS::AssetLoader.send(:asset_version)
+          integrity = asset_integrity(version, "panda-cms-#{version}.js")
+
+          tag_options = {
+            src: js_url
+          }
+          # In CI environment, don't use defer to ensure immediate execution
+          tag_options[:defer] = true unless ENV["GITHUB_ACTIONS"] == "true"
+          # Standalone bundles should NOT use type: "module" - they're regular scripts
+          # Only use type: "module" for importmap/ES module assets
+          if !js_url.include?("panda-cms-assets")
+            tag_options[:type] = "module"
+          end
+          tag_options[:integrity] = integrity if integrity
+          tag_options[:crossorigin] = "anonymous" if integrity
+
+          content_tag(:script, "", tag_options)
+        elsif js_url.include?("panda-cms-assets")
+          # Development assets - check if it's a standalone bundle or importmap
+          defer_option = (ENV["GITHUB_ACTIONS"] == "true") ? {} : {defer: true}
+          javascript_include_tag(js_url, **defer_option)
+        # Standalone bundle - don't use type: "module"
+        else
+          # Importmap asset - use type: "module"
+          defer_option = (ENV["GITHUB_ACTIONS"] == "true") ? {} : {defer: true}
+          javascript_include_tag(js_url, type: "module", **defer_option)
+        end
+      end
+
+      # Include only Panda CMS CSS
+      def panda_cms_stylesheet
+        css_url = Panda::CMS::AssetLoader.css_url
+        return "" unless css_url
+
+        if Panda::CMS::AssetLoader.use_github_assets?
+          # GitHub-hosted assets with integrity check
+          version = Panda::CMS::VERSION
+          integrity = asset_integrity(version, "panda-cms-#{version}.css")
+
+          tag_options = {
+            rel: "stylesheet",
+            href: css_url
+          }
+          tag_options[:integrity] = integrity if integrity
+          tag_options[:crossorigin] = "anonymous" if integrity
+
+          tag(:link, tag_options)
+        else
+          # Development assets
+          stylesheet_link_tag(css_url)
+        end
+      end
+
+      # Get the current Panda CMS version
+      def panda_cms_version
+        Panda::CMS::VERSION
+      end
+
+      # Check if using GitHub-hosted assets
+      def using_github_assets?
+        Panda::CMS::AssetLoader.use_github_assets?
+      end
+
+      # Download and cache assets if needed
+      # Call this in an initializer or controller to pre-cache assets
+      def ensure_panda_cms_assets!
+        Panda::CMS::AssetLoader.ensure_assets_available!
+      end
+
+      # Debug information about asset loading
+      def panda_cms_asset_debug
+        return "" unless Rails.env.development? || Rails.env.test?
+
+        version = Panda::CMS::VERSION
+        js_url = Panda::CMS::AssetLoader.javascript_url
+        css_url = Panda::CMS::AssetLoader.css_url
+        using_github = Panda::CMS::AssetLoader.use_github_assets?
+        compiled_available = Panda::CMS::AssetLoader.send(:compiled_assets_available?)
+
+        # Additional CI debugging
+        asset_file_exists = js_url && File.exist?(Rails.root.join("public#{js_url}"))
+        ci_env = ENV["GITHUB_ACTIONS"] == "true"
+
+        # Check what script tag will be generated
+        script_tag_preview = if using_github
+          tag_options = {src: js_url}
+          tag_options[:defer] = true unless ci_env
+          if !js_url.include?("panda-cms-assets")
+            tag_options[:type] = "module"
+          end
+          "Script tag: <script#{tag_options.map { |k, v| (v == true) ? " #{k}" : " #{k}=\"#{v}\"" }.join}></script>"
+        else
+          "Using development assets"
+        end
+
+        debug_info = [
+          "<!-- Panda CMS Asset Debug Info -->",
+          "<!-- Version: #{version} -->",
+          "<!-- Using GitHub assets: #{using_github} -->",
+          "<!-- Compiled assets available: #{compiled_available} -->",
+          "<!-- JavaScript URL: #{js_url} -->",
+          "<!-- CSS URL: #{css_url || "none"} -->",
+          "<!-- Rails environment: #{Rails.env} -->",
+          "<!-- Asset file exists: #{asset_file_exists} -->",
+          "<!-- Rails root: #{Rails.root} -->",
+          "<!-- CI environment: #{ci_env} -->",
+          "<!-- #{script_tag_preview} -->",
+          "<!-- Params embed_id: #{params[:embed_id] if respond_to?(:params)} -->",
+          "<!-- Compiled at: #{Time.now.utc.iso8601} -->"
+        ]
+
+        debug_info.join("\n").html_safe
+      end
+
+      # Initialize Panda CMS Stimulus application
+      # Call this after the asset tags to ensure proper initialization
+      def panda_cms_stimulus_init
+        javascript_tag(<<~JS, type: "module")
+          // Initialize Panda CMS Stimulus application
+          document.addEventListener('DOMContentLoaded', function() {
+            if (window.pandaCmsStimulus) {
+              console.debug('[Panda CMS] Stimulus application initialized');
+
+              // Set debug mode based on Rails environment
+              const railsEnv = document.body?.dataset?.environment || 'production';
+              window.pandaCmsStimulus.debug = (railsEnv === 'development');
+
+              // Trigger a custom event to signal Panda CMS is ready
+              document.dispatchEvent(new CustomEvent('panda-cms:ready', {
+                detail: {
+                  version: '#{Panda::CMS::VERSION}',
+                  usingGitHubAssets: #{Panda::CMS::AssetLoader.use_github_assets?}
+                }
+              }));
+            } else {
+              console.warn('[Panda CMS] Stimulus application not found. Assets may not have loaded properly.');
+            }
+          });
+        JS
+      end
+
+      # Complete asset loading with initialization
+      # This is the recommended way to include all Panda CMS assets
+      def panda_cms_complete_assets
+        [
+          panda_cms_asset_debug,
+          panda_cms_assets,
+          panda_cms_stimulus_init,
+          # Add immediate JavaScript execution test for CI debugging
+          (Rails.env.test? ? javascript_tag("window.pandaCmsInlineTest = true; console.log('[Panda CMS] Inline script executed');") : "")
+        ].join("\n").html_safe
+      end
+
+      private
+
+      def asset_integrity(version, filename)
+        Panda::CMS::AssetLoader.send(:asset_integrity, version, filename)
+      rescue
+        nil
+      end
+    end
+  end
+end

data/app/helpers/panda/cms/pages_helper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Panda
   module CMS
     module PagesHelper

data/app/helpers/panda/cms/posts_helper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Panda
   module CMS
     module PostsHelper

data/app/helpers/panda/cms/theme_helper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Panda
   module CMS
     module ThemeHelper

data/app/javascript/panda/cms/controllers/editor_form_controller.js

@@ -10,6 +10,10 @@ export default class extends Controller {
 
   connect() {
     this.loadEditorResources();
+    // Enable submit button after a delay as fallback
+    setTimeout(() => {
+      this.enableSubmitButton();
+    }, 1000);
   }
 
   async loadEditorResources() {
@@ -77,6 +81,8 @@ export default class extends Controller {
           holderDiv.dataset.editorInitialized = "true";
           // Add a class to indicate the editor is ready
           holderDiv.classList.add("editor-ready");
+          // Enable the submit button
+          this.enableSubmitButton();
           // Dispatch an event when editor is ready
           this.editorContainerTarget.dispatchEvent(new CustomEvent("editor:ready"));
         },
@@ -119,12 +125,6 @@ export default class extends Controller {
       // Wait for editor to be ready
       await this.editor.isReady;
       console.debug("[Panda CMS] Editor initialized successfully");
-      this.editorContainerTarget.dataset.editorInitialized = "true";
-      holderDiv.dataset.editorInitialized = "true";
-      // Add a class to indicate the editor is ready
-      holderDiv.classList.add("editor-ready");
-      // Dispatch an event when editor is ready
-      this.editorContainerTarget.dispatchEvent(new CustomEvent("editor:ready"));
 
     } catch (error) {
       console.error("[Panda CMS] Editor setup failed:", error);
@@ -133,6 +133,8 @@ export default class extends Controller {
         holderDiv.dataset.editorInitialized = "false";
         holderDiv.classList.remove("editor-ready");
       }
+      // Still enable the submit button even if editor fails
+      this.enableSubmitButton();
     }
   }
 
@@ -190,6 +192,57 @@ export default class extends Controller {
       source: "editorJS",
     };
   }
+  
+  enableSubmitButton() {
+    // Find the submit button in the form and enable it
+    const form = this.element.closest('form');
+    if (form) {
+      const submitButton = form.querySelector('input[type="submit"], button[type="submit"]');
+      if (submitButton) {
+        submitButton.disabled = false;
+      }
+    }
+  }
+
+  async submit(event) {
+    // Prevent the default button click behavior temporarily
+    event.preventDefault();
+    
+    const submitButton = event.target;
+    const form = submitButton.closest('form');
+    
+    // Re-enable the button that was disabled by data-disable-with
+    submitButton.disabled = false;
+    
+    // Ensure editor content is saved before form submission
+    if (this.editor) {
+      try {
+        const outputData = await this.editor.save();
+        outputData.source = "editorJS";
+        const jsonString = JSON.stringify(outputData);
+        this.hiddenFieldTarget.value = jsonString;
+        console.log("[Panda CMS] Editor content saved before submission");
+      } catch (error) {
+        console.error("[Panda CMS] Failed to save editor content:", error);
+      }
+    }
+    
+    // Now trigger the normal form submission (this will let Rails/Turbo handle it properly)
+    if (form) {
+      // Remove our custom action to prevent infinite loop
+      submitButton.removeAttribute('data-action');
+      
+      // Create a new click event that will trigger the normal form submission
+      const clickEvent = new MouseEvent('click', {
+        bubbles: true,
+        cancelable: true,
+        view: window
+      });
+      
+      // Dispatch the click event, which will trigger normal Rails form submission
+      submitButton.dispatchEvent(clickEvent);
+    }
+  }
 
   disconnect() {
     if (this.editor) {

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

@@ -1,16 +1,10 @@
 console.debug("[Panda CMS] Importing Panda CMS Stimulus Controller...")
 
-import { Application as PandaCMSApplication } from "@hotwired/stimulus"
+import { application } from "@hotwired/stimulus-loading"
 
-const pandaCmsApplication = PandaCMSApplication.start()
+console.debug("[Panda CMS] Using shared Stimulus application...")
 
-console.debug("[Panda CMS] Application started...")
-
-// Configure Stimulus development experience
-pandaCmsApplication.debug = false
-window.pandaCmsStimulus = pandaCmsApplication
-
-console.debug("[Panda CMS] window.pandaCmsStimulus available...")
+const pandaCmsApplication = application
 
 console.debug("[Panda CMS] Registering controllers...")
 
@@ -28,6 +22,8 @@ import EditorIframeController from "panda/cms/controllers/editor_iframe_controll
 pandaCmsApplication.register("editor-iframe", EditorIframeController)
 
 console.debug("[Panda CMS] Registering components...")
+import ThemeFormController from "panda/cms/controllers/theme_form_controller";
+pandaCmsApplication.register("theme-form", ThemeFormController);
 
 // Import and register all TailwindCSS Components or just the ones you need
 import { Alert, Autosave, ColorPreview, Dropdown, Modal, Tabs, Popover, Toggle, Slideover } from "tailwindcss-stimulus-components"

data/app/javascript/panda/cms/controllers/slug_controller.js

@@ -14,57 +14,85 @@ export default class extends Controller {
 
   connect() {
     console.debug("[Panda CMS] Slug handler connected...");
-    // Generate path on initial load if title exists
-    if (this.input_textTarget.value) {
-      this.generatePath();
-    }
+    // Don't auto-generate on connect anymore
   }
 
-  generatePath() {
-    const title = this.input_textTarget.value;
-    if (!title) return;
+  generatePath(event) {
+    // Prevent event object from being used as input
+    const title = this.input_textTarget.value.trim();
+    console.debug("[Panda CMS] Generating path from title:", title);
 
-    // Convert title to slug format
-    const slug = title
-      .toLowerCase()
-      .replace(/[^a-z0-9]+/g, "-")
-      .replace(/^-+|-+$/g, "");
+    if (!title) {
+      this.output_textTarget.value = "";
+      return;
+    }
+
+    // Only generate path if output is empty OR user has not manually edited it
+    if (!this.output_textTarget.value || !this.output_textTarget.dataset.userEdited) {
+      // Convert title to slug format
+      const slug = this.createSlug(title);
+      console.debug("[Panda CMS] Generated slug:", slug);
 
-    // Only add year/month prefix for posts
-    if (this.addDatePrefixValue) {
-      // Get current date for year/month
-      const now = new Date();
-      const year = now.getFullYear();
-      const month = String(now.getMonth() + 1).padStart(2, "0");
-
-      // Add leading slash and use date format
-      this.output_textTarget.value = `/${year}/${month}/${slug}`;
-    } else {
-      // Add leading slash for regular pages
-      this.output_textTarget.value = `/${slug}`;
+      // Only add year/month prefix for posts
+      if (this.addDatePrefixValue) {
+        // Get current date for year/month
+        const now = new Date();
+        const year = now.getFullYear();
+        const month = String(now.getMonth() + 1).padStart(2, "0");
+
+        // Add leading slash and use date format
+        this.output_textTarget.value = `/${year}/${month}/${slug}`;
+      } else {
+        // If we have a parent selected, let setPrePath handle it
+        if (this.input_selectTarget.value) {
+          this.setPrePath(slug);
+        } else {
+          // Add leading slash for regular pages
+          this.output_textTarget.value = `/${slug}`;
+        }
+      }
+      console.debug("[Panda CMS] Final path value:", this.output_textTarget.value);
     }
   }
 
-  setPrePath() {
+  setPrePath(slug = null) {
     try {
+      // Don't do anything if we're passed the event object
+      if (slug && typeof slug === 'object') {
+        slug = null;
+      }
+
       const match = this.input_selectTarget.options[this.input_selectTarget.selectedIndex].text.match(/.*\((.*)\)$/);
       if (match) {
-        this.parent_slugs = match[1];
-        const prePath = (this.existing_rootTarget.value + this.parent_slugs).replace(/\/$/, "");
-        // Ensure we don't double up slashes
-        const currentPath = this.output_textTarget.value.replace(/^\//, "");
-        this.output_textTarget.value = `${prePath}/${currentPath}`;
+        const parentPath = match[1].replace(/\/$/, ""); // Remove trailing slash if present
+
+        // If we have a specific slug passed in, use it
+        // Otherwise only use the title-based slug if we have a title
+        const currentSlug = slug ||
+          (this.input_textTarget.value.trim() ? this.createSlug(this.input_textTarget.value.trim()) : "");
+
+        // Set the full path including parent path
+        this.output_textTarget.value = currentSlug
+          ? `${parentPath}/${currentSlug}`
+          : `${parentPath}/`;
+
+        console.debug("[Panda CMS] Set path with parent:", this.output_textTarget.value);
       }
     } catch (e) {
       console.error("[Panda CMS] Error setting pre-path:", e);
+      // Clear the output on error
+      this.output_textTarget.value = "";
     }
   }
 
   createSlug(input) {
-    return input
+    if (!input || typeof input !== 'string') return "";
+    const slug = input
       .toLowerCase()
+      .trim()
       .replace(/[^a-z0-9]+/g, "-")
       .replace(/^-+|-+$/g, "");
+    return slug;
   }
 
   trimStartEnd(str, ch) {
@@ -74,4 +102,9 @@ export default class extends Controller {
     while (end > start && str[end - 1] === ch) --end;
     return start > 0 || end < str.length ? str.substring(start, end) : str;
   }
+
+  // Add handler for manual path edits
+  handlePathInput() {
+    this.output_textTarget.dataset.userEdited = "true";
+  }
 }

data/app/javascript/panda/cms/controllers/theme_form_controller.js

@@ -0,0 +1,25 @@
+import { Controller } from "@hotwired/stimulus";
+
+// Connects to data-controller="theme-form"
+export default class extends Controller {
+  connect() {
+    // Ensure submit button is enabled on connect
+    this.enableSubmitButton();
+  }
+
+  updateTheme(event) {
+    const newTheme = event.target.value;
+    document.documentElement.dataset.theme = newTheme;
+  }
+
+  enableSubmitButton() {
+    // Find the submit button in the form and ensure it's enabled
+    const form = this.element;
+    if (form) {
+      const submitButton = form.querySelector('input[type="submit"], button[type="submit"]');
+      if (submitButton) {
+        submitButton.disabled = false;
+      }
+    }
+  }
+}

data/app/javascript/panda/cms/stimulus-loading.js

@@ -0,0 +1,39 @@
+// Stimulus loading utilities for Panda CMS
+// This provides the loading functionality that would normally come from stimulus-rails
+
+import { Application } from "@hotwired/stimulus"
+
+const application = Application.start()
+
+// Configure debug mode based on environment
+const railsEnv = document.body?.dataset?.environment || "production";
+application.debug = railsEnv === "development"
+window.Stimulus = application
+
+// Auto-registration functionality
+function eagerLoadControllersFrom(context) {
+  const definitions = []
+  for (const path of context.keys()) {
+    const module = context(path)
+    const controller = module.default
+    if (controller && path.match(/[_-]controller\.(js|ts)$/)) {
+      const name = path
+        .replace(/^.*\//, "")
+        .replace(/[_-]controller\.(js|ts)$/, "")
+        .replace(/_/g, "-")
+      definitions.push({ name, module: controller, filename: path })
+    }
+  }
+  return definitions
+}
+
+function lazyLoadControllersFrom(context) {
+  return eagerLoadControllersFrom(context)
+}
+
+// Export the functions that stimulus-loading typically provides
+export {
+  application,
+  eagerLoadControllersFrom,
+  lazyLoadControllersFrom
+}

data/app/javascript/panda_cms/stimulus-loading.js

@@ -0,0 +1,39 @@
+// Stimulus loading utilities for Panda CMS
+// This provides the loading functionality that would normally come from stimulus-rails
+
+import { Application } from "@hotwired/stimulus"
+
+const application = Application.start()
+
+// Configure debug mode based on environment
+const railsEnv = document.body?.dataset?.environment || "production";
+application.debug = railsEnv === "development"
+window.Stimulus = application
+
+// Auto-registration functionality
+function eagerLoadControllersFrom(context) {
+  const definitions = []
+  for (const path of context.keys()) {
+    const module = context(path)
+    const controller = module.default
+    if (controller && path.match(/[_-]controller\.(js|ts)$/)) {
+      const name = path
+        .replace(/^.*\//, "")
+        .replace(/[_-]controller\.(js|ts)$/, "")
+        .replace(/_/g, "-")
+      definitions.push({ name, module: controller, filename: path })
+    }
+  }
+  return definitions
+}
+
+function lazyLoadControllersFrom(context) {
+  return eagerLoadControllersFrom(context)
+}
+
+// Export the functions that stimulus-loading typically provides
+export {
+  application,
+  eagerLoadControllersFrom,
+  lazyLoadControllersFrom
+}

data/app/jobs/panda/cms/application_job.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Panda
   module CMS
     class ApplicationJob < ActiveJob::Base

data/app/jobs/panda/cms/record_visit_job.rb

@@ -1,29 +1,29 @@
+# frozen_string_literal: true
+
 module Panda
   module CMS
     class RecordVisitJob < ApplicationJob
       queue_as :default
 
       def perform(
-        url: nil,
+        path:,
+        user_id: nil,
+        redirect_id: nil,
+        page_id: nil,
         user_agent: nil,
-        referrer: nil,
         ip_address: nil,
-        page_id: nil,
-        current_user_id: nil,
-        params: [],
-        visited_at: nil,
-        redirect_id: nil
+        referer: nil,
+        params: []
       )
         Panda::CMS::Visit.create!(
-          url: url,
+          url: path,
+          user_id: user_id,
+          redirect_id: redirect_id,
+          page_id: page_id,
           user_agent: user_agent,
-          referrer: referrer,
           ip_address: ip_address,
-          page_id: page_id,
-          redirect_id: redirect_id,
-          user_id: current_user_id,
-          params: params,
-          visited_at: visited_at
+          referrer: referer, # TODO: Fix the naming of this column
+          params: params
         )
       end
     end

data/app/mailers/panda/cms/application_mailer.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Panda
   module CMS
     class ApplicationMailer < ActionMailer::Base

data/app/mailers/panda/cms/form_mailer.rb

@@ -1,10 +1,12 @@
+# frozen_string_literal: true
+
 module Panda
   module CMS
     class FormMailer < Panda::CMS::ApplicationMailer
       def notification_email(form:, form_submission:)
         # TODO: Handle fields named just "name", and "email" better
         @submission_data = form_submission.data
-        @sender_name = @submission_data["first_name"].to_s + " " + @submission_data["last_name"].to_s
+        @sender_name = "#{@submission_data["first_name"]} #{@submission_data["last_name"]}"
         @sender_email = @submission_data["email"].to_s
 
         mail(

data/app/models/panda/cms/application_record.rb

@@ -1,7 +1,10 @@
+# frozen_string_literal: true
+
 module Panda
   module CMS
     class ApplicationRecord < ActiveRecord::Base
       self.abstract_class = true
+      self.implicit_order_column = "created_at"
     end
   end
 end