fullstackgtm 0.14.0 → 0.15.0

package/src/rules.ts

@@ -21,6 +21,27 @@ export function requiresHumanInput(value: unknown): boolean {
   return typeof value === "string" && value.startsWith(REQUIRES_HUMAN_PREFIX);
 }
 
+/**
+ * Attribution for duplicate groups: when the provider exposes record-source
+ * provenance (RecordProvenance), name the writer(s) that created the group —
+ * the fix for recurring dupes is upstream in the writer, not in the records.
+ */
+export function provenanceSummary(records: Array<{ provenance?: { source?: string; sourceLabel?: string; sourceId?: string } }>): string {
+  const counts = new Map<string, number>();
+  for (const record of records) {
+    const p = record.provenance;
+    if (!p) continue;
+    const label = p.sourceLabel ?? p.source ?? "unknown source";
+    const key = p.sourceId ? `${label} (${p.sourceId})` : label;
+    counts.set(key, (counts.get(key) ?? 0) + 1);
+  }
+  if (counts.size === 0) return "";
+  const parts = [...counts.entries()]
+    .sort((a, b) => b[1] - a[1])
+    .map(([key, count]) => (count > 1 ? `${key} ×${count}` : key));
+  return ` Created by: ${parts.join(", ")}.`;
+}
+
 export function auditFindingId(ruleId: string, objectId: string) {
   return `finding_${stableHash(`${ruleId}:${objectId}`)}`;
 }
@@ -343,7 +364,7 @@ export const duplicateAccountDomainRule: GtmAuditRule = {
         ruleId: "duplicate-account-domain",
         title: "Accounts share the same domain",
         severity: "warning",
-        summary: `${accounts.length} accounts share ${domain}: ${accounts.map((account) => account.name).join(", ")}.`,
+        summary: `${accounts.length} accounts share ${domain}: ${accounts.map((account) => account.name).join(", ")}.${provenanceSummary(accounts)}`,
         recommendation: "Review the group and merge duplicates so activity and deals roll up once.",
       });
       operations.push({
@@ -383,7 +404,7 @@ export const duplicateContactEmailRule: GtmAuditRule = {
         ruleId: "duplicate-contact-email",
         title: "Contacts share the same email",
         severity: "warning",
-        summary: `${contacts.length} contacts share ${email}.`,
+        summary: `${contacts.length} contacts share ${email}.${provenanceSummary(contacts)}`,
         recommendation: "Merge the duplicates so engagement history and routing stay coherent.",
       });
       operations.push({
@@ -433,7 +454,7 @@ export const duplicateOpenDealRule: GtmAuditRule = {
         severity: "warning",
         summary: `${deals.length} open deals named "${anchor.name}"${
           anchor.accountId ? " on the same account" : ""
-        }: ${deals.map((deal) => deal.id).join(", ")}.`,
+        }: ${deals.map((deal) => deal.id).join(", ")}.${provenanceSummary(deals)}`,
         recommendation:
           "Keep one deal, archive the copies, and fix the integration that is re-creating them.",
       });

package/src/types.ts

@@ -141,11 +141,26 @@ export type CanonicalUser = {
   active?: boolean;
 };
 
+/**
+ * Who created a record, per the provider's read-only record-source fields
+ * (HubSpot: hs_object_source / _label / _id). Populated on read; used to
+ * attribute duplicate findings to the writer that produced them.
+ */
+export type RecordProvenance = {
+  /** Provider source code, e.g. INTEGRATION, API, CRM_UI, IMPORT, FORM. */
+  source?: string;
+  /** Human label, e.g. an integration's name. */
+  sourceLabel?: string;
+  /** Provider-side id of the source (e.g. app id, import id). */
+  sourceId?: string;
+};
+
 export type CanonicalAccount = {
   id: string;
   provider?: CrmProvider;
   crmId?: string;
   identities?: ProviderIdentity[];
+  provenance?: RecordProvenance;
   name: string;
   domain?: string;
   industry?: string;
@@ -163,6 +178,7 @@ export type CanonicalContact = {
   provider?: CrmProvider;
   crmId?: string;
   identities?: ProviderIdentity[];
+  provenance?: RecordProvenance;
   accountId?: string;
   firstName?: string;
   lastName?: string;
@@ -181,6 +197,7 @@ export type CanonicalDeal = {
   provider?: CrmProvider;
   crmId?: string;
   identities?: ProviderIdentity[];
+  provenance?: RecordProvenance;
   accountId?: string;
   ownerId?: string;
   name: string;