fix showing dependencies of status package in manifest mode (#4435)

Author: KristofferC

src/Operations.jl

@@ -2944,7 +2944,27 @@ function print_status(
             printpkgstyle(io, :Project, "No packages added to or removed from $(pathrepr(env.project_file))", ignore_indent; color = Base.info_color())
         end
     else
-        xs = !filter ? xs : eltype(xs)[(id, old, new) for (id, old, new) in xs if (id in uuids || something(new, old).name in names)]
+        if filter
+            # Find packages matching the filter
+            matching_ids = Set{UUID}()
+            for (id, old, new) in xs
+                if (id in uuids || something(new, old).name in names)
+                    push!(matching_ids, id)
+                end
+            end
+            # In manifest mode, also include all dependencies of matching packages
+            if manifest && !isempty(matching_ids)
+                deps_to_add = Set{UUID}()
+                for id in matching_ids
+                    entry = get(env.manifest, id, nothing)
+                    if entry !== nothing
+                        union!(deps_to_add, values(entry.deps))
+                    end
+                end
+                union!(matching_ids, deps_to_add)
+            end
+            xs = eltype(xs)[(id, old, new) for (id, old, new) in xs if id in matching_ids]
+        end
         if isempty(xs)
             printpkgstyle(
                 io, Symbol("No Matches"),

test/new.jl

@@ -2957,6 +2957,17 @@ end
         @test any(l -> occursin(r"\[7876af07\] Example\s*v0\.3\.0", l), statuslines)
         @test any(l -> occursin(r"\[2a0f44e3\] Base64", l), statuslines)
         @test any(l -> occursin(r"\[d6f4376e\] Markdown", l), statuslines)
+        # Test that manifest status with filter shows package and its dependencies (issue #1989)
+        Pkg.add(name = "JSON", version = "0.21.0")  # JSON has dependencies
+        Pkg.status("JSON"; io = io, mode = Pkg.PKGMODE_MANIFEST)
+        statuslines = readlines(io)
+        @test occursin(r"Status `.+Manifest.toml`", first(statuslines))
+        @test any(l -> occursin(r"\[682c06a0\] JSON\s*v0\.21\.0", l), statuslines)
+        # JSON's dependencies (Parsers, Dates, Mmap, Unicode) should also be shown
+        @test any(l -> occursin(r"Parsers", l), statuslines)
+        # But Example and Markdown (not dependencies of JSON) should not be shown
+        @test !any(l -> occursin(r"\[7876af07\] Example", l), statuslines)
+        @test !any(l -> occursin(r"\[d6f4376e\] Markdown", l), statuslines)
     end
     # Diff API
     isolate(loaded_depot = true) do