Skip to content

Bumped to ChatGPT 5.6#269

Merged
sfreeman422 merged 1 commit into
masterfrom
feature/bump-to-5.6
Jul 10, 2026
Merged

Bumped to ChatGPT 5.6#269
sfreeman422 merged 1 commit into
masterfrom
feature/bump-to-5.6

Conversation

@sfreeman422

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings July 10, 2026 12:48
@sfreeman422 sfreeman422 merged commit 7987ab9 into master Jul 10, 2026
11 checks passed

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.

Pull request overview

This PR updates the backend’s OpenAI Responses API usage to target a newer GPT model and explicitly request a medium level of model “reasoning effort” for several key response-generation flows.

Changes:

  • Bump GPT_MODEL from gpt-5.4-mini-2026-03-17 to gpt-5.6-terra.
  • Add reasoning: { effort: 'medium' } to multiple openAi.responses.create(...) calls in AIService (general text + history-based + participation prompts).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
packages/backend/src/ai/ai.service.ts Adds a reasoning option to several OpenAI Responses API calls used by Slack-facing AI features.
packages/backend/src/ai/ai.constants.ts Updates the default GPT model identifier used by the backend.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 200 to 205
tool_choice: 'auto',
instructions: GENERAL_TEXT_INSTRUCTIONS,
input: text,
user: `${userId}-DaBros2016`,
reasoning: { effort: 'medium' },
})
Comment on lines 201 to 205
instructions: GENERAL_TEXT_INSTRUCTIONS,
input: text,
user: `${userId}-DaBros2016`,
reasoning: { effort: 'medium' },
})
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.

2 participants