fix: allow clearing self-targeting identity with empty string

[jonathannorris]

Summary

• Updated UpdateSelfTargetingIdentityArgsSchema to require a string (removed .nullable()).
• In updateSelfTargetingIdentityHandler, treat empty string as null when calling updateUserProfile.
• Kept MCP tool registration using Zod shapes (SDK-native).

Why

• fixes issue where nullable wasn't working with MCP call for updating the self-targeting identity

[cloudflare-workers-and-pages]

Deploying with    Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status	Name	Latest Commit	Updated (UTC)
✅ Deployment successful! View logs	devcycle-mcp-server	929b529	Aug 27 2025, 09:15 PM

[Copilot]

Pull Request Overview

This PR fixes an issue where the MCP tool for updating self-targeting identity couldn't properly clear the identity field using an empty string. The fix removes the nullable validation from the schema and adds logic to convert empty strings to null in the handler.

• Removed .nullable() validation from UpdateSelfTargetingIdentityArgsSchema
• Added empty string to null conversion logic in the update handler
• Updated schema description to clarify that empty strings can be used to clear the identity

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
src/mcp/types.ts	Removed nullable validation from UpdateSelfTargetingIdentityArgsSchema
src/mcp/tools/selfTargetingTools.ts	Added logic to convert empty strings to null when updating user profile

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}