content_signals 0.1.10 → 0.1.15

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0b02489786fd427dcf1ca68cc287bb92f505a51cd0e101e9d748f7f94132acd6
-  data.tar.gz: f3d36298735f85466408db3137fd99a9da2d591391691a5b3ccc4531f3d711a3
+  metadata.gz: e2c1d01c7f6f684b573250d04429a365fa3ab703cb8dddac8ddac0a7131580b8
+  data.tar.gz: bb343e43a050d8007817062d8de52f64f3ed54ee51d459b1a4988689e420083d
 SHA512:
-  metadata.gz: 161a9164367840f86c22d9c91301f5239d6d419000bfe5af0624e01e1def63b6c32044f5ca0771a82f6a3d3bed26577037cecb3abe04183d88309117d8a78366
-  data.tar.gz: 3c34e372467c5e3f4acf24970e13733dbcb0dd90b125b51e52f9822c785e8e4ad1010a193975a31077ae0e933d4f41ee30bf9535c66fac07fa80dcffe5143a79
+  metadata.gz: 4ca85cd8372d706ff93c3c9faa11ecf2920eff399fd7fa276f7f6cd09d7e81b569a8977cb1998189c43790c510500b8530f00a88a2d9860819909bc656eaa85e
+  data.tar.gz: 86dd62f5920ddc4565f4a619ba34ff9c57ef130e12f3a08af719fa11e6895bb28cd869d3278f338f3ef7004e231122b97cf9f1fe399f0d737d771cea3bff21c2

data/lib/content_signals/models/page_view.rb

@@ -6,33 +6,21 @@ module ContentSignals
 
     # Associations
     belongs_to :trackable, polymorphic: true, counter_cache: :page_views_count
-    belongs_to :tenant, optional: true if ContentSignals.configuration.multitenancy?
     belongs_to :user, optional: true
 
+    # tenant_id is a plain string (UUID), not a foreign key — declare explicitly
+    # Guard allows db:migrate to run on a fresh DB before the table exists
+    attribute :tenant_id, :string if connection.table_exists?(table_name) rescue nil
+
     # Validations
     validates :trackable_type, :trackable_id, :visitor_id, :viewed_at, presence: true
     validates :country_code, length: { maximum: 2 }, allow_nil: true
     validates :device_type, inclusion: { in: %w[desktop mobile tablet] }, allow_nil: true
     validates :app_platform, inclusion: { in: %w[hybrid native] }, allow_nil: true
 
-    # Default scope for tenant isolation (when multitenancy is enabled)
-    if ContentSignals.configuration.multitenancy?
-      default_scope -> { where(tenant_id: current_tenant_id) }
-
-      def self.current_tenant_id
-        return nil unless ContentSignals.configuration.multitenancy?
-
-        method_name = ContentSignals.configuration.current_tenant_method
-        return nil unless method_name
-
-        # Try to get tenant_id from Current, controller, or thread
-        if defined?(Current) && Current.respond_to?(method_name)
-          Current.send(method_name)
-        elsif Thread.current[method_name]
-          Thread.current[method_name]
-        end
-      end
-    end
+    # Tenant filtering — use explicitly instead of default_scope to avoid
+    # silent NULL filtering in background jobs, console, and migrations.
+    scope :for_tenant, ->(id) { where(tenant_id: id) }
 
     # Time period scopes
     scope :today, -> { where("viewed_at >= ?", Time.current.beginning_of_day) }

data/lib/content_signals/services/page_view_tracker.rb

@@ -154,8 +154,8 @@ module ContentSignals
       # Prefer X-Real-IP (set by reverse proxies like Caddy/Nginx) over
       # remote_ip which may still contain an internal proxy IP when
       # kamal-proxy overwrites X-Forwarded-For.
-      # real_ip = @request.headers['X-Real-IP']
-      # return real_ip if real_ip.present? && !private_ip?(real_ip)
+      real_ip = @request.headers['X-Real-IP']
+      return real_ip if real_ip.present? && !private_ip?(real_ip)
 
       @request.remote_ip
     end

data/lib/content_signals/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ContentSignals
-  VERSION = "0.1.10"
+  VERSION = "0.1.15"
 end

data/lib/generators/content_signals/templates/create_content_signals_page_views.rb.erb

@@ -1,8 +1,8 @@
 class CreateContentSignalsPageViews < ActiveRecord::Migration<%= migration_version %>
   def change
     create_table :content_signals_page_views do |t|
-      # Multi-tenancy support (optional)
-      t.integer :tenant_id, index: true
+      # Multi-tenancy support (optional, string to support both UUID and integer tenant IDs)
+      t.string :tenant_id, index: true
 
       # Polymorphic trackable (Page, Profile, Event, etc.)
       t.string :trackable_type, null: false

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: content_signals
 version: !ruby/object:Gem::Version
-  version: 0.1.10
+  version: 0.1.15
 platform: ruby
 authors:
 - Ionuț Munteanu Alexandru