Skip to content

fix(twin): /alert/push 捕获 TwinStationError 转 404,对齐同文件其他端点 - #9

Open
leaderwm wants to merge 1 commit into
zhyese:mainfrom
leaderwm:fix/twin-alert-push-404
Open

fix(twin): /alert/push 捕获 TwinStationError 转 404,对齐同文件其他端点#9
leaderwm wants to merge 1 commit into
zhyese:mainfrom
leaderwm:fix/twin-alert-push-404

Conversation

@leaderwm

@leaderwm leaderwm commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

问题

POST /twin/alert/push 未捕获 TwinStationError:非法/未知 stationId 会直接 500。同文件的 station_layout 等端点均有 except twin_service.TwinStationError → BizError(str(exc), 404) 处理,唯 push 漏了。

改动

对齐兄弟端点写法补 try/except 转 404(一行包裹 + 注释)。

验证

  • python -m pytest tests/ -q --ignore=tests/test_api.py -m "not integration":828 passed(3 个失败为 main 上已存在的 test_batch3_infra openai SDK 环境差异,与本 PR 无关,已在未改动代码上复现确认)
  • python scripts/validate_golden.py 通过

非法/未知 stationId 原直接 500;station_layout 等兄弟端点均有
except TwinStationError → BizError(404) 处理,唯 push 漏了
@leaderwm
leaderwm force-pushed the fix/twin-alert-push-404 branch from 3ab8d37 to 4267076 Compare September 4, 2026 10:30
@leaderwm

leaderwm commented Sep 5, 2026

Copy link
Copy Markdown
Contributor Author

CI 红叉已定位并另有 PR 修复:#12

本 PR 的 CI 失败为 main 上预存问题,与本 PR 改动无关——证据:main(8862c31)与本 PR 分支的本地全量失败集完全一致,均为 test_batch3_infra.py 3 例(openai 3.7.0 在 AsyncOpenAI() 构造期即校验凭据,而单测环境无 key);本 PR 涉及的模块相关测试全部通过。

#12(conftest 注入哑 key,仅测试环境变更)合并后,main 与本 PR 的 CI 会自动转绿(PR CI 跑的是与 main 的合并预览)。

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.

1 participant