Skip to content

Commit 3c5e2a3

Browse files
DavertMikclaude
andcommitted
refactor(mcp): rename raceRunOutcome → waitForTestResult, outcome → status
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 736bbbf commit 3c5e2a3

1 file changed

Lines changed: 11 additions & 11 deletions

File tree

bin/mcp-server.js

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -412,18 +412,18 @@ async function cancelRun() {
412412
return true
413413
}
414414

415-
async function raceRunOutcome(runPromise, timeout) {
415+
async function waitForTestResult(runPromise, timeout) {
416416
const pausedPromise = new Promise(resolve => pauseEvents.once('paused', () => resolve('paused')))
417417
const completedPromise = runPromise.then(() => 'completed', () => 'completed')
418418
let timeoutId
419419
const timeoutPromise = new Promise((_, reject) => {
420420
timeoutId = setTimeout(() => reject(new Error(`Timeout after ${timeout}ms`)), timeout)
421421
})
422422
try {
423-
return { outcome: await Promise.race([completedPromise, pausedPromise, timeoutPromise]) }
423+
return { status: await Promise.race([completedPromise, pausedPromise, timeoutPromise]) }
424424
} catch (err) {
425425
await cancelRun()
426-
return { outcome: 'aborted', error: err.message }
426+
return { status: 'aborted', error: err.message }
427427
} finally {
428428
clearTimeout(timeoutId)
429429
}
@@ -1044,13 +1044,13 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
10441044
})()
10451045
pendingRunPromise = runPromise
10461046

1047-
const { outcome, error: abortError } = await raceRunOutcome(runPromise, timeout)
1048-
if (outcome === 'aborted') {
1047+
const result = await waitForTestResult(runPromise, timeout)
1048+
if (result.status === 'aborted') {
10491049
await startShellSession()
1050-
return { content: [{ type: 'text', text: JSON.stringify({ status: 'failed', file: testFile, error: abortError }, null, 2) }] }
1050+
return { content: [{ type: 'text', text: JSON.stringify({ status: 'failed', file: testFile, error: result.error }, null, 2) }] }
10511051
}
10521052

1053-
if (outcome === 'paused') {
1053+
if (result.status === 'paused') {
10541054
const page = await gatherPageBrief()
10551055
return {
10561056
content: [{
@@ -1138,13 +1138,13 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
11381138
})()
11391139
pendingRunPromise = runPromise
11401140

1141-
const { outcome, error: abortError } = await raceRunOutcome(runPromise, timeout)
1142-
if (outcome === 'aborted') {
1141+
const result = await waitForTestResult(runPromise, timeout)
1142+
if (result.status === 'aborted') {
11431143
await startShellSession()
1144-
return { content: [{ type: 'text', text: JSON.stringify({ status: 'failed', file: testFile, error: abortError }, null, 2) }] }
1144+
return { content: [{ type: 'text', text: JSON.stringify({ status: 'failed', file: testFile, error: result.error }, null, 2) }] }
11451145
}
11461146

1147-
if (outcome === 'paused') {
1147+
if (result.status === 'paused') {
11481148
const page = await gatherPageBrief()
11491149
return {
11501150
content: [{

0 commit comments

Comments
 (0)