style: remove inline docblocks, fix copyright year to 2026 on new files

Author: kayjoosten

src/OpenConext/EngineBlock/Logger/Processor/CorrelationIdProcessor.php

@@ -1,7 +1,7 @@
 <?php
 
 /**
- * Copyright 2010 SURFnet B.V.
+ * Copyright 2026 SURFnet B.V.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.

src/OpenConext/EngineBlock/Request/CorrelationId.php

@@ -1,7 +1,7 @@
 <?php
 
 /**
- * Copyright 2010 SURFnet B.V.
+ * Copyright 2026 SURFnet B.V.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -18,9 +18,6 @@
 
 namespace OpenConext\EngineBlock\Request;
 
-/**
- * Immutable value object representing a correlation ID for a SAML authentication flow.
- */
 final class CorrelationId
 {
     public function __construct(public readonly string $correlationId)

src/OpenConext/EngineBlock/Request/CorrelationIdRepository.php

@@ -1,7 +1,7 @@
 <?php
 
 /**
- * Copyright 2010 SURFnet B.V.
+ * Copyright 2026 SURFnet B.V.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -21,13 +21,6 @@
 use Symfony\Component\HttpFoundation\Exception\SessionNotFoundException;
 use Symfony\Component\HttpFoundation\RequestStack;
 
-/**
- * Session-only CRUD for correlation IDs.
- *
- * Stores and retrieves raw correlation ID strings keyed by SAML request ID.
- * Has no knowledge of which correlation ID is currently "active" — that is
- * the responsibility of CorrelationIdService + CurrentCorrelationId.
- */
 final class CorrelationIdRepository
 {
     private const SESSION_KEY = 'CorrelationIds';
@@ -36,15 +29,6 @@ public function __construct(private readonly RequestStack $requestStack)
     {
     }
 
-    /**
-     * Persists a correlation ID for the given request ID.
-     * No-op when no session is available.
-     *
-     * Note: this method unconditionally overwrites any existing entry.
-     * Callers (e.g. CorrelationIdService::mint()) are responsible for the
-     * back-button idempotency guard: call find() first and skip store() if
-     * a value already exists.
-     */
     public function store(string $requestId, CorrelationId $correlationId): void
     {
         try {
@@ -58,10 +42,6 @@ public function store(string $requestId, CorrelationId $correlationId): void
         $session->set(self::SESSION_KEY, $ids);
     }
 
-    /**
-     * Copies the correlation ID from $sourceRequestId to $targetRequestId.
-     * No-op when source is not found or no session is available.
-     */
     public function link(string $targetRequestId, string $sourceRequestId): void
     {
         try {
@@ -80,10 +60,6 @@ public function link(string $targetRequestId, string $sourceRequestId): void
         $session->set(self::SESSION_KEY, $ids);
     }
 
-    /**
-     * Returns the CorrelationId for $requestId, or null when not found.
-     * Returns null when no session is available.
-     */
     public function find(string $requestId): ?CorrelationId
     {
         try {

src/OpenConext/EngineBlock/Request/CorrelationIdService.php

@@ -1,7 +1,7 @@
 <?php
 
 /**
- * Copyright 2010 SURFnet B.V.
+ * Copyright 2026 SURFnet B.V.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -18,17 +18,6 @@
 
 namespace OpenConext\EngineBlock\Request;
 
-/**
- * Orchestrates correlation ID operations across the four SAML legs.
- *
- *  mint()    — generate a new ID for a SAML request (back-button safe)
- *  link()    — copy an existing ID to a new SAML request ID (SP→IdP handoff)
- *  resolve() — look up the ID and push it into CurrentCorrelationId so all
- *              subsequent log entries carry it
- *
- * This is the single entry point used by Corto services. The session
- * interaction is delegated to CorrelationIdRepository.
- */
 final class CorrelationIdService
 {
     public function __construct(
@@ -37,30 +26,18 @@ public function __construct(
     ) {
     }
 
-    /**
-     * Generates and stores a correlation ID for $requestId if none exists yet.
-     * Safe to call multiple times for the same ID (back-button guard).
-     */
     public function mint(string $requestId): void
     {
         if ($this->repository->find($requestId) === null) {
             $this->repository->store($requestId, CorrelationId::mint());
         }
     }
 
-    /**
-     * Copies the correlation ID from $sourceRequestId to $targetRequestId.
-     */
     public function link(string $targetRequestId, string $sourceRequestId): void
     {
         $this->repository->link($targetRequestId, $sourceRequestId);
     }
 
-    /**
-     * Looks up the correlation ID for $requestId and sets it as the active ID
-     * in CurrentCorrelationId so all subsequent log entries carry it.
-     * No-op when $requestId is null or not found.
-     */
     public function resolve(?string $requestId): void
     {
         if ($requestId === null) {

src/OpenConext/EngineBlock/Request/CurrentCorrelationId.php

@@ -1,7 +1,7 @@
 <?php
 
 /**
- * Copyright 2010 SURFnet B.V.
+ * Copyright 2026 SURFnet B.V.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -18,11 +18,6 @@
 
 namespace OpenConext\EngineBlock\Request;
 
-/**
- * Mutable DI singleton that holds the active correlation ID for the current
- * HTTP request. Set once per request by CorrelationIdService::resolve().
- * Read by CorrelationIdProcessor to stamp every log entry.
- */
 final class CurrentCorrelationId
 {
     private ?string $correlationId = null;

src/OpenConext/EngineBlockFunctionalTestingBundle/Features/Context/LoggingContext.php

@@ -1,7 +1,7 @@
 <?php
 
 /**
- * Copyright 2010 SURFnet B.V.
+ * Copyright 2026 SURFnet B.V.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -22,15 +22,6 @@
 use OpenConext\EngineBlockFunctionalTestingBundle\Log\TestLogHandler;
 use PHPUnit\Framework\Assert;
 
-/**
- * Behat context for asserting on structured log output.
- *
- * Injects the in-memory TestLogHandler so scenarios can verify that
- * log records carry the expected structured fields (e.g. correlation_id).
- *
- * Does not extend AbstractSubContext because it performs no browser interactions
- * and has no need for MinkContext.
- */
 class LoggingContext implements Context
 {
     public function __construct(private readonly TestLogHandler $logHandler)

src/OpenConext/EngineBlockFunctionalTestingBundle/Log/TestLogHandler.php

@@ -1,7 +1,7 @@
 <?php
 
 /**
- * Copyright 2010 SURFnet B.V.
+ * Copyright 2026 SURFnet B.V.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -22,12 +22,6 @@
 use Monolog\Level;
 use Monolog\LogRecord;
 
-/**
- * In-memory Monolog handler for Behat log assertions.
- *
- * Collects every log record that passes through it so Behat steps
- * can assert on the presence (or absence) of structured log fields.
- */
 final class TestLogHandler extends AbstractProcessingHandler
 {
     /** @var LogRecord[] */
@@ -43,19 +37,11 @@ protected function write(LogRecord $record): void
         $this->records[] = $record;
     }
 
-    /**
-     * Returns all captured log records since the last reset.
-     *
-     * @return LogRecord[]
-     */
     public function getRecords(): array
     {
         return $this->records;
     }
 
-    /**
-     * Clears all captured log records.
-     */
     public function reset(): void
     {
         $this->records = [];