feat: add Medical Research Summarizer agent - #560
Conversation
|
@adityajoshi18vk-art is attempting to deploy a commit to the aditthyass' projects Team on Vercel. A member of the Team first needs to authorize it. |
|
Note
|
| Layer / File(s) | Summary |
|---|---|
Export form, metadata, and input schema src/agents/definitions/medical-research-summarizer.js |
Converts const medicalResearchSummarizer = ...; export default medicalResearchSummarizer to an inline export default { ... }. Updates description, changes category from Research to Healthcare, updates icon, and adds a provider comment. Renames researchText → paper_text and replaces audience selector options with Patient, Healthcare Practitioner, Both; removes the field's default value and makes it optional. |
System prompt and output contract src/agents/definitions/medical-research-summarizer.js |
Replaces the system prompt with a stricter version mandating a fixed five-section Markdown structure (## Plain English Summary, ## Key Findings, ## Methodology, ## Limitations, ## What This Means), rules against inventing results, abstract-only text handling, audience-conditional section depth, and a required ending disclaimer line. Retains outputType: 'markdown'. |
Estimated code review effort
🎯 2 (Simple) | ⏱️ ~12 minutes
Possibly related PRs
- AditthyaSS/iloveAgents#531: This is the original PR that added the
medical-research-summarizeragent; the current PR is a direct refactor of that same file's export form, input schema, and system prompt.
Suggested labels
healthcare, agent-update
Suggested reviewers
- AditthyaSS
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
| Check name | Status | Explanation |
|---|---|---|
| Description Check | ✅ Passed | Check skipped - CodeRabbit’s high-level summary is enabled. |
| Title check | ✅ Passed | The title 'feat: add Medical Research Summarizer agent' directly and clearly describes the main change—adding a new agent. It is concise, specific, and accurately reflects the core objective of the PR. |
| Linked Issues check | ✅ Passed | The PR fully implements the requirements from issue #69: a Medical Research Summarizer agent that accepts medical research text, generates plain English summaries with key findings, methodology, limitations, and audience-specific takeaways, and is added to the Healthcare domain. |
| Out of Scope Changes check | ✅ Passed | All changes are contained within the single configuration file for the Medical Research Summarizer agent. No modifications to categories.js or manual registry updates were required, keeping changes tightly scoped to the agent definition. |
| Docstring Coverage | ✅ Passed | No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. |
✏️ Tip: You can configure your own custom pre-merge checks in the settings.
✨ Finishing Touches
🧪 Generate unit tests (beta)
- Create PR with unit tests
Comment @coderabbitai help to get the list of available commands and usage tips.
|
Hey @adityajoshi18vk-art! 👋
|
|
hey @adityajoshi18vk-art! 👋 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Pull request overview
This PR updates/adds the Medical Research Summarizer agent definition so it appears under the Healthcare category and produces a more structured, safety-conscious plain-English summary of medical research text (abstracts or full papers), aligning with Issue #69.
Changes:
- Reworked the agent config to
export default { ... }, moved the agent to the Healthcare category, and updated the icon to Stethoscope. - Updated inputs to accept paper text plus an optional audience selector (Patient / Healthcare Practitioner / Both).
- Replaced the system prompt with a stricter Markdown output structure, stronger “don’t invent results” rules, and an explicit informational-not-advice closing note.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| type: 'select', | ||
| options: [ | ||
| 'General Public', | ||
| 'Patients', | ||
| 'Healthcare Professionals', | ||
| 'Researchers', | ||
| ], | ||
| defaultValue: 'General Public', | ||
| required: true, | ||
| options: ['Patient', 'Healthcare Practitioner', 'Both'], | ||
| placeholder: 'Choose who this summary is for', | ||
| required: false, |
adityajoshi18vk-art
changed the title
|
Hi @AditthyaSS, I hope the changes look good. Please let me know if there is anything else left to fix, or if it's ready for your approval! |
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
What this PR does
Adds a new Medical Research Summarizer agent to the Healthcare category.
This agent turns dense medical research abstracts or full papers into plain-English summaries with:
Implementation details
egistry.js\ — no registry edits needed
Closes #69
Summary by CodeRabbit
paper_textand made the audience selection optional