Skip to content

Archive session transcripts before cleanup#88

Merged
olearycrew merged 1 commit intopinchbench:mainfrom
juppytt:feat/transcript-archive
Apr 6, 2026
Merged

Archive session transcripts before cleanup#88
olearycrew merged 1 commit intopinchbench:mainfrom
juppytt:feat/transcript-archive

Conversation

@juppytt
Copy link
Copy Markdown
Contributor

@juppytt juppytt commented Apr 1, 2026

Summary

  • Session transcripts were deleted between tasks by cleanup_agent_sessions, making post-run debugging impossible
  • Now transcripts are copied to results/{run_id}_transcripts/{task_id}.jsonl before cleanup
  • _load_transcript returns the file path alongside the parsed data so it can be archived

Test plan

  • Run a benchmark and verify results/{run_id}_transcripts/ is created with per-task JSONL files
  • Verify transcript content matches what OpenClaw produced
  • Verify existing behavior (grading, cleanup) is unaffected

🤖 Generated with Claude Code

Session transcripts were deleted between tasks by cleanup_agent_sessions,
making post-run debugging impossible. Now transcripts are copied to
results/{run_id}_transcripts/{task_id}.jsonl before cleanup.

Also fixes pre-existing duplicate _remove_readonly function definition
that caused a SyntaxError on import.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@olearycrew
Copy link
Copy Markdown
Member

@juppytt thanks for this contribution!

@olearycrew olearycrew merged commit 5d31e1a into pinchbench:main Apr 6, 2026
1 check failed
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.

2 participants