154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
391
392
393
394
395
396
397
398
399
400
401
402
403
404
405
406
407
408
409
410
411
412
413
414
415
416
417
418
419
420
421
422
423
424
425
426
427
428
429
430
431
432
433
434
435
436
437
438
439
440
441
442
443
444
445
446
447
448
449
450
451
452
453
454
455
456
457
458
459
460
461
462
463
464
465
466
467
468
469
470
471
472
473
474
475
476
477
478
479
480
481
482
483
484
485
486
487
488
489
490
491
492
493
494
495
496
497
498
499
500
501
502
503
504
505
506
507
508
509
510
511
512
513
514
515
516
517
518
519
520
521
522
523
524
525
526
527
528
529
530
531
532
533
534
535
536
537
538
539
540
541
542
543
544
545
546
547
548
549
550
551
552
553
554
555
556
557
558
559
560
561
562
563
564
565
566
567
568
569
570
571
572
573
574
575
576
577
578
579
580
581
582
583
584
585
586
587
588
589
590
591
592
593
594
595
596
597
598
599
600
601
602
603
604
605
606
607
608
609
610
611
612
613
614
615
616
617
618
619
620
621
622
623
624
625
626
627
628
629
630
631
632
633
634
635
636
637
638
639
640
641
642
643
644
645
646
647
648
649
650
651
652
653
654
655
656
657
658
659
660
661
662
663
664
665
666
667
668
669
670
671
672
673
674
675
676
677
678
679
680
681
682
683
684
685
686
687
688
689
690
691
692
693
694
695
696
697
698
699
700
701
702
703
704
705
706
707
708
709
710
711
712
713
714
715
716
717
718
719
720
721
722
723
724
725
726
727
728
729
730
731
732
733
734
735
736
737
738
739
740
741
742
743
744
745
746
747
748
749
750
751
752
753
754
755
756
757
758
759
760
761
762
763
764
765
766
767
768
769
770
771
772
773
774
775
776
777
778
779
780
781
782
783
784
785
786
787
788
789
790
791
792
793
794
795
796
797
798
799
800
801
802
803
804
805
806
807
808
809
810
811
812
813
814
815
816
817
818
819
820
821
822
823
824
825
826
827
828
829
830
831
832
833
834
835
836
837
838
839
840
841
842
843
844
845
846
847
848
849
850
851
852
853
854
855
856
857
858
859
860
861
862
863
864
865
866
867
868
869
870
871
872
873
874
875
876
877
878
879
880
881
882
883
884
885
886
887
888
889
890
891
892
893
894
895
896
897
898
899
900
901
902
903
904
905
906
907
908
909
910
911
912
913
914
915
916
917
918
919
920
921
922
923
924
925
926
927
928
929
930
931
932
933
934
935
936
937
938
939
940
941
942
943
944
945
946
947
948
949
950
951
952
953
954
955
956
957
958
959
960
961
962
963
964
965
966
967
968
969
970
971
972
973
974
975
976
977
978
979
980
981
982
983
984
985
986
987
988
989
990
991
992
993
994
995
996
997
998
999
1000
1001
1002
1003
1004
1005
1006
1007
1008
1009
1010
1011
1012
1013
1014
1015
1016
1017
1018
1019
1020
1021
1022
1023
1024
1025
1026
1027
1028
1029
1030
1031
1032
1033
1034
1035
1036
1037
1038
1039
1040
1041
1042
1043
1044
1045
1046
1047
1048
1049
1050
1051
1052
1053
1054
1055
1056
1057
1058
1059
1060
1061
1062
1063
1064
1065
1066
1067
1068
1069
1070
1071
1072
1073
1074
1075
1076
1077
1078
1079
1080
1081
1082
1083
1084
1085
1086
1087
1088
1089
1090
1091
1092
1093
1094
1095
1096
1097
1098
1099
1100
1101
1102
1103
1104
1105
1106
1107
1108
1109
1110
1111
1112
1113
1114
1115
1116
1117
1118
1119
1120
1121
1122
1123
1124
1125
1126
1127
1128
1129
1130
1131
1132
1133
1134
1135
1136
1137
1138
1139
1140
1141
1142
1143
1144
1145
1146
1147
1148
1149
1150
1151
1152
1153
1154
1155
1156
1157
1158
1159
1160
1161
1162
1163
1164
1165
1166
1167
1168
1169
1170
1171
1172
1173
1174
1175
1176
1177
1178
1179
1180
1181
1182
1183
1184
1185
1186
1187
1188
1189
1190
1191
1192
1193
1194
1195
1196
1197
1198
1199
1200
1201
1202
1203
1204
1205
1206
1207
1208
1209
1210
1211
1212
1213
1214
1215
1216
1217
1218
1219
1220
1221
1222
1223
1224
1225
1226
1227
1228
1229
1230
1231
1232
1233
1234
1235
1236
1237
1238
1239
1240
1241
1242
1243
1244
1245
1246
1247
1248
1249
1250
1251
1252
1253
1254
1255
1256
1257
1258
1259
1260
1261
1262
1263
1264
1265
1266
1267
1268
1269
1270
1271
1272
1273
1274
1275
1276
1277
1278
1279
1280
1281
1282
1283
1284
1285
1286
1287
1288
1289
1290
1291
1292
1293
1294
1295
1296
1297
1298
1299
1300
1301
1302
1303
1304
1305
1306
1307
1308
1309
1310
1311
1312
1313
1314
1315
1316
1317
1318
1319
1320
1321
1322
1323
1324
1325
1326
1327
1328
1329
1330
1331
1332
1333
1334
1335
1336
1337
1338
1339
1340
1341
1342
1343
1344
1345
1346
1347
1348
1349
1350
1351
1352
1353
1354
1355
1356
1357
1358
1359
1360
1361
1362
1363
1364
1365
1366
1367
1368
1369
1370
1371
1372
1373
1374
1375
1376
1377
1378
1379
1380
1381
1382
1383
1384
1385
1386
1387
1388
1389
1390
1391
1392
1393
1394
1395
1396
1397
1398
1399
1400
1401
1402
1403
1404
1405
1406
1407
1408
1409
1410
1411
1412
1413
1414
1415
1416
1417
1418
1419
1420
1421
1422
1423
1424
1425
1426
1427
1428
1429
1430
1431
1432
1433
1434
1435
1436
1437
1438
1439
1440
1441
1442
1443
1444
1445
1446
1447
1448
1449
1450
1451
1452
1453
1454
1455
1456
1457
1458
1459
1460
1461
1462
1463
1464
1465
1466
1467
1468
1469
1470
1471
1472
1473
1474
1475
1476
1477
1478
1479
1480
1481
1482
1483
1484
1485
1486
1487
1488
1489
1490
1491
1492
1493
1494
1495
1496
1497
1498
1499
1500
1501
1502
1503
1504
1505
1506
1507
1508
1509
1510
1511
1512
1513
1514
1515
1516 | class SWEEnv(gym.Env):
"""Gym environment for SWE-bench. This class should handle all communication with the docker container."""
name = "swe_main"
# This prefix will be prepended to the image name when caching task images
cached_image_prefix = "swe-agent-task-env-"
def __init__(self, args: EnvironmentArguments):
super().__init__()
t0 = time.perf_counter()
self.args = args
self.base_commit: str | None = None
self.communicate_output: str | None = None
self.container_name: str | None = args.container_name
self.install_environment = args.install_environment
self.logger = get_logger("SWEEnv")
self.persistent = args.container_name is not None
self.container_mounts = args.container_mounts
self.returncode: None | int = None
if not self.args.verbose:
# fixme: This creates problems if we have multiple instances of this class
self.logger.disabled = True
#: The commit hash of the swe-agent repository
self.commit_sha = None
try:
repo = Repo(REPO_ROOT, search_parent_directories=True)
self.commit_sha = repo.head.object.hexsha
except KeyboardInterrupt:
raise
except Exception as e:
self.logger.exception("Failed to get commit hash for this repo: %s", str(e))
self._github_token: str = keys_config.get("GITHUB_TOKEN", "") # type: ignore
# Load Task Instances
self.data_path = self.args.data_path
self.data = get_instances(
self.data_path,
self.args.base_commit,
self.args.split,
token=self._github_token,
repo_path=self.args.repo_path,
)
#: Instance we're currently processing. Gets set in self.reset.
self.record: dict[str, Any] | None = None
self.logger.info(f"💽 Loaded dataset from {self.data_path}")
# Establish connection with execution container
self.image_name = args.image_name
self.container_obj: docker.models.containers.Container | None = None
self.container: subprocess.Popen | None = None
self.docker_compose: Path | None = None
self.challenge: dict[str, Any] | None = None
self._reset_container()
self.interactive_session: InteractiveSession | None = None
self.idx = 0
self.clean_multi_line_functions = lambda x: x
self.hooks: list[EnvHook] = []
self.logger.debug("Environment initialization took %.2f seconds", time.perf_counter() - t0)
def _get_cached_task_image_name(self) -> str:
assert self.record is not None
inputs: list[str] = [
self.record["repo"],
self.record["base_commit"],
self.args.environment_setup or "no_setup",
]
tag = hashlib.sha256("".join(inputs).encode()).hexdigest()[:50]
return f"{self.cached_image_prefix}{tag}"
def add_hook(self, hook: EnvHook):
"""Add `EnvHook` to the environment.
This allows to inject custom functionality at different stages of the environment
lifecycle, in particular to connect SWE-agent to a new interface (like a GUI).
"""
hook.on_init()
self.hooks.append(hook)
@property
def _repo_name(self) -> str:
"""Name of the local copy of the repository"""
assert self.record is not None
return self.record["repo"].replace("/", "__").replace(" ", "-").replace("'", "")
def _copy_repo(self) -> str:
"""Clone/copy repository/codebase in container
Returns:
folder name of clone
"""
assert self.container_obj is not None
assert self.record is not None # mypy
for hook in self.hooks:
hook.on_copy_repo_started(repo_type=self.record["repo_type"], repo_path=self.record["repo"])
if self.record["repo_type"] == "local":
if "challenge" in self.record:
self.communicate_with_handling(
input=f"mkdir {self._repo_name}", error_msg=f"Failed to create {self._repo_name} in container"
)
for file_name in self.record["challenge"]["files"]:
self.logger.debug(f"Copying file {file_name} to container")
copy_anything_to_container(
self.container_obj,
str(Path(self.record["repo"].removeprefix("local://")) / file_name),
"/" + self._repo_name,
)
else:
copy_anything_to_container(
self.container_obj,
self.record["repo"].removeprefix("local://"),
"/" + self._repo_name,
)
self.communicate_with_handling(
input=f"chown -R root:root {self._repo_name}",
error_msg="Failed to change permissions on copied repository",
)
return self._repo_name
assert self.record["repo_type"] == "github"
token_prefix = ""
if self._github_token:
token_prefix = f"{self._github_token}@"
# fixme: This if statement is brittle and should probably be replaced with better logic
if not self.args.no_mirror and self.record["problem_statement_source"] == "swe-bench":
self.logger.info(f"{self._repo_name} not found in container, cloning...")
clone_url = f"https://{token_prefix}github.com/swe-bench/{self._repo_name}.git"
else:
self.logger.info("Trying to clone from non-mirror...")
clone_url = f"https://{token_prefix}github.com/{self.record['repo']}.git"
clone_method = keys_config.get("SWE_AGENT_CLONE_METHOD", default="shallow", choices=["shallow", "full"])
if len(self.data) > 1 or self.persistent:
msg = "Falling back to full cloning method due to multiple instances or persistent container"
clone_method = "full"
self.logger.debug(msg)
if clone_method == "full":
self.communicate_with_handling(
input=f"git clone {clone_url} {self._repo_name}",
error_msg="Failed to clone repository from conservative method",
timeout_duration=LONG_TIMEOUT,
redact_command_trace=True,
)
else:
base_commit = self.record["base_commit"]
self.communicate_with_handling(
input="&&".join(
(
f"mkdir {self._repo_name}",
f"cd {self._repo_name}",
"git init",
f"git remote add origin {clone_url}",
f"git fetch --depth 1 origin {base_commit}",
"git checkout FETCH_HEAD",
"cd ..",
)
),
error_msg="Failed to clone repository with fast method",
timeout_duration=LONG_TIMEOUT,
)
return self._repo_name
def reset(self, index: int | None = None, apply_test_patch: bool = False) -> tuple[str | None, dict]:
"""
Function to reset container between each task instance.
* Clones instance's repository
* Cleans repository of prior modifications
* Resets environment variables
* Check out base commit
Args:
index: index of task instance to reset to
Returns:
observation: output from container
info: additional information (e.g. debugging information)
"""
info = {}
info["commit_sha"] = self.commit_sha
# Get task instance
self.idx = index if index is not None else self.idx
self.record = self.data[self.idx]
self.idx += 1
# Set query, gold command
self.base_commit = self.record["base_commit"]
self.query = self.record["problem_statement"]
self.challenge = self.record.get("challenge")
self.reward = None
### Reset Container ###
self._init_docker_compose()
if self.args.cache_task_images:
cached_image = self._get_cached_task_image_name()
if image_exists(cached_image):
self.logger.info(f"Restore environment from cached image {cached_image}")
self.close() # stop current container
self._init_container(cached_image=cached_image)
self.communicate("export $(xargs </.env)")
envs = self.communicate("env")
self.logger.debug(f"Environment variables restored from the image:\n{envs}\n")
if apply_test_patch:
self._apply_test_patch()
return None, info
else:
self.logger.info(f"Cached image {cached_image} not found, rebuilding task environment...")
# Init docker network
self._init_docker_network()
# Clone repository if not already cloned
self.communicate(input="cd /")
folders = self.communicate(input="ls").split("\n")
if self._repo_name not in folders:
self._copy_repo()
self._reset_repository()
self._reset_environment_variables()
# Set up environment
self.communicate_with_handling(
"source /root/miniconda3/etc/profile.d/conda.sh",
error_msg="Failed to source conda",
)
system = self.communicate("uname -s").strip().lower()
arch = self.communicate("uname -m").strip().lower()
if system == "linux" and arch == "x86_64":
self.communicate_with_handling(
"apt update; apt install build-essential -y",
error_msg="Failed to install build-essential",
timeout_duration=LONG_TIMEOUT,
)
# Call install environment helper function if specified
if self.install_environment:
self.install_env()
# Install mypy for linting purposes
self.communicate_with_handling("pip install flake8", error_msg="Failed to install flake8 (lint library)")
if self.args.cache_task_images:
envs = self.communicate("env")
self.logger.debug(f"Environment variables to save:\n{envs}\n")
self.communicate("env >> /.env")
assert self.container_obj is not None # mypy
self.container_obj.commit(cached_image)
self.logger.info(f"Container with environment {self.container_obj.id} cached as image {cached_image}")
if apply_test_patch:
self._apply_test_patch()
# Write any metadata to info if necessary
return None, info
def _reset_repository(self) -> None:
"""Clean repository of any modifications + Checkout base commit"""
startup_commands = [
"echo -n > /root/files_to_edit.txt",
f"cd /{self._repo_name}",
"export ROOT=$(pwd -P)",
]
if self.challenge is None:
startup_commands += [
"git status",
"git restore .",
f"git reset --hard {self.base_commit}",
"git clean -fdxq",
]
self.communicate_with_handling(
input=" && ".join(startup_commands),
error_msg="Failed to clean repository",
)
def _reset_environment_variables(self) -> None:
"""Reset environment variables (`CURRENT_FILE`) etc. within container"""
cmd = [
'export CURRENT_FILE=""',
"export CURRENT_LINE=0",
"export SEARCH_RESULTS=()",
"export SEARCH_FILES=()",
"export SEARCH_INDEX=0",
]
self.communicate_with_handling(
input=" && ".join(cmd),
error_msg="Failed to reset environment variables",
)
def reset_for_new_attempt(
self,
) -> None:
"""Compared to `reset`, which prepares the container for a new instance,
this prepares the container for taking another shot at the same instance.
"""
self._reset_repository()
self._reset_environment_variables()
def _apply_test_patch(self):
"""
Apply test patch for oracle setting
"""
assert self.record is not None
path_to_patch = "test.patch"
with open(path_to_patch, "w") as f:
f.write(self.record["test_patch"])
subprocess.run(
f"docker cp {path_to_patch} {self.container_name}:/root/test.patch",
shell=True,
check=False,
)
self.communicate_with_handling(
input="git apply /root/test.patch",
error_msg="Failed to apply test patch correctly",
)
os.remove(path_to_patch)
def _get_edited_files_with_context(self, patch: str) -> dict[str, str]:
"""Get the edited files with context from the patch"""
pf = PatchFormatter(patch, read_method=self.read_file) if patch else None
out = {}
for context_length in [30, 50, 70]:
value = "Empty. No edited files found."
if pf is not None:
value = pf.get_files_str(original=False, context_length=context_length)
out[f"edited_files{context_length}"] = value
return out
def _terminate_interactive_session(self, session_name: str):
if not self.interactive_session:
# Maybe fixing #772
return
try:
self.interactive_session.session_process.terminate()
self.communicate(self.interactive_session.config.exit_command)
except Exception as e:
msg = (
f"Failed to terminate interactive session {session_name}: {e}."
"\nHere's the full traceback\n" + traceback.format_exc()
)
self.logger.warning(msg)
self.interactive_session = None
def _handle_interactive_commands(self, observation: str) -> str:
"""Handle interactive commands in the environment, essentially substituting dummy
output for the actual output of the interactive commands.
Args:
observation: Output from running the interactive command wrappers in the
environment. They will returns some dummy output that will be caught and then
we will run the actual commands in the interactive session and return the
actual output.
Returns:
observation: The observation shown to the model. If no interactive commands
are detected, this is the same as the input observation.
Else, only the output from the interactive commands is returned.
"""
session_name, interactive_commands = get_interactive_commands(observation, logger=self.logger)
if session_name is None:
return observation
if (
session_name is not None
and self.interactive_session is not None
and self.interactive_session.name != session_name
):
return self.interactive_session._get_only_one_interactive_error_message_observation()
observation = ""
for command in interactive_commands:
if command == "START":
# Start the session if previous session does not exist
if self.interactive_session is not None:
return self.interactive_session._get_only_one_interactive_error_message_observation()
assert self.container_name is not None
_observation, self.interactive_session = get_interactive_session(
ctr_name=self.container_name,
ctr_obj=self.container_obj,
cwd="/" + self._repo_name,
session_name=session_name,
config=self.args.interactive_sessions_config[session_name],
logger=self.logger,
)
observation += _observation
elif command == "STOP":
if self.interactive_session is None:
observation = f"Interactive session {session_name!r} is not running, so it cannot be stopped!"
else:
if self.interactive_session.session_process.poll() is None:
self.logger.warning("Session did not quit successfully, terminating.")
self.interactive_session.session_process.terminate()
observation = f"Interactive session {session_name!r} stopped successfully"
self.interactive_session = None
else:
if self.interactive_session is None:
self.logger.warning("Tried to run interactive commands without starting session")
start_command = self.args.interactive_sessions_config[session_name].start_command
observation = f"Interactive session {session_name!r} is not running! please start it first using `{start_command}`"
elif self.interactive_session and self.interactive_session.session_process.poll() is not None:
start_command = self.args.interactive_sessions_config[session_name].start_command
observation = f"Interactive session {session_name!r} was unexpectedly closed! Please start it again using `{start_command}`"
self._terminate_interactive_session(session_name=session_name)
else:
_observation, terminate = self.interactive_session.communicate_with_handling(
command,
timeout_duration=AGENT_ACTION_TIMEOUT,
no_output_timeout_duration=AGENT_ACTION_NO_OUTPUT_TIMEOUT,
)
observation += _observation
if terminate:
self._terminate_interactive_session(session_name=session_name)
observation += "\n"
return observation
def step(self, action: str) -> tuple[str | None, int, bool, AgentInfo]:
"""
Runs an action proposed by the agent in the environment and returns the corresponding output.
Args:
action: command to run in bash shell
Returns:
observation: output from container
reward: Always set to 0
done: whether task is over
info: additional information (e.g. debugging information)
"""
info: AgentInfo = {}
# Make sure to have the right keys even if the submission is missing/empty
info.update(self._get_edited_files_with_context(patch="")) # type: ignore
observation = ""
# Handle special actions
action = action.strip()
if action == "skip":
observation = "Skipped"
info["exit_status"] = "skipped"
return observation, 0, True, info
if action == "exit_forfeit":
observation = "Exited"
info["exit_status"] = action
return observation, 0, True, info
if action in {"exit_context", "exit_cost", "exit_error", "exit_format", "exit_api"}:
try:
observation = self.communicate(input="submit")
submission = self.get_submission(observation)
assert submission is not None and submission.strip() != "", AssertionError("No submission found.")
self.logger.info(f"Found submission: {submission}")
info["exit_status"] = f"submitted ({action})"
info["submission"] = submission
info.update(self._get_edited_files_with_context(patch=submission)) # type: ignore
observation = "Exited (autosubmitted)"
self.logger.info("Exiting with autosubmission")
return observation, 0, True, info
except KeyboardInterrupt:
raise
except:
observation = "Exited"
info["exit_status"] = action
return observation, 0, True, info
# Attempt to run action in container
observation = ""
try:
observation = self.communicate(
input=action,
timeout_duration=AGENT_ACTION_TIMEOUT,
no_output_timeout_duration=AGENT_ACTION_NO_OUTPUT_TIMEOUT,
set_last_action=True,
)
except TimeoutError as e:
try:
observation += e.args[1] if len(e.args) > 1 else ""
observation += self.interrupt()
observation += "\nEXECUTION TIMED OUT"
observation += (
f" BECAUSE NO OUTPUT WAS PRODUCED FOR MORE THAN {AGENT_ACTION_NO_OUTPUT_TIMEOUT} SECONDS.\nPLEASE REFINE YOUR RUNNING COMMAND SO IT WILL PRODUCE OUTPUT IN THE SPECIFIED TIME FRAME."
if isinstance(e, NoOutputTimeoutError)
else f" BECAUSE THE COMMAND WAS RUNNING FOR MORE THAN {AGENT_ACTION_TIMEOUT} SECONDS."
)
except RuntimeError as e:
observation += e.args[1] if len(e.args) > 1 else ""
observation += "\nEXECUTION TIMED OUT AND INTERRUPT FAILED. RESTARTING PROCESS."
info["exit_status"] = "early_exit"
self.logger.warning(f"Failed to interrupt container: {e}\nRESTARTING PROCESS.")
self.reset_container()
return observation, 0, True, info
except RuntimeError as e:
observation += e.args[1] if len(e.args) > 1 else ""
observation += "\nCOMMAND FAILED TO EXECUTE. RESTARTING PROCESS."
info["exit_status"] = "early_exit"
self.logger.warning(f"Failed to execute command: {e}\nRESTARTING PROCESS.")
self.reset_container()
return observation, 0, True, info
except BrokenPipeError as e:
observation += "\nBROKEN PIPE ERROR. RESTARTING PROCESS."
info["exit_status"] = "early_exit"
self.logger.error(f"Broken pipe error: {e}\nRESTARTING PROCESS.")
self.reset_container()
return observation, 0, True, info
except UnicodeError as e:
observation += "\nCOMMAND PRODUCED TOO MANY NON-UNICODE CHARACTERS. PLEASE TRY ANOTHER COMMAND.\nIF YOU WANT TO VIEW BINARY FILES, PLEASE USE `xxd` OR `hexdump` INSTEAD.\n"
self.logger.error(f"Unicode error: {e}")
except Exception:
observation += "\nEXECUTION FAILED OR COMMAND MALFORMED"
self.logger.exception("Unknown exception")
# Record submission and end episode if `submit` keyword found
submission = self.get_submission(observation)
if submission is not None:
if self.validate_submission(submission):
self.logger.info(f"Found submission: {submission}")
info["exit_status"] = "submitted"
info["submission"] = submission if submission.strip() != "" else None
info.update(self._get_edited_files_with_context(patch=submission)) # type: ignore
observation = submission if submission.strip() != "" else None
return observation, 0, True, info
else:
# Currently only validating CTF challenges
assert self.challenge is not None
self.logger.warning(f"Wrong submission found: {submission} (real flag is {self.challenge['flag']})")
observation = "Wrong flag!"
return observation, 0, False, info
observation = self._handle_interactive_commands(observation)
return observation, 0, False, info
def close(self) -> None:
"""
Handle environment shutdown
"""
self.logger.info("Beginning environment shutdown...")
try:
self.communicate(input="exit")
except KeyboardInterrupt:
raise
except:
self.logger.warning("Errors when exiting container", exc_info=True)
assert self.container is not None # mypy
self.container.terminate()
if self.docker_compose is not None:
terminate_docker_compose(self.docker_compose)
if self.interactive_session is not None:
try:
self.interactive_session.session_process.terminate()
except KeyboardInterrupt:
raise
except Exception:
self.logger.warning("Failed to stop interactive session: %s", traceback.format_exc())
self.interactive_session = None
else:
self.logger.info("Interactive session stopped")
self.interactive_session = None
if self.container_obj is None:
pass
elif self.persistent:
# stopping is Podman specific, but doesn't hurt to include
# https://stackoverflow.com/a/32428199/
# Want to avoid https://github.com/princeton-nlp/SWE-agent/issues/496
# Note that container_obj.status might not be updated throughout the container
# lifecycle, so let's get the container_obj again
assert self.container_name
try:
self.container_obj = docker.from_env().containers.get(self.container_name)
except Exception:
self.logger.warning(f"Failed to get fresh container object: {traceback.format_exc()}", exc_info=True)
if self.container_obj.status not in {"paused", "exited", "dead", "stopping"}:
try:
self.container_obj.pause()
except Exception:
self.logger.warning("Failed to pause container.", exc_info=True)
except KeyboardInterrupt:
raise
else:
self.logger.info("Agent container paused")
else:
self.logger.info(f"Agent container status: {self.container_obj.status}")
else:
try:
self.container_obj.remove(force=True)
except KeyboardInterrupt:
raise
except docker.errors.NotFound:
# We already tried to exit the container, so it's actually good if
# it's not found
pass
except Exception:
self.logger.warning("Failed to remove container", exc_info=True)
else:
self.logger.info("Agent container stopped")
for hook in self.hooks:
hook.on_close()
# MARK: Helper functions #
def _reset_container(self) -> None:
if self.container is not None:
try:
self.container.terminate()
except KeyboardInterrupt:
raise
except:
self.logger.warning("Failed to terminate container", exc_info=True)
else:
self.logger.debug("Terminated container")
if self.docker_compose is not None:
try:
terminate_docker_compose(self.docker_compose)
except KeyboardInterrupt:
raise
except:
self.logger.warning("Failed to terminate docker compose", exc_info=True)
else:
self.logger.debug("Terminated docker compose")
self._init_container()
self._init_scripts()
def reset_container(self) -> None:
self.close()
self.container = None
self.container_obj = None
self._reset_container()
@staticmethod
def _get_container_name(image_name: str) -> str:
"""Return name of container"""
process_id = str(os.getpid())
current_time = str(datetime.datetime.now())
unique_string = current_time + process_id
hash_object = hashlib.sha256(unique_string.encode())
image_name_sanitized = image_name.replace("/", "-")
image_name_sanitized = image_name_sanitized.replace(":", "-")
return f"{image_name_sanitized}-{hash_object.hexdigest()[:10]}"
# ctf
def _init_docker_network(self) -> None:
"""
Add the "ctfnet" network interface for all the containers used for CTF challenges
"""
assert self.container_name is not None
if self.challenge is not None:
attach_network_interface_to_container(self.container_name)
# ctf
def _init_docker_compose(self) -> None:
"""
Handles docker compose initialization for challenge with docker compose file.
"""
if self.challenge is not None and self.challenge.get("docker_compose") is not None:
self.docker_compose = get_docker_compose(self.challenge["docker_compose"])
self.logger.info("🌱 Initialized docker compose for challenge")
def _init_container(self, cached_image: str | None = None) -> None:
"""
Handles container initialization. Defines container name and creates it.
If cached_image is provided, it will use that image name instead of the default.
"""
image_name = self.image_name
if cached_image is not None:
image_name = cached_image
self.logger.info(f"Using cached image: {image_name}")
if self.persistent:
assert self.container_name is not None
else:
# Make sure that we get a new container name just in case removing didn't work.
# Might be a fix for https://github.com/princeton-nlp/SWE-agent/issues/451
self.container_name = self._get_container_name(image_name)
self.container, self.parent_pids = get_container(
self.container_name, image_name, persistent=self.persistent, container_mounts=self.container_mounts
)
try:
client = docker.from_env(timeout=600)
except docker.errors.DockerException as e:
if "Error while fetching server API version" in str(e):
msg = "Docker is not running. Please start Docker and try again."
else:
msg = "Unknown docker exception occurred. Are you sure docker is running?"
raise RuntimeError(msg) from e
t0 = time.time()
self.container_obj = None
while time.time() - t0 < 60:
try:
self.container_obj = client.containers.get(self.container_name)
except docker.errors.NotFound:
self.logger.debug("Couldn't find container. Let's wait and retry.")
time.sleep(1)
else:
break
else:
print(f"{self.persistent=}")
available_containers = client.containers.list(all=True)
available_containers_info = json.dumps([str(c.attrs) for c in available_containers], indent=2)
print(available_containers_info)
msg = "Failed to get container object."
raise RuntimeError(msg)
self.logger.info("🌱 Environment Initialized")
def _init_scripts(self):
"""
Initialize custom commands within container
"""
self.communicate_with_handling(
"source /root/.bashrc",
error_msg="Failed to source .bashrc",
)
self.communicate_with_handling(
"mkdir -p /root/commands",
error_msg="Failed to create commands directory",
)
self.communicate_with_handling(
"touch /root/commands/__init__.py",
error_msg="Failed to create __init__.py",
)
self.communicate_with_handling(
"export PATH=$PATH:/root/commands",
error_msg="Failed to add commands directory to PATH",
)
def _communicate_experimental(
self,
input: str,
timeout_duration: int | float = 25,
no_output_timeout_duration: int | float = 25,
) -> str:
"""Experimental version of `_communicate`"""
assert self.container is not None
# Sleep to ensure that the exit code is in the last line
# See https://github.com/princeton-nlp/SWE-agent/issues/595
command_suffix = (
f'EXITSTATUS="$?"; sleep 0.01; echo {PROCESS_DONE_MARKER_START}$EXITSTATUS{PROCESS_DONE_MARKER_END}\n'
)
try:
self.returncode = None
cmd = input if input.endswith("\n") else input + "\n"
cmd += command_suffix
os.write(self.container.stdin.fileno(), cmd.encode()) # type: ignore
time.sleep(0.03)
self.container.stdin.flush() # type: ignore
except BrokenPipeError:
traceback.print_exc()
self.logger.error("Failed to communicate with container. Check docker logs for more information.")
msg = "Failed to communicate with container"
raise RuntimeError(msg)
try:
buffer, exit_code = read_with_timeout_experimental(
self.container, timeout_duration, no_output_timeout_duration
)
except Exception:
msg = f"Read with timeout failed on input:\n---\n{input}\n---"
self.logger.error(msg)
raise
if exit_code == "$EXITSTATUS":
# this sometimes happens if the command badly fails
# for example if you just try to run python with no arguments
# in this case, the error message is usually also garbage, so let's set
# something new.
# See https://github.com/princeton-nlp/SWE-agent/issues/630
buffer = (
"Unkknown error occurred when running the command. Please double check syntax "
"and that you're not running an interactive command."
)
self.logger.warning("Couldn't get real exit code. Setting it to 999")
exit_code = 999
elif not exit_code.isdigit():
# this sometimes happens if the command is being killed, for example radare2
# we set the error to 998 in that case
self.logger.warning("Couldn't get real exit code. Setting it to 998")
exit_code = 998
self.returncode = int(exit_code)
return buffer
def _communicate(
self,
input: str,
timeout_duration: int | float = 25,
no_output_timeout_duration: int | float = 25,
) -> str:
"""Runs command in container and returns output
Args:
input: command to run in container
timeout_duration: duration to wait for output
no_output_timeout_duration: duration to wait when the process stopped produce any output
"""
assert self.container is not None
communicate_method = keys_config.get(
"SWE_AGENT_COMMUNICATE_METHOD", default="end-marker", choices=["end-marker", "processes"]
)
if communicate_method == "end-marker":
return self._communicate_experimental(input, timeout_duration, no_output_timeout_duration)
try:
self.returncode = None
cmd = input if input.endswith("\n") else input + "\n"
os.write(self.container.stdin.fileno(), cmd.encode()) # type: ignore
time.sleep(0.1)
self.container.stdin.flush() # type: ignore
except BrokenPipeError:
traceback.print_exc()
self.logger.error("Failed to communicate with container. Check docker logs for more information.")
msg = "Failed to communicate with container"
raise RuntimeError(msg)
try:
buffer = read_with_timeout(self.container, self.get_pids, timeout_duration)
self.container.stdin.write("echo $?\n") # type: ignore
time.sleep(0.1)
self.container.stdin.flush() # type: ignore
exit_code = read_with_timeout(self.container, self.get_pids, 5).strip()
except Exception as e:
self.logger.error(f"Read with timeout failed on input:\n---\n{input}\n---")
raise e
if not exit_code.isdigit():
msg = f"Failed to get exit code. Output:\n---\n{buffer}\n---"
raise RuntimeError(msg)
self.returncode = int(exit_code)
return buffer
def _check_syntax(self, input: str) -> tuple[str, bool]:
"""
Check syntax of command.
Returns:
output: Output of the command
success: whether the exit code was 0
"""
output = self._communicate(f"/bin/bash -n <<'EOF'\n{input}\nEOF\n")
return output, self.returncode == 0
def communicate(
self,
input: str,
timeout_duration: int | float = 25,
no_output_timeout_duration: int | float | None = None,
*,
set_last_action: bool = False,
redact_command_trace: bool = False,
) -> str:
"""
Sends input to container and returns output
Args:
input: input to send to container
timeout_duration: duration to wait for output
set_last_action: whether to set the LAST_ACTION environment variable
redact_command_trace: Whether to redact the command that is being run when logging
it to trace level
Returns:
output: output from container
"""
assert self.container is not None
if no_output_timeout_duration is None:
no_output_timeout_duration = timeout_duration
if input.strip() != "exit":
if redact_command_trace:
self.logger.log(logging.TRACE, "Input:\nREDACTED") # type: ignore
else:
self.logger.log(logging.TRACE, "Input:\n%s", input) # type: ignore
output, valid = self._check_syntax(input)
if not valid:
return output # shows syntax errors
output = self._communicate(
input,
timeout_duration=timeout_duration,
no_output_timeout_duration=no_output_timeout_duration,
)
self.logger.log(logging.TRACE, "Output:\n%s", output) # type: ignore
self.communicate_output = output
if set_last_action:
# Cannot merge this with last command, because of multiline command
# handling.
last_action_string = shlex.quote(input.strip())
input = f"export LAST_ACTION={last_action_string}"
self._communicate(input, timeout_duration=5, no_output_timeout_duration=5)
return output
else:
self.container.terminate()
self.returncode = 0
self.communicate_output = ""
return ""
def communicate_with_handling(
self, input: str, error_msg: str, timeout_duration: int | float = 25, *, redact_command_trace: bool = False
) -> str:
"""
Wrapper for communicate function that raises error if return code is non-zero
Args:
input: input to send to container
error_msg: error message to raise if return code is non-zero
timeout_duration: duration to wait for output
redact_command_trace: Whether to redact the command that is being run when logging
it to trace level
Returns:
output: output from container
"""
logs = self.communicate(input, timeout_duration=timeout_duration, redact_command_trace=redact_command_trace)
if self.returncode != 0:
self.logger.error(f"{error_msg}: {logs}")
self.close()
msg = f"{error_msg}: {logs}"
raise RuntimeError(msg)
return logs
def get_available_actions(self) -> list[str]:
"""
Returns list of available actions in current environment state
Currently not in use.
"""
return []
def get_pids(self, all_pids: bool = False) -> list[tuple[str, str]]:
"""
Gets list of processes running inside docker container
Args:
all_pids: whether to return all pids, or whether to exclude ps
and parent PIDs
Returns:
list of PIDs
"""
assert self.container_obj is not None
pids = self.container_obj.exec_run("ps -eo pid,comm,ppid --no-headers").output.decode().split("\n")
pids = [x.split() for x in pids if x]
if not all_pids:
# Get just the PIDs of processes that are descendants of parent_pids and not others
pids = [
(x[0], x[1])
for x in pids
if x[1] != "ps"
and x[0] not in self.parent_pids
and x[1] != getattr(self.interactive_session, "name", None)
and x[2] in self.parent_pids
]
return pids
# ctf
def validate_submission(self, submission: str) -> bool:
"""
Function for validating submission for CTF challenges.
Args:
submission: extracted submission
Returns:
validation of the submission found against known flag
"""
submission = submission.strip()
def wrap(s: str) -> str:
return f"flag{{{s}}}"
if self.challenge is not None:
assert "flag" in self.challenge
solution = self.challenge["flag"]
return any(
(
submission == solution,
wrap(submission) == solution,
submission == wrap(solution),
)
)
return True
def get_submission(self, output: str) -> str | None:
"""
Function for extracting diff patch submission at the end of an episode.
Args:
output: `submit` observation
Returns:
submission: diff patch submission
"""
pattern = r"\<\<SUBMISSION\|\|(.*)\|\|SUBMISSION\>\>"
match = re.search(pattern, output, re.DOTALL)
if match is None:
return None
return match.group(1)
def run_shell_script(self, script_path: Path, *, location: str) -> None:
"""Run custom script supplied by user at `script_path`
Args:
script_path: path to script file
location: location of script file 'host' or 'container'
"""
if location == "host":
return self._run_shell_script_host(script_path)
elif location == "container":
raise NotImplementedError
msg = f"Invalid 'location': {location}"
raise ValueError(msg)
def _run_shell_script_host(self, script_path: Path) -> None:
"""Run shell script file (located on host) in container"""
if not script_path.is_file():
msg = f"Script not found at {script_path}"
raise FileNotFoundError(msg)
shell_commands = Path(script_path).read_text().splitlines(keepends=True)
for i, cmd in enumerate(shell_commands):
self.communicate_with_handling(
cmd,
error_msg=f"Failed to execute line {i}.",
timeout_duration=LONG_TIMEOUT,
)
def _get_install_configs(self) -> dict | None:
"""Return config for environment setup"""
assert self.record is not None # mypy
if (
self.record["problem_statement_source"] != "swe-bench" or self.record["repo_type"] == "local"
) and self.args.environment_setup is None:
self.logger.warning(
"install_environment is set to True, but the data path is a GitHub URL "
"without an environment config file (environment_config key/flag). "
"Skipping conda environment installation.",
)
return None
if self.args.environment_setup is not None:
assert isinstance(self.args.environment_setup, (str, os.PathLike))
if Path(self.args.environment_setup).suffix in [".yml", ".yaml"]:
try:
return yaml.safe_load(Path(self.args.environment_setup).read_text())
except Exception as e:
msg = "Environment config file needs to be a yaml file"
raise ValueError(msg) from e
elif Path(self.args.environment_setup).suffix == ".sh":
return {
"shell_script_path": self.args.environment_setup,
}
else:
msg = "Environment config file needs to be a yaml file or shell script"
raise ValueError(msg)
else:
try:
return MAP_REPO_VERSION_TO_SPECS[self.record["repo"]][str(self.record["version"])]
except KeyError as e:
msg = (
"Tried to look up install configs in swe-bench, but failed. "
"You can set a custom environment config with the environment_config key/flag."
)
raise ValueError(msg) from e
def _conda_environment_exists(self, env_name: str) -> bool:
env_check = self.communicate(f"conda env list | grep {env_name}", timeout_duration=LONG_TIMEOUT)
return env_check.strip() != ""
def install_env(self) -> None:
"""
Creates conda environment and installs third party dependencies to allow code execution
"""
t0 = time.perf_counter()
for hook in self.hooks:
hook.on_install_env_started()
install_configs = self._get_install_configs()
if not install_configs:
return
if "shell_script_path" in install_configs:
assert len(install_configs) == 1
self.run_shell_script(Path(install_configs["shell_script_path"]), location="host")
return
assert self.record is not None # mypy
# Create environment if does not exist yet
env_name = f"{self._repo_name}__{self.record['version']}"
if not self._conda_environment_exists(env_name):
self.logger.info(f"{env_name} conda env not found, creating...")
packages = install_configs.get("packages", "")
if packages == "requirements.txt":
# Create conda environment
self.communicate_with_handling(
f"conda create -n {env_name} python={install_configs['python']} -y",
error_msg="Failed to create conda environment",
timeout_duration=LONG_TIMEOUT,
)
self.logger.debug("Created conda environment")
# Write reqs to requirements.txt in docker container
content_reqs = get_requirements(self.record)
copy_file_to_container(self.container_obj, content_reqs, PATH_TO_REQS)
# Create conda environment + install reqs
self.communicate_with_handling(
f"conda activate {env_name}",
error_msg="Failed to activate conda environment",
)
self.communicate_with_handling(
f"pip install -r {PATH_TO_REQS}",
error_msg="Failed to install requirements.txt",
timeout_duration=LONG_TIMEOUT,
)
self.logger.debug("Installed requirements from requirements.txt")
self.communicate(f"rm {PATH_TO_REQS}")
elif packages == "environment.yml":
# Write environment.yml to file
content_env_yml = get_environment_yml(self.record, env_name)
# Hotfix for
if not install_configs.get("no_use_env"):
content_env_yml += f'\n - python={install_configs["python"]}\n'
copy_file_to_container(self.container_obj, content_env_yml, PATH_TO_ENV_YML)
if install_configs.get("no_use_env"):
# Create conda environment
self.communicate_with_handling(
f"conda create -c conda-forge -n {env_name} python={install_configs['python']} -y",
error_msg="Failed to create conda environment",
timeout_duration=LONG_TIMEOUT,
)
self.logger.debug("Created conda environment")
# Install packages
self.communicate_with_handling(
f"conda env update -f {PATH_TO_ENV_YML}",
error_msg="Failed to install environment.yml",
timeout_duration=LONG_TIMEOUT,
)
self.logger.debug("Installed packages from environment.yml")
else:
# Create environment + install packages
self.communicate_with_handling(
f"conda env create --file {PATH_TO_ENV_YML}",
error_msg="Failed to create conda environment with environment.yml",
timeout_duration=LONG_TIMEOUT,
)
self.logger.debug("Created conda environment with environment.yml")
self.communicate(f"rm {PATH_TO_ENV_YML}")
else:
python_env = f"python{install_configs['python']}"
if self._conda_environment_exists(python_env):
self.communicate_with_handling(
f"conda create --name {env_name} --clone {python_env}",
error_msg="Failed to clone conda environment",
timeout_duration=LONG_TIMEOUT,
)
self.logger.debug("Cloned python conda environment")
else:
self.logger.debug(f"Could not find {python_env}, creating new environment")
self.communicate_with_handling(
f"conda create -n {env_name} python={install_configs['python']} -y",
error_msg="Failed to create conda environment",
timeout_duration=LONG_TIMEOUT,
)
self.communicate_with_handling(
f"conda activate {env_name}",
error_msg="Failed to activate conda environment",
)
if packages.strip():
self.communicate_with_handling(
f"conda install {packages} -y",
error_msg="Failed to install packages",
timeout_duration=LONG_TIMEOUT,
)
self.logger.debug("Installed conda packages")
# Install extra pip packages if specified
if install_configs.get("pip_packages"):
self.communicate_with_handling(
f"source activate {env_name} && pip install {' '.join(install_configs['pip_packages'])}",
error_msg="Failed to install pip packages",
timeout_duration=LONG_TIMEOUT,
)
self.logger.debug("Installed extra pip dependencies")
# Activate environment
self.communicate_with_handling(f"conda activate {env_name}", error_msg="Failed to activate conda environment")
# Install repo at base commit
if install_configs.get("pre_install"):
self.logger.info("Running pre-install commands...")
for pre_install_cmd in install_configs["pre_install"]:
self.communicate_with_handling(
pre_install_cmd,
error_msg="Pre-install commands failed to execute successfully",
timeout_duration=LONG_TIMEOUT,
)
self.logger.debug("Ran pre-install commands")
self.logger.info(f"Installing {self._repo_name} at base commit...")
if install_configs.get("install"):
install_cmd = install_configs["install"]
self.communicate_with_handling(
install_cmd,
error_msg="Install command failed to execute successfully",
timeout_duration=LONG_TIMEOUT,
)
self.logger.debug("Ran install command")
if install_configs.get("post_install"):
self.logger.info("Running post-install commands...")
for post_install_cmd in install_configs["post_install"]:
self.communicate_with_handling(
post_install_cmd,
error_msg="Post-install commands failed to execute successfully",
)
self.logger.debug("Ran post-install commands")
self.logger.info("Installation step took %.2f seconds", time.perf_counter() - t0)
def add_commands(self, commands: list[dict]) -> None:
"""
Adds custom commands to container
"""
for command in commands:
name = command["name"]
contents = command["contents"]
copy_file_to_container(self.container_obj, contents, f"/root/commands/{name}")
if command["type"] == "source_file":
self.communicate_with_handling(
f"source /root/commands/{name}",
error_msg=(
f"Failed to source {name}. If you meant to make a script,"
" start the file with a shebang (e.g. #!/usr/bin/env python)."
),
)
elif command["type"] == "script":
self.communicate_with_handling(
f"chmod +x /root/commands/{name}",
error_msg=f"Failed to chmod {name}",
)
elif command["type"] == "utility":
# nothing to do for utility scripts
pass
else:
msg = f"Invalid command type: {command['type']}"
raise ValueError(msg)
def interrupt(self) -> str:
"""
Send interrupt signal to container and exhaust stdout buffer with a communicate call
"""
assert self.container is not None
assert self.container_obj is not None
pids = self.get_pids()
for pid, _ in pids:
# Sending signal several times ensures that the process is dead
for _ in range(3):
self.container_obj.exec_run(f"kill -9 {pid}")
observation = ""
try:
observation += read_with_timeout(self.container, self.get_pids, 20)
except TimeoutError:
pass
try:
# This is a workaround because of bash behaviour
# when sometimes we get the prints of Killed after we press some "Enter" in stdin
self.communicate(input="echo 'interrupted'", timeout_duration=5)
output = self.communicate(input="echo 'interrupted'", timeout_duration=5)
assert output.strip().endswith("interrupted"), "container health check failed"
except TimeoutError:
msg = "Failed to interrupt container"
raise RuntimeError(msg)
return observation
def open_pr(self, *, trajectory, _dry_run: bool = False) -> None:
"""Create PR to repository
Args:
trajectory: Trajectory of actions taken by the agent
_dry_run: Whether to actually push anything or just simulate it
"""
self.logger.info("Opening PR")
# TODO: have better way of handling this
# Adding random string suffix to avoid name conflicts if we had a previously failed run
issue_url = self.args.data_path
try:
issue = get_gh_issue_data(issue_url, token=self._github_token)
except InvalidGithubURL as e:
msg = "Data path must be a github issue URL if --open_pr is set."
raise ValueError(msg) from e
branch_name = f"swe-agent-fix-#{issue.number}-" + str(random.random())[2:10]
self.communicate_with_handling(
input="rm -f model.patch",
error_msg="Failed to remove model patch",
timeout_duration=10,
)
self.communicate_with_handling(
input=f"git checkout -b {branch_name}",
error_msg="Failed to switch to new branch",
timeout_duration=10,
)
self.communicate_with_handling(
input="git add .",
error_msg="Failed to add commits",
timeout_duration=10,
)
dry_run_flag = "--allow-empty" if _dry_run else ""
commit_msg = [
shlex.quote("Fix: {issue.title}"),
shlex.quote("Closes #{issue.number}"),
]
self.communicate_with_handling(
input=f"git commit -m {commit_msg[0]} -m {commit_msg[1]} {dry_run_flag}",
error_msg="Failed to commit changes",
timeout_duration=10,
)
owner, repo, _ = parse_gh_issue_url(issue_url)
# If `--repo_path` was specified with a different github URL, then the record will contain
# the forking user
assert self.record is not None
if self.record["repo_type"] != "github":
# We already validated that `--data_path` is a github issue URL
# so this is the only case where we can reach here
msg = "--repo_path must point to a github URL if --open_pr is set"
raise ValueError(msg)
forker, _ = self.record["repo"].split("/")
head = branch_name
remote = "origin"
if forker != owner:
head = f"{forker}:{branch_name}"
token_prefix = ""
if self._github_token:
token_prefix = f"{self._github_token}@"
fork_url = f"https://{token_prefix}github.com/{forker}/{repo}.git"
self.logger.debug(f"Using fork: {fork_url}")
self.communicate_with_handling(
input=f"git remote add fork {fork_url}",
error_msg="Failed to create new git remote",
timeout_duration=10,
)
remote = "fork"
dry_run_prefix = "echo " if _dry_run else ""
self.communicate_with_handling(
input=f"{dry_run_prefix} git push {remote} {branch_name}",
error_msg=(
"Failed to push branch to remote. Please check your token and permissions. "
"You might want to push to a fork with the push_gh_repo_url option."
),
timeout_duration=10,
)
body = (
f"This is a PR opened by AI tool [SWE Agent](https://github.com/princeton-nlp/SWE-agent/) "
f"to close [#{issue.number}]({issue_url}) ({issue.title}).\n\nCloses #{issue.number}."
)
body += "\n\n" + format_trajectory_markdown(trajectory)
api = GhApi(token=self._github_token)
if not _dry_run:
pr_info = api.pulls.create( # type: ignore
owner=owner,
repo=repo,
title=f"SWE-agent[bot] PR to fix: {issue.title}",
head=head,
base="main",
body=body,
draft=True,
)
self.logger.info(
f"🎉 PR created as a draft at {pr_info.html_url}. Please review it carefully, push "
"any required changes onto the branch and then click "
"'Ready for Review' to bring it to the attention of the maintainers.",
)
def read_file(self, path: str | PurePath) -> str:
"""Read file contents from container
Args:
path: Path to file relative to repository root
Returns:
file_contents: Contents of file as string
"""
path_in_container = f"/{self._repo_name}/{path}"
return self.communicate(f"cat {str(path_in_container)}")
|