chore: remove meticulous from CI (coder#14369)

sreya · web-flow · commit b6d35edebdf8 · 2024-08-20T14:13:13.000-04:00
diff --git a/.github/workflows/meticulous.yaml b/.github/workflows/meticulous.yaml
diff --git a/coderd/httpmw/csp.go b/coderd/httpmw/csp.go
@@ -59,7 +59,7 @@ func CSPHeaders(telemetry bool, websocketHosts func() []string) func(next http.H
 				cspDirectiveConnectSrc: {"'self'"},
 				cspDirectiveChildSrc:   {"'self'"},
 				// https://github.com/suren-atoyan/monaco-react/issues/168
-				cspDirectiveScriptSrc: {"'self' "},
+				cspDirectiveScriptSrc: {"'self'"},
 				cspDirectiveStyleSrc:  {"'self' 'unsafe-inline'"},
 				// data: is used by monaco editor on FE for Syntax Highlight
 				cspDirectiveFontSrc:   {"'self' data:"},
@@ -88,11 +88,6 @@ func CSPHeaders(telemetry bool, websocketHosts func() []string) func(next http.H
 			if telemetry {
 				// If telemetry is enabled, we report to coder.com.
 				cspSrcs.Append(cspDirectiveConnectSrc, "https://coder.com")
-				// These are necessary to allow meticulous to collect sampling to
-				// improve our testing. Only remove these if we're no longer using
-				// their services.
-				cspSrcs.Append(cspDirectiveConnectSrc, meticulousConnectSrc...)
-				cspSrcs.Append(cspDirectiveScriptSrc, meticulousScriptSrc...)
 			}
 
 			// This extra connect-src addition is required to support old webkit
@@ -136,8 +131,3 @@ func CSPHeaders(telemetry bool, websocketHosts func() []string) func(next http.H
 		})
 	}
 }
-
-var (
-	meticulousConnectSrc = []string{"https://cognito-identity.us-west-2.amazonaws.com", "https://user-events-v3.s3-accelerate.amazonaws.com", "*.sentry.io"}
-	meticulousScriptSrc  = []string{"https://snippet.meticulous.ai", "https://browser.sentry-cdn.com"}
-)
diff --git a/site/src/index.tsx b/site/src/index.tsx
@@ -1,4 +1,3 @@
-import { tryLoadAndStartRecorder } from "@alwaysmeticulous/recorder-loader";
 import { createRoot } from "react-dom/client";
 import { App } from "./App";
 
@@ -15,28 +14,4 @@ if (element === null) {
 }
 
 const root = createRoot(element);
-async function startApp() {
-	// Record all sessions on localhost, staging stacks and preview URLs
-	if (isInternal()) {
-		// Start the Meticulous recorder before you initialise your app.
-		// Note: all errors are caught and logged, so no need to surround with try/catch
-		await tryLoadAndStartRecorder({
-			projectId: "Y4uHy1qs0B660xxUdrkLPkazUMPr6OuTqYEnShaR",
-			isProduction: false,
-		});
-	}
-
-	root.render(<App />);
-}
-
-function isInternal() {
-	return (
-		process.env.NODE_ENV === "development" &&
-		(window.location.hostname.indexOf("localhost") > -1 ||
-			window.location.hostname.indexOf("127.0.0.1") > -1)
-	);
-}
-
-startApp().catch((error) => {
-	console.error(error);
-});
+root.render(<App />);
