From 540c2c2f497e8c67916d6436cc46d343b9159894 Mon Sep 17 00:00:00 2001 From: Number531 <120485065+Number531@users.noreply.github.com> Date: Fri, 6 Mar 2026 02:35:26 -0500 Subject: [PATCH] fix: align codeExecutionBridge env checks with featureFlags defaults MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The bridge reads process.env directly (self-contained module) but used === 'true' (opt-in) while featureFlags.js defaults to true (opt-out). When env vars are unset, the bridge silently disabled all three features. Changed to !== 'false' for CODE_EXECUTION_BRIDGE, FILES_API_CHART_EXTRACTION, and CHART_PERSISTENCE — now defaults match featureFlags.js. Explicit =false still disables. Co-Authored-By: Claude Opus 4.6 --- super-legal-mcp-refactored/src/tools/codeExecutionBridge.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/super-legal-mcp-refactored/src/tools/codeExecutionBridge.js b/super-legal-mcp-refactored/src/tools/codeExecutionBridge.js index 89c07def9..90df8a297 100644 --- a/super-legal-mcp-refactored/src/tools/codeExecutionBridge.js +++ b/super-legal-mcp-refactored/src/tools/codeExecutionBridge.js @@ -32,7 +32,7 @@ const OVERALL_TIMEOUT_MS = Number(process.env.CODE_EXECUTION_TIMEOUT_MS) || 300_ * @returns {boolean} */ export function isCodeExecutionBridgeEnabled() { - return process.env.CODE_EXECUTION_BRIDGE === 'true'; + return process.env.CODE_EXECUTION_BRIDGE !== 'false'; } /** @@ -41,7 +41,7 @@ export function isCodeExecutionBridgeEnabled() { * @returns {boolean} */ export function isFilesApiEnabled() { - return process.env.FILES_API_CHART_EXTRACTION === 'true'; + return process.env.FILES_API_CHART_EXTRACTION !== 'false'; } // ─── System Prompt ────────────────────────────────────────────────────────── @@ -130,7 +130,7 @@ async function downloadFilesApiCharts(client, fileIds, finalResult) { * @param {object} finalResult - Result object with charts array */ export async function persistChartsToDisk(finalResult) { - if (process.env.CHART_PERSISTENCE !== 'true' || finalResult.charts.length === 0) return; + if (process.env.CHART_PERSISTENCE === 'false' || finalResult.charts.length === 0) return; const sessionDir = process.env.CURRENT_SESSION_DIR; if (!sessionDir) return;