Skip to content

流式改成默认开,TALOS_STREAM=0 关 - #5

Merged
Jerry-TZ merged 1 commit into
mainfrom
claude/dreamy-faraday-rn209o
Sep 24, 2026
Merged

Jerry-TZ merged 1 commit into
mainfrom
claude/dreamy-faraday-rn209o

Conversation

@Jerry-TZ

Copy link
Copy Markdown
Owner

改动

  • 没设 TALOS_STREAM 就流式;设成 0 / false / no / off(不分大小写、容空格)回到整份返回那条老路
  • README 那段、agent.py 的注释、conftest 的注释跟着改:哪家跑出毛病,先设 TALOS_STREAM=0,再来看是哪一块没拼对

判据

test_streaming_is_on_unless_you_turn_it_off 真的起一个进程查默认值:开关在 import 时读环境变量,而 conftest 为了让几百条用例的假客户端照旧回整份,把每条测试里的 STREAM 都钉成了关 —— 在测试进程里看它永远是关的。cwd 放在空目录,别让本机的 .env 替判据做决定。旧代码下当场红在「没设时没开」。

端到端

走了一遍真的 once()(也就是 agent.py -p 那条路):本地起一个 OpenAI 兼容的假服务吐流式块,在伪终端里跑,TALOS_STREAM 不设:

⠇ 模型在写 write_file(hello.py) · 800 字
⠏ 模型在写 write_file(hello.py) · 2,400 字
  …
⠧ 模型在写 write_file(hello.py) · 14,435 字
  ⚙ write_file({'path': 'hello.py', …})
wrote 14000 chars to hello.py
⠋ 模型在写回复 · 21 字
╭─ 🤖 Talos ─────────────────────╮
│ 写好了,hello.py 已经在工作区里。 │
╰────────────────────────────────╯
  🎫 2 次调用 · 2000+100=2100 tok · 缓存命中 1280

两次请求都带了 stream: true + include_usage。还是没对哪一家真调用过。

验证

  • 345 条判据:343 过、5 skip、1 xfail(原有);TALOS_STREAM=1 下同样绿
  • selfcheck 绿;py3.10 编译通过

🤖 Generated with Claude Code

https://claude.ai/code/session_014gdkHx6rLSMVsUmQiErVSn


Generated by Claude Code

没设 TALOS_STREAM 就流;设成 0 / false / no / off(不分大小写、容空格)回到整份返回那条老路。
README 那段和 agent.py 的注释跟着改:哪家跑出毛病,先关掉它,再来看是哪一块没拼对。

判据要真的起一个进程:开关在 import 时读环境变量,而 conftest 为了让几百条用例的假客户端
照旧回整份,把每条测试里的 STREAM 都钉成了关 —— 在测试进程里看它永远是关的。
cwd 放在空目录,别让本机的 .env 替判据做决定。旧代码下当场红在「没设时没开」。

端到端走了一遍真的 once()(也就是 agent.py -p 那条路),本地起一个 OpenAI 兼容的假服务
吐流式块,在伪终端里跑:转圈从「模型在写 write_file(hello.py) · 800 字」涨到 14,435 字,
文件 14,000 字符原样落地,两次请求都带了 stream + include_usage,token 和缓存命中照常入账。
还是没对哪一家真调用过。

345 判据:343 过、5 skip、1 xfail(原有);TALOS_STREAM=1 下同样绿。selfcheck 绿。

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_014gdkHx6rLSMVsUmQiErVSn
@Jerry-TZ
Jerry-TZ merged commit 9ab573d into main Sep 24, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants