fix: better handling for /dev/null

CHANGELOG.md

@@ -1,5 +1,7 @@
 ## Tools development version
 
+- `ccbr-hooks detect-absolute-paths`: Reduce false positives caused by `/dev/null`. (#174, @kelly-sovacool)
+
 ## Tools 0.5.2
 
 - `ccbr-hooks detect-absolute-paths`: Reduce false positives caused by `/dev/shm` and language comments. (#157, @kelly-sovacool)

README.md

@@ -178,7 +178,7 @@ guidelines](https://CCBR.github.io/Tools/CONTRIBUTING).
 Please cite this software if you use it in a publication:
 
 > Sovacool K., Koparde V., Kuhn S., Tandon M., and Huse S. (2026). CCBR
-> Tools: Utilities for CCBR Bioinformatics Software (version v0.5.1).
+> Tools: Utilities for CCBR Bioinformatics Software (version v0.5.2).
 > DOI: 10.5281/zenodo.13377166 URL: https://ccbr.github.io/Tools/
 
 ### Bibtex entry

src/ccbr_tools/hooks/detect_absolute_paths.py

@@ -58,9 +58,9 @@ def word_is_absolute_path(word):
 
     return word.startswith("/") and not any(
         [
-            word == "/dev/null",
-            word.startswith("/dev/shm"),
             word == "/",  # FP from pathlib. abs-path:ignore
+            word.startswith("/dev/null"),
+            word.startswith("/dev/shm"),
             word.startswith("//"),  # FP from groovy comments. abs-path:ignore
             word.startswith("/*"),  # FP from CSS & groovy comments. abs-path:ignore
             word.startswith("/$"),  # FP from nextflow scripts. abs-path:ignore

tests/test_hooks.py

@@ -37,6 +37,8 @@ def test_word_is_absolute_path():
     assert not hooks.word_is_absolute_path("//")  # groovy comments
     assert not hooks.word_is_absolute_path("/*")  # groovy multiline comments
     assert not hooks.word_is_absolute_path("/*--")  # CSS comments
+    assert not hooks.word_is_absolute_path("/dev/null")  # common shell redirection
+    assert not hooks.word_is_absolute_path("/dev/shm")  # common shell redirection
     assert not hooks.word_is_absolute_path("/$")  # nextflow script