@indigoai-us/hq-cli 5.39.2 → 5.39.3

package/CHANGELOG.md

@@ -2,6 +2,19 @@
 
 ## [Unreleased]
 
+## [5.39.3]
+
+### Fixed
+
+- **`hq members invite` no longer errors out on an existing member (HQ-11).**
+  Inviting (or `--resend`-ing) an email that is already an active member returns
+  `409 MEMBERSHIP_ALREADY_EXISTS`; the command used to print a red error and exit
+  `1`, so a re-run just re-fired the server's (correctly captured) 409 and a
+  bulk-invite script aborted on an already-member. It now treats that 409 as a
+  terminal "already a member" outcome — a plain note, exit `0`, no retry. The
+  409 stays the server's correct signal; only the client's needless re-attempt
+  is removed. Sibling to the hq-console fix.
+
 ## [5.39.0]
 
 ### Added

package/dist/commands/members.js

@@ -1,5 +1,5 @@
 
-!function(){try{var e="undefined"!=typeof window?window:"undefined"!=typeof global?global:"undefined"!=typeof globalThis?globalThis:"undefined"!=typeof self?self:{},n=(new e.Error).stack;n&&(e._sentryDebugIds=e._sentryDebugIds||{},e._sentryDebugIds[n]="83c39bc9-e532-580d-bc3a-5f176eb5cba6")}catch(e){}}();
+!function(){try{var e="undefined"!=typeof window?window:"undefined"!=typeof global?global:"undefined"!=typeof globalThis?globalThis:"undefined"!=typeof self?self:{},n=(new e.Error).stack;n&&(e._sentryDebugIds=e._sentryDebugIds||{},e._sentryDebugIds[n]="b2b63537-b45f-59d6-8783-172b7edcacd6")}catch(e){}}();
 import chalk from "chalk";
 import { ensureCognitoToken } from "../utils/cognito-session.js";
 import { vaultApiFetch, getCompanyUid } from "../utils/vault-api.js";
@@ -366,6 +366,16 @@ export function registerMembersCommand(program) {
             }
         }
         catch (err) {
+            // The target is ALREADY an active member (409 MEMBERSHIP_ALREADY_EXISTS).
+            // Terminal, not a failure: the person is already in the company, so
+            // there's nothing to invite or retry. Print a plain note and exit 0 —
+            // re-attempting just re-fires the server's (correctly captured) 409,
+            // and a bulk-invite script shouldn't abort on an already-member. HQ-11.
+            if (err instanceof InviteHttpError &&
+                err.code === "MEMBERSHIP_ALREADY_EXISTS") {
+                console.log(chalk.yellow(`${target.toLowerCase()} is already a member of this company — nothing to do.`));
+                return;
+            }
             if (err instanceof InviteHttpError) {
                 console.error(chalk.red(formatInviteHttpError(err.status, err.message, err.code)));
                 process.exit(1);
@@ -451,4 +461,4 @@ export function registerMembersCommand(program) {
     });
 }
 //# sourceMappingURL=members.js.map
-//# debugId=83c39bc9-e532-580d-bc3a-5f176eb5cba6
+//# debugId=b2b63537-b45f-59d6-8783-172b7edcacd6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@indigoai-us/hq-cli",
-  "version": "5.39.2",
+  "version": "5.39.3",
   "description": "HQ by Indigo management CLI \u2014 modules and cloud sync",
   "main": "dist/index.js",
   "bin": {

package/src/commands/members.test.ts

@@ -225,6 +225,31 @@ describe("inviteMember", () => {
     ).rejects.toBeInstanceOf(InviteHttpError);
   });
 
+  // HQ-11: inviting an already-active member returns 409 with the machine code
+  // MEMBERSHIP_ALREADY_EXISTS. The code must ride on the InviteHttpError so the
+  // `invite` command can treat it as a terminal "already a member" (no retry).
+  it("surfaces the server `code` (MEMBERSHIP_ALREADY_EXISTS) on the InviteHttpError", async () => {
+    fetchSpy.mockResolvedValueOnce(
+      jsonResponse(409, {
+        error:
+          "Membership already exists for person email:alice@example.com in company cmp_acme",
+        code: "MEMBERSHIP_ALREADY_EXISTS",
+      }),
+    );
+
+    const err = await inviteMember({
+      target: "alice@example.com",
+      role: "member",
+      companyUid: "cmp_acme",
+      callerUid: "prs_admin",
+      token: "test-token",
+    }).catch((e: unknown) => e);
+
+    expect(err).toBeInstanceOf(InviteHttpError);
+    expect((err as InviteHttpError).status).toBe(409);
+    expect((err as InviteHttpError).code).toBe("MEMBERSHIP_ALREADY_EXISTS");
+  });
+
   it("resolves the token when it is nested on the membership row", async () => {
     fetchSpy.mockResolvedValueOnce(
       jsonResponse(200, {

package/src/commands/members.ts

@@ -650,6 +650,22 @@ export function registerMembersCommand(program: Command): void {
             );
           }
         } catch (err) {
+          // The target is ALREADY an active member (409 MEMBERSHIP_ALREADY_EXISTS).
+          // Terminal, not a failure: the person is already in the company, so
+          // there's nothing to invite or retry. Print a plain note and exit 0 —
+          // re-attempting just re-fires the server's (correctly captured) 409,
+          // and a bulk-invite script shouldn't abort on an already-member. HQ-11.
+          if (
+            err instanceof InviteHttpError &&
+            err.code === "MEMBERSHIP_ALREADY_EXISTS"
+          ) {
+            console.log(
+              chalk.yellow(
+                `${target.toLowerCase()} is already a member of this company — nothing to do.`,
+              ),
+            );
+            return;
+          }
           if (err instanceof InviteHttpError) {
             console.error(
               chalk.red(