From 73fccfb0a9da9af71e55e796281312e96b6bbbc3 Mon Sep 17 00:00:00 2001
From: rcmerci <rcmerci@gmail.com>
Date: Fri, 22 May 2026 00:08:45 +0800
Subject: [PATCH] enhance: update :logseq.property.agent/session-id

---
 deps/db/src/logseq/db/frontend/property.cljs       |  8 +++++---
 deps/db/test/logseq/db/frontend/property_test.cljs | 10 ++++++----
 src/test/frontend/worker/migrate_test.cljs         |  8 +++++---
 src/test/logseq/cli/command/agent_test.cljs        |  8 +++++---
 4 files changed, 21 insertions(+), 13 deletions(-)

diff --git a/deps/db/src/logseq/db/frontend/property.cljs b/deps/db/src/logseq/db/frontend/property.cljs
index 06837d88f4..04d562b93f 100644
--- a/deps/db/src/logseq/db/frontend/property.cljs
+++ b/deps/db/src/logseq/db/frontend/property.cljs
@@ -646,10 +646,12 @@
                                        :schema {:type :node
                                                 :public? false
                                                 :hide? true}}
-     :logseq.property.agent/session-id {:title "agent session id"
+     :logseq.property.agent/session-id {:title "Agent Session ID"
                                         :schema {:type :string
-                                                 :public? false
-                                                 :hide? true}}
+                                                 :public? true
+                                                 :hide? true}
+                                        :properties
+                                        {:logseq.property/description "Stores the AgentBridge session ID for a routed task."}}
      :logseq.property/used-template {:title "Used template"
                                      :schema {:type :node
                                               :public? false
diff --git a/deps/db/test/logseq/db/frontend/property_test.cljs b/deps/db/test/logseq/db/frontend/property_test.cljs
index 2f4f0404c6..a2509b7db4 100644
--- a/deps/db/test/logseq/db/frontend/property_test.cljs
+++ b/deps/db/test/logseq/db/frontend/property_test.cljs
@@ -91,13 +91,15 @@
 (deftest agent-session-id-built-in-property
   (let [property (get db-property/built-in-properties :logseq.property.agent/session-id)]
     (testing "schema"
-      (is (= "agent session id" (:title property)))
+      (is (= "Agent Session ID" (:title property)))
       (is (= :string (get-in property [:schema :type])))
       (is (not (contains? (:schema property) :db/cardinality)))
-      (is (= false (get-in property [:schema :public?])))
+      (is (= true (get-in property [:schema :public?])))
       (is (= true (get-in property [:schema :hide?]))))
 
-    (testing "internal built-in logseq property"
-      (is (not (contains? db-property/public-built-in-properties :logseq.property.agent/session-id)))
+    (testing "built-in logseq property"
+      (is (= "Stores the AgentBridge session ID for a routed task."
+             (get-in property [:properties :logseq.property/description])))
+      (is (contains? db-property/public-built-in-properties :logseq.property.agent/session-id))
       (is (db-property/logseq-property? :logseq.property.agent/session-id))
       (is (db-property/internal-property? :logseq.property.agent/session-id)))))
diff --git a/src/test/frontend/worker/migrate_test.cljs b/src/test/frontend/worker/migrate_test.cljs
index 6468d52b9a..751720a000 100644
--- a/src/test/frontend/worker/migrate_test.cljs
+++ b/src/test/frontend/worker/migrate_test.cljs
@@ -197,10 +197,12 @@
              (:kv/value (d/entity @conn :logseq.kv/schema-version))))
       (let [property (d/entity @conn :logseq.property.agent/session-id)]
         (is (some? property))
-        (is (= "agent session id" (:block/title property)))
+        (is (= "Agent Session ID" (:block/title property)))
         (is (= :string (:logseq.property/type property)))
-        (is (false? (:logseq.property/public? property)))
-        (is (true? (:logseq.property/hide? property))))
+        (is (true? (:logseq.property/public? property)))
+        (is (true? (:logseq.property/hide? property)))
+        (is (= "Stores the AgentBridge session ID for a routed task."
+               (:block/title (:logseq.property/description property)))))
       (is (some #(= {:properties [:logseq.property.agent/session-id]}
                     (:migrate-updates %))
                 (:upgrade-result-coll result))))))
diff --git a/src/test/logseq/cli/command/agent_test.cljs b/src/test/logseq/cli/command/agent_test.cljs
index 9e9b035a91..d4fdd3c36f 100644
--- a/src/test/logseq/cli/command/agent_test.cljs
+++ b/src/test/logseq/cli/command/agent_test.cljs
@@ -99,12 +99,14 @@
 
   (testing "agent session id is an internal built-in property"
     (let [property (get db-property/built-in-properties :logseq.property.agent/session-id)]
-      (is (= "agent session id" (:title property)))
+      (is (= "Agent Session ID" (:title property)))
       (is (= :string (get-in property [:schema :type])))
       (is (not (contains? (:schema property) :db/cardinality)))
-      (is (= false (get-in property [:schema :public?])))
+      (is (= true (get-in property [:schema :public?])))
       (is (= true (get-in property [:schema :hide?])))
-      (is (not (contains? db-property/public-built-in-properties :logseq.property.agent/session-id)))
+      (is (= "Stores the AgentBridge session ID for a routed task."
+             (get-in property [:properties :logseq.property/description])))
+      (is (contains? db-property/public-built-in-properties :logseq.property.agent/session-id))
       (is (db-property/logseq-property? :logseq.property.agent/session-id))
       (is (db-property/internal-property? :logseq.property.agent/session-id)))))