-
Notifications
You must be signed in to change notification settings - Fork 428
Adjusted prompt to include support links, adjusted to use discord bot… #756
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.
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
Conversation
… instead of webhook. Message formatter now supports links
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR Summary
Enhanced Discord support integration by replacing webhook with bot implementation and improved message handling system.
- Migrated from Discord webhooks to bot API in
docs/lib/discord-webhook.ts
enabling persistent thread-based conversations tracked by sessionId - Added support contact routing in AI prompt to guide users to Discord/email channels when appropriate
- Implemented link formatting support in
docs/src/components/chat/message-formatter.tsx
with proper styling and accessibility - WARNING: In-memory session mapping in
docs/lib/discord-webhook.ts
may increase API calls due to serverless context resets - New endpoint
docs/src/app/api/discord-webhook/response/route.ts
lacks standard SmartRouteHandler pattern and input validation
5 files reviewed, 2 comments
Edit PR Review Bot Settings | Greptile
😱 Found 1 issue. Time to roll up your sleeves! 😱 🗒️ View all ignored comments in this repo
Need help? Join our Discord for support! |
…ionIds within ai channel
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, although next time try to make the PR smaller (the first, second, and 3rd+4th could be separate PRs each). For AI chatbot prompt changes, feel free to push to dev
directly without a PR (just gave you permissions)
- Resolved conflicts in docs/lib/discord-webhook.ts: kept Discord Bot API implementation with threading support while incorporating improved browser detection - Resolved conflicts in docs/src/app/api/chat/route.ts: combined detailed support guidance with improved limitation handling - Maintained discord_webhook_update functionality while integrating dev improvements
… instead of webhook. Message formatter now supports link
Important
Enhance Discord integration by using a bot for session management, update AI assistant to provide support links, and add link support in message formatting.
discord-webhook.ts
for session threading and message handling.sendLLMResponseToDiscord()
to send AI responses to Discord.findExistingThread()
,createNewThread()
, andsendToThread()
for thread management.route.ts
to include support contact information in AI responses.ai-chat.tsx
to send AI responses to Discord and handle session data.message-formatter.tsx
to render links in AI responses.This description was created by
for f1f5e2b. You can customize this summary. It will automatically update as commits are pushed.