Fix visibility updates during resource upserts

[adamint]

Description

Previously, when upserting a resource, the visibility function passed to UpdateFromResource would return false in the event that any value in the dictionary was false. For example, if container and project are values, container is marked as hidden (false), and then a resource of type project is updated, project will then be set to hidden. This is incorrect. Instead, the value should be visible if it was previously visible in the dictionary or if it's new (ie, when stopping a command, you may see the state Stopping for the first time).

Fixes #9220

Checklist

• Is this feature complete?
  • Yes. Ready to ship.
  • No. Follow-up changes expected.
• Are you including unit tests for the changes and scenario tests if relevant?
  • Yes
  • No
• Did you add public API?
  • Yes
    • If yes, did you have an API Review for it?
      • Yes
      • No
    • Did you add <remarks /> and <code /> elements on your triple slash comments?
      • Yes
      • No
  • No
• Does the change make any security assumptions or guarantees?
  • Yes
    • If yes, have you done a threat model and had a security review?
      • Yes
      • No
  • No
• Does the change require an update in our Aspire docs?
  • Yes
    • Link to aspire-docs issue (consider using one of the following templates):
      • New (or update) doc-idea template
      • New breaking-change template
      • New diagnostic template
  • No

[danmoseley]

does this test all 3 lines of your fix above?

[danmoseley]

/backport to release/9.3

[github-actions]

Started backporting to release/9.3: https://github.com/dotnet/aspire/actions/runs/14986296044