Apply fixed Coderabbitai review.

Author: terabytesoftw

CHANGELOG.md

@@ -7,4 +7,4 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## 0.1.0 Under development
 
-- feat: add framework agnostic debug snapshot contracts, strict JSON hydration, and filesystem persistence.
+- feat: add framework-agnostic debug snapshot contracts, strict JSON hydration, and filesystem persistence.

README.md

@@ -7,6 +7,8 @@ instead of requiring this package directly.
 
 ## Installation
 
+Adapter packages install Debug Core transitively. If you develop an adapter, run:
+
 ```shell
 composer require php-forge/debug-core
 ```

src/Storage/SnapshotStore.php

@@ -383,7 +383,7 @@ private function initialize(): void
      */
     private static function isValidTag(string $tag): bool
     {
-        return $tag !== 'index' && $tag !== '' && preg_match('/\A[A-Za-z0-9._-]+\z/D', $tag) === 1;
+        return $tag !== 'index' && preg_match('/\A(?!-?(?:0|[1-9][0-9]*)\z)[A-Za-z0-9._-]+\z/D', $tag) === 1;
     }
 
     /**

tests/Storage/SnapshotStoreTest.php

@@ -5,7 +5,7 @@
 namespace PHPForge\Debug\Tests\Storage;
 
 use PHPForge\Debug\Storage\{DebugSnapshot, RequestSummary, SnapshotStore, StorageException};
-use PHPUnit\Framework\Attributes\Group;
+use PHPUnit\Framework\Attributes\{Group, TestWith};
 use PHPUnit\Framework\TestCase;
 use Xepozz\InternalMocker\MockerState;
 
@@ -269,6 +269,26 @@ public function testThrowStorageExceptionForNegativeHistorySize(): void
         }
     }
 
+    /**
+     * @param string $tag Numeric tag.
+     */
+    #[TestWith(['0'])]
+    #[TestWith(['7'])]
+    public function testThrowStorageExceptionForNumericTag(string $tag): void
+    {
+        $summary = $this->summary($tag, 1_700_000_000.0);
+
+        $this->expectException(StorageException::class);
+        $this->expectExceptionMessage(
+            "Invalid debug snapshot tag: {$tag}",
+        );
+
+        $this->store()->writeSnapshot(
+            new DebugSnapshot($summary, [], []),
+            10,
+        );
+    }
+
     public function testThrowStorageExceptionForReservedManifestTag(): void
     {
         $summary = $this->summary('index', 1_700_000_000.0);