fix: default text agent endpoint

[HuangHan96]

No description provided.

[claude]

Claude finished @HuangHan96's task —— View job

---

Code Review Complete ✅

I've completed a comprehensive review of PR #348. Here's my analysis:

---

Summary

This PR fixes a typo in the default text agent endpoint configuration, correcting the domain from aidenai.info to aidenai.io. The change is minimal (1 line) but important for proper API connectivity.

---

Review Findings

✅ Code Quality

• Clean and straightforward: Simple configuration fix
• Proper JSON formatting: File remains valid JSON
• Clear intent: Commit message accurately describes the change

✅ Security

• No security concerns: Only changes the domain suffix
• Endpoint remains HTTPS: Maintains secure communication
• No sensitive data exposure: API key field remains empty (as expected for config template)

✅ Performance

• No performance impact: Configuration change only affects endpoint routing
• No additional overhead: Same API structure maintained

⚠️ Consistency Issue Identified

Finding: The codebase has inconsistent usage of domains:

• API endpoints: Now correctly use aidenai.io (this PR + line 30 in same file)
• User-facing URLs: Still reference aidenai.info in locale files

Location: app/locales/zh-CN/general.json lines 197-198 contain invitation text with https://aidenai.info/

Question: Are these intentionally different domains (e.g., aidenai.info for website/downloads, aidenai.io for API endpoints)? If so, this should be documented. If not, the locale files should also be updated.

📋 Testing

• No tests affected: This is a configuration file change
• Manual testing recommended: Verify the endpoint is reachable and returns expected responses
• Suggestion: Consider adding integration tests for default agent connectivity

📝 Documentation

• Missing PR description: Adding context would help reviewers understand:
  • Why the domain changed (typo vs. infrastructure change)
  • Whether this affects existing deployments
  • If aidenai.info vs aidenai.io serve different purposes

---

Recommendations

1. Add PR description explaining the reason for the domain change
2. Clarify domain strategy: Document whether .info and .io serve different purposes
3. Update locale files if aidenai.info should also become aidenai.io
4. Test the endpoint to ensure https://prod.aidenai.io/llm/openai is operational

---

Approval Status

✅ The code change itself is correct and safe to merge - it fixes the endpoint configuration as intended.

⚠️ However, please address the consistency question about locale files before merging to ensure the codebase uses domains consistently.

---