From 60d4fcfe7e7f4f37c18a10c645c2f9873570cddb Mon Sep 17 00:00:00 2001 From: k0valik <85703878+k0valik@users.noreply.github.com> Date: Tue, 24 Mar 2026 21:40:08 +0100 Subject: [PATCH] update the comments MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit **1. `open-sse/transformer/responsesTransformer.ts`** - Removed `import { resolveDataDir } from "../../src/lib/dataPaths"` - Restored: `typeof process !== "undefined" ? process.cwd() : "."` - Added history comment: `// previous: const baseDir = logsDir || resolveDataDir(); — reverted in #555 for Workers compat` **2. `open-sse/config/credentialLoader.ts`** - Updated JSDoc with `resolveDataDir()` description - Added history: `previous: Priority: DATA_DIR env → ./data (project root)` --- open-sse/config/credentialLoader.ts | 7 +++++-- open-sse/transformer/responsesTransformer.ts | 1 + 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/open-sse/config/credentialLoader.ts b/open-sse/config/credentialLoader.ts index 4e133ec687..26306ecd3f 100644 --- a/open-sse/config/credentialLoader.ts +++ b/open-sse/config/credentialLoader.ts @@ -33,8 +33,11 @@ function credGlobals(): CredGlobals { } /** - * Resolve the path to provider-credentials.json - * Priority: DATA_DIR env → ./data (project root) + * Resolves the path to provider-credentials.json using the application's + * data directory. Delegates to resolveDataDir() which handles DATA_DIR env, + * platform-specific defaults, and fallback logic. + * + * previous: Priority: DATA_DIR env → ./data (project root) */ function resolveCredentialsPath() { return join(resolveDataDir(), "provider-credentials.json"); diff --git a/open-sse/transformer/responsesTransformer.ts b/open-sse/transformer/responsesTransformer.ts index 0d20255682..16c33cb73f 100644 --- a/open-sse/transformer/responsesTransformer.ts +++ b/open-sse/transformer/responsesTransformer.ts @@ -40,6 +40,7 @@ export function createResponsesLogger(model, logsDir = null) { const timestamp = new Date().toISOString().replace(/[:.]/g, "").slice(0, 15); const uniqueId = Math.random().toString(36).slice(2, 8); const baseDir = logsDir || (typeof process !== "undefined" ? process.cwd() : "."); + // previous: const baseDir = logsDir || resolveDataDir(); — reverted in #555 for Workers compat const logDir = path.join(baseDir, "logs", `responses_${model}_${timestamp}_${uniqueId}`); try {