feat: Prepend "querychat_" to automatic id

Author: gadenbuie

pkg-py/src/querychat/_querychat.py

@@ -58,7 +58,7 @@ def __init__(
                 "Table name must begin with a letter and contain only letters, numbers, and underscores",
             )
 
-        self.id = id or table_name
+        self.id = id or f"querychat_{table_name}"
 
         self.tools = normalize_tools(tools, default=("update", "query"))
         self.greeting = greeting.read_text() if isinstance(greeting, Path) else greeting

pkg-py/tests/test_client_console.py

@@ -291,5 +291,5 @@ def test_existing_initialization_still_works(self, sample_df):
         )
 
         assert qc is not None
-        assert qc.id == "test_table"
+        assert qc.id == "querychat_test_table"
         assert qc.tools == ("update", "query")

pkg-py/tests/test_querychat.py

@@ -39,7 +39,7 @@ def test_querychat_init(sample_df):
 
     # Verify basic attributes are set
     assert qc is not None
-    assert qc.id == "test_table"
+    assert qc.id == "querychat_test_table"
 
     # Even without server initialization, we should be able to query the data source
     result = qc.data_source.execute_query(

pkg-r/R/QueryChat.R

@@ -178,7 +178,7 @@ QueryChat <- R6::R6Class(
 
       private$.data_source <- normalize_data_source(data_source, table_name)
 
-      self$id <- id %||% table_name
+      self$id <- id %||% sprintf("querychat_%s", table_name)
       self$tools <- tools
 
       if (!is.null(greeting) && file.exists(greeting)) {

pkg-r/tests/testthat/test-QueryChat.R

@@ -44,7 +44,7 @@ describe("QueryChat$new()", {
     withr::defer(qc$cleanup())
 
     expect_equal(qc$data_source$table_name, "my_data")
-    expect_equal(qc$id, "my_data")
+    expect_equal(qc$id, "querychat_my_data")
   })
 
   it("loads greeting from file if file exists", {