fix(skill-mcp): allow object args and strip quotes

[gilbrotheraway]

Summary

This PR fixes two issues with the skill_mcp tool that caused failures when arguments were passed as objects or single-quoted strings (common with LLMs).

1. Schema Fix: Updated arguments parameter schema in src/tools/skill-mcp/tools.ts to accept both string and Record<string, unknown>. This prevents Zod from coercing object inputs into [object Object], which caused JSON parsing failures.
2. Parser Fix: Updated parseArguments function to strip outer single quotes from string inputs before parsing. This handles cases where LLMs wrap the JSON string in single quotes (e.g., '{"key": "value"}'), which previously caused JSON.parse to fail.

These changes ensure the skill_mcp tool robustly handles various argument formats generated by agents.

---

Summary by cubic

Fixes skill_mcp argument handling to accept both JSON strings and objects, and strips outer single quotes before parsing to prevent failures. This makes LLM-generated calls more reliable and prevents JSON parsing and coercion errors.

• Bug Fixes
  • Updated arguments schema to accept string or object, avoiding Zod coercion to "[object Object]".
  • parseArguments now removes outer single quotes before JSON.parse.

^{Written for commit 65a567e. Summary will update on new commits.}

[github-actions]

All contributors have signed the CLA. Thank you! ✅
_{Posted by the CLA Assistant Lite bot.}

[gilbrotheraway]

I have read the CLA Document and I hereby sign the CLA

[cubic-dev-ai]

No issues found across 1 file

Confidence score: 5/5

• Automated review surfaced no issues in the provided summaries.
• No files require special attention.

[kdcokenny]

@gilbrotheraway appreciate the pr but please ensure that the typecheck passes.

[gilbrotheraway]

@gilbrotheraway appreciate the pr but please ensure that the typecheck passes.

done, the typecheck was not showing on the release version i am on, only on beta