TAB does not hide all file entries when before=1

[sje30]

Hi, if I set the context before to something non-zero, e.g. 1 and then do a search, TAB does not hide all the entries in a file; it only hides the first entry.

e.g. if I am in the /etc folder of emacs and do a search for 'bidi' with before:1 then e.g. for the NEWS.25 file I get the folllowing entries:

Search term: bidi change
Search type: string words regexp
Case: smart sensitive ignore
Context: none before:1 after:0

Directory: /usr/local/share/emacs/28.0.90/etc/
Files: all type glob

NEWS.25
315  ** Emacs now supports the latest version of the UBA.
316  The Emacs implementation of the Unicode Bidirectional Algorithm (UBA)
---
319  This includes full support for directional isolates and the
320  Bidirectional Parentheses Algorithm (BPA) specified by these Unicode
---
1472 
1473 ** The new function 'bidi-find-overridden-directionality' allows you to
----
1476 can use this to detect potential phishing of URLs and other links that
1477 exploits bidirectional display reordering.
1478 
1479 ** The new function 'buffer-substring-with-bidi-context' allows you to
----
1481 the visual appearance both of the copied text and the text at
1482 destination, even when the copied text includes mixed bidirectional
----

NEWS.23
2228 character properties.  They are `name', `general-category',
2229 `canonical-combining-class', `bidi-class', `decomposition',
--

If I now hit TAB on NEWS.25, only lines 319 and 320 disappear. This is because in deadgrep--hide the stopping condition in the relevant loop is:

              (while (and
                      (get-text-property (point) 'deadgrep-line-number)
                      (not (bobp)))
                (forward-line))

and the --- separator lines in the output buffer that separate matches in one file do not have the text property deadgrep-line-number. Hence it hides only the first match.

About your environment:
Platform: gnu/linux
Emacs version: 28.0.90
Command: rg --color=ansi --line-number --no-heading --no-column --with-filename --fixed-strings --smart-case --before-context=1 --after-context=0 -- bidi .
default-directory: "/usr/local/share/emacs/28.0.90/etc/"
exec-path: ("/home/stephen/NOBACKUP/compiles/makefile2graph" "//home/stephen/langs/bin/python" "/home/stephen/langs/bin/perl" "//home/stephen/bin" "~/.local/bin" "/home/stephen/langs/bin/R" "/home/stephen/langs/bin/julia" "/home/stephen/langs/bin/sh" "/home/stephen/langs/bin/python" "/home/stephen/langs/bin/perl" "//home/stephen/bin" "/home/stephen/.local/bin" "/usr/local/bin" "/usr/bin" "/bin" "/usr/local/sbin" "/home/stephen/.dotnet/tools" "/usr/lib/jvm/default/bin" "/usr/bin/site_perl" "/usr/bin/vendor_perl" "/usr/bin/core_perl" "/home/stephen/langs/bin/applescript" "/home/stephen/langs/bin/sh" "/home/stephen/.gem/ruby/2.7.0/bin" "/usr/local/libexec/emacs/28.0.90/x86_64-pc-linux-gnu")
tramp-remote-path: (tramp-default-remote-path "/bin" "/usr/bin" "/sbin" "/usr/sbin" "/usr/local/bin" "/usr/local/sbin" "/local/bin" "/local/freeware/bin" "/local/gnu/bin" "/usr/freeware/bin" "/usr/pkg/bin" "/usr/contrib/bin" "/opt/bin" "/opt/sbin" "/opt/local/bin")

Initial output from ripgrep:
"[0m[35m./NEWS.25[0m-[0m[32m315[0m-** Emacs now supports the latest version of the UBA.
[0m[35m./NEWS.25[0m:[0m[32m316[0m:The Emacs implementation of the Unicode [0m[1m[31mBidi[0mrectional Algorithm (UBA)
--
[0m[35m./NEWS.25[0m-[0m[32m319[0m-This includes full support for directional isolates and the
[0m[35m./NEWS.25[0m:[0m[32m320[0m:[0m[1m[31mBidi[0mrectional Parentheses Algorithm (BPA) specified by these Unicode
--
[0m[35m./NEWS.25[0m-[0m[32m1472[0m-
[0m[35m./NEWS.25[0m:[0m[32m1473[0m:** The new function '[0m[1m[31mbidi[0m-find-overridden-directionality' allows you to
--
[0m[35m./NEWS.25[0m-[0m[32m1476[0m-can use this to detect potential phishing of URLs and other links that
[0m[35m./NEWS.25[0m:[0m[32m1477[0m:exploits [0m[1m[31mbidi[0mrectional display reordering.
[0m[35m./NEWS.25[0m-[0m[32m1478[0m-
[0m[35m./NEWS.25[0m:[0m[32m1479[0m:** The new function 'buffer-substring-with-[0m[1m[31mbidi[0m-context' allows you to
--
[0m[35m./NEWS.25[0m-[0m[32m1481[0m-the visual appearance both of the copied text and the text at
[0m[35m./NEWS.25[0m:[0m[32m1482[0m:destination, even when the copied text includes mixed [0m[1m[31mbidi[0mrectional
--
[0m[35m./NEWS.23[0m-[0m[32m2228[0m-character properties.  They are `name', `general-category',
[0m[35m./NEWS.23[0m:[0m[32m2229[0m:`canonical-combining-class', `[0m[1m[31mbidi[0m-class', `decomposition',
"

Please file bugs at https://github.com/Wilfred/deadgrep/issues/new