Skip to content

fix: active テナントで integration 未設定時に警告ログを出す#248

Merged
coji merged 1 commit intomainfrom
fix/warn-active-org-no-integration
Mar 24, 2026
Merged

fix: active テナントで integration 未設定時に警告ログを出す#248
coji merged 1 commit intomainfrom
fix/warn-active-org-no-integration

Conversation

@coji
Copy link
Owner

@coji coji commented Mar 24, 2026

Summary

  • scheduler が active な org の integration 未設定を黙ってスキップしていた問題を修正
  • logger.warn で org 名・slug 付きの警告メッセージを出力し、設定ミスを検知可能にした

closes #246

Test plan

  • pnpm validate pass
  • integration 未設定の active org がある状態で scheduler を動かし、warn ログが出ることを確認

🤖 Generated with Claude Code

Summary by CodeRabbit

  • 改善
    • ジョブスケジューラーが、連携設定のない有効な組織を検出した場合に警告ログを記録するようになりました。これにより、設定に問題がある組織の特定が容易になります。

scheduler が active な org の integration 未設定を黙ってスキップしていたため、
データ収集停止に気づけなかった。warn ログを追加して検知可能にする。

closes #246

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

coderabbitai bot commented Mar 24, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: ed01334c-e809-4e62-b476-d1aebf19a3be

📥 Commits

Reviewing files that changed from the base of the PR and between edd1c45 and dd4dd3d.

📒 Files selected for processing (1)
  • batch/job-scheduler.ts

📝 Walkthrough

Walkthrough

batch/job-scheduler.ts のアクティブな組織ループに対して、integration 設定が欠けている場合に警告ログを出力する機能を追加しました。これまでは、設定が欠けている場合、警告なく次の組織へスキップしていました。

Changes

Cohort / File(s) Summary
ジョブスケジューラー警告ログ
batch/job-scheduler.ts
アクティブな組織で integration が未設定の場合に警告メッセージをログ出力し、処理を続行。設定ミス検知の改善。

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

⚠️ 静かなバグは見つけにくい
でも警告は光る兎の知恵
設定の欠けを叫ぶログが
今、スケジューラーに咲き誇る 🐰✨

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/warn-active-org-no-integration

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.

@coji coji merged commit 0f9ebd1 into main Mar 24, 2026
6 checks passed
@coji coji deleted the fix/warn-active-org-no-integration branch March 24, 2026 12:39
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.

active テナントで integration 未設定時に警告を出す

1 participant