Skip to content

オーナーの返事を反映するresume-のジョブをreq-と同じ最上位で起動する - #38

Open
mashfromband wants to merge 1 commit into
mainfrom
fix/resume-tier-top
Open

mashfromband wants to merge 1 commit into
mainfrom
fix/resume-tier-top

Conversation

@mashfromband

@mashfromband mashfromband commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

概要

オーナーの Discord の返事・新しい指示・停止作業の再開を既存のセッションへ反映する resume- のジョブを、req- と同じ最上位の順位にします。オーナーが答えたのに作業が進まない時間をなくすためです。

変更

  1. queue_priority で resume- を req- と同じ順位 0 にする(同じ順位の中は従来どおり待ち時間の長い順)
  2. lowest_priority_todo_ids(ToDo 18・23)の指定は従来どおり名前より優先する
  3. README の順位の説明を更新

テスト

owner_then_shepherd_then_other_jobs_use_fifo_within_each_class に resume- を追加しました。修正前は失敗し、修正後は通ることを確認しています。

手元では cargo nextest run --locked --workspace が 1186 件すべて成功し、cargo clippy -p cc-jobs --all-targets -D warnings と cargo fmt の確認も通りました。


Note

Low Risk
Small, localized queue ordering change with a parity test; no auth or data-path impact.

Overview
Owner reply / session resume jobs (resume-, e.g. Discord answers reflected into an existing session) now share top priority (tier 0) with direct owner requests (req-), ahead of shepherd (shep-) and other work. Within that tier, FIFO by queue time is unchanged.

queue_priority treats names starting with resume- like req-. lowest_priority_todo_ids / normal_priority_todo_ids still override name-based priority. README documents the updated ordering.

Parity test owner_then_shepherd_then_other_jobs_use_fifo_within_each_class now asserts a resume- job runs between older and newer req- jobs when slots allow.

Reviewed by Cursor Bugbot for commit 5bdf527. Bugbot is set up for automated code reviews on this repo. Configure here.

resume- jobs reflect an owner's Discord answer or new instruction into an
existing session. Waiting behind ordinary work left the owner's reply
unacted on.
@cursor

cursor Bot commented Oct 8, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: cd11c4cf-5c62-4cb8-807a-7a9e87b37661)

@mashfromband

Copy link
Copy Markdown
Contributor Author

別モデルによる読み取りレビューの結果

判定: Must なし

確認した点:

  1. queue_priority で resume- が req- と同じ順位 0 になっている
  2. should_yield でも、同じ順位どうしは平等に扱われる
  3. resume- のジョブは、cc-ask がオーナーの Discord の返事を受けたときだけ起動し、ジョブ自身は作らない
  4. req- と resume- は同じ順位の中で待ち時間の長い順に並ぶので、待ちっぱなしにはならない
  5. テストは、待ち時間の順に req- と resume- が交互に起動することを確かめていて、意味のある検査になっている
  6. ほかで req- を参照している cc-task-title は題名の抽出だけで、順位には関係しない

Suggestion・Nit はありません。

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