fix(skill-mcp): handle pre-parsed object arguments in parseArguments (#675)

ashir6892 · web-flow · commit 0809de826222 · 2026-01-11T18:18:32.000+09:00
diff --git a/src/tools/skill-mcp/tools.ts b/src/tools/skill-mcp/tools.ts
@@ -69,8 +69,11 @@ function formatAvailableMcps(skills: LoadedSkill[]): string {
   return mcps.length > 0 ? mcps.join("\n") : "  (none found)"
 }
 
-function parseArguments(argsJson: string | undefined): Record<string, unknown> {
+function parseArguments(argsJson: string | Record<string, unknown> | undefined): Record<string, unknown> {
   if (!argsJson) return {}
+  if (typeof argsJson === "object" && argsJson !== null) {
+    return argsJson
+  }
   try {
     const parsed = JSON.parse(argsJson)
     if (typeof parsed !== "object" || parsed === null) {
diff --git a/src/tools/skill-mcp/types.ts b/src/tools/skill-mcp/types.ts
@@ -3,6 +3,6 @@ export interface SkillMcpArgs {
   tool_name?: string
   resource_name?: string
   prompt_name?: string
-  arguments?: string
+  arguments?: string | Record<string, unknown>
   grep?: string
 }

Original file line number	Diff line number	Diff line change
`@@ -3,6 +3,6 @@ export interface SkillMcpArgs {`
`3`	`3`	`tool_name?: string`
`4`	`4`	`resource_name?: string`
`5`	`5`	`prompt_name?: string`
`6`		`- arguments?: string`
	`6`	`+ arguments?: string \| Record<string, unknown>`
`7`	`7`	`grep?: string`
`8`	`8`	`}`