From b632a0d946dbb268c38435c181db38237517de80 Mon Sep 17 00:00:00 2001 From: Saravana Achu Mac Date: Sat, 4 Apr 2026 18:28:35 -0700 Subject: [PATCH] refactor(backend): resolve strategy preset legacy client inside repository Made-with: Cursor --- backend/src/services/apiServer.ts | 4 ++-- backend/src/services/strategyPresetRepository.ts | 15 +++++++-------- 2 files changed, 9 insertions(+), 10 deletions(-) diff --git a/backend/src/services/apiServer.ts b/backend/src/services/apiServer.ts index d54ae69..d966d0e 100644 --- a/backend/src/services/apiServer.ts +++ b/backend/src/services/apiServer.ts @@ -1990,7 +1990,7 @@ export class ApiServer { this.app.get('/api/marketplace-presets', this.requireAuth, async (_req, res) => { try { - const presets = await listStrategyPresets(supabaseService); + const presets = await listStrategyPresets(); res.json({ presets }); } catch (error: any) { res.status(500).json({ error: `Failed to load marketplace presets: ${error.message}` }); @@ -2009,7 +2009,7 @@ export class ApiServer { ...(req.body || {}), created_by: authUserId }; - await createStrategyPreset(payload, supabaseService); + await createStrategyPreset(payload); res.status(201).json({ success: true }); } catch (error: any) { res.status(400).json({ error: `Failed to publish preset: ${error.message}` }); diff --git a/backend/src/services/strategyPresetRepository.ts b/backend/src/services/strategyPresetRepository.ts index 4214695..3837867 100644 --- a/backend/src/services/strategyPresetRepository.ts +++ b/backend/src/services/strategyPresetRepository.ts @@ -2,9 +2,8 @@ import { getContainer } from '@bytelyst/cosmos'; import { randomUUID } from 'node:crypto'; import { config } from '../config/index.js'; import logger from '../utils/logger.js'; -import type { supabaseService } from './SupabaseService.js'; +import { supabaseService } from './SupabaseService.js'; -type LegacySupabaseService = typeof supabaseService; const PRESET_CONTAINER = 'strategy_presets'; interface StrategyPresetDocument { @@ -19,11 +18,11 @@ function isCosmosConfigured(): boolean { return Boolean(config.COSMOS_ENDPOINT && config.COSMOS_KEY); } -function getClient(legacyService?: LegacySupabaseService) { - return legacyService?.getClient?.() ?? null; +function getLegacyClient() { + return supabaseService.getClient(); } -export async function listStrategyPresets(legacyService?: LegacySupabaseService): Promise { +export async function listStrategyPresets(): Promise { if (isCosmosConfigured()) { try { const container = getContainer(PRESET_CONTAINER); @@ -43,7 +42,7 @@ export async function listStrategyPresets(legacyService?: LegacySupabaseService) } } - const client = getClient(legacyService); + const client = getLegacyClient(); if (!client) return []; try { @@ -78,7 +77,7 @@ export async function listStrategyPresets(legacyService?: LegacySupabaseService) } } -export async function createStrategyPreset(payload: Record, legacyService?: LegacySupabaseService): Promise { +export async function createStrategyPreset(payload: Record): Promise { if (isCosmosConfigured()) { const container = getContainer(PRESET_CONTAINER); await container.items.upsert({ @@ -90,7 +89,7 @@ export async function createStrategyPreset(payload: Record, leg }); } - const client = getClient(legacyService); + const client = getLegacyClient(); if (!client) { return; }