Skip to content

feat(services-dotnet-asp-platform): complete ASP.NET generation and evidence#149

Merged
LMLiam merged 26 commits into
masterfrom
aspnet-complete-generation
Apr 23, 2026
Merged

feat(services-dotnet-asp-platform): complete ASP.NET generation and evidence#149
LMLiam merged 26 commits into
masterfrom
aspnet-complete-generation

Conversation

@LMLiam

@LMLiam LMLiam commented Apr 20, 2026

Copy link
Copy Markdown
Owner

Why

  • The ASP.NET branch had drifted into generating synthetic runtime behaviour that the DSL does not actually describe, which made the generated service surface misleading.
  • ASP.NET port allocation had regressed from stable, configurable allocation to simple index-based defaults.
  • The fixture, CI, and documentation surface had followed the concrete-controller layout and needed to be brought back into line with an honest extension-oriented design.

What Changed

  • restored stable ASP.NET port allocation, including configured port handling and collision detection
  • replaced the synthetic controller/parser/status-override surface with generated hosting extensions, contract records, abstract controller bases, and response result mapping derived from the REST model
  • added compile-time validation that rejects unsupported request-binding shapes such as reference-typed path or query fields, and fixed nuint default rendering with a 64-bit-safe literal
  • realigned generation tests, script-host tests, native build-tool tests, runtime e2e coverage, README, and CI checks around the generated base-controller layout and user-authored derived controllers
  • kept .microsmith/origins.json output-root discovery and provenance support intact across the updated ASP.NET surface

Validation

  • ./gradlew --no-daemon --no-build-cache --max-workers=1 -Dkotlin.compiler.execution.strategy=in-process :artifact-services-dotnet-asp:jvmKotest :compile-services-dotnet-asp:jvmKotest --stacktrace
  • ./gradlew --no-daemon --no-build-cache --max-workers=1 -Dkotlin.compiler.execution.strategy=in-process :gen:jvmKotest :runtime-scripting:jvmKotest :gradle-plugin:test :maven-plugin:jvmKotest :sbt-plugin:jvmKotest --stacktrace
  • ruby -e 'require "yaml"; %w[.github/workflows/build_test_qodana.yml examples/non-gradle/dotnet/.github/workflows/microsmith.yml examples/gradle/dotnet/.github/workflows/microsmith.yml examples/maven/dotnet/.github/workflows/microsmith.yml examples/sbt/dotnet/.github/workflows/microsmith.yml].each { |path| YAML.load_file(path) }'

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 35b881d85a

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@LMLiam LMLiam force-pushed the aspnet-complete-generation branch from 35b881d to 99a44d6 Compare April 20, 2026 23:40
LMLiam added 21 commits April 21, 2026 13:18
@LMLiam LMLiam merged commit ce3c428 into master Apr 23, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant