pi-voice 0.1.1 → 0.2.0

package/out/main/index.js

@@ -2,16 +2,39 @@ import { systemPreferences, app, BrowserWindow, ipcMain } from "electron";
 import { fileURLToPath } from "node:url";
 import { uIOhook, UiohookKey } from "uiohook-napi";
 import { join } from "node:path";
+import { homedir } from "node:os";
+import pino from "pino";
 import { readFileSync, existsSync, mkdirSync, createWriteStream, unlinkSync, writeFileSync } from "node:fs";
+import { z } from "zod";
 import { GoogleGenAI } from "@google/genai";
 import OpenAI, { toFile } from "openai";
 import { WhisperFullParams, WhisperSamplingStrategy, Whisper } from "@napi-rs/whisper";
-import { homedir } from "node:os";
 import { Readable } from "node:stream";
 import { finished } from "node:stream/promises";
 import { spawn } from "node:child_process";
 import { createAgentSession, SessionManager } from "@mariozechner/pi-coding-agent";
 import { createServer } from "node:net";
+function resolveLogPath() {
+  const envPath = process.env["PI_VOICE_LOG_PATH"];
+  if (envPath) return envPath;
+  const configHome = process.env["XDG_CONFIG_HOME"] || join(homedir(), ".config");
+  return join(configHome, "pi-voice", "daemon.log");
+}
+const logPath = resolveLogPath();
+const logger = pino(
+  {
+    level: "debug"
+  },
+  pino.multistream([
+    // Console output (human-readable via stdout)
+    { level: "debug", stream: process.stdout },
+    // File output (JSON, auto-creates parent directories)
+    {
+      level: "debug",
+      stream: pino.destination({ dest: logPath, mkdir: true, sync: false })
+    }
+  ])
+);
 function getReleaseCodes(binding) {
   const codes = [binding.keycode];
   if (binding.ctrl) {
@@ -67,14 +90,14 @@ class FnHook {
     });
     uIOhook.start();
     this.started = true;
-    console.log(`[FnHook] Started monitoring ${this.displayName}`);
+    logger.info({ key: this.displayName }, "Started monitoring key");
   }
   stop() {
     if (!this.started) return;
     uIOhook.stop();
     this.started = false;
     this.active = false;
-    console.log("[FnHook] Stopped monitoring");
+    logger.info("Stopped monitoring key");
   }
   get isFnDown() {
     return this.active;
@@ -216,50 +239,68 @@ function formatKeyDisplay(binding) {
 }
 const DEFAULT_KEY_STRING = "meta+shift+i";
 const DEFAULT_PROVIDER = "local";
-const VALID_PROVIDERS = ["local", "gemini", "openai"];
 function defaultConfig() {
+  const binding = parseKeyBinding(DEFAULT_KEY_STRING);
   return {
-    key: parseKeyBinding(DEFAULT_KEY_STRING),
-    keyDisplay: formatKeyDisplay(parseKeyBinding(DEFAULT_KEY_STRING)),
+    key: binding,
+    keyDisplay: formatKeyDisplay(binding),
     provider: DEFAULT_PROVIDER
   };
 }
+const configFileSchema = z.object({
+  key: z.string().refine(
+    (v) => {
+      try {
+        parseKeyBinding(v);
+        return true;
+      } catch {
+        return false;
+      }
+    },
+    { message: "Invalid key binding" }
+  ).optional().default(DEFAULT_KEY_STRING),
+  provider: z.enum(["local", "gemini", "openai"]).optional().default(DEFAULT_PROVIDER)
+});
+class ConfigError extends Error {
+  constructor(configPath, details) {
+    super(`Invalid config at ${configPath}:
+${details}`);
+    this.configPath = configPath;
+    this.details = details;
+    this.name = "ConfigError";
+  }
+}
 function loadConfig(cwd) {
   const configPath = join(cwd, ".pi", "pi-voice.json");
+  let raw;
   try {
-    const raw = readFileSync(configPath, "utf-8");
-    const json = JSON.parse(raw);
-    let binding;
-    let display;
-    if (typeof json.key === "string") {
-      binding = parseKeyBinding(json.key);
-      display = formatKeyDisplay(binding);
-    } else {
-      if (json.key !== void 0) {
-        console.warn(`[Config] "key" must be a string in ${configPath}, using default`);
-      }
-      binding = parseKeyBinding(DEFAULT_KEY_STRING);
-      display = formatKeyDisplay(binding);
-    }
-    let provider = DEFAULT_PROVIDER;
-    if (typeof json.provider === "string") {
-      const p = json.provider.toLowerCase();
-      if (VALID_PROVIDERS.includes(p)) {
-        provider = p;
-      } else {
-        console.warn(`[Config] Unknown provider "${json.provider}" in ${configPath}, using default "${DEFAULT_PROVIDER}"`);
-      }
-    }
-    console.log(`[Config] Loaded config: key=${display}, provider=${provider} from ${configPath}`);
-    return { key: binding, keyDisplay: display, provider };
+    raw = readFileSync(configPath, "utf-8");
   } catch (err) {
     if (err.code === "ENOENT") {
-      console.log(`[Config] No config file found at ${configPath}, using default (${DEFAULT_KEY_STRING})`);
-    } else {
-      console.warn(`[Config] Failed to load ${configPath}, using default:`, err);
+      logger.info({ configPath }, "No config file found, using defaults");
+      return defaultConfig();
     }
-    return defaultConfig();
+    throw new ConfigError(configPath, `Failed to read file: ${err.message}`);
   }
+  let json;
+  try {
+    json = JSON.parse(raw);
+  } catch {
+    throw new ConfigError(configPath, "Invalid JSON syntax");
+  }
+  const result = configFileSchema.safeParse(json);
+  if (!result.success) {
+    const details = result.error.issues.map((issue) => {
+      const path = issue.path.length > 0 ? `"${issue.path.join(".")}"` : "(root)";
+      return `  - ${path}: ${issue.message}`;
+    }).join("\n");
+    throw new ConfigError(configPath, details);
+  }
+  const parsed = result.data;
+  const binding = parseKeyBinding(parsed.key);
+  const display = formatKeyDisplay(binding);
+  logger.info({ key: display, provider: parsed.provider, configPath }, "Loaded config");
+  return { key: binding, keyDisplay: display, provider: parsed.provider };
 }
 const DEFAULT_MODEL = "medium-q5_0";
 const HF_BASE_URL = "https://huggingface.co/ggerganov/whisper.cpp/resolve/main";
@@ -271,7 +312,7 @@ function modelFileName(model) {
 }
 async function downloadModel(model, destPath) {
   const url = `${HF_BASE_URL}/${modelFileName(model)}`;
-  console.log(`[Whisper] Downloading model "${model}" from ${url} ...`);
+  logger.info({ model, url }, "Downloading Whisper model");
   const response = await fetch(url, { method: "GET", redirect: "follow" });
   if (!response.ok) {
     throw new Error(
@@ -323,7 +364,7 @@ async function downloadModel(model, destPath) {
     process.stderr.write("\n");
     const { renameSync } = await import("node:fs");
     renameSync(tmpPath, destPath);
-    console.log(`[Whisper] Model saved to ${destPath}`);
+    logger.info({ destPath }, "Whisper model saved");
   } catch (err) {
     try {
       unlinkSync(tmpPath);
@@ -380,9 +421,9 @@ async function getWhisperInstance() {
   if (whisperInitPromise) return whisperInitPromise;
   whisperInitPromise = (async () => {
     const modelPath = await resolveModelPath();
-    console.log("[STT:local] Loading Whisper model from", modelPath, "...");
+    logger.info({ modelPath }, "Loading Whisper model");
     whisperInstance = new Whisper(modelPath);
-    console.log("[STT:local] Whisper model loaded");
+    logger.info("Whisper model loaded");
     return whisperInstance;
   })();
   return whisperInitPromise;
@@ -447,7 +488,7 @@ async function transcribe(audioData, provider = "local") {
       text = await transcribeGemini(Buffer.from(audioData));
       break;
   }
-  console.log(`[STT:${provider}] Transcribed: "${text}"`);
+  logger.info({ provider, text }, "Transcribed");
   return text;
 }
 let geminiClient = null;
@@ -525,8 +566,9 @@ async function* synthesizeStreamGemini(text) {
     totalBytes += leftover.length;
     yield leftover;
   }
-  console.log(
-    `[TTS:gemini] Streamed ${totalBytes} bytes of PCM audio for "${text.substring(0, 50)}..."`
+  logger.info(
+    { provider: "gemini", totalBytes, text: text.substring(0, 50) },
+    "Streamed PCM audio"
   );
 }
 async function* synthesizeStreamOpenAI(text) {
@@ -549,8 +591,9 @@ async function* synthesizeStreamOpenAI(text) {
     yield chunk;
     offset = end;
   }
-  console.log(
-    `[TTS:openai] Streamed ${totalBytes} bytes of PCM audio for "${text.substring(0, 50)}..."`
+  logger.info(
+    { provider: "openai", totalBytes, text: text.substring(0, 50) },
+    "Streamed PCM audio"
   );
 }
 function speakLocal(text) {
@@ -571,8 +614,9 @@ function speakLocal(text) {
     });
     child.on("close", (code) => {
       if (code === 0) {
-        console.log(
-          `[TTS:local] Spoke "${text.substring(0, 50)}${text.length > 50 ? "..." : ""}"`
+        logger.info(
+          { provider: "local", text: text.substring(0, 50) },
+          "Spoke text"
         );
         resolve();
       } else {
@@ -603,13 +647,13 @@ function setSessionCwd(cwd) {
 }
 async function getOrCreateSession() {
   if (session) return session;
-  console.log(`[PiSession] Creating new agent session (cwd: ${sessionCwd})...`);
+  logger.info({ cwd: sessionCwd }, "Creating new agent session");
   const result = await createAgentSession({
     cwd: sessionCwd,
     sessionManager: SessionManager.inMemory()
   });
   session = result.session;
-  console.log("[PiSession] Session created");
+  logger.info("Agent session created");
   return session;
 }
 async function prompt(text, options) {
@@ -618,7 +662,7 @@ async function prompt(text, options) {
     if (event.type === "message_update" && event.assistantMessageEvent.type === "text_end") {
       const content = event.assistantMessageEvent.content.trim();
       if (content.length > 0) {
-        console.log(`[PiSession] Response: ${content}`);
+        logger.info({ content }, "Agent response");
         options?.onTextEnd?.(content);
       }
     }
@@ -633,7 +677,7 @@ function dispose() {
   if (session) {
     session.dispose();
     session = null;
-    console.log("[PiSession] Session disposed");
+    logger.info("Agent session disposed");
   }
 }
 const IPC = {
@@ -706,7 +750,7 @@ function startDaemonServer(handler) {
     });
   });
   server.listen(socketPath);
-  console.log(`[DaemonIPC] Listening on ${socketPath}`);
+  logger.info({ socketPath }, "DaemonIPC listening");
   return socketPath;
 }
 function stopDaemonServer() {
@@ -721,7 +765,7 @@ function stopDaemonServer() {
     } catch {
     }
   }
-  console.log("[DaemonIPC] Server stopped");
+  logger.info("DaemonIPC server stopped");
 }
 const workingCwd = process.env["PI_VOICE_CWD"] || process.cwd();
 let mainWindow = null;
@@ -730,7 +774,7 @@ let currentState = "idle";
 setSessionCwd(workingCwd);
 function setState(state, message) {
   currentState = state;
-  console.log(`[Main] State: ${state}${message ? ` - ${message}` : ""}`);
+  logger.info({ state, message }, "State changed");
 }
 function createWindow() {
   mainWindow = new BrowserWindow({
@@ -764,7 +808,7 @@ function setupIpcHandlers(provider) {
   ipcMain.on(IPC.RECORDING_DATA, async (_event, data) => {
     if (currentState !== "recording") return;
     if (data.byteLength < 1e3) {
-      console.log("[Main] Recording too short, ignoring");
+      logger.info("Recording too short, ignoring");
       setState("idle", "Recording too short");
       return;
     }
@@ -830,7 +874,7 @@ function setupIpcHandlers(provider) {
       }
     } catch (err) {
       const msg = err instanceof Error ? err.message : String(err);
-      console.error("[Main] Pipeline error:", msg);
+      logger.error({ err: msg }, "Pipeline error");
       setState("error", msg);
       setTimeout(() => {
         if (currentState === "error") setState("idle");
@@ -838,7 +882,7 @@ function setupIpcHandlers(provider) {
     }
   });
   ipcMain.on(IPC.RECORDING_ERROR, (_event, error) => {
-    console.error("[Main] Recording error:", error);
+    logger.error({ err: error }, "Recording error");
     setState("error", error);
     setTimeout(() => {
       if (currentState === "error") setState("idle");
@@ -856,8 +900,9 @@ function setupFnHook(config) {
     {
       onFnDown: () => {
         if (currentState !== "idle") {
-          console.log(
-            `[Main] ${config.keyDisplay} pressed but state is ${currentState}, ignoring`
+          logger.info(
+            { key: config.keyDisplay, state: currentState },
+            "Key pressed but not idle, ignoring"
           );
           return;
         }
@@ -876,7 +921,7 @@ function setupFnHook(config) {
     fnHook.start();
   } catch (err) {
     const msg = err instanceof Error ? err.message : String(err);
-    console.error("[Main] FnHook error:", msg);
+    logger.error({ err: msg }, "FnHook error");
     setState("error", msg);
   }
 }
@@ -900,7 +945,7 @@ function handleDaemonCommand(command) {
   }
 }
 function gracefulShutdown() {
-  console.log("[Main] Shutting down...");
+  logger.info("Shutting down...");
   fnHook?.stop();
   dispose();
   stopDaemonServer();
@@ -912,17 +957,28 @@ process.on("SIGTERM", () => {
 });
 const gotLock = app.requestSingleInstanceLock();
 if (!gotLock) {
-  console.log("[Main] Another instance is already running. Exiting.");
+  logger.warn("Another instance is already running. Exiting.");
   app.quit();
 }
 app.whenReady().then(async () => {
-  const config = loadConfig(workingCwd);
+  let config;
+  try {
+    config = loadConfig(workingCwd);
+  } catch (err) {
+    if (err instanceof ConfigError) {
+      logger.error({ err: err.message }, "Config error");
+    } else {
+      logger.error({ err: err instanceof Error ? err.message : String(err) }, "Failed to load config");
+    }
+    app.quit();
+    return;
+  }
   if (config.provider === "local") {
     try {
       await resolveModelPath();
     } catch (err) {
       const msg = err instanceof Error ? err.message : String(err);
-      console.error("[Main] Failed to prepare Whisper model:", msg);
+      logger.error({ err: msg }, "Failed to prepare Whisper model");
       app.quit();
       return;
     }
@@ -932,7 +988,7 @@ app.whenReady().then(async () => {
   setupFnHook(config);
   startDaemonServer(handleDaemonCommand);
   saveRuntimeState(workingCwd);
-  console.log(`[Main] pi-voice daemon started (cwd: ${workingCwd})`);
+  logger.info({ cwd: workingCwd }, "pi-voice daemon started");
 });
 app.on("window-all-closed", () => {
 });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-voice",
-  "version": "0.1.1",
+  "version": "0.2.0",
   "description": "Voice interface for pi coding agent",
   "author": "Yuku Kotani",
   "license": "MIT",
@@ -37,7 +37,9 @@
     "@napi-rs/whisper": "^0.0.4",
     "electron": "^40.2.1",
     "openai": "^6.10.0",
-    "uiohook-napi": "^1.5.4"
+    "pino": "^10.3.1",
+    "uiohook-napi": "^1.5.4",
+    "zod": "^4.3.6"
   },
   "trustedDependencies": [
     "electron",