pi-langfuse 1.3.2 → 1.3.3

package/README.md

@@ -78,7 +78,7 @@ Run any `pi` command with the extension loaded. On first run without configurati
 2. **Langfuse secret key** — starts with `sk-lf-...`
 3. **Langfuse host** — defaults to `https://cloud.langfuse.com`
 
-The extension saves these to a local `config.json` (ignored by git).
+The extension saves these to `~/.pi/agent/pi-langfuse/config.json`, so package updates and reinstalls do not overwrite your Langfuse credentials.
 
 To re-run setup at any time:
 
@@ -86,7 +86,7 @@ To re-run setup at any time:
 /langfuse-setup
 ```
 
-### Method 2: Environment variables
+### Method 2: Environment variables (fallback)
 
 Set these before starting Pi:
 
@@ -96,11 +96,11 @@ export LANGFUSE_SECRET_KEY="sk-lf-xxxx"
 export LANGFUSE_BASE_URL="https://cloud.langfuse.com"  # optional; LANGFUSE_HOST is also supported
 ```
 
-The extension checks the local `config.json` first, then falls back to environment variables.
+The saved config file takes precedence. Environment variables are used only when `~/.pi/agent/pi-langfuse/config.json` does not exist or is incomplete, which avoids drift after re-running `/langfuse-setup`.
 
-### Method 3: Local config.json (development only)
+### Method 3: Persistent config.json
 
-For local development, create a `config.json` in the project root:
+For persistent local configuration, create or update `~/.pi/agent/pi-langfuse/config.json`:
 
 ```json
 {
@@ -110,7 +110,7 @@ For local development, create a `config.json` in the project root:
 }
 ```
 
-> **⚠️ Security**: `config.json` is not tracked by git. Never commit API keys to version control.
+> **⚠️ Security**: Keep `~/.pi/agent/pi-langfuse/config.json` private. Never commit API keys to version control.
 
 ## Usage
 
@@ -160,7 +160,6 @@ pi-langfuse/
 ├── index.ts            # Extension entrypoint and core logic
 ├── package.json        # Package metadata
 ├── tsconfig.json       # TypeScript configuration
-├── config.json         # Local credentials (git-ignored)
 ├── types/
 │   ├── pi-coding-agent.d.ts   # Pi extension API types
 │   └── node-shims.d.ts        # Node.js module shims

package/README_CN.md

@@ -78,7 +78,7 @@ pi link /path/to/pi-langfuse
 2. **Langfuse 密钥** — 以 `sk-lf-...` 开头
 3. **Langfuse 主机地址** — 默认为 `https://cloud.langfuse.com`
 
-扩展会将这些保存到本地的 `config.json`（被 git 忽略）。
+扩展会将这些保存到 `~/.pi/agent/pi-langfuse/config.json`，这样 Pi 更新、重装扩展时不会覆盖你的 Langfuse 凭据。
 
 随时重新运行设置：
 
@@ -86,7 +86,7 @@ pi link /path/to/pi-langfuse
 /langfuse-setup
 ```
 
-### 方式 2：环境变量
+### 方式 2：环境变量（兜底）
 
 在启动 Pi 前设置：
 
@@ -96,11 +96,11 @@ export LANGFUSE_SECRET_KEY="sk-lf-xxxx"
 export LANGFUSE_BASE_URL="https://cloud.langfuse.com"  # 可选；也支持 LANGFUSE_HOST
 ```
 
-扩展会先检查本地的 `config.json`，然后回退到环境变量。
+保存的配置文件优先级更高。只有当 `~/.pi/agent/pi-langfuse/config.json` 不存在或不完整时，扩展才会使用环境变量，这样重新运行 `/langfuse-setup` 后不会出现配置漂移。
 
-### 方式 3：本地 config.json（仅限开发）
+### 方式 3：持久化 config.json
 
-对于本地开发，在项目根目录创建 `config.json`：
+如需使用持久化本地配置，创建或更新 `~/.pi/agent/pi-langfuse/config.json`：
 
 ```json
 {
@@ -110,7 +110,7 @@ export LANGFUSE_BASE_URL="https://cloud.langfuse.com"  # 可选；也支持 LANG
 }
 ```
 
-> **⚠️ 安全提醒**：`config.json` 不会被 git 跟踪。切勿将 API 密钥提交到版本控制。
+> **⚠️ 安全提醒**：请保护好 `~/.pi/agent/pi-langfuse/config.json`。切勿将 API 密钥提交到版本控制。
 
 ## 使用
 
@@ -160,7 +160,6 @@ pi-langfuse/
 ├── index.ts            # 扩展入口和核心逻辑
 ├── package.json        # 包元数据
 ├── tsconfig.json       # TypeScript 配置
-├── config.json         # 本地凭据（git 忽略）
 ├── types/
 │   ├── pi-coding-agent.d.ts   # Pi 扩展 API 类型
 │   └── node-shims.d.ts        # Node.js 模块 shims

package/index.ts

@@ -11,7 +11,7 @@ import { basename } from "node:path";
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 
 import { state, resetRunState } from "./src/state.js";
-import { ensureConfig, promptForConfig, loadConfigFromEnv, loadConfigFromFile } from "./src/config.js";
+import { ensureConfig, promptForConfig, loadConfig } from "./src/config.js";
 import { shutdownRuntime } from "./src/langfuse.js";
 import { getMessageFromEvent, extractAssistantOutput } from "./src/utils.js";
 import { startAgentRun, finishAgentRun } from "./src/handlers/agent.js";
@@ -35,7 +35,7 @@ import {
 
 export default async function (pi: ExtensionAPI) {
   if (!state.config) {
-    state.config = loadConfigFromEnv() || loadConfigFromFile();
+    state.config = loadConfig();
   }
 
   if (state.config) {

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "pi-langfuse",
-	"version": "1.3.2",
+	"version": "1.3.3",
 	"description": "Langfuse extension for Pi coding agent",
 	"repository": {
 		"type": "git",

package/src/config.ts

@@ -1,6 +1,6 @@
-import { readFileSync, existsSync, writeFileSync } from "node:fs";
+import { mkdirSync, readFileSync, existsSync, writeFileSync } from "node:fs";
 import type { Config } from "./types.js";
-import { CONFIG_PATH, DEFAULT_LANGFUSE_HOST } from "./constants.js";
+import { CONFIG_DIR, CONFIG_PATH, DEFAULT_LANGFUSE_HOST } from "./constants.js";
 import { state } from "./state.js";
 import { shutdownRuntime } from "./langfuse.js";
 
@@ -38,7 +38,12 @@ export function loadConfigFromEnv(): Config | null {
   };
 }
 
+export function loadConfig(): Config | null {
+  return loadConfigFromFile() || loadConfigFromEnv();
+}
+
 export function saveConfig(config: Config) {
+  mkdirSync(CONFIG_DIR, { recursive: true });
   writeFileSync(CONFIG_PATH, `${JSON.stringify(config, null, 2)}\n`, "utf-8");
 }
 
@@ -79,7 +84,7 @@ async function saveConfigFromUI(ctx: any, config: Config): Promise<boolean> {
     return true;
   } catch (error) {
     console.warn("📊 Langfuse: Failed to save config.json", error);
-    ctx.ui.notify("Failed to save Langfuse config.json. Check extension directory permissions.", "error");
+    ctx.ui.notify(`Failed to save Langfuse config.json to ${CONFIG_PATH}. Check Pi config directory permissions.`, "error");
     state.config = null;
     return false;
   }
@@ -87,7 +92,7 @@ async function saveConfigFromUI(ctx: any, config: Config): Promise<boolean> {
 
 export async function ensureConfig(ctx: any): Promise<boolean> {
   if (!state.config) {
-    state.config = loadConfigFromEnv() || loadConfigFromFile();
+    state.config = loadConfig();
   }
 
   if (state.config) {
@@ -114,7 +119,7 @@ export async function promptForConfig(ctx: any): Promise<boolean> {
 
   const config = await collectConfigFromUI(ctx, "Manual setup requested");
   if (!config) {
-    state.config = loadConfigFromEnv() || loadConfigFromFile();
+    state.config = loadConfig();
     return false;
   }
 

package/src/constants.ts

@@ -1,11 +1,8 @@
-import { resolve, dirname } from "node:path";
-import { fileURLToPath } from "node:url";
+import { homedir } from "node:os";
+import { resolve } from "node:path";
 
-// Since we are in src/constants.ts, EXT_DIR should point to the parent of src/ (the root of the extension)
-// We use `import.meta.url` which points to `src/constants.ts`.
-// `dirname` gives us `src/`, and `resolve(..., '..')` gives us the root.
-export const EXT_DIR = resolve(dirname(fileURLToPath(import.meta.url)), "..");
-export const CONFIG_PATH = resolve(EXT_DIR, "config.json");
+export const CONFIG_DIR = resolve(homedir(), ".pi", "agent", "pi-langfuse");
+export const CONFIG_PATH = resolve(CONFIG_DIR, "config.json");
 export const DEFAULT_LANGFUSE_HOST = "https://cloud.langfuse.com";
 
 export const MAX_STRING_LENGTH = 12_000;