feat(performance): added reactquery hooks for workflow operations, for logs, fixed logs reloading, fix subscription UI

[waleedlatif1]

Summary

• added reactquery hooks for workflow operations,
• added reactquery hooks for logs,
  • fixed logs reloading
• fix subscription UI bug

Type of Change

• Bug fix
• New feature

Testing

Tested manually

Checklist

• Code follows project style guidelines
• Self-reviewed my changes
• Tests added/updated and passing
• No new warnings introduced
• I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Preview	Comments	Updated (UTC)
docs	Skipped			Nov 16, 2025 0:39am

[greptile-apps]

Greptile Overview

Greptile Summary

This PR migrates workflow and logs operations to React Query for better caching and performance, adds workflow import functionality, and fixes subscription UI styling.

Key Changes

• Migrated workflow/folder/logs data fetching to React Query hooks with proper cache management
• Added comprehensive workflow import supporting JSON and ZIP files with folder structure preservation
• Removed 200+ lines of manual fetch logic from logs component
• Fixed subscription button styling to remove problematic hover transitions

Critical Issue

• Pagination bug in useLogsList hook (logs.ts:150-155): The hook always replaces logs instead of appending when loading more pages, breaking infinite scroll functionality. The setLogs call needs to check filters.page > 1 to determine if logs should be appended.

Confidence Score: 3/5

• This PR has a critical pagination bug that will break logs infinite scroll
• Score reflects one critical bug in logs pagination that will cause functional issues. The infinite scroll will not work correctly - when users scroll to load more logs, page 2+ will replace page 1 instead of appending. The rest of the changes are well-structured React Query migrations with proper patterns.
• apps/sim/hooks/queries/logs.ts requires immediate attention to fix pagination logic

Important Files Changed

File Analysis

Filename	Score	Overview
apps/sim/hooks/queries/logs.ts	2/5	Added React Query hooks for logs fetching. Critical pagination bug: logs are always replaced instead of appended for infinite scroll.
apps/sim/hooks/queries/workflows.ts	5/5	Added React Query hooks for workflow operations with proper optimistic updates and cache invalidation.
apps/sim/hooks/queries/folders.ts	5/5	Added React Query hooks for folder CRUD operations with proper cache management.
apps/sim/app/workspace/[workspaceId]/logs/logs.tsx	3/5	Refactored to use React Query hooks, removing manual fetch logic. Pagination logic relies on broken useLogsList hook.
apps/sim/app/workspace/[workspaceId]/w/hooks/use-import-workflow.ts	5/5	Implemented comprehensive workflow import supporting single/multiple JSON files and ZIP archives with folder structure preservation.

Sequence Diagram

sequenceDiagram
    participant User
    participant LogsPage as Logs Component
    participant ReactQuery as React Query
    participant LogsHook as useLogsList Hook
    participant Store as Filter Store
    participant API as /api/logs

    User->>LogsPage: Scrolls to bottom
    LogsPage->>LogsPage: loadMoreLogs() called
    LogsPage->>Store: setPage(page + 1)
    Store-->>LogsPage: page updated
    Note over LogsPage,ReactQuery: Page change triggers query refetch
    ReactQuery->>LogsHook: Query refetch with new page
    LogsHook->>API: GET /api/logs?page=2&limit=50
    API-->>LogsHook: Returns {logs: [...], hasMore: true}
    LogsHook->>Store: setLogs(logs) - BUG: replaces instead of appends!
    Store-->>LogsPage: Logs replaced (should append)
    LogsPage->>User: Shows only page 2 logs (page 1 lost)

Loading

[greptile-apps]

_{20 files reviewed, 1 comment}

_{Edit Code Review Agent Settings | Greptile}