Address remaining CodeRabbit review comments

- Include time and tenant in gauge buffer dedup key to prevent
  overwrites of distinct snapshots
- Add non-root user to Dockerfile for production safety
- Add getValue() edge case tests for floats, numeric strings,
  and non-numeric fallback

https://claude.ai/code/session_01UCN4sKD3Zkdbimn3QgTHSB

Author: claude

Dockerfile

@@ -12,6 +12,8 @@ FROM php:8.3.3-cli-alpine3.19 as final
 
 LABEL maintainer="team@appwrite.io"
 
+RUN addgroup -S appgroup && adduser -S appuser -G appgroup
+
 WORKDIR /code
 
 COPY --from=step0 /src/vendor /code/vendor
@@ -21,4 +23,6 @@ COPY ./tests /code/tests
 COPY ./src /code/src
 COPY ./phpunit.xml /code/phpunit.xml
 
+USER appuser
+
 CMD [ "tail", "-f", "/dev/null" ]

src/Usage/Usage.php

@@ -151,7 +151,8 @@ public function setGauges(array $gauges, int $batchSize = 1000): bool
     public function collectGauge(array $gauge): self
     {
         $key = ($gauge['metric'] ?? '') . ':' . ($gauge['resource'] ?? '') . ':' . ($gauge['projectId'] ?? '')
-            . ':' . ($gauge['resourceId'] ?? '') . ':' . ($gauge['region'] ?? '') . ':' . ($gauge['period'] ?? '');
+            . ':' . ($gauge['resourceId'] ?? '') . ':' . ($gauge['region'] ?? '') . ':' . ($gauge['period'] ?? '')
+            . ':' . ($gauge['time'] ?? '') . ':' . ($gauge['$tenant'] ?? '');
         $this->gaugeBuffer[$key] = $gauge;
         $this->bufferCount++;
         return $this;

tests/Usage/MetricTest.php

@@ -95,4 +95,27 @@ public function testNonNumericTenantReturnsNull(): void
         $metric = new Metric(['tenant' => 'abc']);
         $this->assertNull($metric->getTenant());
     }
+
+    public function testGetValueWithFloat(): void
+    {
+        $metric = new Metric(['value' => 1.75]);
+        $this->assertSame(1.75, $metric->getValue());
+        $this->assertIsFloat($metric->getValue());
+    }
+
+    public function testGetValueWithNumericString(): void
+    {
+        $metric = new Metric(['value' => '42']);
+        $this->assertSame(42, $metric->getValue());
+
+        $metric = new Metric(['value' => '3.14']);
+        $this->assertSame(3.14, $metric->getValue());
+    }
+
+    public function testGetValueWithNonNumericStringReturnsDefault(): void
+    {
+        $metric = new Metric(['value' => 'abc']);
+        $this->assertSame(0, $metric->getValue());
+        $this->assertSame(99, $metric->getValue(99));
+    }
 }