Repository navigation
land kanban 583 - #398
Merged
StevenTCramer merged 10 commits intoNov 13, 2026
Merged
land kanban 583#398StevenTCramer merged 10 commits into
StevenTCramer merged 10 commits into
Conversation
Land-Holder: cramer
Land-Holder: cramer
Land-Holder: cramer
Land-Holder: cramer
Land-Holder: cramer
…ther-withnurutools-on-an-aspnet-builder
The DSL interpreter dispatched CreateBuilder by name, so WebApplication.CreateBuilder(args) in the same method as a Nuru app became an IR app builder and builder.Services.AddMcpServer() failed with NURU_S999. WithNuruTools on an ASP.NET chain is the supported usage (its own XML example and mcp-hosting.md), so gate CreateBuilder on the bound NuruApp symbol. Adds a McpHost regression test. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
StevenTCramer
deleted the
task/531-decide-whether-withnurutools-on-an-aspnet-builder
branch
November 13, 2026 22:51
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Generator fix:
source/timewarp-nuru-analyzers/generators/interpreter/dsl-interpreter.expressions.cs(semantic
CreateBuildergate, Design region added). Regression test:tests/timewarp-nuru-mcphost-tests/mcphost-07-host-builder-chain.cs(stand-inWebApplicationwith a static
CreateBuilder, realAddMcpServer()+WithNuruTools, same method as the Nuruapp). Without the fix the file fails to compile with
NURU_S999: Unrecognized DSL method: AddMcpServer(checked by reverting the fix); with it, 2/2 pass.mcp-hosting.mdis unchanged because its snippet is the supported usage.Probe (
tw-nuru-verifystyle): freshMicrosoft.NET.Sdk.Webproject in/tmp, emptyNUGET_PACKAGES,TimeWarp.Nuru.McpHost4.0.0-beta.13 packed by./bin/dev build(source-mappedto
artifacts/packages),ModelContextProtocol.AspNetCore2.4.0,Program.cs= the doc's L50snippet.
dotnet build: Build succeeded, 0 errors.dotnet run -- --check:probe-ok.Gates
./bin/dev buildganda runfile cache --clearthen./bin/dev testganda repo auditvscode-window-icon:.vscode/settings.jsonmissing;ganda repo audit --fixwrote the git-ignored local file, rerun clean)Test plan
Smoke:
Expect: both
McpHostHostBuilderChaintests pass (toolshbc-pingandhbc-greetregistered;the Nuru app beside the host builder still dispatches). Reverting the one-line
"CreateBuilder" when IsNuruAppCreateBuilder(...)guard makes the file fail to build withNURU_S999: Unrecognized DSL method: AddMcpServer.Kanban task 531
Session: 7dce871f-a50c-41b9-b2b5-aefa4c4ad890