Critical failures across all dimensions. All 4 tools lack descriptions entirely (0/4 have non-empty descriptions). Parameter descriptions are null for all inputs. No output schemas documented. Tool names use hyphens instead of verb_noun convention. No error handling, validation, or recovery guidance. The two financial tools (buy-a-stock, sell-a-stock) are IRREVERSIBLE operations with zero safety mechanisms, no confirmation patterns, and no audit trails. This server is not production-ready.

• Add non-empty descriptions to all 4 tools (10 - 1024 chars). Example: 'buy-a-stock' → 'Place a market buy order for a stock on NSE exchange. Requires valid trading symbol and quantity. This is an IRREVERSIBLE operation that executes immediately.'
• Add descriptions to all parameters. Example: stock → 'NSE trading symbol (e.g., HDFCBANK, INFY). Must be a valid listed company.' qty → 'Number of shares to buy (1 - 10000). Must be a positive integer.'
• Implement confirmation pattern for buy-a-stock and sell-a-stock: add a dry_run parameter (default true) that returns 'Order preview: would buy 100 shares of HDFCBANK at market price. Call again with dry_run=false to execute.' This prevents accidental trades.
• Document output schemas. Example: buy-a-stock should return {order_id: string, status: 'pending'|'executed'|'rejected', quantity: number, symbol: string, timestamp: ISO8601}.
• Rename tools to verb_noun convention: add-two-number → add_numbers, factorial-of-a-number → compute_factorial, buy-a-stock → buy_stock, sell-a-stock → sell_stock.