Fix Issue #500: "agents-api: create-or-update session throws 'Internal Server Error'"

[HamadaSalhab]

🚀	This description was created by Ellipsis for commit 67bfc1e

fix: resolve 'Internal Server Error' in create_or_update_session

Summary:

Fixes 'Internal Server Error' in create_or_update_session by removing agent assertion, renaming function, and correcting response attribute.

Key points:

• Behavior:
  • Removes assertion assert len(agents) > 0 in create_or_update_session.py to allow zero agents.
  • Corrects response attribute from created_at to updated_at in create_or_update_session() in create_or_update_session.py.
• Renames:
  • Renames create_session_query to create_or_update_session_query in create_or_update_session.py and create_or_update_session.py.

---

Generated with ❤️ by ellipsis.dev

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to 67bfc1e in 10 seconds

More details

• Looked at 42 lines of code in 2 files
• Skipped 0 files when reviewing.
• Skipped posting 1 drafted comments based on config settings.

1. agents-api/agents_api/routers/sessions/create_or_update_session.py:33

• Draft comment:
  The change from created_at to updated_at might be incorrect. Typically, a ResourceCreatedResponse should include a created_at timestamp. Verify if this change aligns with the expected API behavior.
• Reason this comment was not posted:
  Decided after close inspection that this draft comment was likely wrong and/or not actionable:
  The comment is speculative as it asks to verify if the change aligns with the expected API behavior. According to the rules, speculative comments should be removed. The comment does not provide a clear action or refactor suggestion, and it asks for verification, which is not allowed.
  The comment might be pointing out a potential issue with the logic of the code, which could be important to address. However, it does not provide a definitive statement that there is an issue, only a suggestion to verify.
  While the comment might be pointing out a potential issue, it does not align with the rules as it is speculative and asks for verification. The rules clearly state to avoid such comments.
  The comment should be removed as it is speculative and asks for verification, which is against the rules.

Workflow ID: wflow_Yr0JIINtL5OhHQBh

---

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

[creatorrr]

Closing in favor of #503