fix(qqbot): support QQBOT_DATA_DIR environment variable

Phymath · Phymath · commit cb6be91e4f74 · 2026-05-08T08:55:13.000+08:00
Allow users to customize QQBot data directory by setting QQBOT_DATA_DIR environment variable. Falls back to ~/.openclaw/qqbot when not set. Closes [Bug]：#39461
diff --git a/docs/channels/qqbot.md b/docs/channels/qqbot.md
@@ -67,6 +67,7 @@ Default-account env vars:
 
 - `QQBOT_APP_ID`
 - `QQBOT_CLIENT_SECRET`
+- `QQBOT_DATA_DIR` (optional): custom data directory path for QQBot storage. Defaults to `~/.openclaw/qqbot`.
 
 File-backed AppSecret:
 
diff --git a/extensions/qqbot/src/engine/utils/platform.test.ts b/extensions/qqbot/src/engine/utils/platform.test.ts
@@ -1,13 +1,83 @@
 import fs from "node:fs";
 import os from "node:os";
 import path from "node:path";
-import { afterEach, describe, expect, it, vi } from "vitest";
+import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
 import {
   getHomeDir,
+  getQQBotDataDir,
+  getQQBotDataPath,
   resolveQQBotLocalMediaPath,
   resolveQQBotPayloadLocalFilePath,
 } from "./platform.js";
 
+describe("QQBot data directory with QQBOT_DATA_DIR", () => {
+  const createdPaths: string[] = [];
+  let originalDataDir: string | undefined;
+
+  beforeEach(() => {
+    originalDataDir = process.env.QQBOT_DATA_DIR;
+  });
+
+  afterEach(() => {
+    vi.restoreAllMocks();
+    if (originalDataDir === undefined) {
+      delete process.env.QQBOT_DATA_DIR;
+    } else {
+      process.env.QQBOT_DATA_DIR = originalDataDir;
+    }
+    for (const target of createdPaths.splice(0)) {
+      fs.rmSync(target, { recursive: true, force: true });
+    }
+  });
+
+  it("uses QQBOT_DATA_DIR when set", () => {
+    const customDir = fs.mkdtempSync(path.join(os.tmpdir(), "qqbot-custom-data-"));
+    createdPaths.push(customDir);
+    process.env.QQBOT_DATA_DIR = customDir;
+
+    const dataDir = getQQBotDataDir();
+    expect(dataDir).toBe(customDir);
+    expect(fs.existsSync(dataDir)).toBe(true);
+  });
+
+  it("uses QQBOT_DATA_DIR with subpaths", () => {
+    const customDir = fs.mkdtempSync(path.join(os.tmpdir(), "qqbot-custom-data-"));
+    createdPaths.push(customDir);
+    process.env.QQBOT_DATA_DIR = customDir;
+
+    const sessionsDir = getQQBotDataDir("sessions", "test");
+    expect(sessionsDir).toBe(path.join(customDir, "sessions", "test"));
+    expect(fs.existsSync(sessionsDir)).toBe(true);
+  });
+
+  it("falls back to ~/.openclaw/qqbot when QQBOT_DATA_DIR is not set", () => {
+    delete process.env.QQBOT_DATA_DIR;
+    const homeDir = getHomeDir();
+    const expectedDir = path.join(homeDir, ".openclaw", "qqbot");
+
+    const dataDir = getQQBotDataDir();
+    expect(dataDir).toBe(expectedDir);
+  });
+
+  it("expands tilde in QQBOT_DATA_DIR", () => {
+    const homeDir = getHomeDir();
+    const customDir = path.join(homeDir, "custom-qqbot-data");
+    process.env.QQBOT_DATA_DIR = "~/custom-qqbot-data";
+
+    const dataDir = getQQBotDataPath();
+    expect(dataDir).toBe(customDir);
+  });
+
+  it("returns path without creating when using getQQBotDataPath", () => {
+    const customDir = path.join(os.tmpdir(), "qqbot-not-created-yet");
+    process.env.QQBOT_DATA_DIR = customDir;
+
+    const dataPath = getQQBotDataPath("not-created", "subdir");
+    expect(dataPath).toBe(path.join(customDir, "not-created", "subdir"));
+    expect(fs.existsSync(dataPath)).toBe(false);
+  });
+});
+
 describe("qqbot local media path remapping", () => {
   const createdPaths: string[] = [];
 
diff --git a/extensions/qqbot/src/engine/utils/platform.ts b/extensions/qqbot/src/engine/utils/platform.ts
@@ -39,12 +39,21 @@ export function getHomeDir(): string {
   return getPlatformAdapter().getTempDir();
 }
 
-/** Return a path under `~/.openclaw/qqbot` without creating it. */
+/** Return the base QQBot data directory, respecting QQBOT_DATA_DIR env var. */
+function getQQBotDataBaseDir(): string {
+  const envDataDir = process.env.QQBOT_DATA_DIR;
+  if (envDataDir) {
+    return normalizePath(envDataDir);
+  }
+  return path.join(getHomeDir(), ".openclaw", "qqbot");
+}
+
+/** Return a path under `~/.openclaw/qqbot` (or QQBOT_DATA_DIR) without creating it. */
 export function getQQBotDataPath(...subPaths: string[]): string {
-  return path.join(getHomeDir(), ".openclaw", "qqbot", ...subPaths);
+  return path.join(getQQBotDataBaseDir(), ...subPaths);
 }
 
-/** Return a path under `~/.openclaw/qqbot`, creating it on demand. */
+/** Return a path under `~/.openclaw/qqbot` (or QQBOT_DATA_DIR), creating it on demand. */
 export function getQQBotDataDir(...subPaths: string[]): string {
   const dir = getQQBotDataPath(...subPaths);
   if (!fs.existsSync(dir)) {
