salmon-loop 0.3.2 → 0.5.0

package/dist/core/adapters/git/lock-manager.js

@@ -96,8 +96,9 @@ export class FileHandleManager {
                     const fs = await import('fs/promises');
                     await fs.unlink(lockFile);
                 }
-                catch {
-                    // Ignore
+                catch (error) {
+                    // Ignore - best-effort force unlock
+                    getLogger().debug(`[LockManager] force unlock failed: ${error instanceof Error ? error.message : String(error)}`);
                 }
             }
             while (Date.now() - start < LIMITS.lockWaitTimeoutMs) {
@@ -142,9 +143,10 @@ export class FileHandleManager {
                                 continue; // Retry immediately after removing stale lock
                             }
                         }
-                        catch {
+                        catch (error) {
                             // If the lock file is unreadable, we cannot safely determine ownership.
                             // Do not auto-remove in this path; rely on timeout-based recovery instead.
+                            getLogger().debug(`[LockManager] lock file unreadable during stale check: ${error instanceof Error ? error.message : String(error)}`);
                         }
                         // Exponential backoff: delay increases with retry count, capped at 2000ms
                         retryCount++;
@@ -159,8 +161,9 @@ export class FileHandleManager {
                             await mkdir(repoPath, { recursive: true });
                             continue; // Retry immediately
                         }
-                        catch {
+                        catch (error) {
                             // If mkdir fails, just wait and retry
+                            getLogger().debug(`[LockManager] mkdir fallback failed: ${error instanceof Error ? error.message : String(error)}`);
                         }
                         await this.abortableDelay(LIMITS.retry.io.initialDelayMs, hardAbort.signal);
                     }

package/dist/core/ast/parser.js

@@ -24,39 +24,48 @@ export class AstParser {
     static state = InitState.Idle;
     static initPromise = null;
     /**
-     * Get the Parser class from web-tree-sitter, handling different API versions
+     * Get the Parser class from web-tree-sitter, handling different API versions.
+     * Uses `as unknown as` for CJS/ESM interop — the runtime module shape varies
+     * across bundler configurations.
      */
     static getParserClass() {
+        const mod = TreeSitter;
         try {
-            return TreeSitter.Parser || TreeSitter.default?.Parser || TreeSitter;
+            return (mod.Parser ??
+                mod.default?.Parser ??
+                TreeSitter);
         }
         catch (_error) {
             getLogger().degrade(text.ast.degradedApi);
-            return TreeSitter.default || TreeSitter;
+            return (mod.default ?? TreeSitter);
         }
     }
     /**
-     * Get the Language class from web-tree-sitter, handling different API versions
+     * Get the Language class from web-tree-sitter, handling different API versions.
+     * Uses `as unknown as` for CJS/ESM interop.
      */
     static getLanguageClass() {
+        const mod = TreeSitter;
         try {
-            return TreeSitter.Language || TreeSitter.default?.Language;
+            return (mod.Language ?? mod.default?.Language);
         }
         catch (_error) {
             getLogger().degrade(text.ast.degradedApi);
-            return TreeSitter.default?.Language;
+            return mod.default?.Language;
         }
     }
     /**
-     * Get the Query class from web-tree-sitter, handling different API versions
+     * Get the Query class from web-tree-sitter, handling different API versions.
+     * Uses `as unknown as` for CJS/ESM interop.
      */
     static getQueryClass() {
+        const mod = TreeSitter;
         try {
-            return TreeSitter.Query || TreeSitter.default?.Query;
+            return (mod.Query ?? mod.default?.Query);
         }
         catch (_error) {
             getLogger().degrade(text.ast.degradedApi);
-            return TreeSitter.default?.Query;
+            return mod.default?.Query;
         }
     }
     static languages = new Map();

package/dist/core/checkpoint-domain/manifest-store.js

@@ -2,6 +2,7 @@ import { createHash } from 'crypto';
 import { mkdir, open, readFile, rename, stat, unlink, writeFile } from '../adapters/fs/node-fs.js';
 import { defaultPathAdapter } from '../adapters/path/path-adapter.js';
 import { recordAuditEvent } from '../observability/audit-trail.js';
+import { getLogger } from '../observability/logger.js';
 import { getUserCheckpointManifestDir } from '../runtime/paths.js';
 const CHECKPOINT_MANIFEST_FILENAME_V2 = 'manifest.v2.json';
 const CHECKPOINT_MANIFEST_FILENAME_V1 = 'manifest.v1.json';
@@ -131,7 +132,8 @@ async function withManifestLock(repoPath, operation, lockPolicy) {
                 lockPathHash,
             }, { source: 'runtime', severity: 'low', scope: 'session', phase: 'PREFLIGHT' });
         }
-        catch {
+        catch (error) {
+            getLogger().debug(`[ManifestStore] Stale lock reclaim failed: ${error instanceof Error ? error.message : String(error)}`);
             try {
                 const lockStat = await stat(lockPath);
                 const ageMs = Date.now() - lockStat.mtimeMs;
@@ -144,8 +146,9 @@ async function withManifestLock(repoPath, operation, lockPolicy) {
                     }, { source: 'runtime', severity: 'medium', scope: 'session', phase: 'PREFLIGHT' });
                 }
             }
-            catch {
+            catch (error) {
                 // Ignore lock probe failures; retry loop handles contention.
+                getLogger().debug(`[ManifestStore] Lock probe failed: ${error instanceof Error ? error.message : String(error)}`);
             }
         }
     };
@@ -156,7 +159,8 @@ async function withManifestLock(repoPath, operation, lockPolicy) {
             await handle.writeFile(JSON.stringify({ pid: process.pid, createdAtMs: Date.now() }), 'utf8');
             break;
         }
-        catch {
+        catch (error) {
+            getLogger().debug(`[ManifestStore] Lock acquisition attempt failed: ${error instanceof Error ? error.message : String(error)}`);
             await tryClearStaleLock();
             await new Promise((resolve) => setTimeout(resolve, 30 * (attempt + 1)));
         }
@@ -185,14 +189,14 @@ async function withManifestLock(repoPath, operation, lockPolicy) {
         try {
             await handle.close();
         }
-        catch {
-            // ignore
+        catch (error) {
+            getLogger().debug(`[ManifestStore] Failed to close lock handle: ${error instanceof Error ? error.message : String(error)}`);
         }
         try {
             await unlink(lockPath);
         }
-        catch {
-            // ignore
+        catch (error) {
+            getLogger().debug(`[ManifestStore] Failed to unlink lock file: ${error instanceof Error ? error.message : String(error)}`);
         }
     }
 }
@@ -204,7 +208,8 @@ export async function readCheckpointManifest(repoPath) {
         try {
             return normalizeManifest(JSON.parse(raw));
         }
-        catch {
+        catch (error) {
+            getLogger().warn(`[ManifestStore] v2 manifest parse error, triggering self-heal: ${error instanceof Error ? error.message : String(error)}`);
             await selfHealCorruptedManifest({
                 repoPath,
                 manifestPath: manifestV2Path,
@@ -214,15 +219,16 @@ export async function readCheckpointManifest(repoPath) {
             });
         }
     }
-    catch {
-        // fallback to legacy v1 file
+    catch (error) {
+        getLogger().debug(`[ManifestStore] v2 manifest read failed, falling back to v1: ${error instanceof Error ? error.message : String(error)}`);
     }
     try {
         const raw = await readFile(manifestV1Path, 'utf8');
         try {
             return normalizeManifest(JSON.parse(raw));
         }
-        catch {
+        catch (error) {
+            getLogger().warn(`[ManifestStore] v1 manifest parse error, triggering self-heal: ${error instanceof Error ? error.message : String(error)}`);
             await selfHealCorruptedManifest({
                 repoPath,
                 manifestPath: manifestV1Path,
@@ -233,7 +239,8 @@ export async function readCheckpointManifest(repoPath) {
             return createEmptyManifest();
         }
     }
-    catch {
+    catch (error) {
+        getLogger().debug(`[ManifestStore] v1 manifest read failed, returning empty manifest: ${error instanceof Error ? error.message : String(error)}`);
         return createEmptyManifest();
     }
     return createEmptyManifest();
@@ -333,7 +340,8 @@ export async function probeCheckpointHandle(repoPath, checkpointId) {
             return { handle: null, reason: 'not_found' };
         return { handle, reason: 'ok' };
     }
-    catch {
+    catch (error) {
+        getLogger().debug(`[ManifestStore] Manifest parse failed during probe: ${error instanceof Error ? error.message : String(error)}`);
         return { handle: null, reason: 'manifest_parse_error' };
     }
 }

package/dist/core/checkpoint-domain/service.js

@@ -1,3 +1,4 @@
+import { getLogger } from '../observability/logger.js';
 import { CheckpointManager } from '../strata/checkpoint/manager.js';
 import { garbageCollectManifest, linkSessionToCheckpoint, probeCheckpointHandle, readCheckpointManifest, removeCheckpointHandle, upsertCheckpointHandle, } from './manifest-store.js';
 export class GitSnapshotCheckpointService {
@@ -74,8 +75,9 @@ export class GitSnapshotCheckpointService {
                 await this.checkpointManager.deleteSnapshot(input.repoPath, checkpointId);
                 refsRemoved += 1;
             }
-            catch {
+            catch (error) {
                 // Best-effort ref reconciliation; manifest remains source of truth.
+                getLogger().debug(`[CheckpointService] snapshot ref cleanup failed for ${checkpointId}: ${error instanceof Error ? error.message : String(error)}`);
             }
         }
         return { removed: manifestGc.removed, refsRemoved };

package/dist/core/config/limits.js

@@ -2,7 +2,7 @@ export const LIMITS = {
     // Patch safety
     maxFilesChanged: 2,
     maxDiffLines: 200,
-    maxRetries: 2,
+    maxRetries: 5,
     // Context budget (token-based, with char fallback)
     maxContextTokens: 7500, // ~30k chars equivalent
     minContextTokens: 1250, // ~5k chars equivalent

package/dist/core/config/model-pricing.js

@@ -0,0 +1,61 @@
+/**
+ * Per-model token pricing (USD per 1M tokens).
+ *
+ * Used to estimate run cost from token counts.
+ * Prices are approximate and should be updated when providers change rates.
+ */
+// prettier-ignore
+export const MODEL_PRICING = {
+    // Anthropic Claude
+    'claude-opus-4-7': { inputPer1M: 15, outputPer1M: 75 },
+    'claude-opus-4-6': { inputPer1M: 15, outputPer1M: 75 },
+    'claude-sonnet-4-6': { inputPer1M: 3, outputPer1M: 15 },
+    'claude-sonnet-4-5': { inputPer1M: 3, outputPer1M: 15 },
+    'claude-haiku-4-5': { inputPer1M: 0.80, outputPer1M: 4 },
+    'claude-haiku-4-5-20251001': { inputPer1M: 0.80, outputPer1M: 4 },
+    'claude-3-opus': { inputPer1M: 15, outputPer1M: 75 },
+    'claude-3-sonnet': { inputPer1M: 3, outputPer1M: 15 },
+    'claude-3-haiku': { inputPer1M: 0.25, outputPer1M: 1.25 },
+    // OpenAI
+    'gpt-4o': { inputPer1M: 2.50, outputPer1M: 10 },
+    'gpt-4o-mini': { inputPer1M: 0.15, outputPer1M: 0.60 },
+    'gpt-4-turbo': { inputPer1M: 10, outputPer1M: 30 },
+    'gpt-4': { inputPer1M: 30, outputPer1M: 60 },
+    'o1': { inputPer1M: 15, outputPer1M: 60 },
+    'o1-mini': { inputPer1M: 3, outputPer1M: 12 },
+    'o3-mini': { inputPer1M: 1.10, outputPer1M: 4.40 },
+    // Google Gemini
+    'gemini-2.5-pro': { inputPer1M: 1.25, outputPer1M: 10 },
+    'gemini-2.5-flash': { inputPer1M: 0.15, outputPer1M: 0.60 },
+    'gemini-2.0-flash': { inputPer1M: 0.10, outputPer1M: 0.40 },
+};
+/**
+ * Look up pricing for a model ID.
+ * Tries exact match first, then prefix match (e.g. "claude-sonnet-4-6-20250514" matches "claude-sonnet-4-6").
+ * Returns undefined if no pricing is available.
+ */
+export function getModelPricing(modelId) {
+    if (!modelId)
+        return undefined;
+    // Exact match
+    if (MODEL_PRICING[modelId])
+        return MODEL_PRICING[modelId];
+    // Prefix match — try longest prefix first
+    const normalized = modelId.toLowerCase();
+    for (const key of Object.keys(MODEL_PRICING)) {
+        if (normalized.startsWith(key))
+            return MODEL_PRICING[key];
+    }
+    return undefined;
+}
+/**
+ * Estimate cost in USD from token counts and model ID.
+ */
+export function estimateCost(inputTokens, outputTokens, modelId) {
+    const pricing = getModelPricing(modelId);
+    if (!pricing)
+        return undefined;
+    return ((inputTokens / 1_000_000) * pricing.inputPer1M +
+        (outputTokens / 1_000_000) * pricing.outputPer1M);
+}
+//# sourceMappingURL=model-pricing.js.map