src/wrapper/connection_handle.hxx:119

• tags is only read/converted and not mutated; consider taking const zval* tags here (and in the .cxx) to better document intent and prevent accidental modification.

  COUCHBASE_API
  void record_core_meter_operation_duration(std::int64_t duration_us, zval* tags);

Couchbase/LoggingMeter.php:45

• With strict_types=1, prefer !== for string comparisons to avoid unintended type coercion ($name !== ...). Also, explicitly declare method visibility (public function close(): void) for consistency and clarity.

        if ($name != ObservabilityConstants::METER_NAME_OPERATION_DURATION) {
            // The logging meter only uses the db.client.operation.duration metric
            return new NoopValueRecorder();
        }

Couchbase/LoggingMeter.php:51

• With strict_types=1, prefer !== for string comparisons to avoid unintended type coercion ($name !== ...). Also, explicitly declare method visibility (public function close(): void) for consistency and clarity.

    function close(): void
    {
    }