improvement(logs): update logs export route to respect filters

[waleedlatif1]

Summary

• update logs export route to respect filters, previously it ignored some of the filters like time range

Type of Change

• Bug fix

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	Review	Updated (UTC)
docs	Skipped		Dec 23, 2025 6:11pm

[greptile-apps]

Greptile Summary

Fixed logs export to respect time range and other filters by extracting shared filter logic into a centralized module. The export route previously had filter parameters defined but didn't apply time range filters correctly, causing exports to ignore user-selected time windows.

Key changes:

• Created lib/logs/filters.ts with shared LogFilterParamsSchema and buildFilterConditions() to eliminate code duplication
• Refactored export route to use shared filter builders, now correctly applies time range, cost, duration, and all other filters
• Updated logs list route to use shared filter logic while preserving its advanced level filtering for running/pending states
• Fixed frontend to pass time range as startDate parameter to export endpoint
• Moved getStartDateFromTimeRange() utility to shared module for consistency

Confidence Score: 5/5

• This PR is safe to merge with minimal risk
• Well-structured refactoring that centralizes filter logic, fixing a clear bug where exports ignored time range filters. Code follows established patterns, maintains backward compatibility, and the shared filter module improves maintainability. No breaking changes or security concerns.
• No files require special attention

Important Files Changed

Filename	Overview
apps/sim/lib/logs/filters.ts	new shared filter module centralizing log filter logic with comprehensive validation and condition builders
apps/sim/app/api/logs/export/route.ts	refactored to use shared filter module, now correctly applies all filters including time range, cost, and duration
apps/sim/app/workspace/[workspaceId]/logs/logs.tsx	now passes time range filter to export URL, fixing the bug where exports ignored time filters

Sequence Diagram

sequenceDiagram
    participant User
    participant LogsUI as Logs UI
    participant ExportAPI as /api/logs/export
    participant ListAPI as /api/logs
    participant Filters as lib/logs/filters
    participant DB as Database

    Note over User,DB: Log Export Flow (Fixed)
    User->>LogsUI: Click export with filters
    LogsUI->>LogsUI: getStartDateFromTimeRange(timeRange)
    LogsUI->>ExportAPI: GET /api/logs/export?startDate=...&level=...&workflowIds=...
    ExportAPI->>Filters: LogFilterParamsSchema.parse(params)
    ExportAPI->>Filters: buildFilterConditions(params)
    Filters-->>ExportAPI: SQL conditions (with time range!)
    ExportAPI->>DB: SELECT with all filters applied
    DB-->>ExportAPI: Filtered logs
    ExportAPI-->>User: CSV file download

    Note over User,DB: Log List Flow
    User->>ListAPI: GET /api/logs?startDate=...&level=...
    ListAPI->>Filters: LogFilterParamsSchema.parse(params)
    ListAPI->>ListAPI: Build advanced level conditions (running/pending)
    ListAPI->>Filters: buildFilterConditions(params, {useSimpleLevelFilter: false})
    Filters-->>ListAPI: SQL conditions (excluding level)
    ListAPI->>DB: SELECT with combined conditions
    DB-->>ListAPI: Filtered logs
    ListAPI-->>User: JSON response

Loading