generated from amazon-archives/__template_Apache-2.0
-
Notifications
You must be signed in to change notification settings - Fork 1k
feat(aws-healthomics-mcp): genomics file search #1501
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
Open
markjschreiber
wants to merge
43
commits into
awslabs:main
Choose a base branch
from
markjschreiber:feature/genomics-file-search
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+22,301
−49
Open
Changes from all commits
Commits
Show all changes
43 commits
Select commit
Hold shift + click to select a range
3a955ad
feat: add core data models for genomics file search
markjschreiber 918e521
feat(search): implement pattern matching and scoring engine
markjschreiber 2945749
feat: implement file association detection system
markjschreiber 27b3e36
feat: implement S3 search engine with configuration management
markjschreiber a62f7a1
feat:(search) adds a search interface to the healthomics sequence and…
markjschreiber 5e6623b
feat(genomics-search): implement search orchestrator and MCP tool han…
markjschreiber 52e0261
feat(search): adds result ranking and response assembly
markjschreiber 30254d0
docs: add genomics file search capabilities to README and CHANGELOG
markjschreiber 5eca2a7
Fix SearchGenomicsFiles tool: regex patterns, S3 client calls, and fi…
markjschreiber aa4dd8a
perf(s3-search): optimize S3 API calls with lazy loading, caching, an…
markjschreiber ddd784f
Fix genomics file search for HealthOmics reference stores
markjschreiber 7ba4bbe
feat(search): enhance HealthOmics sequence and reference store search…
markjschreiber 5f6407e
feat: performance improvements and minor fixes
markjschreiber 1d1484c
feat: implement efficient storage-level pagination for genomics file …
markjschreiber bc9dfb6
fix: correct the associate of bwa files and fix pyright type errors
markjschreiber 65c6826
feat(tests): implement comprehensive testing framework with MCP Field…
markjschreiber 0f93475
fix(tests): repair healthomics search engine tests
markjschreiber a64c8d5
test(s3): add comprehensive tests for S3SearchEngine
markjschreiber 119c998
fix(tests): fix failing healthomics search engine tests
markjschreiber 317e389
test(result-ranker): achieve 100% test coverage for ResultRanker
markjschreiber 49cd8b9
test(json-response-builder): achieve 100% test coverage for JsonRespo…
markjschreiber 2b94eee
test(config-utils): achieve 100% test coverage for config utilities
markjschreiber 194fbca
feat(s3-utils): optimize bucket validation and achieve 99% coverage
markjschreiber 0364e5c
feat(genomics-search-orchestrator): achieve 49% test coverage with co…
markjschreiber c0b91d4
perf(genomics-search-orchestrator): optimize test performance by 94%
markjschreiber 205883a
feat(healthomics-search-engine): improve test coverage from 61% to 69%
markjschreiber 2c8d7d1
fix: clean up files and reformats some files failing lints
markjschreiber a80a1e0
Merge remote-tracking branch 'upstream/main' into feature/genomics-fi…
markjschreiber 3a93788
security: fix bandit security issues
markjschreiber 2843406
fix(tests): mock AWS account/region methods to prevent credential access
markjschreiber 0a8c6a1
fix: fix pyright issues
markjschreiber 5010d5e
feat: improve test coverage
markjschreiber 298acc4
feat: increases coverage of pagination logic, filtering, fallbacks an…
markjschreiber 23f8a51
fix: mock aws credentials
markjschreiber afa8b02
feat: improve test coverage of exception handling, continuation token…
markjschreiber 9cd825b
fix: pyright type error fixed
markjschreiber 7c48aca
feat: more test coverage to stop codecov nagging me
markjschreiber 8049bea
feat: improvements to branch coverage
markjschreiber f82a873
fix(search): enforce S3 bucket access validation in orchestrator
markjschreiber 1a78421
refactor: rename config_utils to search_config and reorganize models
markjschreiber ebd02bc
Merge remote-tracking branch 'upstream/main' into feature/genomics-fi…
markjschreiber a8e29c6
feat: comprehensive test coverage improvements and code quality enhan…
markjschreiber 3b7c3ad
Merge branch 'main' into feature/genomics-file-search
markjschreiber File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Is this file needed?
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.
It has benefits above and beyond
uv-requirements.txtas it lets CI/CD build with the exact packages used by the developer. It looks like several other servers in the monorepo do also haveuv.lockfiles and they are tracked by git. Maybe something for the maintainers to make a repo wide decision on (and maybe also add to the.gitignorefor the whole repo)?