chore: lint, lower frame default for libcurl test

Author: cat-ph

README.md

@@ -47,7 +47,7 @@ PostHog::init('phc_xxx', [
     ],
     'error_tracking_include_source_context' => true,
     'error_tracking_context_lines' => 5,
-    'error_tracking_max_frames' => 50,
+    'error_tracking_max_frames' => 20,
     'error_tracking_context_provider' => static function (array $payload): array {
         return [
             'distinctId' => $_SESSION['user_id'] ?? null,

example.php

@@ -1,5 +1,7 @@
 <?php
 
+// phpcs:disable PSR1.Files.SideEffects
+
 // PostHog PHP library example
 //
 // This script demonstrates various PostHog PHP SDK capabilities including:
@@ -267,7 +269,11 @@ function flagDependencyExamples()
         [],
         true
     );
-    echo "📊 Beta feature comparison - @example.com: " . json_encode($beta1) . ", regular: " . json_encode($beta2) . "\n";
+    echo "📊 Beta feature comparison - @example.com: "
+        . json_encode($beta1)
+        . ", regular: "
+        . json_encode($beta2)
+        . "\n";
 
     echo "\n🎯 Results Summary:\n";
     echo "   - Flag dependencies evaluated locally: " . ($result1 != $result2 ? "✅ YES" : "❌ NO") . "\n";
@@ -302,7 +308,10 @@ function flagDependencyExamples()
         true
     );
     if ($dependentResult3 !== "breaking-bad") {
-        echo "     ❌ Something went wrong evaluating 'multivariate-root-flag' with pineapple@example.com. Expected 'breaking-bad', got '" . json_encode($dependentResult3) . "'\n";
+        echo "     ❌ Something went wrong evaluating 'multivariate-root-flag' with pineapple@example.com. "
+            . "Expected 'breaking-bad', got '"
+            . json_encode($dependentResult3)
+            . "'\n";
     } else {
         echo "✅ 'multivariate-root-flag' with email pineapple@example.com succeeded\n";
     }
@@ -317,7 +326,10 @@ function flagDependencyExamples()
         true
     );
     if ($dependentResult4 !== "the-wire") {
-        echo "     ❌ Something went wrong evaluating multivariate-root-flag with mango@example.com. Expected 'the-wire', got '" . json_encode($dependentResult4) . "'\n";
+        echo "     ❌ Something went wrong evaluating multivariate-root-flag with mango@example.com. "
+            . "Expected 'the-wire', got '"
+            . json_encode($dependentResult4)
+            . "'\n";
     } else {
         echo "✅ 'multivariate-root-flag' with email mango@example.com succeeded\n";
     }
@@ -491,7 +503,9 @@ function etagPollingExamples()
         if ($beforeEtag === $afterEtag && $beforeEtag !== null) {
             echo "No change (304 Not Modified) - $currentFlagCount flag(s)\n";
         } else {
-            echo "🔄 Flags updated! New ETag: " . ($afterEtag ? substr($afterEtag, 0, 20) . "..." : "none") . " - $currentFlagCount flag(s)\n";
+            echo "🔄 Flags updated! New ETag: "
+                . ($afterEtag ? substr($afterEtag, 0, 20) . "..." : "none")
+                . " - $currentFlagCount flag(s)\n";
         }
 
         $iteration++;

lib/Client.php

@@ -163,7 +163,8 @@ public function capture(array $message)
             $flags = [];
 
             if (count($this->featureFlags) != 0) {
-                # Local evaluation is enabled, flags are loaded, so try and get all flags we can without going to the server
+                // Local evaluation is enabled, flags are loaded, so try and get all flags
+                // we can without going to the server.
                 $flags = $this->getAllFlags($message["distinct_id"], $message["groups"], [], [], true);
             } else {
                 $flags = $this->fetchFeatureVariants($message["distinct_id"], $message["groups"]);
@@ -194,8 +195,11 @@ public function capture(array $message)
      * @param array $additionalProperties Extra properties merged into the event
      * @return bool whether the capture call succeeded
      */
-    public function captureException(\Throwable|string $exception, ?string $distinctId = null, array $additionalProperties = []): bool
-    {
+    public function captureException(
+        \Throwable|string $exception,
+        ?string $distinctId = null,
+        array $additionalProperties = []
+    ): bool {
         $noDistinctIdProvided = $distinctId === null;
         if ($noDistinctIdProvided) {
             $distinctId = Uuid::v4();

lib/ExceptionCapture.php

@@ -10,13 +10,13 @@ class ExceptionCapture
 
     private static bool $includeSourceContext = true;
     private static int $contextLines = 5;
-    private static int $maxFrames = 50;
+    private static int $maxFrames = 20;
 
     public static function configure(array $options = []): void
     {
         self::$includeSourceContext = (bool) ($options['error_tracking_include_source_context'] ?? true);
         self::$contextLines = max(0, (int) ($options['error_tracking_context_lines'] ?? 5));
-        self::$maxFrames = max(0, (int) ($options['error_tracking_max_frames'] ?? 50));
+        self::$maxFrames = max(0, (int) ($options['error_tracking_max_frames'] ?? 20));
     }
 
     /**

lib/PostHog.php

@@ -65,8 +65,11 @@ public static function init(
      * @return bool
      * @throws Exception
      */
-    public static function captureException(\Throwable|string $exception, ?string $distinctId = null, array $additionalProperties = []): bool
-    {
+    public static function captureException(
+        \Throwable|string $exception,
+        ?string $distinctId = null,
+        array $additionalProperties = []
+    ): bool {
         self::checkClient();
         return self::$client->captureException($exception, $distinctId, $additionalProperties);
     }

phpunit.xml

@@ -11,6 +11,7 @@
          stopOnSkipped="false"
          stopOnRisky="false"
          cacheDirectory=".phpunit.cache"
+         bootstrap="test/error_log_mock.php"
 >
   <coverage includeUncoveredFiles="true">
     <report>

test/ErrorTrackingRegistrarTest.php

@@ -2,11 +2,10 @@
 
 namespace PostHog\Test;
 
-require_once 'test/error_log_mock.php';
-
 use PHPUnit\Framework\TestCase;
 use PostHog\Client;
 use PostHog\ErrorTrackingRegistrar;
+use PostHog\ExceptionCapture;
 
 class ErrorTrackingRegistrarTest extends TestCase
 {
@@ -18,6 +17,7 @@ class ErrorTrackingRegistrarTest extends TestCase
     public function setUp(): void
     {
         date_default_timezone_set("UTC");
+        ExceptionCapture::configure([]);
         ErrorTrackingRegistrar::resetForTests();
 
         global $errorMessages;
@@ -104,7 +104,12 @@ public function testExceptionHandlerCapturesFlushesAndChainsPreviousHandler(): v
         $previousCalls = 0;
         $receivedException = null;
 
-        $previousExceptionHandler = static function (\Throwable $exception) use (&$previousCalls, &$receivedException): void {
+        $previousExceptionHandler = static function (
+            \Throwable $exception
+        ) use (
+            &$previousCalls,
+            &$receivedException
+        ): void {
             $previousCalls++;
             $receivedException = $exception;
         };
@@ -161,7 +166,12 @@ public function testExceptionHandlerRethrowsWhenNoPreviousHandlerExists(): void
     public function testErrorHandlerCapturesNonFatalErrorsWithoutRegistrarFrames(): void
     {
         $previousCalls = 0;
-        $previousErrorHandler = static function (int $errno, string $message, string $file, int $line) use (&$previousCalls): bool {
+        $previousErrorHandler = static function (
+            int $errno,
+            string $message,
+            string $file,
+            int $line
+        ) use (&$previousCalls): bool {
             $previousCalls++;
             return true;
         };
@@ -206,7 +216,12 @@ public function testErrorHandlerCapturesNonFatalErrorsWithoutRegistrarFrames():
     public function testErrorHandlerRespectsRuntimeSuppression(): void
     {
         $previousCalls = 0;
-        $previousErrorHandler = static function (int $errno, string $message, string $file, int $line) use (&$previousCalls): bool {
+        $previousErrorHandler = static function (
+            int $errno,
+            string $message,
+            string $file,
+            int $line
+        ) use (&$previousCalls): bool {
             $previousCalls++;
             return true;
         };

test/ExceptionCaptureTest.php

@@ -2,8 +2,6 @@
 
 namespace PostHog\Test;
 
-require_once 'test/error_log_mock.php';
-
 use Exception;
 use PHPUnit\Framework\TestCase;
 use PostHog\Client;
@@ -22,6 +20,7 @@ class ExceptionCaptureTest extends TestCase
     public function setUp(): void
     {
         date_default_timezone_set("UTC");
+        ExceptionCapture::configure([]);
         $this->httpClient = new MockedHttpClient("app.posthog.com");
         $this->client = new Client(
             self::FAKE_API_KEY,