Refactor: sql on conflict; fix unittest; page names to vars

storage/storage.go

@@ -53,11 +53,15 @@ func (p ProviderSQL) GetLastChatByAgent(agent string) (*models.Chat, error) {
 	return &resp, err
 }
 
+// https://sqlite.org/lang_upsert.html
+// on conflict was added
 func (p ProviderSQL) UpsertChat(chat *models.Chat) (*models.Chat, error) {
 	// Prepare the SQL statement
 	query := `
-        INSERT OR REPLACE INTO chats (id, name, msgs, agent, created_at, updated_at)
+        INSERT INTO chats (id, name, msgs, agent, created_at, updated_at)
 	VALUES (:id, :name, :msgs, :agent, :created_at, :updated_at)
+	ON CONFLICT(id) DO UPDATE SET msgs=excluded.msgs,
+	updated_at=excluded.updated_at
         RETURNING *;`
 	stmt, err := p.db.PrepareNamed(query)
 	if err != nil {

storage/storage_test.go

@@ -95,6 +95,7 @@ func TestChatHistory(t *testing.T) {
 	    id INTEGER PRIMARY KEY AUTOINCREMENT,
 	    name TEXT NOT NULL,
 	    msgs TEXT NOT NULL, 
+	    agent TEXT NOT NULL,
 	    created_at TIMESTAMP NOT NULL DEFAULT CURRENT_TIMESTAMP,
 	    updated_at TIMESTAMP NOT NULL DEFAULT CURRENT_TIMESTAMP
 		);`)