@zooid/server 0.0.9 → 0.0.10

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zooid/server",
-  "version": "0.0.9",
+  "version": "0.0.10",
   "type": "module",
   "license": "MIT",
   "author": "Ori Ben",
@@ -21,8 +21,8 @@
     "ulidx": "^2.4.1",
     "yaml": "^2.8.2",
     "zod": "^4.3.6",
-    "@zooid/types": "0.0.9",
-    "@zooid/web": "0.0.9"
+    "@zooid/types": "0.0.10",
+    "@zooid/web": "0.0.10"
   },
   "devDependencies": {
     "@cloudflare/vitest-pool-workers": "^0.12.13",

package/src/db/queries.ts

@@ -422,35 +422,55 @@ export async function upsertServerMeta(
     email?: string | null;
   },
 ): Promise<ServerIdentity> {
-  const tags = meta.tags ? JSON.stringify(meta.tags) : null;
+  const tags = meta.tags !== undefined ? JSON.stringify(meta.tags) : undefined;
+
+  // Build dynamic SET clause — only update fields that were provided.
+  // This distinguishes undefined (not provided → keep existing) from null (clear it).
+  const setClauses: string[] = ["updated_at = datetime('now')"];
+  const setBinds: (string | null)[] = [];
+
+  if (meta.name !== undefined) {
+    setClauses.push('name = ?');
+    setBinds.push(meta.name);
+  }
+  if (meta.description !== undefined) {
+    setClauses.push('description = ?');
+    setBinds.push(meta.description);
+  }
+  if (tags !== undefined) {
+    setClauses.push('tags = ?');
+    setBinds.push(tags);
+  }
+  if (meta.owner !== undefined) {
+    setClauses.push('owner = ?');
+    setBinds.push(meta.owner);
+  }
+  if (meta.company !== undefined) {
+    setClauses.push('company = ?');
+    setBinds.push(meta.company);
+  }
+  if (meta.email !== undefined) {
+    setClauses.push('email = ?');
+    setBinds.push(meta.email);
+  }
 
   await db
     .prepare(
       `INSERT INTO server_meta (id, name, description, tags, owner, company, email, updated_at)
        VALUES (1, ?, ?, ?, ?, ?, ?, datetime('now'))
        ON CONFLICT(id) DO UPDATE SET
-         name = COALESCE(?, server_meta.name),
-         description = ?,
-         tags = ?,
-         owner = ?,
-         company = ?,
-         email = ?,
-         updated_at = datetime('now')`,
+         ${setClauses.join(',\n         ')}`,
     )
     .bind(
+      // INSERT values (use defaults for unspecified fields)
       meta.name ?? 'Zooid',
       meta.description ?? null,
-      tags,
-      meta.owner ?? null,
-      meta.company ?? null,
-      meta.email ?? null,
-      // ON CONFLICT values
-      meta.name ?? null,
-      meta.description ?? null,
-      tags,
+      tags ?? null,
       meta.owner ?? null,
       meta.company ?? null,
       meta.email ?? null,
+      // ON CONFLICT SET values (only the provided fields)
+      ...setBinds,
     )
     .run();
 

package/src/routes/server-meta.test.ts

@@ -126,6 +126,57 @@ describe('Server meta routes', () => {
       expect(body.owner).toBe('bob');
     });
 
+    it('preserves fields not included in partial update', async () => {
+      await authRequest('/api/v1/server', {
+        method: 'PUT',
+        body: JSON.stringify({
+          name: 'My Server',
+          description: 'Original desc',
+          tags: ['ai'],
+          owner: 'alice',
+          company: 'Acme',
+          email: 'alice@acme.com',
+        }),
+      });
+
+      // Update only description — all other fields should be preserved
+      const res = await authRequest('/api/v1/server', {
+        method: 'PUT',
+        body: JSON.stringify({ description: 'Updated desc' }),
+      });
+
+      expect(res.status).toBe(200);
+      const body = (await res.json()) as Record<string, unknown>;
+      expect(body.description).toBe('Updated desc');
+      expect(body.name).toBe('My Server');
+      expect(body.tags).toEqual(['ai']);
+      expect(body.owner).toBe('alice');
+      expect(body.company).toBe('Acme');
+      expect(body.email).toBe('alice@acme.com');
+    });
+
+    it('allows explicitly clearing a field with null', async () => {
+      await authRequest('/api/v1/server', {
+        method: 'PUT',
+        body: JSON.stringify({
+          name: 'My Server',
+          description: 'Some desc',
+          owner: 'alice',
+        }),
+      });
+
+      const res = await authRequest('/api/v1/server', {
+        method: 'PUT',
+        body: JSON.stringify({ description: null }),
+      });
+
+      expect(res.status).toBe(200);
+      const body = (await res.json()) as Record<string, unknown>;
+      expect(body.description).toBeNull();
+      expect(body.name).toBe('My Server');
+      expect(body.owner).toBe('alice');
+    });
+
     it('rejects without auth', async () => {
       const res = await app.request(
         '/api/v1/server',