Skip to content

fix(shell): clean up session temporary files - #107

Open
Saksham1105 wants to merge 1 commit into
knight22-21:mainfrom
Saksham1105:fix/shell-session-temp-cleanup
Open

Saksham1105 wants to merge 1 commit into
knight22-21:mainfrom
Saksham1105:fix/shell-session-temp-cleanup

Conversation

@Saksham1105

Copy link
Copy Markdown
Contributor

Summary

  • Add safe, idempotent cleanup for \ShellSession\ temporary directory and output files
  • Implement context-manager support (_enter_\ / _exit_) on \ShellSession\
  • Register process-exit cleanup via \�texit.register(session.cleanup)\ in
    egister_shell_tool\
  • Add comprehensive regression tests in \ ests/test_shell_tool.py\

Testing

  • Focused test suite: .venv\Scripts\python.exe -m pytest tests/test_shell_tool.py\ (21 passed)
  • Full test suite: .venv\Scripts\python.exe -m pytest tests/\ (847 passed, 2 skipped)
  • Ruff lint check: .venv\Scripts\ruff check devagent/ tests/\ (All checks passed!)

Closes #99

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.

ShellSession leaks temporary directories in tempfile without cleanup on exit

1 participant