Optimize startup project metadata refresh without changing global GetAll loading behavior

[Copilot]

The review flagged startup overhead from calling ProjectService.GetAll(context) (deep eager-load + materialization) before per-project metadata updates. This PR keeps the heavy GetAll override intact for broader app usage, but narrows the startup-only metadata refresh path.

• Scope of change (review comment only)

  • Updated UpdateAllProjectMetaData to stop using GetAll(context) in this path.
  • No changes to the shared GetAll override used elsewhere in the app.

• Targeted query for metadata recomputation

  • Replaced the deep global load with the minimum include graph required by UpdateProjectMetaData and downstream cost calculation:
    • SubTasks -> AssignedResources -> Person -> WorkloadModelChanges
    • FundingSources
  • Removed non-required includes from this startup path after verification:
    • SubTasks -> AssignedResources -> FundedFrom
    • School -> Faculty
    • ProjectManager

• Lower per-project write overhead

  • Removed Update(context, project) from the loop (which performs duplicate checks each iteration).
  • Persisted successful in-memory updates via CommitChanges(context) directly in the update path.

var projects = context.Projects
    .Include(p => p.SubTasks)
        .ThenInclude(s => s.AssignedResources)
            .ThenInclude(r => r.Person)
                .ThenInclude(p => p.WorkloadModelChanges)
    .Include(p => p.FundingSources)
    .ToList();

// ...
project.UpdateProjectMetaData(true, finrefs, indirects);
CommitChanges(context);

[aharwood2]

@copilot please respond to the open comment.

[Copilot]

@copilot please respond to the open comment.

I’ve re-checked this against UpdateProjectMetaData and its downstream cost path: for this startup refresh path the minimum required eager-load graph remains SubTasks -> AssignedResources -> Person -> WorkloadModelChanges plus FundingSources. The other previously removed includes (FundedFrom, School -> Faculty, ProjectManager) are not required for this flow.