aevgarik/paperclip
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #266 from mvanhorn/fix/89-run-log-location

Browse Source 
fix(server): use home-based path for run logs instead of cwd
This commit is contained in:
Dotta
2026-03-07 21:15:56 -06:00
committed by GitHub
parent 0fedd8a395 3f2274cd8d
commit 61551ffea3
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
server/src/services/run-log-store.ts
View File

@@ -2,6 +2,7 @@ import { createReadStream, promises as fs } from "node:fs";
import path from "node:path"; import path from "node:path";
import { createHash } from "node:crypto"; import { createHash } from "node:crypto";
import { notFound } from "../errors.js"; import { notFound } from "../errors.js";
import { resolvePaperclipInstanceRoot } from "../home-paths.js";
export type RunLogStoreType = "local_file"; export type RunLogStoreType = "local_file";
@@ -148,7 +149,7 @@ let cachedStore: RunLogStore | null = null;
export function getRunLogStore() { export function getRunLogStore() {
if (cachedStore) return cachedStore; if (cachedStore) return cachedStore;
const basePath = process.env.RUN_LOG_BASE_PATH ?? path.resolve(process.cwd(), "data/run-logs"); const basePath = process.env.RUN_LOG_BASE_PATH ?? path.resolve(resolvePaperclipInstanceRoot(), "data", "run-logs");
cachedStore = createLocalFileRunLogStore(basePath); cachedStore = createLocalFileRunLogStore(basePath);
return cachedStore; return cachedStore;
} }