Fix: Better client info replacement

[ivanagas]

Changes

In many places in code snippets, <ph_project_api_key> and <ph_client_api_host> aren't being replaced. There seems to be two reasons for this:

1. We only check if the string exactly matches, meaning we miss cases where <ph_project_api_key> is part of a larger string.
2. Because of how the code block works, a <ph_project_api_key> is being split in 3 tokens and not matching the check.

This fixes both these issues. Some feedback requested:

1. I'm not sure either of these approaches are actually good ones.
2. I'm not sure the impact on performance, the replaceClientApiInfo function is two loops which could be problematic.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated (UTC)
posthog	❌ Failed (Inspect)		May 7, 2024 10:58pm

[ivanagas]

Eli did it better