-
Notifications
You must be signed in to change notification settings - Fork 17
improvements | added new prompt | refactoring #8
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
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
| @@ -1,66 +1,32 @@ | ||||||
| from langchain_ollama import ChatOllama | ||||||
| from langchain.agents import create_agent | ||||||
| from langchain_core.runnables import RunnableConfig | ||||||
| from tools.wiki import wiki_search | ||||||
| from tools.math import smart_math | ||||||
| from tools.getNews import asknews_search | ||||||
| from tools.arxiv_tool import arxiv_search | ||||||
| from tools.duckduckgo import duck_duck_go_search, duck_duck_go_search_results | ||||||
| from tools.getDate import get_current_date | ||||||
| from tools.save_md import save_md_locally | ||||||
| from tools.summarize_text import summarize_text | ||||||
| from tools.serpSearch import serp_search | ||||||
| from tools.save_md_plus import save_md_plus | ||||||
| from tools.SearxNG import searx_search | ||||||
| from tools.crossref import crossref_search | ||||||
| from tools.unpaywall import unpaywall_lookup | ||||||
| from tools.europe_pmc import europe_pmc_search | ||||||
| from tools.openlibrary import openlibrary_search | ||||||
| from retriever import add_to_db,query_db | ||||||
| from prompts.research_prompt import KURAMA_RESEARCH_PROMPT_GENERIC, KURAMA_RESEARCH_PROMPT_DEEPRESEARCH | ||||||
| from tools.tools_wrapper import tools | ||||||
| from prompts.research_prompt import KURAMA_RESEARCH_PROMPT_GENERIC, KURAMA_RESEARCH_PROMPT_DEEPRESEARCH,KURAMA_RESEARCH_PROMPT | ||||||
|
||||||
| from prompts.research_prompt import KURAMA_RESEARCH_PROMPT_GENERIC, KURAMA_RESEARCH_PROMPT_DEEPRESEARCH,KURAMA_RESEARCH_PROMPT | |
| from prompts.research_prompt import KURAMA_RESEARCH_PROMPT |
Copilot
AI
Jan 17, 2026
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.
Missing space after comma in the function call. Python style guidelines (PEP 8) recommend adding a space after commas in function arguments for better readability.
| llm = ChatOllama(model="gpt-oss:120b-cloud", temperature=0.7,verbose=True) | |
| llm = ChatOllama(model="gpt-oss:120b-cloud", temperature=0.7, verbose=True) |
| Original file line number | Diff line number | Diff line change | ||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -239,4 +239,71 @@ | |||||||||||||||
|
|
||||||||||||||||
| You are Kurama Research Agent in **Deep Mode**. | ||||||||||||||||
| Your purpose is to produce **multi-page, publication-grade Markdown research reports**, saving each section progressively while maintaining local, persistent knowledge across sessions. | ||||||||||||||||
| """ | ||||||||||||||||
|
|
||||||||||||||||
|
|
||||||||||||||||
| Tools = """ | ||||||||||||||||
| # Role | ||||||||||||||||
| You are **Kurama Research Agent v1.3**, an autonomous research assistant built by Vedas Dixit. | ||||||||||||||||
|
|
||||||||||||||||
| Below are strict rules for you to follow: | ||||||||||||||||
| Do's: | ||||||||||||||||
| -> Internal reason before taking action | ||||||||||||||||
| -> never process sensitive request that makes you search any negative things and abuse tool calling in terms of. "*add thinhs herr" | ||||||||||||||||
|
||||||||||||||||
| -> never process sensitive request that makes you search any negative things and abuse tool calling in terms of. "*add thinhs herr" | |
| -> Never process sensitive requests that require searching for harmful, abusive, or otherwise unsafe content, or that would constitute abuse of tool calling. |
Copilot
AI
Jan 17, 2026
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.
This instruction is unclear and appears incomplete. The phrase "never process sensitive request that makes you search any negative things and abuse tool calling in terms of" lacks grammatical correctness and doesn't clearly define what behavior should be avoided. The instruction should be rewritten to clearly specify what types of requests should be rejected and why.
| -> Internal reason before taking action | |
| -> never process sensitive request that makes you search any negative things and abuse tool calling in terms of. "*add thinhs herr" | |
| -> | |
| -> Internally reason through your plan before taking any action or calling any tool. | |
| -> Do not fulfill or assist with requests involving hate, harassment, self-harm, explicit sexual content, illegal activities, or other clearly harmful behavior. If a user asks for such content, politely refuse and, when appropriate, offer safer alternatives or high-level harm-reduction information. | |
| -> Do not abuse tools (for example, do not use them to search for or distribute harmful, unsafe, or disallowed content). Use tools only when they are necessary and in compliance with these safety rules. |
Copilot
AI
Jan 17, 2026
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.
Empty list item with only "->" and no content. This appears to be an incomplete placeholder that should either be removed or completed with the intended rule.
| -> |
Copilot
AI
Jan 17, 2026
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.
Variable name "Tools" is assigned but never used or exported. This appears to be a prompt template that might have been intended to be used but is not referenced anywhere in the codebase. Consider either removing it or incorporating it into the active prompts.
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| @@ -0,0 +1,38 @@ | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| from tools.wiki import wiki_search | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| from tools.math import smart_math | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| from tools.getNews import asknews_search | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| from tools.arxiv_tool import arxiv_search | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| from tools.duckduckgo import duck_duck_go_search, duck_duck_go_search_results | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| from tools.getDate import get_current_date | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| from tools.save_md import save_md_locally | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| from tools.summarize_text import summarize_text | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| from tools.serpSearch import serp_search | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| from tools.save_md_plus import save_md_plus | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| from tools.SearxNG import searx_search | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| from tools.crossref import crossref_search | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| from tools.unpaywall import unpaywall_lookup | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| from tools.europe_pmc import europe_pmc_search | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| from tools.openlibrary import openlibrary_search | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| from retriever import add_to_db,query_db | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| tools=[ | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| wiki_search, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| smart_math, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| duck_duck_go_search, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| duck_duck_go_search_results, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| searx_search, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| add_to_db, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| query_db, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| asknews_search, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| serp_search, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| arxiv_search, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| crossref_search, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| unpaywall_lookup, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| europe_pmc_search, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| openlibrary_search, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| get_current_date, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| save_md_locally, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| save_md_plus, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| summarize_text | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Comment on lines
+21
to
+37
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| smart_math, | |
| duck_duck_go_search, | |
| duck_duck_go_search_results, | |
| searx_search, | |
| add_to_db, | |
| query_db, | |
| asknews_search, | |
| serp_search, | |
| arxiv_search, | |
| crossref_search, | |
| unpaywall_lookup, | |
| europe_pmc_search, | |
| openlibrary_search, | |
| get_current_date, | |
| save_md_locally, | |
| save_md_plus, | |
| summarize_text | |
| smart_math, | |
| duck_duck_go_search, | |
| duck_duck_go_search_results, | |
| searx_search, | |
| add_to_db, | |
| query_db, | |
| asknews_search, | |
| serp_search, | |
| arxiv_search, | |
| crossref_search, | |
| unpaywall_lookup, | |
| europe_pmc_search, | |
| openlibrary_search, | |
| get_current_date, | |
| save_md_locally, | |
| save_md_plus, | |
| summarize_text |
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.
Missing space after comma in the import statement. Python style guidelines (PEP 8) recommend adding a space after commas in import statements for better readability.