Skip to content

Commit e756e47

Browse files
committed
PR
1 parent df6478c commit e756e47

File tree

3 files changed

+10
-11
lines changed

3 files changed

+10
-11
lines changed

.github/workflows/chat-perf.yml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -86,6 +86,10 @@ jobs:
8686
- name: Transpile source
8787
run: npm run transpile-client
8888

89+
- name: Build copilot extension
90+
run: npm run compile
91+
working-directory: extensions/copilot
92+
8993
- name: Download Electron
9094
run: node build/lib/preLaunch.ts
9195

scripts/chat-simulation/common/perf-scenarios.js

Lines changed: 6 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -316,21 +316,19 @@ const TOOL_CALL_SCENARIOS = {
316316
kind: 'tool-calls',
317317
toolCalls: [
318318
{
319-
toolNamePattern: /replace.?string|apply.?patch|insert.?edit/i,
319+
toolNamePattern: /insert.?edit|replace.?string|apply.?patch/i,
320320
arguments: {
321321
filePath: path.join(FIXTURES_DIR, 'lifecycle.ts'),
322-
oldString: '// perf-benchmark-marker',
323-
newString: '// perf-benchmark-marker (updated)',
324322
explanation: 'Update the benchmark marker comment in lifecycle.ts',
323+
code: '// perf-benchmark-marker (updated)',
325324
},
326325
},
327326
{
328-
toolNamePattern: /replace.?string|apply.?patch|insert.?edit/i,
327+
toolNamePattern: /insert.?edit|replace.?string|apply.?patch/i,
329328
arguments: {
330329
filePath: path.join(FIXTURES_DIR, 'event.ts'),
331-
oldString: '// perf-benchmark-marker',
332-
newString: '// perf-benchmark-marker (updated)',
333330
explanation: 'Update the benchmark marker comment in event.ts',
331+
code: '// perf-benchmark-marker (updated)',
334332
},
335333
},
336334
],
@@ -411,12 +409,11 @@ const TOOL_CALL_SCENARIOS = {
411409
kind: 'tool-calls',
412410
toolCalls: [
413411
{
414-
toolNamePattern: /replace.?string|apply.?patch|insert.?edit/i,
412+
toolNamePattern: /insert.?edit|replace.?string|apply.?patch/i,
415413
arguments: {
416414
filePath: path.join(FIXTURES_DIR, 'lifecycle.ts'),
417-
oldString: '// perf-benchmark-marker',
418-
newString: '// perf-benchmark-marker (fixed)',
419415
explanation: 'Fix the dispose call in the test',
416+
code: '// perf-benchmark-marker (fixed)',
420417
},
421418
},
422419
],

scripts/chat-simulation/common/utils.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -235,8 +235,6 @@ function writeSettings(userDataDir, mockServer) {
235235
'github.copilot.advanced.debug.overrideProxyUrl': mockServer.url,
236236
'github.copilot.advanced.debug.overrideCapiUrl': mockServer.url,
237237
'chat.allowAnonymousAccess': true,
238-
// Start new chat sessions in agent mode so tools are available.
239-
'chat.newSession.defaultMode': 'agent',
240238
// Disable MCP servers — they start async and add unpredictable
241239
// delay that pollutes perf measurements.
242240
'chat.mcp.discovery.enabled': false,

0 commit comments

Comments
 (0)