Skip to content

Commit bce2dbf

Browse files
committed
small fixes like argsSchema
1 parent d1c94a2 commit bce2dbf

File tree

4 files changed

+51
-55
lines changed

4 files changed

+51
-55
lines changed

README.md

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -55,8 +55,7 @@ import { z } from "zod";
5555
// Create an MCP server
5656
const server = new McpServer({
5757
name: "demo-server",
58-
version: "1.0.0",
59-
title: "Demo Server" // Optional display name
58+
version: "1.0.0"
6059
});
6160

6261
// Add an addition tool
@@ -109,9 +108,8 @@ The McpServer is your core interface to the MCP protocol. It handles connection
109108

110109
```typescript
111110
const server = new McpServer({
112-
name: "my-app", // Unique identifier for your server
113-
version: "1.0.0", // Server version
114-
title: "My Application" // Optional display name for UI
111+
name: "my-app",
112+
version: "1.0.0"
115113
});
116114
```
117115

@@ -471,8 +469,7 @@ import { z } from "zod";
471469

472470
const server = new McpServer({
473471
name: "echo-server",
474-
version: "1.0.0",
475-
title: "Echo Server"
472+
version: "1.0.0"
476473
});
477474

478475
server.registerResource(
@@ -507,7 +504,7 @@ server.registerPrompt(
507504
{
508505
title: "Echo Prompt",
509506
description: "Creates a prompt to process a message",
510-
arguments: { message: z.string() }
507+
argsSchema: { message: z.string() }
511508
},
512509
({ message }) => ({
513510
messages: [{

src/examples/server/simpleStreamableHttp.ts

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,7 @@ const useOAuth = process.argv.includes('--oauth');
1717
const getServer = () => {
1818
const server = new McpServer({
1919
name: 'simple-streamable-http-server',
20-
version: '1.0.0',
21-
title: 'Simple Streamable HTTP Server', // Display name for UI
20+
version: '1.0.0'
2221
}, { capabilities: { logging: {} } });
2322

2423
// Register a simple tool that returns a greeting
@@ -94,7 +93,7 @@ const getServer = () => {
9493
{
9594
title: 'Greeting Template', // Display name for UI
9695
description: 'A simple greeting prompt template',
97-
arguments: {
96+
argsSchema: {
9897
name: z.string().describe('Name to include in greeting'),
9998
},
10099
},
@@ -158,10 +157,10 @@ const getServer = () => {
158157
server.registerResource(
159158
'greeting-resource',
160159
'https://example.com/greetings/default',
161-
{
160+
{
162161
title: 'Default Greeting', // Display name for UI
163162
description: 'A simple greeting resource',
164-
mimeType: 'text/plain'
163+
mimeType: 'text/plain'
165164
},
166165
async (): Promise<ReadResourceResult> => {
167166
return {

src/server/mcp.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1011,15 +1011,15 @@ export class McpServer {
10111011
config: {
10121012
title?: string;
10131013
description?: string;
1014-
arguments?: Args;
1014+
argsSchema?: Args;
10151015
},
10161016
cb: PromptCallback<Args>
10171017
): RegisteredPrompt {
10181018
if (this._registeredPrompts[name]) {
10191019
throw new Error(`Prompt ${name} is already registered`);
10201020
}
10211021

1022-
const { title, description, arguments: argsSchema } = config;
1022+
const { title, description, argsSchema } = config;
10231023

10241024
const registeredPrompt = this._createRegisteredPrompt(
10251025
name,

src/server/title.test.ts

Lines changed: 40 additions & 40 deletions
Original file line numberDiff line numberDiff line change
@@ -7,12 +7,12 @@ import { McpServer } from "./mcp.js";
77
describe("Title field backwards compatibility", () => {
88
it("should work with tools that have title", async () => {
99
const [clientTransport, serverTransport] = InMemoryTransport.createLinkedPair();
10-
10+
1111
const server = new McpServer(
1212
{ name: "test-server", version: "1.0.0" },
1313
{ capabilities: {} }
1414
);
15-
15+
1616
// Register tool with title
1717
server.registerTool(
1818
"test-tool",
@@ -25,12 +25,12 @@ describe("Title field backwards compatibility", () => {
2525
},
2626
async () => ({ content: [{ type: "text", text: "result" }] })
2727
);
28-
28+
2929
const client = new Client({ name: "test-client", version: "1.0.0" });
30-
30+
3131
await server.server.connect(serverTransport);
3232
await client.connect(clientTransport);
33-
33+
3434
const tools = await client.listTools();
3535
expect(tools.tools).toHaveLength(1);
3636
expect(tools.tools[0].name).toBe("test-tool");
@@ -40,25 +40,25 @@ describe("Title field backwards compatibility", () => {
4040

4141
it("should work with tools without title", async () => {
4242
const [clientTransport, serverTransport] = InMemoryTransport.createLinkedPair();
43-
43+
4444
const server = new McpServer(
4545
{ name: "test-server", version: "1.0.0" },
4646
{ capabilities: {} }
4747
);
48-
48+
4949
// Register tool without title
5050
server.tool(
5151
"test-tool",
5252
"A test tool",
5353
{ value: z.string() },
5454
async () => ({ content: [{ type: "text", text: "result" }] })
5555
);
56-
56+
5757
const client = new Client({ name: "test-client", version: "1.0.0" });
58-
58+
5959
await server.server.connect(serverTransport);
6060
await client.connect(clientTransport);
61-
61+
6262
const tools = await client.listTools();
6363
expect(tools.tools).toHaveLength(1);
6464
expect(tools.tools[0].name).toBe("test-tool");
@@ -68,25 +68,25 @@ describe("Title field backwards compatibility", () => {
6868

6969
it("should work with prompts that have title using update", async () => {
7070
const [clientTransport, serverTransport] = InMemoryTransport.createLinkedPair();
71-
71+
7272
const server = new McpServer(
7373
{ name: "test-server", version: "1.0.0" },
7474
{ capabilities: {} }
7575
);
76-
76+
7777
// Register prompt with title by updating after creation
7878
const prompt = server.prompt(
7979
"test-prompt",
8080
"A test prompt",
8181
async () => ({ messages: [{ role: "user", content: { type: "text", text: "test" } }] })
8282
);
8383
prompt.update({ title: "Test Prompt Display Name" });
84-
84+
8585
const client = new Client({ name: "test-client", version: "1.0.0" });
86-
86+
8787
await server.server.connect(serverTransport);
8888
await client.connect(clientTransport);
89-
89+
9090
const prompts = await client.listPrompts();
9191
expect(prompts.prompts).toHaveLength(1);
9292
expect(prompts.prompts[0].name).toBe("test-prompt");
@@ -96,33 +96,33 @@ describe("Title field backwards compatibility", () => {
9696

9797
it("should work with prompts using registerPrompt", async () => {
9898
const [clientTransport, serverTransport] = InMemoryTransport.createLinkedPair();
99-
99+
100100
const server = new McpServer(
101101
{ name: "test-server", version: "1.0.0" },
102102
{ capabilities: {} }
103103
);
104-
104+
105105
// Register prompt with title using registerPrompt
106106
server.registerPrompt(
107107
"test-prompt",
108108
{
109109
title: "Test Prompt Display Name",
110110
description: "A test prompt",
111-
arguments: { input: z.string() }
111+
argsSchema: { input: z.string() }
112112
},
113-
async ({ input }) => ({
114-
messages: [{
115-
role: "user",
116-
content: { type: "text", text: `test: ${input}` }
117-
}]
113+
async ({ input }) => ({
114+
messages: [{
115+
role: "user",
116+
content: { type: "text", text: `test: ${input}` }
117+
}]
118118
})
119119
);
120-
120+
121121
const client = new Client({ name: "test-client", version: "1.0.0" });
122-
122+
123123
await server.server.connect(serverTransport);
124124
await client.connect(clientTransport);
125-
125+
126126
const prompts = await client.listPrompts();
127127
expect(prompts.prompts).toHaveLength(1);
128128
expect(prompts.prompts[0].name).toBe("test-prompt");
@@ -133,12 +133,12 @@ describe("Title field backwards compatibility", () => {
133133

134134
it("should work with resources using registerResource", async () => {
135135
const [clientTransport, serverTransport] = InMemoryTransport.createLinkedPair();
136-
136+
137137
const server = new McpServer(
138138
{ name: "test-server", version: "1.0.0" },
139139
{ capabilities: {} }
140140
);
141-
141+
142142
// Register resource with title using registerResource
143143
server.registerResource(
144144
"test-resource",
@@ -148,19 +148,19 @@ describe("Title field backwards compatibility", () => {
148148
description: "A test resource",
149149
mimeType: "text/plain"
150150
},
151-
async () => ({
152-
contents: [{
151+
async () => ({
152+
contents: [{
153153
uri: "https://example.com/test",
154-
text: "test content"
155-
}]
154+
text: "test content"
155+
}]
156156
})
157157
);
158-
158+
159159
const client = new Client({ name: "test-client", version: "1.0.0" });
160-
160+
161161
await server.server.connect(serverTransport);
162162
await client.connect(clientTransport);
163-
163+
164164
const resources = await client.listResources();
165165
expect(resources.resources).toHaveLength(1);
166166
expect(resources.resources[0].name).toBe("test-resource");
@@ -171,21 +171,21 @@ describe("Title field backwards compatibility", () => {
171171

172172
it("should support serverInfo with title", async () => {
173173
const [clientTransport, serverTransport] = InMemoryTransport.createLinkedPair();
174-
174+
175175
const server = new Server(
176-
{
176+
{
177177
name: "test-server",
178178
version: "1.0.0",
179179
title: "Test Server Display Name"
180180
},
181181
{ capabilities: {} }
182182
);
183-
183+
184184
const client = new Client({ name: "test-client", version: "1.0.0" });
185-
185+
186186
await server.connect(serverTransport);
187187
await client.connect(clientTransport);
188-
188+
189189
const serverInfo = client.getServerVersion();
190190
expect(serverInfo?.name).toBe("test-server");
191191
expect(serverInfo?.version).toBe("1.0.0");

0 commit comments

Comments
 (0)