Skip to content

docs(skills): clarify LocalDirLazySkillSource host paths#2993

Closed
MukundaKatta wants to merge 1 commit intoopenai:mainfrom
MukundaKatta:codex/skills-host-path-guidance
Closed

docs(skills): clarify LocalDirLazySkillSource host paths#2993
MukundaKatta wants to merge 1 commit intoopenai:mainfrom
MukundaKatta:codex/skills-host-path-guidance

Conversation

@MukundaKatta
Copy link
Copy Markdown

Summary

  • improve the lazy skills error message to explain that LocalDirLazySkillSource expects a host filesystem path
  • include the configured source path in the error context
  • document the host-path requirement in the sandbox skills guides
  • add a regression test covering the improved error context

Why

Windows users can end up passing a path that exists inside the sandbox image or workspace rather than the original host path. When that happens, the current error is too vague to explain what went wrong. This change makes the failure mode clearer and documents the intended setup.

Fixes #2991.

Verification

  • git diff --check
  • attempted targeted pytest, but local repo dependencies were not installed in this checkout (openai missing), so full test execution was not available here

@github-actions github-actions Bot added documentation Improvements or additions to documentation feature:sandboxes labels Apr 21, 2026
@MukundaKatta
Copy link
Copy Markdown
Author

Closing — lint failure, not going to push fixes.

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 feature:sandboxes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Skill source directory unavailable on Windows

1 participant