Add cached_og_description to pages, computed in background

Adds a `cached_og_description` column to pages, populated by
`CachePageOgDescriptionJob` on every page create/update. The job renders
the page's Liquid template in a logged-out CmsRenderingContext (same
pattern as `content_for_search`) and strips/truncates the result to 160
characters.

`open_graph_description_for_page` now reads straight from the column
instead of calling Rails.cache with a full Liquid render on every
cold-cache request. A migration backfill enqueues the job for all
existing pages.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: nbudin

.rubocop_todo.yml

@@ -75,9 +75,22 @@ Rails/RefuteMethods:
 # Offense count: 3
 # Configuration parameters: ForbiddenMethods, AllowedMethods.
 # ForbiddenMethods: decrement!, decrement_counter, increment!, increment_counter, insert, insert!, insert_all, insert_all!, toggle!, touch, touch_all, update_all, update_attribute, update_column, update_columns, update_counters, upsert, upsert_all
+Rails/HasAndBelongsToMany:
+  Exclude:
+    - 'app/models/page.rb'
+
+Rails/HasManyOrHasOneDependent:
+  Exclude:
+    - 'app/models/page.rb'
+
+Rails/HelperInstanceVariable:
+  Exclude:
+    - 'app/helpers/application_helper.rb'
+
 Rails/SkipsModelValidations:
   Exclude:
     - 'app/controllers/application_controller.rb'
+    - 'app/models/page.rb'
     - 'db/migrate/20251229184620_allow_null_redirect_uri_on_oauth_appliations.rb'
 
 # Offense count: 7

app/helpers/application_helper.rb

@@ -18,13 +18,7 @@ def page_title
   end
 
   def open_graph_description_for_page(page)
-    return page.meta_description if page.meta_description.present?
-
-    Rails
-      .cache
-      .fetch(["open_graph_description", page], expires_in: 1.day) do
-        strip_tags(cms_rendering_context.render_page_content(@page)).gsub(/\s+/, " ").strip.truncate(160)
-      end
+    page.meta_description.presence || page.cached_og_description
   end
 
   def application_entry_path

app/jobs/cache_page_og_description_job.rb

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+class CachePageOgDescriptionJob < ApplicationJob
+  def perform(page)
+    page.update_column(:cached_og_description, page.compute_og_description) # rubocop:disable Rails/SkipsModelValidations
+  end
+end

app/models/page.rb

@@ -11,6 +11,7 @@
 #  invariant                :boolean          default(FALSE), not null
 #  meta_description         :text
 #  name                     :text
+#  og_description           :text
 #  parent_type              :string
 #  skip_clickwrap_agreement :boolean          default(FALSE), not null
 #  slug                     :string
@@ -46,6 +47,7 @@ class Page < ApplicationRecord
 
   before_commit :set_performance_metadata, on: %i[create update]
   after_commit :touch_parent
+  after_commit :enqueue_cache_og_description_job, on: %i[create update]
 
   multisearchable(
     against: %i[name content_for_search],
@@ -84,6 +86,17 @@ def content_for_search
     ""
   end
 
+  def compute_og_description
+    convention = parent.is_a?(Convention) ? parent : nil
+    strip_tags(CmsContentFinder.new(convention).cms_rendering_context.render_page_content(self))
+      .gsub(/\s+/, " ")
+      .strip
+      .truncate(160)
+  rescue StandardError => e
+    Rails.logger.debug e
+    ""
+  end
+
   private
 
   def set_performance_metadata
@@ -95,4 +108,8 @@ def set_performance_metadata
   def touch_parent
     parent.touch if parent&.persisted?
   end
+
+  def enqueue_cache_og_description_job
+    CachePageOgDescriptionJob.perform_later(self)
+  end
 end

db/migrate/20260601000000_add_og_description_to_pages.rb

@@ -0,0 +1,8 @@
+# frozen_string_literal: true
+
+class AddOgDescriptionToPages < ActiveRecord::Migration[8.1]
+  def change
+    add_column :pages, :cached_og_description, :text
+    reversible { |dir| dir.up { Page.find_each { |page| CachePageOgDescriptionJob.perform_later(page) } } }
+  end
+end

db/structure.sql

@@ -2136,7 +2136,8 @@ CREATE TABLE public.pages (
     invariant boolean DEFAULT false NOT NULL,
     skip_clickwrap_agreement boolean DEFAULT false NOT NULL,
     hidden_from_search boolean DEFAULT false NOT NULL,
-    meta_description text
+    meta_description text,
+    cached_og_description text
 );
 
 
@@ -6160,6 +6161,7 @@ ALTER TABLE ONLY public.cms_files_pages
 SET search_path TO "$user", public;
 
 INSERT INTO "schema_migrations" (version) VALUES
+('20260601000000'),
 ('20260524175914'),
 ('20260519000000'),
 ('20260409003354'),

test/factories/pages.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # rubocop:disable Layout/LineLength, Lint/RedundantCopDisableDirective
 # == Schema Information
 #
@@ -10,6 +11,7 @@
 #  invariant                :boolean          default(FALSE), not null
 #  meta_description         :text
 #  name                     :text
+#  og_description           :text
 #  parent_type              :string
 #  skip_clickwrap_agreement :boolean          default(FALSE), not null
 #  slug                     :string

test/models/page_test.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # rubocop:disable Layout/LineLength, Lint/RedundantCopDisableDirective
 # == Schema Information
 #
@@ -10,6 +11,7 @@
 #  invariant                :boolean          default(FALSE), not null
 #  meta_description         :text
 #  name                     :text
+#  og_description           :text
 #  parent_type              :string
 #  skip_clickwrap_agreement :boolean          default(FALSE), not null
 #  slug                     :string