conduyt 1.49.0 → 1.50.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);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "conduyt",
-  "version": "1.49.0",
+  "version": "1.50.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",