Explicit runIds bulk cancel silently excludes runs older than seven days

オープン
#5,110 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る

メンテナーはふだん 1 日以内に返信

まだ誰も着手していません。

  • #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時間
マージ済み PR(30日)
17

環境構築

このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

triggerdotdev/trigger.dev のほかの issue

triggerdotdev/trigger.dev の issue をすべて見る

似ている issue

TypeScript の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。