From e8fad549d2ccdfa4fef156a0cdfee68b751550ff Mon Sep 17 00:00:00 2001 From: Celine Lee Date: Tue, 17 Jun 2025 09:01:24 -0400 Subject: [PATCH 1/2] some minor fixes --- scripts/small_repos/run_claude.sh | 17 ++++++++++------- scripts/small_repos/run_codex.sh | 15 +++++++++------ 2 files changed, 19 insertions(+), 13 deletions(-) diff --git a/scripts/small_repos/run_claude.sh b/scripts/small_repos/run_claude.sh index 9f1a1c1e7..fe3651310 100644 --- a/scripts/small_repos/run_claude.sh +++ b/scripts/small_repos/run_claude.sh @@ -9,7 +9,7 @@ if [ $# -ne 1 ]; then fi directory="$1" -MODEL="codex-mini-latest" +ANTHROPIC_MODEL="claude-3-7-sonnet-20250219" if [ ! -d "$directory" ]; then echo "Error: Directory '$directory' does not exist" @@ -39,8 +39,9 @@ echo "Following the instructions in $base_dir/REFACTOR_INSTRUCTIONS.md..." echo "Running claude planner" # Claude plan -claude --dangerously-skip-permissions -p \ - "Follow the instructions in ../../../prompts/REFACTOR_INSTRUCTIONS.md. Only write PLAN.md. Give the file structure. Do not implement any code. Do not give example usage code." +claude --dangerously-skip-permissions -p </dev/null # Now that it's been refactored, remove all original persona subdirs @@ -70,8 +72,9 @@ pytest tests/ --json-report --json-report-file=report.json --continue-on-collect echo "Running final-check claude impl" # claude implement -claude --dangerously-skip-permissions -p \ - "Read the pytest results in test_output.txt. If they indicate pytest failures, fix them. Stick to the implementation plan and file structure proposed in PLAN.md. IMPORTANT: Create and modify files ONLY in this current working subdirectory ($new_directory/unified) and nowhere else. Implement ALL code and update ALL test file imports until this whole subfolder is a functional standalone repository. Do not stop to ask for confirmation; keep going until the final implementation passes all tests using pytest tests/. If there are no errors, exit." +claude --dangerously-skip-permissions -p < test_output.txt 2>&1 deactivate diff --git a/scripts/small_repos/run_codex.sh b/scripts/small_repos/run_codex.sh index 139133a1d..c44c94ce0 100644 --- a/scripts/small_repos/run_codex.sh +++ b/scripts/small_repos/run_codex.sh @@ -41,8 +41,9 @@ echo "Following the instructions in $base_dir/prompts/REFACTOR_INSTRUCTIONS.md.. echo "Running codex planner" # Codex plan -codex --approval-mode full-auto -q \ - "Follow the instructions in ../../../prompts/REFACTOR_INSTRUCTIONS.md. Only write PLAN.md. Give the file structure. Do not implement any code. Do not give example usage code." +codex --approval-mode full-auto -q </dev/null # Now that it's been refactored, remove all original persona subdirs @@ -71,8 +73,9 @@ pytest tests/ --json-report --json-report-file=report.json --continue-on-collect echo "Running final-check codex impl" # Codex implement -codex --approval-mode full-auto -q \ - "Read the pytest results in test_output.txt. If they indicate pytest failures, fix them. Stick to the implementation plan and file structure proposed in PLAN.md. IMPORTANT: Create and modify files ONLY in this current working subdirectory ($new_directory/unified) and nowhere else. Implement ALL code and update ALL test file imports until this whole subfolder is a functional standalone repository. Do not stop to ask for confirmation; keep going until the final implementation passes all tests using pytest tests/. If there are no errors, exit." +codex --approval-mode full-auto -q < test_output.txt 2>&1 deactivate From 283cba516490cf973c3c35625ae2ef51768c6283 Mon Sep 17 00:00:00 2001 From: Celine Lee Date: Tue, 17 Jun 2025 09:33:09 -0400 Subject: [PATCH 2/2] codex line preferences --- scripts/small_repos/run_codex.sh | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/scripts/small_repos/run_codex.sh b/scripts/small_repos/run_codex.sh index c44c94ce0..4f3f9fda4 100644 --- a/scripts/small_repos/run_codex.sh +++ b/scripts/small_repos/run_codex.sh @@ -41,9 +41,7 @@ echo "Following the instructions in $base_dir/prompts/REFACTOR_INSTRUCTIONS.md.. echo "Running codex planner" # Codex plan -codex --approval-mode full-auto -q </dev/null # Now that it's been refactored, remove all original persona subdirs @@ -73,9 +69,7 @@ pytest tests/ --json-report --json-report-file=report.json --continue-on-collect echo "Running final-check codex impl" # Codex implement -codex --approval-mode full-auto -q < test_output.txt 2>&1 deactivate