Skip to content

Commit 7f9588f

Browse files
authored
make text reply more clear for guessing numbers for LLMs (#102)
* make text reply more clear for guessing numbers for LLMs * fix test cases --------- Co-authored-by: Allen Wyma <>
1 parent e2eddb9 commit 7f9588f

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

examples/secret-mcp/src/tools.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ export function setupServerTools(server: McpServer, env: Env) {
1616
content: [
1717
{
1818
type: 'text',
19-
text: `You guessed ${guess}`,
19+
text: `You guessed ${guess}: your guess was ${guessedCorrectedly}`,
2020
},
2121
],
2222
correct: guessedCorrectedly,

examples/secret-mcp/test/client.test.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -116,7 +116,7 @@ describe('Secret MCP Client Integration Tests', () => {
116116
expect(result).not.toBeUndefined();
117117
expect(result.content).toHaveLength(1);
118118
expect(result.content[0].type).toBe('text');
119-
expect(result.content[0].text).toBe('You guessed 7');
119+
expect(result.content[0].text).toBe('You guessed 7: your guess was true');
120120
expect(result.correct).toBe(true);
121121

122122
await waitOnExecutionContext(ctx);
@@ -136,7 +136,7 @@ describe('Secret MCP Client Integration Tests', () => {
136136
expect(result).not.toBeUndefined();
137137
expect(result.content).toHaveLength(1);
138138
expect(result.content[0].type).toBe('text');
139-
expect(result.content[0].text).toBe('You guessed 5');
139+
expect(result.content[0].text).toBe('You guessed 5: your guess was false');
140140
expect(result.correct).toBe(false);
141141

142142
await waitOnExecutionContext(ctx);
@@ -163,7 +163,7 @@ describe('Secret MCP Client Integration Tests', () => {
163163
expect(result).not.toBeUndefined();
164164
expect(result.content).toHaveLength(1);
165165
expect(result.content[0].type).toBe('text');
166-
expect(result.content[0].text).toBe(`You guessed ${testCase.guess}`);
166+
expect(result.content[0].text).toBe(`You guessed ${testCase.guess}: your guess was ${testCase.guess === 7}`);
167167
expect(result.correct).toBe(testCase.expected);
168168
}
169169

@@ -190,7 +190,7 @@ describe('Secret MCP Client Integration Tests', () => {
190190
expect(result).not.toBeUndefined();
191191
expect(result.content).toHaveLength(1);
192192
expect(result.content[0].type).toBe('text');
193-
expect(result.content[0].text).toBe(`You guessed ${testCase.guess}`);
193+
expect(result.content[0].text).toBe(`You guessed ${testCase.guess}: your guess was ${testCase.guess === 7}`);
194194
expect(result.correct).toBe(testCase.expected);
195195
}
196196

0 commit comments

Comments
 (0)