Skip to content

chore(qa): harnesses clean up their temp folders - #60

Merged
writingdeveloper merged 1 commit into
mainfrom
chore/harness-temp-cleanup
Oct 21, 2026
Merged

writingdeveloper merged 1 commit into
mainfrom
chore/harness-temp-cleanup

Conversation

@writingdeveloper

Copy link
Copy Markdown
Owner

Every harness run left its sandbox in the OS temp directory, and src/main/diagnostics.test.ts left one folder per case. On the development PC that came to 1,379 folders, about 5 GB.

  • qa/sandbox.mjs: sandboxDir(prefix) creates the folder and removes it when the process exits with 0. A failing run keeps it and prints kept for inspection: ….
  • 11 harnesses use it instead of mkdtempSync. The 4 that already removed their sandbox are unchanged.
  • The diagnostics test removes its folders after the file.

Checked: qa/images-remote.mjs and qa/restore-pacing.mjs pass and leave nothing. A full vitest run (1510 passed) leaves nothing.

No app code changes.

🤖 Generated with Claude Code

https://claude.ai/code/session_016NLGwiBY5uEivJqERQzcoY

Every harness run left its sandbox in the OS temp directory, and the diagnostics unit test
left one folder per case: 1,379 folders, about 5 GB, after a few weeks. qa/sandbox.mjs
makes them and removes them when the run exits passing; a failing run keeps them and
prints where.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_016NLGwiBY5uEivJqERQzcoY
@writingdeveloper
writingdeveloper force-pushed the chore/harness-temp-cleanup branch from 12dcca4 to 2db7eb7 Compare October 21, 2026 23:48
@writingdeveloper
writingdeveloper merged commit fdbd8aa into main Oct 21, 2026
16 checks passed
@writingdeveloper
writingdeveloper deleted the chore/harness-temp-cleanup branch October 21, 2026 23:54
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.

1 participant