Skip to content

fix(voice): require local time when speaking schedules - #412

Merged
LUPENGHAN merged 1 commit into
1024XEngineer:mainfrom
znnnnnnn-wil:codex/fix-voice-local-time-prompt
Aug 28, 2026
Merged

fix(voice): require local time when speaking schedules#412
LUPENGHAN merged 1 commit into
1024XEngineer:mainfrom
znnnnnnn-wil:codex/fix-voice-local-time-prompt

Conversation

@znnnnnnn-wil

Copy link
Copy Markdown
Contributor

变更

  • 强制语音查询日程播报使用 starts_at_local 字段。
  • 明确 start_time 仅用于内部判断和工具参数,避免模型直接播报 UTC 时分。

原因

PR #411 已补充本地时间字段,但系统提示词未禁止模型读取 start_time,在 mode=2 下可能出现相差 8 小时的播报。

验证

  • 基于上游最新 main。
  • 仅修改 backend/src/timeflow/intelligence/conversation/agent.py。
  • 已执行 git diff --check。

@codecov

codecov Bot commented Aug 28, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

Flag Coverage Δ
backend 96.57% <ø> (ø)
frontend 91.05% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
...nd/src/timeflow/intelligence/conversation/agent.py 100.00% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@fennoai fennoai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

审阅了完整变更及其调用链:查询工具已返回按日程时区格式化的 starts_at_local,本次系统提示词改为要求播报使用该字段,同时保留 start_time 供内部判断和后续工具参数使用。该约束与现有序列化契约一致,未发现由本 PR 引入的可操作正确性或回归问题。

验证:git diff --check 和 Python compileall 通过;未能运行聚焦 pytest,因为环境未安装 pytest

@LUPENGHAN
LUPENGHAN merged commit 956a53f into 1024XEngineer:main Aug 28, 2026
5 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