Skip to content

fix: local development fixes for skill-bundler branch#33560

Closed
hikariming wants to merge 1 commit intolanggenius:refactor/skill-bundlerfrom
hikariming:fix/skill-bundler-local
Closed

fix: local development fixes for skill-bundler branch#33560
hikariming wants to merge 1 commit intolanggenius:refactor/skill-bundlerfrom
hikariming:fix/skill-bundler-local

Conversation

@hikariming
Copy link
Copy Markdown
Contributor

Changes

This PR fixes two issues found when testing the skill-bundler branch locally:

1. trust_env=False for httpx calls

2. Remove invalid base_path parameter

  • File: api/core/workflow/nodes/llm/node.py
  • compile_one() method doesn't have base_path parameter in its signature
  • Removing the invalid parameter allows skill compilation to work

Testing

Tested locally with:

  • Dify API running from source
  • Plugin daemon in Docker
  • Chatflow with skills working correctly

Related: #33558

1. trust_env=False for httpx calls to plugin daemon
   - Fixes connection issues when running from source on macOS

2. Remove invalid base_path parameter from compile_one calls
   - The method signature doesn't include base_path parameter
@gemini-code-assist
Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Mar 17, 2026
@hikariming hikariming closed this Mar 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant