JobMonitor: preserve source metadata from job properties when resubmitting Helix work items

[Copilot]

Resubmitted Helix jobs from JobMonitor were missing SDK-populated source metadata (Repository, Branch, SourcePrefix, TeamProject). This change aligns resubmission payload construction with Helix SDK submissions by using source metadata from the original job property bag, without parsing Source.

• Source metadata handling

  • Use the original job property bag values (SourcePrefix, TeamProject, Repository, Branch) when constructing the resubmission JobCreationRequest.
  • Avoid sending both Source and source-component fields together.

• Compatibility fallback

  • If source metadata is missing from the property bag, keep current behavior by setting Source directly.

• Targeted test coverage

  • Keep coverage asserting all source-component fields are preserved from properties.
  • Update coverage so that when source metadata properties are missing, resubmission falls back to Source passthrough (no Source parsing fallback).

if (TryGetSourceMetadataFromProperties(details.Properties, out string sourcePrefix, out string teamProject, out string repository, out string branch))
{
    creationRequest.SourcePrefix = sourcePrefix;
    creationRequest.TeamProject = teamProject;
    creationRequest.Repository = repository;
    creationRequest.Branch = branch;
}
else
{
    creationRequest.Source = details.Source;
}

To double check:

• The right tests are in and the right validation has happened. Guidance: https://github.com/dotnet/arcade/blob/main/Documentation/Validation.md

[mmitche]

@copilot Look into how job properties are structured. I believe that all of the source properties you're trying to parse are part of the property bag.

[Copilot]

@copilot Look into how job properties are structured. I believe that all of the source properties you're trying to parse are part of the property bag.

Updated to source these fields from the original job property bag first, then fall back to parsing Source, then to Source passthrough. Addressed in commits 0ac825f and e04b7cb.

[mmitche]

@copilot Do NOT fall back to parsing source.

[Copilot]

@copilot Do NOT fall back to parsing source.

Addressed in caa8529. I removed the Source parsing fallback from JobMonitor resubmission; it now uses source metadata only from the job property bag, and otherwise falls back to passing Source through unchanged.

[mmitche]

@copilot Why would the source not also be set here?

[Copilot]

Because Helix rejects requests that include both Source and any of SourcePrefix/TeamProject/Repository/Branch (400). We mirror the SDK behavior here: use the split source fields when available; otherwise set Source.