Skip to content

Conversation

teddyam
Copy link
Contributor

@teddyam teddyam commented Aug 9, 2025

Added new template to template folder and custom hook to require MCP Server Name and MCP Server Link as inputs.

Comment on lines +12 to +13
const serverName = await rl.question('Enter MCP server name: ');
const mcpUrl = await rl.question('Enter MCP server URL: ');
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It'd be great if the template could provide these, and it would go through all the questions

rl.close();

// Validate the MCP URL
const result = z.string().url().safeParse(mcpUrl);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You're putting this in "new" but it's very specific to a particular template.

"@microsoft/teams.dev": "preview",
"@microsoft/teams.graph": "preview",
"@microsoft/teams.mcp": "preview",
"@microsoft/teams.mcpclient": "^2.0.0-preview.8",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why not preview

const res = await prompt.send(activity.text);
console.log(res);

await send({ type: 'message', text: res.content });
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

don't you need to send a message back?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

send with res.content

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants