Skip to content

Perf/codecovtrigger#151

Merged
Josephrp merged 59 commits intodevfrom
perf/codecovtrigger
Oct 13, 2025
Merged

Perf/codecovtrigger#151
Josephrp merged 59 commits intodevfrom
perf/codecovtrigger

Conversation

@Josephrp
Copy link
Copy Markdown
Collaborator

fix ci

MarioAderman and others added 30 commits October 6, 2025 11:06
Removes all references to non-existent @defer decorator from codebase.
The @defer decorator never existed in Pydantic AI. Tools are correctly
implemented using standard Pydantic AI patterns.

Changes:
- Removed 16 @defer comments from tool files
- Updated README Known Issues section
- All tools continue to work correctly (no functional changes)

Fixes #2
- Implement VLLMModel wrapper around existing VLLMClient
- Add OpenAICompatibleModel for vLLM, llama.cpp, TGI servers
- Provide factory methods (from_vllm, from_llamacpp, from_tgi)
- Include streaming support and message conversion
- Add convenience aliases for VLLMModel and LlamaCppModel
- Replace non-existent OpenAIProvider with OllamaProvider from pydantic_ai
- Remove dataclass decorator to properly inherit from OpenAIChatModel
- Fix factory methods to pass model_name as positional argument
- Add comprehensive test suite with 8 passing tests
- Skip integration tests that require actual vLLM servers
- Add from_config() method to support Hydra DictConfig
- Update all factory methods (from_vllm, from_llamacpp, from_tgi, from_custom) to accept optional config
- Support config override via direct parameters
- Extract generation settings from config (temperature, max_tokens, etc.)
- Add environment variable fallbacks (LLM_BASE_URL, LLM_API_KEY)
- Create config files for llamacpp, tgi, and vllm
- Update tests to cover both config-based and direct parameter approaches
- All 10 tests passing
- Add LLMModelConfig and GenerationConfig datatypes
- Remove redundant vllm_model.py
- Update openai_compatible_model.py with validation
- Rewrite tests to use actual config files (30 tests)
Integrated latest upstream changes while preserving Issue #10 LLM client work.

Conflict resolutions:
- DeepResearch/src/datatypes/__init__.py: Added LLM model types to __all__ without duplicates
- Accepted upstream versions for bioinformatics_tools.py, deep_agent_tools.py, README.md
…in permissions

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
Signed-off-by: Tonic <joseph@legml.ai>
…in permissions

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
Signed-off-by: Tonic <joseph@legml.ai>
Signed-off-by: Tonic <joseph.pollack@emle.eu>
Josephrp and others added 29 commits October 13, 2025 01:32
Signed-off-by: Tonic <joseph.pollack@emle.eu>
Signed-off-by: Tonic <joseph.pollack@emle.eu>
Signed-off-by: Tonic <joseph.pollack@emle.eu>
Signed-off-by: Tonic <joseph.pollack@emle.eu>
Signed-off-by: Tonic <joseph.pollack@emle.eu>
@Josephrp Josephrp merged commit 5933bd8 into dev Oct 13, 2025
6 checks passed
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.

2 participants