Skip to content

centralize analysis type definition#7

Open
shivanshuyadav921 wants to merge 2 commits intoreactome:mainfrom
shivanshuyadav921:make
Open

centralize analysis type definition#7
shivanshuyadav921 wants to merge 2 commits intoreactome:mainfrom
shivanshuyadav921:make

Conversation

@shivanshuyadav921
Copy link
Copy Markdown

@shivanshuyadav921 shivanshuyadav921 commented Mar 26, 2026

Summary
Replaces hardcoded inline analysis type values in the interface
with a centralized definition that the interface now references
directly.

What Changed

  • Introduced a single centralized analysis type definition
  • Interface now depends on that definition instead of duplicating it
  • Removed all hardcoded inline values

Why This Matters

  • Eliminates duplication across the codebase
  • Enables runtime validation (Zod/JSON Schema compatible)
  • Strengthens compile-time type safety
  • Reusable across API layers, MCP tools, validation schemas, and UI

Testing

  • Existing tests pass unchanged
  • No breaking changes

This PR introduces hybrid retrieval (embeddings + API fallback), query routing, result enrichment, and caching to improve the Reactome MCP server’s accuracy, performance, and usability. It also adds standardized error handling and basic diagnostics for better observability, while maintaining backward compatibility.
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