Better encoding

[ertembiyik]

Better support for optionals, arrays and dictionaries

Problem

The current ConvexMobile encoding implementation has limited support for common Swift collection types, leading to compilation errors when trying to use arrays and dictionaries with ConvexEncodable values.

Issue encountered:
When attempting to encode a [String] array, the compiler throws an error:

Generic struct 'Array' requires the types 'String' and '(any ConvexEncodable)?' be equivalent 

Example that fails with the current implementation:

let tools = ["hammer", "screwdriver", "wrench"] let args: [String: any ConvexEncodable] = [ "id": agentId, "tools": tools // ❌ Compilation error ]

The workaround required manual casting:

args["tools"] = tools.map { $0 as ConvexEncodable? } as [ConvexEncodable?]

Solution

This PR enhances the ConvexEncodable protocol support by adding proper extensions for:

1. Generic Array support: Array<Element: ConvexEncodable> replaces the specific [ConvexEncodable?] extension
2. Generic Dictionary support: Dictionary<String, ConvexEncodable> replaces the specific [String: ConvexEncodable?] extension
3. Optional support: Optional<Wrapped: ConvexEncodable> provides clean null encoding
4. Removed unneeded .sorted() call for better performance

After this PR

The problematic example now works seamlessly:

let tools = ["hammer", "screwdriver", "wrench"] let userData = ["name": "John", "email": "john@example.com"] let args: [String: any ConvexEncodable] = [ "id": agentId, "tools": tools, // ✅ Works directly "metadata": userData // ✅ Works directly ]

The changes are backward compatible.

This improvement makes ConvexMobile more intuitive to use with standard Swift collection types while maintaining the same encoding behavior.

---

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

[nipunn1313]

Hi! Thanks for the PR. It may be a little while before we get around to reviewing this as mobile has been stable for a while, but appreciate that energy in the mobile swift area.

[ertembiyik]

Hi! Thanks for the PR. It may be a little while before we get around to reviewing this as mobile has been stable for a while, but appreciate that energy in the mobile swift area.

sorry, wrong branch! will close this one
open a new here #4