Skip to content

Teams通知をWebhook直送へ移行(通常通知はWebhook専用・エラー通知は管理者メール併用)#2

Merged
mitsuiJao merged 6 commits into
mainfrom
copilot/change-notifications-to-webhook
May 26, 2026
Merged

Teams通知をWebhook直送へ移行(通常通知はWebhook専用・エラー通知は管理者メール併用)#2
mitsuiJao merged 6 commits into
mainfrom
copilot/change-notifications-to-webhook

Conversation

Copilot AI commented May 26, 2026

Copy link
Copy Markdown
Contributor

現在の通知経路を、Teams Incoming Webhook への直接送信に対応させる変更です。
レビュー指摘を反映し、通常通知はWebhookのみエラー通知は管理者メールを併用する構成に調整しました(実行時エラーはメールのみ)。

  • 通知送信基盤の更新

    • src/sender.jsfetch による Teams Webhook POST を実装
    • 通常の課題通知 (sendNotification) はWebhook送信のみに統一
    • 認証エラー通知 (sendLoginRequiredNotification) は Teams Webhook + 管理者メール併用
    • 実行時エラー通知 (sendErrorNotification) は 管理者メールのみ 送信(Teams送信なし)
  • 実行フローの更新

    • index.jsloginRequired 時に sendLoginRequiredNotification(...) を送信
    • メイン処理の catchsendErrorNotification(error) を送信
  • 設定モデルの整理

    • src/config.jsTEAMS_WEBHOOK_URL をWebhook通知に利用
    • APIKEY / SENDFROM / SENDTO はエラー時の管理者メール通知用途として維持
  • 運用・補助コードの追従

    • debug_sender.js をWebhook送信デバッグ用に更新
    • ワークフロー(.github/workflows/cron.disable.yml)でWebhook/メール両方の環境変数に対応
    • README.md.env 説明を、メール設定は管理者エラー通知向けである旨に更新
  • 依存関係

    • resend はエラー時の管理者メール通知のため利用継続
// src/sender.js(要旨)
export async function sendNotification(subject, body) {
  await sendTeamsWebhook(subject, body); // 通常通知はWebhookのみ
}

export async function sendLoginRequiredNotification(errorMessage = "") {
  await sendTeamsWebhook(subject, body); // Teams alert
  await sendAdminEmail(subject, body);   // 管理者メール併用
}

export async function sendErrorNotification(error) {
  await sendAdminEmail(subject, body);   // 実行時エラーはメールのみ
}

Copilot AI linked an issue May 26, 2026 that may be closed by this pull request
Copilot AI and others added 2 commits May 26, 2026 02:12
Copilot AI changed the title [WIP] Change notification method from email to webhook Teams通知をメール経由からWebhook直送へ移行 May 26, 2026
Copilot AI requested a review from mitsuiJao May 26, 2026 02:16
@mitsuiJao

Copy link
Copy Markdown
Owner

@copilot
メール通知機能はそのまま残していてほしい。

Copilot AI commented May 26, 2026

Copy link
Copy Markdown
Contributor Author

@copilot
メール通知機能はそのまま残していてほしい。

メール通知機能を復元し、Webhookとメールの両方を送れるように修正しました。対応コミットは beaaba0 です。

Copilot AI changed the title Teams通知をメール経由からWebhook直送へ移行 Teams通知をWebhook直送へ移行(メール通知も併用可能) May 26, 2026
@mitsuiJao

Copy link
Copy Markdown
Owner

@copilot
通知はwebhookのみに、エラー時の管理者通知はメールを併用すること。
teamsへのアラートは全てwebhookにしてほしい

Copilot AI commented May 26, 2026

Copy link
Copy Markdown
Contributor Author

@copilot
通知はwebhookのみに、エラー時の管理者通知はメールを併用すること。
teamsへのアラートは全てwebhookにしてほしい

ご要望どおりに調整しました。通常通知はWebhookのみ、認証エラー/実行時エラーはTeamsへWebhook送信しつつ管理者メールも併用します。対応コミットは 7c664c8 です。

Copilot AI changed the title Teams通知をWebhook直送へ移行(メール通知も併用可能) Teams通知をWebhook直送へ移行(通常通知はWebhook専用・エラー時は管理者メール併用) May 26, 2026
@mitsuiJao

Copy link
Copy Markdown
Owner

@copilot 実行時エラーはteamsにも飛ぶようになってる?メールだけにしてほしい

Copilot AI commented May 26, 2026

Copy link
Copy Markdown
Contributor Author

@copilot 実行時エラーはteamsにも飛ぶようになってる?メールだけにしてほしい

実行時エラー通知はメール専用に変更しました(Teams送信はしません)。対応コミットは 0432f6a です。UI変更はありません。

Copilot AI changed the title Teams通知をWebhook直送へ移行(通常通知はWebhook専用・エラー時は管理者メール併用) Teams通知をWebhook直送へ移行(通常通知はWebhook専用・エラー通知は管理者メール併用) May 26, 2026
@mitsuiJao
mitsuiJao marked this pull request as ready for review May 26, 2026 07:49
Copilot AI review requested due to automatic review settings May 26, 2026 07:49
@mitsuiJao mitsuiJao closed this May 26, 2026
@mitsuiJao mitsuiJao reopened this May 26, 2026
@mitsuiJao
mitsuiJao merged commit 07ad5e5 into main May 26, 2026
1 of 2 checks passed
Copilot AI removed the request for review from Copilot May 26, 2026 08:11
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.

teams通知をwebhookに変更

2 participants