-
Notifications
You must be signed in to change notification settings - Fork 69
Update prompt.py #114
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
Update prompt.py #114
Conversation
Stricter classification prompt for DevRel triage (fixes #93)
WalkthroughThe classification prompt for DevRel triage was rewritten to apply stricter, more explicit criteria for triggering DevRel involvement. The new prompt clarifies when to activate DevRel, updates response formatting, and revises example messages and expected outputs to align with the stricter rules. No changes were made to exported entities. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~7 minutes Assessment against linked issues
Poem
Note ⚡️ Unit Test Generation is now available in beta!Learn more here, or try it out under "Finishing Touches" below. 📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
🧰 Additional context used🧠 Learnings (2)📓 Common learningsbackend/app/classification/prompt.py (1)Learnt from: smokeyScraper 🔇 Additional comments (4)
✨ Finishing Touches
🧪 Generate unit tests
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
smokeyScraper
left a comment
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.
could you please have a workaround for the rest of the prompts and optimize them as much as they can be? with testing on how the model outputs markdown if asked to? it's better if we enforce markdown based approach for text responses.
| Respond ONLY with JSON: | ||
| {{ | ||
| { |
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.
I'm not very sure how this will affect the model response, as the model can also respond with JSON as markdown, I faced this a lot while interacting with the system. Could you please check a bit on this part? A few interactions would be awesome.
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.
Thank you for highlighting this—it's a really important point! I've noticed as well that sometimes the model returns JSON wrapped in Markdown code blocks, which can trip up our downstream processing if we're expecting plain JSON.
To make sure the new prompt works as intended, I'll check a variety of typical user inputs—including ones where the bot is mentioned directly, general greetings, technical questions, and even some off-topic messages. I'll pay close attention to whether the model outputs just raw JSON or still adds Markdown formatting. If I see the model consistently adding code blocks, we can tweak the prompt to explicitly ask for plain JSON only.
If you have any particular examples where this was an issue, let me know and I'll make sure to include them in the checks. Otherwise, I'll report back soon with what I find and suggest any further changes if needed!
Stricter classification prompt for DevRel triage (fixes #93)
Closes #93
📝 Description
This pull request refines the 'DEVREL_TRIAGE_PROMPT' in 'backend/app/classification/prompt.py' to enforce stricter criteria for triggering the DevRel agent. The new prompt ensures the agent is activated only for direct DevRel mentions or clear, repository-related technical queries, minimizing false positives and unnecessary agent involvement. This increases accuracy and improves the user experience.
🔧 Changes Made
DEVREL_TRIAGE_PROMPTto:📷 Screenshots or Visual Changes (if applicable)
🤝 Collaboration
Collaborated with:
@username(optional)✅ Checklist
Summary by CodeRabbit