Add PSR-3/PSR-20 compliance, TrendReporter, and script_filename tracking

PSR compliance:
- PSR-3: LogReporter now type-hints Psr\Log\LoggerInterface instead of
  duck-typing with ?object. psr/log moved to require (from require-dev).
- PSR-20: TimeCollector accepts optional Psr\Clock\ClockInterface for
  testable timing. Falls back to hrtime() when no clock is provided.
  psr/clock added to require.

New features:
- TrendReporter: generates sci-trend.txt with per-script SCI trajectories,
  ASCII sparklines, trend indicators (improved/stable/worse), and a
  chronological history table with delta markers.
- RequestCollector now captures request.script_filename ($_SERVER['SCRIPT_FILENAME']
  or argv[0]) for tracking which PHP file is being profiled.

Both bootstrap.php and phar stub register the trend reporter.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: fullo

.claude/CLAUDE.md

@@ -44,6 +44,9 @@ Before every push to the repository:
 - All classes are `final` with `declare(strict_types=1)`
 - No dependencies in production code (only dev: phpunit, phpstan, php-cs-fixer)
 - Reporter errors are always caught silently — never break the host application
+- PSR-1: Basic Coding Standard
+- PSR-3: Logger Interface (accetteremo un PSR-3 logger opzionale)
+- PSR-20: Clock interface (per testabilità del timing)
 
 ## The phar
 

bin/build-phar.php

@@ -122,9 +122,10 @@
 $__sciProfiler->addCollector(new \SciProfiler\Collector\RequestCollector());
 
 $__sciReporterMap = [
-    'json' => static fn () => new \SciProfiler\Reporter\JsonReporter(),
-    'log'  => static fn () => new \SciProfiler\Reporter\LogReporter(),
-    'html' => static fn () => new \SciProfiler\Reporter\HtmlReporter(),
+    'json'  => static fn () => new \SciProfiler\Reporter\JsonReporter(),
+    'log'   => static fn () => new \SciProfiler\Reporter\LogReporter(),
+    'html'  => static fn () => new \SciProfiler\Reporter\HtmlReporter(),
+    'trend' => static fn () => new \SciProfiler\Reporter\TrendReporter(),
 ];
 foreach ($__sciConfig->getReporters() as $__rName) {
     $__rName = trim($__rName);

bin/sci-profiler.phar

@@ -11,18 +11,14 @@
         }
     ],
     "require": {
-        "php": ">=8.1"
+        "php": ">=8.1",
+        "psr/clock": "^1.0",
+        "psr/log": "^2.0|^3.0"
     },
     "require-dev": {
-        "phpunit/phpunit": "^10.0|^11.0",
-        "phpstan/phpstan": "^1.10",
         "friendsofphp/php-cs-fixer": "^3.0",
-        "psr/log": "^3.0",
-        "psr/clock": "^1.0"
-    },
-    "suggest": {
-        "psr/log": "For PSR-3 compatible logging of SCI results",
-        "psr/clock": "For PSR-20 compatible clock (testing)"
+        "phpstan/phpstan": "^1.10",
+        "phpunit/phpunit": "^10.0|^11.0"
     },
     "autoload": {
         "psr-4": {

composer.json

@@ -14,6 +14,7 @@ final class RequestCollector implements CollectorInterface
 {
     private string $method = '';
     private string $uri = '';
+    private string $scriptFilename = '';
     private int $inputBytes = 0;
     private int $outputBytes = 0;
     private int $responseCode = 0;
@@ -22,6 +23,7 @@ public function start(): void
     {
         $this->method = $_SERVER['REQUEST_METHOD'] ?? 'CLI';
         $this->uri = $_SERVER['REQUEST_URI'] ?? ($_SERVER['SCRIPT_FILENAME'] ?? 'unknown');
+        $this->scriptFilename = $_SERVER['SCRIPT_FILENAME'] ?? ($_SERVER['argv'][0] ?? 'unknown');
 
         // Use CONTENT_LENGTH header when available to avoid reading the entire
         // request body into memory (file uploads, large JSON payloads, etc.).
@@ -48,6 +50,7 @@ public function getMetrics(): array
         return [
             'method' => $this->method,
             'uri' => $this->uri,
+            'script_filename' => $this->scriptFilename,
             'response_code' => $this->responseCode,
             'input_bytes' => $this->inputBytes,
             'output_bytes' => $this->outputBytes,

src/Collector/RequestCollector.php

@@ -4,36 +4,52 @@
 
 namespace SciProfiler\Collector;
 
+use Psr\Clock\ClockInterface;
+
 /**
  * Collects wall time and CPU time metrics.
  *
- * Uses hrtime() for nanosecond-precision wall time
- * and getrusage() for user/system CPU time on supported platforms.
+ * Uses hrtime() for nanosecond-precision wall time by default.
+ * Accepts an optional PSR-20 ClockInterface for testable timing.
+ * Uses getrusage() for user/system CPU time on supported platforms.
+ *
+ * @see https://www.php-fig.org/psr/psr-20/
  */
 final class TimeCollector implements CollectorInterface
 {
-    /** Whether getrusage() is available — checked once, not per call. */
+    /** Whether getrusage() is available — checked once per process. */
     private static bool $hasRusage;
 
     private int $startHrtime = 0;
     private int $stopHrtime = 0;
 
+    private ?\DateTimeImmutable $startTime = null;
+    private ?\DateTimeImmutable $stopTime = null;
+
     /** @var array{ru_utime.tv_sec: int, ru_utime.tv_usec: int, ru_stime.tv_sec: int, ru_stime.tv_usec: int}|null */
     private ?array $startRusage = null;
 
     /** @var array{ru_utime.tv_sec: int, ru_utime.tv_usec: int, ru_stime.tv_sec: int, ru_stime.tv_usec: int}|null */
     private ?array $stopRusage = null;
 
-    public function __construct()
-    {
-        // Cache function availability once per process, not per start()/stop() call.
+    /**
+     * @param ClockInterface|null $clock Optional PSR-20 clock for testability.
+     *                                   When null, uses hrtime() for nanosecond precision.
+     */
+    public function __construct(
+        private readonly ?ClockInterface $clock = null,
+    ) {
         self::$hasRusage ??= function_exists('getrusage');
     }
 
     public function start(): void
     {
         $this->startHrtime = hrtime(true);
 
+        if ($this->clock !== null) {
+            $this->startTime = $this->clock->now();
+        }
+
         if (self::$hasRusage) {
             $this->startRusage = getrusage();
         }
@@ -43,16 +59,29 @@ public function stop(): void
     {
         $this->stopHrtime = hrtime(true);
 
+        if ($this->clock !== null) {
+            $this->stopTime = $this->clock->now();
+        }
+
         if (self::$hasRusage) {
             $this->stopRusage = getrusage();
         }
     }
 
     public function getMetrics(): array
     {
-        $wallTimeNs = $this->stopHrtime - $this->startHrtime;
-        $wallTimeMs = $wallTimeNs / 1_000_000;
-        $wallTimeSec = $wallTimeNs / 1_000_000_000;
+        // When a PSR-20 clock is provided and both timestamps exist,
+        // use the clock for wall time (enables deterministic testing).
+        if ($this->startTime !== null && $this->stopTime !== null) {
+            $wallTimeSec = (float) $this->stopTime->format('U.u')
+                - (float) $this->startTime->format('U.u');
+            $wallTimeNs = (int) ($wallTimeSec * 1_000_000_000);
+            $wallTimeMs = $wallTimeSec * 1_000;
+        } else {
+            $wallTimeNs = $this->stopHrtime - $this->startHrtime;
+            $wallTimeMs = $wallTimeNs / 1_000_000;
+            $wallTimeSec = $wallTimeNs / 1_000_000_000;
+        }
 
         $metrics = [
             'wall_time_ns' => $wallTimeNs,

src/Collector/TimeCollector.php

@@ -4,25 +4,24 @@
 
 namespace SciProfiler\Reporter;
 
+use Psr\Log\LoggerInterface;
 use SciProfiler\Config;
 use SciProfiler\ProfileResult;
 
 /**
  * Writes a human-readable log line per request.
  *
- * If a PSR-3 logger is provided, uses it. Otherwise writes to a plain file.
+ * If a PSR-3 LoggerInterface is provided, uses it. Otherwise writes to a plain file.
+ *
+ * @see https://www.php-fig.org/psr/psr-3/
  */
 final class LogReporter implements ReporterInterface
 {
     use EnsuresOutputDirectory;
 
-    /** @var \Psr\Log\LoggerInterface|null */
-    private ?object $logger;
+    private ?LoggerInterface $logger;
 
-    /**
-     * @param object|null $logger Optional PSR-3 logger instance
-     */
-    public function __construct(?object $logger = null)
+    public function __construct(?LoggerInterface $logger = null)
     {
         $this->logger = $logger;
     }
@@ -40,7 +39,7 @@ public function report(ProfileResult $result, Config $config): void
             $data['memory.memory_peak_mb'] ?? '?',
         );
 
-        if ($this->logger !== null && method_exists($this->logger, 'info')) {
+        if ($this->logger !== null) {
             $this->logger->info($line, $result->toArray());
             return;
         }