@hienlh/ppm 0.8.63 → 0.8.64

package/CHANGELOG.md

@@ -1,5 +1,14 @@
 # Changelog
 
+## [0.8.64] - 2026-03-27
+
+### Fixed
+- **Usage polling dedup**: Concurrent `pollOnce` calls now share a single in-flight fetch instead of hammering the API
+- **429 cooldown floor**: Minimum 60s cooldown on 429 responses, even if `retry-after` header is lower
+
+### Added
+- **Browser preview tests**: Unit tests (12) for tunnel route validation and lifecycle; integration test (6) verifying real Cloudflare tunnel creation and access
+
 ## [0.8.63] - 2026-03-27
 
 ### Changed

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hienlh/ppm",
-  "version": "0.8.63",
+  "version": "0.8.64",
   "description": "Personal Project Manager — mobile-first web IDE with AI assistance",
   "author": "hienlh",
   "license": "MIT",

package/src/server/routes/browser-preview.ts

@@ -21,8 +21,8 @@ interface ActiveTunnel {
   startedAt: number;
 }
 
-/** Active tunnels keyed by port */
-const activeTunnels = new Map<number, ActiveTunnel>();
+/** Active tunnels keyed by port — exported for testing */
+export const activeTunnels = new Map<number, ActiveTunnel>();
 
 /** Start a tunnel for a localhost port */
 browserPreviewRoutes.post("/tunnel", async (c) => {

package/src/services/claude-usage.service.ts

@@ -51,6 +51,10 @@ let pollTimer: ReturnType<typeof setInterval> | null = null;
 
 // Per-token cooldown map: token prefix → earliest allowed fetch time
 const tokenCooldowns = new Map<string, number>();
+const MIN_COOLDOWN_MS = 60_000; // floor: at least 60s cooldown on 429
+
+// Dedup: if a poll is already in-flight, reuse the same promise
+let inflightPoll: Promise<void> | null = null;
 
 // Legacy: Keychain token cache for users without accounts in DB
 let tokenCache: { token: string; timestamp: number } | null = null;
@@ -113,9 +117,10 @@ async function fetchUsageForToken(token: string): Promise<ClaudeUsage> {
   });
   if (res.status === 429) {
     const retryAfter = parseInt(res.headers.get("retry-after") ?? "60", 10);
+    const cooldownMs = Math.max(retryAfter * 1000, MIN_COOLDOWN_MS);
     const cooldownKey = token.substring(0, 20);
-    tokenCooldowns.set(cooldownKey, Date.now() + retryAfter * 1000);
-    throw new Error(`Usage API 429 — cooldown ${retryAfter}s`);
+    tokenCooldowns.set(cooldownKey, Date.now() + cooldownMs);
+    throw new Error(`Usage API 429 — cooldown ${Math.ceil(cooldownMs / 1000)}s`);
   }
   if (!res.ok) throw new Error(`Usage API returned ${res.status}`);
   const raw = (await res.json()) as Record<string, any>;
@@ -228,7 +233,7 @@ async function fetchLegacySingleAccount(): Promise<void> {
   } catch {}
 }
 
-async function pollOnce(): Promise<void> {
+async function pollOnceInternal(): Promise<void> {
   try {
     const hasAccounts = accountService.list().length > 0;
     if (hasAccounts) {
@@ -241,6 +246,13 @@ async function pollOnce(): Promise<void> {
   }
 }
 
+/** Deduped: concurrent callers share a single in-flight fetch */
+async function pollOnce(): Promise<void> {
+  if (inflightPoll) return inflightPoll;
+  inflightPoll = pollOnceInternal().finally(() => { inflightPoll = null; });
+  return inflightPoll;
+}
+
 // ---------------------------------------------------------------------------
 // Public API
 // ---------------------------------------------------------------------------