Skip to content

docs: clarify lazy skill source host paths#2998

Merged
seratch merged 1 commit into
mainfrom
alfozan/clarify-lazy-skill-host-paths
Apr 22, 2026
Merged

docs: clarify lazy skill source host paths#2998
seratch merged 1 commit into
mainfrom
alfozan/clarify-lazy-skill-host-paths

Conversation

@alfozan
Copy link
Copy Markdown
Collaborator

@alfozan alfozan commented Apr 22, 2026

Summary

  • Clarify that LocalDirLazySkillSource(source=LocalDir(src=...)) reads from the SDK host filesystem.
  • Distinguish LocalDir.src from the in-sandbox skills_path destination.
  • Add matching comments to sandbox examples that configure LocalDirLazySkillSource.

Test plan

  • make format
  • make lint
  • make build-docs
  • uv run pytest -n auto --dist loadfile -m "not serial" --ignore=tests/local

Not fully green locally due unrelated environment/local-checkout issues:

  • Full make tests collects ignored tests/local/old/... files in this checkout and fails importing customer_support_bot.
  • Serial tests with tests/local ignored fail during Docker/Testcontainers setup because Colima cannot create the Docker socket mount path.
  • make typecheck fails on existing ignored tests/local files plus existing Temporal example typing issues.

Issue number

Closes #2991

Checks

  • I've added new tests (if relevant)
  • I've added/updated the relevant documentation
  • I've run make lint and make format
  • I've made sure tests pass where not blocked by local environment issues

@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Apr 22, 2026
Copy link
Copy Markdown
Member

@seratch seratch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@seratch seratch merged commit 734d424 into main Apr 22, 2026
10 checks passed
@seratch seratch deleted the alfozan/clarify-lazy-skill-host-paths branch April 22, 2026 01:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Skill source directory unavailable on Windows

2 participants