-
Notifications
You must be signed in to change notification settings - Fork 86
Add rich menu apis #97
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from 2 commits
771051d
60a8d18
87c7e56
5a0c737
6c55078
595c7fd
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -20,6 +20,8 @@ import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js"; | |||||||||||||||||||||||||
| import { StdioServerTransport } from "@modelcontextprotocol/sdk/server/stdio.js"; | ||||||||||||||||||||||||||
| import * as line from "@line/bot-sdk"; | ||||||||||||||||||||||||||
| import { z } from "zod"; | ||||||||||||||||||||||||||
| import pkg from "../package.json" with { type: "json" }; | ||||||||||||||||||||||||||
| import fs from "fs"; | ||||||||||||||||||||||||||
| import { LINE_BOT_MCP_SERVER_VERSION, USER_AGENT } from "./version.js"; | ||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||
| const NO_USER_ID_ERROR = | ||||||||||||||||||||||||||
|
|
@@ -40,6 +42,13 @@ const messagingApiClient = new line.messagingApi.MessagingApiClient({ | |||||||||||||||||||||||||
| }, | ||||||||||||||||||||||||||
| }); | ||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||
| const lineBlobClient = new line.messagingApi.MessagingApiBlobClient({ | ||||||||||||||||||||||||||
| channelAccessToken: channelAccessToken, | ||||||||||||||||||||||||||
| defaultHeaders: { | ||||||||||||||||||||||||||
| "User-Agent": `${pkg.name}/${pkg.version}`, | ||||||||||||||||||||||||||
| }, | ||||||||||||||||||||||||||
| }); | ||||||||||||||||||||||||||
|
||||||||||||||||||||||||||
| const lineBlobClient = new line.messagingApi.MessagingApiBlobClient({ | |
| channelAccessToken: channelAccessToken, | |
| defaultHeaders: { | |
| "User-Agent": `${pkg.name}/${pkg.version}`, | |
| }, | |
| }); | |
| const lineBlobClient = new line.messagingApi.MessagingApiBlobClient({ | |
| channelAccessToken: channelAccessToken, | |
| defaultHeaders: { | |
| "User-Agent": USER_AGENT, | |
| }, | |
| }); |
Please use ./version.js as shown 🙏
Related to #61
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
Hmm, I'm not sure about the best practice for passing images to the MCP Server...
I tried verifying whether we could pass images through chat from AI agents like Claude by changing the tool argument from a path to base64, but I ran into character limits. (like this)
With the method you implemented, we need to prepare images in advance within the local mcp-server repository, but maybe this approach will not work with the npx installation method (though I can't think of any better alternatives...).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This command needs to be discussed further. Therefore, I will separate the PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.




There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This syntax is not available in Node v20.9 or earlier...
Instead, use ./version.js, so this import is unnecessary.
Related to #61