Skip to content

Conversation

@waleedlatif1
Copy link
Collaborator

Summary

  • increase workflow logs limit from 1k to 5k per workflow

Type of Change

  • 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
Copy link

vercel bot commented Feb 11, 2026

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
docs Skipped Skipped Feb 11, 2026 4:10am

Request Review

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 11, 2026

Greptile Overview

Greptile Summary

This PR increases the per-workflow terminal console retention cap (MAX_ENTRIES_PER_WORKFLOW) from 1,000 to 5,000 in apps/sim/stores/terminal/console/store.ts, allowing longer execution log histories to be kept in the persisted console store. The trimming logic remains the same; only the cap value changes, which will increase the amount of data stored per workflow in IndexedDB via the store’s persist middleware.

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk.
  • The PR only changes a single constant (MAX_ENTRIES_PER_WORKFLOW) and leaves the trimming/redaction/persistence logic unchanged; the functional impact is limited to retaining more console entries per workflow as intended.
  • No files require special attention

Important Files Changed

Filename Overview
apps/sim/stores/terminal/console/store.ts Increases MAX_ENTRIES_PER_WORKFLOW from 1000 to 5000; no other logic changes.

Sequence Diagram

sequenceDiagram
  participant UI as UI/Execution events
  participant Store as useTerminalConsoleStore
  participant Persist as persist(indexedDBStorage)
  participant Notify as useNotificationStore

  UI->>Store: addConsole(entry)
  Store->>Store: redactApiKeys (if applicable)
  Store->>Store: prepend entry
  Store->>Store: trim to MAX_ENTRIES_PER_WORKFLOW (per workflow)
  Store->>Persist: persist updated entries
  alt entry has error
    Store->>Notify: addNotification(error)
  end
Loading

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 file reviewed, no comments

Edit Code Review Agent Settings | Greptile

@waleedlatif1 waleedlatif1 merged commit 8a24b56 into staging Feb 11, 2026
7 checks passed
@waleedlatif1 waleedlatif1 deleted the feat/terminal branch February 11, 2026 04:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant