Replace tomasvotruba/unused-public with shipmonk/dead-code-detector

Author: TomasVotruba

composer.json

@@ -51,13 +51,13 @@
         "rector/jack": "^1.1",
         "rector/swiss-knife": "^2.6",
         "shipmonk/composer-dependency-analyser": "^1.8",
+        "shipmonk/dead-code-detector": "^1.4",
         "symfony/process": "^8.1",
         "symplify/easy-coding-standard": "^13.2.13",
         "symplify/phpstan-rules": "^14.13",
         "tomasvotruba/class-leak": "^2.1",
         "tomasvotruba/fast-unit": "^0.1",
         "tomasvotruba/type-coverage": "^2.3.6",
-        "tomasvotruba/unused-public": "^2.2",
         "tracy/tracy": "^2.12"
     },
     "replace": {

phpstan-baseline-dead-code.neon

@@ -0,0 +1,67 @@
+parameters:
+	ignoreErrors:
+		-
+			message: '#^Unused Rector\\Php80\\Contract\\StrStartWithMatchAndRefactorInterface\:\:match$#'
+			identifier: shipmonk.deadMethod
+			count: 1
+			path: rules/Php80/Contract/StrStartWithMatchAndRefactorInterface.php
+
+		-
+			message: '#^Unused Rector\\Php80\\Contract\\StrStartWithMatchAndRefactorInterface\:\:refactorStrStartsWith$#'
+			identifier: shipmonk.deadMethod
+			count: 1
+			path: rules/Php80/Contract/StrStartWithMatchAndRefactorInterface.php
+
+		-
+			message: '#^Unused Rector\\BetterPhpDocParser\\Contract\\PhpDocParser\\PhpDocNodeDecoratorInterface\:\:decorate$#'
+			identifier: shipmonk.deadMethod
+			count: 1
+			path: src/BetterPhpDocParser/Contract/PhpDocParser/PhpDocNodeDecoratorInterface.php
+
+		-
+			message: '#^Unused Rector\\Parallel\\Contract\\SerializableInterface\:\:decode$#'
+			identifier: shipmonk.deadMethod
+			count: 1
+			path: src/Parallel/Contract/SerializableInterface.php
+
+		-
+			message: '#^Unused Rector\\PhpParser\\Printer\\BetterStandardPrinter\:\:pInterpolatedStringPart$#'
+			identifier: shipmonk.deadMethod
+			count: 1
+			path: src/PhpParser/Printer/BetterStandardPrinter.php
+
+		-
+			message: '#^Unused Rector\\PhpParser\\Printer\\BetterStandardPrinter\:\:pStmt_FileNode$#'
+			identifier: shipmonk.deadMethod
+			count: 1
+			path: src/PhpParser/Printer/BetterStandardPrinter.php
+
+		-
+			message: '#^Unused Rector\\Set\\Contract\\SetInterface\:\:getGroupName$#'
+			identifier: shipmonk.deadMethod
+			count: 1
+			path: src/Set/Contract/SetInterface.php
+
+		-
+			message: '#^Unused Rector\\Set\\Contract\\SetInterface\:\:getName$#'
+			identifier: shipmonk.deadMethod
+			count: 1
+			path: src/Set/Contract/SetInterface.php
+
+		-
+			message: '#^Unused Rector\\Set\\Contract\\SetInterface\:\:getSetFilePath$#'
+			identifier: shipmonk.deadMethod
+			count: 1
+			path: src/Set/Contract/SetInterface.php
+
+		-
+			message: '#^Property Rector\\Tests\\BetterPhpDocParser\\PhpDocInfo\\PhpDocInfoPrinter\\AbstractPhpDocInfoPrinterTestCase\:\:\$filePathHelper is never read$#'
+			identifier: shipmonk.deadProperty.neverRead
+			count: 1
+			path: tests/BetterPhpDocParser/PhpDocInfo/PhpDocInfoPrinter/AbstractPhpDocInfoPrinterTestCase.php
+
+		-
+			message: '#^Unused Rector\\Tests\\Caching\\ValueObject\\Storage\\FileCacheStorageTest\:\:provideConfigFilePath$#'
+			identifier: shipmonk.deadMethod
+			count: 1
+			path: tests/Caching/ValueObject/Storage/FileCacheStorageTest.php

phpstan.neon

@@ -1,5 +1,6 @@
 includes:
     - vendor/symplify/phpstan-rules/config/rector-rules.neon
+    - phpstan-baseline-dead-code.neon
 
 rules:
     - Rector\Utils\PHPStan\Rule\RegisterRelatedPolyfillRectorRule
@@ -58,11 +59,10 @@ parameters:
         narrow_param: true
         narrow_return: true
 
-    # see https://github.com/TomasVotruba/unused-public
-    unused_public:
-        methods: true
-        properties: true
-        constants: true
+    # offload breaks cold-cache parallel runs, keep collector data in result cache
+    shipmonkDeadCode:
+        cache:
+            offloadCollectorData: false
 
     ignoreErrors:
         # keepComments() is a generic helper; do not narrow it after the deprecated CombineIfRector stopped calling it