Skip to content

fix(labs): cron alerts in Russian (operator chats) - #125

Merged
mashkovd merged 1 commit into
mainfrom
fix/labs-watch-scan-russian-alerts
May 6, 2026
Merged

mashkovd merged 1 commit into
mainfrom
fix/labs-watch-scan-russian-alerts

Conversation

@mashkovd

@mashkovd mashkovd commented May 6, 2026

Copy link
Copy Markdown
Member

Summary

Operator Telegram chats are Russian-language. The cron alerts and onExit error notifications were English-only, which was unintuitive at a glance. Translate both message templates.

Preview

Alert (main scan step):

⚠️ BTC лонг — сработал watch (cron pre-filter)

Скор: 75/100 (порог 67, направление лонг)
Цена: $XXXXX.XX (Δ за 24ч: +X.XX%)
Funding 4h: -0.0094%
OI растёт: да

Запустите /eth btc или /btc для полного анализа в чат-скилле.
Это наблюдение, не финансовая рекомендация.

onExit error:

🚨 labs-watch-scan упал: workflow=... status=Failed. Проверьте workflows.mctl.ai/...

Trader slang stays in its widely used Russian form (long → лонг, short → шорт, funding, OI). Score line and emojis stay structurally identical so older alert archives still read the same shape.

Test plan

  • python3 -m py_compile on embedded Python passes
  • After ArgoCD sync (~3 min) trigger a manual cron run with current watchlist
  • Verify both operator chats receive the Russian alert
  • Verify scoring + dedup behavior unchanged (this PR only touches text)

🤖 Generated with Claude Code

Operator chats are Russian-language; English-only Telegram alerts were
unintuitive at a glance. Translate the main scan-step alert template and
the onExit error notification. Trader slang (long→лонг, short→шорт,
funding, OI) is kept in its widely used Russian form. Score line and
emojis stay structurally identical so older alert archives still read
the same shape.
@mashkovd

mashkovd commented May 6, 2026

Copy link
Copy Markdown
Member Author

@codex review

@mashkovd

mashkovd commented May 6, 2026

Copy link
Copy Markdown
Member Author

@copilot review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. 🚀

ℹ️ About Codex in GitHub

Your team has set up Codex to 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 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@mashkovd
mashkovd merged commit d40ce5f into main May 6, 2026
@mashkovd
mashkovd deleted the fix/labs-watch-scan-russian-alerts branch May 6, 2026 08:02
mashkovd pushed a commit that referenced this pull request May 22, 2026
#91 Layer-2 merged via PR #132 (PR #124 closed unmerged; Layer 1 deferred
to issue #126). #94 (M4, scoped to Slices 1-2) merged via PR #125 with the
3 P3 nits cleared in PR #134.
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