@markwharton/liquidplanner 2.0.1 → 2.1.1

package/README.md

@@ -208,6 +208,8 @@ const client = new LPClient({
 });
 ```
 
+`LPConfig` extends `ClientConfig` from api-core, which provides the `baseUrl`, `onRequest`, and `retry` fields. `apiToken`, `workspaceId`, and `cache` are LP-specific.
+
 ### Cache TTLs
 
 | Cache Key | Default TTL |

package/dist/client.js

@@ -10,7 +10,7 @@ import { buildAuthHeader, hoursToMinutes, normalizeItemType, filterIs, filterIsN
 import { buildTree, getTreeAncestors } from './tree.js';
 import { parseLPErrorResponse } from './errors.js';
 import { LP_API_BASE } from './constants.js';
-import { TTLCache, batchMap, getErrorMessage, fetchWithRetry, ok, err } from '@markwharton/api-core';
+import { TTLCache, batchMap, getErrorMessage, fetchWithRetry, ok, err, resolveRetryConfig } from '@markwharton/api-core';
 /** Transform raw API item to LPItem, preserving scheduling and effort fields */
 function transformItem(raw) {
     const item = {
@@ -126,13 +126,7 @@ export class LPClient {
             treeTtl: config.cache?.treeTtl ?? 600000,
         };
         // Initialize retry config with defaults if provided
-        if (config.retry) {
-            this.retryConfig = {
-                maxRetries: config.retry.maxRetries ?? 3,
-                initialDelayMs: config.retry.initialDelayMs ?? 1000,
-                maxDelayMs: config.retry.maxDelayMs ?? 10000,
-            };
-        }
+        this.retryConfig = resolveRetryConfig(config.retry);
     }
     /**
      * Route through cache if enabled, otherwise call factory directly.
@@ -262,7 +256,7 @@ export class LPClient {
             return err(`Unexpected response: HTTP ${response.status}`, response.status);
         }
         catch (error) {
-            return err(getErrorMessage(error) || 'Connection failed');
+            return err(getErrorMessage(error, 'Connection failed'));
         }
     }
     /**

package/dist/errors.d.ts

@@ -6,6 +6,7 @@
  * - {"message":"..."} or {"error":"..."}
  * - Plain text
  */
+import { ApiError } from '@markwharton/api-core';
 /**
  * Parsed LP error response
  */
@@ -25,14 +26,12 @@ export interface LPParsedError {
 export declare function parseLPErrorResponse(errorText: string, statusCode: number): LPParsedError;
 /**
  * Custom error class for LP API errors
+ *
+ * Extends ApiError from api-core with LP-specific isDuplicate flag.
  */
-export declare class LPError extends Error {
-    /** HTTP status code */
-    status: number;
+export declare class LPError extends ApiError {
     /** Whether this is a duplicate entry error */
     isDuplicate: boolean;
-    /** Raw error response */
-    rawResponse?: string;
     constructor(message: string, status: number, options?: {
         isDuplicate?: boolean;
         rawResponse?: string;

package/dist/errors.js

@@ -6,7 +6,7 @@
  * - {"message":"..."} or {"error":"..."}
  * - Plain text
  */
-import { parseJsonErrorResponse } from '@markwharton/api-core';
+import { ApiError, parseJsonErrorResponse } from '@markwharton/api-core';
 /** Error code for duplicate entry errors */
 const LP_ERROR_CODE_DUPLICATE = 'duplicate_value';
 /**
@@ -36,14 +36,14 @@ export function parseLPErrorResponse(errorText, statusCode) {
 }
 /**
  * Custom error class for LP API errors
+ *
+ * Extends ApiError from api-core with LP-specific isDuplicate flag.
  */
-export class LPError extends Error {
+export class LPError extends ApiError {
     constructor(message, status, options) {
-        super(message);
+        super(message, status, options);
         this.name = 'LPError';
-        this.status = status;
         this.isDuplicate = options?.isDuplicate ?? false;
-        this.rawResponse = options?.rawResponse;
     }
     /**
      * Create an LPError from an API response

package/dist/index.d.ts

@@ -32,7 +32,7 @@ export { LPClient } from './client.js';
 export { resolveTaskToAssignment } from './workflows.js';
 export type { LPConfig, LPCacheConfig, LPRetryConfig, LPItemType, HierarchyItem, LPItem, LPAncestor, LPWorkspace, LPMember, LPCostCode, LPSyncResult, LPTimesheetEntry, LPTimesheetEntryWithId, LPTaskResolution, LPUpsertOptions, LPAssignmentWithContext, LPFindItemsOptions, LPTreeNode, LPWorkspaceTree, } from './types.js';
 export { ok, err, getErrorMessage } from '@markwharton/api-core';
-export type { Result, RetryConfig } from '@markwharton/api-core';
+export type { Result, RetryConfig, OnRequestCallback, ClientConfig } from '@markwharton/api-core';
 export { hoursToMinutes, normalizeItemType, buildAuthHeader, filterIs, filterIsNot, filterIn, filterGt, filterLt, filterAfter, filterBefore, joinFilters, paginatedFetch, } from './utils.js';
 export type { PaginateOptions } from './utils.js';
 export { buildTree, getTreeAncestors, getTreeHierarchyPath, findInTree, } from './tree.js';

package/dist/index.js

@@ -32,7 +32,7 @@
 export { LPClient } from './client.js';
 // Workflows
 export { resolveTaskToAssignment } from './workflows.js';
-// Re-export Result from api-core
+// Re-exported from @markwharton/api-core
 export { ok, err, getErrorMessage } from '@markwharton/api-core';
 // Utilities
 export { hoursToMinutes, normalizeItemType, buildAuthHeader, filterIs, filterIsNot, filterIn, filterGt, filterLt, filterAfter, filterBefore, joinFilters, paginatedFetch, } from './utils.js';

package/dist/types.d.ts

@@ -4,7 +4,7 @@
  * These types define the data structures used when interacting with
  * the LiquidPlanner API.
  */
-import type { Result, RetryConfig } from '@markwharton/api-core';
+import type { Result, RetryConfig, ClientConfig } from '@markwharton/api-core';
 /**
  * LiquidPlanner item types in the hierarchy
  */
@@ -199,23 +199,13 @@ export type LPRetryConfig = RetryConfig;
 /**
  * LiquidPlanner configuration for API access
  */
-export interface LPConfig {
+export interface LPConfig extends ClientConfig {
     /** Workspace ID to operate on */
     workspaceId: number;
     /** API token for authentication */
     apiToken: string;
-    /** Base URL for LP API (defaults to https://next.liquidplanner.com/api) */
-    baseUrl?: string;
-    /** Optional callback invoked on each API request (for debugging/logging) */
-    onRequest?: (info: {
-        method: string;
-        url: string;
-        description?: string;
-    }) => void;
     /** Enable caching with optional TTL overrides. Omit to disable caching. */
     cache?: LPCacheConfig;
-    /** Retry configuration for transient failures (429, 503). Omit to disable retry. */
-    retry?: LPRetryConfig;
 }
 /**
  * Result of a timesheet sync operation

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@markwharton/liquidplanner",
-  "version": "2.0.1",
+  "version": "2.1.1",
   "description": "LiquidPlanner API client for timesheet integration",
   "type": "module",
   "main": "dist/index.js",
@@ -16,7 +16,7 @@
     "clean": "rm -rf dist"
   },
   "dependencies": {
-    "@markwharton/api-core": "^1.1.0"
+    "@markwharton/api-core": "^1.2.0"
   },
   "devDependencies": {
     "@types/node": "^20.10.0",