conduyt 1.49.0 → 1.51.0

package/dist/index.js

@@ -468,6 +468,7 @@ deals
     if (opts.json !== undefined)
         Object.assign(body, jsonArg(opts.json, "--json"));
     assertFiniteValue(body.value);
+    assertDealSource(body);
     return client.post("/api/v1/deals", body);
 }));
 deals
@@ -477,6 +478,19 @@ deals
     assertUuid(id, "deal id");
     return client.get(`/api/v1/deals/${encodeURIComponent(id)}`);
 }));
+// Conduyt #129: every deal carries a source — it can be changed, never cleared. Validated on the FINAL merged body
+// (flags and --json alike) so the CLI fails locally instead of shipping a guaranteed 422. An omitted source on
+// create is fine: the server derives it (explicit → trigger → contact → "Unknown (created via api)").
+function assertDealSource(body) {
+    if (!("source" in body))
+        return;
+    const v = body.source;
+    if (v === null || typeof v !== "string" || v.trim() === "" || v.trim().toLowerCase() === "null") {
+        throw new Error("A deal's source can be changed, never cleared (every deal carries a source). Pass a non-blank source string, or omit it.");
+    }
+    if (v.length > 500)
+        throw new Error("source must be at most 500 characters.");
+}
 deals
     .command("update <id>")
     .description("Update a deal (PATCH — only the fields you pass change). --stage moves the deal (won/lost status auto-derives from the target stage's flags)")
@@ -487,7 +501,7 @@ deals
     .option("--pipeline <id>", "move to this pipeline UUID")
     .option("--contact <id>", "primary contact UUID")
     .option("--status <status>", "explicit status (open|won|lost); overrides the status auto-derived from --stage")
-    .option("--source <source>", "attribution source for THIS deal (pass 'null' to clear it)")
+    .option("--source <source>", "attribution source for THIS deal (can be changed, never cleared — every deal carries a source)")
     .option("--lost-reason <reason>", "reason for a lost/disqualified deal — some stages REQUIRE it on entry; when the stage has a configured dropdown the API 422 lists the valid lostReasonOptions, resend with one of those exact values")
     .option("--json <json>", "full JSON body, merged over the flags")
     .action(run(async (client, id, opts) => {
@@ -512,14 +526,14 @@ deals
     if (opts.status !== undefined)
         body.status = opts.status;
     if (opts.source !== undefined) {
-        // Mirror the API validator (blank rejected, 500-char cap) so the CLI
-        // fails locally rather than shipping a guaranteed 422.
-        if (opts.source.trim() === "") {
-            throw new Error("--source was provided but is blank. Omit it, or pass 'null' to clear the source.");
+        // Mirror the API validator (Conduyt #129: a deal's source can change, never be cleared; blank rejected,
+        // 500-char cap) so the CLI fails locally rather than shipping a guaranteed 422.
+        if (opts.source.trim() === "" || opts.source.trim().toLowerCase() === "null") {
+            throw new Error("A deal's source can be changed, never cleared (every deal carries a source). Pass the new source, or omit --source.");
         }
         if (opts.source.length > 500)
             throw new Error("--source must be at most 500 characters.");
-        body.source = opts.source === "null" ? null : opts.source;
+        body.source = opts.source;
     }
     if (opts.stage !== undefined) {
         if (opts.stage.trim() === "")
@@ -534,6 +548,7 @@ deals
     if (opts.json !== undefined)
         Object.assign(body, jsonArg(opts.json, "--json"));
     assertFiniteValue(body.value);
+    assertDealSource(body);
     if (Object.keys(body).length === 0)
         throw new Error("Nothing to update. Pass at least one field flag or --json.");
     return client.patch(`/api/v1/deals/${encodeURIComponent(id)}`, body);
@@ -1262,6 +1277,33 @@ appointments
         throw new Error("Nothing to update. Pass at least one field flag or --json.");
     return client.patch(`/api/v1/appointments/${encodeURIComponent(id)}`, body);
 }));
+appointments
+    .command("outcome <id>")
+    .description("Record the outcome (completed | no_show | cancelled | rescheduled): sets the status (rescheduled keeps it) and moves the contact's open deal to the stage mapped in Pipeline settings (POST /appointments/:id/outcome)")
+    .requiredOption("--outcome <outcome>", "completed, no_show, cancelled or rescheduled")
+    .option("--start <iso>", "rescheduled only: new start, ISO-8601 with tz offset")
+    .option("--end <iso>", "rescheduled only: new end, ISO-8601 with tz offset")
+    .action(run(async (client, id, opts) => {
+    assertUuid(id, "appointment id");
+    if (!["completed", "no_show", "cancelled", "rescheduled"].includes(opts.outcome)) {
+        throw new Error("--outcome must be completed, no_show, cancelled or rescheduled");
+    }
+    if (opts.outcome === "rescheduled" && (!opts.start || !opts.end))
+        throw new Error("--outcome rescheduled needs --start and --end");
+    const body = { outcome: opts.outcome };
+    if (opts.start)
+        body.startTime = opts.start;
+    if (opts.end)
+        body.endTime = opts.end;
+    return client.post(`/api/v1/appointments/${encodeURIComponent(id)}/outcome`, body);
+}));
+appointments
+    .command("outcome-undo <id>")
+    .description("Undo the last recorded outcome: status back to what it was, deal back to its previous stage when nobody moved it since (POST /appointments/:id/outcome/undo)")
+    .action(run(async (client, id) => {
+    assertUuid(id, "appointment id");
+    return client.post(`/api/v1/appointments/${encodeURIComponent(id)}/outcome/undo`, {});
+}));
 appointments
     .command("cancel <id>")
     .description("Cancel an appointment (soft-cancel; sets status=cancelled)")

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "conduyt",
-  "version": "1.49.0",
+  "version": "1.51.0",
   "description": "Command-line interface for Conduyt CRM — manage contacts, deals, pipelines, and run insight queries from your terminal.",
   "type": "module",
   "main": "dist/index.js",