Skip to content

Conversation

@wmasfoe
Copy link

@wmasfoe wmasfoe commented Nov 29, 2025

Features

  • Fix dependency conflict between pillow and crawl4ai by downgrading pillow to ~=10.4.0
  • Add missing dependencies structlog and daytona-sdk to requirements.txt
  • Fix incorrect import path for daytona SDK (daytona -> daytona_sdk)
  • Make daytona_api_key optional to allow running without Daytona configuration
  • Add conditional initialization for Daytona client to prevent crashes when API key is not provided

Feature Docs

N/A

Influence

This PR fixes critical startup issues that prevent users from running the project:

  1. uv pip install -r requirements.txt fails due to pillow/crawl4ai version conflict
  2. ModuleNotFoundError for structlog and daytona modules
  3. ValidationError when daytona_api_key is not configured
  4. DaytonaError during Daytona client initialization without API key

After this fix, users can successfully install dependencies and start the project without requiring Daytona configuration.

Result

Before:

× No solution found when resolving dependencies:
  ╰─▶ Because crawl4ai==0.6.3 depends on pillow>=10.4,<11.dev0...

ModuleNotFoundError: No module named 'structlog'
ModuleNotFoundError: No module named 'daytona'

pydantic_core._pydantic_core.ValidationError: 1 validation error for DaytonaSettings
  daytona_api_key - Field required

daytona_sdk.common.errors.DaytonaError: API key or JWT token is required

After:

✅ uv pip install -r requirements.txt - Success
✅ python main.py - Starts without errors
✅ Daytona features gracefully skipped when not configured

Other

Files changed:

  • requirements.txt - Fix pillow version, add structlog and daytona-sdk
  • app/config.py - Make daytona_api_key optional
  • app/daytona/sandbox.py - Fix import path, add conditional initialization
  • app/daytona/tool_base.py - Fix import path, add conditional initialization

@titanism
Copy link

please merge!

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