@dench.com/cli 3.10.1-staging.423 → 3.10.2-staging.428

package/lib/enrichment-gateway.ts

@@ -11,6 +11,7 @@ export type {
 } from "./enrichment-shapes";
 
 import { extractDomain } from "./crm-enrichment";
+import { normalizeGatewayBaseUrl } from "./gateway-url";
 
 export type FetchLike = (
   input: string | URL,
@@ -274,7 +275,7 @@ export function resolveEnrichmentGatewayBaseUrl(
   env: EnrichmentGatewayEnv = process.env,
 ): string {
   const explicit = env.DENCH_GATEWAY_URL?.trim() || env.GATEWAY_URL?.trim();
-  if (explicit) return explicit.replace(/\/+$/, "").replace(/\/v1$/, "");
+  if (explicit) return normalizeGatewayBaseUrl(explicit);
 
   const host = env.DENCH_HOST?.trim();
   if (host && /localhost|127\.0\.0\.1/.test(host)) {
@@ -1690,10 +1691,9 @@ async function callGatewayJson(
 }
 
 function resolveBaseUrl(options: EnrichmentGatewayOptions): string {
-  return (
-    options.baseUrl?.trim().replace(/\/+$/, "").replace(/\/v1$/, "") ??
-    resolveEnrichmentGatewayBaseUrl(options.env)
-  );
+  return options.baseUrl != null
+    ? normalizeGatewayBaseUrl(options.baseUrl.trim())
+    : resolveEnrichmentGatewayBaseUrl(options.env);
 }
 
 async function buildGatewayError(

package/lib/gateway-url.ts

@@ -0,0 +1,12 @@
+/** Remove a URL's trailing slashes in one pass, including long malformed input. */
+export function stripTrailingSlashes(value: string): string {
+  let end = value.length;
+  while (end > 0 && value.charCodeAt(end - 1) === 47) end--;
+  return value.slice(0, end);
+}
+
+/** Gateway callers append their own /v1 path; accept either base URL form. */
+export function normalizeGatewayBaseUrl(value: string): string {
+  const base = stripTrailingSlashes(value);
+  return base.endsWith("/v1") ? base.slice(0, -3) : base;
+}

package/lib/meetings-import.ts

@@ -229,6 +229,44 @@ export function normalizeTranscript(value: unknown): ImportedSegment[] | null {
   return null;
 }
 
+const LINE_BREAK = /[\r\n\u2028\u2029]/;
+
+/** Inspect only the final owner delimiter instead of backtracking over the body. */
+function actionItemOwner(
+  text: string,
+): { text: string; ownerName: string } | null {
+  if (text.endsWith(")")) {
+    const open = text.lastIndexOf("(");
+    const owner = text.slice(open + 1, -1);
+    const body = text.slice(0, open).trimEnd();
+    if (
+      open >= 0 &&
+      owner.length > 0 &&
+      owner.length <= 60 &&
+      !owner.includes(")") &&
+      !LINE_BREAK.test(body)
+    ) {
+      return { text: body.trim(), ownerName: owner.trim() };
+    }
+  }
+  for (let index = text.length - 1; index >= 0; index--) {
+    if (!"-–—".includes(text[index])) continue;
+    // Only the last dash can separate an owner: owners cannot contain dashes.
+    if (
+      !/\s/.test(text[index - 1] ?? "") ||
+      !/\s/.test(text[index + 1] ?? "")
+    ) {
+      return null;
+    }
+    const owner = text.slice(index + 1).trimStart();
+    const body = text.slice(0, index).trimEnd();
+    return owner.length > 0 && owner.length <= 60 && !LINE_BREAK.test(body)
+      ? { text: body.trim(), ownerName: owner.trim() }
+      : null;
+  }
+  return null;
+}
+
 /**
  * `- [x] Send the deck (Hank)` / `Send the deck — Hank` / `{text, owner,
  * done}`. Markdown checkboxes carry completion; a trailing parenthesised
@@ -239,20 +277,18 @@ export function normalizeActionItem(value: unknown): ImportedActionItem | null {
     let text = value.trim().replace(/^[-*•]\s+/, "");
     if (!text) return null;
     let done: boolean | undefined;
-    const box = text.match(/^\[([ xX])\]\s*(.*)$/);
-    if (box) {
-      done = box[1] !== " ";
-      text = box[2].trim();
+    if (text[0] === "[" && " xX".includes(text[1] ?? "") && text[2] === "]") {
+      const body = text.slice(3).trimStart();
+      if (!LINE_BREAK.test(body)) {
+        done = text[1] !== " ";
+        text = body.trim();
+      }
     }
     let ownerName: string | null = null;
-    const paren = text.match(/^(.*?)\s*\(([^()]{1,60})\)\s*$/);
-    const dash = text.match(/^(.*?)\s+[-–—]\s+([^-–—]{1,60})$/);
-    if (paren) {
-      text = paren[1].trim();
-      ownerName = paren[2].trim();
-    } else if (dash) {
-      text = dash[1].trim();
-      ownerName = dash[2].trim();
+    const owner = actionItemOwner(text);
+    if (owner) {
+      text = owner.text;
+      ownerName = owner.ownerName;
     }
     if (!text) return null;
     return { text, ownerName, ...(done !== undefined ? { done } : {}) };
@@ -303,15 +339,28 @@ export function normalizeAttendee(value: unknown): ImportedAttendee | null {
   if (typeof value === "string") {
     const raw = value.trim();
     if (!raw) return null;
-    const isHost = /\((?:host|organi[sz]er)\)\s*$/i.test(raw);
-    const cleaned = raw.replace(/\s*\((?:host|organi[sz]er)\)\s*$/i, "");
-    const angle = cleaned.match(/^(.*?)\s*<([^<>\s]+@[^<>\s]+)>$/);
-    if (angle) {
-      return {
-        name: angle[1].trim() || null,
-        email: angle[2].trim(),
-        ...(isHost ? { isHost } : {}),
-      };
+    const markerStart = raw.lastIndexOf("(");
+    const marker = raw.slice(markerStart).toLowerCase();
+    const isHost = ["(host)", "(organizer)", "(organiser)"].includes(marker);
+    const cleaned = isHost ? raw.slice(0, markerStart).trimEnd() : raw;
+    if (cleaned.endsWith(">")) {
+      const open = cleaned.lastIndexOf("<");
+      const email = cleaned.slice(open + 1, -1);
+      const at = email.indexOf("@", 1);
+      const name = cleaned.slice(0, open).trimEnd();
+      if (
+        open >= 0 &&
+        at >= 1 &&
+        at < email.length - 1 &&
+        !/[<>\s]/.test(email) &&
+        !LINE_BREAK.test(name)
+      ) {
+        return {
+          name: name.trim() || null,
+          email,
+          ...(isHost ? { isHost } : {}),
+        };
+      }
     }
     if (/^[^\s@]+@[^\s@]+$/.test(cleaned)) {
       return { name: null, email: cleaned, ...(isHost ? { isHost } : {}) };
@@ -342,9 +391,27 @@ export function normalizeAttendees(value: unknown): ImportedAttendee[] | null {
         // fall through
       }
     }
-    // Commas inside `Name <email>` are unusual; semicolons and newlines
-    // are the safer separators, but all three are accepted.
-    return normalizeAttendees(trimmed.split(/\s*[;\n]\s*|,\s*(?![^<]*>)/));
+    // A comma splits unless the next angle bracket is a closing one. Track
+    // that bracket while scanning backwards instead of rescanning the tail
+    // for every comma. Semicolons and newlines always separate attendees.
+    const parts: string[] = [];
+    let end = trimmed.length;
+    let nextAngleIsClose = false;
+    for (let index = trimmed.length - 1; index >= 0; index--) {
+      const char = trimmed[index];
+      if (char === ">") nextAngleIsClose = true;
+      else if (char === "<") nextAngleIsClose = false;
+      else if (
+        char === ";" ||
+        char === "\n" ||
+        (char === "," && !nextAngleIsClose)
+      ) {
+        parts.push(trimmed.slice(index + 1, end));
+        end = index;
+      }
+    }
+    parts.push(trimmed.slice(0, end));
+    return normalizeAttendees(parts.reverse());
   }
   if (Array.isArray(value)) {
     return value

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dench.com/cli",
-  "version": "3.10.1-staging.423",
+  "version": "3.10.2-staging.428",
   "description": "Dench agent workspace CLI.",
   "type": "module",
   "bin": {

package/search.ts

@@ -36,6 +36,7 @@ import {
   personFullName,
 } from "./lib/exa-people";
 import { unsupportedParamsForCategory } from "./lib/exa-search-constraints";
+import { normalizeGatewayBaseUrl } from "./lib/gateway-url";
 
 type SearchCliContext = {
   args: string[];
@@ -119,10 +120,6 @@ function assertFlagsSupportedForCategory(
   );
 }
 
-function normalizeGatewayBaseUrl(value: string): string {
-  return value.replace(/\/+$/, "").replace(/\/v1$/, "");
-}
-
 function resolveGatewayBaseUrl(
   ctx?: Pick<SearchCliContext, "gatewayBaseUrl">,
 ): string {

package/tools.ts

@@ -57,6 +57,10 @@ import {
   type ConnectionSource,
   connectionSource,
 } from "./lib/connection-source";
+import {
+  normalizeGatewayBaseUrl,
+  stripTrailingSlashes,
+} from "./lib/gateway-url";
 import {
   isNativeToolRecord,
   nativeToolkit,
@@ -673,9 +677,8 @@ async function runSlackConnectSubcommand(
   toolkit: string,
 ): Promise<void> {
   const slug = ctx.workspaceSlug?.trim() || (await resolveWorkspaceSlug(ctx));
-  const origin = (ctx.appOrigin?.trim() || resolveAppPublicOrigin()).replace(
-    /\/+$/,
-    "",
+  const origin = stripTrailingSlashes(
+    ctx.appOrigin?.trim() || resolveAppPublicOrigin(),
   );
   const redirectUrl = `${origin}/api/slack/install?slug=${
     slug ? encodeURIComponent(slug) : "<workspace-slug>"
@@ -897,10 +900,6 @@ class GatewayHttpError extends ToolCliError {
   }
 }
 
-function normalizeGatewayBaseUrl(value: string): string {
-  return value.replace(/\/+$/, "").replace(/\/v1$/, "");
-}
-
 function resolveGatewayBaseUrl(
   ctx?: Pick<ToolCliContext, "gatewayBaseUrl">,
 ): string {

package/web.ts

@@ -7,6 +7,7 @@
  */
 
 import { assertNoUnknownFlags, getFlag, hasFlag } from "./lib/cli-args";
+import { normalizeGatewayBaseUrl } from "./lib/gateway-url";
 
 type WebCliContext = {
   args: string[];
@@ -32,10 +33,6 @@ function parsePositiveInt(
   return Math.floor(parsed);
 }
 
-function normalizeGatewayBaseUrl(value: string): string {
-  return value.replace(/\/+$/, "").replace(/\/v1$/, "");
-}
-
 function resolveGatewayBaseUrl(
   ctx?: Pick<WebCliContext, "gatewayBaseUrl">,
 ): string {