Remove market review from daily push notification - #1246
Conversation
- Add SearXNG as a compose service with JSON API enabled - Split single analyzer into analyzer-pre (21:00) and analyzer-post (05:00) for US market pre-market and post-market analysis - Remove deprecated `version` field from compose file - Add health check and dependency chain (searxng -> analyzers/server)
- Add SearXNG as a compose service with JSON API enabled - Split single analyzer into analyzer-pre (21:00) and analyzer-post (05:00) for US market pre-market and post-market analysis - Remove deprecated `version` field from compose file - Add health check and dependency chain (searxng -> analyzers/server)
feat(docker): add SearXNG and split pre/post market analyzers
Keep only the decision dashboard (决策仪表盘) in daily push and Feishu doc, removing market review (大盘复盘) section. The market review generation logic is preserved but no longer included in the merged notification or Feishu document output.
github-actions
Bot
added
ai
documentation
🤖 自动审查报告
📁 修改的文件
|
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 5f03652815
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| # Issue #190: 合并推送（个股+大盘复盘） | ||
| if merge_notification and (results or market_report) and not args.no_notify: | ||
| # Issue #190: 合并推送（仅决策仪表盘，不含大盘复盘） | ||
| if merge_notification and results and not args.no_notify: |
There was a problem hiding this comment.
In merge mode (MERGE_EMAIL_NOTIFICATION=true), run_market_review intentionally skips its standalone push, expecting main.py to send a combined message later. This change now gates the combined send on results only, so executions that generate a valid market_report but no successful stock results (for example, empty STOCK_LIST or all stock analyses failing) produce no notification at all. Before this commit, the merged path still sent when market_report existed, so this is a regression in delivery behavior.
Useful? React with 👍 / 👎.
1 participant
Summary
reports/market_review_*.md，但完全不推送通知run_market_review的send_notification设为False，确保无论合并/非合并模式都不推送Test plan
已跳过推送通知）--force-run验证通过