Avoid percent-encoding for "(" and ")" characters.

[Powerbyte7]

The VRChat API does not accept percent-encoded variants of InstanceID, specifically the characters "(" and ")". This causes an error when attempting the following API call:

https://api.vrchat.cloud/api/1/instances/wrld_28aab3e4-953f-4c85-9223-ef29a0873a6e:31124%7Egroup%28grp_b2a19685-c53e-4c5a-9503-744a5373bf2c%29%7EgroupAccessType%28plus%29%7Eregion%28use%29/shortName

{"error":{"message":"\"malformed url\"","status_code":400,"waf_code":26497}}

When using the raw string without encoding, the API yields a 200 response as expected.

https://vrchat.com/api/1/instances/wrld_28aab3e4-953f-4c85-9223-ef29a0873a6e:31124~group(grp_b2a19685-c53e-4c5a-9503-744a5373bf2c)~groupAccessType(plus)~region(use)/shortName

See vrchatapi/vrchatapi-rust#43 for more details.

[binn]

Thanks for tracking this down @Powerbyte7 — the repro in the description made it easy to confirm.

Your fix is incorporated into #39, with credit. The only change is where it lives: src/VRChat.API/Client/WebRequestPathBuilder.cs is generated from templates/WebRequestPathBuilder.mustache, so #39 puts it in the template rather than a sed over the output. That PR removes 17 of the 19 post-generation seds for the same reason — each one keys on a string the generator may rename, and sed exits 0 when it matches nothing, so a stale one fails silently.

Both call sites now go through one helper:

private static string Escape(string value)
{
    return Uri.EscapeDataString(value).Replace("%28", "(").Replace("%29", ")");
}

Worth noting for anyone reading later: this is safe precisely because Uri.EscapeDataString encodes % first, so a literal %28 inside a value becomes %2528 and the replacement cannot corrupt it.