Repository navigation
ci: main の lockfile が動いたら SDK 更新の PR を積み直す - #4460
Merged
Merged
Conversation
衝突した PR には pull_request のワークフローが起きないので、日次の定時だけだと、 main の lockfile が動いた日は次の定時まで SDK の PR にチェックが1本も付かずに止まる。
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
この PR は takecchi 本人ではなく、クローン teto の委譲で動くマネージャー層(mgr-52dc46a6)が書いた。
なぜ
SDK 更新の PR(
automation/claude-agent-sdk)は、日次の定時(0 22 * * *)にだけ積み直される。日中に main のpnpm-lock.yamlが動くと、開いている SDK の PR は main と衝突する。衝突した PR にはpull_requestのワークフローが起きないので、次の定時までチェックが1本も付かずに止まる。no checks reportedのまま止まっていた。手でworkflow_dispatchして積み直すと、MERGEABLE になって必須チェックが走ったpnpm-lock.yamlを動かしたのは4回(chore: update dependency ink to v8 #4371・chore: update non-major dependencies #4369・feat: 添付の中身を S3 互換ストレージへ置けるようにする(#4128 段2) #4309・chore(web): api-client が core を取り込まず import のまま残す(Web の合計 -1,857 B) #3855)何を変えたか
update-claude-sdk.ymlのon:に、main への push でpnpm-lock.yaml/pnpm-workspace.yamlが動いたときを足した。changed=trueになり、いつもどおり検証して積み直すchanged=falseで検証も PR も飛ばして終わる。SDK の PR をマージした直後の push もこちらに当たるconcurrency: update-claude-sdk(前を殺さない)で順番に回る費用
積み直すたびに、検証(
verify-for-sdk-pr.sh)が1回走る(実測では1回 15〜25 分)。上の頻度なら、週に数回ぶん増える。PR が無い日は、install と版の確認だけで終わる。確かめたこと