completion-kit 0.5.11 → 0.5.12

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ed531ae29162bb91d2c463c3ff4eb20b5da469b9b7a21baddf5054a0ccc15041
-  data.tar.gz: b86aea95b2e1cf73abf6514093565dc07b12dc0f4fe5c5c5c8b80db3fbdfa83d
+  metadata.gz: 9c45ef9dab51631cbc2bfc606b6c98c038d967edeea8903efa634571d012e0b0
+  data.tar.gz: 549981db86a07f2d48cafb2b2413f6e1850997087ef553418c3ec9d7d07e0a0a
 SHA512:
-  metadata.gz: 04ae500020e71d52c41073c36a6741bc47b94a06ceec6548720d6022a60ce7422be8a354d627c39a7be8174af2ce65219041c5d99ad175157c7bf4b4eaf8f056
-  data.tar.gz: 261daeeb1555b3aecb8e2e18edb7f14ebdc37f974c2713ecbe12c43a281e8109edc45222be0f6039c325a0428e89c1c11a1a7104f0a36ace2b618fb2ef1cb7e8
+  metadata.gz: ddf258f7c59bc3a92113bcb95211f0314ea6cebe624d163478833f1515f0baafab0c630b9dd8983b7e2bde7ea5b641cb45b2249cd48c08a853422633e93409d9
+  data.tar.gz: 5001137b6be3a376f665cff46480e51b6994774f7e9146b5e464053bedcd5d989e2171b4ea8d792c9312b8c03b2e46e9d6b7515d8037e1c5510bde14caf11b1f

data/app/assets/stylesheets/completion_kit/application.css

@@ -1874,17 +1874,21 @@ tr:hover .ck-chip--publish {
   overflow: auto;
   max-height: 60vh;
   scrollbar-width: thin;
-  scrollbar-color: var(--ck-line-strong) transparent;
+  scrollbar-color: var(--ck-line-strong) var(--ck-bg);
+}
+.ck-csv-table-wrap::-webkit-scrollbar { width: 12px; height: 12px; background: var(--ck-bg); }
+.ck-csv-table-wrap::-webkit-scrollbar-track {
+  background: var(--ck-bg);
+  border-left: 1px solid var(--ck-line);
+  border-top: 1px solid var(--ck-line);
 }
-.ck-csv-table-wrap::-webkit-scrollbar { width: 10px; height: 10px; }
-.ck-csv-table-wrap::-webkit-scrollbar-track { background: transparent; }
 .ck-csv-table-wrap::-webkit-scrollbar-thumb {
   background: var(--ck-line-strong);
-  border-radius: 5px;
-  border: 2px solid var(--ck-bg-strong);
+  border-radius: 6px;
+  border: 3px solid var(--ck-bg);
 }
 .ck-csv-table-wrap::-webkit-scrollbar-thumb:hover { background: var(--ck-muted); }
-.ck-csv-table-wrap::-webkit-scrollbar-corner { background: transparent; }
+.ck-csv-table-wrap::-webkit-scrollbar-corner { background: var(--ck-bg); }
 
 .ck-modal__body .ck-csv-table-wrap {
   margin-top: 0;

data/app/models/completion_kit/mcp_session.rb

@@ -1,8 +1,22 @@
 module CompletionKit
   # MCP session marker — one row per active client session, kept in the
   # database so sessions survive Puma restarts, deploys, and Rails.cache
-  # eviction. Expired rows are opportunistically pruned on every new
-  # session start, so the table stays bounded by recent activity.
+  # eviction.
+  #
+  # Two things to know about why every query goes through `unscoped`:
+  #
+  # 1. CompletionKit::ApplicationRecord applies the host app's tenant_scope as
+  #    a default_scope. MCP sessions are per-CONNECTION, not per-tenant — the
+  #    table has no organization_id column. Letting the tenant_scope into a
+  #    session lookup turns "is this session live?" into either a SQL error
+  #    (no such column) or a false negative (`WHERE 1=0`), which surfaces to
+  #    the client as a spurious "Session not initialized" right after init.
+  # 2. `active?` slides expires_at forward when a session is more than halfway
+  #    through its TTL, so an MCP connection that keeps making calls stays
+  #    alive instead of expiring on the original 1-hour wall clock.
+  #
+  # Expired rows are opportunistically pruned on every new session start, so
+  # the table stays bounded by recent activity.
   class McpSession < ApplicationRecord
     self.table_name = "completion_kit_mcp_sessions"
 
@@ -10,20 +24,31 @@ module CompletionKit
 
     def self.start!
       prune_expired!
-      create!(session_id: SecureRandom.uuid, expires_at: SESSION_TTL.from_now).session_id
+      unscoped.create!(session_id: SecureRandom.uuid, expires_at: SESSION_TTL.from_now).session_id
     end
 
     def self.active?(session_id)
       return false if session_id.blank?
-      where(session_id: session_id).where("expires_at > ?", Time.current).exists?
+
+      row = unscoped.where(session_id: session_id).where("expires_at > ?", Time.current).first
+      return false unless row
+
+      slide_expiry(row)
+      true
     end
 
     def self.destroy_session(session_id)
-      where(session_id: session_id).delete_all
+      unscoped.where(session_id: session_id).delete_all
     end
 
     def self.prune_expired!
-      where("expires_at < ?", Time.current).delete_all
+      unscoped.where("expires_at < ?", Time.current).delete_all
+    end
+
+    def self.slide_expiry(row)
+      half_ttl_from_now = (SESSION_TTL / 2).from_now
+      return if row.expires_at > half_ttl_from_now
+      row.update_column(:expires_at, SESSION_TTL.from_now)
     end
   end
 end

data/lib/completion_kit/version.rb

@@ -1,3 +1,3 @@
 module CompletionKit
-  VERSION = "0.5.11"
+  VERSION = "0.5.12"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: completion-kit
 version: !ruby/object:Gem::Version
-  version: 0.5.11
+  version: 0.5.12
 platform: ruby
 authors:
 - Damien Bastin