Skip to content

Commit 549aac2

Browse files
committed
todo notes
1 parent 9233be0 commit 549aac2

1 file changed

Lines changed: 4 additions & 1 deletion

File tree

lib/messages/inject.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -144,6 +144,10 @@ export const insertPruneToolContext = (
144144
const modelID = userInfo.model.modelID
145145
const isGitHubCopilot =
146146
providerID === "github-copilot" || providerID === "github-copilot-enterprise"
147+
148+
// TODO: This can probably be improved further to only trigger for the appropriate thinking settings
149+
// This setting is also potentially only necessary for claude subscription, API seems to not need this
150+
// validation. See more here: https://platform.claude.com/docs/en/build-with-claude/extended-thinking
147151
const isAnthropic = modelID.includes("claude")
148152

149153
if (isGitHubCopilot) {
@@ -154,7 +158,6 @@ export const insertPruneToolContext = (
154158
}
155159

156160
// Anthropic extended thinking models require a thinking block at the start of its turn
157-
// This can probably be improved further to only trigger for the appropriate thinking settings
158161
if (isAnthropic) {
159162
if (!hasReasoningInCurrentAssistantTurn(messages)) {
160163
return

0 commit comments

Comments
 (0)