salmon-loop 0.3.2 → 0.5.0

package/dist/core/session/pruning-strategy.js

@@ -178,7 +178,8 @@ export class SessionArchiver {
                 })),
             };
         }
-        catch {
+        catch (error) {
+            getLogger().debug(`[SessionPruning] Failed to restore session from archive: ${error instanceof Error ? error.message : String(error)}`);
             return null;
         }
     }

package/dist/core/session/token-tracker.js

@@ -1,6 +1,9 @@
 import path from 'path';
 import { FileAdapter } from '../adapters/fs/index.js';
+import { estimateCost } from '../config/model-pricing.js';
 import { logIgnoredError } from '../observability/ignored-error.js';
+import { getLogger } from '../observability/logger.js';
+import { isRecord } from '../utils/serialize.js';
 /**
  * Token usage tracker for chat sessions.
  * Extracts and accumulates token statistics from LLM execution results.
@@ -21,26 +24,36 @@ export class TokenTracker {
         try {
             const auditRaw = await this.fileAdapter.readFile(result.auditPath, 'utf8');
             const audit = JSON.parse(auditRaw);
-            const eventsRef = audit?.context?.eventsRef;
+            if (!isRecord(audit))
+                return null;
+            const context = isRecord(audit.context) ? audit.context : null;
+            const eventsRef = isRecord(context?.eventsRef) ? context.eventsRef : null;
             if (!eventsRef || typeof eventsRef.path !== 'string')
                 return null;
             const eventsPath = path.isAbsolute(eventsRef.path)
                 ? eventsRef.path
                 : path.join(path.dirname(result.auditPath), eventsRef.path);
             const eventsRaw = await this.fileAdapter.readFile(eventsPath, 'utf8');
-            const events = eventsRaw
-                .split('\n')
-                .filter((line) => line.trim().length > 0)
-                .map((line) => JSON.parse(line));
+            const events = [];
+            for (const line of eventsRaw.split('\n')) {
+                if (line.trim().length === 0)
+                    continue;
+                try {
+                    events.push(JSON.parse(line));
+                }
+                catch (error) {
+                    getLogger().debug(`[TokenTracker] Skipping malformed JSON line in events file: ${error instanceof Error ? error.message : String(error)}`);
+                }
+            }
             let inputTokens = 0;
             let outputTokens = 0;
             for (const event of events) {
-                if (!event || typeof event !== 'object')
+                if (!isRecord(event))
                     continue;
                 if (event.action !== 'llm.usage')
                     continue;
                 const details = event.details;
-                if (!details || typeof details !== 'object')
+                if (!isRecord(details))
                     continue;
                 const promptTokens = details.promptTokens;
                 const completionTokens = details.completionTokens;
@@ -65,11 +78,16 @@ export class TokenTracker {
         }
     }
     /**
-     * Accumulate tokens into session metadata
+     * Accumulate tokens into session metadata.
+     * Computes estimated cost if model pricing is available.
      */
-    static accumulate(session, usage) {
+    static accumulate(session, usage, modelId) {
         session.meta.totalTokens.input += usage.inputTokens;
         session.meta.totalTokens.output += usage.outputTokens;
+        const cost = usage.estimatedCost ?? estimateCost(usage.inputTokens, usage.outputTokens, modelId);
+        if (cost !== undefined) {
+            session.meta.totalTokens.estimatedCost = (session.meta.totalTokens.estimatedCost ?? 0) + cost;
+        }
     }
     /**
      * Estimate token count based on text length

package/dist/core/skills/parser.js

@@ -3,6 +3,7 @@ import { parse as parseYaml } from 'yaml';
 import { z } from 'zod';
 import { text } from '../../locales/index.js';
 import { tryGetLogger } from '../observability/logger.js';
+import { errorMessage } from '../utils/error.js';
 /**
  * Safe logger accessor that never throws when the logger is not yet initialized.
  *
@@ -129,7 +130,7 @@ export class SkillParser {
             parsed = parseYaml(yamlRaw);
         }
         catch (error) {
-            const reason = error instanceof Error ? error.message : String(error);
+            const reason = errorMessage(error);
             const msg = text.skills.yamlParseError(filePath, reason);
             safeLogger().error(msg);
             throw new Error(msg);
@@ -188,7 +189,7 @@ export class SkillParser {
             parsed = parseYaml(yamlRaw);
         }
         catch (error) {
-            const reason = error instanceof Error ? error.message : String(error);
+            const reason = errorMessage(error);
             const msg = text.skills.yamlParseError(filePath, reason);
             safeLogger().error(msg);
             throw new Error(msg);

package/dist/core/skills/permissions.js

@@ -89,9 +89,9 @@ export class SkillPermissionManager {
                 this.policies = [];
             }
         }
-        catch {
+        catch (error) {
             const logger = tryGetLogger();
-            logger?.warn(text.skills.permissionFileLoadError(this.filePath));
+            logger?.warn(`${text.skills.permissionFileLoadError(this.filePath)}: ${error instanceof Error ? error.message : String(error)}`);
             this.policies = [];
         }
     }

package/dist/core/skills/runtime/MicroTaskRunner.js

@@ -67,7 +67,7 @@ export class MicroTaskRunner {
                     cmd,
                     output: output,
                 })),
-                injectedPrompt: injectAction?.params?.prompt || '',
+                injectedPrompt: String(injectAction?.params?.prompt ?? ''),
                 status: plan.shouldAbort ? 'FAILURE' : 'SUCCESS',
             };
         }

package/dist/core/skills/runtime/SkillRunner.js

@@ -2,6 +2,7 @@ import * as crypto from 'crypto';
 import { MicroTaskRunner } from '../../grizzco/dsl/MicroTaskRunner.js';
 import { tryGetLogger } from '../../observability/logger.js';
 import { Phase } from '../../types/index.js';
+import { isRecord } from '../../utils/serialize.js';
 import { emitSkillAuditEvent, generateSkillTraceId, hashSkillArgs } from '../audit.js';
 import { SkillParser } from '../parser.js';
 import { SkillStrategyDSL } from '../strategy.js';
@@ -227,7 +228,9 @@ export async function executeSkill(options) {
                             argsHash,
                             traceId,
                             denyReason: result.error?.code || 'unknown',
-                            denySource: result.meta?.authorization?.source || 'policy',
+                            denySource: isRecord(result.meta) && isRecord(result.meta.authorization)
+                                ? result.meta.authorization.source
+                                : 'policy',
                             durationMs: Date.now() - startedAt,
                         });
                     }
@@ -264,7 +267,7 @@ export async function executeSkill(options) {
                 cmd,
                 output: String(output),
             })),
-            injectedPrompt: String(inject?.params?.prompt ?? ''),
+            injectedPrompt: String((isRecord(inject?.params) ? inject.params.prompt : undefined) ?? ''),
             status,
         };
     }

package/dist/core/slash/steps/slash-execute.js

@@ -2,12 +2,13 @@ function planToDecision(plan, options) {
     const action = plan.actions[0];
     if (!action)
         return { kind: 'consumed' };
+    const p = action.params ?? {};
     if (action.type === 'FORWARD_TEXT') {
-        const next = String(action.params?.input ?? '');
+        const next = String(p.input ?? '');
         return { kind: 'forward', input: next };
     }
     if (action.type === 'UNKNOWN_SLASH') {
-        const cmd = String(action.params?.commandName ?? '');
+        const cmd = String(p.commandName ?? '');
         if (options.unknownSlashPolicy === 'forward_as_text') {
             return { kind: 'forward', input: cmd };
         }
@@ -31,7 +32,8 @@ export function buildSlashExecuteStep(options, meta) {
             const decision = planToDecision(plan, { unknownSlashPolicy: options.unknownSlashPolicy });
             return { ...context, data: { ...data, __decision: decision } };
         }
-        const commandName = String(action.params?.commandName ?? '');
+        const p = action.params ?? {};
+        const commandName = String(p.commandName ?? '');
         const spec = options.registry.find(commandName);
         if (!spec) {
             const decision = planToDecision({ ...plan, actions: [{ type: 'UNKNOWN_SLASH', params: { commandName } }] }, { unknownSlashPolicy: options.unknownSlashPolicy });
@@ -50,8 +52,8 @@ export function buildSlashExecuteStep(options, meta) {
         const req = {
             rawInput: context.input.raw,
             command: spec,
-            argsText: String(action.params?.argsText ?? ''),
-            tokens: Array.isArray(action.params?.tokens) ? action.params.tokens : [],
+            argsText: String(p.argsText ?? ''),
+            tokens: Array.isArray(p.tokens) ? p.tokens : [],
             meta,
         };
         const result = await handler.execute(req);

package/dist/core/slash/strategy.js

@@ -18,7 +18,7 @@ export const SlashStrategyDSL = (engine) => {
         .when((c) => c.input.isSlash && Boolean(c.resolved?.command), (p) => {
         p.setWorker('slash.execute');
         p.addAction('EXECUTE_SLASH', {
-            commandName: engine.ctx.resolved.command.name,
+            commandName: engine.ctx.resolved?.command?.name ?? '',
             argsText: engine.ctx.input.argsText ?? '',
             tokens: engine.ctx.input.tokens ?? [],
         });

package/dist/core/strata/checkpoint/manager.js

@@ -63,7 +63,8 @@ export class CheckpointManager {
                     try {
                         writeTreeProbe = await probeWriteTreeFailure(git);
                     }
-                    catch {
+                    catch (probeError) {
+                        getLogger().debug(`[CheckpointManager] probeWriteTreeFailure failed: ${probeError instanceof Error ? probeError.message : String(probeError)}`);
                         writeTreeProbe = {};
                     }
                 }
@@ -129,7 +130,8 @@ export class CheckpointManager {
         try {
             meta = JSON.parse(msg);
         }
-        catch {
+        catch (error) {
+            getLogger().debug(`[CheckpointManager] Invalid snapshot metadata for ${snapshotHash}: ${error instanceof Error ? error.message : String(error)}`);
             throw new Error(`Invalid snapshot metadata for ${snapshotHash}`);
         }
         if (!meta.staged) {
@@ -268,8 +270,8 @@ export class CheckpointManager {
             try {
                 await rm(tempIndexFile, { force: true });
             }
-            catch {
-                // Ignore cleanup errors
+            catch (error) {
+                getLogger().debug(`[CheckpointManager] Failed to cleanup temp index: ${error instanceof Error ? error.message : String(error)}`);
             }
         }
     }
@@ -292,7 +294,8 @@ export class CheckpointManager {
         try {
             meta = JSON.parse(msg);
         }
-        catch {
+        catch (error) {
+            getLogger().debug(`[CheckpointManager] Invalid snapshot metadata for ${snapshotHash}: ${error instanceof Error ? error.message : String(error)}`);
             throw new Error(`Invalid snapshot metadata for ${snapshotHash}`);
         }
         // 3. Get Parent Commit (Original HEAD)
@@ -339,7 +342,8 @@ export class CheckpointManager {
                 return { hash, timestamp, message, ref };
             });
         }
-        catch {
+        catch (error) {
+            getLogger().debug(`[CheckpointManager] Failed to list snapshots: ${error instanceof Error ? error.message : String(error)}`);
             return [];
         }
     }
@@ -354,8 +358,8 @@ export class CheckpointManager {
             await git.exec(['update-ref', '-d', `refs/s8p/snapshots/${snapshotHash}`]);
             return;
         }
-        catch {
-            // If direct deletion fails (maybe hash mismatch or short hash), try to find the ref
+        catch (error) {
+            getLogger().debug(`[CheckpointManager] Direct snapshot deletion failed, trying ref search: ${error instanceof Error ? error.message : String(error)}`);
         }
         // Fallback: find the ref pointing to this hash
         // Note: This is expensive if there are many snapshots, but safe.
@@ -392,7 +396,8 @@ export class CheckpointManager {
         try {
             meta = JSON.parse(msg);
         }
-        catch {
+        catch (error) {
+            getLogger().debug(`[CheckpointManager] Invalid snapshot metadata for ${snapshotHash}: ${error instanceof Error ? error.message : String(error)}`);
             throw new Error(`Invalid snapshot metadata for ${snapshotHash}`);
         }
         // Staged files: Diff between Parent (HEAD at time of snapshot) and Staged Tree
@@ -475,7 +480,8 @@ export class CheckpointManager {
         try {
             meta = JSON.parse(msg);
         }
-        catch {
+        catch (error) {
+            getLogger().debug(`[CheckpointManager] Invalid backup metadata for ${backupHash}: ${error instanceof Error ? error.message : String(error)}`);
             throw new Error(`Invalid backup metadata for ${backupHash}`);
         }
         // 2. Restore Worktree and Index to the T1 state

package/dist/core/strata/checkpoint/snapshot-create.js

@@ -2,6 +2,7 @@ import { randomBytes } from 'crypto';
 import { tmpdir } from 'os';
 import { join } from 'path';
 import { rm } from '../../adapters/fs/node-fs.js';
+import { getLogger } from '../../observability/logger.js';
 import { normalizePath } from '../../utils/path.js';
 async function getSafeUntrackedFiles(git) {
     // --exclude-standard: Respect .gitignore
@@ -50,8 +51,8 @@ export async function createSnapshotCommitFromStagedTree(input) {
                         await git.exec(['add', '--', file], { env });
                     }
                 }
-                catch {
-                    // Ignore per-file add failures to keep snapshot best-effort.
+                catch (error) {
+                    getLogger().debug(`[SnapshotCreate] Failed to add file ${file} to snapshot index: ${error instanceof Error ? error.message : String(error)}`);
                 }
             }
         }
@@ -71,8 +72,8 @@ export async function createSnapshotCommitFromStagedTree(input) {
         try {
             await rm(tempIndexFile, { force: true });
         }
-        catch {
-            // Ignore cleanup errors.
+        catch (error) {
+            getLogger().debug(`[SnapshotCreate] Failed to clean up temp index file: ${error instanceof Error ? error.message : String(error)}`);
         }
     }
 }

package/dist/core/strata/checkpoint/snapshot-write-tree.js

@@ -1,5 +1,6 @@
 import { join } from 'path';
 import { stat } from '../../adapters/fs/node-fs.js';
+import { getLogger } from '../../observability/logger.js';
 import { classifyGitFailureHint } from './snapshot-audit.js';
 export async function tryWriteTreeWithRetry(git, retryDelaysMs) {
     let attempts = 0;
@@ -28,7 +29,8 @@ export async function probeWriteTreeFailure(git) {
         details.indexLockPresent = true;
         details.indexLockAgeMs = Math.max(0, Math.floor(Date.now() - lockStat.mtimeMs));
     }
-    catch {
+    catch (error) {
+        getLogger().debug(`[SnapshotWriteTree] Index lock not present or unreadable: ${error instanceof Error ? error.message : String(error)}`);
         details.indexLockPresent = false;
     }
     try {
@@ -39,7 +41,8 @@ export async function probeWriteTreeFailure(git) {
             .filter(Boolean).length;
         details.unmergedCount = count;
     }
-    catch {
+    catch (error) {
+        getLogger().debug(`[SnapshotWriteTree] Failed to list unmerged files: ${error instanceof Error ? error.message : String(error)}`);
         details.unmergedCount = undefined;
     }
     try {
@@ -64,7 +67,8 @@ export async function probeWriteTreeFailure(git) {
             });
         }
     }
-    catch {
+    catch (error) {
+        getLogger().debug(`[SnapshotWriteTree] Failed to probe work tree status: ${error instanceof Error ? error.message : String(error)}`);
         details.isInsideWorkTree = undefined;
     }
     return details;

package/dist/core/strata/engine/shadow-merge-engine.js

@@ -269,7 +269,8 @@ export class ShadowMergeEngine {
                 return false;
             return this.guardian.inspect(buffer).isBinary;
         }
-        catch {
+        catch (error) {
+            getLogger().debug(`[ShadowMergeEngine] Binary detection failed for ${relativePath}: ${error instanceof Error ? error.message : String(error)}`);
             return false;
         }
     }
@@ -282,7 +283,8 @@ export class ShadowMergeEngine {
         try {
             return await git.show(ref, relativePath);
         }
-        catch {
+        catch (error) {
+            getLogger().debug(`[ShadowMergeEngine] git show failed for ${relativePath}@${ref}: ${error instanceof Error ? error.message : String(error)}`);
             return null;
         }
     }

package/dist/core/strata/interaction/file-system-provider.js

@@ -81,7 +81,8 @@ export class StrataFileSystemProvider {
             const buffer = await fs.readFile(safePath);
             return this.guardian.inspect(buffer).isBinary;
         }
-        catch {
+        catch (error) {
+            getLogger().debug(`[StrataFileSystem] Binary detection failed for ${filePath}: ${error instanceof Error ? error.message : String(error)}`);
             return false;
         }
     }

package/dist/core/strata/layers/file-state-resolver.js

@@ -3,6 +3,7 @@ import * as path from 'path';
 import * as fs from '../../adapters/fs/node-fs.js';
 import { LIMITS } from '../../config/limits.js';
 import { FileStatus } from '../../grizzco/domain/grizzco-types.js';
+import { getLogger } from '../../observability/logger.js';
 /**
  * FileStateResolver
  * Responsibilities: Accurately scan workspace and index state, implementing the data foundation for Zero Index Access.
@@ -43,9 +44,8 @@ export class FileStateResolver {
                 state.stagedContent = await this.git.show(':0', normalizedPath);
                 state.workingContent = await fs.readFile(absolutePath);
             }
-            catch {
-                // Fallback: If we can't capture content, we might be in a race condition.
-                // But proceed with what we have; strategy layer will handle missing content if needed.
+            catch (error) {
+                getLogger().debug(`[FileStateResolver] Failed to capture MM content for ${normalizedPath}: ${error instanceof Error ? error.message : String(error)}`);
             }
         }
         return state;
@@ -124,8 +124,8 @@ export class FileStateResolver {
                 await fd.close();
             }
         }
-        catch {
-            // If file doesn't exist or can't be read, assume non-binary (safe default for new files)
+        catch (error) {
+            getLogger().debug(`[FileStateResolver] Binary detection failed for ${filePath}: ${error instanceof Error ? error.message : String(error)}`);
             return false;
         }
     }
@@ -137,7 +137,8 @@ export class FileStateResolver {
             const stats = await fs.lstat(filePath);
             return stats.isSymbolicLink();
         }
-        catch {
+        catch (error) {
+            getLogger().debug(`[FileStateResolver] Symlink detection failed for ${filePath}: ${error instanceof Error ? error.message : String(error)}`);
             return false;
         }
     }
@@ -149,7 +150,8 @@ export class FileStateResolver {
             const stats = await fs.stat(filePath);
             return stats.size;
         }
-        catch {
+        catch (error) {
+            getLogger().debug(`[FileStateResolver] File size check failed for ${filePath}: ${error instanceof Error ? error.message : String(error)}`);
             return 0;
         }
     }

package/dist/core/strata/layers/immutable-git-layer.js

@@ -4,6 +4,7 @@
  * Wraps existing CheckpointManager to provide StrataSystem interface
  * for L1 Git snapshot and worktree operations.
  */
+import { getLogger } from '../../observability/logger.js';
 import { CheckpointManager } from '../checkpoint/manager.js';
 /**
  * ImmutableGitLayer Implementation
@@ -34,7 +35,8 @@ export class ImmutableGitLayerImpl {
             const content = await this.checkpointManager.getSnapshotFileContent(process.cwd(), 'HEAD', path);
             return Buffer.from(content);
         }
-        catch {
+        catch (error) {
+            getLogger().debug(`[ImmutableGitLayer] Failed to get file from snapshot: ${error instanceof Error ? error.message : String(error)}`);
             return null;
         }
     }

package/dist/core/strata/layers/shadow-driver/readonly-lock.js

@@ -129,7 +129,8 @@ async function removeLockByToken(lockPath, expectedToken) {
     try {
         await rename(lockPath, swapPath);
     }
-    catch {
+    catch (error) {
+        getLogger().debug(`[ReadonlyLock] Failed to rename lock file for atomic swap: ${error instanceof Error ? error.message : String(error)}`);
         return false;
     }
     try {
@@ -141,18 +142,19 @@ async function removeLockByToken(lockPath, expectedToken) {
         try {
             await rename(swapPath, lockPath);
         }
-        catch {
-            // Best-effort rollback when concurrent updates happen.
+        catch (rollbackError) {
+            getLogger().debug(`[ReadonlyLock] Failed to rollback lock swap (token mismatch): ${rollbackError instanceof Error ? rollbackError.message : String(rollbackError)}`);
             await unlink(swapPath).catch(() => null);
         }
         return false;
     }
-    catch {
+    catch (error) {
+        getLogger().debug(`[ReadonlyLock] Failed to verify lock token after swap: ${error instanceof Error ? error.message : String(error)}`);
         try {
             await rename(swapPath, lockPath);
         }
-        catch {
-            // Best-effort rollback when concurrent updates happen.
+        catch (rollbackError) {
+            getLogger().debug(`[ReadonlyLock] Failed to rollback lock swap (verification error): ${rollbackError instanceof Error ? rollbackError.message : String(rollbackError)}`);
             await unlink(swapPath).catch(() => null);
         }
         return false;

package/dist/core/strata/layers/shadow-driver/shadow-driver.js

@@ -36,7 +36,8 @@ async function pointsToExpectedDependency(sourcePath, targetPath) {
         ]);
         return arePathsEquivalent(resolvedSourcePath, resolvedTargetPath);
     }
-    catch {
+    catch (error) {
+        getLogger().debug(`[ShadowDriver] Failed to verify dependency path equivalence: ${error instanceof Error ? error.message : String(error)}`);
         return false;
     }
 }

package/dist/core/strata/layers/worktree.js

@@ -5,7 +5,9 @@ import { text } from '../../../locales/index.js';
 import { access, readdir, realpath, rm } from '../../adapters/fs/node-fs.js';
 import { GitAdapter } from '../../adapters/git/git-adapter.js';
 import { getLogger } from '../../observability/logger.js';
+import { errorMessage } from '../../utils/error.js';
 import { isPathWithinDirectory, normalizePath } from '../../utils/path.js';
+import { isRecord } from '../../utils/serialize.js';
 import { detectDependencyPaths } from './shadow-driver/strategy.js';
 function resolveEnvironmentMode(options) {
     return options.environmentMode === 'parity' ? 'parity' : 'strict';
@@ -46,7 +48,8 @@ async function tryRealpath(value) {
     try {
         return await realpath(value);
     }
-    catch {
+    catch (error) {
+        getLogger().debug(`[Worktree] Failed to resolve realpath for ${value}: ${error instanceof Error ? error.message : String(error)}`);
         return null;
     }
 }
@@ -73,14 +76,14 @@ async function removeProjectedWorktreeEntries(workPath) {
         worktreeRealPath = await realpath(workPath);
     }
     catch (error) {
-        throw new Error(`Failed to resolve worktree path before git cleanup (${workPath}): ${error instanceof Error ? error.message : String(error)}`);
+        throw new Error(`Failed to resolve worktree path before git cleanup (${workPath}): ${errorMessage(error)}`);
     }
     let entries = [];
     try {
         entries = (await readdir(workPath, { withFileTypes: true }));
     }
     catch (error) {
-        throw new Error(`Failed to enumerate worktree entries before git cleanup (${workPath}): ${error instanceof Error ? error.message : String(error)}`);
+        throw new Error(`Failed to enumerate worktree entries before git cleanup (${workPath}): ${errorMessage(error)}`);
     }
     for (const entry of entries) {
         const name = entry?.name;
@@ -116,7 +119,7 @@ async function pruneWorktreeDependencyRoots(baseRepoPath, worktreePath) {
             getLogger().debug(`Pruned disposable dependency root before worktree cleanup: ${dependencyRoot}`);
         }
         catch (error) {
-            getLogger().debug(`Failed to prune dependency root before worktree cleanup (${dependencyRoot}): ${error instanceof Error ? error.message : String(error)}`);
+            getLogger().debug(`Failed to prune dependency root before worktree cleanup (${dependencyRoot}): ${errorMessage(error)}`);
         }
     }
 }
@@ -243,7 +246,7 @@ export class WorkspaceManager {
                         return true;
                     }
                     catch (error) {
-                        if (error && typeof error === 'object' && error.code === 'ENOENT') {
+                        if (isRecord(error) && error.code === 'ENOENT') {
                             return false;
                         }
                         return true;
@@ -268,11 +271,7 @@ export class WorkspaceManager {
             }
         }
         catch (error) {
-            const msg = error instanceof Error
-                ? error instanceof Error
-                    ? error.message
-                    : String(error)
-                : String(error);
+            const msg = errorMessage(error);
             onEvent?.({
                 type: 'action.fallback',
                 tool: 'git',

package/dist/core/strata/runtime/environment.js

@@ -20,7 +20,8 @@ async function pathExists(target) {
         await stat(target);
         return true;
     }
-    catch {
+    catch (error) {
+        getLogger().debug(`[RuntimeEnvironment] pathExists check failed for ${target}: ${error instanceof Error ? error.message : String(error)}`);
         return false;
     }
 }