Merge pull request #214 from andreaswolf/add-line-number

[FEATURE] Add line number to file path

Author: simonschaufi

e2e/typo3-typoscript/expected-output.txt

@@ -2,7 +2,7 @@
 1 file with changes
 ===================
 
-1) typo3-typoscript/result/cache-hash.typoscript
+1) typo3-typoscript/result/cache-hash.typoscript:3
 
     ---------- begin diff ----------
 @@ @@

e2e/typo3-yaml/expected-output.txt

@@ -2,7 +2,7 @@
 1 file with changes
 ===================
 
-1) typo3-yaml/result/my_form.form.yaml
+1) typo3-yaml/result/my_form.form.yaml:2
 
     ---------- begin diff ----------
 @@ @@

packages/fractor/src/ChangesReporting/Output/ConsoleOutputFormatter.php

@@ -61,6 +61,11 @@ private function reportFileDiffs(array $fileDiffs, bool $absoluteFilePath): void
         $i = 0;
         foreach ($fileDiffs as $fileDiff) {
             $filePath = $absoluteFilePath ? $fileDiff->getAbsoluteFilePath() ?? '' : $fileDiff->getRelativeFilePath();
+            // append line number for faster file jump in diff
+            $firstLineNumber = $fileDiff->getFirstLineNumber();
+            if ($firstLineNumber !== null) {
+                $filePath .= ':' . $firstLineNumber;
+            }
             $message = \sprintf('<options=bold>%d) %s</>', ++$i, $filePath);
             $this->symfonyStyle->writeln($message);
             $this->symfonyStyle->newLine();

packages/fractor/src/Differ/ValueObject/FileDiff.php

@@ -4,8 +4,21 @@
 
 namespace a9f\Fractor\Differ\ValueObject;
 
+use Nette\Utils\Strings;
+
 final readonly class FileDiff
 {
+    /**
+     * @var string
+     */
+    private const FIRST_LINE_KEY = 'first_line';
+
+    /**
+     * @var string
+     * @se https://regex101.com/r/AUPIX4/1
+     */
+    private const FIRST_LINE_REGEX = '#@@(.*?)(?<' . self::FIRST_LINE_KEY . '>\\d+)(.*?)@@#';
+
     public function __construct(
         private string $relativeFilePath,
         private string $diff,
@@ -42,4 +55,14 @@ public function getAppliedRules(): array
     {
         return $this->appliedRules;
     }
+
+    public function getFirstLineNumber(): ?int
+    {
+        $match = Strings::match($this->diff, self::FIRST_LINE_REGEX);
+        // probably some error in diff
+        if (! isset($match[self::FIRST_LINE_KEY])) {
+            return null;
+        }
+        return (int) $match[self::FIRST_LINE_KEY];
+    }
 }