引用されたもとのメッセージが編集されたとき引用した側のメッセージを変える

[nyson411]

close #2915
引用されたもとのメッセージが編集、削除したとき、引用した側のメッセージを変える

[coderabbitai]

Warning

Review limit reached

@nyson411, we couldn't start this review because you've used your available PR reviews for now.

Your plan includes 2 reviews of capacity. Refill in 2 minutes and 21 seconds.

Your organization has run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After more review capacity refills, 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 trial, open-source, and free plans. In all cases, review capacity refills continuously over time.

Please see our FAQ for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 8d0d62fd-78de-4f1b-b35c-f4275ab63118

📥 Commits

Reviewing files that changed from the base of the PR and between 78c1575 and e6269ec.

📒 Files selected for processing (4)

• cmd/wire_gen.go
• service/notification/handlers.go
• service/notification/service.go
• service/ws/target_func.go

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch feature/2915-cited-message

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[Eraxyso]

実装ありがとうございます！方針はLGTMですが、以下の点を確認していただきたいです。

• Query.Limit が未指定のため、検索結果がデフォルトの20件に制限され、引用が多い場合に一部のチャンネルへイベントが届かない可能性があります
• In を指定しない検索は公開チャンネルのみが対象となるため、DMなどの非公開チャンネル内の引用が取得できないように見えます
• 引用先チャンネル一覧を1分間キャッシュしていますが、引用の追加・編集・削除時にinvalidateされないため、最大1分間古い結果が使われる点が許容可能か確認したいです

（影響は限定的なのでそのまま許容してもよさそうですが、ramdosさんなどと相談した方がよいかもしれません）

また、可能であれば、今回の変更に対応するテストも追加していただけるとよさそうです。

[Eraxyso]

エラーをそのまま握り潰しているため、可能であればエラーハンドリングを改善していただきたいです