jpmschweitzer/odysseus
1
0
Fork 0
mirror of https://github.com/pewdiepie-archdaemon/odysseus.git synced 2026-10-18 12:52:20 +02:00
Code Issues Packages Projects Releases Wiki Activity

match completion writes to requested artifact paths

Browse Source
This commit is contained in:
pewdiepie-archdaemon
2026-09-17 23:29:44 +00:00
parent da7e8a9c2f
commit 1c533f9f32
2 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/clean_agent_preview.py
-2
View File
@@ -1374,8 +1374,6 @@ def runtime_required_artifacts(user_text, client_runtime_context):
def execution_targets_required_artifact(tool_name, arguments, required_artifacts): def execution_targets_required_artifact(tool_name, arguments, required_artifacts):
"""Return true only when a successful mutation names a required output.""" """Return true only when a successful mutation names a required output."""
if canonical(tool_name) == 'write_file':
return True
serialized = ( serialized = (
arguments if isinstance(arguments, str) arguments if isinstance(arguments, str)
else json.dumps(arguments or {}, ensure_ascii=False) else json.dumps(arguments or {}, ensure_ascii=False)
tests/test_clean_agent_preview.py
+3
View File
@@ -466,6 +466,9 @@ def test_read_only_python_does_not_satisfy_required_artifact():
assert execution_targets_required_artifact( assert execution_targets_required_artifact(
'write_file', {'path': '/workspace/output.html', 'content': '<html />'}, required, 'write_file', {'path': '/workspace/output.html', 'content': '<html />'}, required,
) )
assert not execution_targets_required_artifact(
'write_file', {'path': '/workspace/analyze.py', 'content': 'print(1)'}, required,
)
def test_read_only_gate_blocks_mutation_and_network_shell(): def test_read_only_gate_blocks_mutation_and_network_shell():