Skip to content

Commit e2b9df0

Browse files
bstee615Copilot
andcommitted
Recognize unified strategy eagerness prompts
Treat the optimized PatchBased02 unified strategy as eagerness-aware when a model supplies an eagerness prompt, while preserving non-eagerness behavior for unrelated strategies. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
1 parent 634e10e commit e2b9df0

2 files changed

Lines changed: 10 additions & 0 deletions

File tree

extensions/copilot/src/platform/inlineEdits/common/dataTypes/xtabPromptOptions.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -574,6 +574,7 @@ export function isEagernessPrompt(options: PromptOptions): boolean {
574574
return (options.eagernessPrompt !== undefined && [
575575
PromptingStrategy.PatchBased02,
576576
PromptingStrategy.PatchBased02WithRecentLineNumbers,
577+
PromptingStrategy.PatchBased02Unified,
577578
PromptingStrategy.PatchBased02WithoutRecentLineNumbers,
578579
].includes(options.promptingStrategy)) // eagerness prompt option is only supported for patch-based strategies
579580
|| [PromptingStrategy.XtabAggressiveness,

extensions/copilot/src/platform/inlineEdits/test/common/xtabPromptOptions.spec.ts

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -159,6 +159,15 @@ describe('isEagernessPrompt', () => {
159159
expect(isEagernessPrompt({ ...DEFAULT_OPTIONS, promptingStrategy: PromptingStrategy.PatchBased02, eagernessPrompt: 'aggressionHighLow' })).toBe(true);
160160
expect(isEagernessPrompt({ ...DEFAULT_OPTIONS, promptingStrategy: PromptingStrategy.PatchBased02 })).toBe(false);
161161
});
162+
163+
it('recognizes the optimized PatchBased02 aggression prompt option', () => {
164+
expect(isEagernessPrompt({ ...DEFAULT_OPTIONS, promptingStrategy: PromptingStrategy.PatchBased02Unified, eagernessPrompt: 'aggressionHighLow' })).toBe(true);
165+
expect(isEagernessPrompt({ ...DEFAULT_OPTIONS, promptingStrategy: PromptingStrategy.PatchBased02Unified })).toBe(false);
166+
});
167+
168+
it('does not recognize eagerness for an unrelated strategy', () => {
169+
expect(isEagernessPrompt({ ...DEFAULT_OPTIONS, promptingStrategy: PromptingStrategy.CopilotNesXtab, eagernessPrompt: 'aggressionHighLow' })).toBe(false);
170+
});
162171
});
163172

164173
describe('GlobalBudgetOptions', () => {

0 commit comments

Comments
 (0)