mirror of
https://github.com/usestrix/strix.git
synced 2026-08-28 05:25:00 +00:00
58df71d3db
* let an agent wait on what it already said An agent that answers in plain text is nudged to call a tool, and the only tool that hands control back takes a required message. So it says the same thing twice: once as text the user has already read, once as the argument it had to supply to stop. Seen on a run whose whole instruction was "hi" - a greeting, then the same greeting again through respond_to_user. message is optional now. The nudge arms the tool with the text that was delivered and says not to repeat it, so an agent that has said its piece can park on it with an empty call. Anything it does want to add it passes normally. Parking still cannot leave the user on silence: an empty call is refused unless something was actually said, and the arming is single use - execution clears it as soon as a turn ends any other way. The interactive prompt now also says to answer and stop in one respond_to_user call, which is what avoids the nudge in the first place. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> * drop the worked example from the interactive prompt "the user greeted you, asked something you can answer outright, or you need a decision" was the run I had been reading, written into a rule that holds whatever the reason. The rule is that replying and stopping is one call; listing occasions only invites the model to check whether this is one of them. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> * drop the arming flag; an empty message just waits Passing the delivered text from execution into the tool, and refusing an empty call without it, was machinery guarding against an agent parking having said nothing. That leaves the user looking at "waiting for your reply" with a cursor in front of them - they type. It does not need a mechanism. What is left is the default on message, and the nudge saying the text already landed. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> * only offer waiting on words that were written The nudge told every agent its text had already been delivered, but it fires whenever a turn leaves the agent running, and a turn can end with no tool call and no text at all - _final_output_preview has carried <none> and <empty> branches all along. An agent that said nothing was being invited to wait on an answer the user never received, leaving them at a bare prompt. It now reads the turn: waiting on what was said is offered only when something was, and otherwise the agent is told plainly that the user has read nothing and to send its message. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> * leave the continuation nudge alone Rewording it meant asserting from the outside whether the agent had spoken, and the nudge fires whenever a turn leaves the agent running - text or no text. The agent knows which it did without being told, so the guidance belongs in its prompt, where the condition is its own to read. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> * say it in the nudge, where the agent is reading An agent stranded by the nudge reasons off the nudge. Told only to call respond_to_user, it supplies a message, and since it has just answered in plain text that message is the same answer again. The system prompt saying otherwise sits thousands of tokens earlier and loses. The clause goes on the line the agent acts on: call respond_to_user, with no message if it has already said it. That reads true whatever the turn did, including one that produced no text, because the agent is the one who knows which — nothing here has to work it out from the outside. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com> |
||
|---|---|---|
| .. | ||
| __init__.py | ||
| test_agent_factory_shell.py | ||
| test_agent_factory_tool_arguments.py | ||
| test_agent_tool_registration.py | ||
| test_api_spec.py | ||
| test_api_spec_targets.py | ||
| test_auth_cli.py | ||
| test_cli_target_list.py | ||
| test_codex_auth.py | ||
| test_codex_streaming.py | ||
| test_compaction.py | ||
| test_config_loader.py | ||
| test_context_budget.py | ||
| test_cost_tracking.py | ||
| test_dedupe_model.py | ||
| test_disable_streaming.py | ||
| test_docker_client_delete.py | ||
| test_e2e_budget_lifecycle.py | ||
| test_execution.py | ||
| test_execution_transient_retry.py | ||
| test_fenced_code.py | ||
| test_go_tui_runtime.py | ||
| test_hooks.py | ||
| test_inputs.py | ||
| test_install_script.py | ||
| test_list_reports.py | ||
| test_llm_extra_headers.py | ||
| test_local_sources.py | ||
| test_model_retry.py | ||
| test_models.py | ||
| test_notes.py | ||
| test_optional_deps.py | ||
| test_output_store.py | ||
| test_packaging.py | ||
| test_provider_hints.py | ||
| test_proxy_client.py | ||
| test_report_pdf.py | ||
| test_report_writer.py | ||
| test_reporting_fields.py | ||
| test_reporting_tool.py | ||
| test_respond_to_user.py | ||
| test_runner_rate_limit.py | ||
| test_runner_root_prompt.py | ||
| test_sarif.py | ||
| test_sarif_stride.py | ||
| test_secret_files.py | ||
| test_session_entries.py | ||
| test_skill_dir_extension.py | ||
| test_state_repo_context.py | ||
| test_stream_idle_timeout.py | ||
| test_tool_call_ids.py | ||
| test_tool_call_limits.py | ||
| test_tui_backend_controller.py | ||
| test_tui_backend_server.py | ||
| test_tui_protocol_conformance.py | ||
| test_tui_resume_history.py | ||
| test_unknown_tool_recovery.py | ||
| test_unraisable_filter.py | ||
| test_update_check.py | ||
| test_usage_subscription.py | ||
| test_viewer.py | ||
| test_viewer_auth.py | ||
| test_viewer_runs_gating.py | ||
| test_wait_dedupe.py | ||