Skip to content

修改跨平台问题#2

Open
0DennisLee0 wants to merge 3 commits into
handsomestWei:mainfrom
0DennisLee0:fix/cross-platform-issues
Open

修改跨平台问题#2
0DennisLee0 wants to merge 3 commits into
handsomestWei:mainfrom
0DennisLee0:fix/cross-platform-issues

Conversation

@0DennisLee0

Copy link
Copy Markdown
  1. 修复mac/linux java命令错误的问题
  2. 修复出现终端颜色有误,导致无法正确解析依赖的问题

@handsomestWei

Copy link
Copy Markdown
Owner

@0DennisLee0 感谢支持🫰,看到pr的改动主要有三点:

  • 1、跨平台适配。这点没有问题。
  • 2、关于decompile时自动创建依赖索引的改动。
    • src/index.tshandleDecompileClass方法入口Line 173行,已经做了预创建。
    • 因此不需要在DecompilerService.tsdecompile方法内再做一次,同时也保持该方法的职责单一性。
    • 可能在第一次使用decompile指令时会偶现提示未创建索引,在客户端侧大模型应该也会根据返回的提示去自主规划执行扫描指令,不影响主流程。
  • 3、关于部分日志级别从error改为log的改动。
    • 目前mcp使用的是stdio工作模式,使用console.log输出后这些字节会直接插进stdout管道,容易污染和打断客户端的JSON解析,容易导致会话异常。
    • 因此保留error方式会更合适。

关于2、3两点改动我不太认同。可以根据建议调整后重新提交pr

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