Skip to content

Commit b2dcb2a

Browse files
waleedlatif1Sg312
authored andcommitted
feat(performance): added reactquery hooks for workflow operations, for logs, fixed logs reloading, fix subscription UI (#2017)
* feat(performance): added reactquery hooks for workflow operations, for logs, fixed logs reloading, fix subscription UI * use useInfiniteQuery for logs fetching
1 parent d2eb33a commit b2dcb2a

File tree

25 files changed

+1222
-1470
lines changed

25 files changed

+1222
-1470
lines changed

apps/sim/app/api/workflows/route.ts

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -118,18 +118,18 @@ export async function POST(req: NextRequest) {
118118

119119
logger.info(`[${requestId}] Creating workflow ${workflowId} for user ${session.user.id}`)
120120

121-
// Track workflow creation
122-
try {
123-
const { trackPlatformEvent } = await import('@/lib/telemetry/tracer')
124-
trackPlatformEvent('platform.workflow.created', {
125-
'workflow.id': workflowId,
126-
'workflow.name': name,
127-
'workflow.has_workspace': !!workspaceId,
128-
'workflow.has_folder': !!folderId,
121+
import('@/lib/telemetry/tracer')
122+
.then(({ trackPlatformEvent }) => {
123+
trackPlatformEvent('platform.workflow.created', {
124+
'workflow.id': workflowId,
125+
'workflow.name': name,
126+
'workflow.has_workspace': !!workspaceId,
127+
'workflow.has_folder': !!folderId,
128+
})
129+
})
130+
.catch(() => {
131+
// Silently fail
129132
})
130-
} catch (_e) {
131-
// Silently fail
132-
}
133133

134134
await db.insert(workflow).values({
135135
id: workflowId,

apps/sim/app/workspace/[workspaceId]/logs/components/sidebar/sidebar.tsx

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@ import '@/components/emcn/components/code/code.css'
2323
interface LogSidebarProps {
2424
log: WorkflowLog | null
2525
isOpen: boolean
26+
isLoadingDetails?: boolean
2627
onClose: () => void
2728
onNavigateNext?: () => void
2829
onNavigatePrev?: () => void
@@ -192,6 +193,7 @@ const BlockContentDisplay = ({
192193
export function Sidebar({
193194
log,
194195
isOpen,
196+
isLoadingDetails = false,
195197
onClose,
196198
onNavigateNext,
197199
onNavigatePrev,
@@ -219,15 +221,6 @@ export function Sidebar({
219221
}
220222
}, [log?.id])
221223

222-
const isLoadingDetails = useMemo(() => {
223-
if (!log) return false
224-
// Only show while we expect details to arrive (has executionId)
225-
if (!log.executionId) return false
226-
const hasEnhanced = !!log.executionData?.enhanced
227-
const hasAnyDetails = hasEnhanced || !!log.cost || Array.isArray(log.executionData?.traceSpans)
228-
return !hasAnyDetails
229-
}, [log])
230-
231224
const formattedContent = useMemo(() => {
232225
if (!log) return null
233226

0 commit comments

Comments
 (0)