From 0553e6ec54b17ac6a2d357193739575e0a6529ec Mon Sep 17 00:00:00 2001
From: Junhyuk Lee <xodn348@users.noreply.github.com>
Date: Tue, 7 Apr 2026 15:38:29 -0500
Subject: [PATCH] fix(#60): fix: replace empty catch blocks with meaningful
 error handling

This fix was generated with AI assistance (Anthropic Claude) and reviewed by a human.
---
 src/common/token-storage.ts | 90 ++++---------------------------------
 1 file changed, 8 insertions(+), 82 deletions(-)

diff --git a/src/common/token-storage.ts b/src/common/token-storage.ts
index d08b88b..3906de1 100644
--- a/src/common/token-storage.ts
+++ b/src/common/token-storage.ts
@@ -1,86 +1,12 @@
-import fs from "node:fs";
-import os from "node:os";
-import path from "node:path";
-import { decryptToken, encryptToken } from "./encryption.js";
-
-const DIR_NAME = "linearis";
-const LEGACY_DIR_NAME = ".linearis";
-const TOKEN_FILE = "token";
-
-export function getTokenDir(): string {
-  if (process.platform === "linux") {
-    const xdgConfig = process.env.XDG_CONFIG_HOME;
-    if (xdgConfig && path.isAbsolute(xdgConfig)) {
-      return path.join(xdgConfig, DIR_NAME);
-    }
-    return path.join(os.homedir(), ".config", DIR_NAME);
-  }
-  return path.join(os.homedir(), LEGACY_DIR_NAME);
-}
-
-function getLegacyTokenDir(): string {
-  return path.join(os.homedir(), LEGACY_DIR_NAME);
-}
-
-function getTokenPath(): string {
-  return path.join(getTokenDir(), TOKEN_FILE);
-}
-
-function getLegacyTokenPath(): string {
-  return path.join(getLegacyTokenDir(), TOKEN_FILE);
-}
-
-export function ensureTokenDir(): void {
-  const dir = getTokenDir();
-  if (!fs.existsSync(dir)) {
-    fs.mkdirSync(dir, { recursive: true, mode: 0o700 });
-  } else {
-    fs.chmodSync(dir, 0o700);
-  }
-}
-
-export function saveToken(token: string): void {
-  ensureTokenDir();
-  const tokenPath = getTokenPath();
-  const encrypted = encryptToken(token);
-  fs.writeFileSync(tokenPath, encrypted, "utf8");
-  fs.chmodSync(tokenPath, 0o600);
-}
-
-export function getStoredToken(): string | null {
-  const tokenPath = getTokenPath();
-  if (!fs.existsSync(tokenPath)) {
-    // on linux, fall back to legacy ~/.linearis/token
-    if (process.platform === "linux") {
-      const legacy = getLegacyTokenPath();
-      if (fs.existsSync(legacy)) {
-        try {
-          const encrypted = fs.readFileSync(legacy, "utf8").trim();
-          return decryptToken(encrypted);
-        } catch {
-          return null;
-        }
-      }
-    }
-    return null;
-  }
   try {
     const encrypted = fs.readFileSync(tokenPath, "utf8").trim();
     return decryptToken(encrypted);
-  } catch {
-    return null;
-  }
-}
-
-export function clearToken(): void {
-  const tokenPath = getTokenPath();
-  if (fs.existsSync(tokenPath)) {
-    fs.unlinkSync(tokenPath);
-  }
-  if (process.platform === "linux") {
-    const legacy = getLegacyTokenPath();
-    if (fs.existsSync(legacy)) {
-      fs.unlinkSync(legacy);
+  } catch (error) {
+    // ENOENT = race between existsSync and readFileSync — treat as missing
+    if ((error as NodeJS.ErrnoException).code !== "ENOENT") {
+      process.stderr.write(
+        `Warning: failed to decrypt stored token: ${error instanceof Error ? error.message : String(error)}\n`,
+      );
     }
-  }
-}
+    return null;
+  }
\ No newline at end of file