Skip to content

fix: Bolt 5のLambdaハンドラー形式に対応 - #76

Merged
eai04191 merged 3 commits into
mainfrom
fix/slack-bolt-v5-lambda-handler
Jul 27, 2026
Merged

fix: Bolt 5のLambdaハンドラー形式に対応#76
eai04191 merged 3 commits into
mainfrom
fix/slack-bolt-v5-lambda-handler

Conversation

@eai04191

@eai04191 eai04191 commented Jul 27, 2026

Copy link
Copy Markdown
Member

issue

変更内容

  • @slack/boltを5.0.0へ更新
  • Bolt 5の2引数Promise形式に合わせてLambdaハンドラーの呼び出しを修正

確認したこと

  • 依存更新だけを適用した状態でnpm run typecheckTS2554で失敗することを確認
  • npm run check
  • npm run typecheck
  • npm test(36件成功)
  • infranpx tsc --noEmit

スクリーンショット

なし

補足事項

Renovate PR #75 の依存更新と、そのcheck失敗の修正を含みます。

PR時のセルフチェック

  • 関数や変数の命名は一目で分かるものになってますか?
  • コメントは適切な量で、誰が見ても分かるコメントになってますか?
  • PR内容に関するテストは書きましたか?
  • PR内容に付随する各種ドキュメントは一緒に修正しましたか?
  • IssueをCloseする場合、完了の定義は満たせていますか?

byCodex

Copilot AI review requested due to automatic review settings July 27, 2026 00:28
@eai04191
eai04191 marked this pull request as ready for review July 27, 2026 00:30

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Bolt 5(@slack/bolt v5.0.0)への追従と、Lambda ハンドラー呼び出し形式の差分を最小限で丁寧に合わせている点が良いです。CI/Infra 側でも Node.js 24 を利用しているため、Bolt 5 の Node.js 20+ 要件とも整合しており、アップデート方針として前向きに進められる内容です。

Changes:

  • @slack/bolt4.7.35.0.0 に更新
  • Bolt 5 の AwsLambdaReceiver ハンドラー(2引数 Promise 形式)に合わせて boltHandler(event, context) 呼び出しへ変更
  • 依存更新に伴う lockfile(package-lock.json)の更新

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
src/handler.ts Bolt 5 の Lambda ハンドラー呼び出しシグネチャ変更に追従(callback 引数の削除)。
package.json @slack/bolt を v5.0.0 に更新。
package-lock.json Bolt 5 および関連依存の解決結果を反映。

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@eai04191
eai04191 merged commit 57d8cd7 into main Jul 27, 2026
3 checks passed
@eai04191
eai04191 deleted the fix/slack-bolt-v5-lambda-handler branch July 27, 2026 00:33
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.

2 participants