Skip to content

Conversation

joshsny
Copy link
Collaborator

@joshsny joshsny commented Sep 2, 2025

Make it clearer - the previous wording was ambiguous.

@Copilot Copilot AI review requested due to automatic review settings September 2, 2025 15:14
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates the data residency section in the README to clarify PostHog's MCP server data handling practices. The change replaces vague language about data processing guarantees with a clear statement about the server's data storage practices.

  • Corrects a spelling error ("gaurantee" → "guarantee")
  • Clarifies that the MCP server doesn't store sensitive data outside the user's cloud region
  • Improves the overall clarity of the data residency explanation

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

@@ -45,7 +45,7 @@ npx @posthog/wizard@latest mcp add

### Data residency

The MCP server is hosted on a Cloudflare worker, this can be located outside of the EU / US, so there is no gaurantee that the data will be processed solely within a specific region.
The MCP server is hosted on a Cloudflare worker which can be located outside of the EU / US, for this reason the MCP server does not store any sensitive data outside of your cloud region.
Copy link
Preview

Copilot AI Sep 2, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[nitpick] There's a spelling error in the original text that was corrected: 'gaurantee' should be 'guarantee'. However, the new text has a grammatical issue - it should use 'therefore' instead of 'for this reason' for better readability, or restructure the sentence for clarity.

Suggested change
The MCP server is hosted on a Cloudflare worker which can be located outside of the EU / US, for this reason the MCP server does not store any sensitive data outside of your cloud region.
The MCP server is hosted on a Cloudflare worker which can be located outside of the EU / US; therefore, the MCP server does not store any sensitive data outside of your cloud region.

Copilot uses AI. Check for mistakes.

@joshsny joshsny requested a review from a team September 2, 2025 15:15
@JonathanLab
Copy link
Collaborator

Is this one good to merge? Better to get it in sooner so people don't get confused @joshsny

@joshsny joshsny merged commit aaa0134 into main Sep 9, 2025
7 checks passed
@joshsny joshsny deleted the data-residency-readme branch September 9, 2025 09:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants