Feature/change default url

[emrgnt-cmplxty]

Important

Update default API base URL and enhance database, CLI, and SDK functionalities with improved testing and user-specific rate limits.

• Behavior:
  • Change default --base-url in command_group.py and async_client.py to https://api.cloud.sciphi.ai.
  • Update CLI examples in system_router.py, users_router.py, and other routers to reflect new default URL.
• Database:
  • Modify QueryBuilder in base.py to use PostgreSQL-style parameterization.
  • Add limits_overrides field to User model in user.py.
  • Enhance PostgresLimitsHandler in limits.py to support user-specific rate limits.
• Testing:
  • Add integration tests for user limits and filters in test_users.py and test_filters.py.
  • Refactor and add unit tests for conversations and documents in test_conversations.py and test_documents.py.
• Misc:
  • Update NaClCryptoProvider in nacl.py to improve API key generation.
  • Refactor R2RAsyncClient in async_client.py to handle binary responses.

^{This description was created by for f36795d. It will automatically update as commits are pushed.}

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to f36795d in 1 minute and 30 seconds

More details

• Looked at 4715 lines of code in 39 files
• Skipped 0 files when reviewing.
• Skipped posting 4 drafted comments based on config settings.

1. py/core/database/base.py:56

• Draft comment:
  Ensure that all parts of the code interacting with self.params are updated to handle the list format correctly. The order of parameters in the list must match the order expected by the SQL query.
• Reason this comment was not posted:
  Comment did not seem useful.

2. py/shared/abstractions/user.py:57

• Draft comment:
  Ensure limits_overrides is correctly handled in all parts of the code where user data is processed, including serialization, deserialization, and database interactions.
• Reason this comment was not posted:
  Comment did not seem useful.

3. py/tests/unit/test_limits.py:214

• Draft comment:
  Remove or replace print statements with assertions to keep the test clean and focused on validation.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The test for test_user_level_override is well-structured, but it includes print statements for debugging. These should be removed or replaced with assertions to ensure the test remains clean and focused on validation rather than logging.

4. py/tests/unit/test_limits.py:187

• Draft comment:
  Consider using dynamically generated UUIDs and email addresses to ensure test isolation and avoid potential conflicts.
• Reason this comment was not posted:
  Confidence changes required: 40%
  The test_user_level_override function is testing user-specific rate limit overrides. The test is comprehensive, but it includes hardcoded UUIDs and email addresses, which could be replaced with dynamically generated values to ensure test isolation and avoid potential conflicts.

Workflow ID: wflow_RD7OS2EFBPqb7aqD

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.