conduyt 1.77.0 → 1.78.0

package/README.md

@@ -35,7 +35,8 @@ conduyt contacts list --limit 25        # list contacts (supports --cursor, --qu
 conduyt contacts get <id>               # fetch one contact (carries `revision`, `hasOutboundSms` and `hasOutboundEmail`: a proven send only)
 conduyt contacts update <id> --first Ann --expected-revision <n>   # PATCH; a contact changed since revision n is refused: 409 contact_changed, exit 1, code and current revision printed
 conduyt contacts merge <primaryId> <secondaryId...> [--confirm [--yes]]   # DESTRUCTIVE: a dry run by default (prints the impact); --confirm shows it, asks y/N, then merges (the losers are deleted); --yes skips the question; moved counts 409 and 503 merge_contacts_busy / merge_budget_exhausted + Retry-After exit 1, never retried
-conduyt contacts duplicates <id>                                    # GET /contacts/{id}/duplicate-proposals; prints { proposals, heldIdentifiers } (open and not_sure possible duplicates; read-only)
+conduyt contacts duplicates <id> [--cursor <c>] [--limit <n>] [--held-cursor <c>] [--held-limit <n>]   # GET /contacts/{id}/duplicate-proposals; prints { proposals, heldIdentifiers, total, nextCursor, membershipRevision, heldIdentifiersTotal, heldIdentifiersNextCursor, heldIdentifiersMembershipRevision } (open and not_sure possible duplicates; keyset paged; read-only)
+conduyt imports duplicates <importId> [--page <n>] [--per-page <n>]  # GET /imports/{id}/duplicate-proposals; the proposals one import raised on the records it created, every status, oldest first; prints { rows, meta: { page, per_page, total } }
 conduyt contacts duplicates decide <id> <proposalId> --as not_sure|different|merge   # POST the decision; not_sure keeps both visible, different is final, merge only records a pair ALREADY merged (never merges); 409 already_decided / not_merged and 403 other_record_not_editable exit 1
 conduyt contacts collaborators add <contactId> <userId> [--expected-revision <n>]      # POST /contacts/{id}/collaborators; prints { added, revision }
 conduyt contacts collaborators remove <contactId> <userId> [--expected-revision <n>]   # DELETE with a body; prints { removed, revision }; 404 when not a collaborator

package/dist/index.js

@@ -259,14 +259,28 @@ contacts
 const PROPOSAL_DECISIONS = ["not_sure", "different", "merge"];
 const duplicates = contacts
     .command("duplicates <id>")
-    .description("List the possible duplicates of contact <id> (GET /contacts/{id}/duplicate-proposals, contacts:view, read-only): prints { proposals, heldIdentifiers }. Each proposal has id (the <proposalId> for `contacts duplicates decide`), status (open | not_sure), rule, raisedHere and `other` { id, name, createdAt, visible, editable }; a record you cannot view is \"a contact you cannot view\" with no id. heldIdentifiers are emails or phones this record was not given because another record owns them (value-free). Decide a pair with `conduyt contacts duplicates decide <id> <proposalId> --as not_sure|different|merge`")
-    .action(run(async (client, id) => {
+    .description("List the possible duplicates of contact <id> (GET /contacts/{id}/duplicate-proposals, contacts:view, read-only): prints { proposals, heldIdentifiers }. Each proposal has id (the <proposalId> for `contacts duplicates decide`), status (open | not_sure), rule, raisedHere and `other` { id, name, createdAt, visible, editable }; a record you cannot view is \"a contact you cannot view\" with no id. heldIdentifiers are emails or phones this record was not given because another record owns them (value-free). Decide a pair with `conduyt contacts duplicates decide <id> <proposalId> --as not_sure|different|merge`. Paging: both lists are keyset paged, newest first; --cursor / --limit page the proposals, --held-cursor / --held-limit the held identifiers. The printed object also carries total and heldIdentifiersTotal (every line of each list, all pages), nextCursor and heldIdentifiersNextCursor (pass back as --cursor / --held-cursor; null on the last page) and membershipRevision / heldIdentifiersMembershipRevision (opaque digests of each list: when a later page answers another revision than page one, discard it and read page one again). A cursor this list did not issue is a 400 invalid_cursor (exit 1)")
+    .option("--cursor <cursor>", "the nextCursor of the previous proposals page, passed back UNCHANGED (opaque); omit for the first page")
+    .option("--limit <n>", "proposals per page, integer 1-100 (default 50); above 100 is refused, not capped")
+    .option("--held-cursor <cursor>", "the heldIdentifiersNextCursor of the previous held identifiers page, passed back UNCHANGED (opaque); omit for the first page")
+    .option("--held-limit <n>", "held identifiers per page, integer 1-100 (default 20)")
+    .action(run(async (client, id, opts) => {
     assertUuid(id, "contact id");
-    return client.get(`/api/v1/contacts/${encodeURIComponent(id)}/duplicate-proposals`);
+    const qs = new URLSearchParams();
+    if (opts.cursor !== undefined)
+        qs.set("cursor", opaqueCursorFlag(opts.cursor, "--cursor"));
+    if (opts.limit !== undefined)
+        qs.set("limit", String(strictIntFlag(opts.limit, "--limit", 1, 100)));
+    if (opts.heldCursor !== undefined)
+        qs.set("held_cursor", opaqueCursorFlag(opts.heldCursor, "--held-cursor"));
+    if (opts.heldLimit !== undefined)
+        qs.set("held_limit", String(strictIntFlag(opts.heldLimit, "--held-limit", 1, 100)));
+    const q = qs.toString();
+    return client.get(`/api/v1/contacts/${encodeURIComponent(id)}/duplicate-proposals${q ? `?${q}` : ""}`);
 }));
 duplicates
     .command("decide <id> <proposalId>")
-    .description("Record a decision on one possible-duplicate pair (POST /contacts/{id}/duplicate-proposals/{proposalId}; contacts:edit on BOTH records). --as is required, exactly one of: not_sure (keep both records visible; the pair stays listed and can be decided again), different (the two are different people: never shown again, FINAL), merge (record that the pair is ALREADY merged; this command NEVER merges: run `conduyt contacts merge` first, else 409 not_merged). Prints { id, status, decidedAt }. Refusals exit 1 with the status, message and code and are never retried: 409 already_decided, 409 not_merged, 403 other_record_not_editable, 404, 400")
+    .description("Record a decision on one possible-duplicate pair (POST /contacts/{id}/duplicate-proposals/{proposalId}; contacts:edit on BOTH records). --as is required, exactly one of: not_sure (keep both records visible; the pair stays listed and can be decided again), different (the two are different people: never shown again, FINAL), merge (record that the pair is ALREADY merged; this command NEVER merges: run `conduyt contacts merge` first, else 409 not_merged). Prints { id, status, decidedAt, previousMembershipRevision, membershipRevision, total }: membershipRevision and total are the proposals list AFTER the decision (the same digest and count `contacts duplicates` answers), previousMembershipRevision the digest just BEFORE it (equal on an idempotent repeat); keep loaded pages only when previousMembershipRevision equals the revision you hold. Refusals exit 1 with the status, message and code and are never retried: 409 already_decided, 409 not_merged, 403 other_record_not_editable, 404, 400")
     .option("--as <decision>", "not_sure | different | merge (required)")
     .action(run(async (client, id, proposalId, opts) => {
     assertUuid(id, "contact id");
@@ -277,6 +291,42 @@ duplicates
         throw new Error(`--as must be exactly one of ${PROPOSAL_DECISIONS.join(", ")} (got "${opts.as}").`);
     return client.post(`/api/v1/contacts/${encodeURIComponent(id)}/duplicate-proposals/${encodeURIComponent(proposalId)}`, { decision: opts.as });
 }));
+/** a whole-number flag in [min, max]: digits only (no sign, decimal, exponent or hex), so "1e1" or "0x10" never pass as 10 or 16 */
+function strictIntFlag(raw, flag, min, max) {
+    const t = raw.trim();
+    if (!/^\d+$/.test(t))
+        throw new Error(`${flag} must be a whole number ${min}-${max} (got ${JSON.stringify(raw)}).`);
+    const n = Number(t);
+    if (n < min || n > max)
+        throw new Error(`${flag} must be a whole number ${min}-${max} (got ${JSON.stringify(raw)}).`);
+    return n;
+}
+/** an opaque cursor is sent exactly as given; a blank one is refused (the API would answer page one, silently restarting) */
+function opaqueCursorFlag(raw, flag) {
+    if (raw.trim() === "")
+        throw new Error(`${flag} was provided but is blank: pass the cursor from the previous page unchanged, or omit it for the first page.`);
+    return raw;
+}
+// Duplicate proposals look (board #95, app f2d638db6): the import results rows "Created, similar record exists".
+const imports = program.command("imports").description("Import jobs (contacts / opportunities / companies imports)");
+imports
+    .command("duplicates <importId>")
+    .description("List the duplicate proposals one import raised on the records it created (GET /imports/{id}/duplicate-proposals, contacts:view, read-only): one row per proposal, every status (open | not_sure | different | merged; a merged pair reads merged), oldest first. Prints { rows, meta: { page, per_page, total } } (total counts every row of the import). Each row: proposalId, status, rowNum (the import file's row number), createdAt, decidedAt, decidedByName, newRecord (the record the import created), similar (the existing record it may duplicate), matchedOn (kinds only, such as phone or phone + company, never a value), keptName; newRecord and similar are { id, name, visible, editable, live }. A record you cannot view has no id and the name \"A contact you cannot view\". Decide a row with `conduyt contacts duplicates decide <newRecord.id> <proposalId> --as ...` (contacts:edit on both records). An owner, admin or view-all key reads any import of the account, any other key only the imports its user ran (404 otherwise). The route is page based: --page / --per-page, not a cursor")
+    .option("--page <n>", "page number, integer 1-100000 (default 1)")
+    .option("--per-page <n>", "rows per page, integer 1-100 (default 50); above 100 is refused, not capped")
+    .option("--cursor <cursor>", "not supported (the import rows page with --page); passing it exits 1 and sends no request")
+    .action(run(async (client, importId, opts) => {
+    if (opts.cursor !== undefined)
+        throw new Error("imports duplicates does not support --cursor (the API pages the import rows with page and per_page); use --page <n> and --per-page <n>.");
+    assertUuid(importId, "import id");
+    const qs = new URLSearchParams();
+    if (opts.page !== undefined)
+        qs.set("page", String(strictIntFlag(opts.page, "--page", 1, 100000)));
+    if (opts.perPage !== undefined)
+        qs.set("per_page", String(strictIntFlag(opts.perPage, "--per-page", 1, 100)));
+    const q = qs.toString();
+    return client.get(`/api/v1/imports/${encodeURIComponent(importId)}/duplicate-proposals${q ? `?${q}` : ""}`);
+}));
 contacts
     .command("merge <primaryId> <secondaryIds...>")
     .description("DESTRUCTIVE merge of duplicate contacts (POST /contacts/merge; owner or admin): <primaryId> survives and receives the records of every <secondaryIds> contact, which are then DELETED (no undelete). By default a DRY RUN: nothing changes, it prints the impact { primary, secondaries, counts, totalRecords, guardCounts, filledFields, identifierOverflow, blockers }. --confirm repeats the dry run, shows the impact (the records that move, and separately the guard counts: chat messages, quick notes, identifiers, outbox rows, which are not in the records total), asks y/N and then merges, sending the values that dry run returned (expectedCounts, expectedGuardCounts, expectedIdentifiers). LOSER ORDER: losers are merged in the order given: for each field the survivor lacks, the first loser in this order with a value wins; reorder to change precedence; the dry run's filledFields shows which fields will be filled. WHAT IS PINNED: the server refuses (409) if the record counts, guard counts or identifier plan moved; other field edits made after this preview (a loser's name, company, source, address) are not detected; --yes skips the question (scripts; needs --confirm). A refusal exits 1 with the status, the API's message and its code and is NEVER retried: 409 \"Merge impact changed\" (preview again), 409 queued work, 400 / 404, and 503 merge_contacts_busy / merge_budget_exhausted with `[Retry-After: <N>s]` (nothing changed: wait that long, then run again).")

package/package.json

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

package/dist/action-config-typos.js

@@ -1,78 +0,0 @@
-/**
- * Local typo guard for automation action configs (card ownership per lane, Codex r1/r2). The server validator is the authority
- * and there the option names are not enforced yet, so a typoed `laneSoruce` on assign_to_user would silently assign the run's
- * deal instead of the lane card. Only the options below are known here.
- *
- * Traversal rule: ONLY the real action positions of the graph are visited, exactly as the graph is shaped: a v1 `actions`
- * array's elements `{type, config}`, and a v2 graph's `nodes[].action` `{type, config}`. For each action of a known type the
- * IMMEDIATE keys of `config` are checked; the config VALUES are opaque and never descended into (a webhook payload may hold any
- * object, including one shaped like an action), and nothing outside those positions (request metadata, other node fields) is
- * scanned. A key that is not a known option but is a case-insensitive match or ONE edit (substitution, insertion, deletion or
- * adjacent transposition) from one is refused; keys further away, and every other action type, pass through.
- * Source of the names: lib/automation-action-registry.ts in the app (assign_to_user, create_deal, close_other_open_deals).
- */
-const KNOWN_ACTION_OPTIONS = {
-    assign_to_user: ["runWindow", "mode", "userId", "userIds", "groupId", "target", "applyTo", "overwriteExisting", "laneSource", "lanePipelineId", "laneStageId"],
-    create_deal: ["pipelineId", "stageId", "dealName", "dealValue", "allowDuplicates", "status", "source", "assignedTo", "leaveUnassigned"],
-    close_other_open_deals: ["lostReason"],
-};
-/** Damerau-Levenshtein (optimal string alignment) distance, case-insensitive */
-function distance(a, b) {
-    const x = a.toLowerCase(), y = b.toLowerCase();
-    const d = Array.from({ length: x.length + 1 }, (_, i) => [i, ...Array(y.length).fill(0)]);
-    for (let j = 1; j <= y.length; j++)
-        d[0][j] = j;
-    for (let i = 1; i <= x.length; i++) {
-        for (let j = 1; j <= y.length; j++) {
-            d[i][j] = Math.min(d[i - 1][j] + 1, d[i][j - 1] + 1, d[i - 1][j - 1] + (x[i - 1] === y[j - 1] ? 0 : 1));
-            if (i > 1 && j > 1 && x[i - 1] === y[j - 2] && x[i - 2] === y[j - 1])
-                d[i][j] = Math.min(d[i][j], d[i - 2][j - 2] + 1);
-        }
-    }
-    return d[x.length][y.length];
-}
-const isObject = (v) => v !== null && typeof v === "object" && !Array.isArray(v);
-function checkAction(action) {
-    if (!isObject(action) || typeof action.type !== "string" || !isObject(action.config))
-        return null;
-    const known = KNOWN_ACTION_OPTIONS[action.type];
-    if (!known)
-        return null;
-    for (const key of Object.keys(action.config)) {
-        if (known.includes(key))
-            continue;
-        const near = known.find((k) => distance(key, k) <= 1);
-        if (near)
-            return { actionType: action.type, key, suggestion: near };
-    }
-    return null;
-}
-/** The first misspelled option key in an actions graph (v1 array of {type, config}, or v2 { nodes: [{ action: {type, config} }] }), or null */
-export function findActionConfigTypo(graph) {
-    if (Array.isArray(graph)) {
-        for (const action of graph) {
-            const t = checkAction(action);
-            if (t)
-                return t;
-        }
-        return null;
-    }
-    if (isObject(graph) && Array.isArray(graph.nodes)) {
-        for (const node of graph.nodes) {
-            const t = isObject(node) ? checkAction(node.action) : null;
-            if (t)
-                return t;
-        }
-    }
-    return null;
-}
-/** The refusal text; the details object is the final token */
-export function actionConfigTypoMessage(t) {
-    return `Unknown option "${t.key}" in ${t.actionType} config: did you mean "${t.suggestion}"? Nothing was sent. ${JSON.stringify(t)}`;
-}
-/** Throws (a plain Error) when the graph carries a misspelled option */
-export function assertNoActionConfigTypo(graph) {
-    const t = findActionConfigTypo(graph);
-    if (t)
-        throw new Error(actionConfigTypoMessage(t));
-}