fix: apply Options service fallback when event service is unset

Client::prepareEvent used `$event->getService() ?: $options->getService()`,
but Event::__construct defaults `$service` to the string `'unknown'`, which
is truthy — so the Elvis operator never fell through to the option. Calls
like `LogtideSdk::init(['service' => 'my-php-app'])` followed by
`captureLog(...)` therefore always shipped `service: "unknown"`.

The fallback now triggers when the event service is `''` or `'unknown'`.
Updated `testCaptureEventFallsToOptionsServiceWhenUnknown`, which had been
written to assert the broken behavior.

Closes logtide-dev/logtide-php#7.

Author: Polliog

CHANGELOG.md

@@ -5,6 +5,12 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.7.4] - 2026-04-14
+
+### Fixed
+
+- Fixed `Options::$service` being silently ignored for any log captured without an explicit per-call service. `Client::prepareEvent` used `$event->getService() ?: $this->options->getService()`, but `Event::__construct` defaults `$service` to the string `'unknown'`, which is truthy — so the Elvis operator never fell through to the option. As a result, calls like `LogtideSdk::init(['service' => 'my-php-app'])` followed by `captureLog(...)` always shipped `service: "unknown"`. The fallback now triggers when the event service is `''` or `'unknown'`. Closes [#7](https://github.com/logtide-dev/logtide-php/issues/7).
+
 ## [0.7.3] - 2026-04-07
 
 ### Fixed

packages/logtide/src/Client.php

@@ -124,7 +124,10 @@ public function getOptions(): Options
 
     private function prepareEvent(Event $event, ?Scope $scope, ?EventHint $hint): ?Event
     {
-        $event->setService($event->getService() ?: $this->options->getService());
+        $service = $event->getService();
+        if ($service === '' || $service === 'unknown') {
+            $event->setService($this->options->getService());
+        }
 
         if ($this->options->getEnvironment() !== null) {
             $event->setEnvironment($this->options->getEnvironment());

packages/logtide/tests/Unit/ClientTest.php

@@ -74,8 +74,6 @@ public function testCaptureEventSetsServiceFromOptions(): void
         $transport = $this->createSpyTransport();
         $client = $this->createClient(['service' => 'my-api'], $transport);
 
-        // Event created without explicit service defaults to 'unknown',
-        // which is falsy in ?: so Options service is applied
         $event = new Event(LogLevel::INFO, 'test');
         $event->setService('');
         $client->captureEvent($event);
@@ -99,14 +97,9 @@ public function testCaptureEventFallsToOptionsServiceWhenUnknown(): void
         $transport = $this->createSpyTransport();
         $client = $this->createClient(['service' => 'my-api'], $transport);
 
-        // captureLog without explicit service creates event with 'unknown' service,
-        // but 'unknown' is truthy so ?: won't fall through. The real service override
-        // happens when passing null to captureLog
         $client->captureLog(LogLevel::INFO, 'test', [], null, null);
 
-        // With no service override in captureLog, the event gets 'unknown' from Event constructor
-        // which is truthy, so it stays as 'unknown'
-        $this->assertSame('unknown', $transport->sentLogs[0]->getService());
+        $this->assertSame('my-api', $transport->sentLogs[0]->getService());
     }
 
     public function testCaptureEventSetsEnvironmentFromOptions(): void