fix: rename --json to --json-data on members update command (#8)

The --json option on members update shadowed the global --json output
flag, preventing users from getting JSON output on this command. Renamed
to --json-data to resolve the conflict.

Co-authored-by: Ben Sabic <bensabic@users.noreply.github.com>

Author: bensabic

src/commands/members.ts

@@ -377,7 +377,7 @@ membersCommand
     []
   )
   .option("--meta-data <key=value>", "Metadata field (repeatable)", collect, [])
-  .option("--json <json>", "JSON data (as JSON string)")
+  .option("--json-data <json>", "JSON data (as JSON string)")
   .option("--login-redirect <url>", "Login redirect URL")
   .action(async (id: string, options: MembersUpdateOptions) => {
     const spinner = yoctoSpinner({ text: "Updating member..." }).start();
@@ -399,8 +399,8 @@ membersCommand
       if (options.metaData?.length) {
         input.metaData = parseKeyValuePairs(options.metaData);
       }
-      if (options.json) {
-        input.json = parseJsonString(options.json);
+      if (options.jsonData) {
+        input.json = parseJsonString(options.jsonData);
       }
       if (options.loginRedirect) {
         input.loginRedirect = options.loginRedirect;

src/lib/types.ts

@@ -152,7 +152,7 @@ export interface MembersCreateOptions {
 export interface MembersUpdateOptions {
   customFields?: string[];
   email?: string;
-  json?: string;
+  jsonData?: string;
   loginRedirect?: string;
   metaData?: string[];
 }

tests/commands/members.test.ts

@@ -160,6 +160,51 @@ describe("members", () => {
     );
   });
 
+  it("update sends custom fields and meta data", async () => {
+    graphqlRequest.mockResolvedValueOnce({ updateMember: mockMember });
+
+    await runCommand(membersCommand, [
+      "update",
+      "mem_1",
+      "--meta-data",
+      "source=cli",
+    ]);
+
+    const call = graphqlRequest.mock.calls[0][0];
+    expect(call.variables.input.memberId).toBe("mem_1");
+    expect(call.variables.input.metaData).toEqual({ source: "cli" });
+  });
+
+  it("update sends --json-data as parsed JSON", async () => {
+    graphqlRequest.mockResolvedValueOnce({ updateMember: mockMember });
+
+    await runCommand(membersCommand, [
+      "update",
+      "mem_1",
+      "--json-data",
+      '{"key":"value"}',
+    ]);
+
+    const call = graphqlRequest.mock.calls[0][0];
+    expect(call.variables.input.memberId).toBe("mem_1");
+    expect(call.variables.input.json).toEqual({ key: "value" });
+  });
+
+  it("update accepts --json-data alongside global --json without conflict", async () => {
+    graphqlRequest.mockResolvedValueOnce({ updateMember: mockMember });
+
+    await runCommand(membersCommand, [
+      "update",
+      "mem_1",
+      "--json-data",
+      '{"key":"value"}',
+      "--json",
+    ]);
+
+    const call = graphqlRequest.mock.calls[0][0];
+    expect(call.variables.input.json).toEqual({ key: "value" });
+  });
+
   it("handles errors gracefully", async () => {
     graphqlRequest.mockRejectedValueOnce(new Error("Unauthorized"));