Single tool with moderate naming, basic description, and incomplete schema. The tool name 'get_sales_orders' follows verb_noun convention correctly, but the description lacks context on prerequisites, API limitations, and downstream integration. Input schema has type definitions but parameters lack comprehensive constraint documentation. Output is returned as plain text rather than structured JSON, violating the response-shaper pattern. No error classification or recovery guidance. No support for pagination despite mentioning a 'top' parameter that could exceed reasonable limits.

• Return structured JSON object with typed fields: {"orders": [{"id": "string", "displayId": "string", "soldToParty": "string", "netAmount": "number", "netAmountCurrency": "string", "createdAt": "ISO8601-date"}], "total": "integer", "count": "integer"} instead of plain text.
• Remove 'api_key' parameter entirely. Inject SAP_API_KEY from environment variable only. Document in tool description: 'Requires SAP_API_KEY environment variable to be set before server startup.'
• Add pagination: introduce 'skip' or 'offset' parameter (default 0) and 'limit' parameter (default 20, max 100). Return 'total_count' and 'has_more' in response to enable agent-driven iteration.
• Expand description to 150-250 characters: 'Retrieve sales orders from SAP Graph API sandbox. Returns up to 100 orders with ID, display ID, sold-to party, net amount, and creation date. Requires valid SAP_API_KEY environment variable. Call this to list available orders before retrieving details.'
• Document 'top' parameter: 'Number of sales orders to retrieve (integer, 1-100, default 3). Note: API may enforce stricter limits; exceeding 100 will return error.'
• Add error recovery guidance in exception handlers. Replace 'HTTP error occurred: {http_err}' with 'HTTP {http_err.response.status_code}: {http_err.response.reason}. Likely causes: invalid API key, SAP service unavailable, or rate limit exceeded. Verify SAP_API_KEY and retry after 60 seconds.'
• Add timeout parameter (default 30s) to requests.get() call to prevent agent from hanging on unresponsive API.