Skip to content

fix: rename POST /threads/{thread_id}/messages operationId to createT… - #167

Merged
swapnilroy-cell merged 2 commits into
masterfrom
fix/duplicate-create-message-operation-id
Oct 30, 2026
Merged

swapnilroy-cell merged 2 commits into
masterfrom
fix/duplicate-create-message-operation-id

Conversation

@swapnilroy-cell

Copy link
Copy Markdown

…hreadMessage

createMessage was used by both POST /messages and
POST /threads/{thread_id}/messages. operationId must be unique across the document, so the thread-scoped operation is renamed.

…hreadMessage

createMessage was used by both POST /messages and
POST /threads/{thread_id}/messages. operationId must be unique across
the document, so the thread-scoped operation is renamed.

Co-authored-by: Cursor <cursoragent@cursor.com>
Copilot AI balanced review requested due to automatic review settings October 30, 2026 10:25

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

The Assistants messages navigation still points to createMessage, causing the generated reference to show the wrong endpoint.

1 open issue
What changed in this PR

This PR resolves an operation ID collision in the OpenAPI specification between message creation and thread-scoped message creation.

Changes:

  • Renames the operation ID for POST /threads/{thread_id}/messages from createMessage to createThreadMessage, distinguishing it from POST /messages.
File Description
openapi.yaml Gives thread-message creation a distinct operation ID.

Review effort: Balanced

Comment thread openapi.yaml Resolved
The nav key createMessage now resolves to POST /messages after the
operationId rename, so the Assistants reference showed the wrong
endpoint. Update the key; path is unchanged.

Co-authored-by: Cursor <cursoragent@cursor.com>
Copilot AI balanced review requested due to automatic review settings October 30, 2026 10:30

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

No active review issues remain for the thread-message operation ID rename.

No open issues 🎉

1 resolved since last review

Review effort: Balanced

@swapnilroy-cell
swapnilroy-cell merged commit 4b6d01b into master Oct 30, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants