@fink-andreas/pi-linear-tools 0.2.1 → 0.4.0

package/src/logger.js

@@ -1,11 +1,50 @@
 /**
- * Structured logging module
+ * Structured logging module (file-first, TUI-safe)
  */
 
+import { mkdirSync, appendFileSync } from 'node:fs';
+import { dirname, join } from 'node:path';
+
 const LOG_LEVELS = ['debug', 'info', 'warn', 'error'];
 let currentLevel = process.env.LOG_LEVEL || 'info';
 let quietMode = false;
 
+const LOG_TO_CONSOLE = String(process.env.PI_LINEAR_TOOLS_LOG_TO_CONSOLE || '').toLowerCase() === 'true';
+const DEFAULT_LOG_FILE = process.env.PI_LINEAR_TOOLS_LOG_FILE
+  || join(process.env.HOME || process.cwd(), '.config', 'pi-linear-tools', 'pi-linear-tools.log');
+
+let logFileReady = false;
+let logFilePath = DEFAULT_LOG_FILE;
+
+function ensureLogFileReady() {
+  if (logFileReady) return;
+  try {
+    mkdirSync(dirname(logFilePath), { recursive: true });
+  } catch {
+    // ignore; fallback handled in writeLogLine
+  }
+  logFileReady = true;
+}
+
+function writeLogLine(line, isError = false) {
+  try {
+    ensureLogFileReady();
+    appendFileSync(logFilePath, `${line}\n`, { encoding: 'utf8' });
+  } catch {
+    // Last-resort fallback is disabled by default to protect TUI.
+    // Only print when explicitly opted in.
+    if (LOG_TO_CONSOLE) {
+      if (isError) console.error(line);
+      else console.log(line);
+    }
+  }
+
+  if (LOG_TO_CONSOLE) {
+    if (isError) console.error(line);
+    else console.log(line);
+  }
+}
+
 /**
  * Enable quiet mode (suppress info/debug/warn, keep only errors)
  */
@@ -35,7 +74,7 @@ function getTimestamp() {
  */
 function maskValue(key, value) {
   const sensitiveKeys = ['apiKey', 'token', 'password', 'secret', 'LINEAR_API_KEY'];
-  if (sensitiveKeys.some(sk => key.toLowerCase().includes(sk.toLowerCase()))) {
+  if (sensitiveKeys.some((sk) => key.toLowerCase().includes(sk.toLowerCase()))) {
     return '***masked***';
   }
   return value;
@@ -50,7 +89,7 @@ function formatLog(level, message, data = {}) {
     timestamp,
     level: level.toUpperCase(),
     message,
-    ...data
+    ...data,
   };
   return JSON.stringify(entry);
 }
@@ -60,7 +99,7 @@ function formatLog(level, message, data = {}) {
  */
 export function debug(message, data = {}) {
   if (shouldLog('debug')) {
-    console.log(formatLog('debug', message, data));
+    writeLogLine(formatLog('debug', message, data));
   }
 }
 
@@ -69,7 +108,7 @@ export function debug(message, data = {}) {
  */
 export function info(message, data = {}) {
   if (shouldLog('info')) {
-    console.log(formatLog('info', message, data));
+    writeLogLine(formatLog('info', message, data));
   }
 }
 
@@ -78,7 +117,7 @@ export function info(message, data = {}) {
  */
 export function warn(message, data = {}) {
   if (shouldLog('warn')) {
-    console.log(formatLog('warn', message, data));
+    writeLogLine(formatLog('warn', message, data));
   }
 }
 
@@ -87,7 +126,7 @@ export function warn(message, data = {}) {
  */
 export function error(message, data = {}) {
   if (shouldLog('error')) {
-    console.error(formatLog('error', message, data));
+    writeLogLine(formatLog('error', message, data), true);
   }
 }
 
@@ -95,13 +134,7 @@ export function error(message, data = {}) {
  * Print startup banner
  */
 export function printBanner() {
-  const banner = `
-╔════════════════════════════════════════════════════════════╗
-║               pi-linear-tools                             ║
-║     Pi extension tools for Linear SDK workflows           ║
-╚════════════════════════════════════════════════════════════╝
-`;
-  console.log(banner);
+  info('pi-linear-tools startup');
 }
 
 /**
@@ -110,8 +143,8 @@ export function printBanner() {
 export function logConfig(config) {
   info('Configuration loaded', {
     ...Object.fromEntries(
-      Object.entries(config).map(([key, value]) => [key, maskValue(key, value)])
-    )
+      Object.entries(config).map(([key, value]) => [key, maskValue(key, value)]),
+    ),
   });
 }
 
@@ -126,3 +159,10 @@ export function setLogLevel(level) {
     warn(`Invalid log level: ${level}. Using: ${currentLevel}`);
   }
 }
+
+/**
+ * Expose active log file path for diagnostics/tests
+ */
+export function getLogFilePath() {
+  return logFilePath;
+}

package/src/settings.js

@@ -7,6 +7,7 @@ import { mkdir, readFile, writeFile } from 'node:fs/promises';
 import { existsSync } from 'node:fs';
 import { dirname, join } from 'node:path';
 import { debug, warn, error as logError } from './logger.js';
+import { OAUTH_CLIENT_ID } from './auth/constants.js';
 
 export function getDefaultSettings() {
   return {
@@ -14,7 +15,7 @@ export function getDefaultSettings() {
     authMethod: 'api-key', // 'api-key' or 'oauth'
     apiKey: null, // Legacy API key (migrated from linearApiKey)
     oauth: {
-      clientId: 'a3e177176c6697611367f1a2405d4a34',
+      clientId: OAUTH_CLIENT_ID,
       redirectUri: 'http://localhost:34711/callback',
     },
     defaultTeam: null,
@@ -49,7 +50,7 @@ function migrateSettings(settings) {
     // Add OAuth config
     if (!migrated.oauth || typeof migrated.oauth !== 'object') {
       migrated.oauth = {
-        clientId: 'a3e177176c6697611367f1a2405d4a34',
+        clientId: OAUTH_CLIENT_ID,
         redirectUri: 'http://localhost:34711/callback',
       };
     }
@@ -73,14 +74,14 @@ function migrateSettings(settings) {
   // Ensure oauth config exists
   if (!migrated.oauth || typeof migrated.oauth !== 'object') {
     migrated.oauth = {
-      clientId: 'a3e177176c6697611367f1a2405d4a34',
+      clientId: OAUTH_CLIENT_ID,
       redirectUri: 'http://localhost:34711/callback',
     };
   }
 
   // Ensure oauth has clientId and redirectUri
   if (migrated.oauth.clientId === undefined) {
-    migrated.oauth.clientId = 'a3e177176c6697611367f1a2405d4a34';
+    migrated.oauth.clientId = OAUTH_CLIENT_ID;
   }
 
   if (migrated.oauth.redirectUri === undefined) {

package/src/shared.js

@@ -0,0 +1,112 @@
+/**
+ * Shared utilities for pi-linear-tools entry points
+ *
+ * Common functions used by both CLI and extension entry points.
+ */
+
+import fs from 'node:fs';
+import path from 'node:path';
+import { pathToFileURL } from 'node:url';
+
+/**
+ * Check if a directory is a pi-coding-agent root
+ * @param {string} dir - Directory path to check
+ * @returns {boolean}
+ */
+export function isPiCodingAgentRoot(dir) {
+  const pkgPath = path.join(dir, 'package.json');
+  if (!fs.existsSync(pkgPath)) return false;
+  try {
+    const pkg = JSON.parse(fs.readFileSync(pkgPath, 'utf8'));
+    return pkg?.name === '@mariozechner/pi-coding-agent';
+  } catch {
+    return false;
+  }
+}
+
+/**
+ * Find the pi-coding-agent root directory
+ * @returns {string|null}
+ */
+export function findPiCodingAgentRoot() {
+  const entry = process.argv?.[1];
+  if (!entry) return null;
+
+  // Method 1: walk up from argv1 (works when argv1 is .../pi-coding-agent/dist/cli.js)
+  {
+    let dir = path.dirname(entry);
+    for (let i = 0; i < 20; i += 1) {
+      if (isPiCodingAgentRoot(dir)) {
+        return dir;
+      }
+      const parent = path.dirname(dir);
+      if (parent === dir) break;
+      dir = parent;
+    }
+  }
+
+  // Method 2: npm global layout guess (works when argv1 is .../<prefix>/bin/pi)
+  // <prefix>/bin/pi  ->  <prefix>/lib/node_modules/@mariozechner/pi-coding-agent
+  {
+    const binDir = path.dirname(entry);
+    const prefix = path.resolve(binDir, '..');
+    const candidate = path.join(prefix, 'lib', 'node_modules', '@mariozechner', 'pi-coding-agent');
+    if (isPiCodingAgentRoot(candidate)) {
+      return candidate;
+    }
+  }
+
+  // Method 3: common global node_modules locations
+  for (const candidate of [
+    '/usr/local/lib/node_modules/@mariozechner/pi-coding-agent',
+    '/usr/lib/node_modules/@mariozechner/pi-coding-agent',
+  ]) {
+    if (isPiCodingAgentRoot(candidate)) {
+      return candidate;
+    }
+  }
+
+  return null;
+}
+
+/**
+ * Import from pi-coding-agent root
+ * @param {string} relativePathFromPiRoot - Path relative to pi root
+ * @returns {Promise<any>}
+ */
+export async function importFromPiRoot(relativePathFromPiRoot) {
+  const piRoot = findPiCodingAgentRoot();
+
+  if (!piRoot) throw new Error('Unable to locate @mariozechner/pi-coding-agent installation');
+
+  const absPath = path.join(piRoot, relativePathFromPiRoot);
+  return import(pathToFileURL(absPath).href);
+}
+
+/**
+ * Parse command line arguments string into tokens
+ * @param {string} argsString - Arguments string to parse
+ * @returns {string[]}
+ */
+export function parseArgs(argsString) {
+  if (!argsString || !argsString.trim()) return [];
+  const tokens = argsString.match(/"[^"]*"|'[^']*'|\S+/g) || [];
+  return tokens.map((t) => {
+    if ((t.startsWith('"') && t.endsWith('"')) || (t.startsWith("'") && t.endsWith("'"))) {
+      return t.slice(1, -1);
+    }
+    return t;
+  });
+}
+
+/**
+ * Read a flag value from command line arguments
+ * @param {string[]} args - Parsed arguments
+ * @param {string} flag - Flag name
+ * @returns {string|undefined}
+ */
+export function readFlag(args, flag) {
+  const idx = args.indexOf(flag);
+  if (idx >= 0 && idx + 1 < args.length) return args[idx + 1];
+  return undefined;
+}