New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

文頭にメンションがない場合のエラー対応 #19

Closed

unirt wants to merge 1 commit into aws-samples:main from unirt:fix-mention-position-validation

Contributor

unirt commented Apr 4, 2025

概要

Slackでのメッセージ送信時に、文末にメンションを付けるとValidationExceptionが発生する問題を修正しました。

修正内容

メンションが文頭にない場合に、適切なエラーメッセージを返すよう修正
空のメッセージに対してもエラーメッセージを返すよう対応
これにより、BedrockのAPIに空のメッセージが送信されることを防ぎ、ValidationExceptionを回避します

テスト方法

文頭にメンションを付けて投稿した場合（正常動作）
文末にメンションを付けて投稿した場合（エラーメッセージが表示される）
メンションだけで内容が空の場合（エラーメッセージが表示される）


          Fix: Add validation for mention position in messages

b20e60f

Contributor Author

unirt commented Apr 4, 2025

remote-swe created this PR unintentionally 😅

unirt closed this

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

Labels

None yet