zinkee 0.1.28 → 0.1.29

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "zinkee",
-  "version": "0.1.28",
+  "version": "0.1.29",
   "description": "CLI for Zinkee API v2",
   "type": "module",
   "bin": {

package/src/api/records.ts

@@ -38,7 +38,6 @@ export interface CreateRecordInput {
 }
 
 export interface UpdateRecordInput {
-  [key: string]: unknown;
   set?: Record<string, unknown>;
   unset?: string[];
 }

package/src/commands/records.test.ts

@@ -412,7 +412,7 @@ describe("records commands", () => {
     });
   });
 
-  it("updates records with the patch-style set and unset contract", async () => {
+  it("wraps raw-file fields into the set payload", async () => {
     const configPath = createApiConfig();
     const output = createStdoutBuffer();
     const fetchMock = createFetchMock();
@@ -423,15 +423,9 @@ describe("records commands", () => {
     writeFileSync(
       rawFile,
       JSON.stringify({
-        set: {
-          status: "inactive",
-          metadata: {
-            imported: true,
-          },
-        },
-        unset: ["legacyCode"],
-        audit: {
-          reason: "sync",
+        status: "inactive",
+        metadata: {
+          imported: true,
         },
       }),
       "utf8",
@@ -472,9 +466,6 @@ describe("records commands", () => {
     );
     expect(init?.method).toBe("PATCH");
     expect(JSON.parse(String(init?.body))).toEqual({
-      audit: {
-        reason: "sync",
-      },
       set: {
         status: "active",
         metadata: {
@@ -484,7 +475,81 @@ describe("records commands", () => {
           tier: "gold",
         },
       },
-      unset: ["legacyCode", "tempField"],
+      unset: ["tempField"],
+    });
+  });
+
+  it("wraps bare raw fields into the set payload", async () => {
+    const configPath = createApiConfig();
+    const output = createStdoutBuffer();
+    const fetchMock = createFetchMock();
+    mockSchemaLookup(fetchMock);
+    fetchMock.mockResolvedValueOnce(
+      jsonResponse({
+        id: "550e8400-e29b-41d4-a716-446655440001",
+        uno: "prueba mod",
+      }),
+    );
+    const program = buildProgram({ configPath, stdout: output.stdout });
+
+    await program.parseAsync([
+      "node",
+      "zinkee",
+      "--json",
+      "records",
+      "update",
+      "contacts",
+      "550E8400-E29B-41D4-A716-446655440001",
+      "--raw",
+      '{"uno":"prueba mod"}',
+    ]);
+
+    const init = fetchMock.mock.calls[1]?.[1];
+
+    expect(fetchMock.mock.calls[1]?.[0]).toBe(
+      "https://api.zinkee.test/api/v2/schemas/schema-1/records/550e8400-e29b-41d4-a716-446655440001",
+    );
+    expect(init?.method).toBe("PATCH");
+    expect(JSON.parse(String(init?.body))).toEqual({
+      set: {
+        uno: "prueba mod",
+      },
+    });
+  });
+
+  it("lets --set override raw fields with the same key", async () => {
+    const configPath = createApiConfig();
+    const output = createStdoutBuffer();
+    const fetchMock = createFetchMock();
+    mockSchemaLookup(fetchMock);
+    fetchMock.mockResolvedValueOnce(
+      jsonResponse({
+        id: "550e8400-e29b-41d4-a716-446655440001",
+      }),
+    );
+    const program = buildProgram({ configPath, stdout: output.stdout });
+
+    await program.parseAsync([
+      "node",
+      "zinkee",
+      "--json",
+      "records",
+      "update",
+      "contacts",
+      "550E8400-E29B-41D4-A716-446655440001",
+      "--raw",
+      '{"uno":"raw value"}',
+      "--set",
+      "uno=set value",
+    ]);
+
+    const init = fetchMock.mock.calls[1]?.[1];
+
+    expect(init?.method).toBe("PATCH");
+    expect(JSON.parse(String(init?.body))).toEqual({
+      set: {
+        uno: "set value",
+      },
     });
   });
 

package/src/commands/records.ts

@@ -421,50 +421,16 @@ function buildRecordCreateInput(
   return { values };
 }
 
-function readRawSetPayload(raw: JsonObject): Record<string, unknown> {
-  const setValue = raw.set;
-
-  if (setValue === undefined) {
-    return {};
-  }
-
-  if (!isJsonObject(setValue)) {
-    throw new CliError(
-      "invalid_cli_usage",
-      'Expected raw update payload field "set" to be a JSON object.',
-    );
-  }
-
-  return { ...setValue };
-}
-
-function readRawUnsetPayload(raw: JsonObject): string[] {
-  const unsetValue = raw.unset;
-
-  if (unsetValue === undefined) {
-    return [];
-  }
-
-  if (!Array.isArray(unsetValue) || !unsetValue.every((value) => typeof value === "string")) {
-    throw new CliError(
-      "invalid_cli_usage",
-      'Expected raw update payload field "unset" to be an array of strings.',
-    );
-  }
-
-  return [...unsetValue];
-}
-
 function buildRecordUpdateInput(
   options: RecordsUpdateCommandOptions,
 ): UpdateRecordInput {
   const raw = readRawObject(options.raw, options.rawFile);
-  const input: UpdateRecordInput = { ...raw };
+  const input: UpdateRecordInput = {};
   const setPayload = {
-    ...readRawSetPayload(raw),
+    ...raw,
     ...buildAssignmentObject(options.set, options.setJson),
   };
-  const unsetPayload = [...readRawUnsetPayload(raw), ...(options.unset ?? [])];
+  const unsetPayload = [...(options.unset ?? [])];
 
   for (const key of Object.keys(setPayload)) {
     const unsetIndex = unsetPayload.indexOf(key);
@@ -684,8 +650,8 @@ export function registerRecordsCommands(
     .option("--set <field=value>", "set a string field", collectKeyValueAssignments, [])
     .option("--set-json <field=json>", "set a structured field", collectJsonAssignments, [])
     .option("--unset <field>", "unset a field", collectStringValues, [])
-    .option("--raw <json>", "raw JSON object payload")
-    .option("--raw-file <path>", "path to a raw JSON object payload")
+    .option("--raw <json>", "raw JSON object of fields to set")
+    .option("--raw-file <path>", "path to a JSON object of fields to set")
     .action(async function action(
       this: Command,
       schema: NormalizedUuidOrSlug,