Skip to content

Fix/inconsistent hyperlinks 22#52

Open
vivekdevaa124 wants to merge 2 commits intovirtualcell:mainfrom
vivekdevaa124:fix/inconsistent-hyperlinks-22
Open

Fix/inconsistent hyperlinks 22#52
vivekdevaa124 wants to merge 2 commits intovirtualcell:mainfrom
vivekdevaa124:fix/inconsistent-hyperlinks-22

Conversation

@vivekdevaa124
Copy link

This PR fixes issue #22 where model hyperlinks were inconsistently displayed as *** or led to the generic vcell.org homepage instead of specific model pages.

Changes:

  • System Prompt Update: Added a new 'Biomodel Link Guidelines' section to backend/app/utils/system_prompt.py. This instructs the LLM to always use the bmKey field to construct direct links in the format https://vcell.org/biomodel/<bmKey>.
  • Backend Bug Fix: Fixed a deprecated langchain import in backend/app/services/knowledge_base_service.py that was preventing the backend from starting correctly (ModuleNotFoundError: No module named 'langchain.text_splitter').
  • Frontend Connectivity: Added NEXT_PUBLIC_API_URL configuration to ensure the chat interface can reach the local backend.

Verification:

  • Backend and Frontend servers start cleanly.
  • Verified via browser testing that AI responses now include correctly formatted, clickable links (e.g., https://vcell.org/biomodel/271989751) instead of the generic homepage.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant