@iflow-mcp/attio-mcp 1.3.6

package/dist/services/create/attio-create.service.d.ts

@@ -0,0 +1,127 @@
+/**
+ * AttioCreateService - Refactored with Strategy Pattern (Issue #552)
+ *
+ * REFACTORED: Implements the Strategy Pattern using resource-specific creators to handle
+ * different Attio resource types. Each creator encapsulates the logic for
+ * creating one type of resource, promoting Single Responsibility Principle.
+ *
+ * This refactoring addresses SRP violations by moving resource-specific logic
+ * from large methods (~139-168 lines) into focused creator classes (~50-80 lines each).
+ *
+ * Key improvements:
+ * - Single Responsibility: Each creator handles one resource type
+ * - Maintainability: Easy to add new resource types
+ * - Testability: Test each creator independently
+ * - Code Reuse: Shared utilities in BaseCreator
+ *
+ * See src/services/create/creators/README.md for full documentation.
+ */
+import type { CreateService } from './types.js';
+import type { AttioRecord } from '../../types/attio.js';
+import type { ResourceCreator } from './creators/types.js';
+/**
+ * Refactored implementation using Strategy Pattern
+ *
+ * Uses resource-specific creators to handle different resource types,
+ * promoting separation of concerns and Single Responsibility Principle.
+ *
+ * @example
+ * ```typescript
+ * const service = new AttioCreateService();
+ * const company = await service.createCompany({
+ *   name: "Acme Corp",
+ *   domain: "acme.com"
+ * });
+ * ```
+ */
+export declare class AttioCreateService implements CreateService {
+    private readonly creators;
+    private readonly context;
+    private taskModule;
+    private converterModule;
+    private noteModule;
+    static readonly SUPPORTED_RESOURCE_TYPES: {
+        readonly COMPANIES: "companies";
+        readonly PEOPLE: "people";
+        readonly TASKS: "tasks";
+        readonly NOTES: "notes";
+    };
+    constructor();
+    /**
+     * Lazy-loads dependencies for non-strategy methods
+     */
+    private ensureDependencies;
+    /**
+     * Creates a company record with domain normalization
+     * Delegates to CompanyCreator strategy
+     */
+    createCompany(input: Record<string, unknown>): Promise<AttioRecord>;
+    /**
+     * Creates a person record with name and email normalization
+     * Delegates to PersonCreator strategy
+     */
+    createPerson(input: Record<string, unknown>): Promise<AttioRecord>;
+    /**
+     * Creates a task record via delegation to tasks object
+     * Delegates to TaskCreator strategy
+     */
+    createTask(input: Record<string, unknown>): Promise<AttioRecord>;
+    /**
+     * Updates a task record via delegation to tasks object
+     *
+     * Note: This method doesn't use strategy pattern as it's an update operation
+     * and the existing logic is simple enough to keep inline
+     */
+    updateTask(taskId: string, input: Record<string, unknown>): Promise<AttioRecord>;
+    /**
+     * Creates a note record via delegation to notes object
+     * Delegates to NoteCreator strategy
+     */
+    createNote(input: {
+        resource_type: string;
+        record_id: string;
+        title: string;
+        content: string;
+        format?: string;
+    }): Promise<Record<string, unknown>>;
+    /**
+     * Lists notes for a resource
+     *
+     * Note: This method doesn't use strategy pattern as it's a read operation
+     * and the existing logic is simple enough to keep inline
+     */
+    listNotes(params: {
+        resource_type?: string;
+        record_id?: string;
+    }): Promise<Record<string, unknown>[]>;
+    /**
+     * Validates and gets a creator for the specified resource type
+     * @private
+     */
+    private getCreator;
+    /**
+     * Finds the closest matching resource type for better error messages
+     * @private
+     */
+    private findClosestResourceType;
+    /**
+     * Calculates simple string similarity score
+     * @private
+     */
+    private calculateSimilarity;
+    /**
+     * Adds a new creator for a resource type (for future extensibility)
+     * @param resourceType - The resource type identifier
+     * @param creator - The creator implementation
+     */
+    addCreator(resourceType: string, creator: ResourceCreator): void;
+    /**
+     * Gets all supported resource types
+     */
+    getSupportedResourceTypes(): string[];
+    /**
+     * Validates if a resource type is supported
+     */
+    isResourceTypeSupported(resourceType: string): boolean;
+}
+//# sourceMappingURL=attio-create.service.d.ts.map

package/dist/services/create/attio-create.service.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"attio-create.service.d.ts","sourceRoot":"","sources":["../../../src/services/create/attio-create.service.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;GAiBG;AAgCH,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,YAAY,CAAC;AAChD,OAAO,KAAK,EAAE,WAAW,EAAa,MAAM,sBAAsB,CAAC;AACnE,OAAO,KAAK,EACV,eAAe,EAEhB,MAAM,qBAAqB,CAAC;AAU7B;;;;;;;;;;;;;;GAcG;AACH,qBAAa,kBAAmB,YAAW,aAAa;IACtD,OAAO,CAAC,QAAQ,CAAC,QAAQ,CAA+B;IACxD,OAAO,CAAC,QAAQ,CAAC,OAAO,CAAyB;IAGjD,OAAO,CAAC,UAAU,CAA2B;IAC7C,OAAO,CAAC,eAAe,CAAgC;IACvD,OAAO,CAAC,UAAU,CAA2B;IAG7C,MAAM,CAAC,QAAQ,CAAC,wBAAwB;;;;;MAK7B;;IA8BX;;OAEG;YACW,kBAAkB;IAchC;;;OAGG;IACG,aAAa,CAAC,KAAK,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,OAAO,CAAC,WAAW,CAAC;IAKzE;;;OAGG;IACG,YAAY,CAAC,KAAK,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,OAAO,CAAC,WAAW,CAAC;IAKxE;;;OAGG;IACG,UAAU,CAAC,KAAK,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,OAAO,CAAC,WAAW,CAAC;IAKtE;;;;;OAKG;IACG,UAAU,CACd,MAAM,EAAE,MAAM,EACd,KAAK,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAC7B,OAAO,CAAC,WAAW,CAAC;IA0BvB;;;OAGG;IACG,UAAU,CAAC,KAAK,EAAE;QACtB,aAAa,EAAE,MAAM,CAAC;QACtB,SAAS,EAAE,MAAM,CAAC;QAClB,KAAK,EAAE,MAAM,CAAC;QACd,OAAO,EAAE,MAAM,CAAC;QAChB,MAAM,CAAC,EAAE,MAAM,CAAC;KACjB,GAAG,OAAO,CAAC,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC;IAKpC;;;;;OAKG;IACG,SAAS,CAAC,MAAM,EAAE;QACtB,aAAa,CAAC,EAAE,MAAM,CAAC;QACvB,SAAS,CAAC,EAAE,MAAM,CAAC;KACpB,GAAG,OAAO,CAAC,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,EAAE,CAAC;IAatC;;;OAGG;IACH,OAAO,CAAC,UAAU;IA4BlB;;;OAGG;IACH,OAAO,CAAC,uBAAuB;IAkB/B;;;OAGG;IACH,OAAO,CAAC,mBAAmB;IAa3B;;;;OAIG;IACH,UAAU,CAAC,YAAY,EAAE,MAAM,EAAE,OAAO,EAAE,eAAe,GAAG,IAAI;IAIhE;;OAEG;IACH,yBAAyB,IAAI,MAAM,EAAE;IAIrC;;OAEG;IACH,uBAAuB,CAAC,YAAY,EAAE,MAAM,GAAG,OAAO;CAMvD"}

package/dist/services/create/attio-create.service.js

@@ -0,0 +1,227 @@
+/**
+ * AttioCreateService - Refactored with Strategy Pattern (Issue #552)
+ *
+ * REFACTORED: Implements the Strategy Pattern using resource-specific creators to handle
+ * different Attio resource types. Each creator encapsulates the logic for
+ * creating one type of resource, promoting Single Responsibility Principle.
+ *
+ * This refactoring addresses SRP violations by moving resource-specific logic
+ * from large methods (~139-168 lines) into focused creator classes (~50-80 lines each).
+ *
+ * Key improvements:
+ * - Single Responsibility: Each creator handles one resource type
+ * - Maintainability: Easy to add new resource types
+ * - Testability: Test each creator independently
+ * - Code Reuse: Shared utilities in BaseCreator
+ *
+ * See src/services/create/creators/README.md for full documentation.
+ */
+import { getLazyAttioClient } from '../../api/lazy-client.js';
+import { debug, error as logError } from '../../utils/logger.js';
+import { CompanyCreator, PersonCreator, TaskCreator, NoteCreator, } from './creators/index.js';
+/**
+ * Refactored implementation using Strategy Pattern
+ *
+ * Uses resource-specific creators to handle different resource types,
+ * promoting separation of concerns and Single Responsibility Principle.
+ *
+ * @example
+ * ```typescript
+ * const service = new AttioCreateService();
+ * const company = await service.createCompany({
+ *   name: "Acme Corp",
+ *   domain: "acme.com"
+ * });
+ * ```
+ */
+export class AttioCreateService {
+    creators;
+    context;
+    // Lazy-loaded dependencies for non-strategy methods
+    taskModule = null;
+    converterModule = null;
+    noteModule = null;
+    // Supported resource types for validation
+    static SUPPORTED_RESOURCE_TYPES = {
+        COMPANIES: 'companies',
+        PEOPLE: 'people',
+        TASKS: 'tasks',
+        NOTES: 'notes',
+    };
+    constructor() {
+        // Initialize resource creators using Strategy Pattern
+        this.creators = new Map();
+        this.creators.set(AttioCreateService.SUPPORTED_RESOURCE_TYPES.COMPANIES, new CompanyCreator());
+        this.creators.set(AttioCreateService.SUPPORTED_RESOURCE_TYPES.PEOPLE, new PersonCreator());
+        this.creators.set(AttioCreateService.SUPPORTED_RESOURCE_TYPES.TASKS, new TaskCreator());
+        this.creators.set(AttioCreateService.SUPPORTED_RESOURCE_TYPES.NOTES, new NoteCreator());
+        // Create shared context for all creators
+        this.context = {
+            client: getLazyAttioClient(), // Lazily gets the client with proper Authorization
+            debug,
+            logError,
+        };
+    }
+    /**
+     * Lazy-loads dependencies for non-strategy methods
+     */
+    async ensureDependencies() {
+        if (!this.taskModule) {
+            this.taskModule = (await import('../../objects/tasks.js'));
+        }
+        if (!this.converterModule) {
+            this.converterModule = (await import('./data-normalizers.js'));
+        }
+        if (!this.noteModule) {
+            this.noteModule = (await import('../../objects/notes.js'));
+        }
+    }
+    /**
+     * Creates a company record with domain normalization
+     * Delegates to CompanyCreator strategy
+     */
+    async createCompany(input) {
+        const creator = this.getCreator('companies');
+        return creator.create(input, this.context);
+    }
+    /**
+     * Creates a person record with name and email normalization
+     * Delegates to PersonCreator strategy
+     */
+    async createPerson(input) {
+        const creator = this.getCreator('people');
+        return creator.create(input, this.context);
+    }
+    /**
+     * Creates a task record via delegation to tasks object
+     * Delegates to TaskCreator strategy
+     */
+    async createTask(input) {
+        const creator = this.getCreator('tasks');
+        return creator.create(input, this.context);
+    }
+    /**
+     * Updates a task record via delegation to tasks object
+     *
+     * Note: This method doesn't use strategy pattern as it's an update operation
+     * and the existing logic is simple enough to keep inline
+     */
+    async updateTask(taskId, input) {
+        // Ensure dependencies are loaded
+        await this.ensureDependencies();
+        const updatedTask = await this.taskModule?.updateTask(taskId, {
+            content: input.content,
+            status: input.status,
+            assigneeId: input.assigneeId,
+            dueDate: input.dueDate,
+            recordIds: input.recordIds,
+        });
+        // Convert task to AttioRecord format
+        if (!updatedTask) {
+            throw new Error('Task update failed - no task returned');
+        }
+        const result = this.converterModule?.convertTaskToAttioRecord(updatedTask, input);
+        if (!result) {
+            throw new Error('Task converter module not available');
+        }
+        return result;
+    }
+    /**
+     * Creates a note record via delegation to notes object
+     * Delegates to NoteCreator strategy
+     */
+    async createNote(input) {
+        const creator = this.getCreator('notes');
+        return creator.create(input, this.context);
+    }
+    /**
+     * Lists notes for a resource
+     *
+     * Note: This method doesn't use strategy pattern as it's a read operation
+     * and the existing logic is simple enough to keep inline
+     */
+    async listNotes(params) {
+        // Ensure dependencies are loaded
+        await this.ensureDependencies();
+        const query = {
+            parent_object: params.resource_type,
+            parent_record_id: params.record_id,
+        };
+        const response = await this.noteModule?.listNotes(query);
+        return response?.data || [];
+    }
+    /**
+     * Validates and gets a creator for the specified resource type
+     * @private
+     */
+    getCreator(resourceType) {
+        // Validate input
+        if (!resourceType || typeof resourceType !== 'string') {
+            throw new Error(`Invalid resource type: expected non-empty string, got ${typeof resourceType}`);
+        }
+        const normalizedType = resourceType.toLowerCase().trim();
+        const creator = this.creators.get(normalizedType);
+        if (!creator) {
+            const supportedTypes = Array.from(this.creators.keys()).sort();
+            const suggestion = this.findClosestResourceType(normalizedType, supportedTypes);
+            throw new Error(`Unsupported resource type: "${resourceType}". ` +
+                `Supported types: ${supportedTypes.join(', ')}.` +
+                (suggestion ? ` Did you mean "${suggestion}"?` : ''));
+        }
+        return creator;
+    }
+    /**
+     * Finds the closest matching resource type for better error messages
+     * @private
+     */
+    findClosestResourceType(input, supportedTypes) {
+        // Simple similarity check - could be enhanced with better algorithms
+        const similarities = supportedTypes.map((type) => ({
+            type,
+            score: this.calculateSimilarity(input, type),
+        }));
+        const best = similarities.reduce((prev, current) => prev.score > current.score ? prev : current);
+        // Only suggest if similarity is reasonable (> 0.5)
+        return best.score > 0.5 ? best.type : null;
+    }
+    /**
+     * Calculates simple string similarity score
+     * @private
+     */
+    calculateSimilarity(a, b) {
+        if (a === b)
+            return 1;
+        if (a.length === 0 || b.length === 0)
+            return 0;
+        // Simple character overlap calculation
+        const setA = new Set(a.toLowerCase());
+        const setB = new Set(b.toLowerCase());
+        const intersection = new Set(Array.from(setA).filter((x) => setB.has(x)));
+        const union = new Set([...Array.from(setA), ...Array.from(setB)]);
+        return intersection.size / union.size;
+    }
+    /**
+     * Adds a new creator for a resource type (for future extensibility)
+     * @param resourceType - The resource type identifier
+     * @param creator - The creator implementation
+     */
+    addCreator(resourceType, creator) {
+        this.creators.set(resourceType, creator);
+    }
+    /**
+     * Gets all supported resource types
+     */
+    getSupportedResourceTypes() {
+        return Array.from(this.creators.keys());
+    }
+    /**
+     * Validates if a resource type is supported
+     */
+    isResourceTypeSupported(resourceType) {
+        if (!resourceType || typeof resourceType !== 'string') {
+            return false;
+        }
+        return this.creators.has(resourceType.toLowerCase().trim());
+    }
+}
+//# sourceMappingURL=attio-create.service.js.map

package/dist/services/create/attio-create.service.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"attio-create.service.js","sourceRoot":"","sources":["../../../src/services/create/attio-create.service.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;GAiBG;AAsCH,OAAO,EAAE,kBAAkB,EAAE,MAAM,0BAA0B,CAAC;AAC9D,OAAO,EAAE,KAAK,EAAE,KAAK,IAAI,QAAQ,EAAE,MAAM,uBAAuB,CAAC;AACjE,OAAO,EACL,cAAc,EACd,aAAa,EACb,WAAW,EACX,WAAW,GACZ,MAAM,qBAAqB,CAAC;AAE7B;;;;;;;;;;;;;;GAcG;AACH,MAAM,OAAO,kBAAkB;IACZ,QAAQ,CAA+B;IACvC,OAAO,CAAyB;IAEjD,oDAAoD;IAC5C,UAAU,GAAsB,IAAI,CAAC;IACrC,eAAe,GAA2B,IAAI,CAAC;IAC/C,UAAU,GAAsB,IAAI,CAAC;IAE7C,0CAA0C;IAC1C,MAAM,CAAU,wBAAwB,GAAG;QACzC,SAAS,EAAE,WAAW;QACtB,MAAM,EAAE,QAAQ;QAChB,KAAK,EAAE,OAAO;QACd,KAAK,EAAE,OAAO;KACN,CAAC;IAEX;QACE,sDAAsD;QACtD,IAAI,CAAC,QAAQ,GAAG,IAAI,GAAG,EAA2B,CAAC;QACnD,IAAI,CAAC,QAAQ,CAAC,GAAG,CACf,kBAAkB,CAAC,wBAAwB,CAAC,SAAS,EACrD,IAAI,cAAc,EAAE,CACrB,CAAC;QACF,IAAI,CAAC,QAAQ,CAAC,GAAG,CACf,kBAAkB,CAAC,wBAAwB,CAAC,MAAM,EAClD,IAAI,aAAa,EAAE,CACpB,CAAC;QACF,IAAI,CAAC,QAAQ,CAAC,GAAG,CACf,kBAAkB,CAAC,wBAAwB,CAAC,KAAK,EACjD,IAAI,WAAW,EAAE,CAClB,CAAC;QACF,IAAI,CAAC,QAAQ,CAAC,GAAG,CACf,kBAAkB,CAAC,wBAAwB,CAAC,KAAK,EACjD,IAAI,WAAW,EAAE,CAClB,CAAC;QAEF,yCAAyC;QACzC,IAAI,CAAC,OAAO,GAAG;YACb,MAAM,EAAE,kBAAkB,EAAE,EAAE,mDAAmD;YACjF,KAAK;YACL,QAAQ;SACT,CAAC;IACJ,CAAC;IAED;;OAEG;IACK,KAAK,CAAC,kBAAkB;QAC9B,IAAI,CAAC,IAAI,CAAC,UAAU,EAAE,CAAC;YACrB,IAAI,CAAC,UAAU,GAAG,CAAC,MAAM,MAAM,CAAC,wBAAwB,CAAC,CAAe,CAAC;QAC3E,CAAC;QACD,IAAI,CAAC,IAAI,CAAC,eAAe,EAAE,CAAC;YAC1B,IAAI,CAAC,eAAe,GAAG,CAAC,MAAM,MAAM,CAClC,uBAAuB,CACxB,CAAoB,CAAC;QACxB,CAAC;QACD,IAAI,CAAC,IAAI,CAAC,UAAU,EAAE,CAAC;YACrB,IAAI,CAAC,UAAU,GAAG,CAAC,MAAM,MAAM,CAAC,wBAAwB,CAAC,CAAe,CAAC;QAC3E,CAAC;IACH,CAAC;IAED;;;OAGG;IACH,KAAK,CAAC,aAAa,CAAC,KAA8B;QAChD,MAAM,OAAO,GAAG,IAAI,CAAC,UAAU,CAAC,WAAW,CAAC,CAAC;QAC7C,OAAO,OAAO,CAAC,MAAM,CAAC,KAAK,EAAE,IAAI,CAAC,OAAO,CAAC,CAAC;IAC7C,CAAC;IAED;;;OAGG;IACH,KAAK,CAAC,YAAY,CAAC,KAA8B;QAC/C,MAAM,OAAO,GAAG,IAAI,CAAC,UAAU,CAAC,QAAQ,CAAC,CAAC;QAC1C,OAAO,OAAO,CAAC,MAAM,CAAC,KAAK,EAAE,IAAI,CAAC,OAAO,CAAC,CAAC;IAC7C,CAAC;IAED;;;OAGG;IACH,KAAK,CAAC,UAAU,CAAC,KAA8B;QAC7C,MAAM,OAAO,GAAG,IAAI,CAAC,UAAU,CAAC,OAAO,CAAC,CAAC;QACzC,OAAO,OAAO,CAAC,MAAM,CAAC,KAAK,EAAE,IAAI,CAAC,OAAO,CAAC,CAAC;IAC7C,CAAC;IAED;;;;;OAKG;IACH,KAAK,CAAC,UAAU,CACd,MAAc,EACd,KAA8B;QAE9B,iCAAiC;QACjC,MAAM,IAAI,CAAC,kBAAkB,EAAE,CAAC;QAEhC,MAAM,WAAW,GAAG,MAAM,IAAI,CAAC,UAAU,EAAE,UAAU,CAAC,MAAM,EAAE;YAC5D,OAAO,EAAE,KAAK,CAAC,OAAiB;YAChC,MAAM,EAAE,KAAK,CAAC,MAAgB;YAC9B,UAAU,EAAE,KAAK,CAAC,UAAoB;YACtC,OAAO,EAAE,KAAK,CAAC,OAAiB;YAChC,SAAS,EAAE,KAAK,CAAC,SAAqB;SACvC,CAAC,CAAC;QAEH,qCAAqC;QACrC,IAAI,CAAC,WAAW,EAAE,CAAC;YACjB,MAAM,IAAI,KAAK,CAAC,uCAAuC,CAAC,CAAC;QAC3D,CAAC;QACD,MAAM,MAAM,GAAG,IAAI,CAAC,eAAe,EAAE,wBAAwB,CAC3D,WAAW,EACX,KAAK,CACN,CAAC;QACF,IAAI,CAAC,MAAM,EAAE,CAAC;YACZ,MAAM,IAAI,KAAK,CAAC,qCAAqC,CAAC,CAAC;QACzD,CAAC;QACD,OAAO,MAAM,CAAC;IAChB,CAAC;IAED;;;OAGG;IACH,KAAK,CAAC,UAAU,CAAC,KAMhB;QACC,MAAM,OAAO,GAAG,IAAI,CAAC,UAAU,CAAC,OAAO,CAAC,CAAC;QACzC,OAAO,OAAO,CAAC,MAAM,CAAC,KAAK,EAAE,IAAI,CAAC,OAAO,CAAC,CAAC;IAC7C,CAAC;IAED;;;;;OAKG;IACH,KAAK,CAAC,SAAS,CAAC,MAGf;QACC,iCAAiC;QACjC,MAAM,IAAI,CAAC,kBAAkB,EAAE,CAAC;QAEhC,MAAM,KAAK,GAAG;YACZ,aAAa,EAAE,MAAM,CAAC,aAAa;YACnC,gBAAgB,EAAE,MAAM,CAAC,SAAS;SACnC,CAAC;QAEF,MAAM,QAAQ,GAAG,MAAM,IAAI,CAAC,UAAU,EAAE,SAAS,CAAC,KAAK,CAAC,CAAC;QACzD,OAAO,QAAQ,EAAE,IAAI,IAAI,EAAE,CAAC;IAC9B,CAAC;IAED;;;OAGG;IACK,UAAU,CAAC,YAAoB;QACrC,iBAAiB;QACjB,IAAI,CAAC,YAAY,IAAI,OAAO,YAAY,KAAK,QAAQ,EAAE,CAAC;YACtD,MAAM,IAAI,KAAK,CACb,yDAAyD,OAAO,YAAY,EAAE,CAC/E,CAAC;QACJ,CAAC;QAED,MAAM,cAAc,GAAG,YAAY,CAAC,WAAW,EAAE,CAAC,IAAI,EAAE,CAAC;QACzD,MAAM,OAAO,GAAG,IAAI,CAAC,QAAQ,CAAC,GAAG,CAAC,cAAc,CAAC,CAAC;QAElD,IAAI,CAAC,OAAO,EAAE,CAAC;YACb,MAAM,cAAc,GAAG,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,QAAQ,CAAC,IAAI,EAAE,CAAC,CAAC,IAAI,EAAE,CAAC;YAC/D,MAAM,UAAU,GAAG,IAAI,CAAC,uBAAuB,CAC7C,cAAc,EACd,cAAc,CACf,CAAC;YAEF,MAAM,IAAI,KAAK,CACb,+BAA+B,YAAY,KAAK;gBAC9C,oBAAoB,cAAc,CAAC,IAAI,CAAC,IAAI,CAAC,GAAG;gBAChD,CAAC,UAAU,CAAC,CAAC,CAAC,kBAAkB,UAAU,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,CACvD,CAAC;QACJ,CAAC;QAED,OAAO,OAAO,CAAC;IACjB,CAAC;IAED;;;OAGG;IACK,uBAAuB,CAC7B,KAAa,EACb,cAAwB;QAExB,qEAAqE;QACrE,MAAM,YAAY,GAAG,cAAc,CAAC,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC;YACjD,IAAI;YACJ,KAAK,EAAE,IAAI,CAAC,mBAAmB,CAAC,KAAK,EAAE,IAAI,CAAC;SAC7C,CAAC,CAAC,CAAC;QAEJ,MAAM,IAAI,GAAG,YAAY,CAAC,MAAM,CAAC,CAAC,IAAI,EAAE,OAAO,EAAE,EAAE,CACjD,IAAI,CAAC,KAAK,GAAG,OAAO,CAAC,KAAK,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,OAAO,CAC5C,CAAC;QAEF,mDAAmD;QACnD,OAAO,IAAI,CAAC,KAAK,GAAG,GAAG,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC,IAAI,CAAC;IAC7C,CAAC;IAED;;;OAGG;IACK,mBAAmB,CAAC,CAAS,EAAE,CAAS;QAC9C,IAAI,CAAC,KAAK,CAAC;YAAE,OAAO,CAAC,CAAC;QACtB,IAAI,CAAC,CAAC,MAAM,KAAK,CAAC,IAAI,CAAC,CAAC,MAAM,KAAK,CAAC;YAAE,OAAO,CAAC,CAAC;QAE/C,uCAAuC;QACvC,MAAM,IAAI,GAAG,IAAI,GAAG,CAAC,CAAC,CAAC,WAAW,EAAE,CAAC,CAAC;QACtC,MAAM,IAAI,GAAG,IAAI,GAAG,CAAC,CAAC,CAAC,WAAW,EAAE,CAAC,CAAC;QACtC,MAAM,YAAY,GAAG,IAAI,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;QAC1E,MAAM,KAAK,GAAG,IAAI,GAAG,CAAC,CAAC,GAAG,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,GAAG,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAElE,OAAO,YAAY,CAAC,IAAI,GAAG,KAAK,CAAC,IAAI,CAAC;IACxC,CAAC;IAED;;;;OAIG;IACH,UAAU,CAAC,YAAoB,EAAE,OAAwB;QACvD,IAAI,CAAC,QAAQ,CAAC,GAAG,CAAC,YAAY,EAAE,OAAO,CAAC,CAAC;IAC3C,CAAC;IAED;;OAEG;IACH,yBAAyB;QACvB,OAAO,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,QAAQ,CAAC,IAAI,EAAE,CAAC,CAAC;IAC1C,CAAC;IAED;;OAEG;IACH,uBAAuB,CAAC,YAAoB;QAC1C,IAAI,CAAC,YAAY,IAAI,OAAO,YAAY,KAAK,QAAQ,EAAE,CAAC;YACtD,OAAO,KAAK,CAAC;QACf,CAAC;QACD,OAAO,IAAI,CAAC,QAAQ,CAAC,GAAG,CAAC,YAAY,CAAC,WAAW,EAAE,CAAC,IAAI,EAAE,CAAC,CAAC;IAC9D,CAAC"}

package/dist/services/create/creators/base-creator.d.ts

@@ -0,0 +1,86 @@
+/**
+ * BaseCreator - Abstract base class for all resource creators
+ *
+ * Provides common functionality and utilities shared across all resource creators.
+ * Implements Strategy Pattern base behavior including error handling, recovery,
+ * and response processing.
+ */
+import type { AttioRecord, JsonObject } from '../../../types/attio.js';
+import type { ResourceCreator, ResourceCreatorContext, RecoveryOptions } from './types.js';
+import { EnhancedApiError } from '../../../errors/enhanced-api-errors.js';
+/**
+ * Abstract base class for resource creators
+ * Provides shared functionality and enforces creator interface
+ */
+export declare abstract class BaseCreator implements ResourceCreator {
+    abstract readonly resourceType: string;
+    abstract readonly endpoint: string;
+    /**
+     * Creates a resource record (implemented by subclasses)
+     */
+    abstract create(input: JsonObject, context: ResourceCreatorContext): Promise<AttioRecord>;
+    /**
+     * Normalizes input data for the specific resource type
+     * Override in subclasses for resource-specific normalization
+     */
+    protected normalizeInput(input: JsonObject): JsonObject;
+    /**
+     * Validates required array field presence
+     */
+    protected assertRequiredArray(payload: JsonObject, field: string, errorMessage: string): void;
+    /**
+     * Creates the API payload for resource creation
+     */
+    protected createPayload(normalizedInput: JsonObject): JsonObject;
+    /**
+     * Processes API response and extracts record
+     */
+    protected processResponse(response: JsonObject, context: ResourceCreatorContext, normalizedInput?: JsonObject): Promise<AttioRecord>;
+    /**
+     * Logs API response metadata for debugging
+     */
+    private logResponseMetadata;
+    /**
+     * Extracts record from response and enriches with ID if needed
+     */
+    private extractAndEnrichRecord;
+    /**
+     * Validates record and attempts recovery if needed
+     */
+    private validateAndRecoverRecord;
+    /**
+     * Determines if recovery should be attempted based on record state
+     */
+    private shouldAttemptRecovery;
+    /**
+     * Finalizes record validation and adds debug logging
+     */
+    private finalizeAndDebugRecord;
+    /**
+     * Enriches record with ID extracted from web_url if missing
+     */
+    protected enrichRecordId(record: JsonObject, response: JsonObject): JsonObject;
+    /**
+     * Attempts to recover record by searching for it
+     * Override in subclasses to implement resource-specific recovery
+     */
+    protected attemptRecovery(context: ResourceCreatorContext, _normalizedInput?: JsonObject): Promise<AttioRecord>;
+    /**
+     * Gets recovery options for this resource type
+     * Override in subclasses to provide resource-specific recovery
+     */
+    protected getRecoveryOptions(): RecoveryOptions | null;
+    /**
+     * Creates enhanced API error with context
+     */
+    /**
+     * Fails fast if auth is missing to avoid confusing "200 {}" responses
+     */
+    protected assertClientHasAuth(context: ResourceCreatorContext): void;
+    protected createEnhancedError(error: Error, context: ResourceCreatorContext, status?: number): EnhancedApiError;
+    /**
+     * Handles API errors during creation
+     */
+    protected handleApiError(err: unknown, context: ResourceCreatorContext, payload?: JsonObject): never;
+}
+//# sourceMappingURL=base-creator.d.ts.map

package/dist/services/create/creators/base-creator.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"base-creator.d.ts","sourceRoot":"","sources":["../../../../src/services/create/creators/base-creator.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAEH,OAAO,KAAK,EAAE,WAAW,EAAE,UAAU,EAAE,MAAM,wBAAwB,CAAC;AACtE,OAAO,KAAK,EACV,eAAe,EACf,sBAAsB,EAEtB,eAAe,EAChB,MAAM,YAAY,CAAC;AACpB,OAAO,EAAE,gBAAgB,EAAE,MAAM,wCAAwC,CAAC;AAU1E;;;GAGG;AACH,8BAAsB,WAAY,YAAW,eAAe;IAC1D,QAAQ,CAAC,QAAQ,CAAC,YAAY,EAAE,MAAM,CAAC;IACvC,QAAQ,CAAC,QAAQ,CAAC,QAAQ,EAAE,MAAM,CAAC;IAEnC;;OAEG;IACH,QAAQ,CAAC,MAAM,CACb,KAAK,EAAE,UAAU,EACjB,OAAO,EAAE,sBAAsB,GAC9B,OAAO,CAAC,WAAW,CAAC;IAEvB;;;OAGG;IACH,SAAS,CAAC,cAAc,CAAC,KAAK,EAAE,UAAU,GAAG,UAAU;IAIvD;;OAEG;IACH,SAAS,CAAC,mBAAmB,CAC3B,OAAO,EAAE,UAAU,EACnB,KAAK,EAAE,MAAM,EACb,YAAY,EAAE,MAAM,GACnB,IAAI;IAIP;;OAEG;IACH,SAAS,CAAC,aAAa,CAAC,eAAe,EAAE,UAAU,GAAG,UAAU;IAQhE;;OAEG;cACa,eAAe,CAC7B,QAAQ,EAAE,UAAU,EACpB,OAAO,EAAE,sBAAsB,EAC/B,eAAe,CAAC,EAAE,UAAU,GAC3B,OAAO,CAAC,WAAW,CAAC;IAcvB;;OAEG;IACH,OAAO,CAAC,mBAAmB;IAe3B;;OAEG;IACH,OAAO,CAAC,sBAAsB;IAK9B;;OAEG;YACW,wBAAwB;IActC;;OAEG;IACH,OAAO,CAAC,qBAAqB;IAQ7B;;OAEG;IACH,OAAO,CAAC,sBAAsB;IAiB9B;;OAEG;IACH,SAAS,CAAC,cAAc,CACtB,MAAM,EAAE,UAAU,EAClB,QAAQ,EAAE,UAAU,GACnB,UAAU;IAYb;;;OAGG;cACa,eAAe,CAC7B,OAAO,EAAE,sBAAsB,EAE/B,gBAAgB,CAAC,EAAE,UAAU,GAC5B,OAAO,CAAC,WAAW,CAAC;IAkEvB;;;OAGG;IACH,SAAS,CAAC,kBAAkB,IAAI,eAAe,GAAG,IAAI;IAItD;;OAEG;IACH;;OAEG;IACH,SAAS,CAAC,mBAAmB,CAAC,OAAO,EAAE,sBAAsB;IAc7D,SAAS,CAAC,mBAAmB,CAC3B,KAAK,EAAE,KAAK,EACZ,OAAO,EAAE,sBAAsB,EAC/B,MAAM,GAAE,MAAY,GACnB,gBAAgB;IA8BnB;;OAEG;IACH,SAAS,CAAC,cAAc,CACtB,GAAG,EAAE,OAAO,EACZ,OAAO,EAAE,sBAAsB,EAC/B,OAAO,CAAC,EAAE,UAAU,GACnB,KAAK;CA4BT"}

package/dist/services/create/creators/base-creator.js

@@ -0,0 +1,221 @@
+/**
+ * BaseCreator - Abstract base class for all resource creators
+ *
+ * Provides common functionality and utilities shared across all resource creators.
+ * Implements Strategy Pattern base behavior including error handling, recovery,
+ * and response processing.
+ */
+import { EnhancedApiError } from '../../../errors/enhanced-api-errors.js';
+import { extractRecordId } from '../../../utils/validation/uuid-validation.js';
+import { extractAttioRecord, assertLooksLikeCreated, isTestRun, debugRecordShape, } from '../extractor.js';
+import { validateRequiredArrayField } from '../validators.js';
+/**
+ * Abstract base class for resource creators
+ * Provides shared functionality and enforces creator interface
+ */
+export class BaseCreator {
+    /**
+     * Normalizes input data for the specific resource type
+     * Override in subclasses for resource-specific normalization
+     */
+    normalizeInput(input) {
+        return input;
+    }
+    /**
+     * Validates required array field presence
+     */
+    assertRequiredArray(payload, field, errorMessage) {
+        validateRequiredArrayField(payload, field, errorMessage);
+    }
+    /**
+     * Creates the API payload for resource creation
+     */
+    createPayload(normalizedInput) {
+        return {
+            data: {
+                values: normalizedInput,
+            },
+        };
+    }
+    /**
+     * Processes API response and extracts record
+     */
+    async processResponse(response, context, normalizedInput) {
+        this.logResponseMetadata(response, context);
+        const extractedRecord = this.extractAndEnrichRecord(response);
+        const finalRecord = await this.validateAndRecoverRecord(extractedRecord, context, normalizedInput);
+        return this.finalizeAndDebugRecord(finalRecord, context);
+    }
+    /**
+     * Logs API response metadata for debugging
+     */
+    logResponseMetadata(response, context) {
+        context.debug(this.constructor.name, `${this.resourceType} API response`, {
+            status: response?.status,
+            statusText: response?.statusText,
+            hasData: !!response?.data,
+            hasNestedData: !!response?.data?.data,
+            dataKeys: response?.data
+                ? Object.keys(response.data)
+                : [],
+        });
+    }
+    /**
+     * Extracts record from response and enriches with ID if needed
+     */
+    extractAndEnrichRecord(response) {
+        const record = extractAttioRecord(response);
+        return this.enrichRecordId(record || {}, response);
+    }
+    /**
+     * Validates record and attempts recovery if needed
+     */
+    async validateAndRecoverRecord(record, context, normalizedInput) {
+        const mustRecover = this.shouldAttemptRecovery(record);
+        if (mustRecover) {
+            return await this.attemptRecovery(context, normalizedInput);
+        }
+        return record;
+    }
+    /**
+     * Determines if recovery should be attempted based on record state
+     */
+    shouldAttemptRecovery(record) {
+        return (!record ||
+            !record.id ||
+            !record.id?.record_id);
+    }
+    /**
+     * Finalizes record validation and adds debug logging
+     */
+    finalizeAndDebugRecord(record, context) {
+        assertLooksLikeCreated(record, `${this.constructor.name}.create`);
+        if (isTestRun()) {
+            context.debug(this.constructor.name, `Normalized ${this.resourceType} record`, debugRecordShape(record));
+        }
+        return record;
+    }
+    /**
+     * Enriches record with ID extracted from web_url if missing
+     */
+    enrichRecordId(record, response) {
+        if (record && (!record.id || !record.id?.record_id)) {
+            const webUrl = record?.web_url || response?.data?.web_url;
+            const rid = webUrl ? extractRecordId(String(webUrl)) : undefined;
+            if (rid) {
+                const existingId = record.id || {};
+                record.id = { ...existingId, record_id: rid };
+            }
+        }
+        return record;
+    }
+    /**
+     * Attempts to recover record by searching for it
+     * Override in subclasses to implement resource-specific recovery
+     */
+    async attemptRecovery(context, 
+    // eslint-disable-next-line @typescript-eslint/no-unused-vars
+    _normalizedInput) {
+        const recoveryOptions = this.getRecoveryOptions();
+        if (!recoveryOptions) {
+            throw this.createEnhancedError(new Error(`${this.resourceType} creation returned empty/invalid record`), context, 500);
+        }
+        for (const filter of recoveryOptions.searchFilters) {
+            try {
+                const searchEndpoint = `${this.endpoint}/search`;
+                const searchFilter = {
+                    [filter.field]: filter.operator === 'contains'
+                        ? { contains: filter.value }
+                        : { eq: filter.value },
+                };
+                const { data: searchResult } = await context.client.post(searchEndpoint, {
+                    filter: searchFilter,
+                    limit: 1,
+                    order: { created_at: 'desc' },
+                });
+                const record = extractAttioRecord(searchResult);
+                const typedRecord = record;
+                const recordId = typedRecord?.id
+                    ? typedRecord.id?.record_id
+                    : undefined;
+                if (recordId) {
+                    context.debug(this.constructor.name, `${this.resourceType} recovery succeeded`, {
+                        recoveredBy: filter.field,
+                        recordId,
+                    });
+                    return typedRecord;
+                }
+            }
+            catch (e) {
+                context.debug(this.constructor.name, `${this.resourceType} recovery attempt failed`, {
+                    field: filter.field,
+                    message: e?.message,
+                });
+            }
+        }
+        throw this.createEnhancedError(new Error(`${this.resourceType} creation and recovery both failed`), context, 500);
+    }
+    /**
+     * Gets recovery options for this resource type
+     * Override in subclasses to provide resource-specific recovery
+     */
+    getRecoveryOptions() {
+        return null;
+    }
+    /**
+     * Creates enhanced API error with context
+     */
+    /**
+     * Fails fast if auth is missing to avoid confusing "200 {}" responses
+     */
+    assertClientHasAuth(context) {
+        const common = context.client?.defaults?.headers?.common ?? {};
+        const direct = context.client?.defaults?.headers ?? {};
+        const auth = (common['Authorization'] ??
+            common['authorization'] ??
+            direct['Authorization'] ??
+            direct['authorization']);
+        if (!auth) {
+            throw new Error('Attio client has no Authorization header.');
+        }
+    }
+    createEnhancedError(error, context, status = 500) {
+        const errorInfo = {
+            operation: 'create',
+            endpoint: this.endpoint,
+            resourceType: this.resourceType,
+            originalError: error,
+            httpStatus: status,
+        };
+        context.logError(this.constructor.name, `${this.resourceType} creation error`, errorInfo);
+        let message;
+        if (status === 500) {
+            message = `invalid request: Attio ${this.resourceType} creation failed with a server error.`;
+        }
+        else {
+            message = `Attio ${this.resourceType} creation failed (${status}): ${error.message}`;
+        }
+        return new EnhancedApiError(message, status, this.endpoint, 'POST', {
+            httpStatus: status,
+            resourceType: this.resourceType,
+            operation: 'create',
+            originalError: error,
+        });
+    }
+    /**
+     * Handles API errors during creation
+     */
+    handleApiError(err, context, payload) {
+        const error = err;
+        const status = error?.response?.status ?? 500;
+        const data = error?.response?.data;
+        const detailMessage = data?.message;
+        context.logError(this.constructor.name, `${this.resourceType} API error details`, {
+            status,
+            errorBody: data,
+            requestPayload: payload,
+        });
+        throw this.createEnhancedError(new Error(detailMessage || error?.message || `${this.resourceType} creation error`), context, status);
+    }
+}
+//# sourceMappingURL=base-creator.js.map