chore: remove useless __esModule

[tomerqodo]

Benchmark PR from qodo-benchmark#277

[greptile-apps]

Greptile Summary

This PR removes the __esModule: true property from 161 vi.mock() calls across 79 test files in the web frontend. The __esModule property was historically needed for Jest when mocking ES modules to indicate that the mock should be treated as an ES module with a default export. However, Vitest (which this project uses) handles ES modules natively and doesn't require this flag - it automatically detects and handles default exports correctly.

The changes are purely cleanup, removing redundant boilerplate that serves no functional purpose in Vitest. All mocks continue to work identically after the removal since Vitest's module mocking system treats { default: ... } exports correctly without needing the explicit __esModule marker.

Confidence Score: 5/5

• This PR is safe to merge with no risk
• The changes are mechanical and consistent across all 79 files, removing an obsolete property that Vitest doesn't require. The __esModule flag was needed for Jest but is redundant in Vitest's native ES module handling. All mocks use the { default: ... } pattern which Vitest handles correctly without the flag. This is a pure cleanup refactor with no functional impact.
• No files require special attention

Important Files Changed

Filename	Overview
web/tests/workflow-parallel-limit.test.tsx	removed unnecessary __esModule: true from vi.mock call
web/app/components/app-sidebar/dataset-info/index.spec.tsx	removed unnecessary __esModule: true from vi.mock call
web/app/components/app/annotation/index.spec.tsx	removed unnecessary __esModule: true from vi.mock call
web/app/components/app/configuration/config/index.spec.tsx	removed 8 unnecessary __esModule: true properties from multiple vi.mock calls
web/app/components/billing/billing-page/index.spec.tsx	removed unnecessary __esModule: true from vi.mock call
web/app/components/datasets/create/index.spec.tsx	removed unnecessary __esModule: true from vi.mock call
web/app/components/base/file-uploader/utils.spec.ts	removed unnecessary __esModule: true from vi.mock call
web/context/modal-context.test.tsx	removed unnecessary __esModule: true from vi.mock call

Sequence Diagram

sequenceDiagram
    participant Dev as Developer
    participant Test as Test Files (79 files)
    participant Vitest as Vitest Mock System
    participant Module as Module Loader

    Note over Dev,Module: Before: With __esModule flag
    Dev->>Test: vi.mock('module', { __esModule: true, default: ... })
    Test->>Vitest: Register mock with __esModule flag
    Vitest->>Module: Load mock (flag ignored, not needed)
    Module-->>Vitest: Return mocked module
    
    Note over Dev,Module: After: Without __esModule flag
    Dev->>Test: vi.mock('module', { default: ... })
    Test->>Vitest: Register mock without flag
    Vitest->>Module: Load mock (auto-detects ES module)
    Module-->>Vitest: Return mocked module (identical behavior)
    
    Note over Dev,Test: Result: 161 __esModule properties removed<br/>No functional change in test execution

Loading