Skip to content

ci: update-dev-tools の cron を週次化し誤フォールバックを除去 - #754

Merged
keito4 merged 1 commit into
mainfrom
fix/update-dev-tools-followup
May 18, 2026
Merged

ci: update-dev-tools の cron を週次化し誤フォールバックを除去#754
keito4 merged 1 commit into
mainfrom
fix/update-dev-tools-followup

Conversation

@keito4

@keito4 keito4 commented May 18, 2026

Copy link
Copy Markdown
Owner

Summary

ADR 0006 で残課題としていた update-dev-tools.yml の 2 件を解消します。

1. cron を毎日 → 毎週月曜に変更

  • '0 5 * * *''0 5 * * 1'
  • DevContainer 再ビルドのコストと patch 追従の必要性を勘案し、Dependabot と同じ週次サイクルに揃える。

2. 1Password CLI 取得元の誤フォールバックを削除

agilebits.com の取得失敗時のフォールバックが 別プロダクト (1Password/connect — Connect サーバー) のリリースを参照しており、もし fallback が発火すると CLI ではない無関係なバージョンに置換されるバグだった。

  • フォールバックを削除し、失敗時は現在バージョンを維持する既存ロジックに任せる
  • curl -scurl -fsS で接続失敗をサイレントに無視せず検知できるようにする

1Password CLI は closed-source で公開 GitHub Releases が存在しないため、agilebits.com の update endpoint が唯一の公式取得元。

Test plan

  • npm run format:check
  • npm run lint
  • npm test(114 tests passed)
  • ローカルで curl -fsS ... | jq -r '.version' の動作確認(2.34.0 取得確認)
  • CI 緑確認

🤖 Generated with Claude Code

ADR 0006 で残課題としていた 2 点を解消する。

- cron `'0 5 * * *'` (毎日) → `'0 5 * * 1'` (毎週月曜)
  DevContainer 再ビルドのコストと patch 追従の必要性を勘案し、
  Dependabot と同じ週次サイクルに揃える。
- 1Password CLI の取得元から `1Password/connect` フォールバックを削除。
  Connect は別プロダクト(Connect サーバー)であり、CLI のバージョンに
  ならない誤参照だった。`agilebits.com` の取得失敗時は現在バージョンを
  維持する既存ロジックに任せる。あわせて `curl -fsS` で接続失敗を
  サイレントに無視せず検知できるようにする。

ADR 0006 の 残課題 セクションを削除し、具体変更 に統合。

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented May 18, 2026

Copy link
Copy Markdown

Warning

Rate limit exceeded

@keito4 has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 4 minutes and 56 seconds before requesting another review.

You’ve run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 5065ccfa-0e1a-4284-b0e5-5072cae16c95

📥 Commits

Reviewing files that changed from the base of the PR and between eae9311 and 4bcfe93.

📒 Files selected for processing (2)
  • .github/workflows/update-dev-tools.yml
  • docs/adr/0006-consolidate-version-updates.md
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/update-dev-tools-followup

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@keito4

keito4 commented May 18, 2026

Copy link
Copy Markdown
Owner Author

🔍 AI Code Review (Local Hook)

Models: Codex (default) / Gemini (default)

✨ Gemini Review

MCP issues detected. Run /mcp list for status.このコード変更をレビューしました。全体として、誤った依存関係の修正とメンテナンスコストの最適化が行われており、非常に適切な変更です。

指摘事項

指摘すべきアクション可能な問題は見つかりませんでした。

補足的な観察事項(改善点)

  • エラーハンドリングの改善: 1Password CLIの取得において curl -fsS を導入したことで、ネットワークエラーや404エラーが発生した際にサイレントに失敗せず、適切にフォールバック(現行バージョンの維持)が行われるようになっています。
  • 正確性の向上: ADRで言及されている通り、別プロダクト(1Password Connect)のリリースを参照していた問題を修正しており、正しい取得元(agilebitsのアップデートエンドポイント)に集約されています。
  • 運用の最適化: 実行頻度を毎日から毎週に変更したことは、DevContainerのビルドコスト削減とDependabotとのサイクル同期の観点から合理的です。

判定: patch is correct

理由: 誤ったフォールバック先(1Password Connect)の削除、エラーハンドリングの強化、およびADRへの正確な記録が行われており、正確性と保守性の両面で改善されているため。

信頼度スコア: 1.0


🤖 Generated by post_pr_ai_review.py hook

@keito4 keito4 self-assigned this May 18, 2026
@keito4
keito4 merged commit 1528e37 into main May 18, 2026
14 checks passed
@keito4
keito4 deleted the fix/update-dev-tools-followup branch May 18, 2026 07:18
@github-actions

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 1.112.5 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

released リリース済み

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant