Revert "fix: use stable per-project tmp log directories (#9)" (#10)

This reverts commit 2f71f86.

Author: kuitos

README.md

@@ -174,7 +174,7 @@ Yes. Set `OPENCODE_MEMORY_AUTODREAM=0`. You can also tune gates with:
 
 ### Logs
 
-Logs are written to `/tmp/opencode-claude-memory/<project-hash>/`:
+Logs are written to `$TMPDIR/opencode-memory-logs/`:
 - `extract-*.log`: automatic memory extraction
 - `dream-*.log`: auto-dream consolidation
 

bin/opencode-memory

@@ -204,7 +204,13 @@ AUTODREAM_STALE_LOCK_SECS=$((60 * 60))
 
 WORKING_DIR="${OPENCODE_MEMORY_DIR:-$(pwd)}"
 
-LOG_BASE_DIR="/tmp/opencode-claude-memory"
+TMP_BASE_DIR="${TMPDIR:-/tmp}"
+while [ "$TMP_BASE_DIR" != "/" ] && [ "${TMP_BASE_DIR%/}" != "$TMP_BASE_DIR" ]; do
+  TMP_BASE_DIR="${TMP_BASE_DIR%/}"
+done
+if [ -z "$TMP_BASE_DIR" ]; then
+  TMP_BASE_DIR="/"
+fi
 
 # Scope lock files at project root granularity (not per-subdirectory).
 PROJECT_SCOPE_DIR="$WORKING_DIR"
@@ -215,7 +221,7 @@ fi
 PROJECT_KEY="$(printf '%s' "$PROJECT_SCOPE_DIR" | cksum | awk '{print $1}')"
 
 # Lock files (prevent concurrent work on the same project)
-LOCK_DIR="/tmp/opencode-memory-locks"
+LOCK_DIR="$TMP_BASE_DIR/opencode-memory-locks"
 mkdir -p "$LOCK_DIR"
 EXTRACT_LOCK_FILE="$LOCK_DIR/${PROJECT_KEY}.extract.lock"
 
@@ -224,7 +230,7 @@ mkdir -p "$STATE_DIR"
 CONSOLIDATION_LOCK_FILE="$STATE_DIR/${PROJECT_KEY}.consolidate-lock"
 
 # Logs
-LOG_DIR="$LOG_BASE_DIR/${PROJECT_KEY}"
+LOG_DIR="$TMP_BASE_DIR/opencode-memory-logs"
 mkdir -p "$LOG_DIR"
 TASK_LOG_PREFIX="$(date +%Y%m%d-%H%M%S)-${PROJECT_KEY}"
 EXTRACT_LOG_FILE="$LOG_DIR/extract-${TASK_LOG_PREFIX}.log"

test/opencode-memory.test.ts

@@ -1,11 +1,10 @@
 import { afterEach, describe, expect, test } from "bun:test"
-import { chmodSync, existsSync, mkdtempSync, mkdirSync, readFileSync, readdirSync, rmSync, statSync, writeFileSync } from "fs"
+import { chmodSync, existsSync, mkdtempSync, mkdirSync, readFileSync, readdirSync, rmSync, writeFileSync } from "fs"
 import { tmpdir } from "os"
-import { dirname, join } from "path"
+import { join } from "path"
 import { spawnSync } from "child_process"
 
 const tempRoots: string[] = []
-const tempLogDirs = new Set<string>()
 const scriptPath = join(process.cwd(), "bin", "opencode-memory")
 
 function makeTempRoot(): string {
@@ -19,42 +18,17 @@ function writeExecutable(filePath: string, content: string): void {
   chmodSync(filePath, 0o755)
 }
 
-function rememberLogDirFromPath(logPath: string): void {
-  tempLogDirs.add(dirname(logPath))
-}
-
-function findRecentExtractLog(startedAt: number): string {
-  const logRoot = join("/tmp", "opencode-claude-memory")
-  const projectDirs = readdirSync(logRoot).map((entry) => join(logRoot, entry))
-
-  const extractLogs = projectDirs.flatMap((dir) =>
-    readdirSync(dir)
-      .filter((name) => name.startsWith("extract-"))
-      .map((name) => join(dir, name)),
-  )
-
-  const recentLog = extractLogs.find((logPath) => statSync(logPath).mtimeMs >= startedAt)
-  expect(recentLog).toBeDefined()
-
-  return recentLog ?? ""
-}
-
 afterEach(() => {
   while (tempRoots.length > 0) {
     const root = tempRoots.pop()
     if (root) {
       rmSync(root, { recursive: true, force: true })
     }
   }
-
-  for (const logDir of tempLogDirs) {
-    rmSync(logDir, { recursive: true, force: true })
-  }
-  tempLogDirs.clear()
 })
 
 describe("opencode-memory wrapper", () => {
-  test("writes extraction logs under a stable per-project /tmp directory", () => {
+  test("normalizes TMPDIR before composing extraction log paths", () => {
     const root = makeTempRoot()
     const fakeBin = join(root, "bin")
     const homeDir = join(root, "home")
@@ -86,7 +60,6 @@ exit 0
 `,
     )
 
-    const startedAt = Date.now()
     const result = spawnSync("bash", [scriptPath, "--help"], {
       cwd: root,
       encoding: "utf-8",
@@ -109,10 +82,7 @@ exit 0
     expect(logPathMatch).not.toBeNull()
 
     const logPath = logPathMatch?.[1].trim() ?? ""
-    rememberLogDirFromPath(logPath)
-    expect(logPath).toMatch(/^\/tmp\/opencode-claude-memory\/[^/]+\/extract-/)
-    expect(statSync(logPath).mtimeMs).toBeGreaterThanOrEqual(startedAt)
-    expect(logPath).toContain("/extract-")
+    expect(logPath.startsWith(join(root, "tmp", "opencode-memory-logs", "extract-"))).toBe(true)
     expect(existsSync(logPath)).toBe(true)
     expect(readFileSync(logPath, "utf-8")).toContain("extraction ok")
   })
@@ -149,7 +119,6 @@ exit 0
 `,
     )
 
-    const startedAt = Date.now()
     const result = spawnSync("bash", [scriptPath, "--help"], {
       cwd: root,
       encoding: "utf-8",
@@ -168,8 +137,11 @@ exit 0
     expect(result.status).toBe(0)
     expect(result.stderr).toBe("")
 
-    const logPath = findRecentExtractLog(startedAt)
-    rememberLogDirFromPath(logPath)
+    const logDir = join(root, "tmp", "opencode-memory-logs")
+    const logFiles = readdirSync(logDir)
+    expect(logFiles).toHaveLength(1)
+
+    const logPath = join(logDir, logFiles[0] ?? "")
     expect(readFileSync(logPath, "utf-8")).toContain("extraction ok")
   })
 })