All 4 tools have zero input parameters and minimal descriptions (10-50 chars). No input schemas are visible in the source code. Tool names follow verb_noun convention (get_*), but descriptions lack context on WHEN to use each tool, what data is returned, or dependencies. No output schemas documented. No error handling guidance.

• Add explicit JSON Schema input definitions for each tool (even if empty, declare it as type: object with properties: {}). Make schemas visible in mcp_server.py via @tool decorators or tool registration calls.
• Expand descriptions to 100-200 chars. Example: 'get_profile: Fetch the authenticated Wealthsimple user's account details (name, email, account type). Call this first to verify login and retrieve account metadata needed for portfolio queries.'
• Document output schemas for each tool. Example: 'Returns {user_id: string, email: string, account_type: string, created_at: ISO8601}'. This enables LLMs to chain tools and extract required fields.
• Add error handling with recovery hints. Wrap API calls in try-except; return structured errors like '{error: "auth_failed", message: "Wealthsimple session expired. Re-authenticate via login tool.", retryable: true}'.
• Add parameter descriptions if any parameters are added in future. Even empty input should be documented: 'Input: (none required), uses authenticated session from environment.'