@t4dhg/mcp-factorial 4.0.0 → 6.0.0

package/dist/api.js

@@ -5,8 +5,12 @@
  */
 import { fetchList, fetchOne, postOne, patchOne, deleteOne, postAction } from './http-client.js';
 import { cache, cached, CACHE_TTL, CacheManager } from './cache.js';
+import { debug } from './config.js';
 import { buildPaginationParams, paginateResponse, sliceForPagination, } from './pagination.js';
 import { AuditAction, auditedOperation } from './audit.js';
+import { validateId } from './utils.js';
+import { ENDPOINTS, endpointWithId, endpointWithAction } from './endpoints.js';
+import { NotFoundError } from './errors.js';
 // ============================================================================
 // Employee endpoints
 // ============================================================================
@@ -19,7 +23,7 @@ export async function listEmployees(options) {
     // For filtered requests, we need to fetch all and filter client-side
     // because the API doesn't reliably filter
     if (options?.team_id || options?.location_id) {
-        const allEmployees = await cached('employees:all', () => fetchList('/employees/employees'), CACHE_TTL.employees);
+        const allEmployees = await cached('employees:all', () => fetchList(ENDPOINTS.employees), CACHE_TTL.employees);
         let filtered = allEmployees;
         if (options.team_id) {
             filtered = filtered.filter(e => e.team_ids?.includes(options.team_id));
@@ -30,7 +34,7 @@ export async function listEmployees(options) {
         return sliceForPagination(filtered, params);
     }
     // Without filters, use pagination directly
-    const employees = await cached(cacheKey, () => fetchList('/employees/employees', { params }), CACHE_TTL.employees);
+    const employees = await cached(cacheKey, () => fetchList(ENDPOINTS.employees, { params }), CACHE_TTL.employees);
     return paginateResponse(employees, params.page, params.limit);
 }
 /**
@@ -41,12 +45,10 @@ export async function listEmployees(options) {
  * if the direct endpoint fails or returns no data.
  */
 export async function getEmployee(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid employee ID. Please provide a positive number.');
-    }
+    validateId(id, 'employee');
     // Try the direct endpoint first
     try {
-        const employee = await cached(`employee:${id}`, () => fetchOne(`/employees/employees/${id}`), CACHE_TTL.employees);
+        const employee = await cached(`employee:${id}`, () => fetchOne(endpointWithId(ENDPOINTS.employees, id)), CACHE_TTL.employees);
         // If we got a valid employee, return it
         if (employee) {
             return employee;
@@ -55,13 +57,13 @@ export async function getEmployee(id) {
     catch (error) {
         // If direct fetch fails with NotFoundError, try fallback
         // (other errors will be re-thrown below)
-        if (!(error instanceof Error && error.message.includes('not found'))) {
+        if (!(error instanceof NotFoundError)) {
             throw error;
         }
     }
     // Fallback: Fetch all employees and filter (same approach as searchEmployees)
     // This works around Factorial API limitations with the individual employee endpoint
-    const allEmployees = await cached('employees:all', () => fetchList('/employees/employees'), CACHE_TTL.employees);
+    const allEmployees = await cached('employees:all', () => fetchList(ENDPOINTS.employees), CACHE_TTL.employees);
     const employee = allEmployees.find(emp => emp.id === id);
     if (!employee) {
         throw new Error(`Employee with ID ${id} not found.`);
@@ -75,7 +77,7 @@ export async function searchEmployees(query) {
     if (!query || query.trim().length < 2) {
         throw new Error('Search query must be at least 2 characters long.');
     }
-    const allEmployees = await cached('employees:all', () => fetchList('/employees/employees'), CACHE_TTL.employees);
+    const allEmployees = await cached('employees:all', () => fetchList(ENDPOINTS.employees), CACHE_TTL.employees);
     const lowerQuery = query.toLowerCase().trim();
     return allEmployees.filter(emp => emp.full_name?.toLowerCase().includes(lowerQuery) ||
         emp.email?.toLowerCase().includes(lowerQuery) ||
@@ -91,17 +93,15 @@ export async function searchEmployees(query) {
 export async function listTeams(options) {
     const params = buildPaginationParams(options);
     const cacheKey = CacheManager.key('teams', options);
-    const teams = await cached(cacheKey, () => fetchList('/teams/teams'), CACHE_TTL.teams);
+    const teams = await cached(cacheKey, () => fetchList(ENDPOINTS.teams), CACHE_TTL.teams);
     return sliceForPagination(teams, params);
 }
 /**
  * Get a specific team by ID
  */
 export async function getTeam(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid team ID. Please provide a positive number.');
-    }
-    return cached(`team:${id}`, () => fetchOne(`/teams/teams/${id}`), CACHE_TTL.teams);
+    validateId(id, 'team');
+    return cached(`team:${id}`, () => fetchOne(endpointWithId(ENDPOINTS.teams, id)), CACHE_TTL.teams);
 }
 // ============================================================================
 // Location endpoints
@@ -112,17 +112,15 @@ export async function getTeam(id) {
 export async function listLocations(options) {
     const params = buildPaginationParams(options);
     const cacheKey = CacheManager.key('locations', options);
-    const locations = await cached(cacheKey, () => fetchList('/locations/locations'), CACHE_TTL.locations);
+    const locations = await cached(cacheKey, () => fetchList(ENDPOINTS.locations), CACHE_TTL.locations);
     return sliceForPagination(locations, params);
 }
 /**
  * Get a specific location by ID
  */
 export async function getLocation(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid location ID. Please provide a positive number.');
-    }
-    return cached(`location:${id}`, () => fetchOne(`/locations/locations/${id}`), CACHE_TTL.locations);
+    validateId(id, 'location');
+    return cached(`location:${id}`, () => fetchOne(endpointWithId(ENDPOINTS.locations, id)), CACHE_TTL.locations);
 }
 // ============================================================================
 // Contract endpoints
@@ -131,13 +129,13 @@ export async function getLocation(id) {
  * List contracts, optionally filtered by employee ID
  */
 export async function listContracts(employeeId, options) {
-    if (employeeId !== undefined && employeeId <= 0) {
-        throw new Error('Invalid employee ID. Please provide a positive number.');
+    if (employeeId !== undefined) {
+        validateId(employeeId, 'employee');
     }
     const params = buildPaginationParams(options);
     // Note: The API doesn't reliably filter by employee_id query param,
     // so we fetch all contracts and filter client-side
-    const allContracts = await cached('contracts:all', () => fetchList('/contracts/contract-versions'), CACHE_TTL.contracts);
+    const allContracts = await cached('contracts:all', () => fetchList(ENDPOINTS.contracts), CACHE_TTL.contracts);
     const filtered = employeeId !== undefined
         ? allContracts.filter(c => c.employee_id === employeeId)
         : allContracts;
@@ -163,32 +161,28 @@ export async function listLeaves(options) {
         queryParams.start_on_gte = options.start_on_gte;
     if (options?.start_on_lte)
         queryParams.start_on_lte = options.start_on_lte;
-    const leaves = await fetchList('/timeoff/leaves', { params: queryParams });
+    const leaves = await fetchList(ENDPOINTS.leaves, { params: queryParams });
     return paginateResponse(leaves, params.page, params.limit);
 }
 /**
  * Get a specific leave by ID
  */
 export async function getLeave(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid leave ID. Please provide a positive number.');
-    }
-    return fetchOne(`/timeoff/leaves/${id}`);
+    validateId(id, 'leave');
+    return fetchOne(endpointWithId(ENDPOINTS.leaves, id));
 }
 /**
  * List all leave types
  */
 export async function listLeaveTypes() {
-    return cached('leave-types:all', () => fetchList('/timeoff/leave-types'), CACHE_TTL.leaves);
+    return cached('leave-types:all', () => fetchList(ENDPOINTS.leaveTypes), CACHE_TTL.leaves);
 }
 /**
  * Get a specific leave type by ID
  */
 export async function getLeaveType(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid leave type ID. Please provide a positive number.');
-    }
-    return fetchOne(`/timeoff/leave-types/${id}`);
+    validateId(id, 'leave type');
+    return fetchOne(endpointWithId(ENDPOINTS.leaveTypes, id));
 }
 /**
  * List allowances with optional filtering by employee
@@ -201,7 +195,7 @@ export async function listAllowances(options) {
     };
     if (options?.employee_id)
         queryParams.employee_id = options.employee_id;
-    const allowances = await fetchList('/timeoff/allowances', { params: queryParams });
+    const allowances = await fetchList(ENDPOINTS.allowances, { params: queryParams });
     return paginateResponse(allowances, params.page, params.limit);
 }
 // ============================================================================
@@ -222,17 +216,15 @@ export async function listShifts(options) {
         queryParams.clock_in_gte = options.clock_in_gte;
     if (options?.clock_in_lte)
         queryParams.clock_in_lte = options.clock_in_lte;
-    const shifts = await fetchList('/attendance/shifts', { params: queryParams });
+    const shifts = await fetchList(ENDPOINTS.shifts, { params: queryParams });
     return paginateResponse(shifts, params.page, params.limit);
 }
 /**
  * Get a specific shift by ID
  */
 export async function getShift(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid shift ID. Please provide a positive number.');
-    }
-    return fetchOne(`/attendance/shifts/${id}`);
+    validateId(id, 'shift');
+    return fetchOne(endpointWithId(ENDPOINTS.shifts, id));
 }
 // ============================================================================
 // Document endpoints (Read-only)
@@ -241,16 +233,14 @@ export async function getShift(id) {
  * List all folders
  */
 export async function listFolders() {
-    return cached('folders:all', () => fetchList('/documents/folders'), CACHE_TTL.default);
+    return cached('folders:all', () => fetchList(ENDPOINTS.folders), CACHE_TTL.default);
 }
 /**
  * Get a specific folder by ID
  */
 export async function getFolder(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid folder ID. Please provide a positive number.');
-    }
-    return fetchOne(`/documents/folders/${id}`);
+    validateId(id, 'folder');
+    return fetchOne(endpointWithId(ENDPOINTS.folders, id));
 }
 /**
  * List documents with optional filtering by folder
@@ -271,20 +261,56 @@ export async function listDocuments(options) {
         const baseParams = new URLSearchParams(queryParams).toString();
         const fullParams = baseParams ? `${baseParams}&${employeeIdsParam}` : employeeIdsParam;
         // Make request with custom query string
-        const documents = await fetchList(`/documents/documents?${fullParams}`);
+        const documents = await fetchList(`${ENDPOINTS.documents}?${fullParams}`);
+        debug(`listDocuments returned ${documents.length} documents`, {
+            sampleDocument: documents[0],
+            missingNames: documents.filter(d => !d.name).length,
+        });
         return paginateResponse(documents, params.page, params.limit);
     }
-    const documents = await fetchList('/documents/documents', { params: queryParams });
+    const documents = await fetchList(ENDPOINTS.documents, { params: queryParams });
+    debug(`listDocuments returned ${documents.length} documents`, {
+        sampleDocument: documents[0],
+        missingNames: documents.filter(d => !d.name).length,
+    });
     return paginateResponse(documents, params.page, params.limit);
 }
 /**
  * Get a specific document by ID
+ *
+ * Note: The Factorial API's individual document endpoint (/documents/documents/{id})
+ * can be unreliable, similar to the employee endpoint. This function implements a fallback
+ * to listing all documents and filtering if the direct endpoint fails or returns no data.
+ *
+ * @param id - The document ID
+ * @returns The document object
+ * @throws Error if document is not found
  */
 export async function getDocument(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid document ID. Please provide a positive number.');
+    validateId(id, 'document');
+    // Try the direct endpoint first
+    try {
+        const document = await fetchOne(endpointWithId(ENDPOINTS.documents, id));
+        // If we got a valid document, return it
+        if (document) {
+            return document;
+        }
+    }
+    catch (error) {
+        // If direct fetch fails with NotFoundError, try fallback
+        if (!(error instanceof NotFoundError)) {
+            throw error;
+        }
+        debug(`getDocument(${id}) - direct endpoint failed, using fallback`);
     }
-    return fetchOne(`/documents/documents/${id}`);
+    // Fallback: Fetch all documents and filter by ID
+    // This works around Factorial API limitations with the individual document endpoint
+    const allDocuments = await fetchList(ENDPOINTS.documents);
+    const document = allDocuments.find(doc => doc.id === id);
+    if (!document) {
+        throw new Error(`Document with ID ${id} not found.`);
+    }
+    return document;
 }
 // ============================================================================
 // Job Catalog endpoints
@@ -293,31 +319,27 @@ export async function getDocument(id) {
  * List all job roles
  */
 export async function listJobRoles() {
-    return cached('job-roles:all', () => fetchList('/job_catalog/roles'), CACHE_TTL.default);
+    return cached('job-roles:all', () => fetchList(ENDPOINTS.jobRoles), CACHE_TTL.default);
 }
 /**
  * Get a specific job role by ID
  */
 export async function getJobRole(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid job role ID. Please provide a positive number.');
-    }
-    return fetchOne(`/job_catalog/roles/${id}`);
+    validateId(id, 'job role');
+    return fetchOne(endpointWithId(ENDPOINTS.jobRoles, id));
 }
 /**
  * List all job levels
  */
 export async function listJobLevels() {
-    return cached('job-levels:all', () => fetchList('/job_catalog/levels'), CACHE_TTL.default);
+    return cached('job-levels:all', () => fetchList(ENDPOINTS.jobLevels), CACHE_TTL.default);
 }
 /**
  * Get a specific job level by ID
  */
 export async function getJobLevel(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid job level ID. Please provide a positive number.');
-    }
-    return fetchOne(`/job_catalog/levels/${id}`);
+    validateId(id, 'job level');
+    return fetchOne(endpointWithId(ENDPOINTS.jobLevels, id));
 }
 // ============================================================================
 // Cache utilities
@@ -342,7 +364,7 @@ export function invalidateCache(resourceType) {
  */
 export async function createEmployee(input) {
     return auditedOperation(AuditAction.CREATE, 'employee', undefined, async () => {
-        const employee = await postOne('/employees/employees', input);
+        const employee = await postOne(ENDPOINTS.employees, input);
         cache.invalidatePrefix('employees');
         return employee;
     }, Object.fromEntries(Object.entries(input).map(([k, v]) => [k, { to: v }])));
@@ -351,11 +373,9 @@ export async function createEmployee(input) {
  * Update an existing employee
  */
 export async function updateEmployee(id, input) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid employee ID. Please provide a positive number.');
-    }
+    validateId(id, 'employee');
     return auditedOperation(AuditAction.UPDATE, 'employee', id, async () => {
-        const employee = await patchOne(`/employees/employees/${id}`, input);
+        const employee = await patchOne(endpointWithId(ENDPOINTS.employees, id), input);
         cache.invalidate(`employee:${id}`);
         cache.invalidatePrefix('employees');
         return employee;
@@ -365,11 +385,9 @@ export async function updateEmployee(id, input) {
  * Terminate an employee (soft delete)
  */
 export async function terminateEmployee(id, terminatedOn, reason) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid employee ID. Please provide a positive number.');
-    }
+    validateId(id, 'employee');
     return auditedOperation(AuditAction.TERMINATE, 'employee', id, async () => {
-        const employee = await patchOne(`/employees/employees/${id}`, {
+        const employee = await patchOne(endpointWithId(ENDPOINTS.employees, id), {
             terminated_on: terminatedOn,
         });
         cache.invalidate(`employee:${id}`);
@@ -385,7 +403,7 @@ export async function terminateEmployee(id, terminatedOn, reason) {
  */
 export async function createTeam(input) {
     return auditedOperation(AuditAction.CREATE, 'team', undefined, async () => {
-        const team = await postOne('/teams/teams', input);
+        const team = await postOne(ENDPOINTS.teams, input);
         cache.invalidatePrefix('teams');
         return team;
     });
@@ -394,11 +412,9 @@ export async function createTeam(input) {
  * Update an existing team
  */
 export async function updateTeam(id, input) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid team ID. Please provide a positive number.');
-    }
+    validateId(id, 'team');
     return auditedOperation(AuditAction.UPDATE, 'team', id, async () => {
-        const team = await patchOne(`/teams/teams/${id}`, input);
+        const team = await patchOne(endpointWithId(ENDPOINTS.teams, id), input);
         cache.invalidate(`team:${id}`);
         cache.invalidatePrefix('teams');
         return team;
@@ -408,11 +424,9 @@ export async function updateTeam(id, input) {
  * Delete a team
  */
 export async function deleteTeam(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid team ID. Please provide a positive number.');
-    }
+    validateId(id, 'team');
     return auditedOperation(AuditAction.DELETE, 'team', id, async () => {
-        await deleteOne(`/teams/teams/${id}`);
+        await deleteOne(endpointWithId(ENDPOINTS.teams, id));
         cache.invalidate(`team:${id}`);
         cache.invalidatePrefix('teams');
     });
@@ -425,7 +439,7 @@ export async function deleteTeam(id) {
  */
 export async function createLocation(input) {
     return auditedOperation(AuditAction.CREATE, 'location', undefined, async () => {
-        const location = await postOne('/locations/locations', input);
+        const location = await postOne(ENDPOINTS.locations, input);
         cache.invalidatePrefix('locations');
         return location;
     });
@@ -434,11 +448,9 @@ export async function createLocation(input) {
  * Update an existing location
  */
 export async function updateLocation(id, input) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid location ID. Please provide a positive number.');
-    }
+    validateId(id, 'location');
     return auditedOperation(AuditAction.UPDATE, 'location', id, async () => {
-        const location = await patchOne(`/locations/locations/${id}`, input);
+        const location = await patchOne(endpointWithId(ENDPOINTS.locations, id), input);
         cache.invalidate(`location:${id}`);
         cache.invalidatePrefix('locations');
         return location;
@@ -448,11 +460,9 @@ export async function updateLocation(id, input) {
  * Delete a location
  */
 export async function deleteLocation(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid location ID. Please provide a positive number.');
-    }
+    validateId(id, 'location');
     return auditedOperation(AuditAction.DELETE, 'location', id, async () => {
-        await deleteOne(`/locations/locations/${id}`);
+        await deleteOne(endpointWithId(ENDPOINTS.locations, id));
         cache.invalidate(`location:${id}`);
         cache.invalidatePrefix('locations');
     });
@@ -465,7 +475,7 @@ export async function deleteLocation(id) {
  */
 export async function createLeave(input) {
     return auditedOperation(AuditAction.CREATE, 'leave', undefined, async () => {
-        const leave = await postOne('/timeoff/leaves', input);
+        const leave = await postOne(ENDPOINTS.leaves, input);
         return leave;
     });
 }
@@ -473,11 +483,9 @@ export async function createLeave(input) {
  * Update a leave request
  */
 export async function updateLeave(id, input) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid leave ID. Please provide a positive number.');
-    }
+    validateId(id, 'leave');
     return auditedOperation(AuditAction.UPDATE, 'leave', id, async () => {
-        const leave = await patchOne(`/timeoff/leaves/${id}`, input);
+        const leave = await patchOne(endpointWithId(ENDPOINTS.leaves, id), input);
         return leave;
     });
 }
@@ -485,22 +493,18 @@ export async function updateLeave(id, input) {
  * Cancel a leave request
  */
 export async function cancelLeave(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid leave ID. Please provide a positive number.');
-    }
+    validateId(id, 'leave');
     return auditedOperation(AuditAction.DELETE, 'leave', id, async () => {
-        await deleteOne(`/timeoff/leaves/${id}`);
+        await deleteOne(endpointWithId(ENDPOINTS.leaves, id));
     });
 }
 /**
  * Approve a leave request
  */
 export async function approveLeave(id, input) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid leave ID. Please provide a positive number.');
-    }
+    validateId(id, 'leave');
     return auditedOperation(AuditAction.APPROVE, 'leave', id, async () => {
-        const leave = await postAction(`/timeoff/leaves/${id}/approve`, input || {});
+        const leave = await postAction(endpointWithAction(ENDPOINTS.leaves, id, 'approve'), input || {});
         return leave;
     });
 }
@@ -508,11 +512,9 @@ export async function approveLeave(id, input) {
  * Reject a leave request
  */
 export async function rejectLeave(id, input) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid leave ID. Please provide a positive number.');
-    }
+    validateId(id, 'leave');
     return auditedOperation(AuditAction.REJECT, 'leave', id, async () => {
-        const leave = await postAction(`/timeoff/leaves/${id}/reject`, input || {});
+        const leave = await postAction(endpointWithAction(ENDPOINTS.leaves, id, 'reject'), input || {});
         return leave;
     });
 }
@@ -524,7 +526,7 @@ export async function rejectLeave(id, input) {
  */
 export async function createShift(input) {
     return auditedOperation(AuditAction.CREATE, 'shift', undefined, async () => {
-        const shift = await postOne('/attendance/shifts', input);
+        const shift = await postOne(ENDPOINTS.shifts, input);
         return shift;
     });
 }
@@ -532,11 +534,9 @@ export async function createShift(input) {
  * Update a shift
  */
 export async function updateShift(id, input) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid shift ID. Please provide a positive number.');
-    }
+    validateId(id, 'shift');
     return auditedOperation(AuditAction.UPDATE, 'shift', id, async () => {
-        const shift = await patchOne(`/attendance/shifts/${id}`, input);
+        const shift = await patchOne(endpointWithId(ENDPOINTS.shifts, id), input);
         return shift;
     });
 }
@@ -544,11 +544,9 @@ export async function updateShift(id, input) {
  * Delete a shift
  */
 export async function deleteShift(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid shift ID. Please provide a positive number.');
-    }
+    validateId(id, 'shift');
     return auditedOperation(AuditAction.DELETE, 'shift', id, async () => {
-        await deleteOne(`/attendance/shifts/${id}`);
+        await deleteOne(endpointWithId(ENDPOINTS.shifts, id));
     });
 }
 // ============================================================================
@@ -559,17 +557,15 @@ export async function deleteShift(id) {
  */
 export async function listProjects(options) {
     const params = buildPaginationParams(options);
-    const projects = await cached(CacheManager.key('projects', options), () => fetchList('/project_management/projects'), CACHE_TTL.default);
+    const projects = await cached(CacheManager.key('projects', options), () => fetchList(ENDPOINTS.projects), CACHE_TTL.default);
     return sliceForPagination(projects, params);
 }
 /**
  * Get a specific project by ID
  */
 export async function getProject(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid project ID. Please provide a positive number.');
-    }
-    return cached(`project:${id}`, () => fetchOne(`/project_management/projects/${id}`), CACHE_TTL.default);
+    validateId(id, 'project');
+    return cached(`project:${id}`, () => fetchOne(endpointWithId(ENDPOINTS.projects, id)), CACHE_TTL.default);
 }
 /**
  * List project tasks
@@ -579,7 +575,7 @@ export async function listProjectTasks(projectId, options) {
     const queryParams = {};
     if (projectId)
         queryParams.project_ids = projectId;
-    const tasks = await fetchList('/project_management/project_tasks', {
+    const tasks = await fetchList(ENDPOINTS.projectTasks, {
         params: queryParams,
     });
     return sliceForPagination(tasks, params);
@@ -588,10 +584,8 @@ export async function listProjectTasks(projectId, options) {
  * Get a specific project task by ID
  */
 export async function getProjectTask(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid task ID. Please provide a positive number.');
-    }
-    return fetchOne(`/project_management/project_tasks/${id}`);
+    validateId(id, 'task');
+    return fetchOne(endpointWithId(ENDPOINTS.projectTasks, id));
 }
 /**
  * List project workers
@@ -601,7 +595,7 @@ export async function listProjectWorkers(projectId, options) {
     const queryParams = {};
     if (projectId)
         queryParams.project_ids = projectId;
-    const workers = await fetchList('/project_management/project_workers', {
+    const workers = await fetchList(ENDPOINTS.projectWorkers, {
         params: queryParams,
     });
     return sliceForPagination(workers, params);
@@ -610,10 +604,8 @@ export async function listProjectWorkers(projectId, options) {
  * Get a specific project worker by ID
  */
 export async function getProjectWorker(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid project worker ID. Please provide a positive number.');
-    }
-    return fetchOne(`/project_management/project_workers/${id}`);
+    validateId(id, 'project worker');
+    return fetchOne(endpointWithId(ENDPOINTS.projectWorkers, id));
 }
 /**
  * List time records
@@ -623,7 +615,7 @@ export async function listTimeRecords(projectWorkerId, options) {
     const queryParams = {};
     if (projectWorkerId)
         queryParams.project_workers_ids = projectWorkerId;
-    const records = await fetchList('/project_management/time_records', {
+    const records = await fetchList(ENDPOINTS.timeRecords, {
         params: queryParams,
     });
     return sliceForPagination(records, params);
@@ -632,10 +624,8 @@ export async function listTimeRecords(projectWorkerId, options) {
  * Get a specific time record by ID
  */
 export async function getTimeRecord(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid time record ID. Please provide a positive number.');
-    }
-    return fetchOne(`/project_management/time_records/${id}`);
+    validateId(id, 'time record');
+    return fetchOne(endpointWithId(ENDPOINTS.timeRecords, id));
 }
 // ============================================================================
 // Projects & Time Tracking - WRITE endpoints
@@ -645,7 +635,7 @@ export async function getTimeRecord(id) {
  */
 export async function createProject(input) {
     return auditedOperation(AuditAction.CREATE, 'project', undefined, async () => {
-        const project = await postOne('/project_management/projects', input);
+        const project = await postOne(ENDPOINTS.projects, input);
         cache.invalidatePrefix('projects');
         return project;
     });
@@ -654,11 +644,9 @@ export async function createProject(input) {
  * Update a project
  */
 export async function updateProject(id, input) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid project ID. Please provide a positive number.');
-    }
+    validateId(id, 'project');
     return auditedOperation(AuditAction.UPDATE, 'project', id, async () => {
-        const project = await patchOne(`/project_management/projects/${id}`, input);
+        const project = await patchOne(endpointWithId(ENDPOINTS.projects, id), input);
         cache.invalidate(`project:${id}`);
         cache.invalidatePrefix('projects');
         return project;
@@ -668,11 +656,9 @@ export async function updateProject(id, input) {
  * Delete a project
  */
 export async function deleteProject(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid project ID. Please provide a positive number.');
-    }
+    validateId(id, 'project');
     return auditedOperation(AuditAction.DELETE, 'project', id, async () => {
-        await deleteOne(`/project_management/projects/${id}`);
+        await deleteOne(endpointWithId(ENDPOINTS.projects, id));
         cache.invalidate(`project:${id}`);
         cache.invalidatePrefix('projects');
     });
@@ -682,29 +668,25 @@ export async function deleteProject(id) {
  */
 export async function createProjectTask(input) {
     return auditedOperation(AuditAction.CREATE, 'project_task', undefined, async () => {
-        return postOne('/project_management/project_tasks', input);
+        return postOne(ENDPOINTS.projectTasks, input);
     });
 }
 /**
  * Update a project task
  */
 export async function updateProjectTask(id, input) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid task ID. Please provide a positive number.');
-    }
+    validateId(id, 'task');
     return auditedOperation(AuditAction.UPDATE, 'project_task', id, async () => {
-        return patchOne(`/project_management/project_tasks/${id}`, input);
+        return patchOne(endpointWithId(ENDPOINTS.projectTasks, id), input);
     });
 }
 /**
  * Delete a project task
  */
 export async function deleteProjectTask(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid task ID. Please provide a positive number.');
-    }
+    validateId(id, 'task');
     return auditedOperation(AuditAction.DELETE, 'project_task', id, async () => {
-        await deleteOne(`/project_management/project_tasks/${id}`);
+        await deleteOne(endpointWithId(ENDPOINTS.projectTasks, id));
     });
 }
 /**
@@ -712,18 +694,16 @@ export async function deleteProjectTask(id) {
  */
 export async function assignProjectWorker(input) {
     return auditedOperation(AuditAction.ASSIGN, 'project_worker', undefined, async () => {
-        return postOne('/project_management/project_workers', input);
+        return postOne(ENDPOINTS.projectWorkers, input);
     });
 }
 /**
  * Remove a worker from a project
  */
 export async function removeProjectWorker(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid project worker ID. Please provide a positive number.');
-    }
+    validateId(id, 'project worker');
     return auditedOperation(AuditAction.UNASSIGN, 'project_worker', id, async () => {
-        await deleteOne(`/project_management/project_workers/${id}`);
+        await deleteOne(endpointWithId(ENDPOINTS.projectWorkers, id));
     });
 }
 /**
@@ -731,29 +711,25 @@ export async function removeProjectWorker(id) {
  */
 export async function createTimeRecord(input) {
     return auditedOperation(AuditAction.CREATE, 'time_record', undefined, async () => {
-        return postOne('/project_management/time_records', input);
+        return postOne(ENDPOINTS.timeRecords, input);
     });
 }
 /**
  * Update a time record
  */
 export async function updateTimeRecord(id, input) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid time record ID. Please provide a positive number.');
-    }
+    validateId(id, 'time record');
     return auditedOperation(AuditAction.UPDATE, 'time_record', id, async () => {
-        return patchOne(`/project_management/time_records/${id}`, input);
+        return patchOne(endpointWithId(ENDPOINTS.timeRecords, id), input);
     });
 }
 /**
  * Delete a time record
  */
 export async function deleteTimeRecord(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid time record ID. Please provide a positive number.');
-    }
+    validateId(id, 'time record');
     return auditedOperation(AuditAction.DELETE, 'time_record', id, async () => {
-        await deleteOne(`/project_management/time_records/${id}`);
+        await deleteOne(endpointWithId(ENDPOINTS.timeRecords, id));
     });
 }
 // ============================================================================
@@ -764,17 +740,15 @@ export async function deleteTimeRecord(id) {
  */
 export async function listTrainings(options) {
     const params = buildPaginationParams(options);
-    const trainings = await cached(CacheManager.key('trainings', options), () => fetchList('/trainings/trainings'), CACHE_TTL.default);
+    const trainings = await cached(CacheManager.key('trainings', options), () => fetchList(ENDPOINTS.trainings), CACHE_TTL.default);
     return sliceForPagination(trainings, params);
 }
 /**
  * Get a specific training by ID
  */
 export async function getTraining(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid training ID. Please provide a positive number.');
-    }
-    return cached(`training:${id}`, () => fetchOne(`/trainings/trainings/${id}`), CACHE_TTL.default);
+    validateId(id, 'training');
+    return cached(`training:${id}`, () => fetchOne(endpointWithId(ENDPOINTS.trainings, id)), CACHE_TTL.default);
 }
 /**
  * List training sessions
@@ -784,17 +758,17 @@ export async function listTrainingSessions(trainingId, options) {
     const queryParams = {};
     if (trainingId)
         queryParams.training_id = trainingId;
-    const sessions = await fetchList('/trainings/sessions', { params: queryParams });
+    const sessions = await fetchList(ENDPOINTS.trainingSessions, {
+        params: queryParams,
+    });
     return sliceForPagination(sessions, params);
 }
 /**
  * Get a specific training session by ID
  */
 export async function getTrainingSession(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid session ID. Please provide a positive number.');
-    }
-    return fetchOne(`/trainings/sessions/${id}`);
+    validateId(id, 'session');
+    return fetchOne(endpointWithId(ENDPOINTS.trainingSessions, id));
 }
 /**
  * List training enrollments
@@ -804,7 +778,7 @@ export async function listTrainingEnrollments(trainingId, options) {
     const queryParams = {};
     if (trainingId)
         queryParams.training_id = trainingId;
-    const enrollments = await fetchList('/trainings/memberships', {
+    const enrollments = await fetchList(ENDPOINTS.trainingMemberships, {
         params: queryParams,
     });
     return sliceForPagination(enrollments, params);
@@ -813,10 +787,8 @@ export async function listTrainingEnrollments(trainingId, options) {
  * Get a specific training enrollment by ID
  */
 export async function getTrainingEnrollment(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid enrollment ID. Please provide a positive number.');
-    }
-    return fetchOne(`/trainings/memberships/${id}`);
+    validateId(id, 'enrollment');
+    return fetchOne(endpointWithId(ENDPOINTS.trainingMemberships, id));
 }
 // ============================================================================
 // Training & Development - WRITE endpoints
@@ -826,7 +798,7 @@ export async function getTrainingEnrollment(id) {
  */
 export async function createTraining(input) {
     return auditedOperation(AuditAction.CREATE, 'training', undefined, async () => {
-        const training = await postOne('/trainings/trainings', input);
+        const training = await postOne(ENDPOINTS.trainings, input);
         cache.invalidatePrefix('trainings');
         return training;
     });
@@ -835,11 +807,9 @@ export async function createTraining(input) {
  * Update a training program
  */
 export async function updateTraining(id, input) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid training ID. Please provide a positive number.');
-    }
+    validateId(id, 'training');
     return auditedOperation(AuditAction.UPDATE, 'training', id, async () => {
-        const training = await patchOne(`/trainings/trainings/${id}`, input);
+        const training = await patchOne(endpointWithId(ENDPOINTS.trainings, id), input);
         cache.invalidate(`training:${id}`);
         cache.invalidatePrefix('trainings');
         return training;
@@ -849,11 +819,9 @@ export async function updateTraining(id, input) {
  * Delete a training program
  */
 export async function deleteTraining(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid training ID. Please provide a positive number.');
-    }
+    validateId(id, 'training');
     return auditedOperation(AuditAction.DELETE, 'training', id, async () => {
-        await deleteOne(`/trainings/trainings/${id}`);
+        await deleteOne(endpointWithId(ENDPOINTS.trainings, id));
         cache.invalidate(`training:${id}`);
         cache.invalidatePrefix('trainings');
     });
@@ -863,29 +831,25 @@ export async function deleteTraining(id) {
  */
 export async function createTrainingSession(input) {
     return auditedOperation(AuditAction.CREATE, 'training_session', undefined, async () => {
-        return postOne('/trainings/sessions', input);
+        return postOne(ENDPOINTS.trainingSessions, input);
     });
 }
 /**
  * Update a training session
  */
 export async function updateTrainingSession(id, input) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid session ID. Please provide a positive number.');
-    }
+    validateId(id, 'session');
     return auditedOperation(AuditAction.UPDATE, 'training_session', id, async () => {
-        return patchOne(`/trainings/sessions/${id}`, input);
+        return patchOne(endpointWithId(ENDPOINTS.trainingSessions, id), input);
     });
 }
 /**
  * Delete a training session
  */
 export async function deleteTrainingSession(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid session ID. Please provide a positive number.');
-    }
+    validateId(id, 'session');
     return auditedOperation(AuditAction.DELETE, 'training_session', id, async () => {
-        await deleteOne(`/trainings/sessions/${id}`);
+        await deleteOne(endpointWithId(ENDPOINTS.trainingSessions, id));
     });
 }
 /**
@@ -893,18 +857,16 @@ export async function deleteTrainingSession(id) {
  */
 export async function enrollInTraining(input) {
     return auditedOperation(AuditAction.ASSIGN, 'training_enrollment', undefined, async () => {
-        return postOne('/trainings/memberships', input);
+        return postOne(ENDPOINTS.trainingMemberships, input);
     });
 }
 /**
  * Remove enrollment from a training
  */
 export async function unenrollFromTraining(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid enrollment ID. Please provide a positive number.');
-    }
+    validateId(id, 'enrollment');
     return auditedOperation(AuditAction.UNASSIGN, 'training_enrollment', id, async () => {
-        await deleteOne(`/trainings/memberships/${id}`);
+        await deleteOne(endpointWithId(ENDPOINTS.trainingMemberships, id));
     });
 }
 // ============================================================================
@@ -915,24 +877,22 @@ export async function unenrollFromTraining(id) {
  */
 export async function listWorkAreas(options) {
     const params = buildPaginationParams(options);
-    const workAreas = await cached(CacheManager.key('work_areas', options), () => fetchList('/locations/work_areas'), CACHE_TTL.locations);
+    const workAreas = await cached(CacheManager.key('work_areas', options), () => fetchList(ENDPOINTS.workAreas), CACHE_TTL.locations);
     return sliceForPagination(workAreas, params);
 }
 /**
  * Get a specific work area by ID
  */
 export async function getWorkArea(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid work area ID. Please provide a positive number.');
-    }
-    return cached(`work_area:${id}`, () => fetchOne(`/locations/work_areas/${id}`), CACHE_TTL.locations);
+    validateId(id, 'work area');
+    return cached(`work_area:${id}`, () => fetchOne(endpointWithId(ENDPOINTS.workAreas, id)), CACHE_TTL.locations);
 }
 /**
  * Create a work area
  */
 export async function createWorkArea(input) {
     return auditedOperation(AuditAction.CREATE, 'work_area', undefined, async () => {
-        const workArea = await postOne('/locations/work_areas', input);
+        const workArea = await postOne(ENDPOINTS.workAreas, input);
         cache.invalidatePrefix('work_areas');
         return workArea;
     });
@@ -941,11 +901,9 @@ export async function createWorkArea(input) {
  * Update a work area
  */
 export async function updateWorkArea(id, input) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid work area ID. Please provide a positive number.');
-    }
+    validateId(id, 'work area');
     return auditedOperation(AuditAction.UPDATE, 'work_area', id, async () => {
-        const workArea = await patchOne(`/locations/work_areas/${id}`, input);
+        const workArea = await patchOne(endpointWithId(ENDPOINTS.workAreas, id), input);
         cache.invalidate(`work_area:${id}`);
         cache.invalidatePrefix('work_areas');
         return workArea;
@@ -955,11 +913,9 @@ export async function updateWorkArea(id, input) {
  * Archive a work area
  */
 export async function archiveWorkArea(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid work area ID. Please provide a positive number.');
-    }
+    validateId(id, 'work area');
     return auditedOperation(AuditAction.ARCHIVE, 'work_area', id, async () => {
-        const workArea = await postAction(`/locations/work_areas/${id}/archive`);
+        const workArea = await postAction(endpointWithAction(ENDPOINTS.workAreas, id, 'archive'));
         cache.invalidate(`work_area:${id}`);
         cache.invalidatePrefix('work_areas');
         return workArea;
@@ -969,11 +925,9 @@ export async function archiveWorkArea(id) {
  * Unarchive a work area
  */
 export async function unarchiveWorkArea(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid work area ID. Please provide a positive number.');
-    }
+    validateId(id, 'work area');
     return auditedOperation(AuditAction.UNARCHIVE, 'work_area', id, async () => {
-        const workArea = await postAction(`/locations/work_areas/${id}/unarchive`);
+        const workArea = await postAction(endpointWithAction(ENDPOINTS.workAreas, id, 'unarchive'));
         cache.invalidate(`work_area:${id}`);
         cache.invalidatePrefix('work_areas');
         return workArea;
@@ -987,34 +941,30 @@ export async function unarchiveWorkArea(id) {
  */
 export async function listJobPostings(options) {
     const params = buildPaginationParams(options);
-    const postings = await cached(CacheManager.key('job_postings', options), () => fetchList('/ats/job_postings'), CACHE_TTL.default);
+    const postings = await cached(CacheManager.key('job_postings', options), () => fetchList(ENDPOINTS.jobPostings), CACHE_TTL.default);
     return sliceForPagination(postings, params);
 }
 /**
  * Get a specific job posting by ID
  */
 export async function getJobPosting(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid job posting ID. Please provide a positive number.');
-    }
-    return cached(`job_posting:${id}`, () => fetchOne(`/ats/job_postings/${id}`), CACHE_TTL.default);
+    validateId(id, 'job posting');
+    return cached(`job_posting:${id}`, () => fetchOne(endpointWithId(ENDPOINTS.jobPostings, id)), CACHE_TTL.default);
 }
 /**
  * List all candidates
  */
 export async function listCandidates(options) {
     const params = buildPaginationParams(options);
-    const candidates = await fetchList('/ats/candidates');
+    const candidates = await fetchList(ENDPOINTS.candidates);
     return sliceForPagination(candidates, params);
 }
 /**
  * Get a specific candidate by ID
  */
 export async function getCandidate(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid candidate ID. Please provide a positive number.');
-    }
-    return fetchOne(`/ats/candidates/${id}`);
+    validateId(id, 'candidate');
+    return fetchOne(endpointWithId(ENDPOINTS.candidates, id));
 }
 /**
  * List all applications
@@ -1024,32 +974,30 @@ export async function listApplications(jobPostingId, options) {
     const queryParams = {};
     if (jobPostingId)
         queryParams.job_posting_id = jobPostingId;
-    const applications = await fetchList('/ats/applications', { params: queryParams });
+    const applications = await fetchList(ENDPOINTS.applications, {
+        params: queryParams,
+    });
     return sliceForPagination(applications, params);
 }
 /**
  * Get a specific application by ID
  */
 export async function getApplication(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid application ID. Please provide a positive number.');
-    }
-    return fetchOne(`/ats/applications/${id}`);
+    validateId(id, 'application');
+    return fetchOne(endpointWithId(ENDPOINTS.applications, id));
 }
 /**
  * List all hiring stages
  */
 export async function listHiringStages() {
-    return cached('hiring_stages:all', () => fetchList('/ats/hiring_stages'), CACHE_TTL.default);
+    return cached('hiring_stages:all', () => fetchList(ENDPOINTS.hiringStages), CACHE_TTL.default);
 }
 /**
  * Get a specific hiring stage by ID
  */
 export async function getHiringStage(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid hiring stage ID. Please provide a positive number.');
-    }
-    return fetchOne(`/ats/hiring_stages/${id}`);
+    validateId(id, 'hiring stage');
+    return fetchOne(endpointWithId(ENDPOINTS.hiringStages, id));
 }
 // ============================================================================
 // ATS (Recruiting) - WRITE endpoints
@@ -1059,7 +1007,7 @@ export async function getHiringStage(id) {
  */
 export async function createJobPosting(input) {
     return auditedOperation(AuditAction.CREATE, 'job_posting', undefined, async () => {
-        const posting = await postOne('/ats/job_postings', input);
+        const posting = await postOne(ENDPOINTS.jobPostings, input);
         cache.invalidatePrefix('job_postings');
         return posting;
     });
@@ -1068,11 +1016,9 @@ export async function createJobPosting(input) {
  * Update a job posting
  */
 export async function updateJobPosting(id, input) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid job posting ID. Please provide a positive number.');
-    }
+    validateId(id, 'job posting');
     return auditedOperation(AuditAction.UPDATE, 'job_posting', id, async () => {
-        const posting = await patchOne(`/ats/job_postings/${id}`, input);
+        const posting = await patchOne(endpointWithId(ENDPOINTS.jobPostings, id), input);
         cache.invalidate(`job_posting:${id}`);
         cache.invalidatePrefix('job_postings');
         return posting;
@@ -1082,11 +1028,9 @@ export async function updateJobPosting(id, input) {
  * Delete a job posting
  */
 export async function deleteJobPosting(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid job posting ID. Please provide a positive number.');
-    }
+    validateId(id, 'job posting');
     return auditedOperation(AuditAction.DELETE, 'job_posting', id, async () => {
-        await deleteOne(`/ats/job_postings/${id}`);
+        await deleteOne(endpointWithId(ENDPOINTS.jobPostings, id));
         cache.invalidate(`job_posting:${id}`);
         cache.invalidatePrefix('job_postings');
     });
@@ -1096,29 +1040,25 @@ export async function deleteJobPosting(id) {
  */
 export async function createCandidate(input) {
     return auditedOperation(AuditAction.CREATE, 'candidate', undefined, async () => {
-        return postOne('/ats/candidates', input);
+        return postOne(ENDPOINTS.candidates, input);
     });
 }
 /**
  * Update a candidate
  */
 export async function updateCandidate(id, input) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid candidate ID. Please provide a positive number.');
-    }
+    validateId(id, 'candidate');
     return auditedOperation(AuditAction.UPDATE, 'candidate', id, async () => {
-        return patchOne(`/ats/candidates/${id}`, input);
+        return patchOne(endpointWithId(ENDPOINTS.candidates, id), input);
     });
 }
 /**
  * Delete a candidate
  */
 export async function deleteCandidate(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid candidate ID. Please provide a positive number.');
-    }
+    validateId(id, 'candidate');
     return auditedOperation(AuditAction.DELETE, 'candidate', id, async () => {
-        await deleteOne(`/ats/candidates/${id}`);
+        await deleteOne(endpointWithId(ENDPOINTS.candidates, id));
     });
 }
 /**
@@ -1126,40 +1066,34 @@ export async function deleteCandidate(id) {
  */
 export async function createApplication(input) {
     return auditedOperation(AuditAction.CREATE, 'application', undefined, async () => {
-        return postOne('/ats/applications', input);
+        return postOne(ENDPOINTS.applications, input);
     });
 }
 /**
  * Update an application
  */
 export async function updateApplication(id, input) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid application ID. Please provide a positive number.');
-    }
+    validateId(id, 'application');
     return auditedOperation(AuditAction.UPDATE, 'application', id, async () => {
-        return patchOne(`/ats/applications/${id}`, input);
+        return patchOne(endpointWithId(ENDPOINTS.applications, id), input);
     });
 }
 /**
  * Delete an application
  */
 export async function deleteApplication(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid application ID. Please provide a positive number.');
-    }
+    validateId(id, 'application');
     return auditedOperation(AuditAction.DELETE, 'application', id, async () => {
-        await deleteOne(`/ats/applications/${id}`);
+        await deleteOne(endpointWithId(ENDPOINTS.applications, id));
     });
 }
 /**
  * Advance an application to the next stage
  */
 export async function advanceApplication(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid application ID. Please provide a positive number.');
-    }
+    validateId(id, 'application');
     return auditedOperation(AuditAction.UPDATE, 'application', id, async () => {
-        return postAction(`/ats/applications/${id}/apply`);
+        return postAction(endpointWithAction(ENDPOINTS.applications, id, 'apply'));
     });
 }
 // ============================================================================
@@ -1173,7 +1107,7 @@ export async function listPayrollSupplements(employeeId, options) {
     const queryParams = {};
     if (employeeId)
         queryParams.employee_id = employeeId;
-    const supplements = await fetchList('/payroll/supplements', {
+    const supplements = await fetchList(ENDPOINTS.payrollSupplements, {
         params: queryParams,
     });
     return sliceForPagination(supplements, params);
@@ -1182,10 +1116,8 @@ export async function listPayrollSupplements(employeeId, options) {
  * Get a specific payroll supplement by ID
  */
 export async function getPayrollSupplement(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid supplement ID. Please provide a positive number.');
-    }
-    return fetchOne(`/payroll/supplements/${id}`);
+    validateId(id, 'supplement');
+    return fetchOne(endpointWithId(ENDPOINTS.payrollSupplements, id));
 }
 /**
  * List tax identifiers
@@ -1195,7 +1127,7 @@ export async function listTaxIdentifiers(employeeId, options) {
     const queryParams = {};
     if (employeeId)
         queryParams.employee_id = employeeId;
-    const identifiers = await fetchList('/payroll_employees/identifiers', {
+    const identifiers = await fetchList(ENDPOINTS.taxIdentifiers, {
         params: queryParams,
     });
     return sliceForPagination(identifiers, params);
@@ -1204,10 +1136,8 @@ export async function listTaxIdentifiers(employeeId, options) {
  * Get a specific tax identifier by ID
  */
 export async function getTaxIdentifier(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid tax identifier ID. Please provide a positive number.');
-    }
-    return fetchOne(`/payroll_employees/identifiers/${id}`);
+    validateId(id, 'tax identifier');
+    return fetchOne(endpointWithId(ENDPOINTS.taxIdentifiers, id));
 }
 /**
  * List family situations
@@ -1217,7 +1147,7 @@ export async function listFamilySituations(employeeId, options) {
     const queryParams = {};
     if (employeeId)
         queryParams.employee_id = employeeId;
-    const situations = await fetchList('/payroll/family_situations', {
+    const situations = await fetchList(ENDPOINTS.familySituations, {
         params: queryParams,
     });
     return sliceForPagination(situations, params);
@@ -1226,9 +1156,7 @@ export async function listFamilySituations(employeeId, options) {
  * Get a specific family situation by ID
  */
 export async function getFamilySituation(id) {
-    if (!id || id <= 0) {
-        throw new Error('Invalid family situation ID. Please provide a positive number.');
-    }
-    return fetchOne(`/payroll/family_situations/${id}`);
+    validateId(id, 'family situation');
+    return fetchOne(endpointWithId(ENDPOINTS.familySituations, id));
 }
 //# sourceMappingURL=api.js.map