onni 0.1.0 → 0.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4adac16f882a9d8562e9441200c4e11f1a7c4dc49d0e577a2d0620ce11db1789
-  data.tar.gz: 95e396d99bf6a7f609fd4b6500d8db02ddf80103c07ce1d13b94e74672a71147
+  metadata.gz: 8297f8be9323c319bbafc426dfea3d9eaee1c96464e491827b7b3493d61bfb90
+  data.tar.gz: c643b87fac64fd84a44326be980a3c40e60549541480740d00458b449fe31e61
 SHA512:
-  metadata.gz: b234b572464676742136aa0b26a006408fdd0907189b3098a1aa794033309a9180b1bb763c7bc86d39c48bde47ff578b5793529ccd2d475906c4f6a2db8e5f03
-  data.tar.gz: '069c548695c4ebeb0a89868ef4351a09a06308199c819f83241a31f03c3e1d1823966b90e5dd4bee418b41489a72ef5f2b0b082a09dfb5562abbcf6154b4f223'
+  metadata.gz: c1985bf48570c1f4caf0fec3372669d1cf843d5e23a53ffb9334ca780ae042a32654a83d1387c0646cce50f52c0ce6f4481ae4a5ad3c31b547f30611ace04569
+  data.tar.gz: 4c9da41c69f15728231c13d059d19a1d40de7d2ed3bc7e9270a74d0795da4e3a93b9f298dc40cbea8766792141f225cad010812621b0f4331f3c67d906b23f96

data/CHANGELOG.md

@@ -1,5 +1,9 @@
 # CHANGELOG
 
+## 0.1.1 (2026-11-12)
+
+- Fix PostgreSQL hit stats upsert so `enabled_hits` / `disabled_hits` record again (ambiguous column with `excluded`)
+
 ## 0.1.0 (2026-11-12)
 
 - Add feature evaluation charts with 24-hour, 7-day, and 30-day history, enabled share, expandable counts, and replaceable views

data/README.md

@@ -202,7 +202,7 @@ Host tips:
 
 ### Other ActiveRecord dialects
 
-PostgreSQL uses the same `excluded.*` upsert form. If a host dialect rejects `excluded.*`, override `Adapters::ActiveRecord#record_stat` / `#record_stats_batch` or patch the `on_duplicate` fragment for that adapter.
+PostgreSQL uses the same `excluded.*` upsert form. The conflict `SET` keeps a bare assignment target and qualifies the existing-row hit column on the RHS (`enabled_hits = onni_feature_stat_buckets.enabled_hits + ...`) so PostgreSQL does not treat the bare name as ambiguous with `excluded`. If a host dialect rejects `excluded.*`, override `Adapters::ActiveRecord#record_stat` / `#record_stats_batch` or patch the `on_duplicate` fragment for that adapter.
 
 ## ActiveAdmin dashboard access
 

data/lib/onni/adapters/active_record.rb

@@ -108,7 +108,10 @@ module Onni
         period_start = Time.current.utc.beginning_of_hour
         now = Time.current
         column = enabled ? "enabled_hits" : "disabled_hits"
-        # Intent: column conflict target matches index_onni_feature_stats_unique (SQLite + PostgreSQL ON CONFLICT).
+        table = ::Onni::StatBucket.table_name
+        # Intent: unique_by matches index_onni_feature_stats_unique. Qualify the existing-row
+        # column on the RHS; bare names are ambiguous with excluded on PostgreSQL ON CONFLICT.
+        # SET targets stay bare (PostgreSQL rejects table-qualified assignment targets).
         ::Onni::StatBucket.upsert(
           {
             feature_key: key.to_s,
@@ -120,7 +123,9 @@ module Onni
             updated_at: now
           },
           unique_by: %i[feature_key period period_start],
-          on_duplicate: ::Arel.sql("#{column} = #{column} + #{weight}, updated_at = excluded.updated_at")
+          on_duplicate: ::Arel.sql(
+            "#{column} = #{table}.#{column} + #{weight}, updated_at = excluded.updated_at"
+          )
         )
       end
     end

data/lib/onni/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Onni
-  VERSION = "0.1.0"
+  VERSION = "0.1.1"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: onni
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.1
 platform: ruby
 authors:
 - Andrei Makarov