feat: update changelog format to use brackets for unreleased entries and improve dependency initialization in commands

Signed-off-by: Felipe Sayão Lobato Abreu <github@mentordosnerds.com>

Author: coisa

src/Changelog/UnreleasedEntryChecker.php

@@ -94,7 +94,7 @@ public function hasPendingChanges(string $workingDirectory, ?string $againstRefe
      */
     private function extractEntries(string $contents): array
     {
-        if (0 === preg_match('/^## Unreleased\s+-\s+.+?(?=^##\s|\z)/ms', $contents, $matches)) {
+        if (0 === preg_match('/^## \[?Unreleased\]?\s+-\s+.+?(?=^##\s|\z)/ms', $contents, $matches)) {
             return [];
         }
 

src/Command/ChangelogCheckCommand.php

@@ -31,12 +31,14 @@
 final class ChangelogCheckCommand extends AbstractCommand
 {
     /**
-     * @param Filesystem|null $filesystem
-     * @param UnreleasedEntryCheckerInterface|null $unreleasedEntryChecker
+     * Initializes the command with necessary dependencies.
+     *
+     * @param Filesystem $filesystem filesystem instance for file operations
+     * @param UnreleasedEntryCheckerInterface $unreleasedEntryChecker checker for pending unreleased entries in the changelog
      */
     public function __construct(
-        ?Filesystem $filesystem = null,
-        private readonly ?UnreleasedEntryCheckerInterface $unreleasedEntryChecker = null,
+        Filesystem $filesystem = new Filesystem(),
+        private readonly UnreleasedEntryCheckerInterface $unreleasedEntryChecker = new UnreleasedEntryChecker(),
     ) {
         parent::__construct($filesystem);
     }
@@ -60,14 +62,16 @@ protected function configure(): void
     }
 
     /**
-     * @param InputInterface $input
-     * @param OutputInterface $output
+     * Executes the command to check for pending unreleased changes in the changelog.
+     *
+     * @param InputInterface $input the input interface for command arguments and options
+     * @param OutputInterface $output the output interface for writing command output
      *
-     * @return int
+     * @return int exit code indicating success (0) or failure (1)
      */
     protected function execute(InputInterface $input, OutputInterface $output): int
     {
-        $hasPendingChanges = ($this->unreleasedEntryChecker ?? new UnreleasedEntryChecker())
+        $hasPendingChanges = $this->unreleasedEntryChecker
             ->hasPendingChanges($this->getCurrentWorkingDirectory(), $input->getOption('against'));
 
         if ($hasPendingChanges) {

tests/Composer/Capability/DevToolsCommandProviderTest.php

@@ -30,6 +30,7 @@
 use FastForward\DevTools\Command\SyncCommand;
 use FastForward\DevTools\Command\SkillsCommand;
 use FastForward\DevTools\Agent\Skills\SkillsSynchronizer;
+use FastForward\DevTools\Changelog\UnreleasedEntryChecker;
 use FastForward\DevTools\Command\PhpDocCommand;
 use FastForward\DevTools\Command\RefactorCommand;
 use FastForward\DevTools\Command\ReportsCommand;
@@ -76,6 +77,7 @@
 #[UsesClass(GitAttributesWriter::class)]
 #[UsesClass(GitIgnoreMerger::class)]
 #[UsesClass(Writer::class)]
+#[UsesClass(UnreleasedEntryChecker::class)]
 final class DevToolsCommandProviderTest extends TestCase
 {
     private DevToolsCommandProvider $commandProvider;