Add emojis to all Telegram bot messages and menus

[Adialia1]

Summary

• Added contextual emojis to all menu buttons (sizing, risk, filters, execution, time exits, keywords, conflict strategy, dry run toggle)
• Enhanced detail lines in position views, trader lists, wallet lists, stats, and trade notifications with relevant emojis
• Added emojis to help text sections, command lists, tips, and how-it-works bullet points
• Updated notifier messages (bot started/stopped, trade executed/failed, risk triggers, daily summary, redemptions) with emojis on every detail line

Test plan

• Verify all Telegram menu buttons render correctly with emojis
• Check that notification messages (trade executed, trade failed, risk triggers) display properly
• Test the help command output formatting
• Verify wallet connection flow messages show emojis correctly
• Test follow trader flow and confirm emojis appear on detail lines

https://claude.ai/code/session_01WCuqjynVrtHj3DbKWXmhBz

In general, unused imports should be removed to keep the codebase clean and avoid confusion. If the module’s side effects are needed but the binding is not, the import should be converted to a side-effect-only import (import './riskManager.js';).

For this specific case, the best minimal fix that does not change existing functionality is to delete the unused RiskManager named import from src/services/userBotManager.ts. Since the rest of the file (in the provided snippet) does not reference RiskManager, and other imports already provide the needed types and classes, removing this one line will not alter runtime behavior or type-checking, it will only remove the dead import. No new methods, fields, or definitions are necessary, and no other lines in this file need adjusting.

In general, unused imports should be removed to keep the codebase clean and avoid misleading future readers about dependencies. Since ClobApiClient is not referenced anywhere in src/services/userBotManager.ts, the safest and most direct fix that preserves behavior is to delete the unused import line.

Concretely, in src/services/userBotManager.ts, remove line 6:

import { ClobApiClient } from '../api/clobApi.js';

No other changes are needed: we do not need to add any new imports, change the constructor, or modify any methods, because nothing in this file relies on ClobApiClient at present.

In general, unused imports should be removed from the file where they are declared. This improves readability, avoids misleading future maintainers into thinking a type or value is relevant there, and can slightly improve tooling performance.

For this specific case in src/services/userBotManager.ts, the best fix is to keep the import of actually used types (TradeSignal, WalletConfig) and remove only CopyConfig from the destructuring import. No other code changes are necessary, and there is no functional impact because removing an unused import does not affect runtime behavior.

Concretely:

• Edit src/services/userBotManager.ts.
• Locate the line:

  import { TradeSignal, WalletConfig, CopyConfig } from '../types/index.js';

• Replace it with:

  import { TradeSignal, WalletConfig } from '../types/index.js';

No additional methods, imports, or definitions are needed.

To fix an unused import, remove it from the import list so that only actually used symbols are imported. This preserves functionality while improving readability and avoiding linter/static-analysis noise.

Concretely, in src/telegram/handlers/positions.ts, on the import line from ../menus.js, remove backButton from the destructuring import while keeping positionsMenu and confirmMenu unchanged. No additional imports, methods, or definitions are required; we are only narrowing the existing import.

To fix the problem, remove the unused import of backButton from ../menus.js. This eliminates dead code, avoids confusion, and has no effect on runtime behavior since backButton is never referenced.

Concretely, in src/telegram/handlers/stats.ts, delete line 3 that imports backButton. No other code changes are needed, and no additional methods, imports, or definitions are required.

To fix the problem, remove the unused backButton named import from the import statement so only the actually used tradingMenu symbol is imported. This eliminates the unused import without changing any existing functionality.

Concretely, in src/telegram/handlers/trading.ts, on the import line currently reading import { tradingMenu, backButton } from '../menus.js';, remove backButton and its trailing comma/space. The resulting line should be import { tradingMenu } from '../menus.js';. No other code changes are needed, and no additional imports or definitions are required.