Fix dashboard asset build ordering in CI graph/inner builds

[Copilot]

The build GitHub Actions job failed because Immediate.Jobs.Dashboard inner target-framework builds attempted to embed Assets/app.js before dashboard frontend assets were generated. This surfaced as CS1566 missing embedded resource errors across net8.0/net10.0/net11.0.

• Build orchestration fix

  • Added EnsureDashboardClientBuild for non-cross-targeting (inner) builds.
  • EnsureDashboardClientBuild invokes BuildDashboardClient via MSBuild with TargetFramework removed, forcing the existing outer/cross-target asset build path to run before embedding.

• Asset inclusion dependency update

  • Updated IncludeDashboardAssets to depend on EnsureDashboardClientBuild instead of directly depending on BuildDashboardClient.
  • Preserves current IsCrossTargetingBuild guards while eliminating the race/order gap seen in CI.

<Target Name="EnsureDashboardClientBuild"
  Condition="'$(DesignTimeBuild)' != 'true' and '$(IsCrossTargetingBuild)' != 'true'">
  <MSBuild Projects="$(MSBuildProjectFullPath)"
    Targets="BuildDashboardClient"
    BuildInParallel="false"
    RemoveProperties="TargetFramework" />
</Target>

<Target Name="IncludeDashboardAssets"
  BeforeTargets="AssignTargetPaths"
  DependsOnTargets="EnsureDashboardClientBuild"
  Condition="'$(IsCrossTargetingBuild)' != 'true'">

[coderabbitai]

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 70f425af-56fe-4d9c-bf80-8dcdd36e0fc8

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[coveralls]

Coverage Report for CI Build 36455032098

Warning

No base build found for commit 61e1c7a on main.
Coverage changes can't be calculated without a base build.
If a base build is processing, this comment will update automatically when it completes.

Coverage: 89.045%

Details

• Patch coverage: No coverable lines changed in this PR.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

Requires a base build to compare against. How to fix this →

---

Coverage Stats

Relevant Lines:	13410
Covered Lines:	11941
Line Coverage:	89.05%
Coverage Strength:	2.88 hits per line

---

💛 - Coveralls