Check whether to load databases after sync mode check

[Keats]

Closes #205

[wes-a2ai]

Config:

[project]
# Note: do not update the lockfile from this project since we're testing whether an upgrade
# url is updated by 10 months to upgrade the version/source of the packages
name = "project-upgrade"
r_version = "4.4"
repositories = [
    {alias = "PPM", url = "https://packagemanager.posit.co/cran/2025-04-12"},
    {alias = "PPM-old", url = "https://packagemanager.posit.co/cran/2025-04-01"}
]

dependencies = [
    "ggplot2"
]

# Sync with PPM commented out
$ rv -c example_projects/project-upgrade/rproject.toml sync
Nothing to do

# Sync with PPM uncommented
$ rv -c example_projects/project-upgrade/rproject.toml sync
Nothing to do

# Upgrade showing only changed packages
$ rv -c example_projects/project-upgrade/rproject.toml upgrade
- ggplot2
+ ggplot2 (3.5.2, binary from https://packagemanager.posit.co/cran/2025-04-12) in 0ms
- pillar
+ pillar (1.10.2, binary from https://packagemanager.posit.co/cran/2025-04-12) in 0ms

[Keats]

You can just do

fn is_matching_resolved_dep(&self, dep: &ResolvedDependency) -> bool {
    self.name == dep.name.as_ref() && self.version == dep.version
}

It can also be inlined since it's only used once

[Keats]

can we just create a new vec rather than mutating? I also don't think it's going to be super useful for that to be parallel, did you notice a difference?