[server][core] do not add redundant bookmarks

server/internal/command/user/bookmarks.go

@@ -57,7 +57,7 @@ func setupBookmarksListCommand() *cobra.Command {
 				os.Exit(1)
 			} else {
 				for _, b := range bookmarks {
-					fmt.Println(b)
+					fmt.Printf("%s %s %s\n", b.ID.String(), b.Name, b.PermissionsString())
 				}
 			}
 		},

server/internal/core/db/user.sql.go

@@ -14,7 +14,7 @@ import (
 const addBookmark = `-- name: AddBookmark :exec
 UPDATE users
 SET
-    bookmarks = bookmarks || to_jsonb(array[$1::uuid]),
+    bookmarks = CASE WHEN bookmarks ? ($1::uuid)::text THEN bookmarks ELSE bookmarks || to_jsonb(array[$1::uuid]) END,
     modified = NOW()
 WHERE username = $2::text
 `

server/sql/queries/user.sql

@@ -42,7 +42,7 @@ WHERE username = $2;
 -- name: AddBookmark :exec
 UPDATE users
 SET
-    bookmarks = bookmarks || to_jsonb(array[@id::uuid]),
+    bookmarks = CASE WHEN bookmarks ? (@id::uuid)::text THEN bookmarks ELSE bookmarks || to_jsonb(array[@id::uuid]) END,
     modified = NOW()
 WHERE username = @username::text;