[v6] #143 - Narrow nullable native types, improve PHPStan level to 8

CHANGELOG.md

@@ -16,6 +16,9 @@ Latest
 * [#242](https://github.com/cleverage/process-bundle/issues/242) Fix InputFileReaderTask: an input that is not a non-empty string (e.g. `null`) throws an explicit `\UnexpectedValueException` (`No file path given as input`) instead of a PHP warning followed by a `TypeError`. Update documentation, add tests.
 * [#244](https://github.com/cleverage/process-bundle/issues/244) Fix MappingTransformer: a missing target property of a `\stdClass` destination (`initial_value` or `keep_input`) threw `Property '...' is not writable`, it is now added when the target is a simple property name (nested paths still throw). Update documentation, add tests.
 
+## BC breaks
+* [#143](https://github.com/cleverage/process-bundle/issues/143) Narrow native types that were never `null` in practice: `AbstractConfigurableTask::getOptions()` and `ProcessState::getContextualizedOptions()` return `array` (instead of `?array`), `CsvResource::$lineNumber` is an `int` and `ProcessHistory::$startDate` a `\DateTimeInterface` (instead of nullable types). Overrides of `getOptions()` must declare an `array` return type (e.g. cleverage/archive-process-bundle >= v2.3). Improve PHPStan level from 7 to 8.
+
 v5.1
 -----
 

docs/03-custom_tasks.md

@@ -93,7 +93,8 @@ available in `CleverAge\ProcessBundle\Model\AbstractConfigurableTask`.
 Based on [Symfony's OptionsResolver Component](https://symfony.com/doc/current/components/options_resolver.html) this
 abstract class allows you to implement its `configureOptions` method to add your requirements, default values and
 normalizers. Options are resolved (once) during the task initialization, and can be read with `getOptions($state)` or
-`getOption($state, $code)`. If the resolution fails during initialization, the error is logged and the options are
+`getOption($state, $code)`. If you override `getOptions()`, it must return an `array` (it was typed `?array` before
+v6.0). If the resolution fails during initialization, the error is logged and the options are
 resolved again (failing the process) when the task is first executed. Going on with the process after an
 initialization failure is deprecated since v5: in v6.0, the process will fail before any task is executed (see
 [initializable tasks](02-task_types.md#initializable-tasks)).

phpstan.neon

@@ -1,5 +1,5 @@
 parameters:
-    level: 7
+    level: 8
     paths:
         - src
         - tests

src/Filesystem/CsvResource.php

@@ -34,7 +34,7 @@ class CsvResource implements WritableStructuredFileInterface, SeekableFileInterf
 
     protected int $headerCount;
 
-    protected ?int $lineNumber = 1;
+    protected int $lineNumber = 1;
 
     protected bool $closed = false;
 

src/Model/AbstractConfigurableTask.php

@@ -40,9 +40,9 @@ public function reset(): void
     }
 
     /**
-     * @return array<string, mixed>|null
+     * @return array<string, mixed>
      */
-    protected function getOptions(ProcessState $state): ?array
+    protected function getOptions(ProcessState $state): array
     {
         if (null === $this->options) {
             $resolver = new OptionsResolver();

src/Model/ProcessHistory.php

@@ -30,7 +30,7 @@ class ProcessHistory implements \Stringable
 
     protected string $processCode;
 
-    protected ?\DateTimeInterface $startDate;
+    protected \DateTimeInterface $startDate;
 
     protected ?\DateTimeInterface $endDate = null;
 

src/Model/ProcessState.php

@@ -296,9 +296,9 @@ public function setContext(array $context): void
     }
 
     /**
-     * @return array<string, mixed>|null
+     * @return array<string, mixed>
      */
-    public function getContextualizedOptions(): ?array
+    public function getContextualizedOptions(): array
     {
         if (!$this->contextualizedOptions) {
             $options = $this->getTaskConfiguration()