Explicit runIds bulk cancel silently excludes runs older than seven days

未關閉
#5,110 1 則留言 0 個 reaction 已指派 0 人 在 GitHub 檢視

維護者通常 1 天內回覆

還沒有人認領這個 Issue。

  • #5111 來自 @granbellllc-png —— 已關閉,未合併
  • #5112 來自 @almagas —— 已關閉,未合併

評估

難度
3/5
預估耗時
1-2 天
新手友好度
35/100
Issue 類型
缺陷
描述清晰度
描述清楚
活躍度
停滯
技術堆疊
next.js, typescript
領域
api, backend

研究方向

Start with convertRunListInputOptionsToFilterRunsOptions in apps/webapp/app/services/runsRepository/runsRepository.server.ts and its call to timeFilters; compare it with freezeRunListFilters in apps/webapp/app/v3/services/bulk/BulkActionV2.server.ts. Add regression coverage for count and paginated selection of explicit IDs at T−30 days and T−1 day, while confirming filter-only calls keep the seven-day default. Two linked pull requests are already closed, so check their changes and discussion before taking this on.

由索引模型根據 Issue 內容生成。

描述

SDK 4.8.3, Trigger.dev Cloud. Observed during a real cancellation; hosted backend revision unknown.

Reproduce

  1. Select two cancellable runs in one environment: one created within seven days, one still DELAYED and older than seven days.
  2. Call runs.bulk.cancel({ runIds: [recentRunId, oldDelayedRunId] }) and poll to completion.
  3. Retrieve the older run directly.

Expected: both explicit IDs are selected. The documentation says the seven-day default applies only to filter selections.

Actual: only recent IDs contribute to counts.total/success; failure is zero. The omitted older run remains DELAYED and directly retrievable. No time filter was supplied. We stopped on the count mismatch.

Matching public source: at bdcbc61ab2871f7cc81f29ec36e42c8676ff6aa9, freezeRunListFilters leaves explicit IDs unbounded. However, convertRunListInputOptionsToFilterRunsOptions calls timeFilters without dates, reintroducing its 7d default for both count and batch selection. This source behavior matches the Cloud observation; its deployed revision is not established.

Suggested regression: count and paginated selection include explicit IDs at T−30 days and T−1 day; filter-only calls without a range retain the seven-day default.

主要語言
TypeScript
星號
16.5k
分支
1.5k
平均合併
1 天 15 小時
30 天內合併 PR
17

環境準備

這個專案沒有提供開發容器、Dockerfile 或貢獻指南,環境需要你自己搭建:先看它的 README,通用步驟見我們的新手貢獻指南。

從這裡開始

  1. 先讀完整個 Issue,再讀專案的貢獻指南。
  2. 在 Issue 下留言說明你要接手 —— 這能避免兩個人做同樣的事。
  3. Fork 儲存庫,在一個分支上完成修改。
  4. 送出 Pull Request,並在描述裡引用這個 Issue 編號。

triggerdotdev/trigger.dev 的其他 Issue

查看 triggerdotdev/trigger.dev 的全部 Issue

相似的 Issue

更多 TypeScript Issue

把新 issue 寄到你的電子郵件信箱

精選適合新手參與的 GitHub issue 摘要。