Surface actionable diagnostic when TypeScript codegen generator is dropped by load failure by sebastienros · Pull Request #18125 · microsoft/aspire

Sébastien Ros (sebastienros) · 2026-06-11T19:31:05Z

Description

When the configured Aspire SDK is out of sync with the installed CLI, running or restoring a TypeScript (Node.js) AppHost could fail with a cryptic, unactionable error:

Failed to run TypeScript (Node.js) AppHost: No code generator found for language: TypeScript. No code generators were discovered in any loaded assembly.

The root cause is a binary mismatch between the bundled apphost server and the integration assemblies on disk. The TypeScript code generator assembly (from the SDK packages) references an Aspire.TypeSystem assembly that the running apphost server cannot provide (for example, an older bundled server that predates that assembly version). During assembly discovery, CodeGeneratorResolver.DiscoverGenerators calls GetTypes(), hits a ReflectionTypeLoadException (whose loader exception is a FileNotFoundException for Aspire.TypeSystem), and silently drops the TypeScript generator. The downstream CodeGenerationService.GenerateCode then threw a plain ArgumentException, which the diagnostic builder did not classify, so the existing actionable IncompatibleAspireSdk diagnostic (with a "run aspire update" remediation hint) was never emitted. The bare exception then propagated across StreamJsonRpc to the CLI and surfaced as a raw stack trace / "unexpected error".

This change routes the swallowed load failure into that existing actionable diagnostic so the user instead sees:

Failed to run TypeScript (Node.js) AppHost: Aspire SDK code generation failed because the installed Aspire CLI appears to be incompatible with the configured SDK version. Run 'aspire update' to align the CLI and SDK and try again.

Approach

The fix is contained entirely within Aspire.Hosting.RemoteHost (the apphost server, which is bundled into the CLI). It has three parts:

CodeGeneratorResolver now retains the ReflectionTypeLoadExceptions it swallows during discovery (via a new DiscoveryResult and GetDiscoveryLoadFailures()), instead of discarding them.
CodeGenerationService.GenerateCode chains the first retained load failure as the inner exception of the ArgumentException it throws when no generator is found.
CodeGenerationDiagnostic recognizes FileNotFoundException in the exception chain (in addition to the existing reflection-load cases) and builds the structured IncompatibleAspireSdk RPC error that the CLI already renders with the actionable "run aspire update" message.

Because both the affected server code and the CLI client catch path (AppHostRpcClient.InvokeCodeGenerationAsync) are shared, this applies to every command that triggers code generation (aspire run, aspire restore, etc.). No public API or codegen exports coverage changed; the CLI-side rendering path was already in place.

Verification

In addition to the unit tests, I ran a functional A/B test against a real TypeScript AppHost configured with a daily SDK, injecting the same Aspire.TypeSystem load failure. With the fix, the message flips from the cryptic "No code generator found..." to the actionable "...incompatible with the configured SDK version. Run 'aspire update'...". This was verified end-to-end via aspire run; aspire restore exercises the same shared RPC and client paths.

Fixes#18110
Fixes#17910

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

…failure A TypeScript AppHost whose configured SDK version diverges from the installed CLI fails with a cryptic "No code generator found for language: TypeScript". The real cause is a binary mismatch: the code-generation assembly references an Aspire.TypeSystem version that is absent on disk, so type discovery raises a ReflectionTypeLoadException that CodeGeneratorResolver swallows. The generator is silently dropped, and GenerateCode then throws a plain ArgumentException that the diagnostic builder does not classify, so the actionable IncompatibleAspireSdk ("run aspire update") diagnostic the CLI knows how to render is never emitted. - CodeGeneratorResolver now retains the swallowed ReflectionTypeLoadExceptions and exposes them via GetDiscoveryLoadFailures(). - GenerateCode chains the captured load failure as the inner exception when no generator is found, so the existing catch path produces the IncompatibleAspireSdk diagnostic with the version-alignment remediation hint. - CodeGenerationDiagnosticBuilder recognizes FileNotFoundException (the actual loader exception for a missing dependency assembly) and captures its file name. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

github-actions · 2026-06-11T19:33:05Z

🚀 Dogfood this PR with:

⚠️ WARNING: Do not do this without first carefully reviewing the code of this PR to satisfy yourself it is safe.

curl -fsSL https://raw.githubusercontent.com/microsoft/aspire/main/eng/scripts/get-aspire-cli-pr.sh | bash -s -- 18125

Or

Run remotely in PowerShell:

iex "& { $(irm https://raw.githubusercontent.com/microsoft/aspire/main/eng/scripts/get-aspire-cli-pr.ps1) } 18125"

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

…mbly Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Only treat a bare FileNotFoundException as an incompatible-SDK signal when its FileName is an assembly display name, so a genuine missing-file IO error during code generation is not misreported as a version mismatch. FNFEs inside an RTLE's loader exceptions are still always treated as assembly-bind failures. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated no new comments.

Sébastien Ros (sebastienros) · 2026-06-11T20:53:15Z

PR #18125 - Functional Testing Report

PR:#18125 - TypeScript AppHost codegen failure -> actionable diagnostic
Head commit tested:4aa3bcaea6
Dogfood CLI version:13.5.0-pr.18125.g4aa3bcae (matches PR head g4aa3bcae) ✅
Platform: macOS arm64
Mode: Local (isolated install, no container)

What the PR fixes

When the installed CLI is older than the configured SDK, the bundled apphost server's
Aspire.Hosting.CodeGeneration.TypeScript references an Aspire.TypeSystem assembly the
server cannot load. CodeGeneratorResolver.DiscoverGenerators swallowed the resulting
ReflectionTypeLoadException (loader exception = FileNotFoundException for
Aspire.TypeSystem), so the TypeScript generator was silently dropped and the user got a
cryptic No code generator found for language: TypeScript instead of the existing actionable
IncompatibleAspireSdk ("run aspire update") diagnostic.

Scenario 1 - No regression on the SHIPPED PR artifact (PASS)

Installed the PR CLI to an isolated dir and ran aspire restore (issue #17910's command,
pure codegen trigger) against a minimal TypeScript AppHost (Aspire.Hosting + Aspire.Hosting.Redis).

⚙️ Restoring SDK code...
✅ SDK code restored successfully for apphost.mts.

Generated .aspire/modules/: aspire.mts (3.0 MB), base.mts, transport.mts,
.codegen-hash. Exit code 0. The TypeScript generator was discovered and ran normally -
the resolver/discovery refactor does not regress the healthy codegen path.

Note: the shipped artifact is the newest build, so its bundled server ships a matching
Aspire.TypeSystem and cannot naturally reproduce the version-mismatch failure. The
shipped server assemblies are also packed inside a single-file aspire-managed executable,
so loose-DLL injection against the artifact is not possible. The diagnostic flip is therefore
proven deterministically against the real production code (below).

Scenario 2 - Diagnostic flip proven against real production code (PASS)

The fix lives in Aspire.Hosting.RemoteHost production types
(CodeGeneratorResolver, CodeGenerationService.GenerateCode, CodeGenerationDiagnosticBuilder).
Re-ran the focused tests that drive these exact types on the current head:

Test run summary: Passed! total: 23 failed: 0

Key deterministic proof - ServiceErrorMessageTests.GenerateCode_GeneratorDroppedByLoadFailure_ThrowsIncompatibleSdkDiagnostic:
reproduces the exact bug shape (TypeScript generator dropped because
FileNotFoundException("Aspire.TypeSystem, Version=42.42.42.42") was the loader exception)
and asserts the result is now LocalRpcException with
ErrorCode == IncompatibleAspireSdk and a non-empty RemediationHint - i.e. the cryptic
ArgumentException has flipped to the actionable update path.

Supporting coverage:

ResolverDiagnosticsTests - resolver captures swallowed load failures via Discovery.LoadFailures.
CodeGenerationDiagnosticBuilderTests.TryCreateRpcException_FileNotFoundLoaderException_CapturesMissingAssemblyName
- FNFE loader exception classified + missing assembly name captured.
FNFE over-broadening guard (addressed from review):
- ..._StandaloneFileNotFound_PlainFilePath_IsNotClassified -> returns null (a genuine missing
  file is NOT misreported as an SDK mismatch).
- ..._StandaloneFileNotFound_AssemblyDisplayName_IsClassified -> a standalone FNFE whose
  file name is an assembly display name (Version/PublicKeyToken) IS classified.

Verdict

Healthy TypeScript codegen works end-to-end on the shipped PR artifact (no regression).
The targeted failure shape now produces the actionable IncompatibleAspireSdk diagnostic
instead of the cryptic message, with a guard preventing genuine missing-file IO errors from
being misclassified.

PR is functionally sound for its stated intent (fixes #18110, #17910).

github-actions · 2026-06-11T21:01:40Z

Re-running the failed jobs in the CI workflow for this pull request because 1 job was identified as retry-safe transient failures in the CI run attempt.
GitHub was asked to rerun all failed jobs for that attempt, and the rerun is being tracked in the rerun attempt.
The job links below point to the failed attempt jobs that matched the retry-safe transient failure rules.

Tests / Hosting-1 / Hosting-1 (windows-latest) - Job-level runner or infrastructure failure matched the transient allowlist.

Copilot

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated no new comments.

James Newton-King (JamesNK)

LGTM. The diagnostic chain is well-structured — the heuristic to distinguish assembly-bind FileNotFoundException from plain file IO is careful, and the test coverage is thorough across all the relevant exception shapes.

David Fowler (davidfowl) · 2026-06-12T06:57:54Z

When the configured Aspire SDK is out of sync with the installed CLI, running or restoring a TypeScript (Node.js) AppHost could fail with a cryptic, unactionable error:

We need to harden this, not just fix the error. I think we need a breakdown of why this is happening. Unless there was some fundamental contract breakage between the apphost server between 13.4 and 13.5 (which I am not aware of), this should not happen. The version of Aspire.Hosting should come from the apphost, the apphost server's version should not impact the ALC where integrations are loaded.

David Fowler (davidfowl)

No issues with improving the error but I think there's more to fix here, this scenario should not be broken.

Replace the negative/substring assertions with a single Assert.Equal against the full SafeMessage so the test pins the actionable message verbatim. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

github-actions · 2026-06-12T15:35:36Z

❓ CLI E2E Tests unknown — 115 passed, 0 failed, 2 unknown (commit b01e4f0)

View all recordings

-	Test	Detail
✅	AddPackageInteractiveWhileAppHostRunningDetached	Recording · Job · CLI logs
✅	AddPackageWhileAppHostRunningDetached	Recording · Job · CLI logs
✅	AgentCommands_AllHelpOutputs_AreCorrect	Recording · Job · CLI logs
✅	AgentInitCommand_DefaultSelection_InstallsDefaultSkills	Recording · Job · CLI logs
✅	AgentInitCommand_MigratesDeprecatedConfig	Recording · Job · CLI logs
✅	AgentInit_NonInteractive_BundleOnlySkillsNotInCatalog	Recording · Job · CLI logs
✅	AgentMcpListResources_ExcludesResourceMarkedWithExcludeFromMcp	Recording · Job · CLI logs
✅	AgentMcpListStructuredLogsReturnsLogsFromStarterApp	Recording · Job · CLI logs
✅	AgentMcpListStructuredLogsReturnsLogsFromStarterApp_DevLocalhost	Recording · Job · CLI logs
✅	AgentMcpListStructuredLogsReturnsLogsFromStarterApp_Isolated	Recording · Job · CLI logs
✅	AllPublishMethodsBuildDockerImages	Recording · Job · CLI logs
✅	AspireAddAndStartWorkAgainstLegacyAppHostTs	Recording · Job · CLI logs
✅	AspireAddPackageVersionToDirectoryPackagesProps	Recording · Job · CLI logs
✅	AspireInitSingleFileAppHostRunsViaDotnetRunAppHost	Recording · Job · CLI logs
✅	AspireInit_ExistingAppHostDir_RecreatesNuGetConfigKeepsFiles	Recording · Job · CLI logs
❔	AspireInit_SolutionFile_BuildsAgainstChannelHive	Recording · Job · CLI logs
✅	AspireStartUpdatesStaleTypeScriptAppHostPath	Recording · Job · CLI logs
✅	AspireUpdateRemovesAppHostPackageVersionFromDirectoryPackagesProps	Recording · Job · CLI logs
✅	AspireUpdateRemovesOrphanAppHostPackageVersionWhenSdkAlreadyCurrent	Recording · Job · CLI logs
✅	Banner_DisplayedOnFirstRun	Recording · Job · CLI logs
✅	Banner_DisplayedWithExplicitFlag	Recording · Job · CLI logs
✅	Banner_NotDisplayedWithNoLogoFlag	Recording · Job · CLI logs
✅	CertificatesClean_RemovesCertificates	Recording · Job · CLI logs
✅	CertificatesTrust_WithNoCert_CreatesAndTrustsCertificate	Recording · Job · CLI logs
✅	CertificatesTrust_WithUntrustedCert_TrustsCertificate	Recording · Job · CLI logs
✅	ConfigSetGet_CreatesNestedJsonFormat	Recording · Job · CLI logs
✅	CreateAndRunAspireStarterProject	Recording · Job · CLI logs
✅	CreateAndRunAspireStarterProjectWithBundle	Recording · Job · CLI logs
✅	CreateAndRunEmptyAppHostProject	Recording · Job · CLI logs
✅	CreateAndRunJavaEmptyAppHostProject	Recording · Job · CLI logs
✅	CreateAndRunJsReactProject	Recording · Job · CLI logs
✅	CreateAndRunPolyglotAppHostWithDevLocalhostUrls	Recording · Job · CLI logs
✅	CreateAndRunPythonReactProject	Recording · Job · CLI logs
✅	CreateAndRunTypeScriptEmptyAppHostProject	Recording · Job · CLI logs
✅	CreateAndRunTypeScriptStarterProject	Recording · Job · CLI logs
✅	CreateJavaAppHostWithViteApp	Recording · Job · CLI logs
✅	CreateTypeScriptAppHostWithViteApp_UsesConfiguredToolchain	Recording · Job · CLI logs
✅	DashboardRunWithAgentMcpListTracesReturnsNoTraces	Recording · Job · CLI logs
✅	DashboardRunWithAgentMcpListTracesReturnsNoTraces_DevLocalhost	Recording · Job · CLI logs
✅	DashboardRunWithOtelTracesReturnsNoTraces	Recording · Job · CLI logs
✅	DashboardRunWithOtelTracesReturnsNoTraces_DevLocalhost	Recording · Job · CLI logs
✅	DeployK8sBasicApiService	Recording · Job · CLI logs
✅	DeployK8sWithExternalHelmChart	Recording · Job · CLI logs
✅	DeployK8sWithGarnet	Recording · Job · CLI logs
✅	DeployK8sWithMongoDB	Recording · Job · CLI logs
✅	DeployK8sWithMySql	Recording · Job · CLI logs
✅	DeployK8sWithPostgres	Recording · Job · CLI logs
✅	DeployK8sWithRabbitMQ	Recording · Job · CLI logs
✅	DeployK8sWithRedis	Recording · Job · CLI logs
✅	DeployK8sWithSqlServer	Recording · Job · CLI logs
✅	DeployK8sWithValkey	Recording · Job · CLI logs
✅	DeployTypeScriptAppToKubernetes	Recording · Job · CLI logs
✅	DescribeCommandResolvesReplicaNames	Recording · Job · CLI logs
✅	DescribeCommandShowsRunningResources	Recording · Job · CLI logs
✅	DetachFormatJsonProducesValidJson	Recording · Job · CLI logs
✅	DetachFormatJsonProducesValidJsonWhenRestartingExistingInstance	Recording · Job · CLI logs
✅	DoPublishAndDeployListStepsWork	Recording · Job · CLI logs
✅	DocsCommand_RendersInteractiveMarkdownFromLocalSource	Recording · Job · CLI logs
✅	DoctorCommand_DetectsDeprecatedAgentConfig	Recording · Job · CLI logs
✅	DoctorCommand_TypeScriptAppHostReportsMissingConfiguredToolchain	Recording · Job · CLI logs
✅	DoctorCommand_WithSslCertDir_ShowsTrusted	Recording · Job · CLI logs
✅	DoctorCommand_WithoutSslCertDir_ShowsPartiallyTrusted	Recording · Job · CLI logs
✅	DotNetRunFileBasedAppHostUsesAspireCliBundle	Recording · Job · CLI logs
✅	DotNetRunProjectAppHostUsesAspireCliBundle	Recording · Job · CLI logs
✅	GatewayWithoutExternalEndpoint_FailsPublishWithGuidance	Recording · Job · CLI logs
✅	GeneratedAspireDevScript_StartsWatchMode_WithConfiguredToolchain	Recording · Job · CLI logs
✅	GlobalMigration_HandlesCommentsAndTrailingCommas	Recording · Job · CLI logs
✅	GlobalMigration_HandlesMalformedLegacyJson	Recording · Job · CLI logs
✅	GlobalMigration_PreservesAllValueTypes	Recording · Job · CLI logs
✅	GlobalMigration_SkipsWhenNewConfigExists	Recording · Job · CLI logs
✅	GlobalSettings_MigratedFromLegacyFormat	Recording · Job · CLI logs
✅	IngressWithoutExternalEndpoint_FailsPublishWithGuidance	Recording · Job · CLI logs
✅	InitTypeScriptAppHost_AugmentsExistingViteRepoInWorkspaceSubdirectory	Recording · Job · CLI logs
✅	InteractiveCSharpInitCreatesExpectedFiles	Recording · Job · CLI logs
✅	InvalidAppHostPathWithComments_IsHealedOnRun	Recording · Job · CLI logs
✅	JavaScriptHostingApisRunFromTypeScriptAppHost	Recording · Job · CLI logs
✅	LatestCliCanStartStableChannelAppHost	Recording · Job · CLI logs
✅	LatestCliCanStartStableChannelTypeScriptAppHost	Recording · Job · CLI logs
✅	LegacySettingsMigration_AdjustsRelativeAppHostPath	Recording · Job · CLI logs
✅	LogsCommandShowsResourceLogs	Recording · Job · CLI logs
✅	OtelLogsReturnsStructuredLogsFromStarterApp	Recording · Job · CLI logs
✅	OtelLogsReturnsStructuredLogsFromStarterAppIsolated	Recording · Job · CLI logs
✅	ProcessCommandCallbackReceivesCliArguments	Recording · Job · CLI logs
✅	PsCommandListsRunningAppHost	Recording · Job · CLI logs
✅	PsFormatJsonOutputsOnlyJsonToStdout	Recording · Job · CLI logs
✅	PublishJavaScriptPatternsGeneratesExpectedDockerComposeArtifacts	Recording · Job · CLI logs
✅	PublishWithConfigureEnvFileUpdatesEnvOutput	Recording · Job · CLI logs
✅	PublishWithDockerComposeServiceCallbackSucceeds	Recording · Job · CLI logs
✅	PublishWithoutOutputPathUsesAppHostDirectoryDefault	Recording · Job · CLI logs
✅	ResourceCommand_FailedExec_ShowsLogPathAndLogHasEntries	Recording · Job · CLI logs
✅	ResourceCommand_SetAndDeleteParameterUpdatesDescribeOutput	Recording · Job · CLI logs
✅	RestoreGeneratesSdkFiles	Recording · Job · CLI logs
✅	RestoreGeneratesSdkFiles_WithConfiguredToolchain	Recording · Job · CLI logs
✅	RestoreRefreshesGeneratedSdkAfterAddingIntegration	Recording · Job · CLI logs
✅	RestoreSupportsConfigOnlyHelperPackageAndCrossPackageTypes	Recording · Job · CLI logs
✅	RunFromParentDirectory_UsesExistingConfigNearAppHost	Recording · Job · CLI logs
✅	RunReportsSyntaxErrorsForDotNetAppHost	Recording · Job · CLI logs
✅	RunReportsSyntaxErrorsForTypeScriptAppHost	Recording · Job · CLI logs
✅	SecretCrudOnDotNetAppHost	Recording · Job · CLI logs
✅	SecretCrudOnTypeScriptAppHost	Recording · Job · CLI logs
✅	StagingChannel_ConfigureAndVerifySettings_ThenSwitchChannels	Recording · Job · CLI logs
✅	StartAndWaitForTypeScriptSqlServerAppHostWithNativeAssets	Recording · Job · CLI logs
✅	StartReportsSyntaxErrorsForDotNetAppHost	Recording · Job · CLI logs
✅	StartReportsSyntaxErrorsForTypeScriptAppHost	Recording · Job · CLI logs
✅	StopAllAppHostsFromAppHostDirectory	Recording · Job · CLI logs
✅	StopJavaPolyglotAppHostUsingApphostDirectory	Recording · Job · CLI logs
✅	StopNonInteractiveSingleAppHost	Recording · Job · CLI logs
✅	StopTypeScriptPolyglotAppHostUsingApphostDirectory	Recording · Job · CLI logs
✅	StopWithNoRunningAppHostExitsSuccessfully	Recording · Job · CLI logs
✅	TerminalAttachFrontend_ShowsViteHelpAndDetaches	Recording · Job · CLI logs
✅	TypeScriptAppHostRunDoesNotDeadlockWhenLazyOptionsInvokeAsyncCallback	Recording · Job · CLI logs
✅	TypeScriptAppHostWithVite_AllowsDifferentGuestPkgManager	Recording · Job · CLI logs
✅	UnAwaitedChainsCompileWithAutoResolvePromises	Recording · Job · CLI logs
❔	UpdateToStable_CSharpEmptyAppHost_KeepsConfigChannel	Recording · Job · CLI logs
✅	UpdateToStable_CSharpSingleFileInit_KeepsConfigChannel	Recording · Job · CLI logs
✅	UpdateToStable_TypeScriptSingleFileInit_KeepsConfigChannel	Recording · Job · CLI logs
✅	UpdateToStable_TypeScript_PreviewsStablePkgsAndKeepsChannel	Recording · Job · CLI logs

_{📹 Recordings uploaded automatically from CI run #27423488724}

Ankit Jain (radical) · 2026-06-12T18:41:09Z

No issues with improving the error but I think there's more to fix here, this scenario should not be broken.

Should this be draft till then?

….0.0.0 The shipped 1.0.0.0 freeze fixed old-CLI + new-SDK skew but regressed the common backward-compat direction (new CLI + stable-channel SDK), where the bundled Aspire.TypeSystem (1.0.0.0) could not satisfy the codegen assemblies' strong-named reference to the real released version (e.g. 13.4.3.0). The CLR only binds when the loaded copy's version is >= the requested version, so a single frozen value must sit ABOVE every shipped real version to satisfy the must-work direction. Freeze at 10001.0.0.0 (a '1.0' seeded high; major slot is the breaking-change bump axis). - Aspire.TypeSystem.csproj: AssemblyVersion 1.0.0.0 -> 10001.0.0.0; drop DisablePackageBaselineValidation (10001 >= package baseline, so CP0003 passes); rewrite rationale comment. - IntegrationLoadContext: explain the bundled high copy satisfies any lower/equal reference; residual old-CLI case is #18125's actionable 'update your CLI' path. - AssemblyLoader.WarnIfSharedAssemblyMismatch: warn only when bundled < libs (the only failing order); bundled >= libs binds and logs at Debug. - AtsSharedContractSurfaceTests / SharedContractSurface: reframe the freeze guard around constant-version binding (content compat, not version) and reflow comments. Fixes the regression behind #18110 and #17910; complements the #18125 diagnostics, which still surface the genuine new-member case (already-shipped old CLI + post-freeze codegen) as an actionable error. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

CopilotAI review requested due to automatic review settings June 11, 2026 19:31

CopilotAI reviewed Jun 11, 2026

Sébastien Ros (sebastienros) changed the base branch from release/13.4 to mainJune 11, 2026 19:31

Sébastien Ros (sebastienros)and others added 2 commits June 11, 2026 12:38

Document test-only consumption of GetDiscoveryLoadFailures
bb86ab9
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Expose discovery result instead of dedicated load-failures accessor
e67fe67
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

CopilotAI review requested due to automatic review settings June 11, 2026 19:42

Copilotstarted reviewing on behalf of Sébastien Ros (sebastienros)June 11, 2026 19:44 View session

CopilotAI reviewed Jun 11, 2026
View reviewed changes

Comment threadsrc/Aspire.Hosting.RemoteHost/CodeGeneration/CodeGeneratorResolver.cs

Sébastien Ros (sebastienros)and others added 2 commits June 11, 2026 12:49

Replace local test assembly fakes with reusable configurable FakeAsse…
2d4ff03
…mbly Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

CopilotAI review requested due to automatic review settings June 11, 2026 19:52

Copilotstarted reviewing on behalf of Sébastien Ros (sebastienros)June 11, 2026 19:54 View session

Potential fix for pull request finding
4aa3bca
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

CopilotAI reviewed Jun 11, 2026
View reviewed changes

Sébastien Ros (sebastienros) marked this pull request as ready for review June 11, 2026 23:13

CopilotAI review requested due to automatic review settings June 11, 2026 23:13

Copilotstarted reviewing on behalf of Sébastien Ros (sebastienros)June 11, 2026 23:14 View session

CopilotAI reviewed Jun 11, 2026
View reviewed changes

James Newton-King (JamesNK) approved these changes Jun 12, 2026
View reviewed changes

Comment threadtests/Aspire.Hosting.RemoteHost.Tests/CodeGenerationDiagnosticBuilderTests.cs Outdated

David Fowler (davidfowl) approved these changes Jun 12, 2026
View reviewed changes

Assert exact incompatible-SDK message in diagnostic test
b01e4f0
Replace the negative/substring assertions with a single Assert.Equal against the full SafeMessage so the test pins the actionable message verbatim. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Sébastien Ros (sebastienros) mentioned this pull request Jun 12, 2026
[release/13.4] Freeze Aspire.TypeSystem AssemblyVersion at 13.4.5.0 to fix cross-ALC codegen binding #18160
Merged

Sébastien Ros (sebastienros) marked this pull request as draft June 12, 2026 22:45

Sébastien Ros (sebastienros) closed this Aug 11, 2026

Sébastien Ros (sebastienros) deleted the sebastienros/ts-apphost-codegen-error branch August 11, 2026 17:23

microsoft-github-policy-serviceBot added this to the 13.6 milestone Aug 11, 2026

Navigation Menu

Uh oh!

Surface actionable diagnostic when TypeScript codegen generator is dropped by load failure - #18125

Surface actionable diagnostic when TypeScript codegen generator is dropped by load failure#18125
Sébastien Ros (sebastienros) wants to merge 7 commits into
mainfrom
sebastienros/ts-apphost-codegen-error

Sébastien Ros (sebastienros) commented Jun 11, 2026•
edited
Loading

Uh oh!

CopilotAI left a comment

Uh oh!

github-actionsBot commented Jun 11, 2026•
edited
Loading

Uh oh!

CopilotAI left a comment

Uh oh!

Uh oh!

CopilotAI left a comment

Uh oh!

Sébastien Ros (sebastienros) commented Jun 11, 2026

Uh oh!

github-actionsBot commented Jun 11, 2026

Uh oh!

CopilotAI left a comment

Uh oh!

James Newton-King (JamesNK) left a comment

Uh oh!

Uh oh!

David Fowler (davidfowl) commented Jun 12, 2026

Uh oh!

David Fowler (davidfowl) left a comment

Uh oh!

github-actionsBot commented Jun 12, 2026

Uh oh!

Ankit Jain (radical) commented Jun 12, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Uh oh!

Conversation

Sébastien Ros (sebastienros) commented Jun 11, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Approach

Verification

Checklist

Uh oh!

CopilotAI left a comment

Choose a reason for hiding this comment

Uh oh!

github-actionsBot commented Jun 11, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

CopilotAI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

CopilotAI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Sébastien Ros (sebastienros) commented Jun 11, 2026

PR #18125 - Functional Testing Report

What the PR fixes

Scenario 1 - No regression on the SHIPPED PR artifact (PASS)

Scenario 2 - Diagnostic flip proven against real production code (PASS)

Verdict

Uh oh!

github-actionsBot commented Jun 11, 2026

Uh oh!

CopilotAI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

James Newton-King (JamesNK) left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

David Fowler (davidfowl) commented Jun 12, 2026

Uh oh!

David Fowler (davidfowl) left a comment

Choose a reason for hiding this comment

Uh oh!

github-actionsBot commented Jun 12, 2026

Uh oh!

Ankit Jain (radical) commented Jun 12, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Sébastien Ros (sebastienros) commented Jun 11, 2026•
edited
Loading

github-actionsBot commented Jun 11, 2026•
edited
Loading