zinkee 0.1.27 → 0.1.29

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "zinkee",
-  "version": "0.1.27",
+  "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

@@ -236,6 +236,110 @@ describe("records commands", () => {
     });
   });
 
+  it("accumulates repeated --select flags into a single field selection", async () => {
+    const configPath = createApiConfig();
+    const output = createStdoutBuffer();
+    const fetchMock = createFetchMock();
+    mockSchemaLookup(fetchMock);
+    fetchMock.mockResolvedValueOnce(
+      jsonResponse({
+        records: [{ recordId: "record-1", values: { name: "Ana" } }],
+        page: { offset: 0, limit: 25, total: 1, count: 1, hasMore: false },
+      }),
+    );
+    const program = buildProgram({ configPath, stdout: output.stdout });
+
+    await program.parseAsync([
+      "node",
+      "zinkee",
+      "--json",
+      "records",
+      "query",
+      "contacts",
+      "--select",
+      "campo-1",
+      "--select",
+      "campo-2",
+    ]);
+
+    const init = fetchMock.mock.calls[1]?.[1];
+
+    expect(JSON.parse(String(init?.body))).toEqual({
+      select: ["campo-1", "campo-2"],
+    });
+  });
+
+  it("mixes comma-separated and repeated --select flags preserving order", async () => {
+    const configPath = createApiConfig();
+    const output = createStdoutBuffer();
+    const fetchMock = createFetchMock();
+    mockSchemaLookup(fetchMock);
+    fetchMock.mockResolvedValueOnce(
+      jsonResponse({
+        records: [{ recordId: "record-1", values: { name: "Ana" } }],
+        page: { offset: 0, limit: 25, total: 1, count: 1, hasMore: false },
+      }),
+    );
+    const program = buildProgram({ configPath, stdout: output.stdout });
+
+    await program.parseAsync([
+      "node",
+      "zinkee",
+      "--json",
+      "records",
+      "query",
+      "contacts",
+      "--select",
+      "a,b",
+      "--select",
+      "c",
+    ]);
+
+    const init = fetchMock.mock.calls[1]?.[1];
+
+    expect(JSON.parse(String(init?.body))).toEqual({
+      select: ["a", "b", "c"],
+    });
+  });
+
+  it("deduplicates repeated fields across --select and --sort flags", async () => {
+    const configPath = createApiConfig();
+    const output = createStdoutBuffer();
+    const fetchMock = createFetchMock();
+    mockSchemaLookup(fetchMock);
+    fetchMock.mockResolvedValueOnce(
+      jsonResponse({
+        records: [{ recordId: "record-1", values: { name: "Ana" } }],
+        page: { offset: 0, limit: 25, total: 1, count: 1, hasMore: false },
+      }),
+    );
+    const program = buildProgram({ configPath, stdout: output.stdout });
+
+    await program.parseAsync([
+      "node",
+      "zinkee",
+      "--json",
+      "records",
+      "query",
+      "contacts",
+      "--select",
+      "id",
+      "--select",
+      "id",
+      "--sort",
+      "name:asc",
+      "--sort",
+      "name:asc",
+    ]);
+
+    const init = fetchMock.mock.calls[1]?.[1];
+
+    expect(JSON.parse(String(init?.body))).toEqual({
+      select: ["id"],
+      sort: [{ field: "name", direction: "ASC" }],
+    });
+  });
+
   it("normalizes UUID selectors for records get", async () => {
     const configPath = createApiConfig();
     const output = createStdoutBuffer();
@@ -308,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();
@@ -319,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",
@@ -368,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: {
@@ -380,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

@@ -233,12 +233,20 @@ function parseJsonKeyValueAssignment(input: string): JsonKeyValueAssignment {
   }
 }
 
+function dedupeStrings(values: string[]): string[] {
+  return [...new Set(values)];
+}
+
 function collectStringValues(value: string, previous: string[] = []): string[] {
   return [...previous, value];
 }
 
+function collectSelectValues(value: string, previous: string[] = []): string[] {
+  return dedupeStrings([...previous, ...parseCommaSeparatedValues(value, "--select")]);
+}
+
 function collectSortValues(value: string, previous: string[] = []): string[] {
-  return [...previous, ...parseCommaSeparatedValues(value, "--sort")];
+  return dedupeStrings([...previous, ...parseCommaSeparatedValues(value, "--sort")]);
 }
 
 function collectWhereExpressions(
@@ -413,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);
@@ -578,9 +552,7 @@ export function registerRecordsCommands(
     .command("query")
     .description("Query records for a schema")
     .argument("<schema>", "schema uuid or slug", parseUuidOrSlugSelector)
-    .option("--select <fields>", "comma-separated field selection", (value) => (
-      parseCommaSeparatedValues(value, "--select")
-    ))
+    .option("--select <fields>", "field selection (comma-separated, repeatable)", collectSelectValues, [])
     .option("--where <expression>", "where expression", collectWhereExpressions, [])
     .option(
       "--search <term>",
@@ -678,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,