Skip to content

補助担当の順位を依頼元の製品へ合わせる - #111

Open
mashfromband wants to merge 1 commit into
mainfrom
fix/todo2353-child-ownership
Open

mashfromband wants to merge 1 commit into
mainfrom
fix/todo2353-child-ownership

Conversation

@mashfromband

@mashfromband mashfromband commented Oct 11, 2026 •

Copy link
Copy Markdown
Contributor

製品担当の子レビューが独立したToDoを持たないとき、補助ツールの作業フォルダーを根拠に情報システム部へ昇格する問題を修正します。子担当の所有分類を稼働中の依頼元へたどり、製品の順位を維持します。

独立したToDoを持つ担当と有限の独立subworkは、その所有分類を保持します。終了・取り消し済みの親へはたどらず、循環も打ち切ります。実割当で確認したRe:X担当100・補助レビュー30の逆転を、合成データの回帰テストに追加しました。

ToDo2353のPR109後の実割当確認で発見しました。関連検査と別モデルレビューを実行中です。PCの空きメモリが13.1GiBで共有ビルドゲートの16GiB条件を満たさないため、ローカル検査は資源の空きを待っています。


Note

Medium Risk
Changes scheduling priority logic used by the job queue; wrong inheritance could reorder production work, but scope is limited to rank resolution for parent/child jobs.

Overview
Fixes job priority for queued child jobs that have no linked ToDo: they no longer pick up the helper/tool project rank (e.g. cc-ask → 30) when they should follow the active parent’s product.

job_ranks now resolves rank in a loop—use linked ToDo ranks when present, otherwise walk parent_job_id through queued/starting/running parents (skip finished/cancelled, stop on cycles), while independent subwork and jobs with their own ToDo still use their own classification.

Adds regression test helper_directory_does_not_promote_a_product_review_to_it for Re:X parent (100), inherited review child (100), SocialPilot-owned child (10), and independent subwork (30).

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

@cursor

cursor Bot commented Oct 11, 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: b4a09a57-b277-4ec0-a0fb-4679115c539d)

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