Skip to content

pass logdir path to support downstream logging#32

Merged
Priyank-Shethia3 merged 1 commit intocooperbench:mainfrom
Priyank-Shethia3:pass-logdir
Feb 7, 2026
Merged

pass logdir path to support downstream logging#32
Priyank-Shethia3 merged 1 commit intocooperbench:mainfrom
Priyank-Shethia3:pass-logdir

Conversation

@Priyank-Shethia3
Copy link
Collaborator

Description

Add optional log_dir parameter to agent runner interface to enable external agents to write output files to the correct location.

Closes #

Changes

  • Added log_dir: str | None parameter to AgentRunner Protocol in agents/__init__.py
  • Updated execute_coop() and _spawn_agent() in runner/coop.py to compute and pass log directory path based on run_name and features
  • Updated execute_solo() and _spawn_solo_agent() in runner/solo.py to compute and pass log directory path based on run_name and features
  • All changes are backward compatible - existing agents ignore the parameter, external agents can opt-in

Checklist

  • Code follows project style
  • Tests pass locally (34/34 unit tests passing)
  • Documentation updated (docstrings added for new parameter)

Co-authored-by: Cursor <cursoragent@cursor.com>
@akhatua2
Copy link
Collaborator

akhatua2 commented Feb 7, 2026

LGTM

@Priyank-Shethia3 Priyank-Shethia3 merged commit b1e7420 into cooperbench:main Feb 7, 2026
3 checks passed
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