fix(client): address code review feedback

- Fix ETag parsing to properly detect weak ETags (check W/ before removing quotes)
- Fix 304 response to correctly parse weak ETags from ifNoneMatch
- Remove redundant array mapping in updateMany method
- Change createMany endpoint to /createMany to avoid conflict with /batch
- Remove deprecated tag from updateMany (it's a convenience method)
- Update README to clarify batch() is the recommended method

Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>

Author: Copilot

packages/client/README.md

@@ -111,12 +111,12 @@ Initializes the client by fetching the system discovery manifest from `/api/v1`.
 - `get<T>(object, id)`: Get single record by ID.
 - `query<T>(object, ast)`: Execute complex query using full AST.
 - `create<T>(object, data)`: Create record.
-- `createMany<T>(object, data[])`: Batch create records.
+- `batch(object, request)`: **Recommended** - Execute batch operations (create/update/upsert/delete) with full control.
+- `createMany<T>(object, data[])`: Batch create records (convenience method).
 - `update<T>(object, id, data)`: Update record.
-- `batch(object, request)`: Execute batch operations (create/update/upsert/delete).
-- `updateMany<T>(object, records[], options?)`: Simplified batch update.
+- `updateMany<T>(object, records[], options?)`: Batch update records (convenience method).
 - `delete(object, id)`: Delete record.
-- `deleteMany(object, ids[], options?)`: Batch delete records.
+- `deleteMany(object, ids[], options?)`: Batch delete records (convenience method).
 
 ### `client.views` (New!)
 - `create(request)`: Create a new saved view.

packages/client/src/index.ts

@@ -151,7 +151,10 @@ export class ObjectStackClient {
         if (res.status === 304) {
           return {
             notModified: true,
-            etag: cacheOptions?.ifNoneMatch ? { value: cacheOptions.ifNoneMatch, weak: false } : undefined
+            etag: cacheOptions?.ifNoneMatch ? { 
+              value: cacheOptions.ifNoneMatch.replace(/^W\/|"/g, ''),
+              weak: cacheOptions.ifNoneMatch.startsWith('W/')
+            } : undefined
           };
         }
 
@@ -161,7 +164,10 @@ export class ObjectStackClient {
 
         return {
           data,
-          etag: etag ? { value: etag.replace(/"/g, ''), weak: etag.startsWith('W/') } : undefined,
+          etag: etag ? { 
+            value: etag.replace(/^W\/|"/g, ''), 
+            weak: etag.startsWith('W/') 
+          } : undefined,
           lastModified: lastModified || undefined,
           notModified: false
         };
@@ -261,7 +267,7 @@ export class ObjectStackClient {
 
     createMany: async <T = any>(object: string, data: Partial<T>[]): Promise<T[]> => {
         const route = this.getRoute('data');
-        const res = await this.fetch(`${this.baseUrl}${route}/${object}/batch`, {
+        const res = await this.fetch(`${this.baseUrl}${route}/${object}/createMany`, {
             method: 'POST',
             body: JSON.stringify(data)
         });
@@ -292,7 +298,7 @@ export class ObjectStackClient {
 
     /**
      * Update multiple records (simplified batch update)
-     * @deprecated Use batch() method for full control
+     * Convenience method for batch updates without full BatchUpdateRequest
      */
     updateMany: async <T = any>(
       object: string, 
@@ -301,7 +307,7 @@ export class ObjectStackClient {
     ): Promise<BatchUpdateResponse> => {
         const route = this.getRoute('data');
         const request: UpdateManyRequest = {
-          records: records.map(r => ({ id: r.id, data: r.data })),
+          records,
           options
         };
         const res = await this.fetch(`${this.baseUrl}${route}/${object}/updateMany`, {