@dlwiest/ts-tonal-client 0.2.1 → 0.3.0

package/dist/index.esm.js

@@ -1,4 +1,5 @@
 import fs from 'fs';
+import os from 'os';
 import path from 'path';
 
 class TonalClientError extends Error {
@@ -15,9 +16,11 @@ class AuthManager {
         this.idToken = '';
         this.refreshToken = '';
         this.tokenExpiresAt = 0;
-        this.isRefreshing = false;
+        this.authPromise = null;
+        this.refreshPromise = null;
         this.authUrl = 'https://tonal.auth0.com/oauth/token';
         this.clientId = 'ERCyexW-xoVG_Yy3RDe-eV4xsOnRHP6L';
+        this.authTimeout = 30000;
         this.username = username;
         this.password = password;
     }
@@ -25,6 +28,18 @@ class AuthManager {
         if (this.isTokenValid()) {
             return this.idToken;
         }
+        const activeAuth = this.authPromise ?? this.authenticateWithPassword();
+        this.authPromise = activeAuth;
+        try {
+            return await activeAuth;
+        }
+        finally {
+            if (this.authPromise === activeAuth) {
+                this.authPromise = null;
+            }
+        }
+    }
+    async authenticateWithPassword() {
         const response = await fetch(this.authUrl, {
             method: 'POST',
             headers: {
@@ -37,6 +52,7 @@ class AuthManager {
                 grant_type: 'password',
                 scope: 'offline_access',
             }),
+            signal: AbortSignal.timeout(this.authTimeout),
         });
         if (!response.ok) {
             const errorText = await response.text();
@@ -51,69 +67,69 @@ class AuthManager {
         }
         const tokenData = await response.json();
         this.idToken = tokenData.id_token;
-        this.refreshToken = tokenData.refresh_token;
+        this.refreshToken = tokenData.refresh_token ?? '';
         this.tokenExpiresAt = Date.now() + (tokenData.expires_in * 1000);
         return this.idToken;
     }
+    invalidateToken() {
+        this.tokenExpiresAt = 0;
+    }
     async getValidToken() {
         if (this.isTokenValid()) {
             return this.idToken;
         }
-        if (this.refreshToken) {
-            if (!this.isRefreshing) {
-                await this.refreshTokens();
-            }
-            else {
-                // Wait for ongoing refresh to complete
-                while (this.isRefreshing) {
-                    await new Promise(resolve => setTimeout(resolve, 100));
-                }
-            }
-            if (this.isTokenValid()) {
-                return this.idToken;
+        if (!this.refreshToken) {
+            return this.authenticate();
+        }
+        const activeRefresh = this.refreshPromise ?? this.refreshTokens();
+        this.refreshPromise = activeRefresh;
+        try {
+            await activeRefresh;
+        }
+        catch {
+            return this.authenticate();
+        }
+        finally {
+            if (this.refreshPromise === activeRefresh) {
+                this.refreshPromise = null;
             }
         }
-        throw new TonalClientError('Token expired and refresh failed. Call authenticate() first.');
+        if (this.isTokenValid()) {
+            return this.idToken;
+        }
+        return this.authenticate();
     }
     isTokenValid() {
         return !!this.idToken && Date.now() < this.tokenExpiresAt - 60000; // 1 minute buffer
     }
     async refreshTokens() {
-        if (this.isRefreshing) {
-            return; // Prevent concurrent refresh attempts
-        }
-        this.isRefreshing = true;
-        try {
-            const response = await fetch(this.authUrl, {
-                method: 'POST',
-                headers: {
-                    'Content-Type': 'application/json',
-                },
-                body: JSON.stringify({
-                    client_id: this.clientId,
-                    grant_type: 'refresh_token',
-                    refresh_token: this.refreshToken,
-                }),
-            });
-            if (!response.ok) {
-                const errorText = await response.text();
-                let errorData;
-                try {
-                    errorData = JSON.parse(errorText);
-                }
-                catch {
-                    errorData = { error: errorText };
-                }
-                throw new TonalClientError(errorData.error_description || errorData.error || 'Token refresh failed', response.status, errorData);
+        const response = await fetch(this.authUrl, {
+            method: 'POST',
+            headers: {
+                'Content-Type': 'application/json',
+            },
+            body: JSON.stringify({
+                client_id: this.clientId,
+                grant_type: 'refresh_token',
+                refresh_token: this.refreshToken,
+            }),
+            signal: AbortSignal.timeout(this.authTimeout),
+        });
+        if (!response.ok) {
+            const errorText = await response.text();
+            let errorData;
+            try {
+                errorData = JSON.parse(errorText);
             }
-            const tokenData = await response.json();
-            this.idToken = tokenData.id_token;
-            this.refreshToken = tokenData.refresh_token;
-            this.tokenExpiresAt = Date.now() + (tokenData.expires_in * 1000);
-        }
-        finally {
-            this.isRefreshing = false;
+            catch {
+                errorData = { error: errorText };
+            }
+            throw new TonalClientError(errorData.error_description || errorData.error || 'Token refresh failed', response.status, errorData);
         }
+        const tokenData = await response.json();
+        this.idToken = tokenData.id_token;
+        this.refreshToken = tokenData.refresh_token ?? this.refreshToken;
+        this.tokenExpiresAt = Date.now() + (tokenData.expires_in * 1000);
     }
 }
 
@@ -129,10 +145,11 @@ class HttpClient {
         return this.makeRequestWithRetry(url, options, expectsBody);
     }
     async makeRequest(url, options = {}, expectsBody = true) {
-        const controller = new AbortController();
-        const timeoutId = setTimeout(() => controller.abort(), this.requestTimeout);
+        let timeoutId;
         try {
             const token = await this.authManager.getValidToken();
+            const controller = new AbortController();
+            timeoutId = setTimeout(() => controller.abort(), this.requestTimeout);
             const response = await fetch(url, {
                 ...options,
                 headers: {
@@ -143,6 +160,7 @@ class HttpClient {
                 signal: controller.signal,
             });
             clearTimeout(timeoutId);
+            timeoutId = undefined;
             if (!response.ok) {
                 const errorText = await response.text();
                 let errorData;
@@ -169,28 +187,35 @@ class HttpClient {
             if (error instanceof Error && error.name === 'AbortError') {
                 throw new TonalClientError('Request timeout', undefined, error);
             }
-            throw new TonalClientError('Request failed', undefined, error);
+            throw new TonalClientError(error instanceof Error ? error.message : 'Request failed', undefined, error);
         }
     }
     async makeRequestWithRetry(url, options = {}, expectsBody = true) {
         let lastError;
+        const canRetry = (options.method ?? 'GET').toUpperCase() === 'GET';
         for (let attempt = 1; attempt <= this.maxRetries; attempt++) {
             try {
                 return await this.makeRequest(url, options, expectsBody);
             }
             catch (error) {
                 lastError = error instanceof TonalClientError ? error : new TonalClientError('Unknown error', undefined, error);
-                // If it's an auth error on first attempt, try to refresh token and retry once
-                if (attempt === 1 && lastError.statusCode && (lastError.statusCode === 401 || lastError.statusCode === 403)) {
+                // If it's an auth error on first attempt, invalidate the token and retry once
+                if (attempt === 1 && (lastError.statusCode === 401 || lastError.statusCode === 403)) {
                     try {
-                        await this.authManager.getValidToken(); // This will refresh if needed
-                        continue; // Retry the request with the new token
+                        this.authManager.invalidateToken();
+                        await this.authManager.getValidToken();
+                        continue;
                     }
-                    catch (refreshError) {
+                    catch {
                         // If refresh fails, continue with normal retry logic
                     }
                 }
-                if (attempt === this.maxRetries || (lastError.statusCode && lastError.statusCode < 500)) {
+                const isAbort = lastError.originalError instanceof Error &&
+                    lastError.originalError.name === 'AbortError';
+                if (attempt === this.maxRetries ||
+                    !canRetry ||
+                    (lastError.statusCode !== undefined && lastError.statusCode < 500) ||
+                    (lastError.statusCode === undefined && !isAbort)) {
                     throw lastError;
                 }
                 const delay = Math.pow(2, attempt - 1) * 1000;
@@ -219,6 +244,9 @@ class WorkoutService {
     }
     async getDailyLifts(userInfo, timeZone) {
         const device = userInfo.recentMobileDevice;
+        if (!device) {
+            throw new TonalClientError('Recent mobile device information is unavailable; daily lifts require a mobile device');
+        }
         const userAgent = device.platform === 'ios'
             ? `Tonal/3004226 CFNetwork/3860.100.1 Darwin/${device.osVersion}`
             : `Tonal/${device.appVersion}`;
@@ -255,8 +283,8 @@ class WorkoutService {
         if (!shareUrl?.trim()) {
             throw new TonalClientError('Share URL is required');
         }
-        const urlPattern = /https:\/\/share\.tonal\.com\/workout\/([a-f0-9-]+)/;
-        const match = shareUrl.match(urlPattern);
+        const urlPattern = /^https:\/\/share\.tonal\.com\/workout\/([a-fA-F0-9-]+)(?:[?#]\S*)?$/;
+        const match = shareUrl.trim().match(urlPattern);
         if (!match) {
             throw new TonalClientError('Invalid share URL format. Expected: https://share.tonal.com/workout/{id}');
         }
@@ -301,9 +329,16 @@ class WorkoutService {
         if (!workoutData.sets?.length) {
             throw new TonalClientError('At least one set is required');
         }
+        // Tonal's PUT behavior is unverified. Omitting an absent shortDescription is equivalent
+        // under full-replace semantics and avoids an unintended clear under merge semantics;
+        // an explicit empty string intentionally clears it. `!= null` also covers null, which
+        // sibling response fields are known to return despite their non-optional types.
         const requestBody = {
             id: workoutData.id,
             title: workoutData.title,
+            ...(workoutData.shortDescription != null
+                ? { shortDescription: workoutData.shortDescription }
+                : {}),
             description: workoutData.description || '',
             coachId: workoutData.coachId || '00000000-0000-0000-0000-000000000000',
             sets: workoutData.sets,
@@ -327,10 +362,11 @@ class WorkoutService {
 }
 
 class CacheManager {
-    constructor(cacheDir = '.cache', defaultTTL = 24 * 60 * 60 * 1000) {
+    constructor(cacheDir = process.env.XDG_CACHE_HOME
+        ? path.join(process.env.XDG_CACHE_HOME, 'ts-tonal-client')
+        : path.join(os.homedir(), '.cache', 'ts-tonal-client'), defaultTTL = 24 * 60 * 60 * 1000) {
         this.cacheDir = cacheDir;
         this.defaultTTL = defaultTTL;
-        this.ensureCacheDir();
     }
     ensureCacheDir() {
         if (!fs.existsSync(this.cacheDir)) {
@@ -351,8 +387,7 @@ class CacheManager {
             const cachedAt = new Date(entry.cachedAt).getTime();
             const now = Date.now();
             const age = now - cachedAt;
-            if (age > entry.ttl) {
-                // Cache expired
+            if (!Number.isFinite(cachedAt) || typeof entry.ttl !== 'number' || age > entry.ttl) {
                 return null;
             }
             return entry.data;
@@ -364,12 +399,15 @@ class CacheManager {
     }
     async set(key, data, ttl) {
         const cachePath = this.getCachePath(key);
+        this.ensureCacheDir();
         const entry = {
             cachedAt: new Date().toISOString(),
             ttl: ttl || this.defaultTTL,
             data,
         };
-        fs.writeFileSync(cachePath, JSON.stringify(entry, null, 2), 'utf-8');
+        const tempPath = `${cachePath}.tmp`;
+        fs.writeFileSync(tempPath, JSON.stringify(entry, null, 2), 'utf-8');
+        fs.renameSync(tempPath, cachePath);
     }
     async invalidate(key) {
         const cachePath = this.getCachePath(key);
@@ -379,7 +417,7 @@ class CacheManager {
     }
     async clear() {
         if (fs.existsSync(this.cacheDir)) {
-            const files = fs.readdirSync(this.cacheDir);
+            const files = fs.readdirSync(this.cacheDir).filter(file => file.endsWith('.json') || file.endsWith('.json.tmp'));
             for (const file of files) {
                 fs.unlinkSync(path.join(this.cacheDir, file));
             }
@@ -388,9 +426,9 @@ class CacheManager {
 }
 
 class MovementService {
-    constructor(httpClient) {
+    constructor(httpClient, cacheDir) {
         this.httpClient = httpClient;
-        this.cacheManager = new CacheManager();
+        this.cacheManager = new CacheManager(cacheDir);
     }
     async getMovements(useCache = true) {
         if (useCache) {
@@ -400,7 +438,12 @@ class MovementService {
             }
         }
         const movements = await this.httpClient.request('/movements');
-        await this.cacheManager.set('movements', movements);
+        try {
+            await this.cacheManager.set('movements', movements);
+        }
+        catch {
+            // Cache writes are best-effort and must not hide a successful API response.
+        }
         return movements;
     }
     async invalidateMovementsCache() {
@@ -504,15 +547,15 @@ class UserService {
 }
 
 class TonalClient {
-    constructor(username, password) {
+    constructor(username, password, cacheDir) {
         this.authManager = new AuthManager(username, password);
         this.httpClient = new HttpClient(this.authManager);
         this.workoutService = new WorkoutService(this.httpClient);
-        this.movementService = new MovementService(this.httpClient);
+        this.movementService = new MovementService(this.httpClient, cacheDir);
         this.userService = new UserService(this.httpClient);
     }
     static async create(credentials) {
-        const client = new TonalClient(credentials.username, credentials.password);
+        const client = new TonalClient(credentials.username, credentials.password, credentials.cacheDir);
         await client.authManager.authenticate();
         return client;
     }
@@ -525,7 +568,32 @@ class TonalClient {
     }
     // User operations
     async getUserInfo() {
-        return this.userService.getUserInfo();
+        if (this.userInfo) {
+            return this.userInfo;
+        }
+        let request = this.userInfoPromise;
+        if (!request) {
+            request = this.userService.getUserInfo();
+            this.userInfoPromise = request;
+        }
+        try {
+            const userInfo = await request;
+            if (this.userInfoPromise === request) {
+                this.userInfo = userInfo;
+                this.userInfoPromise = undefined;
+            }
+            return userInfo;
+        }
+        catch (error) {
+            if (this.userInfoPromise === request) {
+                this.userInfoPromise = undefined;
+            }
+            throw error;
+        }
+    }
+    invalidateUserInfo() {
+        this.userInfo = undefined;
+        this.userInfoPromise = undefined;
     }
     async getGoals() {
         return this.userService.getGoals();

package/package.json

@@ -1,8 +1,8 @@
 {
   "name": "@dlwiest/ts-tonal-client",
-  "version": "0.2.1",
+  "version": "0.3.0",
   "description": "TypeScript client for Tonal API",
-  "main": "dist/index.js",
+  "main": "dist/index.cjs",
   "module": "dist/index.esm.js",
   "types": "dist/index.d.ts",
   "type": "module",
@@ -10,14 +10,14 @@
     ".": {
       "types": "./dist/index.d.ts",
       "import": "./dist/index.esm.js",
-      "require": "./dist/index.js"
+      "require": "./dist/index.cjs"
     }
   },
   "files": [
     "dist"
   ],
   "scripts": {
-    "build": "rollup -c --bundleConfigAsCjs",
+    "build": "node -e \"require('node:fs').rmSync('dist', { recursive: true, force: true })\" && rollup -c --bundleConfigAsCjs",
     "typecheck": "tsc --noEmit",
     "dev": "tsc --watch --noEmit",
     "test": "jest",
@@ -64,7 +64,7 @@
     "client"
   ],
   "author": "dlwiest",
-  "license": "ISC",
+  "license": "MIT",
   "bugs": {
     "url": "https://github.com/dlwiest/ts-tonal-client/issues"
   },

package/dist/examples/create-workout.d.ts

@@ -1 +0,0 @@
-import 'dotenv/config';

package/dist/examples/debug-daily-lifts.d.ts

@@ -1 +0,0 @@
-import 'dotenv/config';

package/dist/examples/delete-workout.d.ts

@@ -1 +0,0 @@
-import 'dotenv/config';

package/dist/examples/edit-workout.d.ts

@@ -1 +0,0 @@
-import 'dotenv/config';

package/dist/examples/estimate-workout.d.ts

@@ -1 +0,0 @@
-import 'dotenv/config';

package/dist/examples/get-achievement-stats.d.ts

@@ -1,2 +0,0 @@
-#!/usr/bin/env tsx
-import 'dotenv/config';

package/dist/examples/get-achievements.d.ts

@@ -1,2 +0,0 @@
-#!/usr/bin/env tsx
-import 'dotenv/config';

package/dist/examples/get-activity-summaries.d.ts

@@ -1,2 +0,0 @@
-#!/usr/bin/env tsx
-import 'dotenv/config';

package/dist/examples/get-current-streak.d.ts

@@ -1 +0,0 @@
-import 'dotenv/config';

package/dist/examples/get-daily-lifts.d.ts

@@ -1 +0,0 @@
-import 'dotenv/config';

package/dist/examples/get-daily-metrics.d.ts

@@ -1 +0,0 @@
-import 'dotenv/config';

package/dist/examples/get-goal-metrics.d.ts

@@ -1 +0,0 @@
-import 'dotenv/config';

package/dist/examples/get-goals.d.ts

@@ -1 +0,0 @@
-import 'dotenv/config';

package/dist/examples/get-home-calendar.d.ts

@@ -1,2 +0,0 @@
-#!/usr/bin/env tsx
-import 'dotenv/config';

package/dist/examples/get-metric-scores.d.ts

@@ -1 +0,0 @@
-import 'dotenv/config';

package/dist/examples/get-movements.d.ts

@@ -1 +0,0 @@
-import 'dotenv/config';

package/dist/examples/get-muscle-readiness.d.ts

@@ -1 +0,0 @@
-import 'dotenv/config';

package/dist/examples/get-program-by-id.d.ts

@@ -1 +0,0 @@
-import 'dotenv/config';

package/dist/examples/get-target-scores.d.ts

@@ -1 +0,0 @@
-import 'dotenv/config';

package/dist/examples/get-training-effect-goals.d.ts

@@ -1 +0,0 @@
-import 'dotenv/config';

package/dist/examples/get-training-types.d.ts

@@ -1 +0,0 @@
-import 'dotenv/config';

package/dist/examples/get-user-info.d.ts

@@ -1 +0,0 @@
-import 'dotenv/config';

package/dist/examples/get-user-settings.d.ts

@@ -1 +0,0 @@
-import 'dotenv/config';

package/dist/examples/get-user-statistics.d.ts

@@ -1,2 +0,0 @@
-#!/usr/bin/env tsx
-import 'dotenv/config';

package/dist/examples/get-user-workouts.d.ts

@@ -1 +0,0 @@
-import 'dotenv/config';

package/dist/examples/get-workout-by-id.d.ts

@@ -1 +0,0 @@
-import 'dotenv/config';

package/dist/examples/get-workout-by-name.d.ts

@@ -1 +0,0 @@
-import 'dotenv/config';

package/dist/examples/get-workout-by-share-url.d.ts

@@ -1 +0,0 @@
-import 'dotenv/config';

package/dist/examples/raw-daily-lifts-test.d.ts

@@ -1 +0,0 @@
-import 'dotenv/config';

package/dist/src/auth/auth-manager.d.ts

@@ -1,15 +0,0 @@
-export declare class AuthManager {
-    private username;
-    private password;
-    private idToken;
-    private refreshToken;
-    private tokenExpiresAt;
-    private isRefreshing;
-    private readonly authUrl;
-    private readonly clientId;
-    constructor(username: string, password: string);
-    authenticate(): Promise<string>;
-    getValidToken(): Promise<string>;
-    private isTokenValid;
-    private refreshTokens;
-}

package/dist/src/client.d.ts

@@ -1,40 +0,0 @@
-import { TonalMovement, TonalSharedWorkout, TonalWorkout, TonalUserInfo, TonalGoal, TonalTrainingEffectGoalsResponse, TonalTrainingType, TonalGoalMetric, TonalUserSettings, TonalDailyMetrics, TonalCurrentStreak, TonalActivitySummary, TonalUserStatistics, TonalAchievementStats, TonalEarnedAchievement, TonalHomeCalendar, TonalWorkoutEstimateSet, TonalWorkoutEstimateResponse, TonalWorkoutCreateRequest, TonalWorkoutUpdateRequest, TonalMuscleReadiness, TonalProgram, TonalTargetScoresResponse, TonalMetricScoresResponse } from './types';
-export declare class TonalClient {
-    private authManager;
-    private httpClient;
-    private workoutService;
-    private movementService;
-    private userService;
-    private constructor();
-    static create(credentials: {
-        username: string;
-        password: string;
-    }): Promise<TonalClient>;
-    getMovements(useCache?: boolean): Promise<TonalMovement[]>;
-    invalidateMovementsCache(): Promise<void>;
-    getUserInfo(): Promise<TonalUserInfo>;
-    getGoals(): Promise<TonalGoal[]>;
-    getTrainingEffectGoals(): Promise<TonalTrainingEffectGoalsResponse>;
-    getTrainingTypes(): Promise<TonalTrainingType[]>;
-    getGoalMetrics(): Promise<TonalGoalMetric[]>;
-    getUserSettings(): Promise<TonalUserSettings>;
-    getDailyMetrics(days?: number): Promise<TonalDailyMetrics[]>;
-    getCurrentStreak(): Promise<TonalCurrentStreak>;
-    getActivitySummaries(): Promise<TonalActivitySummary[]>;
-    getUserStatistics(): Promise<TonalUserStatistics>;
-    getAchievementStats(): Promise<TonalAchievementStats>;
-    getAchievements(): Promise<TonalEarnedAchievement[]>;
-    getHomeCalendar(): Promise<TonalHomeCalendar>;
-    getMuscleReadiness(): Promise<TonalMuscleReadiness>;
-    getProgramById(programId: string): Promise<TonalProgram>;
-    getTargetScores(): Promise<TonalTargetScoresResponse>;
-    getMetricScores(startWeek?: number): Promise<TonalMetricScoresResponse>;
-    getUserWorkouts(offset?: number, limit?: number): Promise<TonalWorkout[]>;
-    getDailyLifts(timeZone?: string): Promise<TonalWorkout[]>;
-    getWorkoutById(workoutId: string): Promise<TonalWorkout>;
-    getWorkoutByShareUrl(shareUrl: string): Promise<TonalSharedWorkout>;
-    estimateWorkoutDuration(sets: TonalWorkoutEstimateSet[]): Promise<TonalWorkoutEstimateResponse>;
-    createWorkout(workoutData: TonalWorkoutCreateRequest): Promise<TonalWorkout>;
-    updateWorkout(workoutData: TonalWorkoutUpdateRequest): Promise<TonalWorkout>;
-    deleteWorkout(workoutId: string): Promise<void>;
-}

package/dist/src/http/http-client.d.ts

@@ -1,12 +0,0 @@
-import { AuthManager } from '../auth/auth-manager';
-export declare class HttpClient {
-    private authManager;
-    private readonly baseUrl;
-    private readonly requestTimeout;
-    private readonly maxRetries;
-    constructor(authManager: AuthManager);
-    request<T>(endpoint: string, options?: RequestInit, expectsBody?: boolean): Promise<T>;
-    private makeRequest;
-    private makeRequestWithRetry;
-    private sleep;
-}

package/dist/src/index.d.ts

@@ -1,4 +0,0 @@
-import { TonalClient } from './client';
-export * from './types';
-export { TonalClient };
-export default TonalClient;

package/dist/src/services/movement-service.d.ts

@@ -1,9 +0,0 @@
-import { HttpClient } from '../http/http-client';
-import { TonalMovement } from '../types';
-export declare class MovementService {
-    private httpClient;
-    private cacheManager;
-    constructor(httpClient: HttpClient);
-    getMovements(useCache?: boolean): Promise<TonalMovement[]>;
-    invalidateMovementsCache(): Promise<void>;
-}