feat: use MinIO to storage chart image #12

Susuperli · 2025-07-28T11:25:19Z

fixed use MinIO to save the chart image base64 and return the url. #10

添加MinIO对象存储支持，将图表图片存储为URL而非Base64数据
重构所有图表工具使用统一的generateChartImageForTool方法
更新文档说明MinIO配置和故障排查指南
添加环境变量配置和调试日志支持

添加MinIO对象存储支持，将图表图片存储为URL而非Base64数据重构所有图表工具使用统一的generateChartImageForTool方法更新文档说明MinIO配置和故障排查指南添加环境变量配置和调试日志支持

Susuperli · 2025-07-28T11:28:56Z

将返回的HTML img标签改为纯文本文件地址显示，提高兼容性

hustcc · 2025-07-28T12:21:14Z

神速～

hustcc · 2025-07-28T12:23:20Z

src/tools/boxplot.ts

+      height,
+      theme,
+      "png",
+      "generateboxplot_chart",


为啥缺了一个下划线？

hustcc · 2025-07-28T12:24:47Z

src/utils/imageHandler.ts

+
+/**
+ * 统一的图片生成和处理方法
+ * 根据配置自动决定返回base64图片数据还是MinIO URL的img标签


中英文，中文和数字之间加一个空格。下同～

hustcc · 2025-07-28T12:25:54Z

src/utils/imageHandler.ts

+      content: [
+        {
+          type: "text",
+          text: `Chart image is available at: ${result}`,


url 也是直接返回，不用加一段描述文本。

hustcc · 2025-07-28T12:27:40Z

src/utils/render.ts

+  return !!(
+    process.env.MINIO_ACCESS_KEY &&
+    process.env.MINIO_SECRET_KEY &&
+    process.env.MINIO_ENDPOINT


minio 需要这么多配置？

access_key跟secret_key是必须有的

hustcc · 2025-07-28T12:29:00Z

src/utils/render.ts

+  }
+
+  // Fallback to base64 if MinIO is not configured
+  return `data:image/png;base64,${buffer.toString("base64")}`;


这个写法不好，方法不纯粹，这个方法可以统一返回 buffer，至于 buffer 怎么处理，可以在外层如处理。

hustcc · 2025-07-28T12:36:16Z

minio 只能本地启动软件，然后使用吗？感觉这个用法稍显复杂啊，有没有更简单的用法。

hustcc · 2025-07-28T12:36:56Z

@Susuperli 有兴趣加一个微信，看我主页有微信号。我们电话聊一下这个 PR。

hustcc · 2025-07-28T12:40:20Z

Readme 的 contributors 中，加一下自己吧，可以放到第二个，贡献第二。

hustcc · 2025-07-28T12:50:38Z

README.md

+- ✅ **Reduced token usage** - URLs are much shorter than Base64 data
+- ✅ **Permanent accessibility** - Charts remain available long-term
+- ✅ **Easy sharing** - Send URLs to others to view charts
+- ✅ **Better UX** - Most MCP clients handle URLs better than Base64


移除，内容精炼到 feature 部分即可。

hustcc · 2025-07-28T12:56:12Z

README.md

+   MINIO_BUCKET_NAME=mcp-echarts
+   ```
+
+3. **Test the connection:**


README.md

hustcc · 2025-07-28T12:57:30Z

README.md

+
+### Fallback Behavior
+
+If MinIO is not configured or unavailable, the system automatically falls back to Base64 data output, ensuring compatibility.


hustcc · 2025-07-28T12:58:43Z

TROUBLESHOOTING.md

@@ -0,0 +1,90 @@
+# Troubleshooting Empty Results


移除，后续按需加。

hustcc · 2025-07-28T13:05:05Z

src/utils/imageHandler.ts

+/**
+ * MCP内容类型
+ */
+export interface MCPContent {


hustcc · 2025-07-28T13:13:33Z

src/utils/imageHandler.ts

+ * @param toolName 工具名称（用于调试日志）
+ * @returns MCP响应内容
+ */
+export async function generateChartImageForTool(


合并成一个方法：generateChartImage

hustcc · 2025-07-28T13:14:39Z

src/utils/render.ts

@@ -1,7 +1,10 @@
+import fs from "node:fs";


这个文件基本不变，保持纯函数。

minio 的单独一个文件，所有相关都放这里面，按需 export。

将`generateChartImageForTool`合并到`generateChartImage`中，简化代码结构将MinIO相关逻辑抽离到单独模块更新所有工具文件使用新的统一函数优化README文档并移除过时的TROUBLESHOOTING文件

更新对象存储提供商集成部分，将标题改为更通用的描述并添加MinIO的详细集成信息

hustcc · 2025-07-29T10:26:37Z

代码有冲突了~

refactor(imageHandler): 修改图表图片返回格式为纯文本地址

c133c11

将返回的HTML img标签改为纯文本文件地址显示，提高兼容性