fix: --resume now reuses existing run directory#192
Closed
jecanore wants to merge 2 commits intoaiming-lab:mainfrom
Closed
fix: --resume now reuses existing run directory#192jecanore wants to merge 2 commits intoaiming-lab:mainfrom
jecanore wants to merge 2 commits intoaiming-lab:mainfrom
Conversation
- Add resolve_config_path() to search for config.arc.yaml then config.yaml - Change --config default to None (auto-detect) on run/validate/doctor - Add _resolve_config_or_exit() helper with init hint on missing config - Add `researchclaw init` subcommand with interactive provider selection - String-based template replacement preserves YAML comments
… new one Previously, `researchclaw run --resume` always generated a fresh run_id and run_dir before checking for checkpoints, so it would look for a checkpoint in the new empty directory, find nothing, and start from scratch. Now --resume auto-detects the most recent run in artifacts/ that has a checkpoint.json, reads the original run_id from it, and resumes from the next stage. Also increases ACP timeout from 600s to 1200s to prevent CODE_GENERATION timeouts on complex experiments. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Collaborator
|
Thanks for the contribution! The core improvements in this PR (resolve_config_path, cmd_init, config auto-detection, resume improvements) have already been incorporated into the current codebase through prior merges. The remaining unique change (_find_latest_run with generic matching) differs from the current approach which matches by topic hash — a more precise strategy. Closing as superseded. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
--resumecreating a new run directory instead of reusing the existing one. Previously,cmd_run()always generated a freshrun_id/run_dirbefore checking checkpoints, so--resumewithout--outputwould always start from scratch._find_latest_run()helper that auto-detects the most recent run inartifacts/with acheckpoint.json, reads the originalrun_idfrom it, and resumes from the correct stage.CODE_GENERATIONstage timeouts on complex experiment prompts.Test plan
_find_latest_run()and resume error handlingtest_search_arxiv_mockunrelated to this change)researchclaw run --resumefinds latest run and resumes from checkpointresearchclaw run --resumewith no existing runs prints clear error🤖 Generated with Claude Code